feat(seer): Seer Code Review page allows inline editing of the connected repos in a modal

[ryan953]

Some changes:

• Confirm when disconnecting a repo is now inline instead of a modal
  • This is because a modal triggering a modal is annoying and destroys the first modal
• Fixed the optimistic cache updates inside the scmIntegrationTree, so toggling things on/off updates right away. The count above 748 also updates of course
  • when the modal is closed we invalidate the cache of the Repos page behind, adding/removing items at that time
• Added margin-left to fix the 'Nothing found' message alignment, it should be indented a lot just like a found item
• Removed an extra redundant <Stack> from SeerRepoTable and moved <ListItemCheckboxProvider> closer to where it's needed.
  • the outer wrapper has a stack already with the same gap already

[cursor]

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.}

[cursor]

Concurrent optimistic rollback can restore stale cache state

Low Severity

The previousData snapshot captured before an optimistic removal doesn't account for concurrent removals. If a user disconnects repo A then repo B in quick succession, previousDataA is [A, B, C] and previousDataB is [B, C]. If repo A's API call fails, the rollback restores the cache to [A, B, C], undoing repo B's legitimate optimistic removal. Even if repo B's API call succeeds, it won't update the cache, leaving B incorrectly shown as connected until queries are invalidated (e.g., on modal close).