Skip to content

fix(auth): request team admin OAuth scope - #1373

Draft
betegon wants to merge 1 commit into
mainfrom
fix/oauth-team-admin-scope
Draft

fix(auth): request team admin OAuth scope#1373
betegon wants to merge 1 commit into
mainfrom
fix/oauth-team-admin-scope

Conversation

@betegon

@betegon betegon commented Aug 6, 2026

Copy link
Copy Markdown
Member

Summary

Add team:admin to the CLI standard OAuth scope set so authenticated Team Admins can create projects through team-scoped API endpoints. Regenerate the documentation projections owned by the OAuth scope generator.

Existing stored OAuth grants may predate this scope. When an interactive command receives a scope-specific 403, the CLI now offers one refresh with the current standard scopes and retries exactly once. It does not start OAuth for --yes, --dry-run, JSON/non-interactive execution, or effective environment-token auth. Organization policy denials are excluded from scope recovery.

This expands what the token may do; it does not grant Team Admin status to users who do not already have that role.

Test plan

  • pnpm exec vitest run test/lib/oauth.test.ts test/lib/api-scope.test.ts test/lib/scope-recovery.test.ts test/lib/auto-auth.test.ts --coverage=false
  • pnpm exec biome check src/cli.ts src/lib/api-scope.ts src/lib/scope-recovery.ts src/lib/oauth.ts test/lib/api-scope.test.ts test/lib/scope-recovery.test.ts test/lib/oauth.test.ts
  • pnpm exec tsc --noEmit
  • pnpm run check:docs-sections

@BYK BYK left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should make sure to refresh with the new scope on failure for existing tokens.

@betegon

betegon commented Aug 7, 2026

Copy link
Copy Markdown
Member Author

Thanks for the heads up! I'll check it out

@betegon
betegon force-pushed the fix/oauth-team-admin-scope branch from 1842ad4 to 1e4ac57 Compare August 7, 2026 10:21
@betegon

betegon commented Aug 7, 2026

Copy link
Copy Markdown
Member Author

Implemented the existing-grant path from the review: on a scope-specific 403 from the effective stored OAuth credential, the CLI offers a one-time refresh with the current standard scope set and retries once. Unattended/non-interactive/JSON runs and env-token auth never enter the device flow, and policy denials are not treated as missing OAuth scopes.

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