@team-agent/installer 0.5.41 → 0.5.43

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

@@ -207,7 +207,10 @@ mod tests {
             t("a", &["b"], TaskStatus::Pending),
             t("b", &["a"], TaskStatus::Pending),
         ];
-        assert_eq!(cycle_ids(&find_dependency_cycle(&tasks)), vec!["a", "b", "a"]);
+        assert_eq!(
+            cycle_ids(&find_dependency_cycle(&tasks)),
+            vec!["a", "b", "a"]
+        );
     }
 
     #[test]
@@ -239,7 +242,10 @@ mod tests {
             t("a", &["b"], TaskStatus::Pending),
             t("b", &["a"], TaskStatus::Pending),
         ];
-        assert_eq!(cycle_ids(&find_dependency_cycle(&tasks)), vec!["a", "b", "a"]);
+        assert_eq!(
+            cycle_ids(&find_dependency_cycle(&tasks)),
+            vec!["a", "b", "a"]
+        );
     }
 
     #[test]
@@ -351,7 +357,14 @@ mod tests {
         let mut node = t("a", &[], TaskStatus::Pending);
         node.last_result_summary = Some("old".to_string());
         let mut tasks = [node];
-        update_task_status(&mut tasks, &TaskId::new("a"), TaskStatus::Running, None, None).unwrap();
+        update_task_status(
+            &mut tasks,
+            &TaskId::new("a"),
+            TaskStatus::Running,
+            None,
+            None,
+        )
+        .unwrap();
         assert_eq!(tasks[0].status, TaskStatus::Running);
         assert_eq!(tasks[0].last_result_summary.as_deref(), Some("old"));
     }
@@ -360,9 +373,14 @@ mod tests {
     fn update_unknown_id_errors() {
         // Python U4:未知 id -> KeyError "Unknown task id: zzz"。Rust -> Runtime。
         let mut tasks = [t("a", &[], TaskStatus::Pending)];
-        let err =
-            update_task_status(&mut tasks, &TaskId::new("zzz"), TaskStatus::Done, None, None)
-                .unwrap_err();
+        let err = update_task_status(
+            &mut tasks,
+            &TaskId::new("zzz"),
+            TaskStatus::Done,
+            None,
+            None,
+        )
+        .unwrap_err();
         assert_eq!(err, ModelError::Runtime("Unknown task id: zzz".to_string()));
     }
 
@@ -371,10 +389,19 @@ mod tests {
         // Python U5:summary=None 只写 artifact_refs
         let mut tasks = [t("a", &[], TaskStatus::Pending)];
         let refs = vec![serde_json::json!({"r": 1})];
-        update_task_status(&mut tasks, &TaskId::new("a"), TaskStatus::Done, None, Some(refs))
-            .unwrap();
+        update_task_status(
+            &mut tasks,
+            &TaskId::new("a"),
+            TaskStatus::Done,
+            None,
+            Some(refs),
+        )
+        .unwrap();
         assert_eq!(tasks[0].status, TaskStatus::Done);
         assert!(tasks[0].last_result_summary.is_none());
-        assert_eq!(tasks[0].artifact_refs, Some(vec![serde_json::json!({"r": 1})]));
+        assert_eq!(
+            tasks[0].artifact_refs,
+            Some(vec![serde_json::json!({"r": 1})])
+        );
     }
 }

package/crates/team-agent/src/model/yaml/tests.rs

