Skip to content

Conversation

@developerkunal
Copy link
Contributor

@developerkunal developerkunal commented Oct 10, 2025

🔧 Changes

This pull request upgrades the auth0 dependency to version 5.0.0 and updates the codebase to use the auth0/legacy import path for all Auth0-related clients and types.
This approach allows us to adopt the new Auth0 SDK while avoiding breaking changes and maintaining compatibility with existing usage patterns.

The auth0/legacy entry point provides backward-compatible exports for the previous v4 client interfaces, enabling a smooth, incremental migration path while we progressively adopt the newer SDK structure.

Dependency and import path updates

  • Upgraded the auth0 dependency from v4.32.0 → v5.0.0 in package.json.

  • Updated all Auth0-related imports (e.g. ManagementClient, AuthenticationClient, Client, ResourceServer, etc.) to use the auth0/legacy import path instead of the root auth0 package.
    This ensures full backward compatibility while allowing partial adoption of v5 features.

  • Updated references in affected modules such as:

    • src/context
    • src/tools/auth0/handlers
    • Related handler utilities

ESLint configuration update

  • Added "import/core-modules": ["auth0/legacy"] under ESLint settings to treat the new import path as a recognized core module, preventing false-positive lint errors.

Template string formatting improvements

  • Simplified template string formatting for warning messages in:

    • src/context/index.ts
    • src/tools/auth0/handlers/default.ts
  • Removed unnecessary line breaks and expressions for improved readability.

📚 References

  • Auth0 SDK v5.0.0 release notes (internal migration guidance)
  • Internal discussion on incremental SDK adoption strategy

🔬 Testing

  • Verified CLI commands continue to function correctly using the legacy SDK interfaces.
  • Confirmed no breaking changes in existing configuration or tenant deployment workflows.
  • Validated build and lint tasks pass under Node.js ≥20.18.1.

📝 Checklist

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • Documentation and comments updated to reflect the new auth0/legacy import path (or N/A)

@developerkunal developerkunal requested a review from a team as a code owner October 10, 2025 09:47
@codecov-commenter
Copy link

codecov-commenter commented Oct 10, 2025

Codecov Report

❌ Patch coverage is 81.25000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.17%. Comparing base (f0fc1b9) to head (d8ee9f1).

Files with missing lines Patch % Lines
src/context/index.ts 66.66% 1 Missing and 1 partial ⚠️
src/tools/auth0/handlers/scimHandler.ts 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1177   +/-   ##
=======================================
  Coverage   80.17%   80.17%           
=======================================
  Files         134      134           
  Lines        5100     5100           
  Branches     1014     1014           
=======================================
  Hits         4089     4089           
  Misses        579      579           
  Partials      432      432           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kushalshit27 kushalshit27 marked this pull request as draft October 10, 2025 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants