draftwright 0.4.26__tar.gz → 0.4.27__tar.gz

{draftwright-0.4.26 → draftwright-0.4.27}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: draftwright
-Version: 0.4.26
+Version: 0.4.27
 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.26 → draftwright-0.4.27}/docs/reference/sheet.md

@@ -9,7 +9,10 @@ with an underscore.
 Use `Sheet(part, projection="first")` or `build_drawing(part, projection="first")`
 for first-angle layout and its matching projection symbol. The CLI equivalent is
 `--projection first`. Explicit `"third"` selects third-angle layout and its symbol;
-omitting `projection` keeps third-angle layout without a symbol.
+omitting `projection` uses third-angle layout and its symbol. Use
+`projection_symbol=False` (CLI: `--no-projection-symbol`) to omit the symbol for an
+embedded illustration; this does not change the projection convention or view layout.
+The visibility choice also survives generated-script replay.
 
 View names retain their physical viewing directions in the working part frame. Changing
 convention changes sheet relationships, not which side of the part a name shows:

{draftwright-0.4.26 → draftwright-0.4.27}/pyproject.toml

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

{draftwright-0.4.26 → draftwright-0.4.27}/src/draftwright/_core.py

@@ -1260,9 +1260,10 @@ class Analysis:
     # Draw a sheet border/frame (#767). When True, `margin` is already the reserved content
     # margin (`_content_margin(True)`), so content clears the frame drawn at `_MARGIN`.
     frame: bool = False
-    # Projection-method symbol (#769): "third" / "first" (ISO 5456-2) or None (omit).
+    # Requested convention; None resolves to third-angle. Symbol visibility is independent.
     projection: str | None = None
     projection_convention: str = "third"
+    projection_symbol: bool = True
     text_position: str = "inline"
     text_orientation: str = "aligned"
     # Draw the ISO 5457 zone-grid border ruler (#768). Implies a frame (the ticks sit on it).
@@ -1481,11 +1482,15 @@ def _add_projection_symbol(dwg, a: Analysis):
     just above the drawn title block (deterministic empty space — the block reserves _TB_H but
     draws shorter). Registered ``projection_symbol`` with an ``is_projection_symbol`` identity
     rider. Unlike the page-spanning frame it is NOT lint-exempt: it's a small, well-placed glyph,
