Merge branch 'main' into fix-4108-all-skipped-baseline

Author: louistrue

.changeset/lucky-donkeys-play.md

@@ -0,0 +1,16 @@
+---
+'@ifc-lite/cli': patch
+---
+
+`extract-entities`: keep spatial containment when extracting a strict subset.
+
+A real exporter writes ONE `IfcRelContainedInSpatialStructure` per storey listing
+every product in it, so the previous keep-the-relation-only-when-every-member-is-kept
+rule dropped containment on every `--product` / `--type` extraction from a real
+model: the extracted products landed outside the spatial tree and a viewer showed
+the storey with nothing under it. The relation's `RelatedElements` set is now
+rewritten down to the kept members instead, and `IfcRelReferencedInSpatialStructure`
+joins `IfcRelAggregates` and `IfcRelContainedInSpatialStructure` in the same
+handling. Relations still emit no dangling reference: the relating spatial parent
+and every non-set reference must be kept, and an empty intersection drops the
+relation. A relation that loses no member emits byte-identical to its source line.

.github/workflows/test.yml

@@ -276,10 +276,11 @@ jobs:
             # scripts/lib/revert-oracle.mjs byte-for-byte (any `test_*.py` /
             # `*_test.py`, any depth) rather than naming known marker
             # directories by hand, so a NEW Python test tree can never fall
-            # outside this filter the way rust/python's and
-            # scripts/perf/evidence's Python test fixtures did before this
+            # outside this filter the way rust/python's and the since-removed
+            # scripts/perf/evidence tree's Python test fixtures did before this
             # fix — both already existed unfiltered when this comment was
-            # written.
+            # written. That evidence tree was retired in #4112; the path no
+            # longer exists and nothing needs to be looked for there.
             python:
               - '**/test_*.py'
               - '**/*_test.py'

AGENTS.md

@@ -347,7 +347,7 @@ Opening voids (`IfcRelVoidsElement`) go through ONE exact path: the prepass reso
 Every element's meshes pass through a single funnel (`build_mesh_data` in `ifc-lite-processing::element`) that applies:
 
 - **Degenerate/sliver drops**: `drop_degenerate_triangles` and `drop_thin_triangles` remove zero-area and needle triangles at every output chokepoint (kernel output, funnel backstop), so CSG residue never reaches the GPU.
-- **Source vertex weld** (`mesh_weld::weld_indexed`): collapses vertices with identical f32 position AND coinciding quantized normal (and UV). The faceted-brep mesher emits per-face geometry that duplicates every shared corner 3-6x; the weld undoes that while keeping creases split, because the normal is part of the merge key. Flat shading is preserved by construction (a cube keeps its 24 vertices), and texture seams stay split via the UV key. A naive position-only weld is deliberately NOT used, since it would smooth creases and break flat shading.
+- **Source vertex weld** (`mesh_weld`, since #4103 run from the router's placement appliers in the OBJECT frame, and at this funnel only for geometry with no cross-occurrence identity to protect — see the `mesh_weld` module doc): collapses vertices with identical f32 position AND coinciding quantized normal (and UV). The faceted-brep mesher emits per-face geometry that duplicates every shared corner 3-6x; the weld undoes that while keeping creases split, because the normal is part of the merge key. Flat shading is preserved by construction (a cube keeps its 24 vertices), and texture seams stay split via the UV key. A naive position-only weld is deliberately NOT used, since it would smooth creases and break flat shading.
 
 The weld and drops are deterministic across native and wasm32 targets.