dgf1988/codewhale
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

fix(tui): harvest final v0.8.40 polish

Browse Source
This commit is contained in:
Hunter Bown
2026-05-21 08:45:04 +08:00
parent b639740b73
commit 67fb788110
5 changed files with 70 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
CHANGELOG.md
+22 -3
View File
@@ -93,6 +93,18 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- **The Feishu/Lark bridge recovers better after restarts.** It now reattaches
to persisted active turns after the long-connection client starts, and text
chunking no longer splits emoji or other multi-code-unit characters.
- **RLM survives non-UTF-8 stdout.** `rlm_eval` now decodes REPL stdout
lossily instead of treating a single invalid byte as a fatal crash, so
binary-adjacent diagnostics can still return a bounded result (#1815,
#1819).
- **Small UI/review reliability fixes landed with the stability branch.**
`/clear` now resets all displayed cost state, grayscale theme previews avoid
luma overflow, `/theme` picker arrow navigation wraps at the list edges, and
encoded JSON review output is parsed before display.
- **New-file writes execute on the first Agent-mode call.** `write_file` now
stays preloaded in Agent mode, so creating a file no longer stops at the
deferred-tool schema hydration message before the normal approval/execution
path (#1825, #1841).
### Changed
@@ -101,6 +113,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
`/clear` (which only matches by the longer pinyin alias `qingping`).
Within each rank tier the menu still falls back to alphabetical name
order for deterministic display (#1811).
- **CNB mirror preflight covers stability-release branches.** The CNB sync
path now recognizes the v0.8.40 stability branch shape before release tags
exist, making the Tencent Lighthouse/Lark deployment path easier to verify
before publishing.
### Thanks
@@ -116,8 +132,9 @@ in #1740, #1743, #1742, and #1744. Thanks to **Nightt
([@nightt5879](https://github.com/nightt5879))** for the Ctrl+C prompt restore
fix in #1764. Thanks to **Ling ([@LING71671](https://github.com/LING71671);
commits as `www17 <ivonrust@gmail.com>`)** for the configurable sub-agent API
timeout in #1808, harvested with `1..=1800` clamping and a fail-fast guard
so a stray `api_timeout_secs = 0` keeps the legacy 120-second default.
timeout in #1808 and the Agent-mode `write_file` preload fix in #1841,
harvested with `1..=1800` clamping and a fail-fast guard so a stray
`api_timeout_secs = 0` keeps the legacy 120-second default.
Thanks to **[@knqiufan](https://github.com/knqiufan)** for the sub-agent
file-write delegation work in #1833, harvested with structured approval-
gate semantics (`Implementer` and `Custom` only, never `Required`-level
@@ -125,7 +142,9 @@ tools) so write-capable children can actually land code without bypassing
the `Required` approval class. Thanks to **[@IIzzaya](https://github.com/IIzzaya)**
for the exact-alias-first slash-completion ordering idea in #1811, landed
with a focused regression test. Thanks to **Bevis** and the community reports
that surfaced the compaction failure mode addressed in this release.
that surfaced the compaction failure mode addressed in this release. Thanks to
**Reid ([@reidliu41](https://github.com/reidliu41))** for the grayscale theme
overflow report and `/theme` picker edge-wrapping patch in #1814.
## [0.8.39] - 2026-05-17
crates/tui/CHANGELOG.md
+22 -3
View File
@@ -93,6 +93,18 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- **The Feishu/Lark bridge recovers better after restarts.** It now reattaches
to persisted active turns after the long-connection client starts, and text
chunking no longer splits emoji or other multi-code-unit characters.
- **RLM survives non-UTF-8 stdout.** `rlm_eval` now decodes REPL stdout
lossily instead of treating a single invalid byte as a fatal crash, so
binary-adjacent diagnostics can still return a bounded result (#1815,
#1819).
- **Small UI/review reliability fixes landed with the stability branch.**
`/clear` now resets all displayed cost state, grayscale theme previews avoid
luma overflow, `/theme` picker arrow navigation wraps at the list edges, and
encoded JSON review output is parsed before display.
- **New-file writes execute on the first Agent-mode call.** `write_file` now
stays preloaded in Agent mode, so creating a file no longer stops at the
deferred-tool schema hydration message before the normal approval/execution
path (#1825, #1841).
### Changed
@@ -101,6 +113,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
`/clear` (which only matches by the longer pinyin alias `qingping`).
Within each rank tier the menu still falls back to alphabetical name
order for deterministic display (#1811).
- **CNB mirror preflight covers stability-release branches.** The CNB sync
path now recognizes the v0.8.40 stability branch shape before release tags
exist, making the Tencent Lighthouse/Lark deployment path easier to verify
before publishing.
### Thanks
@@ -116,8 +132,9 @@ in #1740, #1743, #1742, and #1744. Thanks to **Nightt
([@nightt5879](https://github.com/nightt5879))** for the Ctrl+C prompt restore
fix in #1764. Thanks to **Ling ([@LING71671](https://github.com/LING71671);
commits as `www17 <ivonrust@gmail.com>`)** for the configurable sub-agent API
timeout in #1808, harvested with `1..=1800` clamping and a fail-fast guard
so a stray `api_timeout_secs = 0` keeps the legacy 120-second default.
timeout in #1808 and the Agent-mode `write_file` preload fix in #1841,
harvested with `1..=1800` clamping and a fail-fast guard so a stray
`api_timeout_secs = 0` keeps the legacy 120-second default.
Thanks to **[@knqiufan](https://github.com/knqiufan)** for the sub-agent
file-write delegation work in #1833, harvested with structured approval-
gate semantics (`Implementer` and `Custom` only, never `Required`-level
@@ -125,7 +142,9 @@ tools) so write-capable children can actually land code without bypassing
the `Required` approval class. Thanks to **[@IIzzaya](https://github.com/IIzzaya)**
for the exact-alias-first slash-completion ordering idea in #1811, landed
with a focused regression test. Thanks to **Bevis** and the community reports
that surfaced the compaction failure mode addressed in this release.
that surfaced the compaction failure mode addressed in this release. Thanks to
**Reid ([@reidliu41](https://github.com/reidliu41))** for the grayscale theme
overflow report and `/theme` picker edge-wrapping patch in #1814.
## [0.8.39] - 2026-05-17
crates/tui/src/core/engine/tests.rs
+3
View File
@@ -401,6 +401,7 @@ fn non_yolo_mode_retains_default_defer_policy() {
assert!(!should_default_defer_tool("exec_shell", AppMode::Agent));
assert!(should_default_defer_tool("exec_shell", AppMode::Plan));
assert!(!should_default_defer_tool("read_file", AppMode::Agent));
assert!(!should_default_defer_tool("write_file", AppMode::Agent));
assert!(should_default_defer_tool(
"mcp_read_resource",
AppMode::Agent
@@ -412,6 +413,7 @@ fn model_tool_catalog_applies_native_and_mcp_deferral() {
let catalog = build_model_tool_catalog(
vec![
api_tool("read_file"),
api_tool("write_file"),
api_tool("exec_shell"),
api_tool("project_map"),
],
@@ -427,6 +429,7 @@ fn model_tool_catalog_applies_native_and_mcp_deferral() {
};
assert_eq!(defer_loading("read_file"), Some(false));
assert_eq!(defer_loading("write_file"), Some(false));
assert_eq!(defer_loading("exec_shell"), Some(false));
assert_eq!(defer_loading("project_map"), Some(true));
assert_eq!(defer_loading("list_mcp_resources"), Some(false));
crates/tui/src/core/engine/tool_catalog.rs
+1
View File
@@ -54,6 +54,7 @@ pub(super) fn should_default_defer_tool(name: &str, mode: AppMode) -> bool {
!matches!(
name,
"read_file"
| "write_file"
| "list_dir"
| "grep_files"
| "file_search"
crates/tui/src/tui/theme_picker.rs
+22 -4
View File
@@ -90,15 +90,22 @@ impl ThemePickerView {
}
fn move_up(&mut self) {
if self.selected > 0 {
let len = SELECTABLE_THEMES.len();
if len == 0 {
self.selected = 0;
} else if self.selected == 0 {
self.selected = len - 1;
} else {
self.selected -= 1;
}
}
fn move_down(&mut self) {
let max = SELECTABLE_THEMES.len().saturating_sub(1);
if self.selected < max {
self.selected += 1;
let len = SELECTABLE_THEMES.len();
if len == 0 {
self.selected = 0;
} else {
self.selected = (self.selected + 1) % len;
}
}
}
@@ -313,6 +320,17 @@ mod tests {
assert_eq!(selected_name(&action), Some(ThemeId::Whale.name()));
}
#[test]
fn arrow_navigation_wraps_at_picker_edges() {
let mut v = ThemePickerView::new("system".to_string());
let action = v.handle_key(key(KeyCode::Up));
assert_eq!(selected_name(&action), Some(ThemeId::GruvboxDark.name()));
let action = v.handle_key(key(KeyCode::Down));
assert_eq!(selected_name(&action), Some(ThemeId::System.name()));
}
#[test]
fn enter_commits_with_persist_true() {
let mut v = ThemePickerView::new("system".to_string());