-    so lint covers it and a future mispositioning is caught. Gated on ``a.projection``."""
+    so lint covers it and a future mispositioning is caught. Suppressed only by
+    ``projection_symbol=False``."""
+    if not a.projection_symbol:
+        return
+
     from build123d_drafting import ProjectionSymbol
 
     sym = ProjectionSymbol(
-        a.projection,
+        a.projection_convention,
         draft=draft_preset(
             font_size=dwg.draft.font_size,
             decimal_precision=dwg.draft.decimal_precision,
@@ -1496,7 +1501,8 @@ def _add_projection_symbol(dwg, a: Analysis):
     bx, by = (b.min.X + b.max.X) / 2, (b.min.Y + b.max.Y) / 2
     w, h = b.max.X - b.min.X, b.max.Y - b.min.Y
     # Right side of the title-block column, near the top of its reserved band.
-    cx = a.PAGE_W - _TB_CLEAR - w / 2 - 3
+    # Sheet frames reserve an inner content margin; keep furniture inside it too.
+    cx = a.PAGE_W - max(_TB_CLEAR + 3, a.margin) - w / 2
     cy = _TB_CLEAR + _TB_H - h / 2 - 2
     sym = sym.locate(Location((cx - bx, cy - by, 0)))
     sym.is_projection_symbol = True

{draftwright-0.4.26 → draftwright-0.4.27}/src/draftwright/analysis.py

@@ -781,6 +781,7 @@ def _analyse(
     company="",
     frame: bool = False,
     projection: str | None = None,
+    projection_symbol: bool = True,
     text_position: str = "inline",
     text_orientation: str = "aligned",
     zones: bool = False,
@@ -1442,6 +1443,7 @@ def _analyse(
         company=company,
         frame=frame,
         projection=projection,
+        projection_symbol=projection_symbol,
         text_position=text_position,
         text_orientation=text_orientation,
         projection_convention=convention,

{draftwright-0.4.26 → draftwright-0.4.27}/src/draftwright/annotations/orchestrator.py

@@ -900,8 +900,7 @@ def _auto_annotate(dwg, a: Analysis, *, detail_view: bool = False):
 
     def _s_projection_symbol():
         # ISO 5456-2 projection-method glyph (#769) in the reserved title-block band.
-        if a.projection:
-            _add_projection_symbol(dwg, a)
+        _add_projection_symbol(dwg, a)
 
     def _s_tabulate():
         # Escalate to a hole table when the plan view is too dense to dimension

{draftwright-0.4.26 → draftwright-0.4.27}/src/draftwright/builder.py

@@ -787,8 +787,7 @@ def _assemble(
             _add_sheet_frame(dwg, a)
         if a.zones:  # zone-grid ruler (#768), on the frame
             _add_zone_grid(dwg, a)
-        if a.projection:  # projection-method glyph (#769) — auto path adds it via the orchestrator
-            _add_projection_symbol(dwg, a)
+        _add_projection_symbol(dwg, a)
 
     # The NTS caption is post-fit late furniture too, and goes FIRST: it is tied to the
     # iso block it labels, whereas a table may sit anywhere the sheet has room. Placing
@@ -1196,6 +1195,7 @@ def _build_drawing_once(
     company: str = "",
     frame: bool = False,
     projection: str | None = None,
+    projection_symbol: bool = True,
     zones: bool = False,
     reproducible: bool = False,
     framed_recognition: bool = False,
@@ -1336,6 +1336,7 @@ def _build_drawing_once(
             company=company,
             frame=frame,
             projection=projection,
+            projection_symbol=projection_symbol,
             text_position=text_position,
             text_orientation=text_orientation,
             zones=zones,
@@ -1912,6 +1913,8 @@ def build_drawing(
     _include_iso: bool = True,
     _view_constraints=None,
     _document_input=None,
+    *,
+    projection_symbol: bool = True,
 ) -> Drawing:
     """Build a drawing, protecting required annotations under an explicit scale.
 
@@ -1926,14 +1929,14 @@ def build_drawing(
     recognition frame. Raw remains the default. Other arguments and return semantics are
     unchanged from the one-pass builder.
 
-    ``projection='third'`` adds the matching projection symbol. The default omits the
-    symbol but uses the same third-angle layout. ``projection='first'`` places plan below
+    Third-angle layout and its matching projection symbol are the default.
+    ``projection_symbol=False`` suppresses only the symbol. ``projection='first'`` places plan below
     front and side to its left, keeping the physical viewing directions unchanged.
 
     ``text_position="inline"|"above"`` and ``text_orientation="aligned"|"horizontal"``
     independently select dimension typography. Defaults preserve existing appearance.
     """
-    validate_projection(projection)
+    validate_projection(projection, projection_symbol=projection_symbol)
     _dimension_draft(text_position, text_orientation)
     if scale_policy not in {"strict", "fallback", "permissive"}:
         raise ValueError(
@@ -1964,6 +1967,7 @@ def build_drawing(
         company=company,
         frame=frame,
         projection=projection,
+        projection_symbol=projection_symbol,
         text_position=text_position,
         text_orientation=text_orientation,
         zones=zones,
@@ -2855,6 +2859,8 @@ def make_drawing(
     framed_recognition: bool = False,
     text_position: str = "inline",
     text_orientation: str = "aligned",
+    *,
+    projection_symbol: bool = True,
 ) -> tuple[str, str]:
     """Generate a 4-view technical drawing from a STEP file or build123d object.
 
@@ -2919,6 +2925,7 @@ def make_drawing(
         company=company,
         frame=frame,
         projection=projection,
+        projection_symbol=projection_symbol,
         text_position=text_position,
         text_orientation=text_orientation,
         zones=zones,

{draftwright-0.4.26 → draftwright-0.4.27}/src/draftwright/cli.py

@@ -171,7 +171,12 @@ def main(
     projection: str = typer.Option(
         "",
         "--projection",
-        help="Projection convention: 'first' or 'third' (default: third-angle, no symbol)",
+        help="Projection convention: 'first' or 'third' (default: third-angle)",
+    ),
+    projection_symbol: bool = typer.Option(
+        True,
+        "--projection-symbol/--no-projection-symbol",
+        help="Show the symbol for the selected projection convention",
     ),
     text_position: str = typer.Option("inline", help="Dimension text position: inline or above"),
     text_orientation: str = typer.Option(
@@ -294,6 +299,7 @@ def main(
                 frame=frame,
                 zones=zones,
                 projection=projection or None,
+                projection_symbol=projection_symbol,
                 text_position=text_position,
                 text_orientation=text_orientation,
                 part_expr=source.seam,
@@ -320,6 +326,7 @@ def main(
                 frame=frame,
                 zones=zones,
                 projection=projection or None,
+                projection_symbol=projection_symbol,
                 text_position=text_position,
                 text_orientation=text_orientation,
                 pmi=pmi.value if pmi is not None else "off",
@@ -355,6 +362,7 @@ def main(
                 company=company,
                 frame=frame,
                 projection=projection or None,
+                projection_symbol=projection_symbol,
                 text_position=text_position,
                 text_orientation=text_orientation,
                 zones=zones,

{draftwright-0.4.26 → draftwright-0.4.27}/src/draftwright/sheet.py

@@ -1110,12 +1110,13 @@ class Sheet:
         company=None,
         frame=None,
         projection=None,
+        projection_symbol=True,
         text_position="inline",
         text_orientation="aligned",
         zones=None,
         detail_view=None,
     ):
-        validate_projection(projection)
+        validate_projection(projection, projection_symbol=projection_symbol)
         _dimension_draft(text_position, text_orientation)
         self._part = part
         # (token, feature) entries — identity, not position (#908). `_features` is the
@@ -1201,6 +1202,7 @@ class Sheet:
             ("company", company),
             ("frame", frame),
             ("projection", projection),
+            ("projection_symbol", projection_symbol),
             ("text_position", text_position),
             ("text_orientation", text_orientation),
             ("zones", zones),

{draftwright-0.4.26 → draftwright-0.4.27}/src/draftwright/sheet_emit.py

@@ -2084,6 +2084,7 @@ def emit_sheet_script(
     frame: bool = False,
     zones: bool = False,
     projection: str | None = None,
+    projection_symbol: bool = True,
     text_position: str = "inline",
     text_orientation: str = "aligned",
     object_ref: bool = False,
@@ -2130,6 +2131,7 @@ def emit_sheet_script(
     declaration a person edits; evidence about the run that produced it belongs in the sidecar
     document beside it, where a reader can diff or re-read it without parsing Python (#1460)."""
     _validate_scale_policy(scale, scale_policy)
+    validate_projection(projection, projection_symbol=projection_symbol)
     _dimension_draft(text_position, text_orientation)
     # The script declares this model — `model` plus an envelope when the overall height would
     # otherwise be unnameable under the mirrored (authored) set. BEFORE the import scan, since
@@ -2216,6 +2218,8 @@ def emit_sheet_script(
         ctor.append("zones=True")
     if projection:
         ctor.append(f"projection={projection!r}")
+    if not projection_symbol:
+        ctor.append("projection_symbol=False")
     if text_position != "inline":
         ctor.append(f"text_position={text_position!r}")
     if text_orientation != "aligned":
@@ -2480,6 +2484,7 @@ def generate_sheet_script(
     frame: bool = False,
     zones: bool = False,
     projection: str | None = None,
+    projection_symbol: bool = True,
     text_position: str = "inline",
     text_orientation: str = "aligned",
     pmi: Literal["off", "report", "annotate"] = "off",
@@ -2500,7 +2505,7 @@ def generate_sheet_script(
     surface (flagged inline).
     *part_expr*, when given, overrides the ``part = …`` seam — e.g. the import seam from
     :func:`resolve_object_spec` so the script references a live module (#469)."""
-    validate_projection(projection)
+    validate_projection(projection, projection_symbol=projection_symbol)
     _validate_scale_policy(scale, scale_policy)
     _dimension_draft(text_position, text_orientation)
     is_shape = isinstance(step_file, Shape)
@@ -2578,6 +2583,7 @@ def generate_sheet_script(
                 frame=frame,
                 zones=zones,
                 projection=projection,
+                projection_symbol=projection_symbol,
                 text_position=text_position,
                 text_orientation=text_orientation,
                 pmi=pmi,
@@ -2607,6 +2613,7 @@ def generate_sheet_script(
             frame=frame,
             zones=zones,
             projection=projection,
+            projection_symbol=projection_symbol,
             text_position=text_position,
             text_orientation=text_orientation,
             object_ref=is_shape,

{draftwright-0.4.26 → draftwright-0.4.27}/src/draftwright/view_plan.py

@@ -32,8 +32,10 @@ from types import MappingProxyType
 from typing import Any
 
 
-def validate_projection(projection: str | None) -> None:
-    """Validate the supported convention names before loading drawing input."""
+def validate_projection(projection: str | None, *, projection_symbol: bool = True) -> None:
+    """Validate projection convention and symbol visibility before loading drawing input."""
+    if not isinstance(projection_symbol, bool):
+        raise ValueError("projection_symbol must be a boolean")
     if projection not in (None, "", "first", "third"):
         raise ValueError(f"unknown projection {projection!r}; expected 'first' or 'third'")
 

{draftwright-0.4.26 → draftwright-0.4.27}/tests/test_add_dimension.py

@@ -1151,7 +1151,7 @@ class TestOmittedDimensionsDoNotRender:
 
         drawn = _names(sheet.build())
         assert "m_env_width" in drawn, "the one authored measurement"
-        furniture = ("m_cm", "m_locx", "m_locy", "note_", "title_block")
+        furniture = ("m_cm", "m_locx", "m_locy", "note_", "title_block", "projection_symbol")
         assert not [n for n in drawn if n != "m_env_width" and not n.startswith(furniture)], (
             f"an unauthored dimension reached the page: {sorted(drawn)}"
         )

{draftwright-0.4.26 → draftwright-0.4.27}/tests/test_compiled_plan_boundary.py

@@ -43,6 +43,7 @@ _VALUE_FREE = (
     "m_cm",  # centre marks — sized off the hole they mark (#875)
     "note_",  # the ISO NTS note
     "title_block",
+    "projection_symbol",  # identifies the view convention; no feature measurement
     "section_",  # cutting-plane arrows and label
     "hatch",
     "detail_marker",

{draftwright-0.4.26 → draftwright-0.4.27}/tests/test_issue_1144_transactional_hole_table.py

@@ -1901,7 +1901,7 @@ def test_automatic_table_preserves_authored_through_wording(indicator, projectio
         for feature in model.features
     ]
     drawing = build_drawing(part, model=model, page="A3", projection=projection)
-    assert ("projection_symbol" in drawing.annotations()) is (projection is not None)
+    assert drawing.get_annotation("projection_symbol").method == (projection or "third")
     assert "hole_table_plan" in drawing.annotations()
     table = drawing.get_annotation("hole_table_plan")
     depth_column = table.table_rows[0].index("DEPTH")

{draftwright-0.4.26 → draftwright-0.4.27}/tests/test_issue_1217_the_facility_is_shared.py

@@ -375,9 +375,10 @@ class TestNoMeasuredAnnotationEscapesUnclaimed:
 
     @pytest.mark.slow
     def test_the_furniture_really_is_silent(self):
-        # The other side of the same claim: on nist_ctc_02 the 89 unclaimed annotations are
-        # centre marks, bolt circles, notes and the title block. The allow-list is exactly the
-        # four types measured there — a wider set asserts nothing about types that never occur
+        # The other side of the same claim: on nist_ctc_02 the silent annotations are
+        # centre marks, bolt circles, notes, the projection symbol and the title block.
+        # The allow-list is exactly the types observed — a wider set asserts nothing about
+        # types that never occur
         # (#1225 review, finding 7).
         #
         # "Silent" here means silent TO THE READER, and the title block is the honest edge of
@@ -387,7 +388,7 @@ class TestNoMeasuredAnnotationEscapesUnclaimed:
         # exactly the limit `evidence.py` now states rather than the stronger one it claimed.
         #
         # Built with a title on purpose: without `title=`/`number=` no TitleBlock is placed at
-        # all (88 silent annotations, not 89), and the test would then characterise a sheet the
+        # all, and the test would then characterise a sheet the
         # ratchet never sweeps.
         drawing = _drawing(
             "tests/fixtures/nist_ctc_02_asme1_ap203.stp",
@@ -400,4 +401,10 @@ class TestNoMeasuredAnnotationEscapesUnclaimed:
             and not rendered_numbers(drawing.registry.named(n))
         ]
         assert len(silent) > 50, f"too little furniture to characterise: {len(silent)}"
-        assert set(silent) == {"CenterMark", "CenterlineCircle", "Note", "TitleBlock"}, set(silent)
+        assert set(silent) == {
+            "CenterMark",
+            "CenterlineCircle",
+            "Note",
+            "ProjectionSymbol",
+            "TitleBlock",
+        }, set(silent)

{draftwright-0.4.26 → draftwright-0.4.27}/tests/test_issue_1308_machined_leader_analytics.py

@@ -225,7 +225,10 @@ def test_analytical_machined_leaders_preserve_the_occ_measured_drawing(fixture,
     # This golden isolates analytical-vs-OCC leader lowering.  Keep the established principal
     # topology explicit so automatic redundant-view selection is not mistaken for a leader
     # geometry change (and so the recorded page coordinates remain meaningful).
-    drawing = build_drawing(FIXTURES / fixture, _views=("front", "plan", "side"))
+    # Keep the original furniture policy explicit in this leader-geometry comparison.
+    drawing = build_drawing(
+        FIXTURES / fixture, _views=("front", "plan", "side"), projection_symbol=False
+    )
     actual = {}
     for name, annotation in drawing.iter_annotations():
         box = annotation.bounding_box()

{draftwright-0.4.26 → draftwright-0.4.27}/tests/test_issue_1514_projection_safeguard.py

@@ -39,7 +39,7 @@ def test_supported_layout_retains_third_angle_relationships(asymmetric_plate, pr
     front, plan, side = (drawing.view_bounds(v) for v in ("front", "plan", "side"))
     assert plan[1] > front[3]
     assert side[0] > front[2]
-    assert ("projection_symbol" in drawing.annotations()) == (projection == "third")
+    assert drawing.get_annotation("projection_symbol").method == "third"
 
 
 @pytest.mark.parametrize("script", [False, True])

{draftwright-0.4.26 → draftwright-0.4.27}/tests/test_make_drawing.py

@@ -3318,13 +3318,15 @@ def test_make_drawing_module_entrypoint_runs_cli_help():
         capture_output=True,
         env=env,
         text=True,
+        encoding="cp1252",  # match the child's explicit PYTHONIOENCODING
     )
 
     assert result.returncode == 0, f"stdout:\n{result.stdout}\nstderr:\n{result.stderr}"
     assert "Usage:" in result.stdout  # Typer/rich help capitalises (was argparse "usage:")
     assert "step_file" in result.stdout
-    # rich degrades its box-drawing to ASCII on a cp1252 stream, so help stays safe.
-    assert result.stdout.isascii()
+    # Rich degrades unsupported box drawing on cp1252 streams. Long flag names may
+    # use a representable ellipsis, so safe cp1252 output need not be entirely ASCII.
+    assert not any(glyph in result.stdout for glyph in "╭╮╰╯│─")
 
 
 def test_cli_version_reports_installed_version():
@@ -3765,7 +3767,7 @@ def test_build_drawing_auto_dims_false():
     # cylinder's iso is rescaled off sheet scale — the truthful "ISO VIEW (NTS)" note. The
     # note is furniture, not a dimension, so it belongs here (script↔CLI parity); auto_dims
     # still suppresses every *dimension*.
-    assert set(dwg.annotations()) == {"title_block", "note_iso_nts"}
+    assert set(dwg.annotations()) == {"title_block", "note_iso_nts", "projection_symbol"}
 
 
 @pytest.mark.timeout(60)
@@ -11256,9 +11258,9 @@ class TestProjectionSymbol:
     """#769: the ISO 5456-2 projection-method glyph (third/first-angle) in the reserved
     title-block band, from the helpers 0.14.1 ProjectionSymbol primitive."""
 
-    def test_off_by_default(self):
+    def test_third_angle_symbol_by_default(self):
         dwg = build_drawing(Box(80, 60, 20))
-        assert "projection_symbol" not in dwg.annotations()
+        assert dwg.get_annotation("projection_symbol").method == "third"
         assert dwg._analysis.projection is None
 
     def test_third_renders_in_the_title_block_band(self):

draftwright-0.4.27/tests/test_projection_symbol_default.py

@@ -0,0 +1,164 @@
+"""Projection furniture identifies the resolved layout, including implicit defaults."""
+
+from inspect import Parameter, signature
+from pathlib import Path
+
+import pytest
+from build123d import Box, Cylinder, Pos, export_step
+from typer.testing import CliRunner
+
+from draftwright import Drawing, Sheet, build_drawing, make_drawing
+from draftwright.audit import compare_measurements
+from draftwright.sheet_emit import emit_sheet_script, generate_sheet_script
+
+
+@pytest.fixture(scope="module")
+def plate():
+    part = Box(50, 35, 12) - Pos(-13, -8, 0) * Cylinder(3, 20)
+    model = Sheet.from_part(part).model()
+    assert len([f for f in model.features if f.kind == "hole"]) == 1
+    return part, model
+
+
+@pytest.mark.parametrize("projection", [None, "first", "third"])
+@pytest.mark.parametrize("auto_dims", [True, False])
+def test_visibility_changes_only_furniture(plate, projection, auto_dims):
+    part, model = plate
+    options = dict(model=model, projection=projection, auto_dims=auto_dims, page="A3", scale=1)
+    shown = build_drawing(part, **options)
+    hidden = build_drawing(part, projection_symbol=False, **options)
+    symbol = shown.get_annotation("projection_symbol")
+    assert symbol.method == shown.view_plan.convention == (projection or "third")
+    assert symbol.edges()
+    assert set(shown.annotations()) - set(hidden.annotations()) == {"projection_symbol"}
+    assert not set(hidden.annotations()) - set(shown.annotations())
+    assert shown.view_plan == hidden.view_plan
+    assert all(shown.view_bounds(v) == hidden.view_bounds(v) for v in shown.views)
+    if auto_dims:
+        assert compare_measurements(shown, hidden)["status"] == "preserved"
+    assert not [i for i in shown.lint() if i.severity == "error"]
+
+
+@pytest.mark.parametrize("projection", [None, "first", "third"])
+@pytest.mark.parametrize("visible", [True, False])
+def test_generated_script_replays_symbol_policy(plate, projection, visible, tmp_path, monkeypatch):
+    part, model = plate
+    script = emit_sheet_script(
+        model,
+        "part = supplied_part",
+        str(tmp_path / "drawing"),
+        title="PROJECTION",
+        number="PROJECTION-1",
+        projection=projection,
+        projection_symbol=visible,
+        page="A3",
+        scale=1,
+    )
+    assert ("projection_symbol=False" in script) is (not visible)
+    captured = []
+    monkeypatch.setattr(Drawing, "export", lambda self, *a, **kw: captured.append(self))
+    exec(compile(script, "projection-example.py", "exec"), {"supplied_part": part})
+    (drawing,) = captured
+    assert drawing.view_plan.convention == (projection or "third")
+    assert ("projection_symbol" in drawing.annotations()) is visible
+    if visible:
+        assert drawing.get_annotation("projection_symbol").method == drawing.view_plan.convention
+
+
+@pytest.mark.parametrize("entry", [build_drawing, make_drawing, Sheet, generate_sheet_script])
+@pytest.mark.parametrize("invalid", [None, "false", 0])
+def test_invalid_visibility_refused_before_loading_input(entry, invalid, tmp_path):
+    with pytest.raises(ValueError, match="projection_symbol must be a boolean"):
+        entry(str(tmp_path / "missing.step"), projection_symbol=invalid)
+    assert not list(tmp_path.iterdir())
+
+
+@pytest.mark.parametrize("route", ["direct", "step_script", "object_script"])
+@pytest.mark.parametrize("visible", [True, False])
+def test_cli_policy_reaches_rendered_exports(route, visible, plate, tmp_path, monkeypatch):
+    from draftwright import cli
+
+    part, _ = plate
+    source = tmp_path / "plate.step"
+    export_step(part, source)
+    if route == "object_script":
+        source = tmp_path / "plate_source.py"
+        source.write_text("from build123d import Box\npart = Box(50, 35, 12)\n")
+        source = f"{source}:part"
+    drawings = []
+    original_export = Drawing.export
+
+    def capture(self, *args, **kwargs):
+        drawings.append(self)
+        return original_export(self, *args, **kwargs)
+
+    monkeypatch.setattr(Drawing, "export", capture)
+    stem = tmp_path / "result"
+    args = [str(source), "--out", str(stem), "--no-report", "--format", "svg,pdf,dxf"]
+    if not visible:
+        args.append("--no-projection-symbol")
+    if route != "direct":
+        args.append("--script")
+    result = CliRunner().invoke(cli.app, args)
+    assert result.exit_code == 0, result.output
+    if route != "direct":
+        script = stem.with_suffix(".py").read_text()
+        assert ("projection_symbol=False" in script) is (not visible)
+        exec(compile(script, str(stem.with_suffix(".py")), "exec"), {})
+    assert drawings
+    for drawing in drawings:
+        assert drawing.view_plan.convention == "third"
+        assert ("projection_symbol" in drawing.annotations()) is visible
+    for suffix in ("svg", "pdf", "dxf"):
+        assert Path(f"{stem}.{suffix}").stat().st_size > 100
+
+
+@pytest.mark.parametrize(
+    ("entry", "previous"),
+    [
+        (
+            build_drawing,
+            "step_file out title number tolerance drawn_by scale page auto_dims "
+            "detail_view pmi repair assembly model decorations requested authored trace material "
+            "date revision company frame projection zones scale_policy reproducible "
+            "framed_recognition text_position text_orientation _post_build _required_tables "
+            "_views _include_iso _view_constraints _document_input",
+        ),
+        (
+            make_drawing,
+            "step_file out title number tolerance drawn_by scale page auto_dims "
+            "detail_view pmi assembly material date revision company frame projection zones "
+            "scale_policy reproducible framed_recognition text_position text_orientation",
+        ),
+    ],
+)
+def test_existing_positional_arguments_keep_their_bindings(entry, previous):
+    names = previous.split()
+    sentinels = [object() for _ in names]
+    bound = signature(entry).bind(*sentinels)
+    assert bound.arguments == dict(zip(names, sentinels, strict=True))
+    assert signature(entry).parameters["projection_symbol"].kind is Parameter.KEYWORD_ONLY
+
+
+@pytest.mark.parametrize("projection", ["first", "third"])
+@pytest.mark.parametrize("border", [{"frame": True}, {"zones": True}])
+def test_symbol_clears_framed_content_margin(plate, projection, border):
+    part, model = plate
+    drawing = build_drawing(part, model=model, projection=projection, **border)
+    symbol = drawing.get_annotation("projection_symbol")
+    assert symbol.method == projection
+    # The drawn frame is 10 mm in; content reserves another 6 mm inside it.
+    frame = drawing.get_annotation("sheet_frame").bounding_box()
+    box = symbol.bounding_box()
+    assert box.min.X >= frame.min.X + 6 - 0.1
+    assert box.max.X <= frame.max.X - 6 + 0.1
+    assert box.min.Y >= frame.min.Y + 6 - 0.1
+    assert box.max.Y <= frame.max.Y - 6 + 0.1
+    hidden = build_drawing(
+        part, model=model, projection=projection, projection_symbol=False, **border
+    )
+    # A frame can already trigger label/centreline critique. Enabling its projection
+    # symbol must not add a finding; do not pretend the whole fixture was lint-clean.
+    assert [(i.severity, i.code, i.message) for i in drawing.lint()] == [
+        (i.severity, i.code, i.message) for i in hidden.lint()
+    ]

{draftwright-0.4.26 → draftwright-0.4.27}/tests/test_sheet_emit.py

@@ -2359,7 +2359,11 @@ class TestAuthoredSetRoundTrips:
         regenerated = self._run(src, part)["sheet"].build()
 
         def dims(dwg):
-            return {n for n, _ in dwg.iter_annotations() if not n.startswith(("note_", "title"))}
+            return {
+                n
+                for n, _ in dwg.iter_annotations()
+                if not n.startswith(("note_", "title", "projection_symbol"))
+            }
 
         assert dims(regenerated) == dims(direct), (
             "the regenerated script draws a different set from the model it came from"
@@ -2454,7 +2458,11 @@ class TestAuthoredSetRoundTrips:
         regenerated = self._run(src, part)["sheet"].build()
 
         def dims(dwg):
-            return {n for n, _ in dwg.iter_annotations() if not n.startswith(("note_", "title"))}
+            return {
+                n
+                for n, _ in dwg.iter_annotations()
+                if not n.startswith(("note_", "title", "projection_symbol"))
+            }
 
         assert dims(regenerated) == dims(direct) == set(), (
             "an empty authored set draws no generated dimensions, on both paths"
@@ -2861,6 +2869,7 @@ class TestTheDimensionMirror:
 #: purpose: adding one fails loudly, and whoever adds it names the annotation with the real
 #: name in front of them.
 _SCRIPT_FURNITURE = {
+    "projection_symbol": "projection convention; no feature measurement",
     "m_cm": "centre marks — sized off the hole they mark, not a printed value (#875)",
     "centerline_front": "shows where the front view's axis is; no measurement",
     "centerline_plan": "shows where the plan view's axis is; no measurement",
@@ -4315,7 +4324,7 @@ def test_the_object_reference_doc_example_actually_runs(tmp_path):
         assert f"`{view}`" in section, view
 
     names = sorted(ast.literal_eval(out["ANNOT"]))
-    assert len(names) == 11 and "Eleven annotations" in section, names
+    assert len(names) == 12 and "Twelve annotations" in section, names
     for name in names:
         assert f"`{name}`" in section, f"{name} is on the sheet but the doc does not list it"