Add setup-pr agent and pr-description skill; enable formatting pre-commit hooks - #762
Merged
Merged
Conversation
…mmit hooks Agent sessions had no shared definition of what "ready for review" means, and pre-commit only ran whitespace/EOF fixes, so formatting drift reached CI. - `.claude/agents/setup-pr.md`: a blocking, tier-scaled checklist to run when a branch is ready for review, ending in a drafted PR title and description. Validation scales with the diff (docs-only / small / full) so small changes stay cheap. Cleanup is additive commits only; the agent never pushes or rewrites history. - `.claude/skills/pr-description/SKILL.md`: owns PR-description structure and evidence rules, importing humanify's prose rules by reference with the inversion that the PR description IS the change story. - humanify's trigger is narrowed to comments, docstrings, and reference docs so it no longer competes with pr-description on the same trigger. - pre-commit now runs ruff (check --fix, format) and dprint on the staged files, mirroring `make format_py` / `make format_md`. Scoped to pre-commit only: both rewrite files, which would leave a dirty tree mid-push. - dprint.json excludes `.claude/agents`, for the same reason `.claude/skills` is already excluded — the Markdown formatter mangles YAML frontmatter. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
yliu2-sc
approved these changes
Sep 1, 2026
kmontemayor2-sc
marked this pull request as ready for review
September 1, 2026 18:33
kmontemayor2-sc
requested review from
nshah-sc,
svij-sc,
xgao4-sc and
zfan3-sc
as code owners
September 1, 2026 18:33
github-merge-queue
Bot
removed this pull request from the merge queue due to failed status checks
Sep 1, 2026
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
3 participants
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.
Migrating some skills / etc over :)