#2770 gatorwalk-factory: keep a person's plan feedback and hand it to the next plan
Opened by skunk-ape · 9/30/2026· Shipped 9/30/2026
A person's feedback on a plan is issue-lifecycle's main loop, and gatorwalk drops it.
What issue-lifecycle does
iterate (extensions/models/issue_lifecycle.ts:1342-1490) takes feedback as text. It:
- writes each round to the versioned
feedbackresource (:470, :1411-1437); - stores the revised plan with
feedbackIncorporated, the cumulative list of every round (:1440-1450); - posts
plan_revisedwith the version, the round number and the feedback (:1470-1484).
The next plan is built from all of it.
What gatorwalk does
plan-review.reviseis a manualadvance, andadvancetakes no note (extensions/models/work_item.ts:269-289).- The only place feedback can go is a
declinenote onplan-approval. It stays in the run record andstatus(_lib/gates.ts:456-461), counts only for that stage and cycle, and is never projected: a decline is not an entry trigger (_lib/projection.ts:213-220). - The
planartifact has no field for the feedback it answers (lifecycles/swamp-extensions.yaml:257-281). planinjects onlyclassification(:255-256). Afterreworkorrevise, the planning agent is not handed the review findings, the earlier plan or the feedback, even though its prompt says to address the last review's findings (:253-254).
Why it matters
In the pilot, the first time a person says "change the plan like this", the text lives only in the chat. It does not reach the next planning pass, the Lab history, or the summary report.
Fix direction
These changes are YAML only:
planinjects[classification, plan, plan-review, plan-feedback].- A
plan-feedbackevidence ({feedback, round}) is declared onplan-review.revisegains anevidence-recordedgate on it this cycle, so a revision always carries its reason. - The
planschema gains an optionalfeedbackIncorporated: string[]. plan_revisedfills from the plan payload.
Decision for the implementer: use that evidence, or give advance (or manual transitions) a journaled note that CEL and the projection can read. A note would help every manual exit (reclassify, rework, recheck, new-pr). The evidence needs no format change.
resolve_findings is not part of this. It is covered by re-recording the review (swamp-extensions.md:105-111).
Done when
- A walk test shows that
reviseis refused without feedback, and that the nextplandispatch packet injects the feedback, the prior plan and the review. - Against the Lab fake,
plan_revisedcarries the feedback. swamp-extensions.md's "Plan feedback (iterate)" row says how the feedback is kept.
Shipped
Click a lifecycle step above to view its details.