@team-agent/installer 0.5.55 → 0.5.56

package/Cargo.lock

@@ -575,7 +575,7 @@ dependencies = [
 
 [[package]]
 name = "team-agent"
-version = "0.5.55"
+version = "0.5.56"
 dependencies = [
  "anyhow",
  "chrono",

package/Cargo.toml

@@ -9,7 +9,7 @@ members = ["crates/team-agent", "crates/win-conpty-phase0", "crates/conpty-trans
 
 [workspace.package]
 edition = "2021"
-version = "0.5.55"
+version = "0.5.56"
 license = "AGPL-3.0"
 rust-version = "1.95"
 

package/crates/team-agent/src/messaging/tests/runtime.rs

@@ -771,7 +771,7 @@ fn requeue_after_claim_leader_skips_already_notified() {
     // watchers, one already notified (notified_message_id set), one un-notified.
     // requeue must return ONLY the un-notified watcher; the notified one is NOT
     // requeued and its notified_message_id SURVIVES (clearing it would cause a
-    // second injection). Probed golden: requeued == [w_un] (result_id null,
+    // second injection). Probed golden: requeued == [w_un] (result_id present,
     // prior_state "pending"); notified watcher keeps notified_message_id.
     let ws = tmp_ws("requeue");
     let store = store_for(&ws);
@@ -786,7 +786,7 @@ fn requeue_after_claim_leader_skips_already_notified() {
         "t1",
         "alice",
         "pending",
-        None,
+        Some("res_retryable"),
         None,
     );
     let w_notified = seed_watcher(
@@ -3209,7 +3209,7 @@ fn gate054_rebind_replay_requeues_failed_leader_message_on_attach() {
 }
 
 #[test]
-fn claim_requeues_pending_acceptance_for_leader_only() {
+fn claim_preserves_pending_acceptance_for_all_recipients() {
     let ws = tmp_ws("claim-pending-acceptance");
     let store = store_for(&ws);
     let log = EventLog::new(&ws);
@@ -3263,7 +3263,14 @@ fn claim_requeues_pending_acceptance_for_leader_only() {
         )
         .unwrap()
     };
-    assert_eq!(row(&leader_message), ("accepted".to_string(), None, None));
+    assert_eq!(
+        row(&leader_message),
+        (
+            "submitted_pending_acceptance".to_string(),
+            None,
+            None
+        )
+    );
     assert_eq!(
         row(&worker_message),
         (

package/crates/team-agent/src/messaging/watchers.rs

@@ -415,7 +415,7 @@ pub fn requeue_after_claim_leader(
     let conn = crate::db::schema::open_db(store.db_path())?;
     let mut stmt = conn.prepare(
         "select watcher_id, result_id, status, coalesce(completed_at, created_at) from result_watchers
-         where owner_team_id = ?1 and notified_message_id is null
+         where owner_team_id = ?1 and result_id is not null and notified_message_id is null
          order by created_at, watcher_id",
     )?;
     let rows = stmt.query_map(params![owner_team_id.as_str()], |row| {
@@ -474,9 +474,8 @@ pub fn requeue_after_claim_leader(
 /// to `accepted` so `deliver_pending_messages` replays it through the SAME
 /// pipeline. The `leader_notification_log` PK prevents a second notification
 /// row for watcher-backed messages. A `submitted_pending_acceptance` row has
-/// already crossed the transport boundary, so its recovery is intentionally
-/// at-least-once: the stable message id/receipt token is preserved, but a replay
-/// can repeat the physical submit if a same-pane claim races the receipt window.
+/// already crossed the transport boundary and is claim-immutable. Any future
+/// retry belongs to a separate typed recovery arm, not attach/claim convergence.
 pub(crate) fn requeue_blocked_leader_messages(
     conn: &rusqlite::Connection,
     event_log: &EventLog,
@@ -490,10 +489,9 @@ pub(crate) fn requeue_blocked_leader_messages(
     // `deliver_pending_messages` picks them up as `accepted` and injects
     // exactly once. status `queued_until_leader_attach` is deliberately NOT
     // in the `claim_for_delivery` eligible set (see message_store.rs) so it
-    // could not have churned while the leader was unattached. In contrast,
-    // `submitted_pending_acceptance` is an explicit at-least-once recovery arm:
-    // claim convergence favors an eventual receipt over preserving an
-    // unobservable in-flight submit.
+    // could not have churned while the leader was unattached.
+    // `submitted_pending_acceptance` remains parked and claim-immutable; a
+    // future typed recovery arm must own any deliberate retry.
     let requeued = conn.execute(
         "update messages
          set status = 'accepted',
@@ -504,7 +502,6 @@ pub(crate) fn requeue_blocked_leader_messages(
            and (
              (status = 'failed' and error = 'leader_not_attached')
              or status = 'queued_until_leader_attach'
-             or status = 'submitted_pending_acceptance'
            )",
         params![owner_team_id.as_str(), chrono::Utc::now().to_rfc3339()],
     )?;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@team-agent/installer",
-  "version": "0.5.55",
+  "version": "0.5.56",
   "description": "npx installer for Team Agent",
   "keywords": [
     "codex",
@@ -20,9 +20,9 @@
     "team-agent-installer": "npm/install.mjs"
   },
   "optionalDependencies": {
-    "@team-agent/cli-darwin-arm64": "0.5.55",
-    "@team-agent/cli-darwin-x64": "0.5.55",
-    "@team-agent/cli-linux-x64": "0.5.55"
+    "@team-agent/cli-darwin-arm64": "0.5.56",
+    "@team-agent/cli-darwin-x64": "0.5.56",
+    "@team-agent/cli-linux-x64": "0.5.56"
   },
   "scripts": {
     "postinstall": "node npm/bincheck.mjs",