Souptik Chakraborty 26681abb8c fix(amp,droid): surface Approve-with-Notes feedback instead of dropping it (#1146) ...

The Approve with Notes flow (#1092) emits decision:"approved" with a
feedback field, but both adapters collapsed every approved decision to a
bare "Approved." -- the reviewer's notes were silently lost.

Amp had two drop sites: formatAnnotationFeedback returned null for
anything not "annotated", and handleAnnotateResult early-returned on
approved before formatting was ever reached. Droid's
emitAnnotateDecision printed only "Approved."

Both adapters now emit the shared approved-with-notes prompt (mirroring
DEFAULT_ANNOTATE_APPROVED_WITH_NOTES_PROMPT in
packages/shared/prompts.ts); note-less approvals keep the old behavior.
The amp config override key approvedWithNotes matches the shared prompt
config shape.

Fixes #1137

2026-07-29 22:59:46 -07:00

..

run-plannotator.js

fix(amp,droid): surface Approve-with-Notes feedback instead of dropping it (#1146)

2026-07-29 22:59:46 -07:00

run-plannotator.test.js

fix(amp,droid): surface Approve-with-Notes feedback instead of dropping it (#1146)

2026-07-29 22:59:46 -07:00