feat(auto-reasoning): recognise CJK debug/search keywords in Auto tier
`auto_reasoning::select` is the per-turn classifier that picks `reasoning_effort` for `reasoning_effort = "auto"`. Today it only recognises English keywords (`debug`, `error`, `search`, `lookup`), so a user typing in Chinese or Japanese never trips the tier shifts: "帮我调试代码" stays on `High` instead of escalating to `Max`, "搜索一下文件" stays on `High` instead of dropping to `Low`. For a non-English Auto-mode user that's both wrong-side-of-cheap and wrong-side-of-careful on every turn. Extracts the keyword sets into `HIGH_EFFORT_KEYWORDS` and `LOW_EFFORT_KEYWORDS` constants and adds the Chinese / Japanese vocabulary that maps to the same intents: * HIGH (→ `Max`): 调试 / 错误 / 报错 / 出错 / 崩溃 / 調試 / 錯誤 in Chinese; デバッグ / エラー / バグ in Japanese. * LOW (→ `Low`): 搜索 / 查找 / 查询 in Chinese; 検索 in Japanese. Latin lowercase is preserved (the caller still lowercases the message), and CJK matches the literal form because CJK has no case. Four new tests cover Chinese debug keywords, Japanese debug keywords, Chinese search keywords, the single Japanese search keyword, and a sanity test that ordinary CJK prose (without keyword hits) still returns `High` — matching the English-only behaviour the function already had. All previous tests (`subagent_returns_low`, `debug_or_error_returns_max`, `search_or_lookup_returns_low`, `default_returns_high`) continue to pass — the original English-only paths are unchanged. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
LinQ
@@ -10,8 +10,12 @@ use crate::tui::app::ReasoningEffort;
|
||||
///
|
||||
/// Rules:
|
||||
/// - Sub-agent contexts (`is_subagent == true`) → `Low`
|
||||
/// - Last user message contains `"debug"` or `"error"` → `Max`
|
||||
/// - Last user message contains `"search"` or `"lookup"` → `Low`
|
||||
/// - Last user message contains a high-effort keyword
|
||||
/// (English: `debug`, `error`; Chinese: 调试 / 错误 / 报错 / 出错 /
|
||||
/// 崩溃 / 調試 / 錯誤; Japanese: デバッグ / エラー / バグ) → `Max`
|
||||
/// - Last user message contains a low-effort keyword
|
||||
/// (English: `search`, `lookup`; Chinese: 搜索 / 查找 / 查询;
|
||||
/// Japanese: 検索) → `Low`
|
||||
/// - Everything else → `High`
|
||||
#[must_use]
|
||||
pub fn select(is_subagent: bool, last_msg: &str) -> ReasoningEffort {
|
||||
@@ -21,17 +25,54 @@ pub fn select(is_subagent: bool, last_msg: &str) -> ReasoningEffort {
|
||||
|
||||
let lower = last_msg.to_ascii_lowercase();
|
||||
|
||||
if lower.contains("debug") || lower.contains("error") {
|
||||
if HIGH_EFFORT_KEYWORDS.iter().any(|kw| lower.contains(kw)) {
|
||||
return ReasoningEffort::Max;
|
||||
}
|
||||
|
||||
if lower.contains("search") || lower.contains("lookup") {
|
||||
if LOW_EFFORT_KEYWORDS.iter().any(|kw| lower.contains(kw)) {
|
||||
return ReasoningEffort::Low;
|
||||
}
|
||||
|
||||
ReasoningEffort::High
|
||||
}
|
||||
|
||||
/// Keywords that bump `reasoning_effort` to `Max`. Latin terms are
|
||||
/// lowercase because the caller lowercases the message; CJK has no
|
||||
/// case so the literal form matches as-is. Covers the Chinese and
|
||||
/// Japanese vocabulary a non-English user reaches for when reporting
|
||||
/// the same kind of problem the original `"debug" | "error"` rule was
|
||||
/// trying to catch — without those terms a Chinese-speaking user
|
||||
/// paying for Auto mode silently got `High` even on hard debugging
|
||||
/// tasks.
|
||||
const HIGH_EFFORT_KEYWORDS: &[&str] = &[
|
||||
// English (unchanged from the original keyword set).
|
||||
"debug",
|
||||
"error",
|
||||
// Simplified / Traditional Chinese.
|
||||
"\u{8c03}\u{8bd5}", // 调试
|
||||
"\u{9519}\u{8bef}", // 错误
|
||||
"\u{62a5}\u{9519}", // 报错
|
||||
"\u{51fa}\u{9519}", // 出错
|
||||
"\u{5d29}\u{6e83}", // 崩溃
|
||||
"\u{8abf}\u{8a66}", // 調試
|
||||
"\u{932f}\u{8aa4}", // 錯誤
|
||||
// Japanese.
|
||||
"\u{30c7}\u{30d0}\u{30c3}\u{30b0}", // デバッグ
|
||||
"\u{30a8}\u{30e9}\u{30fc}", // エラー
|
||||
"\u{30d0}\u{30b0}", // バグ
|
||||
];
|
||||
|
||||
/// Keywords that drop `reasoning_effort` to `Low`. Same locale coverage
|
||||
/// as [`HIGH_EFFORT_KEYWORDS`].
|
||||
const LOW_EFFORT_KEYWORDS: &[&str] = &[
|
||||
"search",
|
||||
"lookup",
|
||||
"\u{641c}\u{7d22}", // 搜索
|
||||
"\u{67e5}\u{627e}", // 查找
|
||||
"\u{67e5}\u{8be2}", // 查询
|
||||
"\u{691c}\u{7d22}", // 検索
|
||||
];
|
||||
|
||||
#[cfg(test)]
|
||||
mod tests {
|
||||
use super::*;
|
||||
@@ -67,4 +108,84 @@ mod tests {
|
||||
assert_eq!(select(false, "refactor this module"), ReasoningEffort::High);
|
||||
assert_eq!(select(false, ""), ReasoningEffort::High);
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn chinese_debug_keywords_return_max() {
|
||||
// The original keyword set was English-only; Chinese-speaking
|
||||
// Auto-mode users paid for `High` even on real debugging tasks.
|
||||
for msg in [
|
||||
"\u{5e2e}\u{6211}\u{8c03}\u{8bd5}\u{4ee3}\u{7801}", // 帮我调试代码
|
||||
"\u{8fd9}\u{91cc}\u{6709}\u{4e2a}\u{9519}\u{8bef}", // 这里有个错误
|
||||
"\u{4ee3}\u{7801}\u{62a5}\u{9519}\u{4e86}", // 代码报错了
|
||||
"\u{7a0b}\u{5e8f}\u{51fa}\u{9519}", // 程序出错
|
||||
"\u{7cfb}\u{7edf}\u{5d29}\u{6e83}", // 系统崩溃
|
||||
"\u{4ee3}\u{78bc}\u{8abf}\u{8a66}", // 代碼調試 (zh-Hant)
|
||||
"\u{6709}\u{500b}\u{932f}\u{8aa4}", // 有個錯誤 (zh-Hant)
|
||||
] {
|
||||
assert_eq!(
|
||||
select(false, msg),
|
||||
ReasoningEffort::Max,
|
||||
"expected Max for `{msg}`",
|
||||
);
|
||||
}
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn japanese_debug_keywords_return_max() {
|
||||
for msg in [
|
||||
"\u{30b3}\u{30fc}\u{30c9}\u{3092}\u{30c7}\u{30d0}\u{30c3}\u{30b0}", // コードをデバッグ
|
||||
"\u{30a8}\u{30e9}\u{30fc}\u{304c}\u{51fa}\u{305f}", // エラーが出た
|
||||
"\u{30d0}\u{30b0}\u{3092}\u{4fee}\u{6b63}", // バグを修正
|
||||
] {
|
||||
assert_eq!(
|
||||
select(false, msg),
|
||||
ReasoningEffort::Max,
|
||||
"expected Max for `{msg}`",
|
||||
);
|
||||
}
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn chinese_search_keywords_return_low() {
|
||||
for msg in [
|
||||
"\u{641c}\u{7d22}\u{4e00}\u{4e0b}\u{6587}\u{4ef6}", // 搜索一下文件
|
||||
"\u{5e2e}\u{6211}\u{67e5}\u{627e}\u{5b9a}\u{4e49}", // 帮我查找定义
|
||||
"\u{67e5}\u{8be2}\u{6587}\u{6863}", // 查询文档
|
||||
] {
|
||||
assert_eq!(
|
||||
select(false, msg),
|
||||
ReasoningEffort::Low,
|
||||
"expected Low for `{msg}`",
|
||||
);
|
||||
}
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn japanese_search_keyword_returns_low() {
|
||||
// 検索 → "search"
|
||||
assert_eq!(
|
||||
select(
|
||||
false,
|
||||
"\u{30c9}\u{30ad}\u{30e5}\u{30e1}\u{30f3}\u{30c8}\u{691c}\u{7d22}"
|
||||
),
|
||||
ReasoningEffort::Low,
|
||||
);
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn cjk_default_still_returns_high() {
|
||||
// No keyword hits — ordinary Chinese/Japanese prose stays on
|
||||
// the `High` default like English does.
|
||||
for msg in [
|
||||
"\u{5e2e}\u{6211}\u{5199}\u{4e2a}\u{6d4b}\u{8bd5}", // 帮我写个测试
|
||||
"\u{91cd}\u{6784}\u{8fd9}\u{4e2a}\u{6a21}\u{5757}", // 重构这个模块
|
||||
"\u{30c6}\u{30b9}\u{30c8}\u{3092}\u{66f8}\u{304f}", // テストを書く
|
||||
] {
|
||||
assert_eq!(
|
||||
select(false, msg),
|
||||
ReasoningEffort::High,
|
||||
"expected High for `{msg}`",
|
||||
);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user