@team-agent/installer 0.5.60 → 0.5.62

package/crates/team-agent/src/compiler/tests.rs

@@ -189,7 +189,7 @@ fn front_matter_non_object_errors() {
 // separators=(",",":"))` with the workspace path templated to __WS__.
 // (team-agent-public v0.2.11, /tmp/probe_compiler.py.)
 
-const BASE_NOPROFILE_JSON: &str = r#"{"version":1,"team":{"name":"doc-team","mode":"supervisor_worker","objective":"Compile role docs.","workspace":"__WS__"},"leader":{"id":"leader","role":"leader","provider":"codex","model":"gpt-5.5","tools":["fs_read","fs_list","mcp_team"],"context_policy":{"keep_user_thread":true,"receive_worker_outputs":"business_messages_and_short_summaries","max_worker_result_tokens":2000}},"agents":[{"id":"implementer","role":"Implementation Engineer","provider":"codex","model":"gpt-5.5","auth_mode":"subscription","working_directory":"__WS__","system_prompt":{"inline":"Implement bounded tasks and report result_envelope_v1.","file":null},"tools":["fs_read","fs_write","execute_bash","mcp_team"],"permission_mode":"restricted","preferred_for":["implementer","Implementation Engineer"],"avoid_for":[],"output_contract":{"format":"result_envelope_v1","required_fields":["task_id","status","summary","artifacts"]}}],"routing":{"default_assignee":"implementer","rules":[{"id":"route-implementer","match":{"assignee":["implementer"]},"assign_to":"implementer","priority":10}]},"communication":{"protocol":"mcp_inbox","topology":"leader_centered","worker_to_worker":true,"ack_timeout_sec":60,"result_format":"result_envelope_v1","message_store":{"sqlite":".team/runtime/team.db","mirror_files":".team/messages"}},"runtime":{"backend":"tmux","display_backend":"adaptive","session_name":"team-doc-team","auto_launch":true,"require_user_approval_before_launch":true,"max_active_agents":1,"startup_order":["implementer"],"dangerous_auto_approve":false,"fast":false,"tick_interval_sec":2,"push_min_interval_sec":60,"stuck_timeout_sec":300},"context":{"state_file":"team_state.md","artifact_dir":".team/artifacts","log_dir":".team/logs","summarization":{"worker_full_logs":"retain_outside_leader_context","state_update":"after_each_result"}},"tasks":[{"id":"task_initial","title":"Initial document-driven team task","type":"implementation","assignee":"implementer","deps":[],"acceptance":["Worker reports valid result_envelope_v1"],"status":"pending","requires_tools":["mcp_team"],"files":[],"risk":"low"}]}"#;
+const BASE_NOPROFILE_JSON: &str = r#"{"version":1,"team":{"name":"doc-team","mode":"supervisor_worker","objective":"Compile role docs.","workspace":"__WS__"},"leader":{"id":"leader","role":"leader","provider":"codex","model":"gpt-5.5","tools":["fs_read","fs_list","mcp_team"],"context_policy":{"keep_user_thread":true,"receive_worker_outputs":"business_messages_and_short_summaries","max_worker_result_tokens":2000}},"agents":[{"id":"implementer","role":"Implementation Engineer","provider":"codex","model":"gpt-5.5","auth_mode":"subscription","working_directory":"__WS__","system_prompt":{"inline":"Implement bounded tasks and report result_envelope_v1.","file":null},"tools":["fs_read","fs_write","execute_bash","mcp_team"],"permission_mode":"restricted","communication_mode":"leader_centric","preferred_for":["implementer","Implementation Engineer"],"avoid_for":[],"output_contract":{"format":"result_envelope_v1","required_fields":["task_id","status","summary","artifacts"]}}],"routing":{"default_assignee":"implementer","rules":[{"id":"route-implementer","match":{"assignee":["implementer"]},"assign_to":"implementer","priority":10}]},"communication":{"protocol":"mcp_inbox","topology":"leader_centered","worker_to_worker":true,"ack_timeout_sec":60,"result_format":"result_envelope_v1","message_store":{"sqlite":".team/runtime/team.db","mirror_files":".team/messages"}},"runtime":{"backend":"tmux","display_backend":"adaptive","session_name":"team-doc-team","auto_launch":true,"require_user_approval_before_launch":true,"max_active_agents":1,"startup_order":["implementer"],"dangerous_auto_approve":false,"fast":false,"tick_interval_sec":2,"push_min_interval_sec":60,"stuck_timeout_sec":300},"context":{"state_file":"team_state.md","artifact_dir":".team/artifacts","log_dir":".team/logs","summarization":{"worker_full_logs":"retain_outside_leader_context","state_update":"after_each_result"}},"tasks":[{"id":"task_initial","title":"Initial document-driven team task","type":"implementation","assignee":"implementer","deps":[],"acceptance":["Worker reports valid result_envelope_v1"],"status":"pending","requires_tools":["mcp_team"],"files":[],"risk":"low"}]}"#;
 
 #[test]
 fn compile_base_noprofile_matches_python_dict_order_and_values() {
@@ -259,7 +259,7 @@ tools:
 Implement bounded tasks.
 ";
 
-const RUNTIME_DEFAULTS_JSON: &str = r#"{"version":1,"team":{"name":"doc-team","mode":"supervisor_worker","objective":"Compile role docs.","workspace":"__WS__"},"leader":{"id":"leader","role":"leader","provider":"codex","model":null,"tools":["fs_read","fs_list","mcp_team"],"context_policy":{"keep_user_thread":true,"receive_worker_outputs":"business_messages_and_short_summaries","max_worker_result_tokens":2000}},"agents":[{"id":"implementer","role":"Implementation Engineer","provider":"codex","model":"gpt-5.4","auth_mode":"subscription","working_directory":"__WS__","system_prompt":{"inline":"Implement bounded tasks.","file":null},"tools":["fs_read","mcp_team"],"permission_mode":"restricted","preferred_for":["implementer","Implementation Engineer"],"avoid_for":[],"output_contract":{"format":"result_envelope_v1","required_fields":["task_id","status","summary","artifacts"]}}],"routing":{"default_assignee":"implementer","rules":[{"id":"route-implementer","match":{"assignee":["implementer"]},"assign_to":"implementer","priority":10}]},"communication":{"protocol":"mcp_inbox","topology":"leader_centered","worker_to_worker":true,"ack_timeout_sec":60,"result_format":"result_envelope_v1","message_store":{"sqlite":".team/runtime/team.db","mirror_files":".team/messages"}},"runtime":{"backend":"tmux","display_backend":"ghostty_window","session_name":"team-doc-team","auto_launch":true,"require_user_approval_before_launch":true,"max_active_agents":1,"startup_order":["implementer"],"dangerous_auto_approve":true,"fast":true,"tick_interval_sec":1,"push_min_interval_sec":3,"stuck_timeout_sec":5},"context":{"state_file":"team_state.md","artifact_dir":".team/artifacts","log_dir":".team/logs","summarization":{"worker_full_logs":"retain_outside_leader_context","state_update":"after_each_result"}},"tasks":[{"id":"task_initial","title":"Initial document-driven team task","type":"implementation","assignee":"implementer","deps":[],"acceptance":["Worker reports valid result_envelope_v1"],"status":"pending","requires_tools":["mcp_team"],"files":[],"risk":"low"}]}"#;
+const RUNTIME_DEFAULTS_JSON: &str = r#"{"version":1,"team":{"name":"doc-team","mode":"supervisor_worker","objective":"Compile role docs.","workspace":"__WS__"},"leader":{"id":"leader","role":"leader","provider":"codex","model":null,"tools":["fs_read","fs_list","mcp_team"],"context_policy":{"keep_user_thread":true,"receive_worker_outputs":"business_messages_and_short_summaries","max_worker_result_tokens":2000}},"agents":[{"id":"implementer","role":"Implementation Engineer","provider":"codex","model":"gpt-5.4","auth_mode":"subscription","working_directory":"__WS__","system_prompt":{"inline":"Implement bounded tasks.","file":null},"tools":["fs_read","mcp_team"],"permission_mode":"restricted","communication_mode":"leader_centric","preferred_for":["implementer","Implementation Engineer"],"avoid_for":[],"output_contract":{"format":"result_envelope_v1","required_fields":["task_id","status","summary","artifacts"]}}],"routing":{"default_assignee":"implementer","rules":[{"id":"route-implementer","match":{"assignee":["implementer"]},"assign_to":"implementer","priority":10}]},"communication":{"protocol":"mcp_inbox","topology":"leader_centered","worker_to_worker":true,"ack_timeout_sec":60,"result_format":"result_envelope_v1","message_store":{"sqlite":".team/runtime/team.db","mirror_files":".team/messages"}},"runtime":{"backend":"tmux","display_backend":"ghostty_window","session_name":"team-doc-team","auto_launch":true,"require_user_approval_before_launch":true,"max_active_agents":1,"startup_order":["implementer"],"dangerous_auto_approve":true,"fast":true,"tick_interval_sec":1,"push_min_interval_sec":3,"stuck_timeout_sec":5},"context":{"state_file":"team_state.md","artifact_dir":".team/artifacts","log_dir":".team/logs","summarization":{"worker_full_logs":"retain_outside_leader_context","state_update":"after_each_result"}},"tasks":[{"id":"task_initial","title":"Initial document-driven team task","type":"implementation","assignee":"implementer","deps":[],"acceptance":["Worker reports valid result_envelope_v1"],"status":"pending","requires_tools":["mcp_team"],"files":[],"risk":"low"}]}"#;
 
 #[test]
 fn compile_runtime_front_matter_defaults_match_python() {
@@ -299,7 +299,7 @@ tools:
 Edit and defend the argument.
 ";
 
-const ALIAS_JSON: &str = r#"{"version":1,"team":{"name":"debate-team","mode":"supervisor_worker","objective":"Compile thin role docs.","workspace":"__WS__"},"leader":{"id":"leader","role":"leader","provider":"codex","model":null,"tools":["fs_read","fs_list","mcp_team"],"context_policy":{"keep_user_thread":true,"receive_worker_outputs":"business_messages_and_short_summaries","max_worker_result_tokens":2000}},"agents":[{"id":"editor","role":"Editor and Defender","provider":"claude_code","model":"claude-sonnet-4-6","auth_mode":"subscription","working_directory":"__WS__","system_prompt":{"inline":"Edit and defend the argument.","file":null},"tools":["mcp_team"],"permission_mode":"restricted","preferred_for":["editor","Editor and Defender"],"avoid_for":[],"output_contract":{"format":"result_envelope_v1","required_fields":["task_id","status","summary","artifacts"]}}],"routing":{"default_assignee":"editor","rules":[{"id":"route-editor","match":{"assignee":["editor"]},"assign_to":"editor","priority":10}]},"communication":{"protocol":"mcp_inbox","topology":"leader_centered","worker_to_worker":true,"ack_timeout_sec":60,"result_format":"result_envelope_v1","message_store":{"sqlite":".team/runtime/team.db","mirror_files":".team/messages"}},"runtime":{"backend":"tmux","display_backend":"none","session_name":"team-debate-team","auto_launch":true,"require_user_approval_before_launch":true,"max_active_agents":1,"startup_order":["editor"],"dangerous_auto_approve":false,"fast":false,"tick_interval_sec":2,"push_min_interval_sec":60,"stuck_timeout_sec":300},"context":{"state_file":"team_state.md","artifact_dir":".team/artifacts","log_dir":".team/logs","summarization":{"worker_full_logs":"retain_outside_leader_context","state_update":"after_each_result"}},"tasks":[{"id":"task_initial","title":"Initial document-driven team task","type":"implementation","assignee":"editor","deps":[],"acceptance":["Worker reports valid result_envelope_v1"],"status":"pending","requires_tools":["mcp_team"],"files":[],"risk":"low"}]}"#;
+const ALIAS_JSON: &str = r#"{"version":1,"team":{"name":"debate-team","mode":"supervisor_worker","objective":"Compile thin role docs.","workspace":"__WS__"},"leader":{"id":"leader","role":"leader","provider":"codex","model":null,"tools":["fs_read","fs_list","mcp_team"],"context_policy":{"keep_user_thread":true,"receive_worker_outputs":"business_messages_and_short_summaries","max_worker_result_tokens":2000}},"agents":[{"id":"editor","role":"Editor and Defender","provider":"claude_code","model":"claude-sonnet-4-6","auth_mode":"subscription","working_directory":"__WS__","system_prompt":{"inline":"Edit and defend the argument.","file":null},"tools":["mcp_team"],"permission_mode":"restricted","communication_mode":"leader_centric","preferred_for":["editor","Editor and Defender"],"avoid_for":[],"output_contract":{"format":"result_envelope_v1","required_fields":["task_id","status","summary","artifacts"]}}],"routing":{"default_assignee":"editor","rules":[{"id":"route-editor","match":{"assignee":["editor"]},"assign_to":"editor","priority":10}]},"communication":{"protocol":"mcp_inbox","topology":"leader_centered","worker_to_worker":true,"ack_timeout_sec":60,"result_format":"result_envelope_v1","message_store":{"sqlite":".team/runtime/team.db","mirror_files":".team/messages"}},"runtime":{"backend":"tmux","display_backend":"none","session_name":"team-debate-team","auto_launch":true,"require_user_approval_before_launch":true,"max_active_agents":1,"startup_order":["editor"],"dangerous_auto_approve":false,"fast":false,"tick_interval_sec":2,"push_min_interval_sec":60,"stuck_timeout_sec":300},"context":{"state_file":"team_state.md","artifact_dir":".team/artifacts","log_dir":".team/logs","summarization":{"worker_full_logs":"retain_outside_leader_context","state_update":"after_each_result"}},"tasks":[{"id":"task_initial","title":"Initial document-driven team task","type":"implementation","assignee":"editor","deps":[],"acceptance":["Worker reports valid result_envelope_v1"],"status":"pending","requires_tools":["mcp_team"],"files":[],"risk":"low"}]}"#;
 
 #[test]
 fn compile_provider_models_claude_code_falls_back_to_claude_alias() {
@@ -332,7 +332,7 @@ tools:
 Implement bounded tasks.
 ";
 
-const BUILTIN_JSON: &str = r#"{"version":1,"team":{"name":"default-model-team","mode":"supervisor_worker","objective":"Compile role docs without model fields.","workspace":"__WS__"},"leader":{"id":"leader","role":"leader","provider":"codex","model":null,"tools":["fs_read","fs_list","mcp_team"],"context_policy":{"keep_user_thread":true,"receive_worker_outputs":"business_messages_and_short_summaries","max_worker_result_tokens":2000}},"agents":[{"id":"implementer","role":"Implementation Engineer","provider":"codex","model":"gpt-5.5","auth_mode":"subscription","working_directory":"__WS__","system_prompt":{"inline":"Implement bounded tasks.","file":null},"tools":["mcp_team"],"permission_mode":"restricted","preferred_for":["implementer","Implementation Engineer"],"avoid_for":[],"output_contract":{"format":"result_envelope_v1","required_fields":["task_id","status","summary","artifacts"]}}],"routing":{"default_assignee":"implementer","rules":[{"id":"route-implementer","match":{"assignee":["implementer"]},"assign_to":"implementer","priority":10}]},"communication":{"protocol":"mcp_inbox","topology":"leader_centered","worker_to_worker":true,"ack_timeout_sec":60,"result_format":"result_envelope_v1","message_store":{"sqlite":".team/runtime/team.db","mirror_files":".team/messages"}},"runtime":{"backend":"tmux","display_backend":"none","session_name":"team-default-model-team","auto_launch":true,"require_user_approval_before_launch":true,"max_active_agents":1,"startup_order":["implementer"],"dangerous_auto_approve":false,"fast":false,"tick_interval_sec":2,"push_min_interval_sec":60,"stuck_timeout_sec":300},"context":{"state_file":"team_state.md","artifact_dir":".team/artifacts","log_dir":".team/logs","summarization":{"worker_full_logs":"retain_outside_leader_context","state_update":"after_each_result"}},"tasks":[{"id":"task_initial","title":"Initial document-driven team task","type":"implementation","assignee":"implementer","deps":[],"acceptance":["Worker reports valid result_envelope_v1"],"status":"pending","requires_tools":["mcp_team"],"files":[],"risk":"low"}]}"#;
+const BUILTIN_JSON: &str = r#"{"version":1,"team":{"name":"default-model-team","mode":"supervisor_worker","objective":"Compile role docs without model fields.","workspace":"__WS__"},"leader":{"id":"leader","role":"leader","provider":"codex","model":null,"tools":["fs_read","fs_list","mcp_team"],"context_policy":{"keep_user_thread":true,"receive_worker_outputs":"business_messages_and_short_summaries","max_worker_result_tokens":2000}},"agents":[{"id":"implementer","role":"Implementation Engineer","provider":"codex","model":"gpt-5.5","auth_mode":"subscription","working_directory":"__WS__","system_prompt":{"inline":"Implement bounded tasks.","file":null},"tools":["mcp_team"],"permission_mode":"restricted","communication_mode":"leader_centric","preferred_for":["implementer","Implementation Engineer"],"avoid_for":[],"output_contract":{"format":"result_envelope_v1","required_fields":["task_id","status","summary","artifacts"]}}],"routing":{"default_assignee":"implementer","rules":[{"id":"route-implementer","match":{"assignee":["implementer"]},"assign_to":"implementer","priority":10}]},"communication":{"protocol":"mcp_inbox","topology":"leader_centered","worker_to_worker":true,"ack_timeout_sec":60,"result_format":"result_envelope_v1","message_store":{"sqlite":".team/runtime/team.db","mirror_files":".team/messages"}},"runtime":{"backend":"tmux","display_backend":"none","session_name":"team-default-model-team","auto_launch":true,"require_user_approval_before_launch":true,"max_active_agents":1,"startup_order":["implementer"],"dangerous_auto_approve":false,"fast":false,"tick_interval_sec":2,"push_min_interval_sec":60,"stuck_timeout_sec":300},"context":{"state_file":"team_state.md","artifact_dir":".team/artifacts","log_dir":".team/logs","summarization":{"worker_full_logs":"retain_outside_leader_context","state_update":"after_each_result"}},"tasks":[{"id":"task_initial","title":"Initial document-driven team task","type":"implementation","assignee":"implementer","deps":[],"acceptance":["Worker reports valid result_envelope_v1"],"status":"pending","requires_tools":["mcp_team"],"files":[],"risk":"low"}]}"#;
 
 #[test]
 fn compile_subscription_without_model_uses_builtin_provider_default() {
@@ -372,7 +372,7 @@ tools:
 Bravo body.
 ";
 
-const TWO_AGENTS_JSON: &str = r#"{"version":1,"team":{"name":"doc-team","mode":"supervisor_worker","objective":"Compile role docs.","workspace":"__WS__"},"leader":{"id":"leader","role":"leader","provider":"codex","model":"gpt-5.5","tools":["fs_read","fs_list","mcp_team"],"context_policy":{"keep_user_thread":true,"receive_worker_outputs":"business_messages_and_short_summaries","max_worker_result_tokens":2000}},"agents":[{"id":"alpha","role":"Alpha Worker","provider":"codex","model":"gpt-5.5","auth_mode":"subscription","working_directory":"__WS__","system_prompt":{"inline":"Alpha body.","file":null},"tools":["mcp_team"],"permission_mode":"restricted","preferred_for":["alpha","Alpha Worker"],"avoid_for":[],"output_contract":{"format":"result_envelope_v1","required_fields":["task_id","status","summary","artifacts"]}},{"id":"bravo","role":"Bravo Worker","provider":"codex","model":"gpt-5.5","auth_mode":"subscription","working_directory":"__WS__","system_prompt":{"inline":"Bravo body.","file":null},"tools":["mcp_team"],"permission_mode":"restricted","preferred_for":["bravo","Bravo Worker"],"avoid_for":[],"output_contract":{"format":"result_envelope_v1","required_fields":["task_id","status","summary","artifacts"]}}],"routing":{"default_assignee":"alpha","rules":[{"id":"route-alpha","match":{"assignee":["alpha"]},"assign_to":"alpha","priority":10},{"id":"route-bravo","match":{"assignee":["bravo"]},"assign_to":"bravo","priority":10}]},"communication":{"protocol":"mcp_inbox","topology":"leader_centered","worker_to_worker":true,"ack_timeout_sec":60,"result_format":"result_envelope_v1","message_store":{"sqlite":".team/runtime/team.db","mirror_files":".team/messages"}},"runtime":{"backend":"tmux","display_backend":"adaptive","session_name":"team-doc-team","auto_launch":true,"require_user_approval_before_launch":true,"max_active_agents":2,"startup_order":["alpha","bravo"],"dangerous_auto_approve":false,"fast":false,"tick_interval_sec":2,"push_min_interval_sec":60,"stuck_timeout_sec":300},"context":{"state_file":"team_state.md","artifact_dir":".team/artifacts","log_dir":".team/logs","summarization":{"worker_full_logs":"retain_outside_leader_context","state_update":"after_each_result"}},"tasks":[{"id":"task_initial","title":"Initial document-driven team task","type":"implementation","assignee":"alpha","deps":[],"acceptance":["Worker reports valid result_envelope_v1"],"status":"pending","requires_tools":["mcp_team"],"files":[],"risk":"low"}]}"#;
+const TWO_AGENTS_JSON: &str = r#"{"version":1,"team":{"name":"doc-team","mode":"supervisor_worker","objective":"Compile role docs.","workspace":"__WS__"},"leader":{"id":"leader","role":"leader","provider":"codex","model":"gpt-5.5","tools":["fs_read","fs_list","mcp_team"],"context_policy":{"keep_user_thread":true,"receive_worker_outputs":"business_messages_and_short_summaries","max_worker_result_tokens":2000}},"agents":[{"id":"alpha","role":"Alpha Worker","provider":"codex","model":"gpt-5.5","auth_mode":"subscription","working_directory":"__WS__","system_prompt":{"inline":"Alpha body.","file":null},"tools":["mcp_team"],"permission_mode":"restricted","communication_mode":"leader_centric","preferred_for":["alpha","Alpha Worker"],"avoid_for":[],"output_contract":{"format":"result_envelope_v1","required_fields":["task_id","status","summary","artifacts"]}},{"id":"bravo","role":"Bravo Worker","provider":"codex","model":"gpt-5.5","auth_mode":"subscription","working_directory":"__WS__","system_prompt":{"inline":"Bravo body.","file":null},"tools":["mcp_team"],"permission_mode":"restricted","communication_mode":"leader_centric","preferred_for":["bravo","Bravo Worker"],"avoid_for":[],"output_contract":{"format":"result_envelope_v1","required_fields":["task_id","status","summary","artifacts"]}}],"routing":{"default_assignee":"alpha","rules":[{"id":"route-alpha","match":{"assignee":["alpha"]},"assign_to":"alpha","priority":10},{"id":"route-bravo","match":{"assignee":["bravo"]},"assign_to":"bravo","priority":10}]},"communication":{"protocol":"mcp_inbox","topology":"leader_centered","worker_to_worker":true,"ack_timeout_sec":60,"result_format":"result_envelope_v1","message_store":{"sqlite":".team/runtime/team.db","mirror_files":".team/messages"}},"runtime":{"backend":"tmux","display_backend":"adaptive","session_name":"team-doc-team","auto_launch":true,"require_user_approval_before_launch":true,"max_active_agents":2,"startup_order":["alpha","bravo"],"dangerous_auto_approve":false,"fast":false,"tick_interval_sec":2,"push_min_interval_sec":60,"stuck_timeout_sec":300},"context":{"state_file":"team_state.md","artifact_dir":".team/artifacts","log_dir":".team/logs","summarization":{"worker_full_logs":"retain_outside_leader_context","state_update":"after_each_result"}},"tasks":[{"id":"task_initial","title":"Initial document-driven team task","type":"implementation","assignee":"alpha","deps":[],"acceptance":["Worker reports valid result_envelope_v1"],"status":"pending","requires_tools":["mcp_team"],"files":[],"risk":"low"}]}"#;
 
 #[test]
 fn compile_two_agents_sorted_by_filename_with_routing_and_startup_order() {

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

@@ -1,3 +1,4 @@
+//!
 //! step 6 · compiler — doc-driven team source → canonical `team.spec` dict.
 //!
 //! Truth source (READ-ONLY): `team-agent-public` @ v0.2.11, `team_agent/compiler.py`.
@@ -26,6 +27,7 @@
 use std::fs;
 use std::path::Path;
 
+use crate::communication_mode::CommunicationMode;
 use crate::model::enums::{Provider, ProviderEffort};
 use crate::model::yaml::Value;
 use crate::model::{paths, spec, yaml, ModelError};
@@ -133,6 +135,8 @@ pub fn compile_team(team_dir: &Path) -> Result<Value, ModelError> {
     }
 
     let (team_meta, team_body) = read_front_matter(&team_md)?;
+    let team_communication_mode =
+        communication_mode_field(&team_meta, &team_md)?.unwrap_or_default();
     let mut role_paths = Vec::new();
     if agents_dir.is_dir() {
         for entry in fs::read_dir(&agents_dir)
@@ -170,7 +174,8 @@ pub fn compile_team(team_dir: &Path) -> Result<Value, ModelError> {
     let mut agents = Vec::new();
     let mut agent_ids = Vec::new();
     for path in role_paths {
-        let compiled = compile_role_agent(&path, &team_meta, &workspace_s)?;
+        let compiled =
+            compile_role_agent_with_mode(&path, &team_meta, &workspace_s, team_communication_mode)?;
         agent_ids.push(compiled.id);
         agents.push(compiled.agent);
     }
@@ -371,8 +376,21 @@ pub fn compile_role_agent(
     role_path: &Path,
     team_meta: &Value,
     workspace_s: &str,
+) -> Result<CompiledRole, ModelError> {
+    let team_communication_mode =
+        communication_mode_field(team_meta, role_path)?.unwrap_or_default();
+    compile_role_agent_with_mode(role_path, team_meta, workspace_s, team_communication_mode)
+}
+
+fn compile_role_agent_with_mode(
+    role_path: &Path,
+    team_meta: &Value,
+    workspace_s: &str,
+    team_communication_mode: CommunicationMode,
 ) -> Result<CompiledRole, ModelError> {
     let (meta, body) = read_front_matter(role_path)?;
+    let communication_mode =
+        communication_mode_field(&meta, role_path)?.unwrap_or(team_communication_mode);
     let id = required_string(&meta, role_path, "name")?;
     let role = required_string(&meta, role_path, "role")?;
     let provider = required_string(&meta, role_path, "provider")?;
@@ -404,6 +422,10 @@ pub fn compile_role_agent(
         ),
         ("tools", list_str(tools)),
         ("permission_mode", Value::Str("restricted".to_string())),
+        (
+            "communication_mode",
+            Value::Str(communication_mode.as_str().to_string()),
+        ),
         ("preferred_for", list_str(vec![id.clone(), role.clone()])),
         ("avoid_for", Value::List(Vec::new())),
         (
@@ -469,6 +491,37 @@ pub fn compile_role_agent(
     })
 }
 
+fn communication_mode_field(
+    meta: &Value,
+    path: &Path,
+) -> Result<Option<CommunicationMode>, ModelError> {
+    let Some(raw) = meta.get("communication_mode") else {
+        return Ok(None);
+    };
+    let Some(value) = raw.as_str() else {
+        return Err(ModelError::Validation(format!(
+            "{}: unknown communication_mode (allowed: {})",
+            path.display(),
+            CommunicationMode::ALL
+                .iter()
+                .map(|mode| mode.as_str())
+                .collect::<Vec<_>>()
+                .join("|")
+        )));
+    };
+    CommunicationMode::parse(value).map(Some).ok_or_else(|| {
+        ModelError::Validation(format!(
+            "{}: unknown communication_mode '{value}' (allowed: {})",
+            path.display(),
+            CommunicationMode::ALL
+                .iter()
+                .map(|mode| mode.as_str())
+                .collect::<Vec<_>>()
+                .join("|")
+        ))
+    })
+}
+
 fn map(items: Vec<(&str, Value)>) -> Value {
     Value::Map(items.into_iter().map(|(k, v)| (k.to_string(), v)).collect())
 }

package/crates/team-agent/src/conpty/backend.rs

@@ -1,3 +1,4 @@
+//!
 //! `ConPtyBackend` — implements the existing `Transport` trait using the
 //! named-pipe protocol in `super::protocol`.
 //!

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

@@ -1,3 +1,4 @@
+//!
 //! 0.5.x Windows-native transport Phase 1: ConPTY backend.
 //!
 //! Wire protocol + portable shim state machine live in the

package/crates/team-agent/src/coordinator/backoff.rs

@@ -1,3 +1,4 @@
+//!
 //! daemon 主循环面(`__main__.py`)—— 退避序列 + tick 间隔解析 + 子进程入口。
 
 use thiserror::Error;
@@ -10,7 +11,7 @@ use crate::provider::ProviderAdapter;
 use super::health::{coordinator_pid_path, write_coordinator_metadata};
 use super::tick::{write_coordinator_heartbeat, TickError, TickReport, HEARTBEAT_STATUS_PANIC};
 use super::types::{
-    ErrorLists, MetadataSource, Pid, ProviderRegistry, WorkspacePath, BACKOFF_MAX_SEC,
+    MetadataSource, Pid, ProviderRegistry, WorkspacePath, BACKOFF_MAX_SEC,
     DEFAULT_TICK_INTERVAL_SEC,
 };
 use super::Coordinator;
@@ -122,7 +123,7 @@ pub fn run_daemon(args: DaemonArgs) -> Result<(), DaemonError> {
     run_daemon_with_coordinator_and_boot_tmux(&args, &coordinator, Some(tmux_metadata))
 }
 
-pub fn run_daemon_with_coordinator(
+pub(crate) fn run_daemon_with_coordinator(
     args: &DaemonArgs,
     coordinator: &Coordinator,
 ) -> Result<(), DaemonError> {
@@ -519,14 +520,14 @@ fn current_ppid() -> u32 {
 
 /// 计算 tick 间隔(`_tick_interval`,`__main__.py:104-115`)。读 spec `runtime.tick_interval_sec`,
 /// 缺失/出错 → `DEFAULT_TICK_INTERVAL_SEC`;并确保 schema 存在(`MessageStore(workspace)`)。
-pub fn resolve_tick_interval(workspace: &WorkspacePath) -> Result<f64, TickError> {
+pub(super) fn resolve_tick_interval(workspace: &WorkspacePath) -> Result<f64, TickError> {
     let _ = MessageStore::open(workspace.as_path())?;
     Ok(DEFAULT_TICK_INTERVAL_SEC)
 }
 
 /// 退避序列(`__main__.py:65`):`min(interval * 2^min(failures-1, 5), 60.0)` → 5→10→20→40→60→60s。
 /// unit test 锁死本序列(card §85)。**纯函数,无 I/O,可直接 impl 钉死**(但 ROUND-0 仍占位)。
-pub fn backoff_sleep_sec(interval: f64, consecutive_failures: u32) -> f64 {
+pub(super) fn backoff_sleep_sec(interval: f64, consecutive_failures: u32) -> f64 {
     let failures = consecutive_failures.saturating_sub(1).min(5);
     let exp = i32::try_from(failures).unwrap_or(5);
     (interval * 2f64.powi(exp)).min(BACKOFF_MAX_SEC)
@@ -538,10 +539,6 @@ impl ProviderRegistry for RealProviderRegistry {
     fn adapter_for(&self, provider: Provider) -> Box<dyn ProviderAdapter> {
         crate::provider::get_adapter(provider)
     }
-
-    fn error_lists(&self, _provider: Provider) -> ErrorLists {
-        ErrorLists::default()
-    }
 }
 
 fn sleep_seconds(seconds: f64) {

package/crates/team-agent/src/coordinator/conpty_shim.rs

@@ -1,3 +1,4 @@
+//!
 //! Windows ConPTY shim lifecycle manager.
 //!
 //! 0.5.x Windows portability Batch 6 Option A. The design's
@@ -160,7 +161,7 @@ pub enum ShimError {
 ///    installations.
 /// 2. Env override `TEAM_AGENT_WINDOWS_SHIM_PATH` — tests + CI
 ///    harness (Batch 6 SSH real-machine harness sets this).
-pub fn locate_shim_binary() -> Result<PathBuf, ShimError> {
+fn locate_shim_binary() -> Result<PathBuf, ShimError> {
     if let Ok(explicit) = std::env::var("TEAM_AGENT_WINDOWS_SHIM_PATH") {
         let p = PathBuf::from(&explicit);
         if p.exists() {
@@ -218,7 +219,7 @@ fn fresh_pipe_token() -> String {
 /// downstream `transport_factory::conpty_pipe_ready(...)` opens on
 /// the next factory resolve. The `pipe_token` is NOT persisted (CR
 /// C-1); it stays in `ShimHandle` memory only.
-pub fn spawn_shim_and_handshake(
+fn spawn_shim_and_handshake(
     workspace: &Path,
     team_key: &str,
     workspace_hash: &str,

package/crates/team-agent/src/coordinator/health.rs

@@ -1,3 +1,4 @@
+//!
 //! coordinator 健康/身份 & 只读可观测面:metadata 身份原语 + coordinator 路径 + watch 实时流。
 
 use std::io::{Read, Seek, SeekFrom};

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

@@ -1,3 +1,4 @@
+//!
 //! step 12 · coordinator — daemon lifecycle / single-tick orchestration SKELETON (ROUND-0).
 //!
 //! Card: `docs/phase0/subsystems/12-coordinator.md`.
@@ -10,8 +11,7 @@
 //!   - `coordinator/metadata.py`     (COORDINATOR_PROTOCOL_VERSION=2, pid_is_running, read/write/ok meta)
 //!   - `coordinator/paths.py`        (coordinator.pid / coordinator.json / coordinator.log paths)
 //!   - `watch/__init__.py`           (run_watch / collect_watch_lines / render_event_line / WatchCursor)
-//!   - `abnormal_track.py`           (Gap 32 §4 provider-neutral abnormal track:
-//!     process_abnormal_records / detect_whole_team_gone)
+//!   - `abnormal_track.py`           (Gap 32 §4 provider-neutral whole-team-gone detection)
 //!
 //! 职责(card §职责):per-workspace daemon 生命周期 + 单次 tick 编排。tick 按固定顺序把
 //! step 8-11 的原子操作串成一个只读 + 投递既定 obligation 的回路 —— **绝不**在无 pending

package/crates/team-agent/src/coordinator/orphan.rs

@@ -1,18 +1,11 @@
-//! 孤儿自终止判定(Gap 37b)+ provider-neutral 异常轨(abnormal_track:整队消失检测 + 去重通知)。
-
-use std::hash::{Hash, Hasher};
-
-use serde_json::Value;
-
-use crate::model::enums::Provider;
-use crate::provider::{ProcessLiveness, Signature, TurnId, TurnState};
+//!
+//! 孤儿自终止判定(Gap 37b)+ provider-neutral 整队消失检测。
 
 use super::types::{
-    AbnormalDecision, AbnormalError, AbnormalNotification, AbnormalNotificationState,
-    AbnormalProcessOutput, MarkerStore, ProviderRegistry, TeamPresenceSnapshot, WholeTeamGoneClass,
-    WholeTeamGoneReport, WorkspacePath,
+    MarkerStore, TeamPresenceSnapshot, WholeTeamGoneClass, WholeTeamGoneReport, WorkspacePath,
 };
 
+///
 /// 孤儿自终止判定(`__main__.py:51-59`,Gap 37b)。仅当
 /// `current_ppid != initial_ppid ∧ current_ppid == 1 ∧ !workspace.exists()` 三者**同时**成立 → true。
 /// 少一个条件都不能误杀正常 daemon(card §91)。
@@ -28,104 +21,7 @@ pub fn should_orphan_self_terminate(
 // abnormal_track 公共面(abnormal_track.py)—— Gap 32 §4 provider-neutral
 // ===========================================================================
 
-/// #236 dedicated `worker.abnormal_exit` surface.
 ///
-/// This is intentionally separate from `process_abnormal_records`: the generic
-/// abnormal path is transcript-only, while `worker.abnormal_exit` requires a fresh
-/// latest explicit provider error before leader notification.
-#[derive(Debug, Clone, PartialEq, Eq, serde::Serialize, serde::Deserialize)]
-#[serde(tag = "event")]
-pub enum WorkerAbnormalExitSurface {
-    #[serde(rename = "worker.abnormal_exit")]
-    WorkerAbnormalExit {
-        agent_id: String,
-        provider: Provider,
-        path: String,
-        provider_process_dead: bool,
-        latest_explicit_error: bool,
-        signature: Signature,
-        turn_id: Option<TurnId>,
-        process_liveness: ProcessLiveness,
-    },
-    #[serde(rename = "abnormal_exit.single_signal_suppressed")]
-    SingleSignalSuppressed {
-        agent_id: String,
-        provider: Provider,
-        path: String,
-        provider_process_dead: bool,
-        latest_explicit_error: bool,
-        reason: String,
-    },
-}
-
-/// `process_abnormal_records`(`abnormal_track.py:14`)。把原始 provider session records 经
-/// provider reader 翻译成结构化 fault facts(本 module **不命名 provider、不读屏**),catch-bias
-/// (C9)+ `(signature, turn_id|fingerprint)` 去重(C8)。`registry` 携带 provider + 黑白名单。
-pub fn process_abnormal_records(
-    records: &[Value],
-    registry: &dyn ProviderRegistry,
-    provider: Provider,
-    notification_state: &AbnormalNotificationState,
-) -> Result<AbnormalProcessOutput, AbnormalError> {
-    let lists = registry.error_lists(provider);
-    let mut state = notification_state.clone();
-    let mut notifications = Vec::new();
-    for record in records {
-        let raw = record
-            .get("raw")
-            .and_then(Value::as_str)
-            .unwrap_or_else(|| record.as_str().unwrap_or(""))
-            .to_string();
-        let signature = record
-            .get("signature")
-            .and_then(Value::as_str)
-            .unwrap_or("unknown")
-            .to_string();
-        let signature_lower = signature.to_lowercase();
-        let turn_id = record
-            .get("turn_id")
-            .and_then(Value::as_str)
-            .map(TurnId::new);
-        let bucket = match &turn_id {
-            Some(id) => id.as_str().to_string(),
-            None => stable_fingerprint(record),
-        };
-        let key = format!("{signature}\0{bucket}");
-        if state.seen.contains(&key) {
-            continue;
-        }
-        state.seen.insert(key);
-        if lists.whitelist.iter().any(|needle| raw.contains(needle)) {
-            continue;
-        }
-        let decision =
-            if lists.blacklist.iter().any(|needle| {
-                raw.contains(needle) || signature_lower.contains(&needle.to_lowercase())
-            }) {
-                AbnormalDecision::NotifyBlacklist
-            } else {
-                AbnormalDecision::NotifyDefault
-            };
-        let kind = record.get("kind").and_then(Value::as_str);
-        notifications.push(AbnormalNotification {
-            signature,
-            turn_id,
-            state: if kind == Some("approval") {
-                TurnState::BlockedOnHuman
-            } else {
-                TurnState::Abnormal
-            },
-            decision,
-            provider: Some(provider),
-            raw: record.clone(),
-        });
-    }
-    Ok(AbnormalProcessOutput {
-        notifications,
-        notification_state: state,
-    })
-}
-
 /// `detect_whole_team_gone`(`abnormal_track.py:91`,C10/C13)。coordinator-independent 整队消失检测:
 /// 全死(coordinator + leader + 所有 worker + 所有 session)且非 clean_shutdown/restart →
 /// 写 durable marker + 延迟到下条 leader 命令再 escalate。clean/restart 静默。
@@ -166,14 +62,6 @@ pub fn detect_whole_team_gone(
     )
 }
 
-fn stable_fingerprint(value: &Value) -> String {
-    let mut hasher = std::collections::hash_map::DefaultHasher::new();
-    serde_json::to_string(value)
-        .unwrap_or_else(|_| value.to_string())
-        .hash(&mut hasher);
-    format!("{:016x}", hasher.finish())
-}
-
 fn whole_team_report(
     whole_team_gone: bool,
     classification: WholeTeamGoneClass,

package/crates/team-agent/src/coordinator/runtime_detectors.rs

@@ -1,3 +1,4 @@
+//!
 use std::collections::BTreeMap;
 use std::path::Path;
 

package/crates/team-agent/src/coordinator/runtime_observation.rs

@@ -1,3 +1,4 @@
+//!
 //! Shared coordinator runtime observation seam.
 //!
 //! S0 only defines the typed capture/result surface. Lane 1 fills capture facts;

package/crates/team-agent/src/coordinator/steps/abnormal.rs

@@ -1,3 +1,4 @@
+//!
 //! unit-11 (Stage 4) — coordinator tick `abnormal` step group.
 //!
 //! Abnormal-exit detection + classification step extracted from
@@ -15,6 +16,7 @@ use super::super::health::pid_is_running;
 use super::super::tick::TickError;
 use super::super::types::Pid;
 
+///
 /// #236 `worker.abnormal_exit` watcher.
 ///
 /// Notify only when the bounded transcript/rollout tail contains a latest explicit
@@ -967,6 +969,7 @@ fn abnormal_last_error_observation_cohort(state: &Value, agent_id: &str) -> Opti
 /// P1: Python `_TAIL_BYTES` parity (idle_takeover_wiring.py:13) — RS must not read less.
 const ABNORMAL_TAIL_BYTES: u64 = 131_072;
 
+///
 /// P1: bounded tail read; a partial first line is harmless (the consumer only parses
 /// the latest complete JSONL record) and lossy UTF-8 keeps a mid-codepoint seek safe.
 pub(crate) fn read_tail_text(path: &Path, max_bytes: u64) -> std::io::Result<String> {
@@ -1347,6 +1350,7 @@ impl ApiErrorRecoveryClass {
     }
 }
 
+///
 /// 0.5.36 §7.1: rules keyed on status/error text.
 pub(crate) fn classify_api_error_recovery(
     signature: &str,
@@ -1871,6 +1875,7 @@ fn process_api_error_recovery_intent(
 // path (which owns its own save).
 // ─────────────────────────────────────────────────────────────────────────
 
+///
 /// 0.5.36 §7.3: process all due recovery intents. Called from tick.rs
 /// AFTER atomic_save has flushed the detector-written intent. Reloads a
 /// fresh state each call so the caller's stale in-memory state cannot
@@ -2168,7 +2173,7 @@ fn write_recovery_intent_result(workspace: &Path, agent_id: &str, update: Recove
 mod tests {
     use super::*;
     use crate::coordinator::tick::Coordinator;
-    use crate::coordinator::types::{ErrorLists, ProviderRegistry, WorkspacePath};
+    use crate::coordinator::types::{ProviderRegistry, WorkspacePath};
     use std::io::Write as _;
 
     struct NormalRegistry;
@@ -2180,10 +2185,6 @@ mod tests {
         ) -> Box<dyn crate::provider::ProviderAdapter> {
             crate::provider::get_adapter(provider)
         }
-
-        fn error_lists(&self, _provider: crate::provider::Provider) -> ErrorLists {
-            ErrorLists::default()
-        }
     }
     #[test]
     fn abnormal_stale_error_baselines_then_fresh_alive_error_notifies() {

package/crates/team-agent/src/coordinator/steps/delivery.rs

@@ -1,3 +1,4 @@
+//!
 //! unit-11 (Stage 4) — coordinator tick `delivery` step group.
 //!
 //! Future home for the message-delivery FSM ticks from

package/crates/team-agent/src/coordinator/steps/health_sync.rs

@@ -1,3 +1,4 @@
+//!
 //! unit-11 (Stage 4) — coordinator tick `health_sync` step group.
 //!
 //! Future home for worker-liveness reconciliation steps from

package/crates/team-agent/src/coordinator/steps/mod.rs

@@ -1,3 +1,4 @@
+//!
 //! unit-11 (Stage 4) — coordinator tick steps namespace.
 //!
 //! The 3083 LOC `coordinator/tick.rs` orchestrator runs an ordered
@@ -47,6 +48,7 @@ impl TickStepGroup {
         }
     }
 
+    ///
     /// Canonical ordering as a const slice — exhaustive over the enum.
     pub fn ordered() -> &'static [TickStepGroup] {
         &[

package/crates/team-agent/src/coordinator/steps/persist.rs

@@ -1,3 +1,4 @@
+//!
 //! unit-11 (Stage 4) — coordinator tick `persist` step group.
 //!
 //! Future home for the durable state-persistence step at the end of each

package/crates/team-agent/src/coordinator/steps/runtime_prompts.rs

@@ -1,3 +1,4 @@
+//!
 //! unit-11 (Stage 4) — coordinator tick `runtime_prompts` step group.
 //!
 //! Future home for the startup / approval / abnormal-exit prompt steps

package/crates/team-agent/src/coordinator/steps/session_gate.rs

@@ -1,3 +1,4 @@
+//!
 //! unit-11 (Stage 4) — coordinator tick `session_gate` step group.
 //!
 //! Future home for the provider session-capture + readiness-gate steps