@team-agent/installer 0.5.78 → 0.5.80

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
@@ -0,0 +1,248 @@
1
+ use super::*;
2
+ use crate::state::persist::{load_runtime_state, runtime_state_path, save_runtime_state};
3
+
4
+ fn workspace() -> std::path::PathBuf {
5
+ use std::sync::atomic::{AtomicUsize, Ordering};
6
+ static NEXT: AtomicUsize = AtomicUsize::new(0);
7
+ let path = std::env::temp_dir().join(format!(
8
+ "watch-188-{}-{}",
9
+ std::process::id(),
10
+ NEXT.fetch_add(1, Ordering::Relaxed)
11
+ ));
12
+ std::fs::create_dir_all(path.join(".team/runtime")).unwrap();
13
+ path
14
+ }
15
+
16
+ #[test]
17
+ fn complete_empty_roster_prunes_but_missing_roster_and_paused_members_survive() {
18
+ let mut state = json!({
19
+ "agents": {},
20
+ "teams": {
21
+ "empty": {"agents": {}, "coordinator": {"abnormal_exit_watch": {"old": {}}}},
22
+ "unknown": {"coordinator": {"abnormal_exit_watch": {"old": {}}}},
23
+ "paused": {"agents": {"p": {"status": "paused"}}, "coordinator": {
24
+ "abnormal_exit_watch": {"p": {"last_notified_key": "keep"}, "gone": {}}
25
+ }}
26
+ },
27
+ "coordinator": {"abnormal_exit_watch": {"unknown": {}}}
28
+ });
29
+ compact_after_merge(&mut state, None, &[]);
30
+ assert_eq!(
31
+ state["teams"]["empty"]["coordinator"]["abnormal_exit_watch"],
32
+ json!({})
33
+ );
34
+ assert_eq!(
35
+ state["teams"]["unknown"]["coordinator"]["abnormal_exit_watch"],
36
+ json!({"old": {}})
37
+ );
38
+ assert_eq!(
39
+ state["teams"]["paused"]["coordinator"]["abnormal_exit_watch"],
40
+ json!({"p": {"last_notified_key": "keep"}})
41
+ );
42
+ assert_eq!(
43
+ state["coordinator"]["abnormal_exit_watch"],
44
+ json!({"unknown": {}})
45
+ );
46
+ assert!(!needs_compaction(&state));
47
+ }
48
+
49
+ #[test]
50
+ fn root_legacy_union_is_not_migrated_by_bare_agent_name() {
51
+ let mut state = json!({"agents": {}, "teams": {
52
+ "a": {"agents": {"w": {}}}, "b": {"agents": {"w": {}}}
53
+ }, "coordinator": {"abnormal_exit_watch": {"w": {"last_notified_key": "legacy"}, "gone": {}}}});
54
+ compact_after_merge(&mut state, None, &[]);
55
+ assert_eq!(
56
+ state["coordinator"]["abnormal_exit_watch"]
57
+ .as_object()
58
+ .unwrap()
59
+ .len(),
60
+ 1
61
+ );
62
+ for team in ["a", "b"] {
63
+ let projection = crate::state::projection::project_top_level_view(&state, team);
64
+ assert!(projection["coordinator"]
65
+ .get("abnormal_exit_watch")
66
+ .is_none());
67
+ }
68
+ }
69
+
70
+ #[test]
71
+ fn fallback_preserves_other_coordinator_fields_and_owned_watch() {
72
+ let state = json!({"coordinator": {"tick": 9, "abnormal_exit_watch": {"w": {}}}, "teams": {
73
+ "a": {"agents": {}},
74
+ "b": {"agents": {}, "coordinator": {"abnormal_exit_watch": {"owned": {}}, "tick": 2}}
75
+ }});
76
+ let a = crate::state::projection::project_top_level_view(&state, "a");
77
+ let b = crate::state::projection::project_top_level_view(&state, "b");
78
+ assert_eq!(a["coordinator"], json!({"tick": 9}));
79
+ assert_eq!(b["coordinator"], state["teams"]["b"]["coordinator"]);
80
+ }
81
+
82
+ fn observed() -> Value {
83
+ let mut state = json!({"active_team_key": "a", "agents": {"w": {
84
+ "provider": "codex", "rollout_path": "/fixture/w.jsonl", "spawn_epoch": 1
85
+ }}, "coordinator": {"abnormal_exit_watch": {"w": {"last_notified_key": "keep"}}}});
86
+ state["coordinator"]["abnormal_exit_watch"]["w"]["watch_identity"] =
87
+ observation_identity(&state, "w").unwrap();
88
+ state
89
+ }
90
+
91
+ #[test]
92
+ fn observation_identity_separates_same_name_team_path_and_spawn_generation() {
93
+ let initial = observed();
94
+ let mut unchanged = initial.clone();
95
+ begin_observation(&mut unchanged, "w");
96
+ begin_observation(&mut unchanged, "w");
97
+ assert_eq!(
98
+ unchanged, initial,
99
+ "same observation retains notification dedupe"
100
+ );
101
+ for (pointer, value) in [
102
+ ("/active_team_key", json!("b")),
103
+ ("/agents/w/rollout_path", json!("/fixture/other.jsonl")),
104
+ ("/agents/w/spawn_epoch", json!(2)),
105
+ ("/agents/w/provider", json!("claude")),
106
+ ("/agents/w/spawned_at", json!("new-incarnation")),
107
+ ] {
108
+ let mut changed = initial.clone();
109
+ if pointer.ends_with("spawned_at") {
110
+ changed["agents"]["w"]["spawned_at"] = value;
111
+ } else {
112
+ *changed.pointer_mut(pointer).unwrap() = value;
113
+ }
114
+ begin_observation(&mut changed, "w");
115
+ assert!(changed["coordinator"]["abnormal_exit_watch"]
116
+ .get("w")
117
+ .is_none());
118
+ }
119
+ }
120
+
121
+ #[test]
122
+ fn legacy_unknown_identity_is_retained_until_member_is_observed() {
123
+ let mut state = observed();
124
+ state["coordinator"]["abnormal_exit_watch"]["w"]
125
+ .as_object_mut()
126
+ .unwrap()
127
+ .remove("watch_identity");
128
+ let legacy = state.clone();
129
+ compact_after_merge(&mut state, None, &[]);
130
+ assert_eq!(
131
+ state, legacy,
132
+ "membership compaction cannot guess legacy identity"
133
+ );
134
+ begin_observation(&mut state, "w");
135
+ assert!(state["coordinator"]["abnormal_exit_watch"]
136
+ .get("w")
137
+ .is_none());
138
+ }
139
+
140
+ #[test]
141
+ fn unchanged_normal_save_compacts_large_old_snapshots_and_cannot_resurrect_them() {
142
+ let ws = workspace();
143
+ let watch: serde_json::Map<String, Value> = (0..100)
144
+ .map(|n| {
145
+ (
146
+ format!("old-{n}"),
147
+ json!({"last_notified_key": "x".repeat(100)}),
148
+ )
149
+ })
150
+ .collect();
151
+ let mut state =
152
+ json!({"agents": {}, "teams": {}, "coordinator": {"abnormal_exit_watch": watch}});
153
+ for n in 0..8 {
154
+ state["teams"][format!("team-{n}")] =
155
+ json!({"agents": {}, "coordinator": state["coordinator"].clone()});
156
+ }
157
+ let original = serde_json::to_vec(&state).unwrap().len();
158
+ // Warm the old cache without invoking the load-time migrations, which
159
+ // could otherwise compact before the no-change save under test.
160
+ crate::state::persist::save_runtime_state_without_migrations(&ws, &state).unwrap();
161
+ let stale = state;
162
+ assert_eq!(
163
+ serde_json::from_slice::<Value>(&std::fs::read(runtime_state_path(&ws)).unwrap())
164
+ .unwrap(),
165
+ stale
166
+ );
167
+ save_runtime_state(&ws, &stale).unwrap();
168
+ let first = std::fs::read(runtime_state_path(&ws)).unwrap();
169
+ assert!(
170
+ first.len() < original / 10,
171
+ "size must depend on actual members, not watch x teams"
172
+ );
173
+ save_runtime_state(&ws, &stale).unwrap();
174
+ assert_eq!(std::fs::read(runtime_state_path(&ws)).unwrap(), first);
175
+ std::fs::remove_dir_all(ws).unwrap();
176
+ }
177
+
178
+ #[test]
179
+ fn latest_roster_merge_precedes_pruning_and_explicit_deletion_removes_watch() {
180
+ let ws = workspace();
181
+ let latest = json!({"active_team_key": "a", "session_name": "team-a", "agents": {
182
+ "w": {"agent_id": "w", "provider": "codex", "status": "paused"}
183
+ }, "coordinator": {"abnormal_exit_watch": {"w": {"last_notified_key": "keep"}}}});
184
+ save_runtime_state(&ws, &latest).unwrap();
185
+ let mut stale = load_runtime_state(&ws).unwrap();
186
+ stale["agents"] = json!({});
187
+ save_runtime_state(&ws, &stale).unwrap();
188
+ let merged = load_runtime_state(&ws).unwrap();
189
+ assert!(merged["agents"].get("w").is_some());
190
+ assert_eq!(
191
+ merged["coordinator"]["abnormal_exit_watch"]["w"]["last_notified_key"],
192
+ "keep"
193
+ );
194
+ crate::state::persist::save_runtime_state_with_deleted_agents(&ws, &stale, &["w"]).unwrap();
195
+ let removed = load_runtime_state(&ws).unwrap();
196
+ assert!(removed["agents"].get("w").is_none());
197
+ assert_eq!(removed["coordinator"]["abnormal_exit_watch"], json!({}));
198
+ std::fs::remove_dir_all(ws).unwrap();
199
+ }
200
+
201
+ #[test]
202
+ fn unknown_team_retains_history_until_observation_but_never_authorizes_reuse() {
203
+ let mut state = observed();
204
+ state["active_team_key"] = Value::Null;
205
+ let before = state.clone();
206
+ assert!(observation_identity(&state, "w").is_none());
207
+ compact_after_merge(&mut state, None, &[]);
208
+ assert_eq!(state, before);
209
+ assert!(!matches_observation(
210
+ &state,
211
+ "w",
212
+ &state["coordinator"]["abnormal_exit_watch"]["w"]
213
+ ));
214
+ begin_observation(&mut state, "w");
215
+ assert!(state["coordinator"]["abnormal_exit_watch"]
216
+ .get("w")
217
+ .is_none());
218
+ }
219
+
220
+ #[test]
221
+ fn malformed_team_catalog_does_not_authorize_root_pruning() {
222
+ let mut state =
223
+ json!({"agents": {}, "teams": [], "coordinator": {"abnormal_exit_watch": {"w": {}}}});
224
+ let before = state.clone();
225
+ compact_after_merge(&mut state, None, &[]);
226
+ assert_eq!(state, before);
227
+ assert!(!needs_compaction(&state));
228
+ }
229
+
230
+ #[test]
231
+ fn lock_held_latest_membership_protects_rows_without_roster_stub_fields() {
232
+ let latest = json!({"agents": {"unknown-row": {}}, "teams": {
233
+ "new-sibling": {"agents": {"sibling": {}}}
234
+ }});
235
+ let mut incoming = json!({"agents": {}, "coordinator": {"abnormal_exit_watch": {
236
+ "unknown-row": {}, "sibling": {}, "gone": {}
237
+ }}});
238
+ compact_after_merge(&mut incoming, Some(&latest), &[]);
239
+ assert_eq!(
240
+ incoming["coordinator"]["abnormal_exit_watch"],
241
+ json!({"unknown-row": {}, "sibling": {}})
242
+ );
243
+ compact_after_merge(&mut incoming, Some(&latest), &["unknown-row"]);
244
+ assert_eq!(
245
+ incoming["coordinator"]["abnormal_exit_watch"],
246
+ json!({"sibling": {}})
247
+ );
248
+ }
@@ -0,0 +1,144 @@
1
+ //! Scope and lifecycle boundaries for the abnormal-exit observation cache.
2
+ use std::collections::BTreeSet;
3
+
4
+ use serde_json::{json, Value};
5
+
6
+ fn members(state: &Value) -> Option<BTreeSet<String>> {
7
+ Some(state.get("agents")?.as_object()?.keys().cloned().collect())
8
+ }
9
+
10
+ // A missing roster is unknown. Only a complete, lock-merged roster authorizes
11
+ // discarding absent members; liveness, pause status and record age do not.
12
+ fn scopes(state: &Value) -> Vec<(Option<String>, Option<BTreeSet<String>>)> {
13
+ let mut root = members(state);
14
+ if state.get("teams").is_some_and(|teams| !teams.is_object()) {
15
+ root = None;
16
+ }
17
+ let mut scopes = Vec::new();
18
+ if let Some(teams) = state.get("teams").and_then(Value::as_object) {
19
+ for (team, entry) in teams {
20
+ let roster = members(entry);
21
+ root = match (root, &roster) {
22
+ (Some(mut union), Some(keys)) => {
23
+ union.extend(keys.iter().cloned());
24
+ Some(union)
25
+ }
26
+ _ => None,
27
+ };
28
+ scopes.push((Some(team.clone()), roster));
29
+ }
30
+ }
31
+ scopes.push((None, root));
32
+ scopes
33
+ }
34
+
35
+ pub(crate) fn needs_compaction(state: &Value) -> bool {
36
+ scopes(state).into_iter().any(|(team, roster)| {
37
+ let entry = team.as_ref().map_or(state, |key| &state["teams"][key]);
38
+ roster.is_some_and(|keys| {
39
+ entry
40
+ .pointer("/coordinator/abnormal_exit_watch")
41
+ .and_then(Value::as_object)
42
+ .is_some_and(|watch| watch.keys().any(|key| !keys.contains(key)))
43
+ })
44
+ })
45
+ }
46
+
47
+ /// Call only after the persistence lock's latest-roster merge.
48
+ pub(crate) fn compact_after_merge(state: &mut Value, latest: Option<&Value>, deleted: &[&str]) {
49
+ let latest_scopes = latest.map(scopes).unwrap_or_default();
50
+ for (team, mut roster) in scopes(state) {
51
+ // The normal merge may omit a roster row with no allowlisted stub
52
+ // fields, or a whole sibling absent from a stale input. Membership in
53
+ // lock-held latest still forbids pruning its observation. Only explicit
54
+ // deletion authority can subtract it from this retention union.
55
+ if let Some((_, latest_roster)) = latest_scopes.iter().find(|(key, _)| key == &team) {
56
+ roster = match (roster, latest_roster) {
57
+ (Some(mut keys), Some(latest_keys)) => {
58
+ keys.extend(
59
+ latest_keys
60
+ .iter()
61
+ .filter(|key| !deleted.contains(&key.as_str()))
62
+ .cloned(),
63
+ );
64
+ Some(keys)
65
+ }
66
+ _ => None,
67
+ };
68
+ }
69
+ let Some(keys) = roster else { continue };
70
+ let entry = match team {
71
+ Some(team) => &mut state["teams"][&team],
72
+ None => &mut *state,
73
+ };
74
+ if let Some(watch) = entry
75
+ .pointer_mut("/coordinator/abnormal_exit_watch")
76
+ .and_then(Value::as_object_mut)
77
+ {
78
+ watch.retain(|key, _| keys.contains(key));
79
+ }
80
+ }
81
+ }
82
+
83
+ /// New observations carry their team and cohort, never just a bare agent id.
84
+ pub(crate) fn observation_identity(state: &Value, agent_id: &str) -> Option<Value> {
85
+ let agent = state.get("agents")?.get(agent_id)?;
86
+ let provider = crate::provider::wire::parse_provider(agent.get("provider")?.as_str()?)?;
87
+ let path = ["rollout_path", "transcript_path", "session_log_path"]
88
+ .into_iter()
89
+ .find_map(|key| agent.get(key).and_then(Value::as_str))
90
+ .filter(|path| !path.is_empty())?;
91
+ let spawn_epoch = agent.get("spawn_epoch").and_then(Value::as_u64);
92
+ let spawned_at = agent
93
+ .get("spawned_at")
94
+ .and_then(Value::as_str)
95
+ .filter(|value| !value.is_empty());
96
+ // Preserve the watcher's existing legacy cohort: missing spawn fields
97
+ // remain explicit nulls, never a fabricated generation. A later known
98
+ // cohort changes this identity and establishes a new baseline.
99
+ let team = state
100
+ .get("active_team_key")
101
+ .and_then(Value::as_str)
102
+ .filter(|team| !team.is_empty())
103
+ .map(str::to_string)
104
+ .or_else(|| {
105
+ let team = super::projection::team_state_key(state);
106
+ (team != super::projection::CURRENT_TEAM_ALIAS).then_some(team)
107
+ })?;
108
+ Some(json!({
109
+ "team_key": team,
110
+ "agent_id": agent_id,
111
+ "provider": crate::provider::wire::provider_wire(provider),
112
+ "path": path,
113
+ "spawn_epoch": spawn_epoch,
114
+ "spawned_at": spawned_at,
115
+ }))
116
+ }
117
+
118
+ pub(crate) fn matches_observation(state: &Value, agent_id: &str, row: &Value) -> bool {
119
+ observation_identity(state, agent_id)
120
+ .is_some_and(|identity| row.get("watch_identity") == Some(&identity))
121
+ }
122
+
123
+ /// Legacy/foreign-cohort facts cannot be reused as current observation evidence.
124
+ /// This is invoked only when actually observing the member, not for paused or
125
+ /// otherwise ineligible members. The next scan establishes the ordinary baseline.
126
+ pub(crate) fn begin_observation(state: &mut Value, agent_id: &str) {
127
+ let identity = observation_identity(state, agent_id);
128
+ if let Some(watch) = state
129
+ .pointer_mut("/coordinator/abnormal_exit_watch")
130
+ .and_then(Value::as_object_mut)
131
+ {
132
+ if watch.get(agent_id).is_some_and(|row| {
133
+ identity
134
+ .as_ref()
135
+ .is_none_or(|identity| row.get("watch_identity") != Some(identity))
136
+ }) {
137
+ watch.remove(agent_id);
138
+ }
139
+ }
140
+ }
141
+
142
+ #[cfg(test)]
143
+ #[allow(clippy::unwrap_used, clippy::expect_used, clippy::panic)]
144
+ mod tests;
@@ -15,6 +15,7 @@
15
15
  //! 绝不 in-place truncate,绝不让审计/重试失败拖垮可见的原 state。
