draftwright 0.4.34__tar.gz → 0.4.35__tar.gz

{draftwright-0.4.34 → draftwright-0.4.35}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: draftwright
-Version: 0.4.34
+Version: 0.4.35
 Summary: Automated technical-drawing generation for build123d
 Project-URL: Homepage, https://github.com/pzfreo/draftwright
 Project-URL: Repository, https://github.com/pzfreo/draftwright

{draftwright-0.4.34 → draftwright-0.4.35}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "draftwright"
-version = "0.4.34"
+version = "0.4.35"
 description = "Automated technical-drawing generation for build123d"
 readme = "README.md"
 license = { file = "LICENSE" }

{draftwright-0.4.34 → draftwright-0.4.35}/src/draftwright/annotations/leaders.py

@@ -1469,6 +1469,17 @@ def _annotation_fixed_ink(dwg, name, annotation, *, max_components=None):
         components.append(_FixedInkComponent(f"{name}:label", box=label, owner=owner, kind=kind))
         if exhausted():
             return _FIXED_INVENTORY_EXHAUSTED
+
+    # The stock helper Leader has a closed analytical description here: segment zero is
+    # lowered by ``_leader_ink_polygons`` (shaft plus arrowhead), every later shelf segment
+    # is lowered at exact line width, and ``label_bbox`` carries its text. Walking and
+    # meshing all of its OCC faces only re-proves those same components. Dense real-part
+    # sheets can contain dozens of leaders and were spending seconds per assembly on that
+    # duplicate validation. Keep the exact-type gate: routed/custom leader subclasses may
+    # render extra ink and must continue through the conservative residual-face scan.
+    if type(annotation).__name__ == "Leader" and segments and label is not None:
+        return tuple(components)
+
     remaining = None if max_components is None else max_components - len(components)
     residual = _rendered_residual_components(
         name,

{draftwright-0.4.34 → draftwright-0.4.35}/tests/test_issue_1166_cross_pass_feature_leaders.py

@@ -1659,6 +1659,26 @@ def test_empty_faces_accept_an_explicit_complete_fixed_ink_footprint(tiny_box_dw
     assert set(components[0].polygons[0]) == set(polygon)
 
 
+def test_stock_leader_fixed_ink_does_not_remesh_closed_analytical_geometry(
+    tiny_box_dwg, monkeypatch
+):
+    drawing = tiny_box_dwg
+    leader = Leader((10.0, 10.0, 0.0), (20.0, 20.0, 0.0), "HOLE", drawing.draft)
+
+    def unexpected_faces():
+        raise AssertionError("a stock Leader's published ink is already complete")
+
+    monkeypatch.setattr(leader, "faces", unexpected_faces)
+    components = _annotation_fixed_ink(drawing, "stock_leader", leader, max_components=8)
+
+    assert components is not _FIXED_INVENTORY_EXHAUSTED
+    assert [component.name for component in components] == [
+        "stock_leader:segment:0",
+        "stock_leader:segment:1",
+        "stock_leader:label",
+    ]
+
+
 @pytest.mark.parametrize("raising_attribute", ["segments", "fixed_ink_polygons", "label_bbox"])
 def test_raising_fixed_metadata_uses_the_unavailable_inventory_contract(
     raising_attribute, tiny_box_dwg