Skip to content

feat(search): add Zoom and Google Meet providers - #8483

Merged
waleedlatif1 merged 2 commits into
stagingfrom
codex/meeting-search-providers
Sep 30, 2026
Merged

waleedlatif1 merged 2 commits into
stagingfrom
codex/meeting-search-providers

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Sep 30, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Add member-authorized Zoom meeting search and bounded Google Meet transcript and generated-note discovery through the existing live Search owners.
  • Isolate Zoom Search OAuth from workflow credentials, pin its read scopes, and preserve current account permissions and signed document identities.
  • Add provider-specific query and enrollment documentation; render the Lucid logo white on its dark brand tile.

Companion: https://github.com/simstudioai/mothership/pull/558

Type of Change

  • New feature

Testing

  • Passed the full 19-task workspace suite (34,949 app tests; 25 existing skips), 322 repository-script tests, 15 PostgreSQL/Redis OAuth integration tests, and 58 HTTP provider checks.
  • Verified independent negative controls for authorization, initial/runtime scope pinning, authenticated pagination, unsupported provider kinds, response bounds, and changed document revisions.
  • Passed all 26 workspace type checks, 52 audits, lint, generators, and registry/documentation checks. Reviewed the rendered Lucid icons.
  • HTTP suites use synthetic provider data. Live Zoom and Google account acceptance remains pending account authorization; no claim of live account verification.

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 Sep 30, 2026 11:00pm UTC

Request Review

@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 Sep 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.

@greptile-apps

greptile-apps Bot commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Medium risk] Adds Zoom and Google Meet search integrations.

The PR appears safe to merge based on the reviewed changes.

Summary

The PR adds member-authorized Zoom meeting search and bounded Google Meet artifact discovery, with OAuth configuration, search contracts, UI availability, documentation, and synthetic HTTP coverage.

  • Zoom Search uses a separate, read-scoped OAuth registration and signed search continuations.
  • Google Meet searches recent conference transcripts and exposes generated-note document links without claiming to read note bodies.
  • No actionable new finding was established in the changes since the previous review.
Diagram
%%{init: {'theme': 'neutral'}}%%
flowchart LR
  Member[Member account] --> Search[Live Search]
  Search --> Zoom[Zoom MCP: meeting search and asset read]
  Search --> Meet[Google Meet API: bounded conference and artifact reads]
  Zoom --> Results[Scoped results and citations]
  Meet --> Results
  ZoomOAuth[Separate read-scoped Zoom OAuth app] --> Zoom
Loading

Reviews (2) · Last reviewed commit: "fix(search): bind meeting pagination and..."

@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 41 files

Tip: cubic can generate docs of your entire codebase and keep them up to date. Try it here.

Fix all with cubic | Re-trigger cubic

Comment thread apps/sim/lib/mcp/oauth/provider.ts
Comment thread apps/sim/lib/sim-search/live/application.ts
Comment thread apps/sim/lib/api/contracts/mothership-assistant-tools.ts

@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 41 files

Confidence score: 5/5

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

Tip: cubic can generate docs of your entire codebase and keep them up to date. Try it here.

Re-trigger cubic

@github-actions github-actions Bot added the requires-mothership-merge Has a companion PR on the mothership/copilot side — merge in lockstep label Sep 30, 2026
@github-actions

Copy link
Copy Markdown

⚠️ Cross-repo companion check

One or more companion PRs aren't merged into staging yet. Merging this without them will leave copilot and sim out of sync — merge them in lockstep.

  • ❌ simstudioai/mothership#558 — OPEN, not merged (targets staging) — feat(search): add Zoom and Google Meet companion contracts

@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 Sep 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 46 files

Confidence score: 5/5

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

Tip: cubic can generate docs of your entire codebase and keep them up to date. Try it here.

Re-trigger cubic

@waleedlatif1
waleedlatif1 merged commit 41adb36 into staging Sep 30, 2026
33 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/meeting-search-providers branch September 30, 2026 23:16

This branch was successfully deployed

1 active deployment
Preview — a99e6407 Deployed Sep 30, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

requires-mothership-merge Has a companion PR on the mothership/copilot side — merge in lockstep

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant