Skip to content

Conversation

@kushalshit27
Copy link
Contributor

🔧 Changes

  • Added support for use_for_organization_discovery boolean field in organization discovery domain operations
  • Added new screen type pre-login-organization-picker to prompts handler for organization picker screen rendering

Examples

YAML format
organizations:
  - name: acme
    display_name: Acme Inc
    discovery_domains:
      - domain: login.acme.com
        status: verified
        use_for_organization_discovery: true
      - domain: auth.acme.com
        status: pending
        use_for_organization_discovery: false
JSON format
{
  "name": "acme",
  "display_name": "Acme Inc",
  "discovery_domains": [
    {
      "domain": "login.acme.com",
      "status": "verified",
      "use_for_organization_discovery": true
    },
    {
      "domain": "auth.acme.com",
      "status": "pending",
      "use_for_organization_discovery": false
    }
  ]
}

🔬 Testing

  • All existing unit tests updated to reflect the new use_for_organization_discovery field
  • Tests cover create, update, and export scenarios for organization discovery domains with the new field

📝 Checklist

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

…ts type adjustments

- src/tools/auth0/handlers/organizations.ts: add 'use_for_organization_discovery' property to domain schema
- src/tools/auth0/handlers/organizations.ts: refactor domain update logic for better clarity
- src/tools/auth0/handlers/organizations.ts: handle 'use_for_organization_discovery' in domain creation
- src/tools/auth0/handlers/prompts.ts: update ScreenRenderer type to Management.GetAculResponseContent
- src/tools/auth0/handlers/organizations.ts: improve mapping of existing discovery domains to include remote IDs
- src/tools/auth0/handlers/organizations.ts: filter out undefined updated domains during mapping
- test/context/directory/organizations.test.js: include use_for_organization_discovery in JSON organizations
- test/context/yaml/organizations.test.js: add use_for_organization_discovery to YAML organizations
- test/tools/auth0/handlers/organizations.tests.js: update sampleDiscoveryDomain and assertions for use_for_organization_discovery
@kushalshit27 kushalshit27 changed the title feat: add support for use_for_organization_discovery organizations discovery domains feat: add support for use_for_organization_discovery organizations discovery-domains Jan 28, 2026
@codecov-commenter
Copy link

codecov-commenter commented Jan 28, 2026

Codecov Report

❌ Patch coverage is 78.57143% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.29%. Comparing base (0c86dbd) to head (099f1d5).

Files with missing lines Patch % Lines
src/tools/auth0/handlers/organizations.ts 78.57% 3 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1283   +/-   ##
=======================================
  Coverage   80.28%   80.29%           
=======================================
  Files         146      146           
  Lines        5823     5825    +2     
  Branches     1193     1193           
=======================================
+ Hits         4675     4677    +2     
  Misses        655      655           
  Partials      493      493           

☔ 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 ready for review January 29, 2026 07:32
@kushalshit27 kushalshit27 requested a review from a team as a code owner January 29, 2026 07:32
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.

4 participants