fix(build): OUT_DIR for compile/publish, tests/generated mirror for local review

cargo publish verify forbids build.rs from writing outside OUT_DIR. build.rs always writes OUT_DIR/html_tests.rs (compiled via include! in tests/mod.rs); local (non-CI, non-target/package) builds additionally mirror per-case .rs under tests/generated/ for human review. CI (CI=true) and cargo package/publish verify (target/package) skip the mirror, so publish verification no longer sees source-tree modification.

Author: TtTRz

float-pigment-forest/build.rs

@@ -1,9 +1,17 @@
 // HTML -> imperative Rust translator: each tests/cases/<topic>/<case>.html
 // is translated into a `#[test]` that exercises the TestCtx high-level API
 // (create_node / create_text / set_style / append + layout_imperative +
-// getters). Output goes to tests/generated/<topic>/<case>.rs plus a nested
-// mod tree (tests/generated/mod.rs → <topic>/mod.rs → <case>.rs), which
-// tests/mod.rs pulls in via `mod generated;`.
+// getters).
+//
+// Output strategy:
+//   - Always: OUT_DIR/html_tests.rs (single file, all #[test] inlined),
+//     pulled in by tests/mod.rs via `include!`. This is what compiles.
+//   - Local (non-CI, non-publish-verify) builds ALSO mirror per-case .rs
+//     files under tests/generated/<topic>/<case>.rs (+ mod tree) for human
+//     review. CI (CI=true) and `cargo package`/`publish` verify
+//     (CARGO_MANIFEST_DIR under target/package) skip the mirror — CI doesn't
+//     need it, and publish verify forbids build.rs from writing outside
+//     OUT_DIR.
 //
 // The translator walks the parsed DOM and emits imperative calls in
 // document order. Layout assertions (data-expect-*) are collected during
@@ -19,13 +27,13 @@ use float_pigment_mlp::{
 fn main() {
     let manifest = std::env::var("CARGO_MANIFEST_DIR").unwrap();
     let cases_dir = PathBuf::from(&manifest).join("tests/cases");
-    let gen_dir = PathBuf::from(&manifest).join("tests/generated");
-    // 清空旧生成（删整个 generated 再重建，避免残留）
-    let _ = fs::remove_dir_all(&gen_dir);
-    fs::create_dir_all(&gen_dir).unwrap();
+    let out = std::env::var("OUT_DIR").unwrap();
+    let output_file = PathBuf::from(&out).join("html_tests.rs");
 
-    // topic -> Vec<(case_name, body, ignore)>
-    let mut by_topic: std::collections::BTreeMap<String, Vec<(String, String, bool)>> =
+    let mut tests = String::from("// AUTO-GENERATED by build.rs. Do not edit.\n\n");
+    // topic -> Vec<(case_name, name_ident, body, ignore)>, used to mirror
+    // per-case .rs files under tests/generated in local builds.
+    let mut by_topic: std::collections::BTreeMap<String, Vec<(String, String, String, bool)>> =
         std::collections::BTreeMap::new();
 
     if cases_dir.exists() {
@@ -39,28 +47,54 @@ fn main() {
             let mut parts = rel_str.split('/');
             let topic = parts.next().unwrap_or("misc").to_string();
             let name = parts.collect::<Vec<_>>().join("_");
+            let topic_ident = topic.replace('-', "_");
+            let name_ident = name.replace('-', "_");
+            let fn_name = format!("html_{}_{}", topic_ident, name_ident);
             let html = fs::read_to_string(&abs).unwrap_or_default();
             let ignore = html.contains("data-ignore=\"true\"");
+            let ignore_attr = if ignore { "#[ignore]\n" } else { "" };
             let body = translate_html(&html);
+            // OUT_DIR/html_tests.rs — compiled via include! in tests/mod.rs.
+            tests.push_str(&format!(
+                "{ignore_attr}#[test]\nfn {fn_name}() {{\n    use crate::TestCtx;\n{body}}}\n\n"
+            ));
             by_topic
                 .entry(topic)
                 .or_default()
-                .push((name, body, ignore));
+                .push((name, name_ident, body, ignore));
         }
     }
 
-    // 生成每 topic 目录 + mod.rs + 各 case .rs。
-    // topic/name 原样用作目录名/文件名（文件系统允许 `-`），但需 sanitize
-    // 为合法 Rust 标识符（`-` 非法）用于 mod 名与 fn 名。当前 cases 无 `-`，
-    // 此 sanitize 为防御性 —— 与旧 build.rs `replace(['/', '\\', '-', ' '], "_")` 一致。
+    fs::write(&output_file, tests).unwrap();
+
+    // Local builds also mirror per-case .rs under tests/generated/ for review.
+    let is_ci = std::env::var("CI").is_ok();
+    let is_publish_verify = manifest.contains("target/package");
+    if !is_ci && !is_publish_verify {
+        mirror_to_generated(&manifest, &by_topic);
+    }
+
+    println!("cargo:rerun-if-changed=tests/cases");
+    println!("cargo:rerun-if-changed=build.rs");
+}
+
+/// Mirror per-case .rs files (+ nested mod tree) under tests/generated/ for
+/// human review. Not compiled (tests/mod.rs uses the OUT_DIR include!), so
+/// purely for readability.
+fn mirror_to_generated(
+    manifest: &str,
+    by_topic: &std::collections::BTreeMap<String, Vec<(String, String, String, bool)>>,
+) {
+    let gen_dir = PathBuf::from(manifest).join("tests/generated");
+    let _ = fs::remove_dir_all(&gen_dir);
+    fs::create_dir_all(&gen_dir).unwrap();
     let mut top_mod = String::from("// AUTO-GENERATED by build.rs. Do not edit.\n\n");
-    for (topic, cases) in &by_topic {
+    for (topic, cases) in by_topic {
         let topic_dir = gen_dir.join(topic);
         fs::create_dir_all(&topic_dir).unwrap();
         let topic_ident = topic.replace('-', "_");
         let mut topic_mod = String::from("// AUTO-GENERATED. Do not edit.\n\n");
-        for (name, body, ignore) in cases {
-            let name_ident = name.replace('-', "_");
+        for (name, name_ident, body, ignore) in cases {
             let fn_name = format!("html_{}_{}", topic_ident, name_ident);
             let ignore_attr = if *ignore { "#[ignore]\n" } else { "" };
             let case_rs = format!(
@@ -73,8 +107,6 @@ fn main() {
         top_mod.push_str(&format!("mod {topic_ident};\n"));
     }
     fs::write(gen_dir.join("mod.rs"), top_mod).unwrap();
-    println!("cargo:rerun-if-changed=tests/cases");
-    println!("cargo:rerun-if-changed=build.rs");
 }
 
 /// Parse the HTML and emit imperative TestCtx calls.

float-pigment-forest/tests/mod.rs

@@ -677,4 +677,4 @@ fn convert_grid_auto(grid_auto: GridAuto) -> LayoutGridAuto<Len> {
     }
 }
 
-mod generated;
+include!(concat!(env!("OUT_DIR"), "/html_tests.rs"));