chore: typecheck test files; add CI workflow

[shadowbrush]

Summary

npm run typecheck ignored every test file — tsconfig.json excluded src/**/*.test.ts, so type errors in tests were invisible. This ports the split hadrontool-twilio established (87f8659), where enabling it immediately surfaced two real type errors in that repo's tests:

• tsconfig.json excludes only node_modules/dist → typecheck covers tests.
• New tsconfig.build.json (extends the base, re-adds the test excludes) → npm run build still keeps test files out of dist/.
• Also adds the family-standard CI workflow — this repo had none, so nothing ran typecheck or tests on push/PR at all. node-version pinned to the engines floor (>=22.12.0).

Verification

• 4 test files now appear in tsc --listFiles — zero latent errors in this repo (the tests were clean).
• npm run build emits no test artifacts into dist/.
• npm test: 56 passed.

🤖 Generated with Claude Code

[gemini-code-assist]

Code Review

This pull request separates the build configuration from the main TypeScript configuration by introducing a new tsconfig.build.json file and updating the build script in package.json. This ensures test files are excluded during the build while remaining available for type-checking. Feedback highlights a non-standard glob pattern in the TypeScript exclusion list and notes that the CI workflow file mentioned in the pull request description is missing from the changes.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[gemini-code-assist]

In TypeScript tsconfig.json glob patterns, ** is a recursive directory wildcard and should be followed by a slash and a pattern (e.g., **/*), or you can simply specify the directory path (e.g., "src/test") to exclude the directory and all of its contents. Using "src/test/**" is non-standard and may not be parsed correctly by tsc, which could result in files under src/test/ not being excluded during the build and being emitted to the dist/ directory.

Suggested change

	"exclude": ["node_modules", "dist", "src/**/*.test.ts", "src/test/**"]
	"exclude": ["node_modules", "dist", "src/**/*.test.ts", "src/test"]

[gemini-code-assist]

The pull request description mentions adding the "family-standard CI workflow" pinned to Node.js >=22.12.0. However, no CI workflow files (such as .github/workflows/ci.yml) are included in the changes of this pull request. Please add the missing workflow file to complete the implementation.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d7fc3230db

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Add tsconfig.build.json to the Docker image

When this build script is invoked by the production Dockerfile, only package*.json, tsconfig.json, and src are copied before RUN npm run build (Dockerfile:28-32); the new tsconfig.build.json is never present in the image. Because this line now points tsc at that missing file, a clean docker build fails with a missing-config error before any deployment image can be produced.

Useful? React with 👍 / 👎.

[Copilot]

Pull request overview

This PR updates TypeScript configuration so npm run typecheck includes test files while npm run build still excludes them from emitted output, and adds a basic CI workflow to run typechecking and tests on pushes/PRs.

Changes:

• Remove test-file excludes from tsconfig.json so typechecking covers tests.
• Add tsconfig.build.json and point npm run build at it to keep tests out of dist/.
• Add a GitHub Actions workflow to run npm ci, npm run typecheck, and npm test.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File	Description
tsconfig.json	Stops excluding tests so tsc --noEmit typechecks them.
tsconfig.build.json	Build-only TS config that re-excludes tests from emitted output.
package.json	Updates build script to compile with the build tsconfig.
.github/workflows/ci.yml	Adds CI job to install deps, typecheck, and run tests.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[shadowbrush]

Addressed in the latest commit:

• Codex (P1 — Dockerfile): Correct and important — the image never copied tsconfig.build.json, so npm run build (now tsc -p tsconfig.build.json) would fail in a clean docker build. CI doesn't build the image (Komodo does, on merge), so it was uncaught. Fixed: COPY tsconfig.json tsconfig.build.json ./. Verified by reproducing the Dockerfile's exact COPY set in a clean dir and building — exit 0, zero test files emitted.
• Gemini (src/test/** glob): Dropped it entirely rather than reworking it — there is no src/test/ directory in this repo; src/**/*.test.ts already excludes every test file from the build (confirmed dist/ has zero test artifacts).
• Gemini (CI workflow "missing"): Stale — this comment was on an earlier commit. .github/workflows/ci.yml is present in the PR and is green.
• Copilot (node-version: '>=22.12.0'): Not a failure in practice — actions/setup-node@v4 resolves the range and CI ran green. Leaving as the engines-floor spec, consistent with the PR intent.