Skip to content

P42: strict checkjs for scripts/run-web-ai-eval.mjs#43

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

P42: strict checkjs for scripts/run-web-ai-eval.mjs#43
lidge-jun merged 1 commit into
mainfrom
chore/strict-migration-p42

Conversation

@lidge-jun
Copy link
Copy Markdown
Owner

P42: strict checkjs for scripts/run-web-ai-eval.mjs

50-line CLI entry that wraps runWebAiEval (already checked).

Changes

  • // @ts-check directive
  • JSDoc on main(argv) and printHuman(result)
  • Inline /** @type {any} */ cast on values.concurrency (parseArgs returns string|undefined; runWebAiEval expects number|null|undefined). No runtime cast is compile-time only.change
  • tsconfig.checkjs.json: append entry

Gates

0
0
ok
473 passed, 12 skipped

VERDICT-B: annotation only, no new bindings, no signature changes.

50-line CLI wrapper around runWebAiEval. Adds // @ts-check, JSDoc on
main()/printHuman(), and an inline /** @type {any} */ cast on
values.concurrency to bridge parseArgs string output to runWebAiEval's
number param without changing runtime behavior.

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