Skip to content

fix(hooks): point post-git-ops.sh fallback at dist/cli.js, not src/cli.js (#1979) - #2192

Merged
carlos-alm merged 1 commit into
mainfrom
fix/issue-1979
Jul 30, 2026
Merged

fix(hooks): point post-git-ops.sh fallback at dist/cli.js, not src/cli.js (#1979)#2192
carlos-alm merged 1 commit into
mainfrom
fix/issue-1979

Conversation

@carlos-alm

Copy link
Copy Markdown
Contributor

Summary

  • `.claude/hooks/post-git-ops.sh` had the same broken fallback build path already fixed for `update-graph.sh` in update-graph.sh hook fallback build path references nonexistent src/cli.js #1836: `src/cli.js` has never existed in this repo (CLI entry point is TypeScript at `src/cli.ts`, compiled to `dist/cli.js`).
  • The `else` branch (hit when the `codegraph` binary isn't on PATH) silently failed (stderr to `/dev/null`), so `BUILD_OK` stayed `0` and the full rebuild after a rebase/merge/pull never happened for contributors without a global `codegraph` install.
  • Confirmed no other active hook/script has this same bug pattern remaining (swept all `src/cli.js` references repo-wide — the rest are historical dogfood reports, example CLI/MCP transcripts, and ROADMAP/BACKLOG history, not functional fallback paths).

Verification

Closes #1979

…i.js (#1979)

Same broken fallback build path already fixed for update-graph.sh in
#1836: src/cli.js has never existed in this repo (CLI entry point is
TypeScript at src/cli.ts, compiled to dist/cli.js). The else branch
(hit when the codegraph binary isn't on PATH) silently failed
(stderr to /dev/null), so BUILD_OK stayed 0 and the full rebuild after
a rebase/merge/pull never happened for contributors without a global
codegraph install.

Closes #1979

docs check acknowledged: hook-script-only fix, no README/CLAUDE.md/ROADMAP surface change
@greptile-apps

greptile-apps Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Corrects the post-git-operation hook’s local CLI fallback to invoke the compiled dist/cli.js entry point instead of the nonexistent src/cli.js.

Confidence Score: 5/5

The PR appears safe to merge, with the fallback now targeting the repository’s actual compiled CLI entry point.

The one-line change replaces a nonexistent JavaScript source path with dist/cli.js, the configured executable generated by the repository build, without altering hook control flow.

Important Files Changed

Filename Overview
.claude/hooks/post-git-ops.sh Updates the fallback to the canonical compiled CLI entry point, matching the package configuration and existing hook conventions.

Reviews (1): Last reviewed commit: "fix(hooks): point post-git-ops.sh fallba..." | Re-trigger Greptile

@carlos-alm
carlos-alm merged commit 6b80650 into main Jul 30, 2026
26 checks passed
@carlos-alm
carlos-alm deleted the fix/issue-1979 branch July 30, 2026 11:36
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

post-git-ops.sh hook fallback build path references nonexistent src/cli.js

1 participant