docs: add memory consolidation skip ticket
This commit is contained in:
parent
9fb11e25a4
commit
ab73051ddc
3
TODO.md
3
TODO.md
|
|
@ -10,6 +10,9 @@
|
|||
- llm-worker のエラー耐性
|
||||
- ストリーム途中失敗時の継続 → [tickets/llm-worker-stream-continuation.md](tickets/llm-worker-stream-continuation.md)
|
||||
- E2E テストハーネス(`tests/e2e/`、opt-in) → [tickets/e2e-harness.md](tickets/e2e-harness.md)
|
||||
- メモリ機構
|
||||
- consolidation skip 表示と invalid staging の観測性 → [tickets/memory-consolidation-skip-observability.md](tickets/memory-consolidation-skip-observability.md)
|
||||
|
||||
- TUI 拡充
|
||||
- navigation mode / block focus の設計 → [tickets/tui-navigation-mode-design.md](tickets/tui-navigation-mode-design.md)
|
||||
- spawned child Pod の一覧と一時 attach → [tickets/tui-spawned-pod-panel.md](tickets/tui-spawned-pod-panel.md)
|
||||
|
|
|
|||
48
tickets/memory-consolidation-skip-observability.md
Normal file
48
tickets/memory-consolidation-skip-observability.md
Normal file
|
|
@ -0,0 +1,48 @@
|
|||
# メモリ機構: consolidation skip 表示と invalid staging の観測性
|
||||
|
||||
## 背景
|
||||
|
||||
`memory-audit-log` 実装後、TUI actionbar に `memory consolidation skipped: no_staging_entries` が表示されるようになった。これは一見すると「staging が無い」状態に見えるが、実際の workspace には `.insomnia/memory/_staging/*.json` が残っている場合がある。
|
||||
|
||||
現状の原因は二つある。
|
||||
|
||||
1. `run_consolidate_once` が成功後に backlog drain のため loop し、直後の空確認で `skipped: no_staging_entries` を emit する。これにより、直前の `completed_record_changes` 表示が actionbar 上で上書きされる。
|
||||
2. 旧 schema の staging file は `source.session_id` を持ち、現行 schema が要求する `source.segment_id` を持たないため parse で skip される。結果として、ファイルは存在するのに valid entry が 0 件となり、`no_staging_entries` と記録される。
|
||||
|
||||
この状態は機能停止ではないが、観測面として misleading であり、memory 機構の稼働状況を人間が誤解しやすい。
|
||||
|
||||
## 方針
|
||||
|
||||
Audit log には worker の skip / no-op を残しつつ、actionbar には人間が見る価値のある状態だけを出す。特に successful consolidation の直後に発生する drain 終端確認の `no_staging_entries` は actionbar に出さない。
|
||||
|
||||
また、staging directory にファイルがあるが current schema として読めない場合は、`no_staging_entries` ではなく invalid staging が存在することを audit log から分かるようにする。
|
||||
|
||||
## 要件
|
||||
|
||||
- consolidation が `completed` した直後の drain 確認で発生する idle skip が、直前の actionbar 表示を上書きしない。
|
||||
- 例: `completed_record_changes` の直後に `no_staging_entries` を actionbar へ出さない。
|
||||
- audit log へ残すかどうかは実装判断でよいが、UI event としては抑制する。
|
||||
- 通常の post-run check で staging が本当に空の場合も、actionbar に `no_staging_entries` を毎回出さない。
|
||||
- idle/no-op 状態は audit log 側で確認できればよい。
|
||||
- `list_staging_entries` あるいはその呼び出し側で、invalid / parse-failed staging file の存在を区別できるようにする。
|
||||
- 少なくとも invalid count が audit reason または structured field から分かる。
|
||||
- 例: `no_valid_staging_entries invalid=6`。
|
||||
- `threshold_not_reached` と `no_valid_staging_entries` / `invalid_staging_entries` は区別される。
|
||||
- 既存の old schema staging を自動 migration / delete しない。
|
||||
- `source.session_id` と `source.segment_id` は意味が違う可能性があるため、この ticket では観測性改善に留める。
|
||||
- 必要なら後続で archive/drop/migration 方針を決める。
|
||||
|
||||
## 完了条件
|
||||
|
||||
- successful consolidation の直後に actionbar が `no_staging_entries` で上書きされない。
|
||||
- staging directory に parse 不能な `.json` がある場合、audit log が `no_staging_entries` ではなく invalid staging の存在を示す。
|
||||
- staging が本当に空の periodic/post-run check は actionbar にノイズを出さない。
|
||||
- consolidation skip / invalid staging の挙動を確認する test がある。
|
||||
- `cargo fmt --check` と関連 crate の test が通る。
|
||||
|
||||
## 範囲外
|
||||
|
||||
- old schema staging file の自動 migration。
|
||||
- old schema staging file の自動削除 / archive。
|
||||
- actionbar transient notice 汎用 API の実装。
|
||||
- memory audit log の保存形式の大幅変更。
|
||||
Loading…
Reference in New Issue
Block a user