david/pi-loop
Watch
1
0
Fork
You've already forked pi-loop
 0
Code Issues 20 Pull requests 1 Projects Releases Packages Wiki Activity Actions

documentation: add ADR-021 no-code-change comment and implementation plan #312

Merged
david merged 1 commit from documentation/2026-09-06/adr-021-no-code-change-comment into main 2026-09-05 18:48:30 +00:00
Conversation 0 Commits 1 Files changed 2 +531
david commented 2026-09-05 18:48:12 +00:00
Owner
Copy link

Commits two new docs that record the design for commenting on the source issue when the implement agent declares no code change needed:

  • docs/adr/021-no-code-change-comment.md — ADR-021: when the implement-agent no-code-change.json short-circuit fires, pi-loop posts one deterministic comment to the source issue (never auto-closes; issue close stays a human action). Covers the comment-only rule, deterministic body composition, all-four-provider unconditional posting, fatal-on-failure (attributed to the write-back stage), the at-most-once no-code-change-comment.json marker artifact, resume mapping, and surfacing.
  • docs/implementation-plan-no-code-change-comment.md — step-by-step plan mapping to ADR-021 §1–§9 (comment-body builder, marker artifact, post-and-mark service, pipeline wiring, summary/CLI line, resume mapping, unit seams, gated resume E2E, doc updates).

Why now: the design is settled and these docs capture the decision before implementation starts. No code changes — documentation only.

Commits two new docs that record the design for commenting on the source issue when the implement agent declares no code change needed: - **`docs/adr/021-no-code-change-comment.md`** — ADR-021: when the implement-agent `no-code-change.json` short-circuit fires, pi-loop posts one deterministic comment to the source issue (never auto-closes; issue close stays a human action). Covers the comment-only rule, deterministic body composition, all-four-provider unconditional posting, fatal-on-failure (attributed to the write-back stage), the at-most-once `no-code-change-comment.json` marker artifact, resume mapping, and surfacing. - **`docs/implementation-plan-no-code-change-comment.md`** — step-by-step plan mapping to ADR-021 §1–§9 (comment-body builder, marker artifact, post-and-mark service, pipeline wiring, summary/CLI line, resume mapping, unit seams, gated resume E2E, doc updates). Why now: the design is settled and these docs capture the decision before implementation starts. No code changes — documentation only.
david merged commit 3940e369a2 into main 2026-09-05 18:48:30 +00:00
david deleted branch documentation/2026-09-06/adr-021-no-code-change-comment 2026-09-05 18:48:30 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
batch

   
bug

   
chore
Maintenance tasks, cleanup, housekeeping

  
documentation

   
enhancement

   
pi-loop

   
ready

   
resume
No labels
batch
bug
chore
documentation
enhancement
pi-loop
ready
resume
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
david/pi-loop!312
No description provided.