Merge pull request #51 from Kletternaut/fix/remove-invalid-comments-in-templates

fix: remove invalid ;; comments from template (KiCAD parse error, regression from b98c94b)

CHANGELOG.md

@@ -2,6 +2,61 @@
 
 All notable changes to the KiCAD MCP Server project are documented here.
 
+## [2.2.2-alpha] - 2026-03-01
+
+### New MCP Tools
+
+- `route_pad_to_pad` – Convenience wrapper around `route_trace` that looks up pad positions
+  automatically. Accepts `fromRef`/`fromPad`/`toRef`/`toPad` instead of raw XY coordinates.
+  Auto-detects net from pad assignment (overridable via `net` param). Saves ~2 tool calls per
+  connection (~64 calls for a full TMC2209 board compared to the 3-step get_pad_position flow).
+  Live tested: ESP32 ↔ TMC2209 STEP/DIR traces routed without prior coordinate lookup. ✅
+
+- `copy_routing_pattern` – Now registered as MCP tool in TypeScript layer (`routing.ts`).
+  Was previously implemented in Python but missing from the MCP tool registry.
+  Parameters: `sourceRefs`, `targetRefs`, `includeVias?`, `traceWidth?`.
+
+### Bug Fixes
+
+- `add_schematic_component` / `DynamicSymbolLoader`: ignored project-local `sym-lib-table`.
+  `find_library_file()` only searched global KiCAD install directories, causing "library not
+  found" errors for any symbol in a project-local `.kicad_sym` file. Fix: added `project_path`
+  parameter; reads project `sym-lib-table` first via new `_resolve_library_from_table()` helper
+  before falling back to global dirs. `project_path` is auto-derived from the schematic path.
+
+- `place_component`: ignored project-local `fp-lib-table`. `FootprintLibraryManager` was
+  initialised once at server start without a project path, so self-created `.kicad_mod`
+  footprints were never found. Fix: new `boardPath` parameter in TypeScript + Python;
+  `_handle_place_component` wrapper recreates `FootprintLibraryManager(project_path=…)` whenever
+  the active project changes (cached to avoid redundant recreation).
+
+- `copy_routing_pattern`: copied 0 traces when pads had no net assignments. The filter
+  `track.GetNetname() in source_nets` always returned empty when pads were placed without net
+  assignment. Fix: geometric fallback using bounding box of source footprint pads ±5mm
+  tolerance. Response includes `filterMethod` field indicating which mode was used
+  (`"net-based"` or `"geometric (pads have no nets)"`).
+
+- `template_with_symbols.kicad_sch`, `template_with_symbols_expanded.kicad_sch`: restored
+  format version `20250114` (KiCAD 9) after upstream commit `2b38796` accidentally downgraded
+  both files to `20240101`. KiCAD 9 rejects schematics with outdated version numbers.
+
+- **CRITICAL: `template_with_symbols_expanded.kicad_sch`**: removed 7 invalid `;;` comment
+  lines introduced by upstream commit `b98c94b`. KiCAD's S-expression parser does not support
+  any comment syntax — it expects every non-empty, non-whitespace line to start with `(`.
+  The comments (`;; PASSIVES`, `;; SEMICONDUCTORS`, `;; INTEGRATED CIRCUITS`, `;; CONNECTORS`,
+  `;; POWER/REGULATORS`, `;; MISC`, `;; TEMPLATE INSTANCES (...)`) caused KiCAD 9 to reject
+  every schematic created from this template with a hard parse error:
+  > `Expecting '(' in <file>.kicad_sch, line 8, offset 5`
+  **Action required for existing projects:** delete every line beginning with `;;` from any
+  `.kicad_sch` file created between upstream commit `b98c94b` and this fix.
+
+### Maintenance
+
+- `.gitignore`: added `*.kicad_pcb.bak`, `*.kicad_pro.bak` alongside existing `-bak` variants;
+  consolidated personal/local files under `myContribution/`.
+
+---
+
 ## [2.2.1-alpha] - 2026-02-28
 
 ### New MCP Tools

python/templates/template_with_symbols_expanded.kicad_sch

@@ -5,7 +5,6 @@
   (paper "A4")
 
   (lib_symbols
-    ;; PASSIVES
     (symbol "Device:R" (pin_numbers hide) (pin_names (offset 0)) (in_bom yes) (on_board yes)
       (property "Reference" "R" (at 2.032 0 90)
         (effects (font (size 1.27 1.27)))
@@ -182,7 +181,6 @@
         )
       )
     )
-    ;; SEMICONDUCTORS
     (symbol "Device:D" (pin_numbers hide) (pin_names (offset 1.016) hide) (in_bom yes) (on_board yes)
       (property "Reference" "D" (at 0 2.54 0)
         (effects (font (size 1.27 1.27)))
@@ -497,7 +495,6 @@
         )
       )
     )
-    ;; INTEGRATED CIRCUITS
     (symbol "Amplifier_Operational:LM358" (pin_names (offset 0.127)) (in_bom yes) (on_board yes)
       (property "Reference" "U" (at 0 5.08 0)
         (effects (font (size 1.27 1.27)) (justify left))
@@ -546,7 +543,6 @@
         )
       )
     )
-    ;; CONNECTORS
     (symbol "Connector_Generic:Conn_01x02" (pin_names (offset 1.016) hide) (in_bom yes) (on_board yes)
       (property "Reference" "J" (at 0 2.54 0)
         (effects (font (size 1.27 1.27)))
@@ -635,7 +631,6 @@
         )
       )
     )
-    ;; POWER/REGULATORS
     (symbol "Regulator_Linear:AMS1117-3.3" (pin_names (offset 0.254)) (in_bom yes) (on_board yes)
       (property "Reference" "U" (at -3.81 3.175 0)
         (effects (font (size 1.27 1.27)))
@@ -670,7 +665,6 @@
         )
       )
     )
-    ;; MISC
     (symbol "Switch:SW_Push" (pin_numbers hide) (pin_names (offset 1.016) hide) (in_bom yes) (on_board yes)
       (property "Reference" "SW" (at 1.27 2.54 0)
         (effects (font (size 1.27 1.27)) (justify left))
@@ -721,7 +715,6 @@
     )
   )
 
-  ;; TEMPLATE INSTANCES (placed offscreen at -100, -110, etc.)
   (symbol (lib_id "Device:R") (at -100 -100 0) (unit 1)
     (in_bom no) (on_board no) (dnp yes)
     (uuid 00000000-0000-0000-0000-000000000001)