docs(snapshots): Add sentry-cli reference page for snapshots#17464
Open
runningcode wants to merge 2 commits intomtopo27/snapshots-ea-product-docsfrom
Open
docs(snapshots): Add sentry-cli reference page for snapshots#17464runningcode wants to merge 2 commits intomtopo27/snapshots-ea-product-docsfrom
runningcode wants to merge 2 commits intomtopo27/snapshots-ea-product-docsfrom
Conversation
Add a dedicated /cli/snapshots/ page documenting the experimental `sentry-cli build snapshots` command: basic usage, git metadata auto-detection behavior, --selective sharding, and the full flag reference. Move the flag table out of the uploading-snapshots product page and link to the new CLI page instead. Expand the Android guide's one-line sentry-cli mention into a short section with a concrete example for users who don't use the Sentry Android Gradle Plugin.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
Comment on lines
+91
to
+101
| | `--vcs-provider <PROVIDER>` | VCS provider (for example, `github`). Auto-detected from the git remote. | | ||
| | `--head-repo-name <OWNER/REPO>` | Repository in `owner/repo` format. Auto-detected from the git remote. | | ||
| | `--base-repo-name <OWNER/REPO>` | Base repository in `owner/repo` format (required for PRs from forks). | | ||
| | `--head-ref <BRANCH>` | Head branch name. Auto-detected in CI. | | ||
| | `--base-ref <BRANCH>` | Base branch name (PR only). Auto-detected from the remote tracking branch. | | ||
| | `--pr-number <NUMBER>` | Pull request number. Auto-detected in GitHub Actions `pull_request` workflows. | | ||
| | `--no-git-metadata` | Skip automatic git metadata detection. | | ||
| | `--force-git-metadata` | Force git metadata collection outside CI. | | ||
| | `--header <KEY:VALUE>` | Custom header attached to all requests. | | ||
| | `--log-level <LEVEL>` | Logging verbosity: `trace`, `debug`, `info`, `warn`, or `error`. | | ||
| | `--quiet` | Suppress output while preserving exit codes. Alias: `--silent`. | |
There was a problem hiding this comment.
Bug: The new documentation for sentry-cli build snapshots omits the --url flag, which was present in the previous documentation, creating a documentation regression.
Severity: MEDIUM
Suggested Fix
Restore the --url flag and its description to the flag reference table in docs/cli/snapshots.mdx. If the flag is no longer functional or is deprecated, this should be explicitly documented rather than silently removing it.
Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.
Location: docs/cli/snapshots.mdx#L80-L101
Potential issue: The documentation for `sentry-cli build snapshots` was moved to a new
page, `docs/cli/snapshots.mdx`. In the process, the `--url <URL>` flag, which was
present in the old documentation, was omitted from the flag reference table in the new
file. The new page is presented as the "full flag reference," making this omission a
documentation regression. This can negatively impact users of self-hosted Sentry
instances who may rely on this flag, as they will no longer find it in the canonical
reference. The removal appears to be unintentional.
Did we get this right? 👍 / 👎 to inform future reviews.
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
1 participant
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.
DESCRIBE YOUR PR
Add a dedicated
/cli/snapshots/page forsentry-cli build snapshots, the command that underpins Snapshots uploads. Makes the command discoverable from the CLI left nav and documents behavior that wasn't previously covered.docs/cli/snapshots.mdx— basic usage, git metadata auto-detection (head/base SHA, VCS provider, PR number),--selectivesharding,--diff-threshold, and a full flag reference.sentry-cli build snapshotsflag table out ofdocs/product/snapshots/uploading-snapshots/index.mdx— that page now links to the CLI reference.sentry-climention indocs/platforms/android/snapshots/index.mdxinto a short section with an example for Android users who don't use the Sentry Android Gradle Plugin.Stacked on top of
mtopo27/snapshots-ea-product-docs.IS YOUR CHANGE URGENT?
Help us prioritize incoming PRs by letting us know when the change needs to go live.
SLA
Thanks in advance for your help!
PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes: