Skip to content

fix(planning): route interview acceptance criteria into export-brief #5461

Description

@kyle-sexton

No related issue: split out of #5191 after a repro pass on planning 0.46.0 (origin/main at 4d1d9d8).

Summary

Finding 18 of #5191, reported against planning 0.42.0: export-brief has no way to receive acceptance criteria gathered in conversation, so the session hand-merged them into PLAN.md.

Fix

Have export_brief read the restated acceptance section.

Verification

Repro pass on origin/main, playwright-cli driving Chromium against a copy of plugins/planning/surface/, plus round.sh, watch.sh and curl. Verdict: reproduces.

export_brief hard-codes - none recorded in the interview surface under "Acceptance criteria" (plugins/planning/surface/exporters.py:647-651) and never reads doc["restatement"], although restate accepts an acceptance section (round.py:654-661). Live: restate with acceptance "- AC one is testable ...", then export-brief: output still says "none recorded in the interview surface". context/surface.md:206 tells the session to merge by hand.

The part of finding 18 that overlaps #4653 (how commitments reach the Brief, exporters.py:629-655) is on #4653.

Related

Refs #5191. #4653 tracks the commitment and export gaps behind findings 9, 17 and part of 18.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    priority: mediumReal value, no hard deadline; normal backlog flow.work-class: scopedA briefed fix or small feature; blast radius bounded by the brief, tests exist.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions