Skip to content

feat: global /autodream skill, shared tmo/repo_slug, validated noise-gate knobs - #158

Merged
STRML merged 7 commits into
mainfrom
cleanup/dedup-validate-comments
Oct 19, 2026
Merged

STRML merged 7 commits into
mainfrom
cleanup/dedup-validate-comments

Conversation

@STRML

@STRML STRML commented Oct 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds a global /autodream skill. It prints now triaging N issues from dates A to B (or no issues to address!), starts a dream through autodream-now.sh, recommends a --dangerously-skip-permissions session, then walks the open questions one per turn. install.sh links it into ~/.claude/skills (--no-skill opts out).
  • Moves tmo and repo_slug into bin/portable.sh; they were byte-identical copies in prune-worktrees.sh and triage-candidates.sh.
  • Rejects a non-integer AUTODREAM_MIN_USER_TURNS / AUTODREAM_MIN_MINUTES at startup. Before, the noise gate passed them to jq --argjson with stderr discarded, so a bad value let every session through to a model call.
  • Rewrites change-narration comments in run.sh and tests/run-all.sh.

New files: bin/pending-triage.sh, bin/report-format.sh (marker parsing moved out of review.sh), skills/autodream/SKILL.md, tests/pending-triage.sh, tests/install-skill.sh.

Status

Check Result
Full suite 2711 passed, 4 failed. The 4 fail on unchanged main in this sandbox (adapter-claude needs ps, two review-cmux interactive assertions, prune-worktrees watchdog timing). I did not re-compare names on the last run.
New suites pending-triage 20/20, install-skill 29/29
shellcheck no new findings on the new files; install.sh unchanged at 2 blocks
/debate:run tight 3 of 4 seats answered (executor, auditor, antigravity); the deepseek seat failed twice. Three rounds, every finding fixed except the one below.

Debate fixes: a half-triaged report no longer looks addressed (## Triage in progress until the last question), config sourced with set +u and the environment wins, the skill is linked only where its scripts live, link() reports a failed ln -s, future-dated reports are ignored, the remaining count subtracts logged decisions, a zero-padded marker (08) reads as decimal, and any run of trailing slashes on the install dir is accepted.

Open follow-ups

  • A decision logged as several bullets under ## Triage in progress over-subtracts the remaining count (floored at 1). The skill logs one line per decision, so this is left.
  • Run ./install.sh after merge to link the skill; the live install was not touched.
  • Not in this PR: removing the L2 argv fallback and the round-1 review-marker migration, and sharing the L2 system prompt and report-split awk. Each needs its own /debate:run tight.

Test plan

  • TMPDIR=/tmp/cdup bash tests/run-all.sh
  • bash tests/pending-triage.sh, bash tests/install-skill.sh, bash tests/review-skip.sh
  • /debate:run tight on the branch diff
  • ./install.sh on a real ~/.claude, then /autodream in a fresh session

STRML added 7 commits October 18, 2026 11:06
Both were byte-identical in prune-worktrees.sh and triage-candidates.sh. tests/portable.sh now pins them.
…comments

AUTODREAM_MIN_USER_TURNS and AUTODREAM_MIN_MINUTES went to jq --argjson with stderr discarded, so a non-number gated nothing and sent every session to a model. They are checked at startup now, with a test. Seven comments that narrated a past change now state the constraint.
install.sh links skills/autodream into the skills dir (--no-skill skips it, AUTODREAM_SKILLS_DIR moves it). pending-triage.sh prints 'now triaging N issues from dates A to B' or 'no issues to address!' from the reports that have open questions and no Triage decisions heading. review.sh and pending-triage.sh share the marker parsing through report-format.sh.
… it can run

Review findings. The skill logs under Triage in progress and writes Triage decisions after the last question, so an interrupted session does not hide the rest. A custom install dir links the skill only when AUTODREAM_SKILLS_DIR is set. pending-triage.sh sources the config with nounset off, lets the environment win, and ignores future-dated reports. link() returns failure when ln -s fails, and the skill says to work in the repository a question is about, to leave a rebuilding date out of the walk, and to skip the dream off macOS.
…g slash on the install dir

pending-triage.sh subtracts the decisions logged under Triage in progress, and one question always stays until the heading is renamed. install.sh compares the install dir without a trailing slash.
…ink, and log decisions as bullets

codex-review round 1: P1 an unknown report was hidden as an empty queue, P1 a user-owned symlink was replaced, P2 plain decision lines were not counted. Swept pending-triage.sh, SKILL.md and install.sh for the same classes (unknown state treated as absence, an existing link treated as ours, writer/parser format mismatch).
@STRML
STRML merged commit ab3c565 into main Oct 19, 2026
3 checks passed
@STRML
STRML deleted the cleanup/dedup-validate-comments branch October 25, 2026 04:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant