Add ADR skill for architecture decision records - #29
Merged
Conversation
Coding rules (rules/) auto-activate per file type via Claude Code native globs: frontmatter. Each complements existing hooks — rules guide how to write, hooks catch what you missed. Covers Go, TypeScript, Python, Rust, and Shell. ADR skill (skills/adr/) generates Architecture Decision Records in docs/adr/ for capturing the why behind technical choices.
Plugins don't support a rules/ directory — only commands/, agents/, skills/, hooks/. Rules need to live in ~/.claude/rules/ with paths: frontmatter (not globs:) per Claude Code docs. Moved there directly since this is personal tooling, not distributed.
Address review findings: ensure docs/adr/ exists before listing, and clarify how to update old ADR status when superseding.
Address tri-review consensus finding — ls | tail is fragile for determining next ADR number. Now extracts numeric prefixes and sorts numerically.
The rule said "never modify" then immediately said "update the status line" — contradiction. Now explicitly scopes the no-modify rule to substantive content only. Also clarifies the empty-output signal for the numbering command.
5uck1ess
changed the title
CI validate-counts requires README and ROADMAP to reflect actual skill count (17 → 18 after adding ADR skill).
The bold header line also needed updating, not just the table row.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
skills/adr/): Context-activated skill for generating Architecture Decision Records indocs/adr/. Captures the why behind technical choices so agents and humans don't reverse intentional decisions.grep -oE + sort -nNote: This PR was part of a larger effort that also installed language-specific coding rules at
~/.claude/rules/(Go, TypeScript, Python, Rust, Shell). Those are user-level config outside the repo, not part of this merge.Test plan
docs/adr/0001-*.md0002