Skip to content

Add git-blame-historic-context skill#4373

Open
AryanGodara wants to merge 4 commits into
mainfrom
aryan/git-blame-skill
Open

Add git-blame-historic-context skill#4373
AryanGodara wants to merge 4 commits into
mainfrom
aryan/git-blame-skill

Conversation

@AryanGodara
Copy link
Copy Markdown
Member

@AryanGodara AryanGodara commented May 1, 2026

What this is

A standalone procedure for using git blame to recover historic context before flagging unusual-looking code. Often, certain decisions led to sub-optimal-looking code, and those decisions are codified in git history rather than the code itself. This skill formalises the lookup + decision rubric.

Ships with /blame-context <path>:<line>, a slash command that wraps the procedure end-to-end. Use either the slash command or the procedure manually.

Why split out

Extracted from #4351 because the procedure is reusable beyond PR review — order-debug sessions and ad-hoc code investigations both want the same lookup play. Splitting keeps each consumer's surface focused.

Files

  • docs/skills/git-blame-historic-context.md — the procedure + decision rubric.
  • .claude/commands/blame-context.md — slash command wrapping the procedure.

Reviewing this in isolation

The skill body is self-contained. The "Used by" footer links to ../COW_PR_REVIEW_SKILL.md, which lands with #4351; that link will 404 until that PR merges. Nothing else here depends on #4351.

How to try it

/blame-context crates/driver/src/domain/competition/solution/settlement.rs:444
/blame-context crates/observe/src/panic_hook.rs:14-15

Worked sample output for both is in the skill file's Examples section.

@AryanGodara AryanGodara changed the title Add git-blame-historic-context skill" Add git-blame-historic-context skill May 1, 2026
@AryanGodara AryanGodara force-pushed the aryan/git-blame-skill branch from 07f2227 to ff73770 Compare May 5, 2026 13:45
@AryanGodara AryanGodara marked this pull request as ready for review May 5, 2026 13:48
@AryanGodara AryanGodara requested a review from a team as a code owner May 5, 2026 13:48
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces a new Claude command and detailed documentation for the 'git blame historic context' skill, which assists developers in understanding the rationale behind unusual code by tracing it back to its originating PR. The feedback focuses on improving the automation of the gh pr view command by removing unnecessary repository flags that would require manual variable substitution, allowing the CLI to infer the context automatically.

Comment thread .claude/commands/blame-context.md Outdated
Comment thread docs/skills/git-blame-historic-context.md Outdated
@AryanGodara AryanGodara force-pushed the aryan/git-blame-skill branch from 9b89fe4 to 8cb5c1d Compare May 5, 2026 14:27
Comment thread docs/skills/git-blame-historic-context.md Outdated
Comment thread docs/skills/git-blame-historic-context.md Outdated
Comment thread docs/skills/git-blame-historic-context.md
Comment thread docs/skills/git-blame-historic-context.md Outdated
Comment thread docs/skills/git-blame-historic-context.md Outdated
@AryanGodara AryanGodara requested a review from jmg-duarte May 11, 2026 07:32
@AryanGodara AryanGodara force-pushed the aryan/git-blame-skill branch from 4cc1201 to 88c64b3 Compare May 11, 2026 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants