altair823-org/kebab
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 40 Wiki Activity

fix(cli): honor --config flag in kb ask (P4-3 follow-up) #24

Merged
altair823 merged 1 commits from fix/cli-ask-honor-config-flag into main 2026-05-01 16:08:25 +00:00
Conversation 1 Commits 1 Files Changed 1 +2 -1
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
crates/kb-cli/src/main.rs
View File

@@ -320,6 +320,7 @@ fn run(cli: &Cli) -> anyhow::Result<()> {
temperature,
seed,
} => {
let cfg = kb_config::Config::load(cli.config.as_deref())?;
claude-reviewer-01 commented 2026-05-01 15:57:42 +00:00
Review
Copy Link

P3-5 hotfix와 동일 패턴 — Config::load(cli.config.as_deref())로 한 번 빌드 후 *_with_config 변형에 thread. P4-3 시점에 ask body가 추가되었지만 CLI가 여전히 kb_app::ask(no-config) 직접 호출하던 회귀를 닫음. 한 줄 수정인데 사용자 입장에서는 "왜 --config가 무시되지?" 디버깅 비용이 컸던 케이스.

P3-5 hotfix와 동일 패턴 — Config::load(cli.config.as_deref())로 한 번 빌드 후 *_with_config 변형에 thread. P4-3 시점에 ask body가 추가되었지만 CLI가 여전히 `kb_app::ask`(no-config) 직접 호출하던 회귀를 닫음. 한 줄 수정인데 사용자 입장에서는 "왜 --config가 무시되지?" 디버깅 비용이 컸던 케이스.
let opts = kb_app::AskOpts {
k: *k,
explain: *explain,
@@ -331,7 +332,7 @@ fn run(cli: &Cli) -> anyhow::Result<()> {
// wires up a real `mpsc::Sender` here.
stream_sink: None,
};
let ans = kb_app::ask(query, opts)?;
let ans = kb_app::ask_with_config(cfg, query, opts)?;
if cli.json {
println!("{}", serde_json::to_string(&wire::wire_answer(&ans))?);
} else {