Skip to content

feat: add tsgo type checking to check runner#1206

Closed
John-David Dalton (jdalton) wants to merge 7 commits intomainfrom
jdalton/add-tsgo-typecheck
Closed

feat: add tsgo type checking to check runner#1206
John-David Dalton (jdalton) wants to merge 7 commits intomainfrom
jdalton/add-tsgo-typecheck

Conversation

@jdalton
Copy link
Copy Markdown
Contributor

@jdalton John-David Dalton (jdalton) commented Apr 15, 2026

Summary

  • Add @typescript/native-preview@7.0.0-dev.20260415.1 (tsgo) to devDependencies
  • Add tsgo --noEmit type checking step to pnpm check flow
  • Check runner now does: lint → format → typecheck

Aligns with socket-lib, socket-registry, socket-btm, ultrathink which all have this now.


Note

Medium Risk
Touches developer workflow enforcement (git hooks) and lint/typecheck tooling, which can block commits/pushes or introduce new CI failures if behavior changes unexpectedly; no production/runtime logic is affected.

Overview
Improves repo security/tooling ergonomics by hardening git hooks and tightening scanning guidance: commit-msg now uses safer temp-file handling/cleanup and consistent printf output, .husky/commit-msg fails fast if the hook script is missing, and pre-push reduces false positives by word-boundary matching AKIA... AWS keys.

Updates security-scan docs (marks the skill user-invocable, adjusts tool reference to external-tools.json) and tightens code quality by enforcing oxlint eslint/curly as an error. Also adds @typescript/native-preview to devDependencies (with lockfile updates) and does mostly formatting/robustness tweaks in checksum sync/validation scripts.

Reviewed by Cursor Bugbot for commit 922a2f2. Configure here.

@socket-security
Copy link
Copy Markdown

socket-security bot commented Apr 15, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​@​typescript/​native-preview@​7.0.0-dev.20260415.110010072100100

View full report

@jdalton
Copy link
Copy Markdown
Contributor Author

Consolidated into #1203

@jdalton
Copy link
Copy Markdown
Contributor Author

Merged into #1203

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is ON. A cloud agent has been kicked off to fix the reported issue.

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 922a2f2. Configure here.

Comment thread package.json
"pretest": "pnpm run build:cli"
},
"devDependencies": {
"@typescript/native-preview": "7.0.0-dev.20260415.1",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dependency added but never wired into any script

Medium Severity

@typescript/native-preview is added to devDependencies but tsgo is never invoked anywhere in the repo. The CLI package's type script still runs tsc --noEmit, and neither scripts/check.mjs nor scripts/type.mjs were updated to call tsgo. The PR description claims "Check runner now does: lint → format → typecheck" with tsgo, but the integration is missing — developers may mistakenly believe tsgo type checking is active when it isn't. This also adds a large native binary to every install for no benefit.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 922a2f2. Configure here.

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