Skip to content

feat(auth): Make bare sentry auth run login - #1380

Merged
BYK merged 6 commits into
mainfrom
feat/auth-default-login
Aug 7, 2026
Merged

feat(auth): Make bare sentry auth run login#1380
BYK merged 6 commits into
mainfrom
feat/auth-default-login

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Make sentry auth the same as sentry auth login, and teach onboarding/installer copy to recommend the shorter form.

Default command

The auth route default is now login instead of status, so bare sentry auth starts the login flow. sentry auth login still works.

Onboarding copy

Curl installer welcome text, help example, README/quick start, getting-started docs, and skill docs now say sentry auth.

Skill example matching

Bare group examples like sentry auth are associated with the route's default subcommand so generated skill references keep login examples.

Checks: vitest for setup/help/command-suggestions/skill-markdown (78 passed). Manually verified sentry auth starts login.

Requested by David Cramer.

--

View Junior Session [Sentry]

Default the auth route to login so `sentry auth` matches
`sentry auth login`, and point installer/onboarding copy at
the shorter command.

Co-Authored-By: David Cramer <david@sentry.io>
@BYK
BYK marked this pull request as ready for review August 6, 2026 22:44
@BYK BYK added the jared Trigger the Jared agent to work on stuff label Aug 6, 2026
@BYK BYK self-assigned this Aug 6, 2026
@BYK

BYK commented Aug 6, 2026

Copy link
Copy Markdown
Member

Jared, would be ideal if sentry auth would do sentry auth login when logged out and sentry auth status when logged in

@cursor cursor Bot 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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 2d54da3. Configure here.

Comment thread packages/cli/src/commands/cli/setup.ts
sentry-junior Bot and others added 2 commits August 6, 2026 22:52
Bare `sentry auth` now logs in when logged out and shows status when
already authenticated. Login-only flags still force the login path.

Co-Authored-By: David Cramer <david@sentry.io>
Pad the shorter `sentry auth` line so descriptions line up with
`sentry --help` in the post-install Get started output.

Co-Authored-By: David Cramer <david@sentry.io>
@github-actions github-actions Bot added the risk: high PR risk score: high label Aug 6, 2026
github-actions Bot and others added 2 commits August 6, 2026 23:11
Group associateCodeBlocks doc-context params into an options object to
clear useMaxParams, and reflow the test array to biome's formatting.
@jared-outpost

jared-outpost Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

@BYK this is done — bare sentry auth now runs login when logged out and status when logged in (a hidden default dispatcher on the auth route, in b1310a3). Login-only flags like --token/--url still force the login path even when authenticated. sentry auth login and sentry auth status continue to work explicitly.

Also fixed the misaligned installer welcome columns cursor flagged, and cleared a couple of biome lint failures the dispatcher work introduced (useMaxParams on associateCodeBlocks, plus a formatter nit). CI is green on 234adb9.

Comment thread packages/cli/src/commands/auth/default.ts Outdated
Dispatch into unwrapped login/status generators via __rawFunc so auth
guards, rc-URL trust checks, telemetry, and output rendering run once
under the bare-auth command settings.
@BYK
BYK merged commit 196b219 into main Aug 7, 2026
31 checks passed
@BYK
BYK deleted the feat/auth-default-login branch August 7, 2026 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

jared Trigger the Jared agent to work on stuff risk: high PR risk score: high

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant