Skip to content

feat(webhook-relay): append a mark-processed prompt with event_id to room notices - #411

Merged
liplus-lin-lay merged 1 commit into
mainfrom
lin-410-mark-processed-prompt
Oct 13, 2026
Merged

liplus-lin-lay merged 1 commit into
mainfrom
lin-410-mark-processed-prompt

Conversation

@liplus-lin-lay

Copy link
Copy Markdown
Member

Closes #410

変更

  • ルームに流す github-webhook-mcp の知らせすべてに、英語の一行 If this is your own operation, mark it processed now: event_id=<id> を付ける。
  • 置き場は URL の行の直前(ブリッジの要約は URL を最終行に置く。github-webhook-mcp 0.12.0 の server/index.js で確認)。URL が無ければ末尾に足す。
  • 付けるのは crates/webhook-bridge の prompted() で、Notices::notice が通す。署名あり・署名なし・読み待ちの期限切れ・ブリッジ終了時の drain のどれにもちょうど一度付く。署名のある知らせは 署名: <名前> が先頭の行のまま。
  • アプリ自身は今も既読にしない(fix(webhook): stop marking events processed from the app #180)。
  • docs/1-room.md と docs/4-status.md、src-tauri/src/webhook.rs のモジュール説明を同じ PR で更新。

テスト

  • cargo test(crates/webhook-bridge): 26 件すべて成功。追加したのは、URL の直前に入ること、URL が無いときは末尾、どの知らせにも自分の id で一度だけ付くこと、署名のある知らせの先頭が 署名: のままであること。
  • cargo clippy --all-targets -- -D warnings と cargo fmt --check も通過。

— Claude Lin

🤖 Generated with Claude Code

https://claude.ai/code/session_01NYQn8NK3atdbSGNYFKDEjx

…room notices

Every notice the app relays from github-webhook-mcp now carries one English
line, `If this is your own operation, mark it processed now: event_id=<id>`,
placed just before the URL line (the bridge's summary ends with it) or at the
end when there is none. It is added in `Notices::notice`, so signed, unsigned,
deadline-expired and drained notices all get it exactly once, and a signed
notice still opens with `署名:`. The app itself still marks nothing (#180).

ルームの知らせを読んだセッションが、自分の作業の知らせならその場で既読にするよう、知らせそのものに案内の一行を載せる。docs/1-room.md と docs/4-status.md も同じ変更で更新した。

#410

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NYQn8NK3atdbSGNYFKDEjx

@liplus-lin-lay liplus-lin-lay left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

自己レビュー(auto モード)。

  • 受け入れ条件: 中継される全ての知らせに、自分の操作なら既読にする一行が event_id 付きで一度だけ入る。Notices::notice() の一か所で付けていて、署名あり・なし・時間切れ・終了時の吐き出しすべてが通る。署名行は後から先頭に付くので一行目のまま。Lay のレビュー(ルーム)でも同じ確認。
  • event_id はアプリが get_event に渡しているものと同じで、mark_processed にそのまま使える。
  • 範囲の逸脱: rustfmt による無関係なテスト行の折り返しのみ。挙動への影響なし。
  • 版の種類: patch(中継の文面に一行足すだけ)。
  • ワークスペース側の後回し: per-session clone pullcept-lin-410 をマージ後に削除。
  • 次: マージ。

— Claude Lin

@liplus-lin-lay
liplus-lin-lay merged commit d9a2d8f into main Oct 13, 2026
2 checks passed
@liplus-lin-lay
liplus-lin-lay deleted the lin-410-mark-processed-prompt branch October 13, 2026 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(webhook-relay): append a mark-processed prompt with event_id to room notices

1 participant