@team-agent/installer 0.4.10 → 0.5.0

package/Cargo.lock

@@ -566,7 +566,7 @@ dependencies = [
 
 [[package]]
 name = "team-agent"
-version = "0.4.10"
+version = "0.5.0"
 dependencies = [
  "anyhow",
  "chrono",

package/Cargo.toml

@@ -9,7 +9,7 @@ members = ["crates/team-agent"]
 
 [workspace.package]
 edition = "2021"
-version = "0.4.10"
+version = "0.5.0"
 license = "AGPL-3.0"
 rust-version = "1.95"
 

package/crates/team-agent/src/cli/adapters.rs

@@ -993,6 +993,7 @@ fn command_version(command: &str) -> Value {
 }
 
 fn provider_command(provider: &str) -> &str {
+    // E2E provider command grammar, not persisted provider identity parsing.
     match provider {
         "claude" | "claude_code" => "claude",
         "gemini" | "gemini_cli" => "gemini",

package/crates/team-agent/src/cli/diagnose.rs

@@ -1,5 +1,6 @@
 //! diagnose/preflight/wait-ready CLI helpers.
 use super::*;
+use crate::provider::wire::{command_name, provider_wire};
 use crate::transport::Transport;
 
 pub(crate) fn diagnose_runtime(state: &Value, backend: &dyn Transport) -> (Value, Value) {
@@ -781,23 +782,9 @@ pub(crate) fn provider_doctor_checks() -> Value {
     Value::Object(providers)
 }
 
-fn provider_wire(provider: crate::provider::Provider) -> &'static str {
-    match provider {
-        crate::provider::Provider::Claude => "claude",
-        crate::provider::Provider::ClaudeCode => "claude_code",
-        crate::provider::Provider::Codex => "codex",
-        crate::provider::Provider::Copilot => "copilot",
-        crate::provider::Provider::GeminiCli => "gemini_cli",
-        crate::provider::Provider::Fake => "fake",
-    }
-}
-
 fn provider_command(provider: crate::provider::Provider) -> &'static str {
     match provider {
-        crate::provider::Provider::Claude | crate::provider::Provider::ClaudeCode => "claude",
-        crate::provider::Provider::Codex => "codex",
-        crate::provider::Provider::Copilot => "copilot",
-        crate::provider::Provider::GeminiCli => "gemini",
         crate::provider::Provider::Fake => "team-agent fake-worker",
+        other => command_name(other),
     }
 }

package/crates/team-agent/src/cli/emit.rs

@@ -31,7 +31,7 @@ pub fn run(argv: &[String], cwd: &Path) -> ExitCode {
     let Some(command) = argv.first().map(String::as_str) else {
         return emit_missing_subcommand_usage();
     };
-    if command == "codex" || command == "claude" || command == "copilot" {
+    if is_leader_passthrough_command(command) {
         return match cmd_leader_passthrough(command, &argv[1..], cwd) {
             Ok(result) => emit_result(result),
             Err(error) => emit_cli_error(command, &argv[1..], cwd, &error),
@@ -189,6 +189,13 @@ const DISPATCH_COMMANDS: &[&str] = &[
 ];
 
 const SPEC_ONLY_HELP_COMMANDS: &[&str] = &["start", "purge-agent"];
+// Command grammar, not provider identity parsing: these are top-level CLI
+// passthrough verbs for starting a leader under a provider executable.
+const LEADER_PASSTHROUGH_COMMANDS: &[&str] = &["codex", "claude", "copilot"];
+
+fn is_leader_passthrough_command(command: &str) -> bool {
+    LEADER_PASSTHROUGH_COMMANDS.contains(&command)
+}
 
 fn emit_missing_subcommand_usage() -> ExitCode {
     emit_usage_error("the following arguments are required: {codex,claude,...,doctor}");
@@ -206,7 +213,7 @@ fn is_known_subcommand(command: &str) -> bool {
 fn command_help(command: Option<&str>) -> String {
     match command {
         None => {
-            let mut commands = vec!["codex", "claude", "copilot"];
+            let mut commands = LEADER_PASSTHROUGH_COMMANDS.to_vec();
             commands.extend_from_slice(DISPATCH_COMMANDS);
             commands.extend_from_slice(SPEC_ONLY_HELP_COMMANDS);
             format!(
@@ -276,7 +283,7 @@ fn emit_unknown_subcommand_usage(command: &str) -> ExitCode {
 
 /// 在已知子命令里找与 `input` 最接近的一个(Levenshtein ≤ 阈值)。无足够接近者 → None。
 fn nearest_subcommand(input: &str) -> Option<&'static str> {
-    let mut candidates: Vec<&'static str> = vec!["codex", "claude", "copilot"];
+    let mut candidates: Vec<&'static str> = LEADER_PASSTHROUGH_COMMANDS.to_vec();
     candidates.extend_from_slice(DISPATCH_COMMANDS);
     candidates.extend_from_slice(SPEC_ONLY_HELP_COMMANDS);
     // 阈值随长度放宽,但短词收紧,避免 'x' 误配任何东西。
@@ -868,7 +875,7 @@ fn resolve_cli_path(cwd: &Path, path: &Path) -> PathBuf {
 
 fn send_args(args: &[String], cwd: &Path) -> Result<SendArgs, CliError> {
     let parsed = parse_args(args);
-    let target = if parsed.targets.is_some() {
+    let target = if parsed.targets.is_some() || parsed.pane.is_some() {
         None
     } else {
         parsed.positionals.first().cloned()
@@ -1747,6 +1754,57 @@ mod tests {
         );
     }
 
+    #[test]
+    fn send_pane_positionals_are_message_not_target() {
+        let cwd = tmp_workspace();
+        let args = send_args(&cli_argv(&["--pane", "%1596", "hello"]), &cwd).unwrap();
+        assert_eq!(args.pane.as_deref(), Some("%1596"));
+        assert_eq!(args.target, None);
+        assert_eq!(args.targets, None);
+        assert_eq!(args.message, vec!["hello".to_string()]);
+
+        let args = send_args(
+            &cli_argv(&["--pane", "%1596", "multi", "word", "message"]),
+            &cwd,
+        )
+        .unwrap();
+        assert_eq!(args.target, None);
+        assert_eq!(
+            args.message,
+            vec![
+                "multi".to_string(),
+                "word".to_string(),
+                "message".to_string()
+            ]
+        );
+
+        let _ = std::fs::remove_dir_all(&cwd);
+    }
+
+    #[test]
+    fn send_pane_still_rejects_to_and_empty_message() {
+        let cwd = tmp_workspace();
+        let args = send_args(
+            &cli_argv(&["--pane", "%1596", "--to", "worker", "hello"]),
+            &cwd,
+        )
+        .unwrap();
+        let err = cmd_send(&args).unwrap_err();
+        assert!(
+            matches!(err, CliError::Usage(ref message) if message.contains("--pane and TARGET/--to are mutually exclusive")),
+            "expected --pane/--to mutual-exclusion usage error, got {err:?}"
+        );
+
+        let args = send_args(&cli_argv(&["--pane", "%1596"]), &cwd).unwrap();
+        let err = cmd_send(&args).unwrap_err();
+        assert!(
+            matches!(err, CliError::Usage(ref message) if message == "--pane requires a non-empty message"),
+            "expected empty-message usage error, got {err:?}"
+        );
+
+        let _ = std::fs::remove_dir_all(&cwd);
+    }
+
     #[test]
     fn ux_quick_start_workspace_resolves_relative_agents_dir_inside_workspace() {
         let cwd = tmp_workspace();

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

@@ -29,6 +29,10 @@ use std::path::Path;
 use crate::model::enums::{Provider, ProviderEffort};
 use crate::model::yaml::Value;
 use crate::model::{paths, spec, yaml, ModelError};
+use crate::provider::wire::{
+    builtin_provider_model as wire_builtin_provider_model, is_claude_family,
+    parse_canonical_provider, provider_model_keys,
+};
 
 pub const IGNORED_OWNER_TEAM_ID_FIELD: &str = "owner_team_id";
 
@@ -417,16 +421,8 @@ pub fn compile_role_agent(
                 _ => None,
             })
             .unwrap_or("");
-        let provider_enum = match provider_str {
-            "claude" => Provider::Claude,
-            "claude_code" => Provider::ClaudeCode,
-            "codex" => Provider::Codex,
-            "copilot" => Provider::Copilot,
-            "gemini_cli" => Provider::GeminiCli,
-            "fake" => Provider::Fake,
-            _ => Provider::Codex, // unknown providers caught elsewhere
-        };
-        if effort.is_claude_only() && !matches!(provider_enum, Provider::Claude | Provider::ClaudeCode) {
+        let provider_enum = parse_canonical_provider(provider_str).unwrap_or(Provider::Codex);
+        if effort.is_claude_only() && !is_claude_family(provider_enum) {
             return Err(ModelError::Validation(format!(
                 "{}: effort '{}' is only supported by claude/claude_code (provider: {provider_str})",
                 role_path.display(),
@@ -543,19 +539,14 @@ fn resolve_model(role_meta: &Value, team_meta: &Value, provider: &str) -> Value
 
 fn provider_model(team_meta: &Value, provider: &str) -> Option<String> {
     let models = team_meta.get("provider_models")?;
-    string_field(models, provider).or_else(|| match provider {
-        "claude_code" => string_field(models, "claude"),
-        "claude" => string_field(models, "claude_code"),
-        _ => None,
-    })
+    let provider = parse_canonical_provider(provider)?;
+    provider_model_keys(provider)
+        .iter()
+        .find_map(|key| string_field(models, key))
 }
 
 fn builtin_provider_model(provider: &str) -> Option<&'static str> {
-    match provider {
-        "claude" | "claude_code" => Some("claude-sonnet-4-6"),
-        "codex" => Some("gpt-5.5"),
-        _ => None,
-    }
+    parse_canonical_provider(provider).and_then(wire_builtin_provider_model)
 }
 
 fn non_empty_trimmed(text: &str) -> Option<String> {

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

@@ -31,8 +31,8 @@ pub fn should_orphan_self_terminate(
 /// #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 the
-/// provider-process-dead AND latest-explicit-error gate before leader notification.
+/// 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 {

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

@@ -64,7 +64,7 @@ fn detect_compaction(
             "coordinator.compaction_observed",
             json!({
                 "agent_id": fact.agent_id.as_str(),
-                "provider": provider.map(provider_name),
+                "provider": provider.map(runtime_event_provider_name),
                 "team": team,
                 "compaction_count": current,
                 "stuck_loop": false,
@@ -82,7 +82,7 @@ fn detect_compaction(
             "compaction_threshold_crossed.recommend_reset",
             json!({
                 "agent_id": fact.agent_id.as_str(),
-                "provider": provider.map(provider_name),
+                "provider": provider.map(runtime_event_provider_name),
                 "team": team,
                 "compaction_count": current,
                 "threshold": threshold,
@@ -239,7 +239,7 @@ fn detect_leader_api_error(
         json!({
             "leader_session_uuid": leader_session_uuid,
             "error_class": error_class,
-            "provider": provider.map(provider_name),
+            "provider": provider.map(runtime_event_provider_name),
             "partial_response_streamed": partial_response_streamed,
             "worker_dispatch_just_before": [],
             "retry_count": 0,
@@ -545,7 +545,8 @@ fn leader_receiver_provider(receiver: Option<&Value>) -> Option<Provider> {
     serde_json::from_value(Value::String(raw.to_string())).ok()
 }
 
-fn provider_name(provider: Provider) -> &'static str {
+// Event-field grammar, not provider identity parsing.
+fn runtime_event_provider_name(provider: Provider) -> &'static str {
     match provider {
         Provider::Claude => "claude",
         Provider::ClaudeCode => "claude_code",