Skip to content

fix(hook): support git-managed paths and local providers - #583

Merged
di-sukharev merged 1 commit into
masterfrom
codex/fix-git-hook-reliability
Aug 22, 2026
Merged

fix(hook): support git-managed paths and local providers#583
di-sukharev merged 1 commit into
masterfrom
codex/fix-git-hook-reliability

Conversation

@di-sukharev

Copy link
Copy Markdown
Owner

Summary

  • resolve the effective hooks directory through Git so hook setup works with relative core.hooksPath values, submodules, and linked worktrees
  • centralize provider configuration requirements and allow Ollama, MLX, and llama.cpp hooks to run without a fake API key
  • add subprocess E2E coverage for hook path variants and credential behavior, with Git configuration isolated from the host environment
  • rebuild the tracked CLI bundle so the fixes ship to npm consumers

Test plan

  • npm run lint
  • npm run test:unit -- --runInBand
  • OCO_TEST_SKIP_VERSION_CHECK=true npx jest test/e2e/cliBehavior.test.ts test/e2e/geminiBehavior.test.ts test/e2e/gitPush.test.ts test/e2e/oneFile.test.ts test/e2e/noChanges.test.ts --runInBand

Closes #224
Closes #438

@di-sukharev
di-sukharev merged commit a8649ba into master Aug 22, 2026
5 checks passed
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.

[Bug]: Hook needs env vars set even when using ollama [Bug]: opencommit hook set not working when git project is a submodule

1 participant