thoughtspot / mcp-server / 25719255237

90%

push

github

Fix Docker build after prepare hook addition (#153)

* Fix Docker build after prepare hook addition

## Summary

Prevent Docker image builds from failing after the repo-managed prepare hook was
added. The image installed dependencies before copying the hook installer
script, so npm ci invoked prepare and failed before the rest of the repo was
available.

## What Changed

- copy scripts/install-git-hooks.mjs into the image before npm ci
- keep the existing Docker layer ordering and cache behavior otherwise unchanged

## Semantics

- this only affects container builds; local npm install and npm ci behavior is
  unchanged
- the hook installer still exits cleanly in non-git environments once the script
  is present in the image

## Validation

- ran npm run prepare in a stripped temp directory containing package.json,
  tsconfig.json, and scripts/install-git-hooks.mjs
- confirmed the prepare step exits cleanly with the expected non-git checkout
  message

* Optimize Docker cache for prepare hook

Use a placeholder install-git-hooks.mjs during npm ci so the
 dependency layer stays keyed to package metadata instead of the
 local hook script.

Validation:
- docker build -t thoughtspot-mcp-server:fixdockerprepare-v2 .

718 of 824 branches covered (87.14%)

Branch coverage included in aggregate %.

1524 of 1661 relevant lines covered (91.75%)

940.43 hits per line