Skip to content

docs: sync CONTRIBUTING conventions + recover pre-commit hooks#31

Merged
jal-co merged 2 commits into
mainfrom
docs/sync-contributing-conventions
May 31, 2026
Merged

docs: sync CONTRIBUTING conventions + recover pre-commit hooks#31
jal-co merged 2 commits into
mainfrom
docs/sync-contributing-conventions

Conversation

@jal-co

@jal-co jal-co commented May 31, 2026

Copy link
Copy Markdown
Owner

What

Two related contributor-doc / tooling changes:

  1. Recovers the pre-commit hooks (.pre-commit-config.yaml + the CONTRIBUTING "Git hooks" section) that were authored alongside ci: align commit-check + add license, pre-commit hooks #30 but landed after it was merged, so they never reached main.
  2. Syncs CONTRIBUTING.md with the updated global conventions — adds a Versioning section (Semantic Versioning 2.0.0) and clarifies breaking-change signalling.

Why

Recovery: #30 was squash-merged at a 2-commit snapshot; the 3rd commit (ci: add pre-commit hooks) was pushed after the merge and was silently dropped. main is missing .pre-commit-config.yaml and the "Git hooks" docs. This restores them.

Versioning sync: The global AGENTS.md now documents Conventional Branch + Conventional Commits 1.0.0 + SemVer 2.0.0. CONTRIBUTING already covered branches and commits but had no versioning guidance — even though the package publishes to npm with vX.Y.Z tags. This closes the gap.

Closes #

Type

  • docs — Documentation
  • ci — CI/CD changes

How

  • .pre-commit-config.yaml: pins commit-check v2.7.0, check-message (commit-msg) + check-branch (pre-push), sharing .github/cchk.toml with CI.
  • CONTRIBUTING Versioning section: the feat→MINOR / fix→PATCH / breaking→MAJOR mapping, 0.y.z initial-development note, tag/release rules. Verified the publish workflow triggers on GitHub Release (not raw tag push) and documented it accurately.
  • Breaking-change line now notes the BREAKING CHANGE: footer in addition to !.

Testing

  • commit-check --branch / --message — pass locally
  • npm run lint (tsc --noEmit) — passes
  • npm test — 62/62 pass
  • No application code changed.

Screenshots

N/A — contributor docs / CI config.

jal-co added 2 commits May 30, 2026 19:25
Adds a pre-commit config running the same commit-check validation as
CI (Conventional Commit messages, Conventional Branch names), so
contributors catch issues before pushing instead of on the PR. Hooks
share the .github/cchk.toml policy with CI. Documents the one-time
setup in CONTRIBUTING.

This change was authored alongside #30 but landed after that PR was
merged, so it never made it onto main.
Adds a Versioning section documenting Semantic Versioning 2.0.0 and the
feat->MINOR / fix->PATCH / breaking->MAJOR mapping, matching the new
global AGENTS.md conventions. Notes the 0.y.z initial-development
status, tag/release rules, and the automated publish-on-release flow.
Also clarifies that breaking changes may use the BREAKING CHANGE footer
in addition to the ! marker.
@vercel

vercel Bot commented May 31, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
scn-stack Ready Ready Preview, Comment May 31, 2026 2:26am

Request Review

@github-actions github-actions Bot added docs Documentation changes ci CI/CD and workflow changes labels May 31, 2026
@jal-co jal-co merged commit f1b82c3 into main May 31, 2026
5 checks passed
@jal-co jal-co deleted the docs/sync-contributing-conventions branch May 31, 2026 02:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci CI/CD and workflow changes docs Documentation changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant