ts-review

Review TypeScript diffs for type-design and type-safety issues.

1|Updated Jul 31, 2023
One-click install
npx skills add https://github.com/austintheriot/dotfiles --skill ts-review-austintheriot
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: ts-review
Source: https://github.com/austintheriot/dotfiles/tree/main/.claude/skills/ts-review
Command: npx skills add https://github.com/austintheriot/dotfiles --skill ts-review-austintheriot

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

It helps you catch subtle TypeScript type-design and type-safety issues that typically slip past tsc and ESLint, especially in changed code.

Core Features & Use Cases

  • Diff-based expert review: Reviews the current branch diff against main by default, or a targeted file/PR when specified.
  • Type-system footgun detection: Flags invalid states, unsafe escapes (any/unknown/assertions), generic/inference issues, and boundary-safety problems.
  • Structured, referenceable findings: Outputs severity-labeled findings with file and line references, without posting PR comments.
  • Use cases: Apply it for TypeScript review requests, type-safety audits, or expert opinions on changed code before merge.

Quick Start

Ask the skill to review your changes by sending: /ts-review src/components/Button.tsx.

Frequently Asked Questions about ts-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I catch TypeScript type design issues that pass tsc and ESLint?

TypeScript type design issues that bypass tsc and ESLint include unsafe type escapes, invalid state representations, and generic inference flaws. An expert code review targets these specific footguns by analyzing type boundaries and inference logic to flag subtle type-safety problems in changed code.

How do I run a TypeScript type safety audit on a pull request diff?

Run a TypeScript type safety audit on a pull request diff by targeting the changed code against the main branch. Specify a file path, PR number, or branch diff to generate severity-labeled findings with exact file and line references without posting PR comments or reporting compiler errors.

What is the best way to review TypeScript generics and inference in changed code?

The best way to review TypeScript generics and inference in changed code is through an expert type-safety audit that analyzes type design and inference logic. This approach identifies generic issues and boundary-safety problems beyond standard lint findings, producing structured severity-labeled findings with file and line references.

Can I review a specific file path instead of the full branch diff for TypeScript issues?

Yes, you can review a specific file path instead of the full branch diff for TypeScript issues. By providing an explicit path or PR number, the review targets that specific file, while the default behavior reviews the entire current branch diff against main, excluding generated and lock or build artifacts.

Does this TypeScript review tool automatically post comments to the pull request?

No, this TypeScript review tool does not automatically post comments to the pull request. It produces severity-labeled findings with file and line references as output for the user to review, ensuring you maintain control over what feedback is shared on the pull request.

Why should I use a dedicated TypeScript review instead of just fixing tsc and ESLint errors?

You should use a dedicated TypeScript review because tsc and ESLint miss subtle type-design and type-safety issues like invalid state representations, unsafe assertions, and generic inference flaws. A specialized review analyzes type boundaries and footguns to catch these structural problems before merge.