@team-agent/installer 0.5.101 → 0.5.103

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

@@ -140,23 +140,6 @@ fn activity_status_serde_byte_locked() {
     );
 }
 
-#[test]
-fn alert_type_serde_byte_locked() {
-    // scheduler.py:38 _ALERT_TYPES set members.
-    assert_eq!(
-        serde_json::to_string(&AlertType::Stuck).unwrap(),
-        "\"stuck\""
-    );
-    assert_eq!(
-        serde_json::to_string(&AlertType::IdleFallback).unwrap(),
-        "\"idle_fallback\""
-    );
-    assert_eq!(
-        serde_json::to_string(&AlertType::CrossWorkerDeadlock).unwrap(),
-        "\"cross_worker_deadlock\""
-    );
-}
-
 #[test]
 fn check_status_serde_byte_locked() {
     assert_eq!(
@@ -266,20 +249,6 @@ fn provider_sdk_calls_is_zero_only_when_all_three_zero() {
     .is_zero());
 }
 
-#[test]
-fn alert_type_all_is_sorted_full_set() {
-    // scheduler.py:269 sorted(_ALERT_TYPES) =
-    //   ['cross_worker_deadlock','idle_fallback','stuck'].
-    assert_eq!(
-        AlertType::all(),
-        [
-            AlertType::CrossWorkerDeadlock,
-            AlertType::IdleFallback,
-            AlertType::Stuck
-        ]
-    );
-}
-
 #[test]
 fn activity_status_idle_takeover_gate_uncertain_never_idle() {
     // THE IRON LAW (bug-071/077/085): only Idle passes; Uncertain/Working/Stuck

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

@@ -1,78 +1,5 @@
 use super::*;
 
-// ── F1 [P1 byte-shape] — stuck_cancel suppression snapshot.delivered_message_ids must use golden's
-// _DELIVERED_MESSAGE_STATUSES = {visible, submitted, delivered, acknowledged} (scheduler.py:27,376-383),
-// sorted. The Rust delivered_message_ids (scheduler.rs:243) used {injected, visible, submitted,
-// submitted_unverified, delivered} — the WRONG set (reviewer's explicit warning: don't copy the
-// activity.rs deadlock query set): it EXCLUDES 'acknowledged' and INCLUDES 'injected'/'submitted_unverified'.
-// (The snapshot also must carry BOTH assigned_task_ids AND delivered_message_ids — that structural half
-// is already present; this pins the STATUS SET.)
-#[test]
-fn stuck_cancel_snapshot_delivered_message_ids_uses_golden_status_set() {
-    let ws = tmp_ws("f1_delivered");
-    crate::state::persist::save_runtime_state(
-        &ws,
-        &serde_json::json!({
-            "active_team_key": "teamX",
-            "agents": {"w1": {"status": "running", "provider": "codex"}}
-        }),
-    )
-    .unwrap();
-    let store = crate::message_store::MessageStore::open(&ws).unwrap();
-    // golden-delivered (must be IN delivered_message_ids): acknowledged + visible.
-    let m_ack = store
-        .create_message(None, "leader", "w1", "ack-me", None, true, Some("teamX"))
-        .unwrap();
-    store.mark(&m_ack, "delivered", None).unwrap();
-    store.mark(&m_ack, "acknowledged", None).unwrap();
-    let m_vis = store
-        .create_message(None, "leader", "w1", "vis", None, true, Some("teamX"))
-        .unwrap();
-    store.mark(&m_vis, "visible", None).unwrap();
-    // NOT golden-delivered (must be EXCLUDED): injected.
-    let m_inj = store
-        .create_message(None, "leader", "w1", "inj", None, true, Some("teamX"))
-        .unwrap();
-    store.mark(&m_inj, "injected", None).unwrap();
-    drop(store);
-    let _ = stuck_cancel(&ws, "w1", None, "leader").unwrap();
-    let state = crate::state::persist::load_runtime_state(&ws).unwrap();
-    let snapshot =
-        &state["coordinator"]["suppressed_idle_alerts"]["teamX"]["w1"]["stuck"]["snapshot"];
-    assert!(
-        snapshot.get("assigned_task_ids").is_some(),
-        "F1: snapshot must carry assigned_task_ids; got {snapshot}"
-    );
-    let delivered = snapshot
-        .get("delivered_message_ids")
-        .and_then(serde_json::Value::as_array)
-        .map(|a| {
-            a.iter()
-                .filter_map(|v| v.as_str().map(str::to_string))
-                .collect::<Vec<_>>()
-        })
-        .unwrap_or_else(|| panic!("F1: snapshot must carry delivered_message_ids; got {snapshot}"));
-    assert!(
-        delivered.contains(&m_ack),
-        "F1: golden _DELIVERED_MESSAGE_STATUSES includes 'acknowledged' — the acknowledged message must be \
-         in delivered_message_ids; the Rust set omits it. got {delivered:?}"
-    );
-    assert!(
-        delivered.contains(&m_vis),
-        "F1: 'visible' is delivered (golden); got {delivered:?}"
-    );
-    assert!(
-        !delivered.contains(&m_inj),
-        "F1: 'injected' is NOT in golden _DELIVERED_MESSAGE_STATUSES — it must be EXCLUDED; the Rust set \
-         wrongly includes it. got {delivered:?}"
-    );
-    let mut sorted = delivered.clone();
-    sorted.sort();
-    assert_eq!(
-        delivered, sorted,
-        "F1: delivered_message_ids must be sorted (golden sorts); got {delivered:?}"
-    );
-}
 // ── collect --result-file INGEST SEMANTIC (golden results.py:58-73) ──────────────────────────────
 // Golden `collect(result_file=…)` INDEPENDENTLY INGESTS a standalone result: a valid result_envelope_v1
 // is `store.add_result(envelope)`'d (results.py:73) regardless of any in-flight delivery, then the

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

@@ -979,32 +979,6 @@ fn requeue_delivery_exhausted_watchers_reopens_only_exhausted() {
     );
 }
 
-// ════════════════════════════════════════════════════════════════════════
-// GROUP P — stuck_cancel owner-gate + invalid alert type refusal.
-// scheduler.py:262-294.
-// ════════════════════════════════════════════════════════════════════════
-
-#[test]
-fn stuck_cancel_none_alert_type_expands_to_all() {
-    // alert_type None == Python "all" → sorted(_ALERT_TYPES) expansion.
-    let ws = tmp_ws("stuckcancel");
-    let out = stuck_cancel(&ws, "w1", None, "leader").unwrap();
-    // The suppression result must enumerate all three alert types.
-    let types = out.get("alert_types").and_then(|v| v.as_array()).map(|a| {
-        a.iter()
-            .filter_map(|x| x.as_str().map(str::to_string))
-            .collect::<Vec<_>>()
-    });
-    assert_eq!(
-        types,
-        Some(vec![
-            "cross_worker_deadlock".to_string(),
-            "idle_fallback".to_string(),
-            "stuck".to_string()
-        ])
-    );
-}
-
 // ════════════════════════════════════════════════════════════════════════
 // GROUP Q — collect intake (results.py:45-167): valid result advances task,
 // returns collected_results + delivered_messages + results counts shape.

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

@@ -219,26 +219,6 @@ impl WorkerRuntimeState {
     }
 }
 
