Skip to content

plan-*-review: shared GSTACK REVIEW REPORT rewrite deletes earlier review's summary; AUQ option label easy to omit #3219

Closed

Description

@virajs13

Two issues seen running /plan-design-review then /plan-eng-review on the same plan file (gstack 1.91.101.0).

1. Report rewrite deletes another review's completion summary. plan-design-review writes its Completion Summary box inside the ## GSTACK REVIEW REPORT section. plan-eng-review's "Write to the plan file" step deletes that whole section (heading to next ## or EOF) and appends a fresh report, so the design review's per-pass scores, decision count and deferrals disappear from the file; only its one-line table row survives.

Suggestion: have each plan-*-review write its completion summary as its own ## <Review> completion summary section above the terminal report, and document that everything under ## GSTACK REVIEW REPORT is regenerated by the next review.

2. AskUserQuestion option label easy to drop. The decision-brief format describes options in prose (label, description, (recommended) on the label). In a 7-pass review the model repeatedly wrote the long description and omitted the short label field, causing InputValidationError round trips. Suggestion: show the per-option shape as explicit fields (label: short, description: long) in the Format section and add "every option has both label and description; (recommended) goes on the label" to the pre-emit self-check.

Activity

  1. capy-ai commented on Oct 17, 2026

    @capy-ai
    Contributor

    Item 1 (a report rewrite deleting an earlier review's summary) is fixed in #3225, released in v1.91.109.0 (86091dd): /plan-design-review and /plan-devex-review now write their summaries in their own sections above the report. Item 2 (the AskUserQuestion label) is still open and is tracked with #3222 for the next batched wave, so I'm leaving this issue open.

  2. capy-ai commented on Oct 17, 2026

    @capy-ai
    Contributor

    Closing: item 1 shipped in #3225 (v1.91.109.0, 86091dd). Item 2 (the AskUserQuestion label) is tracked in PR #3222.

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

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions