Skip to content

feat: add schema and data options for apps management commands - #1664

Merged
developerkunal merged 7 commits into
mainfrom
DXCDT-2107-agent-rollout-for-apps
Sep 17, 2026
Merged

developerkunal merged 7 commits into
mainfrom
DXCDT-2107-agent-rollout-for-apps

Conversation

@ramya18101

@ramya18101 ramya18101 commented Sep 16, 2026 •

Copy link
Copy Markdown
Contributor

🔧 Changes

Extends the JSON-input / schema-discovery surface to the auth0 apps commands,
mirroring the pattern already applied to actions, roles, users, and apis
with machine-readable schemas and whole-payload JSON).

📚 References

Follows the same pattern as the merged actions/roles/users/apis PRs

🔬 Testing

  • Added internal/cli/apps_json_test.go:
    • TestAppsListCmdInvalidQuery — invalid --query JSON is rejected before any
      API call.
    • TestAppsCreateCmdData — a valid --data payload is sent as POST /clients;
      a schema-invalid payload ({"name":123}) fails validation and never reaches
      the API.
    • TestAppsUpdateCmdData — --data with an ID is sent as PATCH /clients/<id>.
  • make lint, make build, make test-unit, and make docs (check-docs) all pass.
  • Manual (live tenant), for reviewers:
    • auth0 apps create --schema / auth0 apps list --schema — prints schema, no write.
    • auth0 apps create --data '{"name":"myapp","app_type":"spa"}'
    • auth0 apps update <app-id> --data '{"description":"updated"}'
    • auth0 apps list --query '{"app_type":"spa"}'
    • Mutual exclusion: auth0 apps create --data '{"name":"x"}' --name y errors.

📝 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)

@ramya18101
ramya18101 changed the base branch from main to DXCDT-2107-agent-rollout-for-apis September 16, 2026 08:22
@ramya18101 ramya18101 changed the title Dxcdt 2107 agent rollout for apps feat: add schema and data options for apps management commands Sep 16, 2026
Base automatically changed from DXCDT-2107-agent-rollout-for-apis to main September 17, 2026 07:24
@ramya18101
ramya18101 marked this pull request as ready for review September 17, 2026 07:29
@ramya18101
ramya18101 requested a review from a team as a code owner September 17, 2026 07:29

@developerkunal developerkunal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@developerkunal
developerkunal enabled auto-merge (squash) September 17, 2026 07:37
@developerkunal
developerkunal merged commit 9d06c10 into main Sep 17, 2026
6 checks passed
@developerkunal
developerkunal deleted the DXCDT-2107-agent-rollout-for-apps branch September 17, 2026 07: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.

2 participants