Skip to content

fix(search): gate Zoom rollout and correct Lucid tiles - #8491

Merged
waleedlatif1 merged 1 commit into
stagingfrom
codex/meeting-search-rollout
Oct 1, 2026
Merged

waleedlatif1 merged 1 commit into
stagingfrom
codex/meeting-search-rollout

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Sep 30, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Add an organization-scoped, default-off rollout for Zoom Search enrollment and retrieval. Keep standard workflow Zoom OAuth and Google Meet behavior unchanged.
  • Preserve Remove source and Disconnect for saved Zoom approvals/accounts while disabled; expose optional availability on read responses only.
  • Keep the Lucid glyph transparent and apply its background through shared tile metadata.

Type of Change

  • Bug fix

Testing

  • 1,534 nearby regression tests across 165 files; 33 real PostgreSQL/Redis integration tests.
  • Three independent guard-removal controls failed at the intended boundary, followed by a restored green integration run.
  • All 26 workspace typechecks, 52 audits, lint, block registry and generated contract/catalog checks passed. Worker contract and CLI inventory checks remain aligned.
  • Browser verified Google Meet Connect stays available and Zoom is hidden while disabled, plus Lucid rendering in light/dark tiles and as a bare glyph. Real provider-account OAuth acceptance is not claimed.

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

vercel Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
docs Ready Ready Preview Oct 1, 2026 12:32am 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 30 files

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

Fix all with cubic | Re-trigger cubic

Comment thread apps/sim/lib/credentials/managed-mcp.ts
@greptile-apps

greptile-apps Bot commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Medium risk] Adds feature flag for Zoom search and updates integration UI.

The PR appears safe to merge, with a non-blocking job-lookup fallback regression worth addressing.

Summary

The PR gates Zoom MCP Search setup, enrollment, and retrieval by organization while retaining management of saved connections. It also moves the Lucid tile background into shared connector metadata and adds Trigger run receipts for early job lookup.

  • Google Meet’s separate rollout checks are removed.
  • Search read contracts expose optional, server-computed availability.
  • Trigger receipts supplement tag-based job discovery and cancellation.
Diagram
%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Stable job ID] --> B[Trigger run ID lookup]
  B --> C[Database receipt lookup]
  C --> D[Trigger tag lookup]
  D --> E[Job status or cancellation]
Loading

Reviews (2) · Last reviewed commit: "fix(search): gate Zoom rollout and corre..."

Comment thread apps/sim/lib/credential-groups/oauth.ts Outdated
Comment thread apps/sim/lib/knowledge/application/search-integrations.ts Outdated
@waleedlatif1 waleedlatif1 changed the title fix(search): gate meeting providers and correct Lucid tiles fix(search): gate Zoom rollout and correct Lucid tiles Oct 1, 2026
@waleedlatif1
waleedlatif1 force-pushed the codex/meeting-search-rollout branch from d75fc5f to 3ac918b Compare October 1, 2026 00:30
@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 1, 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 29 files

Confidence score: 5/5

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

Re-trigger cubic

@waleedlatif1
waleedlatif1 merged commit dd54d31 into staging Oct 1, 2026
23 of 24 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/meeting-search-rollout branch October 1, 2026 00:39

This branch was successfully deployed

1 active deployment
Preview — 3ac918b6 Deployed Oct 1, 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