@team-agent/installer 0.5.77 → 0.5.78

package/Cargo.lock

@@ -581,7 +581,7 @@ dependencies = [
 
 [[package]]
 name = "team-agent"
-version = "0.5.77"
+version = "0.5.78"
 dependencies = [
  "anyhow",
  "chrono",

package/Cargo.toml

@@ -9,7 +9,7 @@ members = ["crates/team-agent", "crates/win-conpty-phase0", "crates/conpty-trans
 
 [workspace.package]
 edition = "2021"
-version = "0.5.77"
+version = "0.5.78"
 license = "AGPL-3.0"
 rust-version = "1.95"
 

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

@@ -183,10 +183,18 @@ fn quickstart_human(value: &Value) -> String {
     append_reminder(out, crate::cli::QUICK_START_REMINDER)
 }
 
-fn append_send_guidance(value: &mut Value, workspace: &Path, team: Option<&str>) {
-    if value.get("ok").and_then(Value::as_bool) != Some(true) {
+pub(crate) fn append_send_guidance(value: &mut Value, workspace: &Path, team: Option<&str>) {
+    let existing = value.get("summary").and_then(Value::as_str) == Some("existing runtime");
+    if value.get("ok").and_then(Value::as_bool) != Some(true) && !existing {
         return;
     }
+    let team = value
+        .get("team")
+        .and_then(Value::as_str)
+        .filter(|team| !team.is_empty())
+        .map(str::to_string)
+        .or_else(|| team.map(str::to_string));
+    let team = team.as_deref();
     let agents = value
         .get("agent_ids")
         .and_then(Value::as_array)
@@ -204,11 +212,36 @@ fn append_send_guidance(value: &mut Value, workspace: &Path, team: Option<&str>)
     }
 }
 
+#[cfg(test)]
+pub(crate) fn split_shell_argv(command: &str) -> Vec<String> {
+    let mut argv = Vec::new();
+    let mut current = String::new();
+    let mut chars = command.chars().peekable();
+    let mut quote: Option<char> = None;
+    while let Some(ch) = chars.next() {
+        match (quote, ch) {
+            (None, '\'') | (None, '"') => quote = Some(ch),
+            (Some(q), c) if c == q => quote = None,
+            (None, c) if c.is_whitespace() => {
+                if !current.is_empty() {
+                    argv.push(std::mem::take(&mut current));
+                }
+            }
+            (_, c) => current.push(c),
+        }
+    }
+    if !current.is_empty() {
+        argv.push(current);
+    }
+    argv
+}
+
 pub(crate) fn send_command(agent: &str, workspace: &Path, team: Option<&str>) -> Option<String> {
     let workspace = workspace.to_str()?;
     let mut command = format!(
-        "team-agent send {} \\\"MESSAGE\\\" --workspace {}",
+        "team-agent send {} {} --workspace {}",
         shell_quote(agent),
+        shell_quote("MESSAGE"),
         shell_quote(workspace)
     );
     if let Some(team) = team {
@@ -528,6 +561,42 @@ pub fn cmd_allow_peer_talk(args: &AllowPeerTalkArgs) -> Result<CmdResult, CliErr
 
 /// `cmd_diagnose`(`parser.py:298`)。
 pub fn cmd_diagnose(args: &DiagnoseArgs) -> Result<CmdResult, CliError> {
+    let team_hint = args
+        .team
+        .as_deref()
+        .filter(|team| !team.is_empty())
+        .unwrap_or("current");
+    // Check the caller-supplied workspace before RuntimeOnly synthesis or
+    // transport setup, which can create `.team` paths and make an empty
+    // directory look like an unbound team.
+    if !crate::cli::diagnose::workspace_has_existing_team_runtime(&args.workspace, team_hint) {
+        let (issues, suggested_repairs) =
+            crate::cli::diagnose::missing_team_runtime_issues_and_repairs(team_hint);
+        let event_log = args
+            .workspace
+            .join(".team")
+            .join("logs")
+            .join("events.jsonl");
+        return Ok(CmdResult::from_json(
+            json!({
+                "event_log": event_log.to_string_lossy().to_string(),
+                "issues": issues,
+                "ok": false,
+                "providers": provider_doctor_checks(),
+                "runtime": {
+                    "workspace": args.workspace.to_string_lossy().to_string(),
+                    "team_key": team_hint,
+                    "session_name": Value::Null,
+                    "leader_receiver": Value::Null,
+                    "agent_count": 0,
+                    "message_count": 0,
+                    "result_count": 0,
+                },
+                "suggested_repairs": suggested_repairs,
+            }),
+            args.json,
+        ));
+    }
     let selected = crate::state::selector::resolve_active_team(
         &args.workspace,
         args.team.as_deref(),
@@ -556,7 +625,12 @@ pub fn cmd_diagnose(args: &DiagnoseArgs) -> Result<CmdResult, CliError> {
             )),
         };
     let (issues, suggested_repairs) =
-        diagnose_runtime_for_workspace(&selected.run_workspace, &state, backend.as_ref());
+        diagnose_runtime_for_workspace(
+            &selected.run_workspace,
+            &state,
+            backend.as_ref(),
+            Some(selected.team_key.as_str()),
+        );
     let ok = issues.as_array().is_some_and(Vec::is_empty);
     Ok(CmdResult::from_json(
         json!({
@@ -1347,17 +1421,24 @@ pub fn cmd_restart(args: &RestartArgs) -> Result<CmdResult, CliError> {
 
 /// `cmd_start_agent`(`commands.py:348`)。
 pub fn cmd_start_agent(args: &StartAgentArgs) -> Result<CmdResult, CliError> {
-    Ok(CmdResult::from_json(
-        lifecycle_port::start_agent(
-            &args.workspace,
-            &args.agent,
-            args.force,
-            !args.no_display,
-            args.allow_fresh,
-            args.team.as_deref(),
-        )?,
-        args.json,
-    ))
+    let mut value = lifecycle_port::start_agent(
+        &args.workspace,
+        &args.agent,
+        args.force,
+        !args.no_display,
+        args.allow_fresh,
+        args.team.as_deref(),
+    )?;
+    if value.get("agent_ids").is_none() {
+        if let Some(object) = value.as_object_mut() {
+            object.insert(
+                "agent_ids".to_string(),
+                json!([args.agent.as_str()]),
+            );
+        }
+    }
+    append_send_guidance(&mut value, &args.workspace, args.team.as_deref());
+    Ok(CmdResult::from_json(value, args.json))
 }
 
 /// `cmd_stop_agent`(`commands.py:359`)。
@@ -1384,17 +1465,24 @@ pub fn cmd_reset_agent(args: &ResetAgentArgs) -> Result<CmdResult, CliError> {
 
 /// `cmd_add_agent`(`commands.py:373`)。
 pub fn cmd_add_agent(args: &AddAgentArgs) -> Result<CmdResult, CliError> {
-    Ok(CmdResult::from_json(
-        lifecycle_port::add_agent(
-            &args.workspace,
-            &args.agent,
-            &args.role_file,
-            !args.no_display,
-            args.team.as_deref(),
-            args.force,
-        )?,
-        args.json,
-    ))
+    let mut value = lifecycle_port::add_agent(
+        &args.workspace,
+        &args.agent,
+        &args.role_file,
+        !args.no_display,
+        args.team.as_deref(),
+        args.force,
+    )?;
+    if value.get("agent_ids").is_none() {
+        if let Some(object) = value.as_object_mut() {
+            object.insert(
+                "agent_ids".to_string(),
+                json!([args.agent.as_str()]),
+            );
+        }
+    }
+    append_send_guidance(&mut value, &args.workspace, args.team.as_deref());
+    Ok(CmdResult::from_json(value, args.json))
 }
 
 /// `cmd_fork_agent`(`commands.py:383`)。
@@ -1540,8 +1628,10 @@ pub fn cmd_doctor(args: &DoctorArgs) -> Result<CmdResult, CliError> {
 mod tests {
     #![allow(clippy::unwrap_used)]
 
-    use super::{agent_pane_id, append_send_guidance, quickstart_human, send_command};
-    use serde_json::json;
+    use super::{
+        agent_pane_id, append_send_guidance, quickstart_human, send_command, split_shell_argv,
+    };
+    use serde_json::{json, Value};
     use std::path::{Path, PathBuf};
 
     // E13:happy 人类输出必须带 attach 块(此前 else 分支只打 summary 丢 attach_commands)。
@@ -1585,13 +1675,37 @@ mod tests {
         );
     }
 
+    #[test]
+    fn send_command_message_is_one_argv_token() {
+        let command = send_command(
+            "worker name; echo unsafe",
+            Path::new("/tmp/my workspace"),
+            Some("team-a"),
+        )
+        .unwrap();
+        let argv = split_shell_argv(&command);
+        assert_eq!(
+            argv,
+            [
+                "team-agent",
+                "send",
+                "worker name; echo unsafe",
+                "MESSAGE",
+                "--workspace",
+                "/tmp/my workspace",
+                "--team",
+                "team-a"
+            ]
+        );
+    }
+
     #[test]
     fn send_guidance_is_copyable_and_preserves_explicit_scope() {
         let command = send_command("worker name; echo unsafe", Path::new("/tmp/my workspace"), Some("team-a"))
             .unwrap();
         assert_eq!(
             command,
-            "team-agent send 'worker name; echo unsafe' \\\"MESSAGE\\\" --workspace '/tmp/my workspace' --team team-a"
+            "team-agent send 'worker name; echo unsafe' MESSAGE --workspace '/tmp/my workspace' --team team-a"
         );
     }
 
@@ -1625,6 +1739,24 @@ mod tests {
         assert!(value.get("send_commands").is_none());
     }
 
+    #[test]
+    fn existing_runtime_gets_canonical_send_guidance() {
+        let mut value = json!({
+            "ok": false,
+            "summary": "existing runtime",
+            "agent_ids": ["worker"]
+        });
+        append_send_guidance(&mut value, Path::new("/tmp/ws"), Some("team-a"));
+        let command = value
+            .pointer("/send_commands/0")
+            .and_then(Value::as_str)
+            .unwrap();
+        assert!(command.contains("send worker"));
+        assert!(command.contains("--team team-a"));
+        let parts: Vec<&str> = command.split_whitespace().collect();
+        assert_eq!(parts[3], "MESSAGE");
+    }
+
     #[test]
     fn agent_pane_id_resolves_session_window_even_with_recorded_pane_id() {
         let state = json!({