-/// 告警类型 (card §49;`scheduler.py:38` `_ALERT_TYPES`)。`stuck_cancel` 还接 `all` (展开全集)。
-#[derive(Debug, Clone, Copy, PartialEq, Eq, Hash, Serialize, Deserialize)]
-#[serde(rename_all = "snake_case")]
-pub enum AlertType {
-    Stuck,
-    IdleFallback,
-    CrossWorkerDeadlock,
-}
-
-impl AlertType {
-    /// `stuck_cancel(alert_type="all")` → `sorted(_ALERT_TYPES)` 全集 (`scheduler.py:269`)。
-    pub fn all() -> [AlertType; 3] {
-        [
-            AlertType::CrossWorkerDeadlock,
-            AlertType::IdleFallback,
-            AlertType::Stuck,
-        ]
-    }
-}
-
 /// selftest check 状态 (card §47;`diagnose/comms.py`)。§19 必变 enum。
 #[derive(Debug, Clone, Copy, PartialEq, Eq, Hash, Serialize, Deserialize)]
 #[serde(rename_all = "snake_case")]

package/crates/team-agent/src/model/spec.rs

@@ -1135,19 +1135,10 @@ mod tests {
         );
     }
 
-    // 0.5.66 bypass 单源 §4.1:7 触发面共用同一 spec 校验层——compile_team 是它们的共同
-    // 必经点(quick-start/restart/add/fork 都调用),缺字段一律 fail-loud。此处验证
-    // compile 级校验确实 fail-loud,即各触发面(经 compile_team)都被覆盖。
+    // 0.5.66 bypass 单源 §4.1:7 个生命周期入口共享编译后的 spec。
+    // 缺失 role frontmatter 值时由编译器注入 true；此处锁定公共编译入口的默认结果。
     #[test]
