mochan-tk / mochan-tk/agentic-dev-kit-for-copilot

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

Open
#162 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

retro:candidate
Dominant language
Shell
Stars
5
Forks
0
Avg merge
16h 36m
Merged PRs (30d)
15

Description

Occurrence: https://github.com/mochan-tk/agentic-dev-kit-for-copilot/issues/119#issuecomment-5745158146

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.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the linked occurrence and inspect the described read-audit-post sequence, including the Python check, heredoc, and gh api -X POST command. The issue explicitly records the failure class only and authorizes no skill or script changes, so there is no implementation or completion criterion beyond preserving that record.

Written by the indexing model from the issue text.

Assessment

Tech stack
github, python, shell
Domain
api, devtools
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Active
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.