Every plan review now ends by telling you, in one line, whether anything is still unresolved.
- Mandatory unresolved-decisions status in the GSTACK REVIEW REPORT that appears as the final line before the approval prompt
- Blocking approval gate that refuses to allow plan approval unless the report's final non-whitespace line contains the unresolved decisions status
- Static and E2E tests pinning the mandatory unresolved decisions status across every report consumer
- /plan-devex-review endpoint now logs a review entry when a plan-review skill finishes
The GSTACK REVIEW REPORT closes with the open decisions, or "NO UNRESOLVED DECISIONS" in plain sight, before you approve.
When a plan-review skill finishes and hands you the plan to approve, its report now ends with a mandatory unresolved-decisions verdict.
If decisions are still open, it lists each one and what breaks if you ship it deferred.
If nothing is open, it prints the exact line NO UNRESOLVED DECISIONS.
A token-reduction pass had made this line optional, so a clean plan and a plan hiding an open question rendered the same.
Now the line is never omitted, it is always the last thing you read before the approval prompt, and the approval gate refuses to let the plan through without it.
Added: Mandatory unresolved-decisions status in the GSTACK REVIEW REPORT.
Generated into all six report consumers from scripts/resolvers/review.ts.
The report always ends with either the exact unbolded sentinel or a **UNRESOLVED DECISIONS:** bullet block; never omitted, always the final line.
Added: Blocking approval gate that refuses ExitPlanMode unless the report's final non-whitespace line is the unresolved status.
Added: Static and E2E tests pinning the mandatory status across every report consumer.
Fixed: /plan-devex-review never logged a review entry.