-    fn test_all_7_triggers_run_field_check() {
-        // compile_team 是 7 触发面的 spec 校验必经点:
-        //   quick-start(quick_start.rs:233 compile_team)
-        //   restart(rebuild.rs:3245 rebuild_runtime_spec_from_roles → compile_team)
-        //   add-agent(add_agent.rs:331 compile_role_agent)
-        //   clone-agent(clone_agent.rs:62 → add_agent)
-        //   fork-agent(fork_agent.rs:96 validate_spec)
-        //   start-agent / reset-agent(操作已编译 spec,团队创建时经 compile_team 校验)
-        // 验证 compile_team 对缺字段的角色文档 fail-loud。
+    fn test_all_7_triggers_use_compiled_default_bypass() {
         let team = std::env::temp_dir().join(format!(
             "ta-spec-7trigger-{}",
             std::process::id()
@@ -1159,17 +1150,22 @@ mod tests {
             "---\nname: t7\nobjective: 7 trigger.\nprovider: codex\n---\n\nteam.\n",
         )
         .unwrap();
-        // 角色文档故意缺 dangerously_skip_permissions
+        // 角色文档故意省略该字段；纯文本默认必须写入 compiled spec。
         std::fs::write(
             team.join("agents").join("w.md"),
             "---\nname: w\nrole: Worker\nprovider: codex\ntools:\n  - mcp_team\n---\n\nw.\n",
         )
         .unwrap();
-        let err = crate::compiler::compile_team(&team).unwrap_err().to_string();
-        assert!(
-            err.contains("missing front matter field dangerously_skip_permissions")
-                && err.contains("This field must be declared explicitly; it controls whether the agent launches with permission prompts bypassed."),
-            "compile_team must fail-loud on missing field (shared by all 7 triggers); got {err}"
+        let spec = crate::compiler::compile_team(&team)
+            .expect("compile_team must materialize the default bypass value");
+        let agent = spec
+            .get("agents")
+            .and_then(Yaml::as_list)
+            .and_then(|agents| agents.first())
+            .expect("compiled spec has a worker");
+        assert_eq!(
+            agent.get("dangerously_skip_permissions"),
+            Some(&Yaml::Bool(true))
         );
         let _ = std::fs::remove_dir_all(&team);
     }

package/crates/team-agent/src/provider/mod.rs

@@ -51,6 +51,7 @@ pub(crate) mod adapters;
 pub mod approvals;
 pub mod classify;
 pub mod faults;
+pub mod model_catalog;
 pub mod session;
 pub mod session_scan;
 pub mod startup_prompt;