docs: align install, auth, config, and dev docs with current CLI behavior - #1644
Draft
cursor[bot] wants to merge 5 commits into
Draft
cursor[bot] wants to merge 5 commits into
cursor[bot] wants to merge 5 commits into
Conversation
- Place SENTRY_VERSION/SENTRY_INSTALL_DIR on the bash side of the pipe; a prefix on curl never reaches the installer - Document SENTRY_INIT=1, the -v short flag, and the installer telemetry opt-out in getting-started - Clarify that only 'auth login --url' saves the instance URL; logins via SENTRY_HOST/SENTRY_URL need the variable kept set or 'cli defaults url' - Note that package manager installs need 'sentry cli setup' for completions and agent skills Co-authored-by: Miguel Betegón <miguelbetegongarcia@gmail.com>
…t-out - List all 'sentry cli defaults' keys (headers, ca-cert, agent-skills, graphics) and fix the 'sentry upgrade' reference - Add yarn to install method detection; include brew in the 'cli setup --method' brief and fix the stale '--keep-config' path - Correct when agent skills are auto-installed (existing ~/.claude and ~/.agents only; package manager installs need 'cli setup') and document that --no-agent-skills is persisted - Expand the destructive command list in agent safety guidance - Update the plugins README layout for packages/cli and references/ - Regenerate skill files Co-authored-by: Miguel Betegón <miguelbetegongarcia@gmail.com>
- Add a target syntax section to the commands overview: a bare name matches a project first and falls back to an org only for commands that accept one; use <org>/ to target the org explicitly - Use my-org/ in the 'alert metrics create' example - Describe the full 'sentry init' bare-slug behavior (project, then org, then new project) - Update the CONTRIBUTING target syntax list Co-authored-by: Miguel Betegón <miguelbetegongarcia@gmail.com>
- Add the generate:schema/generate:docs bootstrap step needed on a fresh checkout before cli, typecheck, and tests - Use 'pnpm run cli <args>': pnpm forwards a literal '--', so 'pnpm run cli -- --help' fails - Clarify that SENTRY_CLIENT_ID is optional for local runs, required for build/bundle, and that .env.local lives in packages/cli - Fix the contributing test commands and root the generated project tree at packages/cli (drop the stale docs/ entry) - Fix stale docs/src paths and relative links in packages/cli/AGENTS.md - List apps/local in the root README Co-authored-by: Miguel Betegón <miguelbetegongarcia@gmail.com>
The root AGENTS.md was replaced with instructions for an unrelated agent (referencing src/agents/models.ts and scripts/sync-skills.mjs, which do not exist here) as a side effect of #1443. Point it back at packages/cli/AGENTS.md and the lore-managed .lore.md section. Co-authored-by: Miguel Betegón <miguelbetegongarcia@gmail.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Updates the Sentry CLI documentation to match the current CLI implementation: installer usage, self-hosted login,
cli defaults, agent skill installation, bare org/project targets, and the development setup in the pnpm workspace.What changed
Installation and distribution
SENTRY_VERSION/SENTRY_INSTALL_DIRon thebashside of the pipe (curl … | SENTRY_VERSION=0.42.2 bash). A prefix oncurlnever reaches the installer. Fixed in getting-started and the installer's--helptext.SENTRY_INIT=1, the-vshort form of--version, and theSENTRY_CLI_NO_TELEMETRY=1opt-out for installer error reporting.sentry cli setupfor completions and agent skills.clipage adds yarn to install method detection.Authentication and self-hosted usage
sentry auth login --urlsaves the instance as the default URL. A login done throughSENTRY_HOST/SENTRY_URLneeds the variable kept set, orsentry cli defaults url.--urlfor self-hosted and mentionsSENTRY_CLIENT_ID.Configuration
sentry cli defaultskey (headers,ca-cert,agent-skills,graphics, plus the existing ones), and theclipage has examples for the new keys.sentry upgradecorrected tosentry cli upgrade.Commands and usage examples
<org>/targets the org explicitly (matches fix(resolve): use an org slug only when no project matches #1622).alert metrics createexample usesmy-org/.sentry inittarget table describes the full bare-slug behavior (existing project, then org, then new project).Agent skills and plugins
~/.claude/~/.agentsdirectories, that package manager installs needcli setup, and that--no-agent-skillsis remembered. Re-enable withsentry cli defaults agent-skills on.issue mergeandtrial start.packages/cli,references/, and the.well-known/skillspublishing path.SKILL.mdand thealert/cliskill references.Development documentation
pnpm run generate:schema+generate:docsbootstrap. A fresh checkout can't runcli,typecheck, or tests without the generated API schema.pnpm run cli -- --helpchanged topnpm run cli --help: pnpm forwards the literal--, so the old form fails with "Command not found: --help".SENTRY_CLIENT_IDis optional for local runs (falls back to the public sentry.io client ID) and required forbuild/bundle..env.locallives inpackages/cliand is loaded automatically..env.localcopy path fixed. Test commands replaced with ones that actually isolate a file or watch (pnpm --filter sentry exec vitest …).packages/cli/AGENTS.md: fixed staledocs/src/...paths and broken relative links, and described the fullgenerate:docssequence.apps/local.AGENTS.mdrestored to a pointer topackages/cli/AGENTS.mdplus the lore section. fix(span): Handle invalid trace ID in tryAutoSplitSpanArg #1443 had replaced it with instructions for an unrelated agent (in its own commit, so it's easy to drop if unwanted).Implementation updates
packages/cli/install: only the--helpusage examples changed (env var placement). Install logic is unchanged.cli setup --methodbrief now listsbrew, which the parser already accepts and the Homebrew formula passes.cli uninstall --keep-configbrief now names the XDG config dir instead of~/.sentry.script/generate-docs-sections.ts: the generated project tree is rooted atpackages/cli/and no longer lists adocs/directory.Validation
pnpm run generate:schemaandpnpm run generate:docs: pass; regenerated output is committed.check:fragments,check:docs-sections,check:env-coverage,check:stale-refs: pass.pnpm run lint: pass.pnpm exec tsc --noEmit, run after generation: pass.TZ=UTC pnpm run test:unit: 464 files, 9,982 passed, 16 skipped.pnpm --filter sentry-cli-docs run build: pass, 50 pages.sentry …example from the hand-written docs and skill files and checked each command path, flag, and positional count against the live route tree. No mismatches.