Skip to content

P49: strict checkjs for web-ai/gemini-live.mjs#51

Merged
lidge-jun merged 1 commit into
mainfrom
chore/strict-migration-p49
May 6, 2026
Merged

P49: strict checkjs for web-ai/gemini-live.mjs#51
lidge-jun merged 1 commit into
mainfrom
chore/strict-migration-p49

Conversation

@lidge-jun
Copy link
Copy Markdown
Owner

VERDICT-B per-file // @ts-check + JSDoc; appended to tsconfig.checkjs-dom.json. No runtime change. 4 gates green (473 tests pass).

VERDICT-B (per-file // @ts-check + JSDoc, no runtime change).

- Add // @ts-check + typedef block (Deps/Input/Page = any)
- JSDoc @param {any} on every named function
- Inline /** @type {any} */ casts on:
  - defineCapability arrow callbacks
  - page.evaluate destructure params
  - shadowRoot target.value access (Element vs HTMLInputElement)
  - caught error e.message access (no new runtime binding)
  - .catch(e =>) callback param
  - copied.status on CaptureCopyResult union
  - WebAiSession -> FinalizeSession at finalizeProviderTab call
  - .map(file => contextPack iteration
  - usedFallbacks let typed as any[]
- Append web-ai/gemini-live.mjs to tsconfig.checkjs-dom.json

All 4 gates green: tsc-dom, tsc-node, node --check, npm test (473 passed).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@lidge-jun lidge-jun force-pushed the chore/strict-migration-p49 branch from ed1b2b5 to adc0117 Compare May 6, 2026 01:17
@lidge-jun lidge-jun merged commit d180ff5 into main May 6, 2026
2 checks passed
@lidge-jun lidge-jun deleted the chore/strict-migration-p49 branch May 6, 2026 01:19
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