Commit Graph

8 Commits

Author SHA1 Message Date
CodeWhale Bot e81ed33fcc fix(release): finish the v0.9.6 candidate
Move first-run usage disclosure into the native TUI, keep telemetry unarmed until the decision, and order all telemetry writes and delivery against persistent opt-out with fail-closed setup-state handling.

Replace the narrow wide-terminal rail with a responsive full-screen ocean canvas, preserve readable prose measure, and remove per-call padding inside grouped tool activity.

Verified with formatting, clippy, locale/version/budget gates, the 10,120-test TUI library suite, telemetry/config suites, focused UI/PTy coverage, and independent release/UI review. The sandbox-only loopback acceptance rerun remains explicitly environmental.

Agent assistance: Claude and Codex were used for implementation analysis and verification.
2026-08-09 22:41:53 -07:00
CodeWhale Bot 5798de8b39 build(release): native musl for Linux ARM64, and reconciled copy
Three v0.9.5 benchmark tasks could not launch Codewhale at all:
mteb-leaderboard, mteb-retrieve, and pytorch-model-recovery run older ARM64
images, and the aarch64-unknown-linux-gnu artifact built on ubuntu-24.04-arm
requires GLIBC_2.39.

Release and nightly now build aarch64-unknown-linux-musl on the native ARM
runner, alongside the x64 musl build that has shipped since v0.8.65. Both
gain a static check (no ELF INTERP) plus a launch smoke on the matching
native runner, so a dynamically linked or non-starting binary fails the
build rather than the user's install. docs/INSTALL.md drops the arm64 glibc
floor section and describes the v0.9.6 matrix.

Also reconciles the copy the earlier commits invalidated: setup ratification
text and all 15 locales stop attributing execution doctrine to "mode
prompts", docs/MODES.md states Auto-Review's actual deterministic
allow/deny behavior, and the source-structure, dead-code, and
runtime-contract budgets are re-measured — the last locking down 25
decreased ceilings and the new mode-agnostic prompt-stage digests.
2026-08-09 00:52:42 -07:00
CodeWhale Bot 18769cdbe8 fix(release): 0.9.4 stall, budget, and UI polish for session-title fix
- runtime-contract: regenerate tool_catalog for todo_write sole surface (plan/act/operate full/active now list todo_write, not work_update); bumps bytes/sha + prompt stages (agent plugins work)
- web: public-surface-contract expects todo_write (matches docs/RUNTIME_SIMPLIFICATION_DESIGN already)
- source-structure: 676325 -> 676652 (+327) — 321 for bf69e7ff5 session-title fix plus 6 for stall/UI tweaks; durable test asset
- engine: raise no_user_input_continues 12 -> 20 (6 sites) to stop false 'hit (12)' stops on long todo_write loops; preserves anti-runaway
- subagent: GENERAL/PLAN intros now say todo_write, not work_update (child priming fix)
- palette: WHALE_TEXT_HINT #8491AA -> #8A99B3 (+0.4 contrast)
- tui: add BehavioralTipTodoWrite + 15 locale keys (hint: track with todo_write)
- subagent tests: fmt fixes for isolated_fleet_roster_with + assert! expansion

Refs: efcf47a1d, 21ed173cf, ec5747f7d, #5258
2026-08-07 04:32:55 -07:00
CodeWhale Bot 21ed173cf1 fix(tui): pre-release repair batch for 0.9.4 — stall honesty, idle wakes, truncation recovery, wait ergonomics
- turn_loop: a mid-stream chunk-timeout now counts toward the stream-error
  budget (stall with nothing streamed retries transparently; an exhausted
  budget fails the turn with the real reason instead of ending Completed
  over a frozen block).
- idle engine: a finished background shell task wakes and starts an ordinary
  runtime turn even without an active goal; a dead provider route claims the
  completion once and reports where the output lives.
- subagent: over-budget final reports spill to a session artifact and the
  truncation footer names the retrieve_tool_result ref; write failures
  degrade to the honest no-ref footer. Test-only wrappers marked cfg(test).
- waits: agents/wait and agent action=wait default to 30 s and cap at 120 s
  (blocked waits deafen the session; settled children report back as
  sentinels). Bash action=wait honors timeout_secs/timeout aliases and
  block; result metadata reports the real wait_timeout_ms.
- todo_write canonical naming: constructor is new(); work_update/TodoWrite/
  todo stay hidden compat aliases; user-visible copy and docs updated.
- behavioral tips: DurableStateWritten fires on successful remember calls;
  enum allow removed. voice.rs and work_surface model use let-chains.
- test: Windows path-separator tolerant artifact footer assertion.
- changelog: 0.9.4 additions (Agent Plugins v1.0.0, send_later, /advisor,
  quiet mode, automation forms, resume_from, transport resilience,
  durability, zh-Hant, update chip, RLM groundwork, stall/wake/truncation/
  wait fixes). Dead-code budget re-baselined to 452.
