Adom Library

@@ -2338,11 +2338,21 @@ fn emit_symbol_block(xml: &mut String, sym: &ParsedSymbol, symbol_art: Option<&S     deduped_names } -pub(crate) fn generate_lbr(sym: &ParsedSymbol, fp_content: Option<&str>, symbol_art: Option<&SymbolArt>, symbol_art_scale: f64, pin_map: &std::collections::HashMap<String, String>, part_attrs: &[(String, String)]) -> String {+pub(crate) fn generate_lbr(sym: &ParsedSymbol, fp_content: Option<&str>, symbol_art: Option<&SymbolArt>, symbol_art_scale: f64, pin_map: &std::collections::HashMap<String, String>, part_attrs: &[(String, String)]) -> Result<String, String> {     generate_lbr_gates(sym, fp_content, symbol_art, symbol_art_scale, pin_map, part_attrs, &[], &[]) } -pub(crate) fn generate_lbr_gates(sym: &ParsedSymbol, fp_content: Option<&str>, symbol_art: Option<&SymbolArt>, symbol_art_scale: f64, pin_map: &std::collections::HashMap<String, String>, part_attrs: &[(String, String)], alts: &[AltGate], variants: &[PackageVariant]) -> String {+pub(crate) fn generate_lbr_gates(sym: &ParsedSymbol, fp_content: Option<&str>, symbol_art: Option<&SymbolArt>, symbol_art_scale: f64, pin_map: &std::collections::HashMap<String, String>, part_attrs: &[(String, String)], alts: &[AltGate], variants: &[PackageVariant]) -> Result<String, String> {+    // An explicitly supplied zero-pad footprint is not an absent footprint.+    // Reject this unsupported off-board export before inventing copper or writing output.+    if fp_content.is_some_and(|km| parse_mod_pads(km).is_empty()) {+        return Err("[ZERO_PAD_FOOTPRINT] Supplied footprint has no parsed electrical pads; off-board/mechanical export is unsupported. No placeholder copper was generated. Keep the schematic-only component separate from its actual PCB connector.".into());+    }+    for variant in variants {+        if parse_mod_pads(&variant.km).is_empty() {+            return Err(format!("[ZERO_PAD_FOOTPRINT] Package variant {} has no parsed electrical pads; export refused without inventing copper.", variant.suffix));+        }+    }     let mut xml = String::with_capacity(8192);      xml.push_str("<?xml version=\"1.0\" encoding=\"utf-8\"?>\n");@@ -2627,7 +2637,7 @@ Place them within the pad they serve, or they will import as unconnected copper.     xml.push_str("</devicesets>\n");      xml.push_str("</library>\n</drawing>\n</eagle>");-    xml+    Ok(xml) }  // ── Linter ──@@ -3536,7 +3546,10 @@ The geometry is fine but a BOM cannot order this part - pass --part-metadata or                 println!("HINT: {} alternate symbol(s) as extra gates of {}: {}", alts.len(), parsed.name,                     alts.iter().map(|a| format!("{} ({})", a.gate, a.sym.name)).collect::<Vec<_>>().join(", "));             }-            let lbr = generate_lbr_gates(&parsed, fp_content.as_deref(), art.as_ref(), symbol_art_scale, &pin_map, &part_attrs, &alts, &pkg_variants);+            let lbr = generate_lbr_gates(&parsed, fp_content.as_deref(), art.as_ref(), symbol_art_scale, &pin_map, &part_attrs, &alts, &pkg_variants).unwrap_or_else(|e| {+                eprintln!("ERROR: {e}");+                process::exit(2);+            });              // Losses are decided here, once, before anything is linted or             // written: the parser only collected the margin pads.@@ -5142,3 +5155,37 @@ mod lint_tests {         assert!(r.errors.iter().any(|e| e.code == "empty-packages"), "{:?}", r.errors);     } }++#[cfg(test)]+mod zero_pad_export_tests {+    use super::*;+    const SENSOR: &str = include_str!("../tests/fixtures/zero-pad/sensor.kicad_sym");+    const MECHANICAL: &str = include_str!("../tests/fixtures/zero-pad/sensor.kicad_mod");+    const ELECTRICAL: &str = r#"(footprint "two-pin"+      (pad "1" smd rect (at -2 0) (size 1 1) (layers "F.Cu"))+      (pad "2" smd rect (at 2 0) (size 1 1) (layers "F.Cu")))"#;+    #[test]+    fn explicit_zero_pad_footprint_is_refused_without_copper() {+        let sym = parse_kicad_sym(SENSOR, None).unwrap();+        let err = generate_lbr(&sym, Some(MECHANICAL), None, 0.38, &Default::default(), &[]).unwrap_err();+        assert!(err.contains("ZERO_PAD_FOOTPRINT"));+    }+    #[test]+    fn zero_pad_variant_is_refused_as_well() {+        let sym = parse_kicad_sym(SENSOR, None).unwrap();+        let variants = [PackageVariant { suffix: "-offboard".into(), km: MECHANICAL.into(), description: None }];+        let err = generate_lbr_gates(&sym, Some(ELECTRICAL), None, 0.38, &Default::default(), &[], &[], &variants).unwrap_err();+        assert!(err.contains("ZERO_PAD_FOOTPRINT"));+        assert!(err.contains("-offboard"));+    }+    #[test]+    fn real_pads_remain_exact_and_absent_footprint_stays_distinct() {+        let sym = parse_kicad_sym(SENSOR, None).unwrap();+        let xml = generate_lbr(&sym, Some(ELECTRICAL), None, 0.38, &Default::default(), &[]).unwrap();+        assert_eq!(xml.matches("<smd ").count(), 2);+        assert!(xml.contains("x=\"-2.0000\""));+        assert!(xml.contains("x=\"2.0000\""));+        let placeholder = generate_lbr(&sym, None, None, 0.38, &Default::default(), &[]).unwrap();+        assert!(placeholder.contains("placeholder footprint"));+    }+}