Add agent skills (humanify, ace-it) + git guidance; repair skill frontmatter#713
Draft
kmontemayor2-sc wants to merge 4 commits into
Draft
Add agent skills (humanify, ace-it) + git guidance; repair skill frontmatter#713kmontemayor2-sc wants to merge 4 commits into
kmontemayor2-sc wants to merge 4 commits into
Conversation
Add a `humanify` skill under .claude/skills that guides agents to write code comments, docstrings, and PR/commit descriptions for a hurried teammate reading the diff — comment the why at a different level than the code, write the standing state rather than the change, and trim prose without collapsing scannable structure. Extend .claude/development.md with two git rules: never amend an already-pushed commit (add a new commit instead), and propagate fixes up a stacked branch chain with `git merge` only after asking, never by rebasing or force-pushing. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Stop mdformat from corrupting SKILL.md YAML frontmatter by excluding .claude/skills from the MD_FILES set, then reconstruct proper ----delimited frontmatter (name + description [+ argument-hint]) for every skill; mdformat had folded these fields into a heading and escaped the argument-hint brackets. Refresh the humanify skill to the trimmed revision and add an "Enumerations become lists" section. Add a new ace-it skill covering how to write fewer, real-behavior tests and avoid mock-plumbing assertions. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
kmontemayor2-sc
changed the title
Replace the inline find excludes with an explicit MD_EXCLUDE_DIRS list so the set of unformatted directories is easy to read and extend. Adds experimental sub-projects, plan docs, build/dist output, pytest cache, and the .sdd/.superpowers agent dirs to the caches/venv/tools/skills already skipped. Real docs under .claude (development.md, formatting.md) stay formatted. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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.
What
Adds and repairs agent guidance under
.claude/:.claude/development.md— two additions to the## Gitsection: neveramend an already-pushed commit (add a new commit instead), and propagate fixes
up a stacked branch chain with
git merge(ask first), never by rebasing orforce-pushing.
.claude/skills/humanify/SKILL.md(new) — a skill for writing codecomments, docstrings, and PR/commit descriptions: comment the why, write the
standing state rather than the change, turn comma-runs into lists, and trim
prose without collapsing scannable structure.
.claude/skills/ace-it/SKILL.md(new) — a skill for writing fewer,real-behavior tests: assert on real outputs instead of
mock.call_args, giveglue code e2e coverage rather than mock-plumbing unit tests, and match test
count to risk.
mdformatwas corrupting the YAML frontmatter ofevery
SKILL.md(foldingname/description/argument-hintinto a headingand escaping brackets). The
Makefilenow excludes.claude/skillsfrom themarkdown file set, and the frontmatter of all skills (
codex-verify,pragmatic,watch-action,humanify,ace-it) is reconstructed as proper----delimited YAML. Real docs under.claude/(development.md,formatting.md) are still formatted.Why
Codifies recurring guidance so agents write comments and tests that survive
refactors, and stops
mdformatfrom silently breaking skill frontmatter (whichthe skill loader needs to parse).