Skip to content

feat: #13 - Exhaustive PR comment resolution with mandatory replies#14

Merged
iamladi merged 1 commit intomainfrom
feat/exhaustive-pr-comment-resolution
Feb 9, 2026
Merged

feat: #13 - Exhaustive PR comment resolution with mandatory replies#14
iamladi merged 1 commit intomainfrom
feat/exhaustive-pr-comment-resolution

Conversation

@iamladi
Copy link
Owner

@iamladi iamladi commented Feb 9, 2026

Summary

Rewrites /address-pr-comments to guarantee every PR comment receives a GitHub reply — no feedback silently skipped.

  • Mandatory replies: Every non-author, non-bot comment gets a reply (fix with SHA, explanation, acknowledgment, clarification request, or deferral)
  • Auto-commit per fix: Each code fix gets its own commit referencing reviewer and comment ID
  • 3 comment types: Adds review summary fetching alongside review comments and issue comments, all with --paginate
  • Safe reply posting: All reply bodies piped via jq --arg + --input - to prevent shell injection
  • Idempotent re-runs: Detects existing replies before posting to prevent duplicates
  • Line-drift safety: Uses diff_hunk anchor text instead of API line numbers
  • Preflight checks: gh auth, clean working tree, git identity — with reasoning for each
  • Constitution compliance: Scoring system uses judgment dimensions instead of point tables; reply templates replaced with quality criteria; fix workflow framed as principles

Plan

See plans/exhaustive-pr-comment-resolution.md — reviewed by GPT-5.2-Codex and Gemini 3 Pro (16 blindspot findings addressed).

Constitution compliance review: reviews/constitution-compliance-address-pr-comments.md (4.4/10 → transformed)

Closes #13

Test plan

  • Run bun run validate — plugin validation passes
  • Run /address-pr-comments on a PR with mixed comment types (code fix, question, praise, unclear suggestion)
  • Verify all comments receive GitHub replies
  • Verify each fix has its own commit with descriptive message
  • Run command twice on same PR — verify no duplicate replies (idempotency)
  • Test with dirty working tree — verify preflight abort
  • Test autonomous mode (--auto) — verify below-threshold comments get replies

…g-based design

Rewrites the command to guarantee every PR comment receives a GitHub
reply (fix with SHA, explanation, acknowledgment, or deferral). Applies
constitution compliance transformations to replace rigid point tables,
hardcoded reply strings, and fill-in-the-blank templates with judgment
dimensions, quality criteria, and principled guidance.

Closes #13
@iamladi iamladi merged commit 468780a into main Feb 9, 2026
1 check passed
@iamladi iamladi deleted the feat/exhaustive-pr-comment-resolution branch February 9, 2026 19:01
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.

Exhaustive PR comment resolution with mandatory replies

1 participant