@@ -53,7 +53,10 @@ fn skill_front_matter_roundtrip() {
 
 #[test]
 fn block_scalar_basic() {
-    roundtrip_eq("msg: |\n  line one\n  line two\n", "msg: |\n  line one\n  line two\n");
+    roundtrip_eq(
+        "msg: |\n  line one\n  line two\n",
+        "msg: |\n  line one\n  line two\n",
+    );
 }
 
 #[test]
@@ -86,12 +89,18 @@ fn int_plus_and_underscore() {
 
 #[test]
 fn non_ints_stay_strings() {
-    roundtrip_eq("a: 1.5\nb: 0x1f\nc: 1e3\nd: _1\n", "a: \"1.5\"\nb: \"0x1f\"\nc: \"1e3\"\nd: \"_1\"\n");
+    roundtrip_eq(
+        "a: 1.5\nb: 0x1f\nc: 1e3\nd: _1\n",
+        "a: \"1.5\"\nb: \"0x1f\"\nc: \"1e3\"\nd: \"_1\"\n",
+    );
 }
 
 #[test]
 fn bools_and_nulls() {
-    roundtrip_eq("a: true\nb: False\nc: NULL\nd: ~\n", "a: true\nb: false\nc: null\nd: null\n");
+    roundtrip_eq(
+        "a: true\nb: False\nc: NULL\nd: ~\n",
+        "a: true\nb: false\nc: null\nd: null\n",
+    );
 }
 
 #[test]
@@ -119,7 +128,10 @@ fn list_of_maps() {
 
 #[test]
 fn scalar_list_mixed() {
-    roundtrip_eq("a:\n  - 1\n  - two\n  - true\n", "a:\n  - 1\n  - \"two\"\n  - true\n");
+    roundtrip_eq(
+        "a:\n  - 1\n  - two\n  - true\n",
+        "a:\n  - 1\n  - \"two\"\n  - true\n",
+    );
 }
 
 #[test]
@@ -130,7 +142,10 @@ fn empty_value_is_null() {
 #[test]
 fn inline_comment_not_stripped() {
     // 整行注释被跳过;行内 `#` 不被当注释,留在值里。
-    roundtrip_eq("# top\na: 1  # inline?\nb: 2\n", "a: \"1  # inline?\"\nb: 2\n");
+    roundtrip_eq(
+        "# top\na: 1  # inline?\nb: 2\n",
+        "a: \"1  # inline?\"\nb: 2\n",
+    );
 }
 
 #[test]
@@ -213,7 +228,10 @@ fn dump_mixed_list() {
         Value::Map(Vec::new()),
         Value::List(Vec::new()),
     ]);
-    assert_eq!(dumps(&v), "- 1\n- \"two\"\n-\n  - 3\n  - 4\n- k: \"v\"\n- {}\n-\n");
+    assert_eq!(
+        dumps(&v),
+        "- 1\n- \"two\"\n-\n  - 3\n  - 4\n- k: \"v\"\n- {}\n-\n"
+    );
 }
 
 #[test]

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

@@ -86,7 +86,10 @@ impl Value {
     }
     /// dict get:首个匹配 key 的值(`insert_ordered` 已去重 → 首即唯一)。非 Map → `None`。
     pub fn get(&self, key: &str) -> Option<&Value> {
-        self.as_map()?.iter().find(|(k, _)| k == key).map(|(_, v)| v)
+        self.as_map()?
+            .iter()
+            .find(|(k, _)| k == key)
+            .map(|(_, v)| v)
     }
     /// Python 真值语义:None/Null/false/0/""/空集 → false。
     pub fn is_truthy(&self) -> bool {
@@ -330,7 +333,8 @@ fn parse_scalar(raw: &str) -> Value {
     if raw == "{}" {
         return Value::Map(Vec::new());
     }
-    if (raw.starts_with('"') && raw.ends_with('"')) || (raw.starts_with('\'') && raw.ends_with('\''))
+    if (raw.starts_with('"') && raw.ends_with('"'))
+        || (raw.starts_with('\'') && raw.ends_with('\''))
     {
         if raw.len() < 2 {
             // 单个引号字符:不是合法的成对引号,落到末尾 return raw。
@@ -576,10 +580,7 @@ fn dump(value: &Value, indent: usize) -> Vec<String> {
                                 lines.extend(dump(child, indent + 4));
                             } else {
                                 // 注意:此分支**不**对多行字符串做 `|`,与 Python 一致。
-                                lines.push(format!(
-                                    "{pad}{prefix}{key}: {}",
-                                    format_scalar(child)
-                                ));
+                                lines.push(format!("{pad}{prefix}{key}: {}", format_scalar(child)));
                             }
                             first = false;
                         }

package/crates/team-agent/src/packaging/install.rs

@@ -105,7 +105,9 @@ pub fn uninstall(opts: &UninstallOptions) -> Result<UninstallOutcome, PackagingE
 /// `--target all` fan-out 两者;`--dest` 不能与 `--target all` 组合(`commands.py:453` → Err)。
 /// 拷前清陈旧残留(修 `dirs_exist_ok` 残留);`--dry-run` 只报告不落地。
 /// // REAL-MACHINE-E2E:真拷 / removed_stale 需文件系统;dry-run 与 plan 可单测。
-pub fn install_skill(opts: &SkillInstallOptions) -> Result<Vec<SkillInstallOutcome>, PackagingError> {
+pub fn install_skill(
+    opts: &SkillInstallOptions,
+) -> Result<Vec<SkillInstallOutcome>, PackagingError> {
     if opts.target == SkillTarget::All && opts.dest.is_some() {
         return Err(PackagingError::InvalidOptions(
             "--dest cannot be combined with --target all".to_string(),
@@ -120,9 +122,9 @@ pub fn install_skill(opts: &SkillInstallOptions) -> Result<Vec<SkillInstallOutco
     for target in targets {
         let dest = match &opts.dest {
             Some(dest) => SkillDestDir(dest.clone()),
-            None => target
-                .dest_dir(&home)
-                .ok_or_else(|| PackagingError::InvalidOptions("target all has no single dest".to_string()))?,
+            None => target.dest_dir(&home).ok_or_else(|| {
+                PackagingError::InvalidOptions("target all has no single dest".to_string())
+            })?,
         };
         let mut removed_stale = Vec::new();
         if !opts.dry_run {
@@ -169,7 +171,9 @@ pub fn diagnose_path(bin_dir: &BinDir) -> Result<PathHint, PackagingError> {
         .map(PathBuf::from)
         .collect();
     if entries.iter().any(|p| p == &bin_dir.0) {
-        return Ok(PathHint::OnPath { bin_dir: bin_dir.0.clone() });
+        return Ok(PathHint::OnPath {
+            bin_dir: bin_dir.0.clone(),
+        });
     }
     let executable_bit_set = bin_dir.0.join("team-agent").metadata().is_ok_and(|m| {
         #[cfg(unix)]
@@ -197,14 +201,21 @@ pub fn diagnose_path(bin_dir: &BinDir) -> Result<PathHint, PackagingError> {
 }
 
 fn home_dir() -> PathBuf {
-    std::env::var_os("HOME").map(PathBuf::from).unwrap_or_else(|| PathBuf::from("."))
+    std::env::var_os("HOME")
+        .map(PathBuf::from)
+        .unwrap_or_else(|| PathBuf::from("."))
 }
 
 fn default_skill_source() -> PathBuf {
-    PathBuf::from(env!("CARGO_MANIFEST_DIR")).join("skills").join("team-agent")
+    PathBuf::from(env!("CARGO_MANIFEST_DIR"))
+        .join("skills")
+        .join("team-agent")
 }
 
-fn atomic_replace_binary(source: &Path, dest: &Path) -> Result<AtomicReplaceOutcome, PackagingError> {
+fn atomic_replace_binary(
+    source: &Path,
+    dest: &Path,
+) -> Result<AtomicReplaceOutcome, PackagingError> {
     if !source.exists() {
         return Err(PackagingError::Io(std::io::Error::new(
             std::io::ErrorKind::NotFound,
@@ -293,7 +304,10 @@ fn staging_dir_for(dest: &Path) -> Result<PathBuf, PackagingError> {
         .file_name()
         .and_then(|name| name.to_str())
         .unwrap_or("skill");
-    let parent = dest.parent().map(Path::to_path_buf).unwrap_or_else(|| PathBuf::from("."));
+    let parent = dest
+        .parent()
+        .map(Path::to_path_buf)
+        .unwrap_or_else(|| PathBuf::from("."));
     std::fs::create_dir_all(&parent)?;
     Ok(parent.join(format!(".{name}.ta-staging-{}", std::process::id())))
 }

package/crates/team-agent/src/packaging/migrate.rs

@@ -11,14 +11,12 @@ use crate::db::migration::schema_diagnosis_workspace;
 /// **§84**:只调 step 3/11/12 的 trait 入口,注入 mock 时 provider 调用计数 = 0;绝不触发 prompt/token。
 pub fn doctor(opts: &DoctorOptions) -> Result<DoctorStatus, PackagingError> {
     if opts.fix && opts.gate.is_none() {
-        return Err(PackagingError::InvalidOptions("--fix requires --gate".to_string()));
+        return Err(PackagingError::InvalidOptions(
+            "--fix requires --gate".to_string(),
+        ));
     }
-    let gate_blockers = crate::diagnose::doctor_gate_blockers(
-        &opts.workspace,
-        opts.gate,
-        opts.fix,
-        opts.confirm,
-    )?;
+    let gate_blockers =
+        crate::diagnose::doctor_gate_blockers(&opts.workspace, opts.gate, opts.fix, opts.confirm)?;
     if !gate_blockers.is_empty() {
         return Ok(DoctorStatus::HasBlockers {
             blockers: gate_blockers,

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

@@ -53,7 +53,15 @@
 //! 统一加(原子替换/migration·repair 段建议局部等同 daemon 门),本骨架不加。
 
 // ROUND-0 skeleton:fn body 全 unimplemented!() → import/field/method 暂未被用;P2 porter 落实现时移除。
-#![allow(dead_code, unused_imports, unused_variables, clippy::result_large_err, clippy::doc_overindented_list_items, clippy::doc_lazy_continuation, clippy::io_other_error)]
+#![allow(
+    dead_code,
+    unused_imports,
+    unused_variables,
+    clippy::result_large_err,
+    clippy::doc_overindented_list_items,
+    clippy::doc_lazy_continuation,
+    clippy::io_other_error
+)]
 // §10:原子替换二进制 / migration·repair 路径实现层禁 unwrap/expect/panic(unimplemented!() stub 不被拦);
 // tests 子模块各自 allow。
 #![deny(clippy::unwrap_used, clippy::expect_used, clippy::panic)]

package/crates/team-agent/src/packaging/repair.rs

@@ -14,11 +14,18 @@ pub fn repair_schema(workspace: &Path) -> Result<MigrationOutcome, PackagingErro
     match fix_schema_layout(workspace, SCHEMA_VERSION)? {
         FixResult::Missing(diagnosis) => Ok(MigrationOutcome::UpToDate { diagnosis }),
         FixResult::Blocked { reason } => Ok(MigrationOutcome::Blocked { reason }),
-        FixResult::Fixed { diagnosis, rebuilds } if rebuilds.is_empty() => {
-            Ok(MigrationOutcome::UpToDate { diagnosis })
-        }
-        FixResult::Fixed { diagnosis, rebuilds } => {
-            Ok(MigrationOutcome::Migrated { fix: FixResult::Fixed { diagnosis, rebuilds } })
-        }
+        FixResult::Fixed {
+            diagnosis,
+            rebuilds,
+        } if rebuilds.is_empty() => Ok(MigrationOutcome::UpToDate { diagnosis }),
+        FixResult::Fixed {
+            diagnosis,
+            rebuilds,
+        } => Ok(MigrationOutcome::Migrated {
+            fix: FixResult::Fixed {
+                diagnosis,
+                rebuilds,
+            },
+        }),
     }
 }

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

@@ -130,9 +130,21 @@ fn version_current_is_not_a_hand_copied_python_drift_literal() {
         env!("CARGO_PKG_VERSION"),
         "single source of truth = CARGO_PKG_VERSION"
     );
-    assert_ne!(v.as_str(), "0.1.4", "must not hand-copy pyproject.toml drift source");
-    assert_ne!(v.as_str(), "0.2.11", "must not hand-copy package.json drift source");
-    assert_ne!(v.as_str(), "dev", "must not be install.mjs:54 'dev' fallback");
+    assert_ne!(
+        v.as_str(),
+        "0.1.4",
+        "must not hand-copy pyproject.toml drift source"
+    );
+    assert_ne!(
+        v.as_str(),
+        "0.2.11",
+        "must not hand-copy package.json drift source"
+    );
+    assert_ne!(
+        v.as_str(),
+        "dev",
+        "must not be install.mjs:54 'dev' fallback"
+    );
 }
 
 #[test]
@@ -143,8 +155,11 @@ fn no_literal_version_string_hardcoded_in_packaging_code() {
     // 0.1.4 / 0.2.11 legitimately appear in doc/line comments documenting the bug, so we strip
     // comment text first and scan only executable code. This is the one place where
     // "double-source-drift forbidden" is statically checked against the source itself.
-    let src = std::fs::read_to_string(concat!(env!("CARGO_MANIFEST_DIR"), "/src/packaging/types.rs"))
-        .expect("read own source");
+    let src = std::fs::read_to_string(concat!(
+        env!("CARGO_MANIFEST_DIR"),
+        "/src/packaging/types.rs"
+    ))
+    .expect("read own source");
     // Production region only (the #[cfg(test)] mod uses these literals as golden anti-examples):
     let prod = match src.find("#[cfg(test)]") {
         Some(i) => &src[..i],
@@ -375,7 +390,8 @@ fn doctor_drifted_db_emits_schema_layout_drift_blocker() {
     //   "team.db physical layout drift detected"  (EXACT string, commands.py:248).
     let ws = seed_workspace_with_drifted_db("blocker");
     let opts = doctor_opts(&ws);
-    let status = doctor(&opts).expect("doctor on drifted workspace should succeed (returns blockers)");
+    let status =
+        doctor(&opts).expect("doctor on drifted workspace should succeed (returns blockers)");
     match status {
         DoctorStatus::HasBlockers { blockers } => {
             let drift = blockers
@@ -410,7 +426,10 @@ fn doctor_status_has_blockers_carries_typed_source() {
     };
     let json = serde_json::to_string(&status).unwrap();
     assert!(json.contains("\"status\":\"has_blockers\""), "got: {json}");
-    assert!(json.contains("\"source\":\"schema_layout_drift\""), "got: {json}");
+    assert!(
+        json.contains("\"source\":\"schema_layout_drift\""),
+        "got: {json}"
+    );
     // detail 精确 == commands.py:248 schema_error 文本.
     assert!(
         json.contains("team.db physical layout drift detected"),
@@ -508,7 +527,10 @@ fn diagnose_path_not_on_path_npmrc_prefix_is_none_no_npm() {
     let bin = BinDir(PathBuf::from("/zzz-definitely-not-on-path-9f3a"));
     let hint = diagnose_path(&bin).expect("diagnose off-path bin");
     match hint {
-        PathHint::NotOnPath { bin_dir, diagnostic } => {
+        PathHint::NotOnPath {
+            bin_dir,
+            diagnostic,
+        } => {
             assert_eq!(bin_dir, PathBuf::from("/zzz-definitely-not-on-path-9f3a"));
             // Rust 无 npm → 绝不重新引入 .npmrc 解析.
             assert_eq!(diagnostic.npmrc_prefix, None);
@@ -556,11 +578,7 @@ fn skill_opts(target: SkillTarget, dest: Option<PathBuf>, dry_run: bool) -> Skil
 #[test]
 fn install_skill_dest_with_target_all_is_invalid() {
     // commands.py:453-454 — `--dest cannot be combined with --target all`.
-    let opts = skill_opts(
-        SkillTarget::All,
-        Some(PathBuf::from("/custom/dest")),
-        false,
-    );
+    let opts = skill_opts(SkillTarget::All, Some(PathBuf::from("/custom/dest")), false);
     let err = install_skill(&opts).expect_err("dest + all must error");
     match err {
         PackagingError::InvalidOptions(msg) => assert!(
@@ -620,7 +638,11 @@ fn install_skill_dry_run_explicit_dest_single_target() {
 #[ignore = "REAL-MACHINE-E2E: real copytree + stale diff removal (fixes dirs_exist_ok=True residue)"]
 fn install_skill_real_copy_removes_stale_files() {
     // 修 commands.py:480 dirs_exist_ok 残留:Rust 拷前清旧 SKILL,记录 removed_stale.
-    let opts = skill_opts(SkillTarget::Codex, Some(PathBuf::from("/tmp/ta-skill-real")), false);
+    let opts = skill_opts(
+        SkillTarget::Codex,
+        Some(PathBuf::from("/tmp/ta-skill-real")),
+        false,
+    );
     let outcomes = install_skill(&opts).expect("real install-skill");
     assert!(!outcomes[0].dry_run);
     // 真路径下若有旧残留,removed_stale 非空 (具体值依 fixture).
@@ -815,12 +837,33 @@ fn atomic_replace_outcome_serde_tag_outcome() {
 // ───────────────────────────────────────────────────────────────────────
 
 #[test]
+#[serial_test::serial(env)]
 fn install_skill_dry_run_is_pure_no_provider_state() {
     // §84:install-skill 只拷文件;dry-run 连文件都不动 → 纯函数式可重复.
+    //
+    // 0.5.43 debt-sweep (§6.2): even dry-run reads ambient HOME to
+    // build the target skill path. Parallel real-copy sibling tests
+    // hold the same `ENV_LOCK_PKG` + `HomeGuard::set` guards; without
+    // matching guards here, two dry-run runs can observe HOME after
+    // a real-copy test swapped it, producing spurious diffs. Same
+    // env critical section as install/uninstall.
+    let _g = ENV_LOCK_PKG.lock().unwrap_or_else(|p| p.into_inner());
+    let home = std::env::temp_dir().join(format!(
+        "ta-0543-pkg-dry-{}-{}",
+        std::process::id(),
+        line!()
+    ));
+    let _ = std::fs::remove_dir_all(&home);
+    std::fs::create_dir_all(&home).expect("create dry-run HOME");
+    let _h = HomeGuard::set(&home);
     let opts = skill_opts(SkillTarget::Claude, None, true);
     let first = install_skill(&opts).expect("dry-run 1");
     let second = install_skill(&opts).expect("dry-run 2");
-    assert_eq!(first, second, "dry-run install-skill must be deterministic & side-effect free");
+    assert_eq!(
+        first, second,
+        "dry-run install-skill must be deterministic & side-effect free"
+    );
+    let _ = std::fs::remove_dir_all(&home);
 }
 
 // ───────────────────────────────────────────────────────────────────────
@@ -892,7 +935,11 @@ fn install_skill_all_real_copies_to_three_provider_locations() {
     assert_eq!(outcomes.len(), 3, "all → codex+claude+copilot");
 
     for sub in [".codex", ".claude", ".copilot"] {
-        let dest = home.join(sub).join("skills").join("team-agent").join("SKILL.md");
+        let dest = home
+            .join(sub)
+            .join("skills")
+            .join("team-agent")
+            .join("SKILL.md");
         assert!(dest.exists(), "{sub}: SKILL.md must exist after install");
         assert_eq!(
             std::fs::read(&dest).unwrap(),

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

@@ -51,8 +51,11 @@ pub enum SkillTarget {
 impl SkillTarget {
     /// `All` fan-out 的单目标全集(表驱动唯一真相源:新增 provider 只改这里,
     /// install/uninstall/install-skill 三处共用,杜绝漏装/漏卸)。
-    pub const SINGLE_TARGETS: [SkillTarget; 3] =
-        [SkillTarget::Codex, SkillTarget::Claude, SkillTarget::Copilot];
+    pub const SINGLE_TARGETS: [SkillTarget; 3] = [
+        SkillTarget::Codex,
+        SkillTarget::Claude,
+        SkillTarget::Copilot,
+    ];
 
     /// 单目标 → 对应 provider(`All` 无单一 provider → `None`)。与 [`Provider`] 对齐,防散字符串再生。
     pub fn provider(self) -> Option<Provider> {
@@ -67,9 +70,15 @@ impl SkillTarget {
     /// `_skill_dest_dir`:`~/.codex|.claude|.copilot/skills/team-agent`(`All` fan-out 全集,非单 dir → None)。
     pub fn dest_dir(self, home: &Path) -> Option<SkillDestDir> {
         match self {
-            Self::Codex => Some(SkillDestDir(home.join(".codex").join("skills").join("team-agent"))),
-            Self::Claude => Some(SkillDestDir(home.join(".claude").join("skills").join("team-agent"))),
-            Self::Copilot => Some(SkillDestDir(home.join(".copilot").join("skills").join("team-agent"))),
+            Self::Codex => Some(SkillDestDir(
+                home.join(".codex").join("skills").join("team-agent"),
+            )),
+            Self::Claude => Some(SkillDestDir(
+                home.join(".claude").join("skills").join("team-agent"),
+            )),
+            Self::Copilot => Some(SkillDestDir(
+                home.join(".copilot").join("skills").join("team-agent"),
+            )),
             Self::All => None,
         }
     }
@@ -240,7 +249,10 @@ pub enum PathHint {
     /// bin 已在 PATH。
     OnPath { bin_dir: PathBuf },
     /// bin 不在 PATH —— 携带诊断(保留 WSL/`.npmrc` 等价提示)。// REAL-MACHINE-E2E: 真探 PATH。
-    NotOnPath { bin_dir: PathBuf, diagnostic: PathDiagnostic },
+    NotOnPath {
+        bin_dir: PathBuf,
+        diagnostic: PathDiagnostic,
+    },
 }
 
 /// 「bin 不在 PATH」诊断(`bincheck.mjs:39-65` 自由 console.error → typed struct)。
@@ -280,7 +292,10 @@ pub enum AtomicReplaceOutcome {
     /// 跨卷 `EXDEV` → copy+fsync+rename fallback 成功。// REAL-MACHINE-E2E: 需跨卷真机。
     ReplacedCrossDevice { backup: PathBuf },
     /// 替换失败 → 已回滚到 `.previous`(原 dest 仍可用)。// REAL-MACHINE-E2E.
-    RolledBack { restored_from: PathBuf, error: String },
+    RolledBack {
+        restored_from: PathBuf,
+        error: String,
+    },
 }
 
 /// `uninstall` 结果(`install.mjs:109-130`)。默认保留 runtime/workspace(有 team 在跑勿 purge)。

package/crates/team-agent/src/platform/argv.rs

@@ -299,7 +299,10 @@ mod tests {
         // (windows) — both must resolve our own ppid.
         let my = std::process::id();
         let ppid = parent_pid(my);
-        assert!(ppid.is_some(), "parent_pid must resolve own pid on both unix and windows");
+        assert!(
+            ppid.is_some(),
+            "parent_pid must resolve own pid on both unix and windows"
+        );
         assert!(ppid.unwrap() > 0);
     }
 

package/crates/team-agent/src/platform/file_lock.rs

@@ -263,10 +263,7 @@ pub enum LockError {
 /// Existing product callers do NOT use this — they need their own
 /// metadata/waiter/held_long event machinery. This wrapper exists for
 /// simple future callers.
-pub fn try_lock_exclusive(
-    path: &Path,
-    timeout: Duration,
-) -> Result<FileLockGuard, LockError> {
+pub fn try_lock_exclusive(path: &Path, timeout: Duration) -> Result<FileLockGuard, LockError> {
     let file = File::options()
         .read(true)
         .write(true)
@@ -399,20 +396,37 @@ mod tests {
     #[test]
     fn timeout_error_carries_path_and_seconds() {
         // Convenience wrapper's timeout error shape.
-        let dir = std::env::temp_dir().join("ta-b2-timeout");
+        //
+        // 0.5.43 debt-sweep (§6.2): the pre-0.5.43 fixed
+        // `ta-b2-timeout/timeout-lock.tmp` path let parallel test
+        // workers race for the same file across cargo threads. Each
+        // run now allocates a per-process + monotonic-atomic dir so
+        // `--test-threads=2` cannot false-fail. Timeout shape and
+        // `timeout-lock.tmp` basename are preserved so downstream
+        // guards keep firing.
+        static N: std::sync::atomic::AtomicU64 = std::sync::atomic::AtomicU64::new(0);
+        let dir = std::env::temp_dir().join(format!(
+            "ta-b2-timeout-{}-{}",
+            std::process::id(),
+            N.fetch_add(1, std::sync::atomic::Ordering::Relaxed)
+        ));
         std::fs::create_dir_all(&dir).unwrap();
         let path = dir.join("timeout-lock.tmp");
-        let _hold = try_lock_exclusive(&path, Duration::from_secs(1))
-            .expect("first acquire must succeed");
+        let _hold =
+            try_lock_exclusive(&path, Duration::from_secs(1)).expect("first acquire must succeed");
         let err = try_lock_exclusive(&path, Duration::from_millis(150))
             .expect_err("second acquire must timeout");
         match err {
-            LockError::Timeout { timeout_secs, path: p } => {
+            LockError::Timeout {
+                timeout_secs,
+                path: p,
+            } => {
                 assert!(timeout_secs > 0.0);
                 assert!(p.ends_with("timeout-lock.tmp"), "path suffix: {p}");
             }
             other => panic!("expected Timeout, got {other:?}"),
         }
         let _ = std::fs::remove_file(&path);
+        let _ = std::fs::remove_dir_all(&dir);
     }
 }