th-e77603: declarative scrape rules + aider/goose/crush/cline harnesses

[brentrager]

Pearl th-e77603 (epic th-faa590) · fixes th-d2a1e4

Problem

SmoothFlow could only learn state for harnesses with hooks. Hookless coding CLIs (aider, goose, crush, cline…) show idle / working / needs-you only on their terminal, and the flat [state.scrape] lists can't describe real TUIs: crush draws its permission modal over a footer that still says esc cancel, cline's composer looks idle for the whole turn, aider streams answers with no marker at all. And prompt_as = "paste" pasted on a 4 s timer, straight into aider's first-run question.

Solution

• [[state.scrape.rules]] (crates/smooth-flow/src/scrape.rs): ordered rules, first match wins, evaluated before the flat lists (which keep their exact precedence). A rule is match regexes over a scope (tail / pane / last_line / cursor_line / title) with all, unless, unless_below, quiet_ms, changed_within_ms, alternate_screen, spinner. Pure over a PaneObservation; ANSI is stripped first.
• Engine: one display-message per tick for title / alt-screen / cursor row, a per-session "pane last changed" clock, the deciding rule's name in the needs-you detail, a scraped needs-you clears on the next idle, and paste waits for a scraped idle (never into a needs-you; 90 s fallback; manifests with no idle signal keep the 4 s timer).
• resume.mode = "continue_latest" for CLIs that only continue "the latest conversation here".
• Built-ins: aider, goose, crush, cline.
• Docs: Harness-Manifests.md — rule semantics + load/timing guidance, the shipped table (live vs fixture-only), how cmux and orca detect hookless state (file:line), and why Claude's detect.rs is expressible as rules.

Verification

• Live, real CLIs through the engine (tests/scrape_live.rs, #[ignore]), fresh scratch $HOME, isolated installs, local mock OpenAI server (no accounts, no keys): aider 0.86.2, goose 1.50.0, crush 0.94.2, cline 3.0.61 each went launch → first-run needs-you → working → idle, steer → working → idle, tool/edit request → needs-you → deny → idle, kill + resume → back.
• Fixture-only: auggie 0.36.0 and kiro-cli 2.21.4 stop at sign-in walls (not pressed); wall captures are committed and tested never to read idle/working under any built-in. No built-in manifest for either.
• 44 real pane fixtures + tests/scrape_fixtures.rs (verdict + deciding rule per fixture, resize/padding/SGR invariants, time monotonicity, detect.rs-as-rules equivalence on 5 000 generated panes). Unit/property tests in scrape.rs, harness.rs, engine.rs, plus a live fake-CLI test for the paste gate and needs-you clearing.
• cargo test -p smooai-smooth-flow green (168 + 6), cargo clippy -p smooai-smooth-flow --all-targets has no warnings in the files this PR touches. Not run locally: smooth-daemon / smooth-cli test builds (disk under 6 GiB on the build machine). Their only changes are list assertions that now count BUILTIN rather than assuming four, so CI is the check for those.

Harness conformance (#598) — expected failure

tests/conformance/{aider,goose,crush,cline}.toml enrol the four in the conformance suite, with screens cut from the live captures. Once this PR is rebased onto #598, screen_problems gets patched here to count [[state.scrape.rules]] and to feed painted screens a quiet time. Until follow-up th-5a2314 (harness-conformance: manifest-driven [steer] approve_keys / deny_keys) lands, the needs_you → approve step fails for all four. The engine approves with keystroke 1; aider wants y/Enter, goose Enter, crush Enter, cline y. This is expected, not a regression.

🤖 Generated with Claude Code

[changeset-bot]

🦋 Changeset detected

Latest commit: 6200ada

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@smooai/smooth	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR