cso: don't block the remediation roadmap on a human - #2862
Open
EmilianoU26 wants to merge 1 commit into
Open
Conversation
An unattended `/cso` audit stops at Finding 1 and delivers nothing. The roadmap asks one AskUserQuestion per finding, so a spawned session — where nobody is reading this turn — has no way past the first one. The findings are already written by then; the roadmap is the last step, not a gate. When no human will answer (`SESSION_KIND: spawned`, or an auto-decide result), take the RECOMMENDATION as the disposition and mark it `AUTO-CHOSEN (unconfirmed):`, then close the report with those lines under **Decisions still owed a human** so the ones never confirmed stay visible. Two rules keep that from becoming a licence: - The spawned-session exception still applies to irreversible options. If the recommendation is C) Accept risk, record D) Defer instead — a risk nobody accepted is deferred, not accepted. - Read-only still holds. A) Fix now NAMES the change; it does not make it. Interactive and Conductor sessions are unchanged: ask, or render the prose brief. Edited in `cso/SKILL.md.tmpl`; `cso/SKILL.md` is the regenerated output.⚠️ This does not fit under the parity ratchet, and I could not make it fit without your call. `/cso` has roughly 150 bytes of skeleton headroom; this adds ~1100, so the suite reports `skeleton 62740 > maxSkeletonBytes 61800`. I tried the architecturally correct move first — carving it into a new `cso/sections/roadmap-dispositions.md` so it loads on demand — and it still fails: registering ANY new section costs ~380 bytes of skeleton for its manifest index entry and its pointer, which is more than the headroom. Compressing the block to ~150 bytes leaves the claim without the procedure that makes it actionable. So the trade is yours: bump the ratchet, or name something in the `/cso` skeleton that should move to a section first. I did not raise the cap or trim someone else's paragraph to make room. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
Merging to
After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
An unattended
/csoaudit stops at Finding 1 and delivers nothing. The roadmapasks one AskUserQuestion per finding, so a spawned session — where nobody is
reading this turn — has no way past the first one. The findings are already
written by then; the roadmap is the last step, not a gate.
When no human will answer (
SESSION_KIND: spawned, or an auto-decide result),take the RECOMMENDATION as the disposition and mark it
AUTO-CHOSEN (unconfirmed):, then close the report with those lines under Decisions stillowed a human so the ones never confirmed stay visible.
Two rules keep that from becoming a licence:
recommendation is C) Accept risk, record D) Defer instead — a risk nobody
accepted is deferred, not accepted.
Interactive and Conductor sessions are unchanged: ask, or render the prose brief.
Edited in
cso/SKILL.md.tmpl;cso/SKILL.mdis the regenerated output.without your call.
/csohas roughly 150 bytes of skeleton headroom; this adds~1100, so the suite reports
skeleton 62740 > maxSkeletonBytes 61800.I tried the architecturally correct move first — carving it into a new
cso/sections/roadmap-dispositions.mdso it loads on demand — and it stillfails: registering ANY new section costs ~380 bytes of skeleton for its manifest
index entry and its pointer, which is more than the headroom. Compressing the
block to ~150 bytes leaves the claim without the procedure that makes it
actionable.
So the trade is yours: bump the ratchet, or name something in the
/csoskeleton that should move to a section first. I did not raise the cap or trim
someone else's paragraph to make room.
🤖 Generated with Claude Code