Warn if exactly one of manufacturer or mpn is set (XOR)

Author: akhilles

crates/pcb-sch/src/physical.rs

@@ -3622,7 +3622,6 @@ mod tests {
 
         let heap = Heap::new();
         let range = heap.alloc(physical_range(3.0, 3.6, PhysicalUnit::Volts));
-        let gnd_str = heap.alloc("0V");
 
         // Get the range from heap
         let range_val = range.downcast_ref::<PhysicalRange>().unwrap();

crates/pcb-zen-core/src/lang/component.rs

@@ -1060,6 +1060,30 @@ where
                         .map(|s| s.to_owned())
                 });
 
+            // Warn if exactly one of manufacturer or mpn is set (XOR)
+            if final_manufacturer.is_some() != final_mpn.is_some() {
+                if let Some(call_site) = eval_ctx.call_stack_top_location() {
+                    use crate::Diagnostic;
+                    use starlark::errors::EvalSeverity;
+                    let body = if final_manufacturer.is_some() {
+                        "MPN must be specified if manufacturer is specified"
+                    } else {
+                        "Manufacturer must be specified if MPN is specified"
+                    };
+                    let diag = Diagnostic {
+                        path: call_site.filename().to_string(),
+                        span: Some(call_site.resolve_span()),
+                        severity: EvalSeverity::Warning,
+                        body: body.to_string(),
+                        call_stack: None,
+                        child: None,
+                        source_error: None,
+                        suppressed: false,
+                    };
+                    eval_ctx.add_diagnostic(diag);
+                }
+            }
+
             // If datasheet is not explicitly provided, try to get it from properties, then symbol properties
             // Skip empty strings and "~" (KiCad's placeholder for no datasheet) - prefer None over empty
             let final_datasheet = datasheet_val

crates/pcb-zen-core/tests/snapshots/component__component_manufacturer_from_symbol.snap

@@ -2,6 +2,7 @@
 source: crates/pcb-zen-core/tests/component.rs
 expression: output
 ---
+Warning: test.zen:2:1-10:2 MPN must be specified if manufacturer is specified
 {
     <root>: Module {
         path: <root>,

crates/pcb-zen-core/tests/snapshots/component__component_with_manufacturer.snap

@@ -2,6 +2,7 @@
 source: crates/pcb-zen-core/tests/component.rs
 expression: output
 ---
+Warning: test.zen:2:1-14:2 MPN must be specified if manufacturer is specified
 {
     <root>: Module {
         path: <root>,