gh-credit-harvest

gh-credit-harvest

Harvest exactly one community PR into the real landing branch with full authorship and machine-readable credit, verified green, then thank the contributor. A PR is evidence: judge it from code, tests, comments, and checks, never the title. Do not merge, close, tag, or publish without Hunter's approval — this skill lands a credited commit and posts thanks; the workflow closes the PR.

When to use

• You have approval to land ONE specific community PR into a release branch.
• The PR is not yet on the landing branch (if it is, close-with-credit instead — see gh-close-issues).
• The landing branch may be local-only (e.g. <release-branch>); a main-based "mergeable" flag does not prove it lands cleanly.

Workflow

1. Find the real landing branch (the one Hunter named, not always main) and fetch the PR head:

   git switch <release-branch>
   git fetch origin pull/<N>/head
   git log -1 --format='%H %an <%ae>' FETCH_HEAD   # author to preserve
   

2. Review from evidence, not the title. Read the diff, tests, linked issue, comments, and CI:

   gh pr view <N> --repo Hmbown/CodeWhale --json title,author,files,statusCheckRollup
   gh pr diff <N> --repo Hmbown/CodeWhale
   

3. Test mergeability against the REAL landing branch (local-only branches lie via the main flag):

   git merge-tree $(git merge-base HEAD FETCH_HEAD) HEAD FETCH_HEAD   # empty/clean = no conflict
   

4. Land it, preferring cherry-pick — it preserves the original author automatically:

   git cherry-pick <sha>            # one or more commits from FETCH_HEAD
   

5. If it conflicts, spans noise, or needs squashing, re-apply the narrow slice and commit with explicit author + credit trailers. Resolve --author and the co-author from .github/AUTHOR_MAP (fall back to numeric noreply):

   gh api users/<handle> --jq '"\(.id)+\(.login)@users.noreply.github.com"'
   git commit --author="Name <[email protected]>" -m "fix(scope): what changed (#<N>)" \
     -m "Harvested from PR #<N> by @<handle>" \
     -m "Co-authored-by: Name <[email protected]>"
   

   The Harvested from PR #<N> by @<handle> line in the body is what .github/workflows/auto-close-harvested.yml matches to auto-close with credit once the commit reaches main.
6. Format and run the focused tests for the touched crate — only land green:

   cargo fmt --all
   cargo test -p <crate>            # the crate(s) the PR touched, not the whole workspace
   git log -1 --format='%an <%ae>%n%(trailers:key=Co-authored-by)'  # credit the human, from .github/AUTHOR_MAP
   

7. Post a brief, warm, specific thank-you on the PR — name what the change fixed, no drama. Leave the PR open; the workflow closes it with credit when the commit lands on main:

   gh pr comment <N> --repo Hmbown/CodeWhale \
     --body "Thank you @<handle> — clean fix for <the specific bug>. Harvested into the v0.8.61 lane with your authorship preserved; it'll auto-close with credit once it reaches main."