Repository navigation
artifact-audit: 2026-08-25 pass — 8 editorial fixes, 9 hard-stop-adjacent flags - #18
Merged
Merged
Conversation
Monthly artifact-audit pass (4-agent fan-out vs knowledge/ + primary docs read this pass). Applies only editorial, High-confidence corrections; every hard-stop-touching finding is flagged in the PR body for a human, not edited. - templates/ralph/run.sh: cost-meter comment pointed at a nonexistent `claude cost` subcommand (cli-reference: no such subcommand; /cost is in-session only) — now names real wiring options. - templates/goal/recipes.md: /goal clear alias list missing `none`; noted /clear also removes the active goal (goal docs read directly). - templates/loop/loop.md: /review → /code-review (alias since v2.1.223). - templates/ui-audit/PROMPT.md: dead RUN.md cross-reference (file doesn't exist) — caps live in the runbook's launch invocation. - runbooks/overnight-build.md: --remote is now a deprecated alias of --cloud (claude-code-on-the-web docs read directly). - runbooks/first-loop.md + guardrails/checklist.md: stop command for /goal is `/goal clear`; Ctrl+C only stops a non-interactive goal (goal docs). - runbooks/staying-current.md: Routines have no Monthly schedule preset — closest preset + `/schedule update` cron (routines docs read directly). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01ERMTeuPfKdgfgMTCVgdRpK
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.
Artifact drift
Monthly
artifact-auditpass (2026-08-25). Four parallel agents audited skills / templates / guardrails / runbooks+docs againstknowledge/(source of truth, read-only) and primary docs read this pass (cli-reference, goal, scheduled-tasks, routines, claude-code-on-the-web — load-bearing facts independently spot-verified by the orchestrator). No prior artifact-audit PR was open. Open update-knowledge PR #17 touches onlyknowledge/files, which this audit never edits — no conflict. Clean (no drift):guardrails/budget.env, the self-edit gate (.github/workflows/self-edit-guard.yml+self-edit-guard.py— fully consistent with CLAUDE.md and the SKILL.md sentinels),.claude/skills/update-knowledge/SKILL.md,README.md,CLAUDE.md,runbooks/babysit-prs.md,templates/ralph/PROMPT.md, ui-audit PREFLIGHT/PAGES/FINDINGS.Fixed (editorial, High-confidence, no control changed)
templates/ralph/run.shclaude cost/costis in-session only — Hightemplates/goal/recipes.md/goal clearalias list missingnone;/clearalso removes the goaltemplates/loop/loop.md/loop 15m /review 1234/reviewis an alias of/code-reviewsince v2.1.223 (KB primer §4, High) — canonical name usedtemplates/ui-audit/PROMPT.mdRUN.mdtwice — file doesn't exist anywhere in the repo; hard-stop caps were reachable only through the dead pointerrunbooks/ui-audit-overnight.md's launch invocationrunbooks/overnight-build.md--remotepresented as the cloud flag--cloudis current; "The older--remotespelling still works as a deprecated alias" (web docs read this pass) — Highrunbooks/first-loop.md+guardrails/checklist.mdCtrl+Cfor/goal" as the stop command/goal clear; Ctrl+C stops a non-interactive goal — High. (Checklist edit is inguardrails/— flagging that location for your attention; it corrects a kill-switch instruction, doesn't touch the three hard stops orbudget.env.)runbooks/staying-current.md/schedule updatecron — HighFlagged for you (hard-stop / control-adjacent — not edited)
CLAUDE_CODE_MAX_SUBAGENTS_PER_SESSIONstill presented as a live cap in 3 artifacts (high) — the var's default-200 cap was removed in v2.1.224 and it is gone from the docs entirely (primer §4, re-verified 2026-08-17; you already approved the same correction forupdate-knowledge/SKILL.mdin PR knowledge: Aug 10–17 pass — CC v2.1.227–233, Practical Loop Engineering, Agent Plugins GA, roborev v0.65.0 #16 / a829697). Still cited as a working control in:.claude/skills/artifact-audit/SKILL.md:92–93— claims it "bounds the ~4-agent fan-out" (this skill's own iteration hard stop);guardrails/README.md:40–46— lists the default-200 per-session cap as current and says "set them lower explicitly";runbooks/staying-current.md:79, 88–90, 143— recommends=12as both Routines' fan-out hard cap (that env var now does nothing).Suggested replacement everywhere: the surviving native backstops
CLAUDE_CODE_MAX_CONCURRENT_SUBAGENTS(default 20, concurrency) andCLAUDE_CODE_MAX_SUBAGENT_SPAWN_DEPTH(default 3, depth — flipped twice in one week, re-verify), with an explicit note that no native cap on total lifetime spawns remains.templates/ralph/run.shbudget hard stop is a silent no-op by default (high) —cost_so_far()returns0unless the user wires a meter, so the ceiling check can never fire; the header still says "ALL THREE hard stops baked in". Options: add per-call--max-budget-usdtoAGENT_CMD(print-mode; subagent spend counts; total exposure = per-call cap ×MAX_ITERATIONS), and/or refuse to start (or warn loudly) when the meter is unwired — mirroringSUCCESS_CHECK's fail-closed default.templates/goal/recipes.mdcarries none of the three hard stops (high) — no recipe has a bound; the goal docs' own pattern is a turn clause ("…or stop after 20 turns"), and the headless recipe can take--max-budget-usd/--max-turns(both print-mode, verified this pass). Suggested: a "Bound it" rule + capped headless example.runbooks/staying-current.md:78describes the branch-push floor via a toggle that doesn't exist (high) — current routines docs have no "Allow unrestricted branch pushes" option. Real mechanism:claude/-prefixed pushes always accepted; other branches rejected only if protected / carry someone else's commits / have someone else's open PR. The "human merges every PR" floor is only real ifmainhas GitHub branch protection — recommend enabling it and rewording the cell.guardrails/README.mdgateway/ceiling section behind the KB (medium) — still recommends Helicone (in maintenance mode since the Mintlify acquisition — KB says don't build new guardrails on it); missing Databricks Unity AI Gateway (GA Aug 4, enforced proactive budgets); missing--max-budget-usdhalting background subagents (v2.1.217) and in-product gateway spend-limit surfacing (v2.1.225).guardrails/checklist.mdmissing a Routines item (medium) — Routines run with no permission prompts (primer §4, High); the checklist steers laptop-closed runs to Routines without saying so. Suggested checkbox: scope the environment's network access, repos, and connectors tightly..claude/skills/new-loop/SKILL.md:58–59(medium) — budget step offers "or remind the user of the session budget" — an alert, not a ceiling (contradicts CLAUDE.md's non-negotiable). Suggested: name--max-budget-usd(v2.1.217+) / SDKmax_budget_usdfor the/goal+/looppatterns instead of the reminder..claude/skills/loop-guardrails/SKILL.md:32–35(medium) — hard stop knowledge: 2026-06-22 update pass (7-day delta from Jun 15) #3's mechanism list predates--max-budget-usdenforcement; an agent auditing an in-session loop today would miss the canonical native fix. Suggested: add the flag (headless) + gateway-enforced budget options.templates/loop/loop.md(medium) — no budget note, and/loopgenuinely has no native $ ceiling (--max-budget-usdis print-mode only, verified). Suggested honest note: bounds are the 7-day expiry + session scope; watch/usage; for enforced budgets run headless via ralph.Notes (no action taken)
/code-reviewremains invocable from a scheduled/loopfire is Medium-confidence — verify in a live session before relying onloop.md's third starter invocation.runbooks/babysit-prs.mdoptional addition: docs now warn auto-fix replies post under your GitHub account and can trip comment-triggered automation (Atlantis, Terraform Cloud).--remote— a candidate for the nextupdate-knowledgepass (this audit never editsknowledge/).--max-budget-usdnow includes the 1.1× data-residency premium, v2.1.239) contradict nothing here; revisit guardrails wording after it merges.The ui-audit overnight runbook's alert-only budget on its recommended interactive path is a documented trade-off (it already concedes this and points at
run.sh) — it becomes fixable if you take flag 2.🤖 Generated with Claude Code
https://claude.ai/code/session_01ERMTeuPfKdgfgMTCVgdRpK
Generated by Claude Code