From 2a7c96909c80732d74d26474f2f74b09e2c68444 Mon Sep 17 00:00:00 2001 From: Hare Date: Sat, 13 Jun 2026 11:38:08 +0900 Subject: [PATCH] ticket: reopen companion notify --- .yoi/tickets/00001KTTW04W2/item.md | 4 ++-- .yoi/tickets/00001KTTW04W2/thread.md | 22 ++++++++++++++++++++++ 2 files changed, 24 insertions(+), 2 deletions(-) diff --git a/.yoi/tickets/00001KTTW04W2/item.md b/.yoi/tickets/00001KTTW04W2/item.md index 09797e95..e1c05d79 100644 --- a/.yoi/tickets/00001KTTW04W2/item.md +++ b/.yoi/tickets/00001KTTW04W2/item.md @@ -1,8 +1,8 @@ --- title: 'Orchestrator進捗をAutoKickなしでCompanionへ通知する' -state: 'closed' +state: 'planning' created_at: '2026-06-11T08:15:24Z' -updated_at: '2026-06-12T15:44:42Z' +updated_at: '2026-06-13T02:37:58Z' assignee: null queued_by: 'workspace-panel' queued_at: '2026-06-11T10:31:56Z' diff --git a/.yoi/tickets/00001KTTW04W2/thread.md b/.yoi/tickets/00001KTTW04W2/thread.md index b6ca1dbf..62d7481d 100644 --- a/.yoi/tickets/00001KTTW04W2/thread.md +++ b/.yoi/tickets/00001KTTW04W2/thread.md @@ -514,3 +514,25 @@ Non-blocking risk: - Added `minijinja` dependency to `crates/tui`; it is already used elsewhere in the workspace, and `Cargo.lock` / `package.nix` were updated with passing Nix build. --- + +--- + + + +## State changed + +実運用で、実装が Ticket の意図を逸脱していることが確認されたため reopen する。 + +Reopen reason: +- 現実の挙動は、Panel を開いているだけで periodic reload に連動して Companion へ progress notice を繰り返し送る形になっている。 +- Ticket は Orchestrator progress を Companion に共有することを求めていたが、Panel を actionless な notification feed source にすること、長文/スナップショット的 progress notice を Panel refresh ごとに送ることは要求していない。 +- `auto_run: false` は idle Pod を起こさないだけで、Notify は Companion history/model context に入るため、通知 spam / context 汚染として扱うべきだった。 +- Progress notice の発生源は Panel の定期 reload ではなく、Pod/Orchestrator の明示的な進捗イベント、または明示 user action に紐づくべきである。 + +Next planning/fix question: +- 既存 `Method::Notify { auto_run }` 互換部分は残すか。 +- Panel reload から Companion への automatic dispatch を削除または default-off にするか。 +- Orchestrator/Pod 起点でどの明示イベントだけを Companion に通知するか。 +- 送る場合の payload を snapshot ではなく bounded event notice に縮小するか。 + +Until this is resolved, current Companion progress notice implementation should be treated as defective and not closed.