chore: Clean up workspace dependencies and add ai-review tooling

[KiwiGaze]

Summary

• Remove unused tsx dependency from root devDependencies
• Remove unused @simplifying/errors dependency from packages/uploads
• Register tools/* directory in pnpm-workspace.yaml for internal tooling
• Add tools/ai-review package with vitest configuration

Test plan

• pnpm install resolves cleanly
• pnpm type-check:all passes
• pnpm test:run passes
• pnpm build succeeds

Summary by CodeRabbit

• Chores

  • Removed an unused development tooling dependency
  • Removed an unused runtime dependency from the uploads package
  • Expanded workspace configuration to include a tools directory
  • Added new AI review tooling infrastructure

• Tests

  • Included test tooling/configuration for the new AI review tooling

• Documentation

  • Added a note about clean Bash command styling in internal docs

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

---

Walkthrough

Adds a new tools/ai-review package (package.json + Vitest config), registers tools/* in the pnpm workspace, removes tsx from the root devDependencies, removes @simplifying/errors from packages/uploads, and adds a CLAUDE.md styling bullet.

Changes

Cohort / File(s)	Summary
Workspace & Root pnpm-workspace.yaml, package.json	Registered tools/* in workspace patterns; removed tsx devDependency from root package.json.
Uploads Package packages/uploads/package.json	Removed dependency @simplifying/errors from packages/uploads.
New AI Review Tool tools/ai-review/package.json, tools/ai-review/vitest.config.ts	Added @simplifying/ai-review package with TypeScript + Vitest devDependencies and a Vitest config including test include pattern.
Docs / Guidelines CLAUDE.md	Added a bullet enforcing no inline comments in Bash commands.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• refactor(core): Move web app to apps/web/ (R-7 PR3) #140 — modifies workspace patterns (both change pnpm-workspace.yaml workspace entries).
• refactor(tooling): make lint scripts layout-aware for R-7 #138 — touches tools/ai-review scripts/configs related to the new package added here.
• chore: Add PR and uncommitted change review checklists #114 — extends the tools/ai-review tooling (scripts and checks) that this PR introduces.

🚥 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: removing unused dependencies and adding the new ai-review tooling package, with the workspace pattern update as a supporting change.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Peer Dependency Consistency	✅ Passed	No packages in packages/* directory declare peerDependencies; PR only removed a regular dependency from packages/uploads.
Api Contract Layer Conventions	✅ Passed	No API route handlers were added, modified, or touched by changes in this PR. The check only applies to new/modified route handlers and their dependencies.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch chore/r9-dependency-tooling-cleanup-workspace-test-decision

---

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