16
16
  #![deny(clippy::unwrap_used, clippy::expect_used, clippy::panic)]
17
17
 
18
+ pub(crate) mod abnormal_watch;
18
19
  pub mod identity;
19
20
  // Stage 0 of identity-boundary unified plan (architect direction 2026-06-23):
20
21
  // shared identity primitives. Pure additive — Stage 1+ consume these.
@@ -492,7 +492,8 @@ fn persist_runtime_state_with_merge_options_and_expected(
492
492
  // Exact-seed cleanup must observe lock-held disk, not a pre-lock cache or
493
493
  // byte-equality hit. A matching in-memory document would otherwise no-op
494
494
  // and leave the seed, or skip copying a concurrent owner.
495
- let skip_pre_lock_fast_path = exact_owner_seed_to_clear.is_some()
495
+ let skip_pre_lock_fast_path = super::abnormal_watch::needs_compaction(&migrated)
496
+ || exact_owner_seed_to_clear.is_some()
496
497
  || expected_owner_receiver.is_some()
497
498
  || exact_owner_receiver_to_clear.is_some()
498
499
  || exact_owner_receiver_to_restore.is_some();
@@ -542,7 +543,7 @@ fn persist_runtime_state_with_merge_options_and_expected(
542
543
  ));
543
544
  }
