review(회차1): 회차 1 지적 5건 반영
- p9-dogfooding-feedback.md item 14: README 오타 (READE → README) - p9-fb-11.md frontmatter: depends_on=[p9-fb-14] 추가 (14.unblocks 와 양방향 정합) - p9-fb-01.md Behavior contract: '14 번과 wiring' 모호 cross-ref 정정 — cancel wiring 은 p9-fb-04, TUI 신호는 p9-fb-03 - plan File Structure: 'tasks/HOTFIXES.md — n/a (skip)' 자기모순 제거 → 별도 HOTFIXES 절로 분리 - plan task 4 handler: let _ = data_only; 제거, pattern binding 자체를 data_only: _ 로 변경 (관용적) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -24,10 +24,11 @@
|
||||
- `crates/kebab-cli/src/main.rs` — add `Cmd::Reset` arm + handler
|
||||
- `crates/kebab-cli/src/wire.rs` — `wire_reset` helper
|
||||
- `README.md` — `kebab reset` in 명령 표 + Quick start safety note
|
||||
- `tasks/HOTFIXES.md` — n/a (new feature, not deviation; skip)
|
||||
|
||||
**Delete:** none.
|
||||
|
||||
**HOTFIXES:** n/a — 신규 기능이지 deviation 아님. `tasks/HOTFIXES.md` 는 건드리지 않음.
|
||||
|
||||
---
|
||||
|
||||
## Task 1: `kebab-store-sqlite::truncate_embedding_records`
|
||||
@@ -728,12 +729,16 @@ In `fn run(cli: &Cli) -> anyhow::Result<()>`, just above the `Cmd::Doctor =>` ar
|
||||
```rust
|
||||
Cmd::Reset {
|
||||
all,
|
||||
data_only,
|
||||
data_only: _,
|
||||
vector_only,
|
||||
config_only,
|
||||
yes,
|
||||
} => {
|
||||
use kebab_app::ResetScope;
|
||||
// `--data-only` explicit OR no scope flag at all → DataOnly.
|
||||
// The `data_only: _` binding above is intentional — clap's
|
||||
// `group = "reset_scope"` already enforces mutual exclusion,
|
||||
// so the flag's presence does not change the resolved scope.
|
||||
let scope = if *all {
|
||||
ResetScope::All
|
||||
} else if *vector_only {
|
||||
@@ -741,8 +746,6 @@ In `fn run(cli: &Cli) -> anyhow::Result<()>`, just above the `Cmd::Doctor =>` ar
|
||||
} else if *config_only {
|
||||
ResetScope::ConfigOnly
|
||||
} else {
|
||||
// `--data-only` explicit OR no scope flag at all → DataOnly
|
||||
let _ = data_only;
|
||||
ResetScope::DataOnly
|
||||
};
|
||||
|
||||
|
||||
@@ -220,7 +220,7 @@ search / ask 의 query input box 에 모든 키가 입력으로 들어가 e=expl
|
||||
- `--json` 모드 호환: `answer.v1` 에 `conversation_id` / `turn_index` 필드 (13 번에서 추가) — 외부 도구가 session 추적 가능.
|
||||
- **외부 AI 통합 효과** (README 의 외부 AI 섹션): Claude Code skill / MCP server 도 `--session` 으로 conversation context 보존. 이 부분이 multi-turn CLI 의 진짜 가치 — 내장 TUI 만 쓰는 사용자보다 외부 wrapper 사용자가 큼.
|
||||
|
||||
**spec 영향**: §7 RAG 절 multi-turn 정책 + §externalAI 통합 절 (READE 와 ARCHITECTURE 동기화) 에 session 모델 추가. CLI flag 표 (`--session` / `--repl`) README 갱신.
|
||||
**spec 영향**: §7 RAG 절 multi-turn 정책 + §externalAI 통합 절 (README 와 ARCHITECTURE 동기화) 에 session 모델 추가. CLI flag 표 (`--session` / `--repl`) README 갱신.
|
||||
|
||||
### 15. search 결과 캐싱 (incremental invalidation)
|
||||
|
||||
|
||||
@@ -56,7 +56,7 @@ pub fn ingest_with_config_progress(
|
||||
|
||||
- progress event 발신은 best-effort. receiver drop 되면 이후 send 무시 (panic 금지).
|
||||
- 이벤트 ordering: `ScanStarted < ScanCompleted < (AssetStarted < AssetFinished)* < Completed|Aborted`. embed batch 는 asset 사이 임의 위치.
|
||||
- `Aborted` 는 cancellation token (p9-fb-04) trigger 시. 혼자 발생 X — 14 번과 wiring.
|
||||
- `Aborted` 이벤트는 cancellation token (p9-fb-04) trigger 시에만 발신. CLI / TUI 의 cancel 신호 wiring 은 각각 p9-fb-04, p9-fb-03 에서 구현.
|
||||
- `--json` CLI 는 line-delimited 형태로 dump (`schema_version=ingest_progress.v1`) — 별도 task (p9-fb-02).
|
||||
|
||||
## Test plan
|
||||
|
||||
@@ -4,7 +4,7 @@ component: kebab-tui (ask pane)
|
||||
task_id: p9-fb-11
|
||||
title: "Ask answer markdown rendering (bold/italic/code/list/table)"
|
||||
status: planned
|
||||
depends_on: []
|
||||
depends_on: [p9-fb-14]
|
||||
unblocks: []
|
||||
contract_source: ../../docs/superpowers/specs/2026-04-27-kebab-final-form-design.md
|
||||
contract_sections: [§7 RAG, §10 UX]
|
||||
|
||||
Reference in New Issue
Block a user