[frankieramirez]

Prepare Bushel 0.3.6 with the merged audit fixes. Cargo.toml and Cargo.lock change the application version and replace yanked chacha20 0.10.1 with 0.10.2; structural comparison against current main verifies every other package version and dependency edge is preserved. A separate integration correction fixes ASCII health-banner punctuation with regression tests.

Integrated main: 0f24599 (PR119, PR125, PR126 and PR127).
Final head: ba7e1ae.
Final tree: ca20bb65a0140ff656724ee03c8cbe4d1abb4bd6.
Integration commit 9088e0d preserves published release ancestry with parents 3ef3a04 and 0f24599.

Proposed release notes

• Settings toggles preserve comments, unknown fields and unrelated settings. Invalid, unsupported-lossless, externally changed and symlinked config writes are refused while preserving file bytes and usable settings. Cargo-installed users receive the crates.io upgrade command. Process Exit follows delivered stdout/stderr, including buffered final lines; descendants holding pipes have a bounded read-wait budget (Fix Cargo update hints, selective config persistence, and stream exit ordering #119; bushel update tells cargo users to install git HEAD, not crates.io #97, Invalid or richer config is overwritten by a settings toggle; write one key #98, Stream Exit can overtake final output lines; send Exit after readers drain #104).
• Interactive shell execution passes through Runner, Client and Engine with mock coverage and consistent session error handling (Route interactive exec through Runner and Engine #125; Exec bypasses the Runner seam; route it through Runner and Engine #109).
• Engine reads are scoped, cancellable and ordered; inspect caches track resource identity; slow polling and failed reads remain visible; container states and image references are interpreted consistently. Restart failures retain bounded stop attribution; quitting warns about pending commands and can wait for completion. Stopping a selected container retains its already-fetched log tail while still selected (Fix engine read ownership, state semantics, poll health and quit handling #126; Quit kills in-flight actions, pulls and service starts without warning #95, Failed polls leave stale or empty lists looking live; show poll health #96, Inspect cache keyed by bare name across panes; key by pane and name #99, Stats poll has no in-flight guard; make it single-flight like containers #100, Inspect cache pins failures and stale results; retry and invalidate on refresh #102, Lists show 'none' while loading or after a failed read; add load state #103, Logs vanish when a container stops; keep them under a stopped marker #107, Engine interprets raw CLI state strings; Client should own container state #111, Networks, stats, inspect and log results skip the stale-result guard #112, Pull guard compares typed name to canonical name; compare canonical forms #116, Failed restart's stop gets announced as external; keep attribution until polled #117, Selection moves spawn uncancelled backlog and inspect reads; abort or debounce #118).
• Message history and command previews can be scrolled; service-start output and filter/prompt cursors remain visible; log rendering is bounded to the viewport. Reduced-motion and ASCII behavior, non-colour focus cues and dim-text contrast are improved (Fix UI scrolling, input, motion and ASCII rendering #127; Message log clips and cannot scroll; give it help's scroll keys #93, Dim text is about 3:1 on every ground; raise it to 4.5:1 #94, Log view rewraps the entire ring each frame, not the window #101, Service-down screen traps the message log; Esc must close overlays there #105, Table layout never draws the filter; typing after / is invisible #106, Reduced motion still animates spinners; render static glyphs instead #108, --ascii still draws Unicode icons; route every glyph through Theme #110, List/detail focus shown only by glyph colour; add a non-colour cue #113, Log ring evicts by shifting a Vec; use a VecDeque ring #114, Prompt input clips past about 40 chars; scroll the field #115).
• Health-banner punctuation follows the ASCII theme for stats errors and failed image/volume/network reads. External Unicode error text remains intact (separate integration correction in this PR).
• Replace yanked chacha20 0.10.1 with 0.10.2, fixing upstream SSE2 backend behavior without other dependency version changes. Upstream: chacha20: fix use of SSE4.1 intrinsic in SSE2 backend RustCrypto/stream-ciphers#580 and chacha20: mark v0.10.0 and v0.10.1 as yanked RustCrypto/stream-ciphers#583.

All four audit PRs are merged and included. Compatibility claims and TESTED_* ranges remain unchanged. Native stopped-container backlog retrieval and macOS 27 support are not claimed.

Regression and validation

The independent coordinator review found stats banners' middle dot and resource-read banners' em dash/middle dot bypassed Theme. Four new render tests failed against the integrated source before the fix (stats, images, volumes, networks). The fix converts only application-owned suffixes through Theme::chrome, leaving external gists untouched. All 24 UI interaction tests then pass, including the external Unicode preservation test. Cases cover Rail/Table at 55x20 and 120x40.

Full exact-head checks passed from a clean checkout with an isolated build target:

• cargo fmt --check
• cargo clippy --locked --all-targets -- -D warnings
• cargo test --locked: 362 passed (209 library, 11 binary, 90 engine, 10 quit, 10 read ownership, 2 README, 6 stop attribution, 24 UI interaction)
• docs/schema/keymap/config checks; docs version 0.3.6
• generated man page and Bash/Zsh/Fish completions; nonempty/content checks and Bash syntax
• binary version: bushel 0.3.6
• cargo publish --dry-run --locked: package verified, no yanked warning, no upload
• official cargo-dist 0.32.0 dist plan --tag=v0.3.6: stable release, expected ARM archive/installers/assets/checksums
• clean source/head/tree and final four-file diff; precise lock invariants

Remote CI passed on the exact final head: https://github.com/frankieramirez/bushel/actions/runs/37610809782 — macOS fmt/clippy/test and Linux docs/assets/publish dry-run jobs all completed successfully. Release planning passed: https://github.com/frankieramirez/bushel/actions/runs/37610810205. CodeRabbit status is success.
Evidence: /tmp/bushel-release-final-ba7e1ae and /tmp/bushel-release-final-9088e0d/health-banners-{red,green}.txt.

Release controls and limits

Draft only. Coordinator reviews the final version PR and explicitly directs tagging/publication. No tag, release, actual cargo upload or deploy hook invocation was performed.

PR events plan only; tag pushes publish. Existing HOMEBREW_TAP_TOKEN is required to update the tap before announcement. BUSHEL_SH_DEPLOY_HOOK absence fails site deployment. CARGO_REGISTRY_TOKEN absence silently skips crates.io upload; same-version manual publication is then required. No credentials or repository settings are created or changed here.

Linux cloud verification cannot exercise Apple Containers virtualization or real interactive terminal restoration. Physical/self-hosted Apple silicon validation is mandatory before expanding the tested CLI range or claiming macOS 27 support. Existing cloud GitHub CLI GH_TOKEN is invalid; connector writes and git work.

Summary by CodeRabbit

• Bug Fixes

  • Health banners now format log hints and last-good-state messages consistently across layouts.

• Tests

  • Expanded coverage for health banners across resource types, layouts, terminal sizes, and ASCII and Unicode error messages.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: 251b1105-90ec-4bca-9df3-42b109398deb

📥 Commits

Reviewing files that changed from the base of the PR and between 0f24599 and ba7e1ae.

⛔ Files ignored due to path filters (1)

• Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (3)

• Cargo.toml
• src/ui/draw.rs
• tests/ui_interaction.rs

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

The package version changes from 0.3.5 to 0.3.6. Health-banner hints now use the chrome formatter. Interaction tests cover the banners across panes, layouts, sizes, and error text.

Changes

Health Banner Formatting

Layer / File(s)	Summary
Banner formatting and coverage src/ui/draw.rs, tests/ui_interaction.rs, Cargo.toml	The stats-unavailable and list-failure banners format their hints through th.chrome. Interaction tests check banner text across panes and layouts at two sizes, including ASCII and Unicode errors. The package version changes to 0.3.6.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to ba7e1

The reviewed changes are mergeable after normal checks; no concrete current-head risk remains identified.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 66.67% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 9 functions across 2 files. (1 skipped: 1…	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly identifies the 0.3.6 release preparation and matches the version update and release-focused changes.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

Docstring coverage is 66.67% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 9 functions across 2 files. (1 skipped: 1 unsupported.)

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{A rabbit checks the banner glow,
The hints now wear their chrome just so.
Through panes and sizes, tests take flight,
ASCII and Unicode read right.
The rabbit hops, pleased with the show.}

_{Comment @coderabbitai help to get the list of available commands.}