From e0c4dbdc7335989a02fe72443e689772284f481d Mon Sep 17 00:00:00 2001 From: Hare Date: Sun, 26 Apr 2026 23:30:46 +0900 Subject: [PATCH] =?UTF-8?q?notification-naming=E5=AE=8C=E4=BA=86?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- TODO.md | 1 - tickets/notification-naming-cleanup.md | 70 ------------------- tickets/notification-naming-cleanup.review.md | 42 ----------- 3 files changed, 113 deletions(-) delete mode 100644 tickets/notification-naming-cleanup.md delete mode 100644 tickets/notification-naming-cleanup.review.md diff --git a/TODO.md b/TODO.md index 2867aad1..9a3ff87f 100644 --- a/TODO.md +++ b/TODO.md @@ -11,7 +11,6 @@ - [ ] サブミット入力 - [ ] protocol Segment 化 → [tickets/submit-segment-protocol.md](tickets/submit-segment-protocol.md) - [ ] TUI 補完 + 型付き atom 化 → [tickets/submit-tui-completion.md](tickets/submit-tui-completion.md) -- [ ] Notification 用語リネーム (Event::Alert / Method::Notify) → [tickets/notification-naming-cleanup.md](tickets/notification-naming-cleanup.md) - [ ] メモリ機構 - [ ] ファイル形式 + Linter 土台 → [tickets/memory-file-format.md](tickets/memory-file-format.md) - [ ] memory / Knowledge 検索ツール → [tickets/memory-search-tools.md](tickets/memory-search-tools.md) diff --git a/tickets/notification-naming-cleanup.md b/tickets/notification-naming-cleanup.md deleted file mode 100644 index 71b1579e..00000000 --- a/tickets/notification-naming-cleanup.md +++ /dev/null @@ -1,70 +0,0 @@ -# Notification 用語のリネーム - -## 背景 - -Pod 周辺に二系統の「notification」があり、用語が衝突して読みづらい: - -- **outbound** (Pod → Client): `Event::Notification` / `Notifier::notify` — ユーザー向けの運用診断（compaction 失敗、tool 出力 truncated 等） -- **inbound** (External → Pod): `Method::Notify` / `NotificationBuffer` — 外部から Pod の LLM context に「メモを置く」経路 - -両者とも `Notification` / `Notify` を共有しており、コードを読む側が毎回どちらの経路かを文脈から判別する必要がある。 - -`Method::Notify` の動詞名は「呼び出し側がやる行為」として意味的に正しいので残し、**outbound 側の名詞**を `Alert` に倒して非対称を入れることで区別する。 - -## 要件 - -### outbound (Pod → Client) — `Alert` にリネーム - -protocol: - -- `Event::Notification(Notification)` → `Event::Alert(Alert)` -- `protocol::Notification` 構造体 → `protocol::Alert` -- `NotificationLevel` → `AlertLevel` -- `NotificationSource` → `AlertSource` - -pod: - -- `crates/pod/src/ipc/notifier.rs` → `alerter.rs` -- `Notifier` → `Alerter` -- `Notifier::notify(...)` → `Alerter::alert(...)` -- `Notifier::subscribe_with_snapshot()` の戻り値 `Vec` → `Vec` - -tui: - -- `Block::Notification` → `Block::Alert`（描画スタイルは変更しない） - -### inbound (External → Pod) — `Notify` 据置 + 衛生整理 - -- `Method::Notify` 据置（動詞として正しい） -- `NotificationBuffer` → `NotifyBuffer`（メソッド名と揃える） -- `PendingNotification` → `PendingNotify` -- `notify_wrapper` プロンプト名 据置 -- LLM 向け wrapper ラベル `[Notification]` 据置（LLM への見え方は変えない） - -### protocol wire 互換 - -開発初期段階のため wire 互換は意図的に切る。client / pod を同時更新する前提で、旧名を吸収する `#[serde(rename)]` 等は入れない。 - -## 範囲外 - -- 通知の content / level / source の意味論変更（純粋な rename のみ） -- `Notifier` / `NotificationBuffer` の buffer 容量や snapshot 挙動の変更 -- TUI 側 `Block::Alert` の描画スタイルの変更 - -## 完了条件 - -- 上記 rename がすべて反映されている -- ビルドが通り、既存テストが通る -- grep で旧名（outbound 文脈の `Notification` / `Notifier` / 動詞 `notify`）が消えている - -## 参照 - -- `crates/protocol/src/lib.rs`（`Event::Notification`, `Notification` struct, `NotificationLevel/Source`） -- `crates/pod/src/ipc/notifier.rs`（outbound、`Notifier`） -- `crates/pod/src/ipc/notification_buffer.rs`（inbound、`NotificationBuffer`, `PendingNotification`） -- `crates/tui/`（`Block::Notification` 描画） - -## Review -- 状態: Approve with follow-up -- レビュー詳細: [./notification-naming-cleanup.review.md](./notification-naming-cleanup.review.md) -- 日付: 2026-04-26 diff --git a/tickets/notification-naming-cleanup.review.md b/tickets/notification-naming-cleanup.review.md deleted file mode 100644 index c85e0492..00000000 --- a/tickets/notification-naming-cleanup.review.md +++ /dev/null @@ -1,42 +0,0 @@ -# Review: Notification 用語のリネーム - -## 前提・要件の確認 - -### outbound (Pod → Client) — Alert 系 -- `Event::Notification(Notification)` → `Event::Alert(Alert)`: `crates/protocol/src/lib.rs:151`, `crates/protocol/src/lib.rs:178-185` で完了。 -- `NotificationLevel/Source` → `AlertLevel/AlertSource`: `crates/protocol/src/lib.rs:187-201` で完了。serde の `rename_all = "snake_case"` も維持されており wire 名は `warn` / `error` / `pod` / `worker` / `compactor` / `agents_md` のまま (元と一致)。 -- `notifier.rs` → `alerter.rs`、`Notifier` → `Alerter`、`notify()` → `alert()`: `crates/pod/src/ipc/alerter.rs:26-87` で完了。`subscribe_with_snapshot` の戻り値も `(Vec, …)` (l.78)。 -- `Block::Notification` → `Block::Alert`: `crates/tui/src/block.rs:24-28`。レンダリング側 (`crates/tui/src/ui.rs:316-335`) も match arm が更新されており、style (`MessageKind::NoticeWarn/Error` と prefix `[notice]` / `[notice error]`) は据置 — 範囲外要件を尊重している。 -- イベント名のテスト: `event_alert_format` (`crates/protocol/src/lib.rs:464-479`) で `event=alert` の wire 形を確認。 - -### inbound (External → Pod) — Notify 据置 + 衛生整理 -- `Method::Notify` 据置: `crates/protocol/src/lib.rs:18` 健在、`method_notify_json_roundtrip` (l.343) も保持。 -- `notification_buffer.rs` → `notify_buffer.rs`、`NotificationBuffer/PendingNotification` → `NotifyBuffer/PendingNotify`: `crates/pod/src/ipc/notify_buffer.rs:20-67`。 -- `format_notification` → `format_notify`: 同 l.74。 -- `notify_wrapper` プロンプト名と `[Notification]` ラベル据置: `crates/pod/src/ipc/notify_buffer.rs:78,119` および `crates/pod/src/prompt/catalog.rs:447,484` の test asserts、`crates/pod/tests/controller_test.rs:413` の test も `[Notification]` を期待 — 範囲外要件と整合。 - -### protocol wire 互換 -- 旧 `notification` event 名を吸収する `#[serde(rename)]` 等は入っていない (`crates/protocol/src/lib.rs:151` の `Alert(Alert)` は `rename_all = "snake_case"` 経由で wire 上 `event=alert` に変わる)。チケットの「wire 互換は意図的に切る」方針通り。 - -### 完了条件 -- ビルド通過 (cargo build): pod / protocol / tui で warning は既存の `end_scope` のみ、本変更に起因しない。 -- テスト: `cargo test -p pod -p protocol -p tui` で 205 件 PASS (ユニット+統合+doc 含む。0 fail / 0 ignored)。 -- grep: `Notifier|NotificationLevel|NotificationSource|NotificationBuffer|PendingNotification|notification_buffer|push_notification|attach_notifier|notifier` がコード上で 0 ヒット。残る `Notification` 一致は LLM 向け wrapper ラベル `[Notification]` (据置) と inbound 概念を指す散在コメントのみ。 - -## アーキテクチャ・スコープ -- レイヤ境界尊重: 変更は protocol / pod (ipc, controller, pod 本体) / tui に限定。llm-worker は触れていない。 -- 命名方針: `Alerter` / `alert()` の対 `Notify` (動詞) という非対称設計はチケット背景と一致しており、用語衝突を解消する最小限の手当てになっている。 -- 不必要な変更の有無: outbound 一式の rename と inbound 側の衛生 (`NotifyBuffer` / `PendingNotify` / `format_notify`) は要件に明記されている範囲のみ。`pending_notifications` フィールド名 → `pending_notifies` や `push_notification` → `push_notify` まで踏み込んだのは、フィールド型自体が `NotifyBuffer` / `PendingNotify` に変わっていることを踏まえると naming consistency 上妥当。`run_for_notification` メソッド名は `Method::Notify` 由来の "notify-driven run" を表すので残しても問題ない (今回触っていない)。 -- ファイル rename: ticket 通り `notifier.rs` → `alerter.rs`、`notification_buffer.rs` → `notify_buffer.rs` の 2 つだけで、過剰な移動なし。`mv` を使った旨 (history 保全) も妥当。 - -## 指摘事項 - -### Non-blocking / Follow-up -- `crates/pod/src/controller.rs:49` の doc comment が `Thin wrapper over `Alerter::notify`` のまま (実体は `Alerter::alert`)。1 行修正で解消可能。 - -### Nits -- `crates/protocol/src/lib.rs:39` の "via the notification buffer" や `crates/pod/src/controller.rs:255,258,346,474,480,657` の "notification buffer" 等の散在コメントは、文脈上 inbound (Notify) の話なので意味は通る。ただし型名が `NotifyBuffer` に揃ったので "notify buffer" に揃えると読み手のノイズが減る。ticket の「衛生整理」を保守的に解釈すれば任意項目。 -- `crates/pod/src/pod.rs:91,330,363,365,372,581,585`、`crates/pod/src/ipc/interceptor.rs:4,43,46`、`crates/pod/src/ipc/event.rs:10`、`crates/pod/src/prompt/agents_md.rs:23` 等の doc comment の "notification" も同様 — inbound 側は "notify" 動詞・"alert" 名詞 (outbound) の使い分けが定義されたので、コメントもこの軸で揃え直すと表面の用語衝突がさらに減る。本リネームの影響範囲外と見るなら据置で問題ない。 - -## 判断 -Approve with follow-up — 要件は完全に満たされ、ビルド/テストもクリーン。`controller.rs:49` の doc comment 修正は trivial なので、別チケット化せず本作業の最後に直しておくのが望ましい。コメント表面の用語ノイズは Nits で残るが、ticket の rename スコープ内で必要な分は果たされている。