544
545
  }
545
- if let Some(latest) = latest {
546
+ if let Some(latest) = latest.as_ref() {
546
547
  let deleted = deleted_agent_ids
547
548
  .iter()
548
549
  .copied()
@@ -572,7 +573,7 @@ fn persist_runtime_state_with_merge_options_and_expected(
572
573
  .collect::<BTreeSet<_>>();
573
574
  apply_persist_merge_contract(
574
575
  &mut migrated,
575
- &latest,
576
+ latest,
576
577
  &deleted,
577
578
  skip_capture_backfill_team_key,
578
579
  &skip_capture_backfill,
@@ -583,6 +584,7 @@ fn persist_runtime_state_with_merge_options_and_expected(
583
584
  exact_owner_receiver_to_restore,
584
585
  )?;
585
586
  }
587
+ super::abnormal_watch::compact_after_merge(&mut migrated, latest.as_ref(), deleted_agent_ids);
586
588
  // Stage 3 save-output strip second pass (defence-in-depth): after the
587
589
  // `preserve_latest_roster_entries` lock-held merge, a future addition
588
590
  // to the preserve path could re-introduce root owner fields from the
@@ -486,7 +486,13 @@ pub fn project_top_level_view(state: &Value, team_key: &str) -> Value {
486
486
  .is_some_and(|o| o.contains_key("coordinator"))
487
487
  && !p.contains_key("coordinator")
488
488
  {
489
- p.insert("coordinator".to_string(), state["coordinator"].clone());
489
+ let mut coordinator = state["coordinator"].clone();
490
+ // Workspace legacy watch facts have no proven owner in this team.
491
+ // Preserve other coordinator fields, but never broadcast this cache.
492
+ if let Some(coordinator) = coordinator.as_object_mut() {
493
+ coordinator.remove("abnormal_exit_watch");
494
+ }
495
+ p.insert("coordinator".to_string(), coordinator);
490
496
  }
491
497
  Value::Object(p)
492
498
  }
@@ -355,6 +355,34 @@ fn backend_with(default: MockResp, queued: Vec<MockResp>) -> (TmuxBackend, Recor
355
355
  (TmuxBackend::with_runner(Box::new(runner)), recorded)
356
356
  }
357
357
 
358
+ #[test]
359
+ fn provider_exit_receipt_accepts_only_shell_exit_status() {
360
+ for (text, expected) in [
361
+ ("", None),
362
+ ("running", None),
363
+ ("[team-agent] pi exited with 0", None),
364
+ ("256", None),
365
+ ("0\n", Some(0)),
366
+ ("17\n", Some(17)),
367
+ ("143\n", Some(143)),
368
+ ] {
369
+ let (backend, recorded) = backend_with(MockResp::Out(ok(text)), Vec::new());
370
+ assert_eq!(
371
+ backend.provider_exit_status(&PaneId::new("%47")).unwrap(),
372
+ expected
373
+ );
374
+ let calls = recorded.lock().unwrap();
375
+ assert_eq!(calls.len(), 1);
376
+ assert_eq!(calls[0][4], "%47");
377
+ assert_eq!(calls[0][5], "#{@team-agent-provider-exit-status}");
378
+ }
379
+ let (backend, _) = backend_with(MockResp::Out(fail(1, "pane unavailable")), Vec::new());
380
+ assert_eq!(
381
+ backend.provider_exit_status(&PaneId::new("%47")).unwrap(),
382
+ None
383
+ );
384
+ }
385
+
358
386
  fn backend_with_stdin(
359
387
  default: MockResp,
360
388
  queued: Vec<MockResp>,
@@ -2656,24 +2656,22 @@ fn append_caller_context_assignments(parts: &mut Vec<String>, provider: Option<&
2656
2656
  ));
2657
2657
  }
2658
2658
 
2659
- /// 0.4.x (CR R6): single-source marker prefix. The exit marker emitted by
2660
- /// `leader_shell_wrapper_command` and the substring detected by
2661
- /// `leader_provider_health` MUST share this prefix exactly. Format:
2659
+ /// Public exit-marker text emitted by `leader_shell_wrapper_command`. Format:
2662
2660
  /// `"[team-agent] {provider_label} exited with {rc}"`.
2663
2661
  pub const LEADER_PROVIDER_EXIT_MARKER_PREFIX: &str = "[team-agent]";
2664
2662
  pub const LEADER_PROVIDER_EXIT_MARKER_SUFFIX: &str = "exited with";
2663
+ const PROVIDER_EXIT_STATUS_OPTION: &str = "@team-agent-provider-exit-status";
2665
2664
 
2666
2665
  /// 0.4.x (CR R6): build the leader exit marker text for `provider_label`.
2667
- /// Used by both the shell wrapper (printf source) and the health check
2668
- /// (capture substring) so they cannot drift.
2666
+ /// Used by the shell wrapper for human-readable output.
2669
2667
  /// ---
2670
2668
  /// purpose: 单一来源地拼出 leader pane 的 provider 退出标记文本
2671
2669
  /// params:
2672
2670
  /// provider_label: 人类可读的 provider 名,原样嵌入标记
2673
2671
  /// returns: 前缀 + provider 名 + 后缀 拼成的标记(不含退出码,退出码由 shell 的 printf 补)
2674
2672
  /// boundary:
2675
- /// - 写标记的 shell wrapper 与读标记的健康检查必须都用这一个函数,禁止各写一份字面量
2676
- /// - 这是内容信号:pane 里出现同样文本(如 cat 日志)就会误触发,判定强度到此为止
2673
+ /// - shell wrapper 统一使用此显示标记,进程状态另由监督 shell 收据证明
2674
+ /// - 仅供显示;历史正文可包含相同文本,不得据此判定进程退出
2677
2675
  /// ---
2678
2676
  pub fn leader_provider_exit_marker(provider_label: &str) -> String {
2679
2677
  format!(
@@ -2905,6 +2903,11 @@ pub fn leader_shell_wrapper_command(
2905
2903
  parts.push(key.clone());
2906
2904
  parts.push("&&".to_string());
2907
2905
  }
2906
+ // Clear any prior receipt before invoking the provider. Terminal content,
2907
+ // including replayed sessions, must never be treated as an exit receipt.
2908
+ parts.push(format!(
2909
+ "{{ tmux -S \"${{TMUX%%,*}}\" set-option -p -t \"$TMUX_PANE\" {PROVIDER_EXIT_STATUS_OPTION} running >/dev/null 2>&1 || :; }} &&"
2910
+ ));
2908
2911
  // 3. env exports + provider (NO `exec` so the provider is a child).
2909
2912
  // 0.4.x ordering fix: skip keys present in env_unset so KEY=val does not
2910
2913
  // re-introduce a just-unset variable from the inherited env map.
@@ -2918,6 +2921,9 @@ pub fn leader_shell_wrapper_command(
2918
2921
  parts.push(";".to_string());
2919
2922
  // 4. exit marker + inert shell tail
2920
2923
  parts.push("rc=$?;".to_string());
2924
+ parts.push(format!(
2925
+ "tmux -S \"${{TMUX%%,*}}\" set-option -p -t \"$TMUX_PANE\" {PROVIDER_EXIT_STATUS_OPTION} \"$rc\" >/dev/null 2>&1;"
2926
+ ));
2921
2927
  parts.push("printf".to_string());
2922
2928
  // CR R6: marker text comes from single-source `leader_provider_exit_marker`.
2923
2929
  parts.push(shell_quote(&format!(
@@ -2930,8 +2936,8 @@ pub fn leader_shell_wrapper_command(
2930
2936
  }
2931
2937
 
2932
2938
  fn inert_pane_tail_command() -> String {
2933
- // `sh` is deliberate: provider-exit health checks use the shell basename
2934
- // to reach the exit-marker branch. `sh -c` does not read commands from
2939
+ // `sh` is deliberate: exited-wrapper input classifiers recognize shell
2940
+ // basenames. `sh -c` does not read commands from
2935
2941
  // stdin; disabling echo also prevents input from appearing accepted.
2936
2942
  let script = r#"trap '' INT QUIT; stty -echo 2>/dev/null; printf "%s\n" "[team-agent] Provider exited; this pane no longer accepts input. Restart from another pane with the appropriate team-agent start command."; while :; do sleep 3600 & wait "$!"; done"#;
2937
2943
  format!("exec /bin/sh -c {}", shell_quote(script))
@@ -3743,6 +3749,22 @@ impl Transport for TmuxBackend {
3743
3749
  Ok(Some(output.stdout.trim().to_string()))
3744
3750
  }
3745
3751
 
3752
+ fn provider_exit_status(&self, pane: &PaneId) -> Result<Option<u8>, TransportError> {
3753
+ let argv = self.tmux_argv(&[
3754
+ "tmux".to_string(),
3755
+ "display-message".to_string(),
3756
+ "-p".to_string(),
3757
+ "-t".to_string(),
3758
+ pane.as_str().to_string(),
3759
+ format!("#{{{PROVIDER_EXIT_STATUS_OPTION}}}"),
3760
+ ]);
3761
+ let output = self.runner.run(&argv)?;
3762
+ if !output.success {
3763
+ return Ok(None);
3764
+ }
3765
+ Ok(output.stdout.trim().parse::<u8>().ok())
3766
+ }
3767
+
3746
3768
  fn liveness(&self, pane: &PaneId) -> Result<PaneLiveness, TransportError> {
3747
3769
  let argv = self.tmux_argv(&[
3748
3770
  "tmux".to_string(),
@@ -981,6 +981,12 @@ pub trait Transport: Send + Sync {
981
981
  /// pane 存活三态(`PaneLiveness`,bug-085 穷尽 match;unknown ≠ dead ≠ live)。
982
982
  fn liveness(&self, pane: &PaneId) -> Result<PaneLiveness, TransportError>;
983
983
 
984
+ /// Exit status recorded by the provider's supervising wrapper, never inferred
985
+ /// from terminal text. None means running or no authoritative receipt.
986
+ fn provider_exit_status(&self, _pane: &PaneId) -> Result<Option<u8>, TransportError> {
987
+ Ok(None)
988
+ }
989
+
984
990
  /// Cheap direct pane existence check when a backend can prove it. `Ok(None)`
985
991
  /// preserves the existing Unknown boundary.
986
992
  fn has_pane(&self, pane: &PaneId) -> Result<Option<bool>, TransportError> {
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@team-agent/installer",
3
- "version": "0.5.78",
3
+ "version": "0.5.80",
4
4
  "description": "npx installer for Team Agent",
5
5
  "keywords": [
6
6
  "codex",
@@ -20,9 +20,9 @@
20
20
  "team-agent-installer": "npm/install.mjs"
21
21
  },
22
22
  "optionalDependencies": {
23
- "@team-agent/cli-darwin-arm64": "0.5.78",
24
- "@team-agent/cli-darwin-x64": "0.5.78",
25
- "@team-agent/cli-linux-x64": "0.5.78"
23
+ "@team-agent/cli-darwin-arm64": "0.5.80",
24
+ "@team-agent/cli-darwin-x64": "0.5.80",
25
+ "@team-agent/cli-linux-x64": "0.5.80"
26
26
  },
27
27
  "scripts": {
28
28
  "postinstall": "node npm/bincheck.mjs",