chore: add toolkit settings migration - #1597
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Co-Authored-By: OpenAI Codex <noreply@openai.com>
Contributor
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit c344c23. Configure here.
Co-Authored-By: OpenAI Codex <noreply@openai.com>
BYK
added a commit
to getsentry/toolkit
that referenced
this pull request
Sep 30, 2026
## Summary - Import `packages/cli` and `apps/cli-docs` with their unsquashed history from CLI release `0.45.0`, then advance the imported CLI history to the latest `getsentry/cli` main. - Add both packages to the pnpm workspace, restore root patch authority under pnpm 11, and replace Turbo with pnpm-native orchestration and affected-package PR validation. - Keep MCP deployment, smoke, eval, and clean-consumer workflows scoped to their product closures. ## Import provenance - Release source: `03f636f8d1fe6553acf22595eb029ddc8efc79eb`; initial filtered CLI tip: `d5ffbcefa8977382127dc752dd0c78e34fc90ccd`; initial filtered docs tip: `6620138c1cb167babb45987cb2fc499adc00805a`. - Initial import merges: CLI `7d81b40ab93b2f57240cdab3d1b813963c18b062`; docs `86902bac7b2e4ac072ec667b4ab5b11d310450b0`. Both imported trees were byte-identical to their source release trees. The release commit made no docs-tree change, so the filtered docs tip is its parent. - Earlier source-main refresh: `fc1140092883409046438c8bb5d6f7f939eb9659`, with filtered CLI tip `6b08e9029dc7e092bfebaa3870ca116e7dfdfc89` and filtered docs tip `4358ec2729a60ef034f3866680fc5fd7bce62ce0`. - **Latest CLI source main:** `c1337081e382f28a67a0f7594b542e9429402d1c`, imported as filtered CLI commit `5eb94845654fa8e3856a8b0473cb24261c3169d0` by merge `8ff191c70201086a2b391ab3999ef63f6afaeb1d`. Its filtered parent `e34a7c2b31f0501328ab43b4a5a5be31639b8322` maps to upstream `6f9d32308b58633b00fbfe67f288d956d4419d9c`. The latest upstream commit changes only CLI error reporting and its tests; the upstream docs tree is unchanged. The filtered ancestry retains the earlier import history. Toolkit-specific integration changes remain in the destination tree. ## Validation - Frozen pnpm 11 offline install, workspace typecheck/build (`pnpm run tsc`), and workspace lint pass. Lint reports two pre-existing warnings outside this change. - CLI unit suite passes: 479 files, 10,183 passed, 13 skipped. Focused error-reporting tests pass: 92 tests. - The workspace `pnpm run test` reached the MCP core suite after the CLI suite passed, then hit the local command's 10-minute timeout. Exact-head CI validates the remaining packages. - Earlier import validation passed the CLI docs static build, generated-definition and documentation checks, workflow YAML/actionlint checks, the compiled Linux CLI binary, and the CLI E2E suite (144 passed, 3 skipped). Full-range `git diff --check` reports trailing whitespace inherited from the original byte-identical release import. Integration commits pass `git diff --check`; preserving imported history avoids rewriting the source commits. Closes getsentry/cli#1239. Administrative migration is tracked separately in getsentry/cli#1597. The source repository must not be archived until that workflow has migrated settings, open PRs have destination replacements, this integration PR has merged, and release/deployment cutover gates pass.
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
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.
Adds one-off migration tooling for the Toolkit move into
getsentry/sentry-mcp.Settings workflow security properties:
workflow_dispatchfrom the source default branch and pins both repositories by numeric IDpermissions: {}and never checks out or executes repository codeSENTRY_MCP_MIGRATION_PAT, never from workflow inputsproductionenvironment settings without printing valuesOpen-PR migration tooling:
mainValidation:
git diff --checkpassAfter a successful settings run, delete
SENTRY_MCP_MIGRATION_PAT, revoke the PAT, and remove the workflow. Review the generated open-PR migration plan before any execution.Part of #1239.