Skip to content

Add a pre-push check suite - #23

Merged
dgsienko merged 8 commits into
mainfrom
check-suite
Oct 23, 2026
Merged

dgsienko merged 8 commits into
mainfrom
check-suite

Conversation

@dgsienko

Copy link
Copy Markdown
Owner

Ported from the setup in enterprise-data-services. On push, one headless claude -p session (read-only tools) runs three repo-local skills on origin/main...<pushed sha>:

  • check-docs: docs the diff makes false, prose tells, comment density
  • check-scope: changes that don't trace to the commit messages, speculative code
  • check-tests: tests that can't fail, behavior changes with no test

A ⚠️ finding blocks the push. The suite skips itself when claude is missing, the diff is empty, the push only deletes, or there's no verdict. SKIP=check-suite git push bypasses it on purpose.

The hook is now tracked in .githooks/pre-push (ruff check, ruff format, pytest, then the suite). Enable it once per clone with git config core.hooksPath .githooks. There's no new dependency.

The last two commits are the linked-worktree test fixes also on #22.

@dgsienko
dgsienko merged commit 0644ff4 into main Oct 23, 2026
4 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.

1 participant