Skip to content

feat(extension): bring Sim chat and page context to Chrome - #9454

Merged
waleedlatif1 merged 9 commits into
stagingfrom
codex/browser-extension
Oct 30, 2026
Merged

waleedlatif1 merged 9 commits into
stagingfrom
codex/browser-extension

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Oct 29, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Add a Chrome side panel using Sim's existing chat components, authentication, history, permissions, and Mothership engine. Organization chat is the default; workspaces remain optional, and search-only members use Assistant.
  • Capture a page or selection explicitly through the toolbar, shortcut, context menu, or composer + menu. Use shared attachment chips and previews, compact header controls, and platform tooltips. Hide the extension's model picker and label its speed toggle “Fast mode”.
  • Enforce bounded extraction, document/window/conversation isolation, exact-origin messaging, replay exclusion, and minimal permissions. Keep page bytes out of extension storage and persisted drafts while preserving ordinary text and files.
  • Preserve active drafts through transient refetch failures, revalidate access and chat policy while the panel is active, preserve drafts during Assistant fallback, fail closed on revocation, keep forks inside the extension, and preserve completed responses until saved history confirms them. Recover workspace selection after temporary failures or session expiry, retain drafts through permission-policy outages, and enforce policy independently of access-request settings. Restricted panels retain navigation back to chat. Use the shared production build for authenticated browser acceptance.

Type of Change

  • New feature

Testing

  • All nine cleanup passes completed. Root lint, 28 workspace type-checks, 62 audits, test-type ratchet, workflow lint, generated artifacts, and block-registry validation passed.
  • Root test suite: 42,015 passed, 8 skipped.
  • Production app build passed. Authenticated production acceptance: 33 checks passed, including real Mothership responses in workspace chat, organization chat, and Assistant; queue editing, reload, forking, ownership, first-open capture, idle access and permission revocation, shared-workspace navigation, attachment recovery, and full-app replay privacy. Live streaming and idle access revocation use native browser timing; simulated cache freshness runs afterward. Organization UI regression suite: 17 checks passed.
  • Packaged Chromium transport: 18 checks passed. Native Chrome QA: 10 checks passed, including toolbar/shortcut/context-menu capture, real permission denial/grant/revocation, file picking, panel reopening, and window isolation.
  • Targeted regressions were reproduced before their fixes or by removing their guards. Production acceptance produced 72 screenshots, covering light/dark themes, 320px/390px layouts, menus and scroll edges, previews, attachments, queues, and recovery states. The local gallery includes six additional native Chrome screenshots. CI uploads reports and screenshots.
  • Chrome Web Store publication and deployed enterprise SSO remain release checks.

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)

@waleedlatif1
waleedlatif1 requested a review from a team as a code owner October 29, 2026 21:36
@vercel

vercel Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Oct 30, 2026 9:16am UTC

Request Review

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 69 files

This PR changes authentication, authorization, or input validation. Ultrareviews find 2.4x more serious bugs than standard reviews. Comment @cubic-dev-ai ultrareview to run one.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/app/extension/[workspaceId]/workspace.tsx Outdated
Comment thread .github/workflows/browser-extension.yml Outdated
Comment thread apps/sim/app/extension/providers.tsx Outdated
Comment thread apps/extension/scripts/test-e2e.ts Outdated
Comment thread apps/extension/turbo.json Outdated
Comment thread apps/extension/scripts/build.ts
Comment thread apps/sim/stores/mothership-drafts/store.ts Outdated
Comment thread apps/extension/src/extract.ts Outdated
Comment thread apps/extension/src/sidepanel.tsx Outdated
Comment thread apps/extension/src/extract.ts Outdated
@greptile-apps

greptile-apps Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The changes appear safe to merge; no new blocking issue was found.

Summary

Adds a Chrome side panel that shares Sim chat, sign-in, history, and explicit page attachments.

  • Chrome users can open Sim chat in a side panel and choose an organization or workspace.
  • People can attach a bounded page snapshot to Sim chat and view it with the message.
  • Sim chat keeps pending drafts and completed replies while supporting the extension composer.

No new actionable issues were found. The seven supplied, unnumbered review threads were rechecked against the current code; their fixes remain present.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Chrome side panel] --> B[Hosted Sim chat]
  C[User attaches page or selection] --> D[Bounded page snapshot]
  D --> B
  B --> E[Existing chat request and saved history]
  F[Current workspace policy] --> G{Chat allowed?}
  G -->|Yes| B
  G -->|No| H[Access required]
  H --> I[Back to organization chat]
Loading

Reviews (7) · Last reviewed commit: "fix(extension): enforce policy independe..." · Reviewed by Greptile

Comment thread apps/sim/app/extension/providers.tsx
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

@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 cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 96 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/hooks/queries/organization.ts
Comment thread apps/sim/scripts/test-browser-extension-e2e.ts Outdated
Comment thread .github/workflows/browser-extension.yml
Comment thread apps/extension/scripts/test-e2e.ts
Comment thread apps/sim/app/extension/[workspaceId]/header.tsx Outdated
@waleedlatif1
waleedlatif1 force-pushed the codex/browser-extension branch from 145901b to 074689c Compare October 30, 2026 04:03
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

@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 cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 96 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/app/extension/[workspaceId]/header.tsx
Comment thread apps/sim/app/extension/o/[organizationId]/organization.tsx Outdated
Comment thread apps/sim/app/extension/workspaces.tsx Outdated
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

@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 cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 102 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/app/extension/workspaces.tsx Outdated
Comment thread apps/sim/app/extension/workspaces.tsx Outdated
@waleedlatif1
waleedlatif1 force-pushed the codex/browser-extension branch from 4ad99bb to 5e21f8e Compare October 30, 2026 07:28
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

@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 cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 103 files

Confidence score: 5/5

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

You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/app/extension/[workspaceId]/workspace.tsx Outdated
@waleedlatif1
waleedlatif1 force-pushed the codex/browser-extension branch from 5e21f8e to 9c9151f Compare October 30, 2026 08:32
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

@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 cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 104 files

Confidence score: 5/5

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

You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/app/extension/[workspaceId]/workspace.tsx
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

@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 cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 106 files

Confidence score: 5/5

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

You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.

Turn on auto-fix | Re-trigger cubic

@waleedlatif1
waleedlatif1 merged commit 5facc5f into staging Oct 30, 2026
77 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/browser-extension branch October 30, 2026 16:35

This branch was previously deployed

1 inactive deployment
Preview — 47eadc47 Deployed Oct 30, 2026 by vercel[bot]
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.

1 participant