thoughtspot / mcp-server / 24856089817

90%

push

github

Add pre-commit lint hook (#120)

* Add pre-commit lint hook

## Summary

Add a repo-managed pre-commit hook that runs Biome lint checks on staged files
before commits.

## What Changed

- add a `lint:staged` script that runs Biome against staged files only
- add a tracked `.githooks/pre-commit` template for local commits
- add a small installer and wire it into `prepare` so the hook is installed
  on `npm ci` and `npm install`
- avoid overwriting unrelated local pre-commit hooks by requiring the repo
  marker before reinstalling

## Validation

- `npm run hooks:install`
- `./.git/hooks/pre-commit`

* Fix hook installer edge cases

## Summary

Harden the git hook installer so it skips safely in CI and does not fail when
existing hooks cannot be inspected or git already points at the shared hooks
path.

## What Changed

- catch hook inspection failures and skip installation instead of breaking
  package setup
- skip the installer when `CI` is set
- normalize source and target hook paths and avoid copying the hook onto itself

## Validation

- `npm run hooks:install`
- `CI=1 node ./scripts/install-git-hooks.mjs`

* Run staged biome check in pre-commit

## Summary

Update the shared pre-commit hook to run Biome check on staged files so
formatting, import organization, and lint-safe fixes happen before commit.

## What Changed

- add a `check:staged` npm script using `biome check --write --staged`
- update the tracked pre-commit hook to run staged Biome checks instead of
  lint-only
- re-stage files after Biome writes changes so the commit includes the updated
  content

## Validation

- `npm run hooks:install`
- `./.git/hooks/pre-commit`

299 of 369 branches covered (81.03%)

Branch coverage included in aggregate %.

919 of 1044 relevant lines covered (88.03%)

228.3 hits per line