Skip to content

Accept external hooks that satisfy kit config - #548

Merged
sandstream merged 1 commit into
mainfrom
fix/external-hook-policy
Aug 29, 2026
Merged

Accept external hooks that satisfy kit config#548
sandstream merged 1 commit into
mainfrom
fix/external-hook-policy

Conversation

@sandstream

Copy link
Copy Markdown
Owner

Summary

  • accept externally managed hooks as up-to-date when they contain all configured hook commands
  • make kit fix report missing commands instead of overwriting non-kit hooks
  • align this repo's .githooks with the configured pre-commit/pre-push policy

Verification

  • npm run build
  • git diff --check
  • node dist/cli.js adr check
  • KIT_NON_INTERACTIVE=1 KIT_BUMBLEBEE=0 KIT_NO_FAILURE_SIM=1 KIT_NO_UPDATE_CHECK=1 KIT_AUDIT_ANCHOR=0 node --test --test-timeout=180000 --test-concurrency=2 dist/check-hooks.test.js dist/hooks.test.js dist/cli.test.js
  • node dist/cli.js check
  • node dist/cli.js review

Notes

  • global kit may still report these hooks as not managed until this branch is installed or released; local node dist/cli.js check passes
  • full npm test remains excluded from pre-commit until the known suite timeouts are fixed

@github-actions

Copy link
Copy Markdown

✅ Docker image built successfully

  • CLI image: docker.io/sandstream/kit:test

@sandstream
sandstream force-pushed the fix/external-hook-policy branch from 0e914f5 to 3e1cfc5 Compare August 27, 2026 10:03
@github-actions

Copy link
Copy Markdown

✅ Docker image built successfully

  • CLI image: docker.io/sandstream/kit:test

@sandstream
sandstream force-pushed the fix/external-hook-policy branch from 3e1cfc5 to 485ab56 Compare August 29, 2026 22:41
@github-actions

Copy link
Copy Markdown

✅ Docker image built successfully

  • CLI image: docker.io/sandstream/kit:test

@sandstream
sandstream merged commit 6b3c98e into main Aug 29, 2026
19 checks passed
@sandstream
sandstream deleted the fix/external-hook-policy branch August 29, 2026 23:00
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.

1 participant