Skip to content

Add real TypeScript typecheck#10

Open
GHX5T-SOL wants to merge 1 commit into
TheStack-ai:mainfrom
GHX5T-SOL:chore/1-real-tsc-typecheck
Open

Add real TypeScript typecheck#10
GHX5T-SOL wants to merge 1 commit into
TheStack-ai:mainfrom
GHX5T-SOL:chore/1-real-tsc-typecheck

Conversation

@GHX5T-SOL
Copy link
Copy Markdown

Fixes #1

Summary:

  • Replaces the syntax-only typecheck script with tsc --noEmit.
  • Adds TypeScript and Node 22 typings, then removes the old local Node shim declarations.
  • Wires the default npm test path through real typechecking before build/test execution.
  • Updates tsconfig.json to ESNext/Bundler resolution so tsc checks the authored source imports while the existing build script continues rewriting runtime import specifiers.
  • Fixes the strict type errors surfaced by the real compiler pass.

Validation:

  • npm run typecheck
  • npm test (131 tests)
  • git diff --check HEAD~1 HEAD
  • git show --format= --patch HEAD | gitleaks stdin --no-banner --redact --timeout 30

Notes:

  • This does not add CI wiring; the issue notes that as later work.
  • The build path is unchanged and still uses scripts/build.mjs.
  • Prepared with AI assistance and verified locally.

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.

tooling: add real TypeScript typechecking (tsc --noEmit)

1 participant