internal: add lock for writing target/packages.json (#412)

Cargo.toml

@@ -87,7 +87,7 @@ log = "0.4.20"
 env_logger = "0.10.0"
 thiserror = "1.0"
 test-log = "0.2"
-fs4 = { version = "0.8.3", features = ["sync"] }
+fs4 = { version = "0.10.0", features = ["sync"] }
 ariadne = { version = "0.4.1", features = ["auto-color"] }
 clap_complete = { version = "4.5.4" }
 schemars = "0.8"

crates/moonbuild/src/check/normal.rs

@@ -18,7 +18,7 @@
 
 use anyhow::Context;
 
-use moonutil::common::{MoonbuildOpt, MooncOpt};
+use moonutil::common::{FileLock, MoonbuildOpt, MooncOpt};
 use moonutil::module::{convert_mdb_to_json, ModuleDB, ModuleDBJSON};
 use n2::load::State;
 use std::io::BufWriter;
@@ -43,6 +43,8 @@ pub fn write_pkg_lst(module: &ModuleDB, target_dir: &Path) -> anyhow::Result<()>
 
     let mj = convert_mdb_to_json(module);
     let pkg_json = target_dir.join("packages.json");
+    // packages.json now placed in target/, should be protected for mutil-thread write
+    let _lock = FileLock::lock(target_dir)?;
 
     // if the file exist and the old content is the same as the new content in `module`, don't rewrite it
     // otherwise we create and write

crates/moonutil/src/common.rs

@@ -22,7 +22,7 @@ use crate::module::{MoonMod, MoonModJSON};
 use crate::package::{convert_pkg_json_to_package, MoonPkg, MoonPkgJSON, Package};
 use anyhow::{bail, Context};
 use clap::ValueEnum;
-use fs4::FileExt;
+use fs4::fs_std::FileExt;
 use indexmap::IndexMap;
 use serde::{Deserialize, Serialize};
 use std::collections::{HashMap, HashSet};