diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-22 14:13:37 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-22 14:13:37 -0500 |
| commit | 977b420257bdf1545b1c58d5a6a7202b5f4ec53f (patch) | |
| tree | 85834b4f30e4af7570c33d301c04813a6654f5ad /debug | |
| parent | e46174ae7ae443bbc94efcc5cc2059efa2cae5c0 (diff) | |
| download | rulesets-977b420257bdf1545b1c58d5a6a7202b5f4ec53f.tar.gz rulesets-977b420257bdf1545b1c58d5a6a7202b5f4ec53f.zip | |
docs(commands): tighten respond-to-review PR-feedback workflow
Two audit-pass fixes to respond-to-review, plus a stale-reference tidy in respond-to-cj-comments.
The Gather step fetched a flat comment list, which misses thread resolution and re-processes feedback that's already settled. It now pulls unresolved review threads via gh api graphql (skipping resolved ones), keeps REST only for top-level conversation comments, and resolves a thread only after the fix is verified.
The commit guidance suggested "fix: Address review — [description]", which puts the review process into git log against commits.md and used a non-ASCII dash. It now names the actual fix and leaves the how-it-surfaced detail in the PR thread.
respond-to-cj-comments lost a stale "humanizer" mention, since that skill is voice now. Two adjacent audit items came out moot: it no longer embeds an absolute path, and its humanizer/emacsclient fallback was overtaken by the /voice migration and the in-place VERIFY pattern.
Diffstat (limited to 'debug')
0 files changed, 0 insertions, 0 deletions