fix(integrations): complete account connections in place

[waleedlatif1]

Summary

• Keep account connection and reconnect on the originating Integrations page, with automatic refresh and a same-tab fallback when popups are unavailable.
• Complete managed MCP authorization directly while preserving invitation submission and existing Search flows.
• Correlate browser and desktop completion per attempt, prevent overlapping starts, and preserve the selected OAuth credential during reconnect.

Type of Change

• Bug fix

Testing

• 197 focused and neighboring tests; five new callback/state regressions demonstrated failing before the fix.
• Browser/Electron E2E: 13 lifecycle checks covering completion, cancellation, reconnect, preserved page state, popup fallback, separate sessions, and invitation compatibility. JSON report and screenshot generated; CI uploads failure artifacts.
• App and desktop type checks, lint, 52 audits, block registry validation, and docs manifest check.

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing (new tests pass the test-audit authoring gate)
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Skipped		Oct 1, 2026 12:48am UTC

[greptile-apps]

Confidence Score: 5/5

[Medium risk] Adds account connection completion flow for integrations.

The PR appears safe to merge; no outstanding findings remain.

Summary

The PR completes managed account authorization through the originating Integrations page, using correlated popup or desktop handoffs and a same-tab fallback. Since the previous review, it adds a shared guard against overlapping connection attempts and extends the desktop fixture to exercise MCP accounts.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Integrations Connect or Reconnect] --> B[Start correlated authorization]
  B --> C{Desktop or web?}
  C -->|Desktop| D[Native browser handoff]
  C -->|Web popup| E[Provider popup]
  C -->|Popup unavailable| F[Same-tab authorization]
  D --> G[Completion page]
  E --> G
  F --> G
  G --> H[Refresh accounts or return to Integrations]

Loading

_{Reviews (2) · Last reviewed commit: "fix(integrations): preserve active accou..."}

[cubic-dev-ai]

All reported issues were addressed across 26 files

<sub>Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic</sub>

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

[cubic-dev-ai]

No issues found across 27 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

_{Re-trigger cubic}