cdkd: Skill for Claude Code

PR Review Recommendation

Decide how much review rigor a PR warrants and surface the dispatch prompts. The tiers say what a PR needs AT MINIMUM.

The recommended tier is a FLOOR, not a cap, and wall-clock / token cost is never a reason to come in under it or to stop at it (CLAUDE.md → "Cost is not a tiebreaker"): when unsure which tier applies, take the higher one. Reviewers are read-only agents that run in parallel.

The skill itself never spawns reviewers — it reads PR stats, applies the heuristic, and prints a recommendation; the main session orchestrator issues the Agent calls.

Steps

1. A PUSH is not a round-completion signal — a REPLY is. Check for one before opening a round on a head you have not reviewed.

   Decide it from WHOSE TURN IT IS, never from a push timestamp. A round is a conversation turn: your comment ends yours, the author's ends theirs. Two push-timestamp spellings were tried and both skip the wait SILENTLY — a committed timeline event carries the COMMIT date, not the push time (and that is the ordinary non-force-push shape, so the error is the common case), and an empty $PUSH makes jq's .created_at > "" true for every comment ever written, which reads as "they replied". Neither errors.

   PR=<N>; REPO=go-to-k/cdkd
   ME=$(gh api user -q .login)
   THEM=$(gh pr view "$PR" --repo "$REPO" --json author -q .author.login)
   # All three surfaces: issue comments, review BODIES, and review-thread
   # replies. A contributor answering from "Files changed" -- GitHub's default
   # -- writes only the last two, and `issues/<N>/comments` never shows them.
   said() { # said <login> -> newest ISO timestamp, or empty
     { gh api "repos/$REPO/issues/$PR/comments" --paginate -q ".[]|select(.user.login==\"$1\")|.created_at"
       gh api "repos/$REPO/pulls/$PR/comments"  --paginate -q ".[]|select(.user.login==\"$1\")|.created_at"
       gh api "repos/$REPO/pulls/$PR/reviews"   --paginate -q ".[]|select(.user.login==\"$1\" and .submitted_at!=null)|.submitted_at"
     } | sort | tail -1
   }
   MINE=$(said "$ME"); THEIRS=$(said "$THEM")