feat(kebab-core + kebab-store-sqlite): p9-fb-17 chat session storage (V005) #80

Merged

altair823 merged 2 commits from feat/p9-fb-17-chat-storage into main

2026-05-03 05:41:01 +00:00

Author	SHA1	Message	Date
altair823	952ed1615f	review(p9-fb-17): 회차 1 nit 반영 - `append_turn` 의 doc 은 "wrap in one transaction" 보장하지만 실제 코드는 auto-commit `conn.execute` 두 번이라 두번째 실패 시 first row 가 commit 된 채 inconsistent 됨. 진짜 transaction 으로 교체: `conn.transaction()` → `tx.execute(insert)` → `tx.execute(update)` → `tx.commit()`. SQLite BEGIN 으로 감싸 두 statement atomic. `lock_conn()` 이 `MutexGuard<Connection>` 반환하므로 `let mut conn = self.lock_conn(); let tx = conn.transaction()` 패턴 가능 (MutexGuard 의 DerefMut 활용). 9 chat_sessions 테스트 + clippy 통과. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-03 05:40:40 +00:00
altair823	c97e8e00ef	feat(kebab-core + kebab-store-sqlite): p9-fb-17 chat session storage (V005) 도그푸딩 item 13/14 (multi-turn 영속화) — TUI Ask 의 "이전 대화 이어가기" + 향후 CLI `--session foo` (p9-fb-18) backing store. session header + per-turn 두 테이블, ON DELETE CASCADE 로 reset --data-only 가 한꺼번에 wipe. ## 핵심 변경 - SQLite V005 migration `chat_sessions` (session_id PK + created_at + updated_at + title + config_snapshot_json) + `chat_turns` (turn_id PK + session_id FK ON DELETE CASCADE + turn_index + question + answer + citations_json + created_at + UNIQUE(session_id, turn_index)) + `idx_chat_turns_session(session_id, turn_index)`. 모두 `STRICT`. - `kebab_core::ChatSessionRepo` trait (6 method): create_session / get_session / list_sessions(limit, ORDER BY updated_at DESC) / delete_session / append_turn / list_turns(ORDER BY turn_index ASC) - `kebab_core::{ChatSessionRow, ChatTurnRow}` structs — Serialize + Deserialize 둘 다 (CLI / wire 출력 호환) - `kebab-store-sqlite::SqliteStore` impl 신규 모듈 `chat_sessions.rs`. `append_turn` 이 insert + parent updated_at bump 같은 connection 에서 처리. - frozen design §5 에 §5.7a chat_sessions / chat_turns 절 신설 (full schema + trait 메서드 6 개 명시). ## HOTFIXES (V004 → V005) spec p9-fb-17 의 `V004__chat_sessions.sql` 가 p9-fb-19 의 `V004__kv.sql` (이미 머지) 와 refinery migration number 충돌. 무중단 정정: `V005__chat_sessions.sql` 로 시프트. schema / 동작 동일, 파일명 만 이동. HOTFIXES entry 추가. ## 테스트 - 9 신규 integration unit (create/get roundtrip, missing→None, PK collision error, append+list ordered, dup turn_index error, append bumps updated_at, delete CASCADE turns, list_sessions ORDER BY updated_at DESC, list_sessions LIMIT) - workspace 전체 `cargo test --workspace --no-fail-fast -j 1` exit 0 - `cargo clippy --workspace --all-targets -- -D warnings` clean ## 문서 - frozen design §5.7a 신설 - HANDOFF: 2026-05-03 entry - HOTFIXES: V004 → V005 rename rationale - spec status planned → in_progress ## Out of scope - session 검색 / 필터 UI (p9-fb-18 의 `kebab ask --session list` 같은 admin command 가 후속) - 다른 store backend (postgres 등) — trait 만 정의, impl 은 SQLite unblocks p9-fb-18 (CLI session/repl). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-05-03 05:37:53 +00:00

Author

SHA1

