用 Codex 或 Claude 帮你安装 复制这段 Prompt,粘贴到 Codex、Claude 或其他助手里,让它检查 Skill 页面并帮你完成安装。
直接命令不会经过审查 Prompt;运行前请先检查来源。
npx skills add https://github.com/irahardianto/awesome-agv --skill typescript-idioms命令会保持在同一行。复制前请横向滚动并检查完整内容。
想先保存到本地?可下载 SkillsMP 当前能够提供的文件。
基于 SOC 职业分类
正在显示 SKILL.md
| name | typescript-idioms |
| description | TypeScript strict mode, type narrowing, Zod validation, vitest, ESLint flat config. |
| paths | ["**/*.ts","**/*.tsx","**/tsconfig.json"] |
TypeScript's type system is your documentation, your test, and your specification — all at once. Make the type system encode the invariants of your domain so that invalid states are unrepresentable. Lean into the compiler.
Scope: This file covers TypeScript-specific type system and language idioms. For framework-specific patterns, see the respective idiom skill (Vue, React, Angular, Next.js, Hono). For file layout, see
references/project-structure.md. For detailed safety, SAST patterns, and performance patterns, seereferences/ts-patterns-and-anti-patterns.md. For quality commands, see@.agents/rules/code-idioms-and-conventions.md. For logging library, see@.agents/skills/logging-implementation/SKILL.md.
Loading guards:
- Plain JavaScript (no
tsconfig.json): load@.agents/skills/javascript-idioms/SKILL.mdinstead — this skill assumes strict-mode TS.- Hono backend:
honohas no repo-level marker file, so it is not auto-detected. Ifhonoappears inpackage.jsondependencies, co-load@.agents/skills/hono-idioms/SKILL.mdalongside this skill.- Test-file naming diverges by framework: see
references/project-structure.md§ Test Organization for the reconciliation rule before creating test files.
Load these before writing code in the matching context — not after.
| Situation | Reference to Load |
|---|---|
| Starting a new project or setting up file layout | references/project-structure.md |
| Choosing packages, tsconfig template, or vitest config | references/recommended-dependencies.md |
| Writing code that handles user input, async operations, or I/O | references/ts-patterns-and-anti-patterns.md |
| Defining Zod schemas or validating API/env boundaries | references/zod-patterns.md |
tsx for development execution (replaces ts-node)All TypeScript projects MUST have strict mode enabled. If a project does not have these settings, fix tsconfig.json before proceeding.
{
"compilerOptions": {
"strict": true,
"noImplicitAny": true,
"strictNullChecks": true,
"strictFunctionTypes": true,
"strictBindCallApply": true,
"strictPropertyInitialization": true,
"noImplicitThis": true,
"useUnknownInCatchVariables": true,
"alwaysStrict": true,
"noUnusedLocals": true,
"noUnusedParameters": true,
Use unknown instead of any
// ❌ any disables type checking
function processPayload(payload: any) {
console.log(payload.id); // No error if id doesn't exist
}
// ✅ unknown forces narrowing
function processPayload(payload: unknown) {
if (typeof payload === 'object' && payload !== null && 'id' in payload) {
console.log(payload.id);
}
}
Discriminated Unions for state machines
// ❌ Optional properties lead to impossible states
type State = {
status: 'loading' | 'success' | 'error';
data?: string;
error?: Error;
};
// ✅ Discriminated union makes impossible states unrepresentable
type State =
| { status: 'loading' }
| { status: 'success'; data: string }
| { : ; : };
strictNullChecks (always).?.) over explicit checks
// ❌ Verbose
const city = user && user.address && user.address.city;
// ✅ Concise
const city = user?.address?.city;
??) over Logical OR (||)
// ❌ Fails on 0 or ''
const count = input.count || 10;
// ✅ Only falls back on null/undefined
const count = input.count ?? 10;
using declarations — TypeScript 5.2+)
Requires lib: ["es2022"] or higher in tsconfig.json. Available in all Node.js 24 LTS projects.
// ✅ Automatic cleanup — resource disposed when scope exits
{
using file = await openFile('data.csv');
// file is automatically closed when block exits, even on throw
}
Always throw Error instances, never primitives
// ❌ Loses stack trace, breaks instanceof checks
throw 'Something went wrong';
throw { message: 'fail' };
// ✅ Proper error with stack trace
throw new Error('Something went wrong');
Custom error classes for domain errors
export class NotFoundError extends Error {
constructor(public readonly resource: string, public readonly id: string) {
super(`${resource} not found: ${id}`);
this.name = 'NotFoundError';
}
}
Type-safe error narrowing
try {
await api.createUser(data);
} catch (err) {
// ❌ unsafe — err is unknown
console.log(err.message);
// ✅ narrowed
if (err instanceof NotFoundError) {
.(err., err.);
} (err ) {
.(err.);
}
}
Always use async/await over raw Promises.
Use Promise.all for parallel operations.
// ❌ Sequential
const users = await getUsers();
const posts = await getPosts();
// ✅ Parallel
const [users, posts] = await Promise.all([getUsers(), getPosts()]);
Use Promise.allSettled when some can fail.
Avoid .then().catch() chains.
Abort long-running operations with AbortController
const controller = new AbortController();
const response = await fetch(url, { signal: controller.signal });
// Cancel if needed
controller.abort();
Never use async callbacks in Array.forEach
// ❌ forEach ignores returned promises — operations run detached
items.forEach(async (item) => {
await process(item);
});
// ✅ Use for...of for sequential
for (const item of items) {
await process(item);
}
.(items.( (item)));
TypeScript types do not exist at runtime. Any data crossing an I/O boundary must be validated.
import { z } from 'zod';
const UserSchema = z.object({
id: z.string().uuid(),
name: z.string().min(2),
age: z.number().int().nonnegative(),
});
type User = z.infer<typeof UserSchema>;
function parseUser(data: unknown): User {
return UserSchema.parse(data);
}
For advanced Zod patterns (transforms, discriminated unions, branded types, error formatting), see
references/zod-patterns.md.
Use Map/Set over plain objects for dynamic keys
// ❌ Plain objects as maps — prototype pollution risk, string-only keys
const cache: Record<string, User> = {};
// ✅ Map — any key type, no prototype chain, O(1) has/get/set
const cache = new Map<string, User>();
Use structuredClone() for deep copies (not JSON round-trip)
// ❌ Lossy — drops undefined, functions, Date objects, BigInt
const copy = JSON.parse(JSON.stringify(original));
// ✅ Handles circular refs, Date, RegExp, Map, Set, ArrayBuffer
const copy = structuredClone(original);
Prefer immutable array methods (ES2023+)
// ❌ Mutates original array
const sorted = arr.sort((a, b) => a - b);
const reversed = arr.reverse();
// ✅ Returns new array, original unchanged
const sorted = arr.toSorted((a, b) => a - b);
const reversed = arr.toReversed();
const withReplacement = arr.with(, );
Never use raw fetch spread throughout the codebase. Centralize it to handle tokens, retries, and errors.
Testability requirement: Always define an interface first. The concrete class implements it. This lets tests inject a
FakeHttpClientwithout network calls.
// ✅ Define interface first — enables test doubles (architectural rule: I/O isolation)
export interface HttpClient {
get<T>(path: string, schema: z.ZodType<T>): Promise<T>;
post<T>(path: string, body: unknown, schema: z.ZodType<T>): Promise<T>;
delete(path: string): Promise<void>;
}
// ✅ Production implementation
export class ApiClient implements HttpClient {
constructor(private readonly baseUrl: string) {}
async get<T>(path: string, schema: z.ZodType<T>): Promise<T> {
const res = await fetch(`${this.baseUrl}${path}`, {
signal: AbortSignal.timeout(10_000),
});
if (!res.ok) throw (res., res.);
: = res.();
schema.(data);
}
post<T>(: , : , : z.<T>): <T> {
res = (, {
: ,
: { : },
: .(body),
: .(),
});
(!res.) (res., res.);
: = res.();
schema.(data);
}
(: ): <> {
res = (, {
: ,
: .(),
});
(!res.) (res., res.);
}
}
{
: { : ; : }[] = [];
responses = <, >();
(: , : ): {
..(path, response);
}
get<T>(: , : z.<T>): <T> {
..({ : , path });
schema.(..(path));
}
post<T>(: , : , : z.<T>): <T> {
..({ : , path });
schema.(..(path));
}
(: ): <> {
..({ : , path });
}
}
Parameter object over positional arguments
// ❌ Positional — error-prone, order-sensitive, boolean traps
function createUser(name: string, email: string, isAdmin: boolean, isActive: boolean) {}
// ✅ Named parameters — self-documenting, order-independent
interface CreateUserParams {
name: string;
email: string;
isAdmin: boolean;
isActive: boolean;
}
function createUser(params: CreateUserParams) {}
Branded/Opaque types for domain primitives — see Type System Idioms §6. Never pass bare string or number for domain IDs.
Discriminated unions over inheritance — prefer union types with a type or kind discriminant over class hierarchies. Invalid states become compile errors.
Parse, don't validate — convert raw input into typed, validated domain objects at the boundary. Downstream code works with the typed form and never re-validates.
// ❌ Validate at every call site
function processOrder(orderId: string) {
if (!isValidUuid(orderId)) ();
}
orderId = .(rawInput);
(orderId);
export default. Use named exports for better refactoring and intellisense.index.ts) sparingly. They can cause circular dependencies.// ✅ Ensures type is erased at runtime
import type { User } from './types';
import { parseUser } from './parser';
NEVER suppress these rules — they signal structural problems that must be fixed:
| Rule | What It Signals | What To Do Instead |
|---|---|---|
@typescript-eslint/no-explicit-any | Type safety disabled | Use unknown and narrow |
@typescript-eslint/no-floating-promises | Unhandled async operation | Add await or void |
@typescript-eslint/no-unsafe-assignment | Unsafe type flow | Type the source properly |
@typescript-eslint/no-unnecessary-condition | Dead code or logic bug | Remove the condition |
complexity | Function too complex | Decompose into smaller functions |
Acceptable suppressions (with mandatory // SUPPRESS: comment):
| Rule | When Acceptable |
|---|---|
@typescript-eslint/no-non-null-assertion | After runtime validation proves non-null |
@typescript-eslint/ban-ts-comment | @ts-expect-error with explanation (never @ts-ignore) |
no-console | In CLI tools or development scripts |
Rule of thumb: If you're about to write // eslint-disable, stop and ask: "Am I suppressing a real design problem?" If yes, fix the design.
Use Vitest over Jest. It's faster, ESM-native, and requires zero config for TS.
AAA Pattern (Arrange, Act, Assert).
Test behavior, not implementation.
Test async errors by type, not message
Use vi.spyOn for interaction verification
Use satisfies for type-checked test fixtures
const mockUser = { id: '1', name: 'Test' } satisfies Partial<User>;
Test coverage is non-negotiable for new code:
if/else, switch arm, error path) MUST be exercised@vitest/coverage-v8 to verify coverage locally before committing# Quick coverage check
vitest run --coverage
# Coverage with thresholds
vitest run --coverage --coverage.thresholds.lines=80
Test double selection — choose the right tool:
| Approach | When to Use |
|---|---|
| Hand-written fake (implement interface) | Simple interface, few methods, need stateful behavior |
vi.fn() / vi.spyOn() | Verify call counts, argument matching |
msw (Mock Service Worker) | HTTP boundary mocking — intercepts at network level |
Parameterized it.each / test.each |
tsc --noEmitis the TypeScript equivalent of Rust'scargo check— type-checks without producing output. It is the fastest possible feedback during TDD cycles.
| Phase | Command | Purpose |
|---|---|---|
| TDD / rapid iteration | tsc --noEmit | Type-check only, no emit — fastest loop |
| Pre-commit | eslint . | Static analysis — must pass with zero warnings |
| Pre-commit | prettier --write . | Formatting — non-negotiable, always run |
| Pre-commit | vitest run | Unit tests — must all pass |
| Coverage verification | vitest run --coverage | Verify before merging |
| Unused dep audit | knip | Run before releases |
Rules:
tsc build during TDD cycles — tsc --noEmit is sufficient and significantly faster.eslint . must pass with zero warnings before any commit. Warnings are treated as errors.prettier --write . is non-negotiable — all code must be formatted before committing.knip reports unused exports or dependencies, remove them before the release.Document all exported items:
@param + @returns + @throws// ❌ Undocumented exported function
export function parseUserToken(token: string): UserId { ... }
// ✅ Documented
/**
* Parses and validates a signed user token, returning the extracted UserId.
*
* @param token - JWT signed token from the Authorization header
* @returns Validated UserId branded type
* @throws {InvalidTokenError} if the token is expired, malformed, or signature invalid
*/
export function parseUserToken(token: string): UserId { ... }
npm audit or pnpm audit in CIpackage.json with ^ for libraries (^3.0.0)package-lock.json or pnpm-lock.yaml) — for both apps and librariesknip before releasescrypto.randomUUID() over uuid packagestructuredClone() over lodash deep cloneArray.toSorted() over lodash sortObject.groupBy() over lodash groupByFor the full curated dependency list with versions, see
references/recommended-dependencies.md.
Never scatter process.env calls throughout the codebase
// ❌ Scattered, typo-prone, no validation
const port = process.env.PORT || '3000';
const dbUrl = process.env.DATABASE_URL;
// ✅ Centralized, validated at startup
import { z } from 'zod';
const EnvSchema = z.object({
PORT: z.coerce.number().default(3000),
DATABASE_URL: z.string().url(),
NODE_ENV: z.enum(['development', 'production', 'test']).default('development'),
});
export const env = EnvSchema.parse(process.env);
Fail fast on missing required config at boot, not at first use
For type coercion traps, prototype pollution, scope bugs, security vulnerabilities, collection pitfalls, and performance invariants, see
references/ts-patterns-and-anti-patterns.md. Load it before writing any code handling user input, async operations, or I/O.For performance patterns, see
perf-optimizationskill.
Type narrowing with in and typeof
function handle(val: string | number) {
if (typeof val === 'string') {
// val is string
}
}
Satisfies operator for checking without widening
type Colors = 'red' | 'green' | 'blue';
type RGB = [number, number, number];
// ❌ Record widens the specific keys
const palette1: Record<Colors, RGB> = { red: [255, 0, 0], green: [0, 255, 0], blue: [0, 0, 255] };
// ✅ satisfies keeps exact type
const palette2 = { red: [255, 0, 0], green: [0, 255, 0], blue: [0, 0, 255] } satisfies Record<Colors, RGB>;
readonly everywhere
// ❌ Mutable arrays
function sum(numbers: number[]): number { ... }
// ✅ Readonly arrays
function sum(numbers: readonly number[]): number { ... }
Opaque/Nominal typing for domain primitives
type UserId = string & { readonly __brand: 'UserId' };
type OrderId = string & { readonly __brand: 'OrderId' };
function getUser(id: UserId): User { ... }
// ❌ Compile error — OrderId is not assignable to UserId
getUser(orderId);
// ✅ Explicit creation
const userId = 'u-123' as UserId;
getUser(userId);
Template literal types for string patterns
type EventName = `on${Capitalize<string>}`;
type Route = `/${string}`;
NoInfer<T> to prevent unwanted inference (TS 5.4+)
function createFSM<S extends string>(initial: S, transitions: Record<S, NoInfer<S>[]>) { ... }
Exhaustive error handling with Result<T, E> discriminated union
Use this when you want to make errors part of the return type (no throw/catch required).
// Define once, reuse everywhere
type Ok<T> = { ok: true; value: T };
type Err<E> = { ok: false; error: E };
type Result<T, E> = Ok<T> | Err<E>;
// Helper constructors eliminate boilerplate
const ok = <T>(value: T): Ok<T> => ({ ok: true, value });
const err = <E>(error: E): Err<E> => ({ ok: false, error });
// Usage — no try/catch, caller is forced to handle the error case
function divide(a: number, b: number): Result<number, string> {
if (b === 0) return err('Division by zero');
return ok(a / b);
}
const result = divide(10, 0);
if (!result.ok) {
console.error(result.error); // 'Division by zero'
} else {
console.log(result.value); // number
}
Handle timeouts with AbortSignal.timeout()
const response = await fetch(url, {
signal: AbortSignal.timeout(5000),
});
Use Object.groupBy() for grouping (ES2024)
Use Set for O(1) lookups instead of Array.includes in loops
Early returns to reduce nesting — use guard clauses instead of nested if/else.
// ❌ Deep nesting
function handle(req: Request) {
if (req.auth) {
if (req.auth.isValid) {
if (req.body) {
return process(req.body);
}
}
}
}
// ✅ Guard clauses
function handle(req: Request) {
if (!req.auth) return unauthorized();
if (!req.auth.isValid) return forbidden();
if (!req.body) return badRequest();
return process(req.body);
}
Keep function complexity low (cyclomatic complexity < 10)
| Same logic, multiple input/output pairs |
Snapshot (expect().toMatchSnapshot()) | Large outputs — JSON responses, CLI output |
Prefer hand-written fakes for repository interfaces — they are simpler to debug and don't couple tests to implementation details. Use
vi.fn()when you genuinely need interaction verification.