@team-agent/installer 0.5.105 → 0.5.106

package/crates/team-agent/src/transport/test_support.rs

@@ -29,6 +29,7 @@ struct OfflineState {
     target_snapshots: VecDeque<Vec<PaneInfo>>,
     windows: Vec<WindowName>,
     pane_presence: BTreeMap<String, bool>,
+    dead_panes: BTreeMap<String, bool>,
     spawn_failures: BTreeMap<String, String>,
     spawned_panes_addressable: bool,
     liveness: BTreeMap<String, PaneLiveness>,
@@ -67,6 +68,7 @@ impl Default for OfflineState {
             target_snapshots: VecDeque::new(),
             windows: Vec::new(),
             pane_presence: BTreeMap::new(),
+            dead_panes: BTreeMap::new(),
             spawn_failures: BTreeMap::new(),
             spawned_panes_addressable: true,
             liveness: BTreeMap::new(),
@@ -144,6 +146,13 @@ impl OfflineTransport {
         self
     }
 
+    pub fn with_pane_dead(self, pane: impl Into<String>, dead: bool) -> Self {
+        self.with_state(|state| {
+            state.dead_panes.insert(pane.into(), dead);
+        });
+        self
+    }
+
     pub fn with_spawn_failure(self, window: impl Into<String>, error: impl Into<String>) -> Self {
         self.with_state(|state| {
             state.spawn_failures.insert(window.into(), error.into());
@@ -523,6 +532,13 @@ impl Transport for OfflineTransport {
         Ok(self.with_state(|state| state.pane_current_commands.get(&pane_id).cloned()))
     }
 
+    fn pane_is_dead(&self, pane: &PaneId) -> Result<Option<bool>, TransportError> {
+        Ok(self.with_state(|state| {
+            state.calls.push("pane_is_dead");
+            state.dead_panes.get(pane.as_str()).copied()
+        }))
+    }
+
     fn liveness(&self, pane: &PaneId) -> Result<PaneLiveness, TransportError> {
         Ok(self.with_state(|state| {
             state.calls.push("liveness");
@@ -582,6 +598,25 @@ impl Transport for OfflineTransport {
         Ok(self.with_state(|state| state.session_owner.clone()))
     }
 
+    fn set_session_owner_with_generation(
+        &self,
+        _session: &SessionName,
+        workspace: &Path,
+        team: &str,
+        generation: &str,
+    ) -> Result<(), TransportError> {
+        self.with_state(|state| {
+            state.calls.push("set_session_owner_with_generation");
+            state.session_owner = Some(super::SessionOwner {
+                workspace: workspace.canonicalize().unwrap_or_else(|_| workspace.to_path_buf())
+                    .to_string_lossy().to_string(),
+                team: team.to_string(),
+                generation: generation.to_string(),
+            });
+        });
+        Ok(())
+    }
+
     fn list_windows(&self, _session: &SessionName) -> Result<Vec<WindowName>, TransportError> {
         Ok(self.with_state(|state| {
             state.calls.push("list_windows");

package/crates/team-agent/src/transport/tests/wire.rs

@@ -366,16 +366,45 @@ fn inject_empty_payload_reports_empty_text_send_keys_and_turn_not_required() {
     assert_eq!(report.attempts, 1);
 }
 
+#[cfg(unix)]
+#[test]
+#[ignore = "requires native tmux 3.5+ and python3; owns an isolated socket/session"]
+fn submit_key_protocol_native_tmux_is_return_in_legacy_and_extended_modes() {
+    let fixture = std::path::Path::new(env!("CARGO_MANIFEST_DIR"))
+        .join("tests/fixtures/submit_key_protocol_probe.py");
+    let output = std::process::Command::new("python3")
+        .arg(fixture)
+        .args([
+            "--submit-key",
+            crate::transport::tmux_submit_key_name(Key::Enter),
+        ])
+        .output()
+        .expect("run isolated raw-PTY submit diagnostic");
+    assert!(
+        output.status.success(),
+        "production submit key is not Return: stdout={} stderr={}",
+        String::from_utf8_lossy(&output.stdout),
+        String::from_utf8_lossy(&output.stderr),
+    );
+    assert!(String::from_utf8_lossy(&output.stdout)
+        .contains("ProductionSubmitReturn=verified-three-modes"));
+}
+
 #[test]
 fn empty_inject_is_single_direct_send_keys_never_buffer() {
     // 命令构造 golden(STEP-9):空文本禁走 buffer,只发一条 send-keys 提交键
     // (tmux_io.py:42 —— tmux 拒空 buffer 会卡 trust prompt)。
-    // golden empty_inject_calls 仅 1 条:send-keys -t %7 C-m（锁定阳性 CR）。
+    // Return 与 Ctrl-M 只在 legacy 模式等价；提交必须保持逻辑 Enter。
     assert_eq!(
         tmux_empty_inject_argv(&PaneId::new("%7"), Key::Enter),
-        vec!["tmux", "send-keys", "-t", "%7", "C-m"]
+        vec!["tmux", "send-keys", "-t", "%7", "Enter"]
+    );
+    assert_eq!(
+        crate::transport::tmux_send_submit_argv(&PaneId::new("%7"), Key::Enter),
+        vec!["tmux", "send-keys", "-t", "%7", "Enter"]
     );
-    assert_eq!(crate::transport::tmux_submit_key_name(Key::Enter), "C-m");
+    assert_eq!(crate::transport::tmux_submit_key_name(Key::Enter), "Enter");
+    assert_eq!(crate::transport::tmux_submit_key_name(Key::Down), "Down");
 }
 
 #[test]

package/crates/team-agent/src/transport.rs

@@ -13,7 +13,7 @@
 //!     - name: tmux_inject_text_argv
 //!       what: 文本注入三步 argv(装 buffer→paste→删 buffer)的唯一构造点
 //!     - name: tmux_send_submit_argv
-//!       what: 提交键 argv 的唯一构造点,Enter 锁定为 C-m
+//!       what: 提交键 argv 的唯一构造点,Enter 保持逻辑 Return 而非 Ctrl-M
 //!     - name: tmux_capture_argv
 //!       what: capture-pane argv 的唯一构造点
 //!     - name: tmux_query_argv
@@ -994,6 +994,12 @@ pub trait Transport: Send + Sync {
         Ok(None)
     }
 
+    /// Native dead-pane observation. Only Some(true) proves no live pane
+    /// process; a missing/unsupported sample must remain unknown.
+    fn pane_is_dead(&self, _pane: &PaneId) -> Result<Option<bool>, TransportError> {
+        Ok(None)
+    }
+
     /// Cheap direct pane existence check when a backend can prove it. `Ok(None)`
     /// preserves the existing Unknown boundary.
     fn has_pane(&self, pane: &PaneId) -> Result<Option<bool>, TransportError> {
@@ -1085,6 +1091,17 @@ pub trait Transport: Send + Sync {
 
     // —— LIFECYCLE(SL)——
 
+    /// Before stopping a seat, preserve its session if removing these targets
+    /// would otherwise destroy it. This is not called by team shutdown/restart.
+    /// Backends whose sessions outlive their panes need no special handling.
+    fn preserve_session_before_stop(
+        &self,
+        _session: &SessionName,
+        _targets: &[Target],
+    ) -> Result<(), TransportError> {
+        Ok(())
+    }
+
     fn kill_server(&self) -> Result<(), TransportError> {
         Ok(())
     }
@@ -1199,21 +1216,18 @@ pub fn tmux_send_keys_argv(pane: &PaneId, keys: &[Key]) -> Vec<String> {
 
 /// 投递提交键的 tmux 拼写。
 ///
-/// ledger.p0enter 锁定对：`C-j`/LF 5/5 不成回合，`C-m`/CR 5/5 成回合。
-/// `send-keys Enter` 在 raw PTY 上也是 `0d`，但提交路径只发锁定阳性名
-/// `C-m`，避免落到 LF 族。菜单 `send_keys([Down, Enter])` 不走这里。
+/// `Enter` 必须保持逻辑 Return，不能替换成控制组合 `C-m`。
+/// legacy PTY 中两者都是 CR，但启用 extended keys 后 `C-m` 会成为
+/// CSI-u / modifyOtherKeys Ctrl-M，Pi 不把它当提交；`Enter` 仍发送 CR。
 /// ---
-/// purpose: 投递提交键的 tmux 拼写:Enter 锁定为 C-m(CR),其余键回落通用翻译
+/// purpose: 投递提交键复用通用逻辑键翻译,避免协议相关的控制键别名
 /// params:
-///   key: 提交键;只有 Enter 有专门拼写
-/// returns: 提交用键名;非 Enter 与 tmux_key_name 同值(含空串情形)
+///   key: 逻辑提交键
+/// returns: 与 tmux_key_name 同值(含空串情形)
 /// boundary: 只管拼写,不管何时提交、提交是否成功;不发送、不验证
 /// ---
 pub fn tmux_submit_key_name(key: Key) -> &'static str {
-    match key {
-        Key::Enter => "C-m",
-        other => tmux_key_name(other),
-    }
+    tmux_key_name(key)
 }
 
 /// 投递提交：`send-keys -t <pane> <submit-spelling>`。
@@ -1478,7 +1492,7 @@ pub fn tmux_inject_text_argv(
 
 /// 空文本 inject:纯 `send-keys -t <target> <submit_key>`,**禁** buffer 路径
 /// (tmux 拒空 buffer 会卡 trust prompt;tmux_io.py:42)。
-/// `Key::Enter` 在此译成锁定阳性 `C-m`，不是菜单用的 `Enter`。
+/// `Key::Enter` 保持逻辑 `Enter`，不会在 extended keys 下误发 Ctrl-M。
 /// ---
 /// purpose: 空载荷注入(唤醒/ping)的 argv:只发提交键,不碰 buffer
 /// params:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@team-agent/installer",
-  "version": "0.5.105",
+  "version": "0.5.106",
   "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.105",
-    "@team-agent/cli-darwin-x64": "0.5.105",
-    "@team-agent/cli-linux-x64": "0.5.105"
+    "@team-agent/cli-darwin-arm64": "0.5.106",
+    "@team-agent/cli-darwin-x64": "0.5.106",
+    "@team-agent/cli-linux-x64": "0.5.106"
   },
   "scripts": {
     "postinstall": "node npm/bincheck.mjs",

package/skills/team-agent/references/recovery-runbook.md

@@ -54,7 +54,7 @@ Diagnosis:
 
 ```bash
 team-agent status --workspace "$WORKSPACE" --json --detail
-team-agent collect --workspace "$WORKSPACE" --json
+team-agent inbox "$AGENT_ID" --workspace "$WORKSPACE" --team "$TEAM" -n 3 --json
 ```
 
 What to look for:
@@ -167,7 +167,7 @@ Validate:
 
 ```bash
 team-agent status --workspace "$WORKSPACE" --json --detail
-team-agent collect --workspace "$WORKSPACE" --json
+team-agent inbox "$AGENT_ID" --workspace "$WORKSPACE" --team "$TEAM" -n 3 --json
 ```
 
 The expected result is `status="restarted"` or an equivalent running projection without duplicate-session errors. Do not treat a partial failure on one worker as permission to reset the whole team.