Skip to content

retro-candidate: Epic record audit exit status was not chained to the post step #162

Description

@mochan-tk

Occurrence: #119 (comment)

The coordinator audits every Epic record before posting it with a small Python check (non-ASCII count must be 0; no closing keyword followed by an issue reference) that ends with sys.exit(1) on a hit. In this occurrence the audit reported a hit (the word closed followed by a colon and the reference to issue 120, in the sentence listing the sixteen finished sub-issues) but the gh api -X POST that followed was written on its own line after the heredoc instead of being chained with &&, so the post ran regardless of the audit's exit status. The comment is an issue comment, where closing keywords have no effect, and all cited issues were already closed, so nothing changed on GitHub; the record is otherwise accurate and stays unedited. The failure class is a fail-closed check whose result is not wired to the action it guards. The plan-management and session-orchestration skills describe the audit intent but do not require the post step to be conditional on the audit, so any agent scripting the same read-audit-post sequence can repeat it.

This issue records the failure class only (first occurrence); it does not authorize skill or script changes.

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

    retro:candidateObserved scaffold friction; promote to a retro: PR at the 2nd occurrence

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions