Skip to content

Conversation

@szokeasaurusrex
Copy link
Member

@szokeasaurusrex szokeasaurusrex commented Dec 2, 2025

Description

Removed the deprecated --use-artifact-bundle flag from the sentry-cli sourcemaps upload command and its corresponding useArtifactBundle option from the Node.js API. This flag was a no-op that only emitted a deprecation warning, and its removal simplifies the CLI and API by eliminating an unused and misleading option.

Issues


Open in Cursor Open in Web


Co-authored-by: Daniel Szoke daniel.szoke@sentry.io

@cursor
Copy link

cursor bot commented Dec 2, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@linear
Copy link

linear bot commented Dec 2, 2025

@szokeasaurusrex szokeasaurusrex force-pushed the cursor/remove-sourcemaps-upload-artifact-bundle-option-gpt-5.1-codex-602d branch from f3f2e82 to 14a1fe7 Compare December 3, 2025 10:34
@szokeasaurusrex szokeasaurusrex changed the title Remove sourcemaps upload artifact bundle option Remove sourcemaps upload artifact bundle option (#3002) Dec 3, 2025
szokeasaurusrex added a commit that referenced this pull request Dec 3, 2025
### Description
Removed the deprecated `--use-artifact-bundle` flag from the `sentry-cli sourcemaps upload` command and its corresponding `useArtifactBundle` option from the Node.js API. This flag was a no-op that only emitted a deprecation warning, and its removal simplifies the CLI and API by eliminating an unused and misleading option.

### Issues
* resolves: #2352
* resolves: CLI-20

---
<a href="https://cursor.com/background-agent?bcId=bc-2137bde5-9602-436e-9260-e3a6e192ae9c"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-cursor-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-cursor-light.svg"><img alt="Open in Cursor" src="https://cursor.com/open-in-cursor.svg"></picture></a>&nbsp;<a href="https://cursor.com/agents?id=bc-2137bde5-9602-436e-9260-e3a6e192ae9c"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-web-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-web-light.svg"><img alt="Open in Web" src="https://cursor.com/open-in-web.svg"></picture></a>

___

Co-authored-by: Daniel Szoke <daniel.szoke@sentry.io>
@szokeasaurusrex szokeasaurusrex force-pushed the cursor/remove-sourcemaps-upload-artifact-bundle-option-gpt-5.1-codex-602d branch from 14a1fe7 to 3f526bf Compare December 3, 2025 10:42
### Description
Removed the deprecated `--use-artifact-bundle` flag from the `sentry-cli sourcemaps upload` command and its corresponding `useArtifactBundle` option from the Node.js API. This flag was a no-op that only emitted a deprecation warning, and its removal simplifies the CLI and API by eliminating an unused and misleading option.

### Issues
* resolves: #2352
* resolves: CLI-20

---
<a href="https://cursor.com/background-agent?bcId=bc-2137bde5-9602-436e-9260-e3a6e192ae9c"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-cursor-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-cursor-light.svg"><img alt="Open in Cursor" src="https://cursor.com/open-in-cursor.svg"></picture></a>&nbsp;<a href="https://cursor.com/agents?id=bc-2137bde5-9602-436e-9260-e3a6e192ae9c"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-web-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-web-light.svg"><img alt="Open in Web" src="https://cursor.com/open-in-web.svg"></picture></a>

___

Co-authored-by: Daniel Szoke <daniel.szoke@sentry.io>
@szokeasaurusrex szokeasaurusrex force-pushed the cursor/remove-sourcemaps-upload-artifact-bundle-option-gpt-5.1-codex-602d branch from 3f526bf to 2a930a0 Compare December 3, 2025 10:43
@szokeasaurusrex szokeasaurusrex changed the title Remove sourcemaps upload artifact bundle option (#3002) chore(sourcemaps): Remove --use-artifact-bundle (#3002) Dec 3, 2025
@szokeasaurusrex szokeasaurusrex changed the title chore(sourcemaps): Remove --use-artifact-bundle (#3002) chore(sourcemaps): Remove --use-artifact-bundle Dec 3, 2025
@szokeasaurusrex szokeasaurusrex changed the base branch from v3 to graphite-base/3002 December 3, 2025 10:44
@szokeasaurusrex szokeasaurusrex changed the base branch from graphite-base/3002 to szokeasaurusrex/no-project-release-endpoint-calls December 3, 2025 10:44
@szokeasaurusrex szokeasaurusrex marked this pull request as ready for review December 3, 2025 10:44
@szokeasaurusrex szokeasaurusrex requested review from a team as code owners December 3, 2025 10:44
@szokeasaurusrex szokeasaurusrex requested review from andreiborza, loewenheim and stephanie-anderson and removed request for a team and stephanie-anderson December 3, 2025 10:44
Copy link
Member Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

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