Skip to content

chore: setup husky and commitlint similar to systemfsoftware - #3

Merged
ryanleecode merged 2 commits into
mainfrom
husky-setup
Aug 30, 2026
Merged

chore: setup husky and commitlint similar to systemfsoftware#3
ryanleecode merged 2 commits into
mainfrom
husky-setup

Conversation

@systemfsoftware-maker

Copy link
Copy Markdown
Collaborator

Summary

Commits are now validated on the client and vendored history is ignored. commit-msg runs commitlint (no AI co-authors, shape-aware type checks) via deno run npm:@commitlint/cli; pre-commit formats and lints staged non-vendored files before they enter the index.

Details

  • Husky hooks (commit-msg, pre-commit, prepare-commit-msg) skip repos/ and subtrees.toml (annotated grep ^(repos/.*|subtrees\.toml)$), skip merge during MERGE_HEAD, strip AI Co-authored-by trailers
  • commitlint (commitlint.config.ts) mirrors systemfsoftware rules: type-enum, no-ai-coauthors, type-matches-diff-shape (docs/test/CI/lockfile/tooling), scope-enum (repo/deps/release/ci/scripts/hooks), footer ignores for Squashed ' and git-subtree-dir:
  • Pure Deno — no package.json/pnpm-lock.yaml; tasks use deno run npm:* with nodeModulesDir: auto and deno install populates node_modules/.deno

Verification

  • deno task checkdprint check + deno lint + deno check scripts/*.ts passes
  • echo "chore: test" | deno run --allow-read --allow-run --allow-env --allow-sys npm:@commitlint/cli — passes; echo "bad" | ... — fails type-empty/subject-empty as expected

Configure husky commit-msg, pre-commit and prepare-commit-msg hooks
with vendored repos/ skip, commitlint with no-ai-coauthors and
type-matches-diff-shape rules via pure Deno (deno run npm:*), no
pnpm/package.json mixing
Mirrors systemfsoftware pre-push (check-remote-main) — refuses push
when HEAD does not contain origin/main
@ryanleecode
ryanleecode merged commit 1f78be8 into main Aug 30, 2026
1 check passed
@ryanleecode
ryanleecode deleted the husky-setup branch August 30, 2026 04:47
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.

2 participants