2026-08-07 02:45:59 -07:00
CodeWhale Bot a50b6532bd feat(tui): harness/continual ledger, effort, recommend, voice ASR, ui split — 0.9.4 final bundle
- continual_harness ledger (bounded 24, PromptNote/SubagentSpec/SkillHint) + harness tool (overview/refine/remove) for persistent working context
- /effort + /thinking aliases for ReasoningEffort
- skills/recommend: deterministic skill ranker for /plugin suggest|recommend
- voice: free ASR stack (GROQ_ASR_URL, LOCAL_WHISPER_BINS, detect_free_asr)
- tui/ui split: apply/event_loop/handlers/dispatch/motion/overlays/provider_routes/release_check/session_state/frame/terminal extracted from 19k-line ui.rs
- model catalog: Muse Spark 1.2 + ultra effort + cache_read costs
- provider_defaults: gpt-5.5→5.6, muse-spark-1.1→1.2 canonical routes
- locales: CmdEffortDescription, PickerActionSetStartupDefault, operate hint refresh
- docs + budgets resynced; drop stale HANDOFF/RELEASE_EVIDENCE working notes
- .gitignore: ignore .ralph/ agent scratch
2026-08-06 00:35:36 -07:00
Hmbown 7c8a3f89b6 chore(tui): drop unused session/fleet stubs and orphan ops ledgers
Remove never-constructed SaveSession path and unused App helpers, delete
stale ops drafts (keep active handoff), and ratchet the dead-code budget
after cargo check + budget pass (449).
2026-08-04 23:52:08 -07:00
Hmbown f471f555d2 chore(tui): sweep verified-dead ToolRegistry API surface (#4785)
First dead-code sweep slice for #4785. All in
crates/tui/src/tools/registry.rs; 181 net production lines removed
(264 incl. orphaned inline tests), 16 #[allow(dead_code)] attributes
gone (482 -> 466, budget locked in via --update).

Deleted items (zero production callers; verified with rg across
crates/** and confirmed by warning-free cargo check/clippy in this
binary crate, where rustc flags unreachable pub items):

- ToolRegistry::len / is_empty        (test-only callers, in-file)
- ToolRegistry::execute               (test-only; live path is
                                       execute_full{,_with_context})
- ToolRegistry::filter_by_capability  (only caller: its own test)
- ToolRegistry::read_only_tools       (only caller: its own test)
- ToolRegistry::approval_required_tools   (no callers anywhere)
- ToolRegistry::approval_suggested_tools  (no callers anywhere)
- ToolRegistry::set_context / context_mut (no callers anywhere)
- ToolRegistry::remove                (test-only; live remove_tool
                                       also invalidates api_cache)
- ToolRegistry::clear                 (test-only)
- ToolRegistryBuilder::with_agent_tools
    (legacy allow_shell wrapper; comment admitted test-only)
- ToolRegistryBuilder::with_full_agent_surface{,_policy}
    (legacy wrappers; only caller was each other)

Attribute-only removals (items have live production callers):
- ToolRegistry::names      (engine.rs configure_plugin_tools x2)
- struct McpToolAdapter    (built by with_mcp_tools; all fields read)

Orphaned test code deleted: test_registry_remove, test_registry_clear,
test_registry_execute, test_registry_execute_unknown_tool,
test_filter_by_capability, test_read_only_tools. Tests re-pointed at
live APIs instead of deleted shims: remove_tool_invalidates_api_tools_cache
(remove_tool), legacy_file_aliases_replay_through_canonical_dispatch
(execute_full), three with_agent_tools_* tests rewritten against
with_agent_tools_policy(ShellPolicy::None/Full).

Gates (isolated CODEWHALE_HOME, CARGO_TARGET_DIR=.../v093-deadcode-sweep-a):
- cargo fmt --all -- --check                          PASS
- cargo test -p codewhale-tui --bin codewhale-tui --locked tools::registry
  36 passed, 0 failed
- ... tools::      1391 passed, 0 failed
- ... core::engine  435 passed, 0 failed (one flaky first-run failure,
  passed twice on rerun)
- cargo clippy -p codewhale-tui --all-features --locked -- -D warnings
  (+ brief's allow list)                                 PASS
- python3 scripts/check-dead-code-budget.py            PASS (466 at budget)
- python3 scripts/check-source-structure-budget.py     PASS
  (645410 <= 645603; was 645674, i.e. the +71 overage is offset)
2026-07-30 22:50:20 -07:00
Hmbown c1b47796ce chore: land the bounded dead-code slice and stop the wall regrowing
#4785 asks for a 481-attribute sweep. This is not that sweep — it is the part
that can land without judgment calls, plus the gate that makes the sweep worth
doing.

The measurements are why the gate comes first:

    issue filed   464 attributes / 143 files
    2026-07-26    426 / 111
    2026-07-28    481 / 155

The sweep was working and the total still rose, because two large landings this
week added state whose accessors only their own tests read. A sweep is a
snapshot; a budget is a direction.

scripts/check-dead-code-budget.py counts `allow(dead_code)` across crates/ and
fails when the total exceeds a committed ceiling. Lowering it is free; raising
it needs `--update` and a sentence in the PR, so growth becomes a visible
decision rather than an accident. On failure it prints per-crate now-vs-budget
and marks which crate grew. Wired into Lint next to the other check-*.py gates.

This covers a real blind spot: CI's clippy runs without `--all-targets`, so it
never lints cfg(test) or integration-test code — and a prior strip-and-check
measured 197 attributes alive *only* because a test references them.

The three zero-risk removals from the issue's cluster list:

- `codew_legacy_shim.rs` -> `codew.rs`. The issue calls this out as a rename,
  not a deletion: it is the live `codew` binary and its own docs call it
  permanent. Every sweep flags the filename; the binary name is unchanged.
- `clap` dropped from app-server and `thiserror` from lane — no source in
  either crate references them. Cargo.lock loses exactly those two edges and no
  package versions move.

The budget baseline is recorded at today's 482, not at an aspirational number,
so the gate starts truthful and the sweep can ratchet it down.
2026-07-27 18:53:25 -07:00