From 76f83a08944b2e866343dcc336f07a0d2c399e62 Mon Sep 17 00:00:00 2001 From: Hare Date: Mon, 11 May 2026 21:31:26 +0900 Subject: [PATCH] review: memory usage metrics --- tickets/memory-usage-metrics.md | 6 +++++ tickets/memory-usage-metrics.review.md | 36 ++++++++++++++++++++++++++ 2 files changed, 42 insertions(+) create mode 100644 tickets/memory-usage-metrics.review.md diff --git a/tickets/memory-usage-metrics.md b/tickets/memory-usage-metrics.md index 1ea2f722..52f66ec9 100644 --- a/tickets/memory-usage-metrics.md +++ b/tickets/memory-usage-metrics.md @@ -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 経路 diff --git a/tickets/memory-usage-metrics.review.md b/tickets/memory-usage-metrics.review.md new file mode 100644 index 00000000..2a07db27 --- /dev/null +++ b/tickets/memory-usage-metrics.review.md @@ -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 として記録される。 +- `#` 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 の観測に留まっている。マージしてよい。