Skip to content

P33: strict checkjs for web-ai/action-intent.mjs#34

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

P33: strict checkjs for web-ai/action-intent.mjs#34
lidge-jun merged 1 commit into
mainfrom
chore/strict-migration-p33

Conversation

@lidge-jun
Copy link
Copy Markdown
Owner

P33: web-ai/action-intent.mjs strict checkjs

VERDICT-B per-file // @ts-check + JSDoc. No runtime change.

Changes

  • web-ai/action-intent.mjs: ActionIntentInput typedef + JSDoc on 4 functions
  • OPERATION_BY_INTENT typed as Readonly<Record<string,string>>
  • tsconfig.checkjs-dom.json: append web-ai/action-intent.mjs

Gates

  • typecheck dom + node
  • node --check
  • npm test (473 passed, 12 skipped)

Annotate action-intent.mjs with // @ts-check + JSDoc and add to
tsconfig.checkjs-dom.json (depends on self-heal which is in DOM config).

- ActionIntentInput typedef
- OPERATION_BY_INTENT typed as Readonly<Record<string,string>>
- 4 functions JSDoc'd (createActionIntent, serializeActionIntent,
  requiredEvidenceForOperation, patternToHint)
- Inline cast for semanticTarget.testIds (not in SemanticTarget typedef)
- All 4 gates pass

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@lidge-jun lidge-jun merged commit 47d8be4 into main May 5, 2026
2 checks passed
@lidge-jun lidge-jun deleted the chore/strict-migration-p33 branch May 6, 2026 01:44
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