One entryMerged PR· Claudavel· PHP
PR #61 merged: feat: ship .claude assets — /commit CDR command + permissions allowlist
additions
+170
deletions
-1
files changed
9
Closes #43, first slice of #59.
The package named 'Laravel + Claude Code configured to work together' now actually ships Claude Code assets. claudavel:install publishes:
| Asset | Purpose |
|---|---|
.claude/commands/commit.md | The /commit command from #43 — writes a CDR to docs/context/ per non-trivial commit, then backfills the SHA |
.claude/settings.json | Conservative permission allowlist (pint, pest, artisan test, git reads) |
docs/context/.gitkeep | Home for the decision records |
Design decisions (deviations from the issue as written, flagged rather than silent):
- CDRs skip trivial commits — typo/dep-bump/formatting diffs commit without a record; the issue's helptobuy history implies every commit gets one, but on high-frequency repos that's noise. Easy to flip by editing the command.
--no-verifyon the SHA-backfill amend — hooks already ran on the real commit; re-running them on the amend doubles formatter/test time for nothing.- No AI-attribution footers, ever — baked into the command prompt.
- Published as a
.claude/commands/command (as specced in #43) rather than a skill —/commitinvokes either; can migrate to skills later without breaking anyone. - Gotcha fixed en route: the repo's
.gitignorehad.claude/unanchored, which silently ignored the new stubs — now root-anchored (/.claude/).
Tests: 3 new (stub content, settings JSON validity, installer wiring) + required-stubs list extended. 66 passed (258 assertions), pint clean.