Repository navigation
Conversation
Removes the Code Mappings tab from the integration configuration page. Code mappings are now managed from the project's Connected Repositories panel and the org Repositories page. This deletes the tab's components and specs, drops the now-unused codeMappings tab type and the analytics view and event only that tab sent, and shows the Settings tab only when other tabs exist. Refs https://linear.app/getsentry/project/code-mappings-refactor-d3987c38f5f1/overview Co-authored-by: Cursor <cursoragent@cursor.com>
5ab37d1 to
173c4e2
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
There are 2 total unresolved issues (including 1 from previous review).
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 173c4e2. Configure here.
| ); | ||
|
|
||
| type Tab = 'settings' | 'codeMappings' | 'userMappings' | 'teamMappings'; | ||
| type Tab = 'settings' | 'userMappings' | 'teamMappings'; |
There was a problem hiding this comment.
Stale link to removed mappings tab
Medium Severity
Removing the codeMappings tab leaves the Add Code Owner empty state linking to ?tab=codeMappings on the integration settings page. That query no longer matches a tab, so the link falls back to Settings or User Mappings instead of a place to create mappings. Users with no mappings cannot finish CODEOWNERS setup from that modal.
Reviewed by Cursor Bugbot for commit 173c4e2. Configure here.
The nth-child(3)/nth-child(4) selectors assumed Code Mappings was the second tab. Navigate directly to the correct tab via the URL query param instead, which is robust to tab additions or removals. Co-authored-by: Cursor <cursoragent@cursor.com>
…tests" This reverts commit 8e4067b.


Removes the Code Mappings tab from the integration configuration page. Code mappings are now managed from the project's Connected Repositories panel and the org Repositories page. This deletes the tab's components and specs, drops the now-unused
codeMappingstab type and the analytics view and event only that tab sent, and shows the Settings tab only when other tabs exist.Part of the Code Mappings Refactor project.
Depends on #129491 and #129513, which must merge first.