Skip to content

fix(planning): keep interview titles, revision banners and apply refusals accurate #7754

Closed

Description

@kyle-sexton

Parent

Refs #5795

Depends on

  • none: on the frontier now

Outcome

Three text defects from the interview flow are fixed:

  • A live question that supersedes another gets its own short title. Today 14 live questions reused their archived predecessor's yes/no short title while asking a different question.
  • The "Recommendation revised" banner no longer reads "Your note: ..." over Claude's own explanation; Claude's reason is labeled as Claude's.
  • round.sh apply refusing a file for a duplicate id says that nothing was written and names the first duplicate.

Key interfaces

  • round.py: question validation (duplicate id exit today), the add path that supersedes a question, and revise.
  • Interview page revised banner (renders the revised text as is).
  • Interview skill surface.md ops table rules for short and revised.

Done when

  • Adding a superseding question whose short equals its predecessor's is refused or warned (test in the round.py suite).
  • The revised banner labels the text as Claude's reason, and the skill tells the session not to write "Your note" in revised (UI test).
  • apply on a file with a duplicate id prints that nothing was written and names the first duplicate id (test).
  • The planning surface and Python test suites pass.

Out of scope

Activity

  1. added
    priority: mediumReal value, no hard deadline; normal backlog flow.
    needs-humanHuman-in-the-loop required; autonomous sessions must not resolve items carrying this.
    on Oct 17, 2026
  2. added theissue type on Oct 17, 2026
  3. kyle-sexton commented on Oct 17, 2026

    @kyle-sexton
    MemberAuthor

    Ratified in the owner's batch of decomposed children (2026-10-17).

  4. removed
    needs-humanHuman-in-the-loop required; autonomous sessions must not resolve items carrying this.
    on Oct 17, 2026
  5. kyle-sexton commented on Oct 18, 2026

    @kyle-sexton
    MemberAuthor

    Agent Brief

    Type: Bug
    Summary: Keep interview short titles, revision banners and apply refusals accurate.

    Current behavior:

    • A live question that supersedes another can reuse its predecessor's short title while asking a different question.
    • The 'Recommendation revised' banner can read 'Your note: ...' over Claude's own explanation.
    • round.sh apply refusing a file for a duplicate id prints only duplicate id: <id>, without saying nothing was written.

    Desired behavior:

    • Adding a superseding question whose short equals its predecessor's is refused or warned.
    • The revised banner labels the text as Claude's reason, and the surface ops rules tell the session not to write 'Your note' in revised.
    • apply on a file with a duplicate id says nothing was written and names the first duplicate id.

    Key interfaces:

    • round.py question validation, the superseding add path, and revise.
    • Interview page revised banner.
    • Interview surface ops table rules for short and revised (the context surface spoke, not SKILL.md).

    Acceptance criteria:

    • test_round.py gains cases for the superseding short-title refusal or warning and for the duplicate-id message, and scripts/run-test-file.sh plugins/planning/surface/test_round.py exits 0.
    • The browser suite gains a case asserting the revised banner labels the text as Claude's reason (read from the diff), and scripts/run-test-file.sh plugins/planning/surface/surface.test.sh exits 0.
    • The surface spoke tells the session not to write 'Your note' in revised (read from the diff); the interview SKILL.md is not edited.
    • A planning changelog fragment is added and scripts/check-changelog-fragments.sh exits 0.

    Out of scope:


    This was generated by AI during backlog shaping. Held, not admitted: this brief is drafted and checked, but the issue lane would stop on it today. It is admitted, by adding agent-ready, once the lane can run the skill-quality check scripts (Automation Lanes' PR #7912). Held under the owner's 2026-10-18 lane-narrowing course correction.

  6. added
    needs-humanHuman-in-the-loop required; autonomous sessions must not resolve items carrying this.
    agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.
    work-class: scopedA briefed fix or small feature; blast radius bounded by the brief, tests exist.
    and removed
    needs-humanHuman-in-the-loop required; autonomous sessions must not resolve items carrying this.
    on Oct 18, 2026
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

    agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.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

    Fields

    No fields configured for Bug.

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions