unic / unic/unic-agents-plugins
bug(unic-archon-dlc): /pr-review's review-gate cannot re-pause, its message names an unresolved path, and prune_slug_dirs is off
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 1
- Forks
- 0
- Avg merge
- 16h 43m
- Merged PRs (30d)
- 19
Description
What this is
Three leftovers from #439's grilling on 2026-09-04, approved by the maintainer as a spin-out and separated from #439 because none of them changes what /build produces. #439's first spin-out shape ("no generated artefact is committed to a merged branch") died in that grilling: cleanup.prune_slug_dirs already removes <artifacts_dir>/<slug>/ once a slug's pull request is merged or closed, and it defaults to false, so the accumulation on develop has a built, unused fix.
/pr-review'sreview-gatehas noon_reject. A reject cancels the run. The findings are on disk (synthesizeandreconcilewrite fourSESSION/*files before the gate), so a reject loses nothing untilarchon complete <branch> --forcedeletes the worktree. A human who wants to read before deciding has no re-pause, unlike/build'sbuild-pr-gate, whoseon_rejectre-pauses withmax_attempts: 3.- The gate message names
SESSION/comment.mdandSESSION/threads.jsonas paths a human should open, andSESSIONis a constant the message never resolves, so the human is pointed at a path that does not exist as written. - Whether
cleanup.prune_slug_dirsshould default totrue. Session artefacts stay committed by design (#439 criterion 6 and the 30 712-character report that ruled out a comment); the mechanism that prunes them after merge exists and is off. - Whether that pruning wants its own gate mode. Added 2026-09-04 22:30 on the maintainer's words in #439's grilling: "Creo que esto puede gozar de una configuracion AFK|HITL tambien, no? Y un gate para HITL, claro. Va al B." So the question is not only the default but whether
prune_slug_dirsbecomes agates.<box>-shaped setting with anapproval:node inhitl. Note the collision with the two-valued gate vocabulary #439 measured (hitl | afk, no middle) before adding a third instance of it.
Open question, to be grilled before criteria
Is an on_reject that only re-pauses a gate, or a workaround for a schema with no on_approve (ADR-0011 line 43)? The maintainer's own experience in run 2 was "the only lever was destroying the run", and item 1 above is that lever.
Related
#439 (source), #458 (/qa's unread node outputs, not absorbed here), #440 (re-entry idiom).
Contributor guide
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 /pr-review review-gate and compare its reject behavior with /build's build-pr-gate, including the on_reject setting. Inspect the gate message references to SESSION/comment.md and SESSION/threads.json, then review cleanup.prune_slug_dirs and ADR-0011 line 43. Done means the reject flow, displayed paths, and pruning configuration questions have explicit criteria and tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- cli, tooling
- Issue type
- Bug
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Active
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100