Fix duplicate names in composite components

Author: gpeairs

CHANGELOG.md

@@ -4,6 +4,10 @@ The format of this changelog is based on
 [Keep a Changelog](https://keepachangelog.com/), and this project adheres to
 [Semantic Versioning](https://semver.org/).
 
+## Unreleased
+
+  - Fixed issue causing duplicate `Cell` names with paths and composite components, where rendering would use the component's name rather than a unique name
+
 ## 1.4.2 (2025-07-16)
 
   - Removed invalid keyword constructor without type parameters for `@compdef`-ed components with type parameters, so it can be overridden without warnings

src/paths/paths.jl

@@ -393,6 +393,14 @@ function DeviceLayout._geometry!(cs::CoordinateSystem, p::Path)
     return addref!(cs, p)
 end
 
+function DeviceLayout.coordsys_name(p::Path)
+    # If empty, rendering path directly
+    isempty(p._geometry) && return name(p._geometry)
+    # If not empty, p._geometry holds `p` (e.g. `build!` was called)
+    # and the path coordsys needs its own uniquename
+    return uniquename(name(p))
+end
+
 """
     path_in(h::PointHook)
 

src/render/render.jl

@@ -96,7 +96,10 @@ function _render!(
         cur_cell = if already_seen
             memoized_cells[cur_cs]
         else
-            Cell{S}(name(cur_cs))
+            Cell{S}(coordsys_name(cur_cs))
+        end
+        if name(cur_cell) == "cs_pz"
+            @show cur_cs
         end
 
         # If it's a new CS, render the contents, push refs to the stack, and add to memoized_cells

src/schematics/components/components.jl

@@ -414,3 +414,5 @@ function single_component_expr(compname, comptype; params...)
         $escname
     end
 end
+
+DeviceLayout.coordsys_name(comp::AbstractComponent) = name(geometry(comp))

src/structures.jl

@@ -216,5 +216,7 @@ function flatten end
 
 # type of coordinate system created by flatten
 function coordsys_type end # defined elsewhere, once CoordinateSystem is defined
+# name used by CoordinateSystem representation
+coordsys_name(geom::GeometryStructure) = name(geom)
 
 Base.isempty(geom::GeometryStructure) = isempty(elements(geom)) && isempty(refs(geom))

test/test_schematicdriven.jl

@@ -617,10 +617,12 @@ end
     g3 = copy(g)
     @test length(find_components(TestCompVariant, g3)) == 18
     @test length(find_components(TestCompVariant, g3, depth=2)) == 12
-    @test length(find_components(c -> name(c)[1:1] == "z", g3, depth=2)) == 4
+    # There are two unique components named "z" appearing a combined 4 times up to depth 2
+    @test length(find_components(c -> name(c) == "z", g3, depth=2)) == 4
     @test length(
-        unique(component.(g3[find_components(c -> name(c)[1:1] == "z", g3, depth=2)]))
+        unique(component.(g3[find_components(c -> name(c) == "z", g3, depth=2)]))
     ) == 2
+
     g3_flat = SchematicDrivenLayout.flatten(g3)
     g3 = SchematicDrivenLayout.flatten(g3, depth=1)
     @test length(components(g3)) == 14
@@ -633,6 +635,11 @@ end
 
     @test origin(floorplan3, find_components(c -> name(c)[1:1] == "z", g3)[end]) ≈
           Point(200.2μm, 0.1μm)
+    # Make sure that cells for unique "z" have unique names when rendering
+    c = Cell(floorplan3.coordinate_system)
+    a = []
+    traverse!(a, c)
+    @test length(findall(x -> name(last(x)) == DeviceLayout.coordsys_name(zline), a)) == 2
 
     ### Full composite component
     p = Path(Point(0μm, 0μm), name="pz")
@@ -691,6 +698,12 @@ end
 
     @test hooks(bq2, "bq1", :xy) == hooks(bq2, 1 => :xy) # using subcomp name or index=>hsym
     @test keys(SchematicDrivenLayout.subcomponents(bq2)) == (:bq1, :bq2, :pz)
+    # test creating Cell without build uses path's coordsys_name
+    c = Cell(geometry(bq2); map_meta=_ -> GDSMeta())
+    a = []
+    traverse!(a, c)
+    @test DeviceLayout.coordsys_name(p) != name(p)
+    @test isempty(findall(x -> name(last(x)) == name(p), a))
 
     g = SchematicGraph("comp")
     bq2_node = add_node!(g, bq2)