Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

feat(build): Add build download command - #3221

#3221
Merged
runningcode merged 7 commits into
mastergetsentry/sentry-cli:masterfrom
no/build-downloadgetsentry/sentry-cli:no/build-downloadCopy head branch name to clipboard
Mar 24, 2026
Merged

feat(build): Add build download command#3221
runningcode merged 7 commits into
mastergetsentry/sentry-cli:masterfrom
no/build-downloadgetsentry/sentry-cli:no/build-downloadCopy head branch name to clipboard

Conversation

@runningcode

@runningcode runningcode commented Mar 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds sentry-cli build download subcommand to download installable builds (IPA/APK) by build ID
  • Downloads to a temp file first to avoid partial files on failure

This only works for installable builds. Non-installable builds will show an error message Build {build_id} is not installable.
This requires a personal auth token with the org:read scope

Fixes EME-272

Add a new `sentry-cli build download` subcommand that downloads
installable builds (IPA/APK) from a project by build ID.

The command fetches install details from the API, then downloads
the installable artifact with the correct binary format. For iOS
builds, the plist manifest format is automatically swapped to IPA.
The default output filename uses the correct extension based on
the platform (e.g., .apk for Android, .ipa for iOS).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@linear-code

linear-code Bot commented Mar 17, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Mar 17, 2026

Copy link
Copy Markdown
Contributor
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 3b0c857

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Comment thread src/commands/build/download.rs
runningcode and others added 2 commits March 17, 2026 18:49
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@runningcode
runningcode marked this pull request as ready for review March 19, 2026 08:59
@runningcode
runningcode requested review from a team and szokeasaurusrex as code owners March 19, 2026 08:59
Comment thread src/api/mod.rs

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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.

Comment thread src/api/mod.rs
@szokeasaurusrex

Copy link
Copy Markdown
Member

Please check and resolve the open comments from the AI reviewers, before I do a full review. If any comments are inaccurate or non-actionable, please indicate that with a 👎 and mark the comment as resolved. Thanks! 🙏

@runningcode

Copy link
Copy Markdown
Contributor Author

Resolved!

@szokeasaurusrex szokeasaurusrex 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.

Looks good, but could you please add a test for this new functionality (preferably in this PR, otherwise in a follow up)

runningcode and others added 2 commits March 23, 2026 13:21
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Custom headers that should be attached to all requests
in key:value format.

-p, --project <PROJECT>

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.

Are builds project or org scoped?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good point, the project arg is optional here i just tested.

@runningcode
runningcode enabled auto-merge (squash) March 24, 2026 14:58
Builds are org-scoped, so the project flag is unnecessary on the
download subcommand. Also marks the command as [EXPERIMENTAL] to
match the pattern used by the snapshots subcommand.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.