Skip to content

fix(session-control): notify the room once per operation result #417

Closed

Description

@lipluscodex

Codex Luna

目的

席操作の自動通知を実行結果の1通にし、受付と結果のほぼ同文が連続する表示を解消する。マスターから実席テスト中に指摘され、2026-10-15 18:45 JSTに修正指示を受けた。

前提

src-tauri/src/session_control.rs:225 の新規受付と、同ファイル :292,307 の実行失敗/結果で notice を呼び、:276 の共通の長い本文を部屋へ投稿している。実席操作 lin-stop-lay-20261015-1 では受付から結果まで9msで、ほぼ同じ文面が2通届いた。

要求・制約

  • 受付時のルーム投稿を取り除き、通常の成功または失敗の実行結果を1通通知する。
  • 受付の永続記録、accepted/running状態の照会、操作IDの重複排除、自己再起動を継続するアプリworkerは維持する。
  • 失敗通知と画面へのエラー通知を保持する。長時間操作の受付通知やタイマー等の仕組みは今回追加しない。
  • 要件・利用ドキュメントに結果通知のみであることを反映する。

対象

src-tauri/src/session_control.rs、関連する利用仕様の docs/1-room.md および要件。変更範囲を小さく保つ。

— Codex Luna

Activity

  1. added
    bug動いていない、壊れている
    ready本文が実装開始できる形まで収束している状態。ただし更新は継続可能
    on Oct 15, 2026
  2. self-assigned this
    on Oct 15, 2026
  3. added
    waitingExternal dependency wait (CI / dependent issue / environment)
    and removed
    in-progress着手中、実装または検証が進行中
    on Oct 15, 2026
  4. lipluscodex commented on Oct 15, 2026

    @lipluscodex
    MemberAuthor

    Codex Luna

    実装とpushを完了し、PR #418 の既存CIを待機中。対象commitは 11ffd0b。

    — Codex Luna

  5. added
    review-pending実装フェーズ終了、orchestration (review / merge / close) 待ち
    and removed
    waitingExternal dependency wait (CI / dependent issue / environment)
    on Oct 15, 2026
  6. lipluscodex commented on Oct 15, 2026

    @lipluscodex
    MemberAuthor

    Codex Luna

    PR #418を自己レビュー・CI成功後にマージしました。確定SHAは bffea5af7543377acd0e22d4b0c49d87d3aaa595。

    稼働環境 D:/Users/hal/Code/pullcept への反映は、前回成功した独立ヘルパーと同じ方式で行います。D:/Users/hal/Codex_Luna/.artifacts/pullcept-417-runtime-update/ の status.json とログに更新・新ビルド・新しいアプリ系譜・HTTP応答・保存済み話題/会話再開IDの照合結果を残し、ヘルパーが成功/失敗を本issueへ追記します。

    このコメント時点ではアプリ停止前。実際の通知件数の確認は、更新後の席再接続後に行います。ソース上では受付時の投稿を削除し、成功/失敗の結果通知と状態照会を保持しました。

    — Codex Luna

  7. lipluscodex commented on Oct 15, 2026

    @lipluscodex
    MemberAuthor

    Codex Luna: マスターの重複通知修正依頼に基づき、Pullcept の実行環境を Issue #417 の bffea5a に更新し、開発アプリを再起動しました。

    • 実行元: D:/Users/hal/Code/pullcept(main、bffea5af7543377acd0e22d4b0c49d87d3aaa595)
    • 新しい Pullcept PID: 4940、Vite http://localhost:1420 は HTTP 200。Cargo dev 完了、新しい native 実行ファイルの hash と更新時刻を確認しました。
    • 保存済み topic/native conversation IDs の一致を確認。ユーザー設定ファイルの一致: True。
    • 既存の席は再接続・再起動が必要です。席の自動復帰や各席の動作確認は完了条件に含めていません。
    • 通知件数の実機テストは未実施です。ここで確認したのは更新した source、新プロセス、ビルド、HTTP、保存済み識別子です。
    • 実行記録: D:/Users/hal/Codex_Luna/.artifacts/pullcept-417-runtime-update/status.json(stdout/stderr・operations.log も同じディレクトリ)。

    — Codex Luna

  8. lipluscodex commented on Oct 15, 2026

    @lipluscodex
    MemberAuthor

    Codex Luna

    稼働環境への反映と通知件数を確認しました(2026-10-15 18:53 JST)。

    • runtime main は修正マージ bffea5af7543377acd0e22d4b0c49d87d3aaa595。独立ヘルパー結果はsuccess、新app PID4940、HTTP200、configと保存済み話題/会話ID一致、clean checkout。記録は D:/Users/hal/Codex_Luna/.artifacts/pullcept-417-runtime-update/status.json。
    • 再接続後の3席がroom_connected=trueであることをMCP一覧で確認。
    • 操作 luna-notice-once-stale-check-20261015-1 に、現在のLayのPTYとは一致しない既知の旧PTYを指定。想定どおり対象変更として拒否され、ルームにはfailed結果の1通だけが届いた。
    • 話題ログJSONLでも同操作IDを含む投稿は1行。受付通知はなく、状態照会にはaccepted_atとfailed結果が残ることを確認。
    • 確認前後で3席のPTYがすべて同じ、接続状態もtrue。現在の席は終了していない。

    これは失敗結果を使った通知経路の実機確認です。成功操作を伴う追加の席終了/再起動は行っていません。

    — Codex Luna

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bug動いていない、壊れているready本文が実装開始できる形まで収束している状態。ただし更新は継続可能review-pending実装フェーズ終了、orchestration (review / merge / close) 待ち

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions