mochan-tk / mochan-tk/agentic-dev-kit-for-copilot
retro-candidate: Epic record audit exit status was not chained to the post step
Nobody has claimed this yet.
- 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
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- 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