PH3.4 — Publish and discard: synchronous final validation, 409 with findings, warning acknowledgement #26

New issue

Open

opened 2026-09-11 09:02:14 +00:00 by bart · 0 comments

bart commented

2026-09-11 09:02:14 +00:00

Collaborator

Parent

Row PH3.4 in docs/12-implementation-plan.md (branch, owned paths and doc section are defined there).

What to build

A planner confirms a reviewed draft: publish runs the validator synchronously, refuses with 409 and the findings if any error remains, requires acknowledgement of warnings, supersedes the previous published version per date and publishes assignments to technicians. A draft can be discarded.

Acceptance criteria

Publish with an E_ finding → 409 with the findings in the body; nothing written
Publish with unacknowledged W_ findings → 409; with ack_warnings → published, acknowledgements audited
Superseding is per date (S14) and never mutates prior published assignments
Discard moves the draft to discarded with an audit row; S17 passes

Blocked by

#20 — Plan orchestration: snapshot builder, result writer, solver_run/plan_version lifecycle, publish/supersede
#19 — Validator: one rule per E_/W_/I_ code over snapshot and result
#24 — Triage: accept, reject, accept-all, pin and rejection locks, E_UNTRIAGED, I_GAP_UNFILLED, I_REJECTED_ELSEWHERE

## Parent Row **PH3.4** in `docs/12-implementation-plan.md` (branch, owned paths and doc section are defined there). ## What to build A planner confirms a reviewed draft: publish runs the validator synchronously, refuses with 409 and the findings if any error remains, requires acknowledgement of warnings, supersedes the previous published version per date and publishes assignments to technicians. A draft can be discarded. ## Acceptance criteria - [ ] Publish with an `E_` finding → 409 with the findings in the body; nothing written - [ ] Publish with unacknowledged `W_` findings → 409; with `ack_warnings` → published, acknowledgements audited - [ ] Superseding is per date (S14) and never mutates prior published assignments - [ ] Discard moves the draft to discarded with an audit row; S17 passes ## Blocked by - #20 — Plan orchestration: snapshot builder, result writer, solver_run/plan_version lifecycle, publish/supersede - #19 — Validator: one rule per E_/W_/I_ code over snapshot and result - #24 — Triage: accept, reject, accept-all, pin and rejection locks, E_UNTRIAGED, I_GAP_UNFILLED, I_REJECTED_ELSEWHERE

bart added the

ready-for-agent

label

2026-09-11 09:02:14 +00:00

bart added a new dependency

2026-09-11 09:02:27 +00:00

#20 T-E — Plan orchestration: snapshot builder, result writer, solver_run/plan_version lifecycle, publish/supersede

bart added a new dependency

2026-09-11 09:02:28 +00:00

#19 T-D — Validator: one rule per E_/W_/I_ code over snapshot and result

bart added a new dependency

2026-09-11 09:02:28 +00:00