Repository navigation
chore(git-hooks): run the guardrails git hooks from the claude-code-plugins lefthook remote - #108
Conversation
… jobs Commit the guardrails plugin's commit-msg-convention, pre-commit-content and reference-verify entries as lefthook jobs over plugin-owned copies in .githooks/guardrails/ (guardrails 0.55.4), and the subject_pattern they enforce in .claude/source-control.md, so every clone gets them from lefthook install instead of per-machine hook settings. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
claude-review reviewed No findings. I reviewed the single changed file,
|
|
claude-security-review reviewed Security review: PR #108 (head 04e624b)The PR diff is one new file, critical-findings: 0 Findings (1)
No manifest fields are touched, so there are no |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 40581c4daa
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
…lugins lefthook remote Replace the vendored .githooks/guardrails/ copies, their lefthook jobs and the lint exclusions they needed with one lefthook remotes: entry pointing at melodic-software/claude-code-plugins (ref main, refetched every 24h). The commit subject rule comes from docs/conventions/source-control/commit-convention.yml, so .claude/source-control.md keeps no subject_pattern. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
No related issue: rolls out the guardrails lefthook remote (claude-code-plugins#9504)
Summary
Runs the guardrails plugin's three git-hook entries in every clone of this repository from one committed lefthook
remotes:entry, so each machine gets the same hooks fromlefthook installand picks up upstream fixes automatically:commit-msg-convention(commit-msg): the subject against the rule indocs/conventions/source-control/commit-convention.yml.pre-commit-content(pre-commit): secrets and machine-specific paths on added lines.reference-verify(pre-push): CLI flags,/plugin:skillreferences and deleted paths on pushed lines.Fix
lefthook.yml(new; the repository had no hook manager): oneremotes:entry,git_url: https://github.com/melodic-software/claude-code-plugins,ref: main,refetch_frequency: 24h, configplugins/guardrails/git-hooks/lefthook-remote.yml.main. The earlier revision of this PR vendored copies under.githooks/guardrails/and added a.claude/source-control.mdcarryingsubject_pattern; both are gone. The subject rule now comes from the standards-syncedcommit-convention.yml.Verification
install.mjs verify --repo <worktree>(rc 0):install.mjs plan --scope remoteafterwards: no decisions left.lefthook install:sync hooks: ✔️(pre-push, commit-msg, pre-commit). This PR's commit ran the remotepre-commit-content(0.04 s) andcommit-msg-convention(0.04 s), and its push ranreference-verify.Observation runs from the earlier revision, still valid because the remote runs the same hook scripts:
reference-verifyorigin/main~100..origin/main(102 commits in all), 20 changed files, 5 of them.md/.sh, 0.5 spre-commit-contentcommit-msg-conventionInitial commitRelated
🤖 Generated with Claude Code