Skip to content

refactor(google-meet): localize internal declarations#102010

Merged
vincentkoc merged 1 commit into
mainfrom
refactor/deadcode-all-surfaces-20260708-v80
Jul 8, 2026
Merged

refactor(google-meet): localize internal declarations#102010
vincentkoc merged 1 commit into
mainfrom
refactor/deadcode-all-surfaces-20260708-v80

Conversation

@vincentkoc

Copy link
Copy Markdown
Member

What Problem This Solves

Eleven Google Meet implementation declarations were exported even though they have no repository consumers and are not re-exported by the plugin entrypoint.

Why This Change Was Made

Keep CLI manifest shapes, internal Meet API response types, the OAuth token result, the realtime strategy alias, and the audio buffer constant local to their owning modules. The exported runtime functions and supported plugin entrypoint remain unchanged.

User Impact

None. This is export-surface cleanup only; Google Meet CLI, config, OAuth, API, and transport behavior are unchanged.

Evidence

  • Fresh codebase-memory graph: 281,366 nodes / 1,134,640 edges.
  • MCP inbound-edge query found only each symbol's defining edge; repository/docs search found no imports, consumers, or entrypoint re-exports.
  • Testbox tbx_01kwzxqeaarkf2p9n587zsr0jj: pnpm check:changed passed for extension production and test lanes.
  • Five focused Google Meet suites passed: 156/156 tests.
  • Full build passed, including TypeScript declaration emit, runtime postbuild, plugin SDK export checks, and Control UI build.
  • Fresh autoreview: local 0.84 and branch 0.90; no accepted/actionable findings.

@vincentkoc

Copy link
Copy Markdown
Member Author

Land-ready proof for 550c5949ce654ad7be599a595adc7d877ec2a002:

  • Fresh full-clone codebase-memory index: 281,366 nodes / 1,134,640 edges; schema and live symbol query passed.
  • MCP inbound-edge inspection found only each declaration's defining edge. Repository and docs searches found no imports, consumers, or Google Meet entrypoint re-exports.
  • Testbox tbx_01kwzxqeaarkf2p9n587zsr0jj: pnpm check:changed passed for extension production and test lanes.
  • Five focused Google Meet suites passed: 156/156 tests.
  • Full build passed, including declaration emit, runtime postbuild, plugin SDK export checks, and Control UI build.
  • Fresh autoreview is clean: local 0.84 and branch 0.90.
  • scripts/pr review-validate-artifacts 102010 and OPENCLAW_TESTBOX=1 scripts/pr prepare-run 102010 passed with zero findings.

@vincentkoc
vincentkoc marked this pull request as ready for review July 8, 2026 04:02
@vincentkoc
vincentkoc merged commit 827f2c4 into main Jul 8, 2026
118 of 120 checks passed
@vincentkoc
vincentkoc deleted the refactor/deadcode-all-surfaces-20260708-v80 branch July 8, 2026 04:02
@vincentkoc

Copy link
Copy Markdown
Member Author

Merged via squash.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant