chore: simplify dev and test tooling

[Waishnav]

Adding a test currently requires manually extending the package test command, while local server reloads are maintained by a custom watcher and process supervisor. This makes *.test.ts naming the source of truth while preserving serial execution, and keeps type checking focused on TypeScript only.

Local development now uses tsx’s built-in watch mode directly, removing the custom reload supervisor while preserving source-change restarts and clean shutdown behavior.

Summary by CodeRabbit

• Chores
  • Updated the development command to use the TypeScript CLI with automatic reload support.
  • Simplified test execution to discover and run all TypeScript tests sequentially.
  • Streamlined type checking to use the project’s TypeScript configuration.
  • Removed the legacy development server supervisor.

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0a787f4a-d4a0-43e4-99c1-f60051146525

📥 Commits

Reviewing files that changed from the base of the PR and between bde69d0 and 89e2e74.

📒 Files selected for processing (2)

• package.json
• scripts/dev-server.mjs

💤 Files with no reviewable changes (1)

• scripts/dev-server.mjs

Included review availability: Your plan provides up to 8 included reviews per hour; 4 remain after this review.

---

📝 Walkthrough

Walkthrough

The development command now runs the CLI in watch mode. The test command discovers TypeScript tests through a glob with serial concurrency. The typecheck command now runs only tsc. The previous development server supervisor was deleted.

Changes

Development workflow

Layer / File(s)	Summary
Package script execution changes package.json, scripts/dev-server.mjs	The dev script now uses tsx watch with src/cli.ts serve. The test script runs all src/**/*.test.ts files with serial concurrency. The typecheck script runs only TypeScript compilation. The development server supervisor was removed.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 89e2e

The PR simplifies development, testing, and type-checking commands without introducing an actionable merge-blocking risk; it is merge-ready after normal checks and review.

Poem

A rabbit watches the CLI glow
Tests hop through files in a row
Type checks run clean and bright
The old server sleeps tonight
Fresh scripts guide the flow

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main changes to development and test tooling. It is concise and specific.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/dev-test-tooling

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[greptile-apps]

Greptile Summary

The PR simplifies development and test tooling by replacing the custom development supervisor with tsx watch, automatically discovering TypeScript tests, and limiting typecheck to the TypeScript compiler.

• Uses serial Node test-runner discovery for src/**/*.test.ts.
• Runs the development server directly through tsx watch.
• Deletes the custom recursive watcher and process supervisor.
• Removes schema validation execution from the typecheck command while retaining it through test discovery.

Confidence Score: 4/5

The development-tooling change should be fixed before merging because the server no longer recovers automatically after an unexpected startup exit.

The new tsx watch command waits for watched source changes after its child exits, while the deleted supervisor retried crashes; failures caused by external configuration can therefore leave local development stopped even after that configuration is corrected.

Files Needing Attention: package.json, scripts/dev-server.mjs

Important Files Changed

Filename	Overview
package.json	Simplifies test, typecheck, and development commands, but the new development command loses automatic recovery after startup failures.
scripts/dev-server.mjs	Deletes the custom watcher and supervisor, including its delayed restart behavior after unexpected child exits.

_{Reviews (1): Last reviewed commit: "chore(dev): use tsx watch for reloads" | Re-trigger Greptile}