@team-agent/installer 0.4.11 → 0.5.1

package/Cargo.lock

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

package/Cargo.toml

@@ -9,7 +9,7 @@ members = ["crates/team-agent"]
 
 [workspace.package]
 edition = "2021"
-version = "0.4.11"
+version = "0.5.1"
 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!(
@@ -218,7 +225,7 @@ fn command_help(command: Option<&str>) -> String {
         Some("quick-start") => "usage: team-agent quick-start [TEAMDIR] [--workspace WORKSPACE] [--name NAME] [--team-id TEAM|--team TEAM] [--yes] [--no-display] [--json]\n\ndefaults: display_backend=adaptive; set display_backend: none in TEAM.md or pass --no-display to use one worker window per agent.".to_string(),
         Some("start") => "usage: team-agent start [TEAMDIR] [--yes] [--fresh] [--json]".to_string(),
         Some("compile") => "usage: team-agent compile --team TEAM [--out FILE] [--json]".to_string(),
-        Some("send") => "usage: team-agent send TARGET MESSAGE... [--workspace WORKSPACE] [--team TEAM] [--targets AGENTS] [--task TASK] [--sender SENDER] [--watch-result] [--requires-ack|--no-ack] [--no-wait] [--timeout SECONDS] [--confirm-human] [--message-id ID] [--json]".to_string(),
+        Some("send") => "usage: team-agent send TARGET MESSAGE... [--workspace WORKSPACE] [--team TEAM] [--targets AGENTS] [--to-name NAME] [--pane PANE] [--task TASK] [--sender SENDER] [--watch-result] [--requires-ack|--no-ack] [--no-wait] [--timeout SECONDS] [--confirm-human] [--message-id ID] [--json]\n\nMVP: name-based cross-workspace addressing assumes trusted local caller; no auth gate.".to_string(),
         Some("fallback-send-leader") => "usage: team-agent fallback-send-leader --workspace WORKSPACE --team TEAM --sender AGENT --message-id ID --content TEXT --primary-error ERROR [--task TASK] [--json]\n\nEmergency one-shot fallback only after a leader-bound MCP send transport failure; restart-agent after use.".to_string(),
         Some("fallback-report-result") => "usage: team-agent fallback-report-result --workspace WORKSPACE --team TEAM --agent-id AGENT --task-id TASK --result-json JSON --primary-error ERROR [--json]\n\nEmergency one-shot fallback only after a report_result MCP transport failure; persists the result DB row, then restart-agent after use.".to_string(),
         Some("allow-peer-talk") => "usage: team-agent allow-peer-talk A B [--workspace WORKSPACE] [--json]".to_string(),
@@ -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' 误配任何东西。
@@ -661,6 +668,7 @@ struct ParsedArgs {
     out: Option<PathBuf>,
     auth_mode: Option<String>,
     pane: Option<String>,
+    to_name: Option<String>,
     provider: Option<String>,
     message_id: Option<String>,
     content: Option<String>,
@@ -743,6 +751,7 @@ fn parse_args(args: &[String]) -> ParsedArgs {
             "--out" => parsed.out = next_arg(args, &mut i).map(PathBuf::from),
             "--auth-mode" => parsed.auth_mode = next_arg(args, &mut i),
             "--pane" => parsed.pane = next_arg(args, &mut i),
+            "--to-name" => parsed.to_name = next_arg(args, &mut i),
             "--provider" => parsed.provider = next_arg(args, &mut i),
             "--message-id" => parsed.message_id = next_arg(args, &mut i),
             "--content" => parsed.content = next_arg(args, &mut i),
@@ -755,6 +764,9 @@ fn parse_args(args: &[String]) -> ParsedArgs {
             other if other.starts_with("--pane=") => {
                 parsed.pane = Some(other.trim_start_matches("--pane=").to_string());
             }
+            other if other.starts_with("--to-name=") => {
+                parsed.to_name = Some(other.trim_start_matches("--to-name=").to_string());
+            }
             other if other.starts_with("--provider=") => {
                 parsed.provider = Some(other.trim_start_matches("--provider=").to_string());
             }
@@ -868,7 +880,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() || parsed.to_name.is_some() {
         None
     } else {
         parsed.positionals.first().cloned()
@@ -896,6 +908,7 @@ fn send_args(args: &[String], cwd: &Path) -> Result<SendArgs, CliError> {
         json: parsed.json,
         message_id: parsed.message_id,
         pane: parsed.pane.clone(),
+        to_name: parsed.to_name.clone(),
     })
 }
 
@@ -1595,6 +1608,8 @@ mod tests {
                     "--workspace",
                     "--team",
                     "--targets",
+                    "--to-name",
+                    "--pane",
                     "--watch-result",
                     "--timeout",
                     "--json",
@@ -1747,6 +1762,84 @@ 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_to_name_positionals_are_message_not_target() {
+        let cwd = tmp_workspace();
+        let args = send_args(&cli_argv(&["--to-name", "team-a/qa", "hello"]), &cwd).unwrap();
+        assert_eq!(args.to_name.as_deref(), Some("team-a/qa"));
+        assert_eq!(args.target, None);
+        assert_eq!(args.targets, None);
+        assert_eq!(args.message, vec!["hello".to_string()]);
+
+        let args = send_args(
+            &cli_argv(&["--to-name=team-a/qa", "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/cli/mod.rs

@@ -57,6 +57,7 @@ pub mod diagnose;
 pub mod emit;
 pub mod helpers;
 pub mod leader;
+pub mod named_address;
 pub mod profile;
 pub mod send;
 pub mod status;
@@ -66,6 +67,7 @@ pub use adapters::*;
 pub use diagnose::*;
 pub use emit::*;
 pub use leader::*;
+pub use named_address::*;
 pub use profile::*;
 pub use send::*;
 pub use status::*;