review: memory usage metrics

This commit is contained in:
Keisuke Hirata 2026-05-11 21:31:26 +09:00
parent da4f4cc954
commit 9e615a41f0
2 changed files with 42 additions and 0 deletions

View File

@ -132,6 +132,12 @@ estimated_total_resident_exposure_tokens
- consolidation / Doctor 側から report API を呼べる
- `n/Mtoken` や session 浮上率を Knowledge 化候補や tidy protection の主判断として使っていない
## レビュー状態
- `d581a35 feat: add memory usage event metrics` を review 済み。結果は `tickets/memory-usage-metrics.review.md`
- 指摘 1 件tidy hints の古い未接続文言)は amend 済み。
- 判断: approve / merge 可。
## 参照
- `docs/plan/memory.md` §使用頻度メトリクス / §判断ルール / §retrieval 経路

View File

@ -0,0 +1,36 @@
# Review: Memory usage metrics
## 対象
- Ticket: `tickets/memory-usage-metrics.md`
- Branch: `memory-usage-metrics`
- Reviewed commit: `d581a35 feat: add memory usage event metrics`
## 確認内容
- 明示使用回数ログは workspace-local append-only JSONL として `.insomnia/memory/_usage/events.jsonl` に保存される。
- `MemoryRead``use` event として記録される。
- `#<slug>` Knowledge ref は解決成功時に `KnowledgeRef` source の `use` event として記録される。
- `/workflow` invocation は workflow record の `WorkflowInvoke` source の `use` event として記録される。
- `MemoryQuery` / `KnowledgeQuery` は usage を記録せず、検索 hit を使用回数に含めていない。
- resident injection は `resident_exposure` として `use_count` から分離されている。
- report は record ごとの `use_count`, `last_used_at`, `source_breakdown`, `resident_exposure_count`, `estimated_tokens_per_injection`, `estimated_total_resident_exposure_tokens` を返す。
- consolidation input は usage report を evidence として渡し、Knowledge 化や tidy protection の hard decision は実装していない。
- 旧方針の `n/Mtoken`, cumulative-token window, session 浮上率, frequency threshold 判定は入っていない。
## 指摘と対応
- 指摘: tidy hints に「Explicit-invoke metrics が未接続」という古い文言が残っており、今回の evidence report 接続と矛盾していた。
- 対応: 同 commit を amend し、usage evidence は soft context として扱う説明へ修正した。
## 検証
- `cargo test -p memory` passed
- `cargo test -p pod` passed
- `cargo fmt --check` は既存の unrelated rustfmt 差分により失敗するが、今回変更ファイルは対象 diff に含まれていない。
## 判断
Approve.
チケットの簡略化後仕様に対して、実装は過剰な frequency / session-rate 判定を持ち込まず、明示使用回数と resident exposure cost の観測に留まっている。マージしてよい。