Message

Date

altair823

952ed1615f

review(p9-fb-17): 회차 1 nit 반영

- `append_turn` 의 doc 은 "wrap in one transaction" 보장하지만 실제
  코드는 auto-commit `conn.execute` 두 번이라 두번째 실패 시 first
  row 가 commit 된 채 inconsistent 됨. 진짜 transaction 으로 교체:
  `conn.transaction()` → `tx.execute(insert)` → `tx.execute(update)`
  → `tx.commit()`. SQLite BEGIN 으로 감싸 두 statement atomic.

`lock_conn()` 이 `MutexGuard<Connection>` 반환하므로 `let mut conn =
self.lock_conn(); let tx = conn.transaction()` 패턴 가능 (MutexGuard
의 DerefMut 활용).

9 chat_sessions 테스트 + clippy 통과.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

2026-05-03 05:40:40 +00:00

altair823

c97e8e00ef

feat(kebab-core + kebab-store-sqlite): p9-fb-17 chat session storage (V005)

도그푸딩 item 13/14 (multi-turn 영속화) — TUI Ask 의 "이전 대화
이어가기" + 향후 CLI `--session foo` (p9-fb-18) backing store. session
header + per-turn 두 테이블, ON DELETE CASCADE 로 reset --data-only 가
한꺼번에 wipe.

## 핵심 변경

- **SQLite V005 migration** `chat_sessions` (session_id PK + created_at
  + updated_at + title + config_snapshot_json) + `chat_turns` (turn_id
  PK + session_id FK ON DELETE CASCADE + turn_index + question +
  answer + citations_json + created_at + UNIQUE(session_id, turn_index))
  + `idx_chat_turns_session(session_id, turn_index)`. 모두 `STRICT`.
- **`kebab_core::ChatSessionRepo`** trait (6 method): create_session /
  get_session / list_sessions(limit, ORDER BY updated_at DESC) /
  delete_session / append_turn / list_turns(ORDER BY turn_index ASC)
- **`kebab_core::{ChatSessionRow, ChatTurnRow}`** structs — Serialize
  + Deserialize 둘 다 (CLI / wire 출력 호환)
- **`kebab-store-sqlite::SqliteStore`** impl 신규 모듈 `chat_sessions.rs`.
  `append_turn` 이 insert + parent updated_at bump 같은 connection
  에서 처리.
- **frozen design §5** 에 §5.7a chat_sessions / chat_turns 절 신설
  (full schema + trait 메서드 6 개 명시).

## HOTFIXES (V004 → V005)

spec p9-fb-17 의 `V004__chat_sessions.sql` 가 p9-fb-19 의
`V004__kv.sql` (이미 머지) 와 refinery migration number 충돌. 무중단
정정: `V005__chat_sessions.sql` 로 시프트. schema / 동작 동일, 파일명
만 이동. HOTFIXES entry 추가.

## 테스트

- 9 신규 integration unit (create/get roundtrip, missing→None, PK
  collision error, append+list ordered, dup turn_index error,
  append bumps updated_at, delete CASCADE turns, list_sessions
  ORDER BY updated_at DESC, list_sessions LIMIT)
- workspace 전체 `cargo test --workspace --no-fail-fast -j 1` exit 0
- `cargo clippy --workspace --all-targets -- -D warnings` clean

## 문서

- frozen design §5.7a 신설
- HANDOFF: 2026-05-03 entry
- HOTFIXES: V004 → V005 rename rationale
- spec status planned → in_progress

## Out of scope

- session 검색 / 필터 UI (p9-fb-18 의 `kebab ask --session list`
  같은 admin command 가 후속)
- 다른 store backend (postgres 등) — trait 만 정의, impl 은 SQLite

unblocks p9-fb-18 (CLI session/repl).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

2026-05-03 05:37:53 +00:00

feat(kebab-core + kebab-store-sqlite): p9-fb-17 chat session storage (V005) #80

2 Commits