Skip to content

Set up pre-commit hooks with prek: Biome (lint + format) and TypeScript type check#53

Merged
sjquant merged 2 commits into
mainfrom
claude/setup-precommit-hooks-eN77q
May 14, 2026
Merged

Set up pre-commit hooks with prek: Biome (lint + format) and TypeScript type check#53
sjquant merged 2 commits into
mainfrom
claude/setup-precommit-hooks-eN77q

Conversation

@sjquant
Copy link
Copy Markdown
Owner

@sjquant sjquant commented May 13, 2026

  • Add @biomejs/biome and @j178/prek as dev dependencies
  • Initialize biome.json with recommended rules, tabs, double quotes
  • Add .pre-commit-config.yaml with local hooks for biome check and tsc --noEmit
  • Add lint, format, typecheck scripts to package.json
  • Apply Biome formatting across codebase
  • Fix TS2790 error in cli.test.ts (delete on optional property)
  • Add biome-ignore for intentional control-char regex in repo.ts

https://claude.ai/code/session_01XYJUro7Kh2ae1pwDTdN1fp

claude added 2 commits May 13, 2026 14:53
…pt type check

- Add @biomejs/biome and @j178/prek as dev dependencies
- Initialize biome.json with recommended rules, tabs, double quotes
- Add .pre-commit-config.yaml with local hooks for biome check and tsc --noEmit
- Add lint, format, typecheck scripts to package.json
- Apply Biome formatting across codebase
- Fix TS2790 error in cli.test.ts (delete on optional property)
- Add biome-ignore for intentional control-char regex in repo.ts

https://claude.ai/code/session_01XYJUro7Kh2ae1pwDTdN1fp
- typecheck hook: replace always_run with types_or=[ts,tsx] so it only
  fires when TypeScript files are staged
- biome.json: disable noNonNullAssertion in test files (valid pattern
  after .not.toBeNull() assertion)
- biome.json: override JSON files to 2-space indentation
- package.json: add prepare script to auto-install prek hook on pnpm install

https://claude.ai/code/session_01XYJUro7Kh2ae1pwDTdN1fp
@sjquant sjquant merged commit 6e5406c into main May 14, 2026
3 checks passed
@sjquant sjquant deleted the claude/setup-precommit-hooks-eN77q branch May 14, 2026 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants