draftwright 0.4.31__tar.gz → 0.4.33__tar.gz

{draftwright-0.4.31 → draftwright-0.4.33}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: draftwright
-Version: 0.4.31
+Version: 0.4.33
 Summary: Automated technical-drawing generation for build123d
 Project-URL: Homepage, https://github.com/pzfreo/draftwright
 Project-URL: Repository, https://github.com/pzfreo/draftwright
@@ -684,7 +684,7 @@ Classifier: Programming Language :: Python :: 3.14
 Classifier: Topic :: Scientific/Engineering
 Requires-Python: <3.15,>=3.10
 Requires-Dist: antlr4-python3-runtime<4.10
-Requires-Dist: build123d-drafting-helpers>=0.15.4
+Requires-Dist: build123d-drafting-helpers>=0.15.5
 Requires-Dist: build123d<0.11,>=0.9.0; python_full_version < '3.13'
 Requires-Dist: build123d<0.12,>=0.11; python_full_version >= '3.13'
 Requires-Dist: cadquery-ocp-novtk!=7.9.3.1.1; python_full_version >= '3.13'
@@ -692,7 +692,7 @@ Requires-Dist: numpy>=1.24
 Requires-Dist: ocp-gordon<0.3
 Requires-Dist: pillow>=10
 Requires-Dist: pypdfium2>=4
-Requires-Dist: quiddity==0.3.2
+Requires-Dist: quiddity==0.3.3
 Requires-Dist: reportlab>=4.0
 Requires-Dist: rich>=10.11
 Requires-Dist: steputils==0.1

{draftwright-0.4.31 → draftwright-0.4.33}/docs/reference/recogniser-capabilities.md

@@ -70,11 +70,19 @@ issue. The contract test derives package record outputs, converter registries, l
 and emitter branches independently, so copying a new name into the declaration alone cannot make CI
 pass.
 
-### Quiddity 0.3.2 runtime contract
+### Quiddity 0.3.3 runtime contract
 
-The runtime pins the published `quiddity==0.3.2` package, following the 0.2.4 adoption in
+The runtime pins the published `quiddity==0.3.3` package. This release adds material-side
+reconstruction facts to fillets, planar chamfers and angled steps, paired ramps, and open-section
+profiles. Draftwright consumes these records to describe and annotate the recognised part; it does
+not reconstruct a cut solid from them, so the new `side`, `corner`, `opening_direction`,
+`half_width`, and `material_side` fields do not enter the drafting IR. The exact consumer schema
+declarations advance with the provider, and the section-recess adapter accepts document schema 4
+while continuing to reject unknown later schemas.
+
+Version 0.3.2 followed the 0.2.4 adoption in
 [#1486](https://github.com/pzfreo/draftwright/pull/1486) and the migration in
-[#1471](https://github.com/pzfreo/draftwright/issues/1471). Version 0.3.2 advances the public
+[#1471](https://github.com/pzfreo/draftwright/issues/1471). It advanced the public
 `Slot` record to schema 2: `end_radius` distinguishes two semicircular stadium ends and
 `corner_radius` distinguishes rounded rectangular corners. Draftwright consumes
 `end_radius` for through-slot radius requirements; rounded-corner rectangular slots remain a

{draftwright-0.4.31 → draftwright-0.4.33}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "draftwright"
-version = "0.4.31"
+version = "0.4.33"
 description = "Automated technical-drawing generation for build123d"
 readme = "README.md"
 license = { file = "LICENSE" }
@@ -22,8 +22,8 @@ dependencies = [
     # This novtk release lacks OCP.GccEnt on macOS. The exclusion must reach
     # published wheels, not only uv's development resolver constraints.
     "cadquery-ocp-novtk!=7.9.3.1.1 ; python_full_version >= '3.13'",
-    "build123d-drafting-helpers>=0.15.4",
-    "quiddity==0.3.2",
+    "build123d-drafting-helpers>=0.15.5",
+    "quiddity==0.3.3",
     "numpy>=1.24",
     # PNG raster export (SVG→PDF→PNG). Both permissively licensed (Pillow HPND; pypdfium2
     # Apache-2.0 wrapping Google PDFium BSD-3) and ship pre-built wheels with NO native system

{draftwright-0.4.31 → draftwright-0.4.33}/src/draftwright/__init__.py

@@ -57,6 +57,7 @@ _LAZY = {
     # to reach, and the pytest filterwarnings entry naming it paid that on every invocation.
     "SoftDeprecationWarning": "draftwright._warnings",
     "ScaleCompletenessWarning": "draftwright._warnings",
+    "ProjectionGeometryWarning": "draftwright._warnings",
 }
 
 
@@ -93,7 +94,11 @@ _sys.modules[__name__].__class__ = _DraftwrightModule
 
 
 if TYPE_CHECKING:  # static analysers / IDEs — no runtime import, no kernel cost
-    from draftwright._warnings import ScaleCompletenessWarning, SoftDeprecationWarning
+    from draftwright._warnings import (
+        ProjectionGeometryWarning,
+        ScaleCompletenessWarning,
+        SoftDeprecationWarning,
+    )
     from draftwright.builder import ScaleIncompatibilityError, build_drawing, make_drawing
     from draftwright.compose import choose_scale
     from draftwright.document import Document, DocumentBuildError, DocumentResult
@@ -147,6 +152,7 @@ __all__ = [
     "Sheet",
     "ScaleIncompatibilityError",
     "ScaleCompletenessWarning",
+    "ProjectionGeometryWarning",
     "ViewPlanIncomplete",
     "ViewConstraints",
     "ViewSpec",

{draftwright-0.4.31 → draftwright-0.4.33}/src/draftwright/_core.py

@@ -1689,7 +1689,13 @@ def _add_title_block(dwg, a: Analysis):
         bx + cell["max_x"],
         margins.bottom + cell["max_y"],
     )
-    place_annotation(dwg.registry, dwg.items, tb, "title_block")
+    place_annotation(
+        dwg.registry,
+        dwg.items,
+        tb,
+        "title_block",
+        feature=dwg._build.general_tolerance_source,
+    )
 
 
 def _make_sheet_frame(a: Analysis) -> Compound:
@@ -1768,6 +1774,60 @@ def _add_scale_note(dwg, a: Analysis):
     place_annotation(dwg.registry, dwg.items, note, "scale_note")
 
 
+def _add_default_surface_finish(dwg, a: Analysis):
+    """Draw an imported document-wide ISO 1302 requirement without a leader.
+
+    The reserved band above the title block is sheet furniture space.  A document default
+    belongs there: hanging it from model geometry would falsely narrow its scope to one face.
+    """
+    requirement = dwg._build.default_surface_finish_source
+    if requirement is None:
+        return
+
+    from build123d_drafting import SurfaceFinish
+
+    symbol = SurfaceFinish(
+        f"Ra {requirement.ra}",
+        (0, 0),
+        draft=draft_preset(
+            font_size=dwg.draft.font_size,
+            decimal_precision=dwg.draft.decimal_precision,
+            font_path=PLEX_SANS_CONDENSED,
+        ),
+    )
+    box = symbol.bounding_box()
+    width, height = box.max.X - box.min.X, box.max.Y - box.min.Y
+    label = symbol.label_bbox
+    label_x = (label[0] + label[2] - box.min.X - box.max.X) / 2
+    label_y = (label[1] + label[3] - box.min.Y - box.max.Y) / 2
+    # Occupy the left end of the title-block furniture band. Scale and projection method use
+    # its right end, leaving independent document controls readable on every standard sheet.
+    cx = a.PAGE_W - a.TB_W - _title_margins(a).right + width / 2
+    cy = _title_block_top(a) + _TB_FURNITURE_GAP + height / 2
+    symbol = symbol.locate(Location((cx - box.min.X - width / 2, cy - box.min.Y - height / 2, 0)))
+    symbol.pdf_text_relative_specs = (
+        (
+            symbol.label,
+            label_x,
+            label_y,
+            dwg.draft.font_size,
+            PLEX_SANS_CONDENSED,
+            "IBM Plex Sans Condensed",
+            "REGULAR",
+            "center",
+            "middle",
+        ),
+    )
+    symbol.is_default_surface_finish = True
+    place_annotation(
+        dwg.registry,
+        dwg.items,
+        symbol,
+        "default_surface_finish",
+        feature=requirement,
+    )
+
+
 def _add_projection_symbol(dwg, a: Analysis):
     """Place the ISO 5456-2 projection-method glyph (#769) in the reserved title-block band,
     just above the drawn title block (deterministic empty space — the block reserves _TB_H but

{draftwright-0.4.31 → draftwright-0.4.33}/src/draftwright/_warnings.py

@@ -45,3 +45,7 @@ class ScaleCompletenessWarning(UserWarning):
     Callers can filter this policy decision independently of unrelated user warnings while
     still inspecting the complete machine-readable record on ``Drawing.scale_decision``.
     """
+
+
+class ProjectionGeometryWarning(UserWarning):
+    """A projector emitted geometry outside the source shape's possible view envelope."""

{draftwright-0.4.31 → draftwright-0.4.33}/src/draftwright/annotations/_common.py

@@ -3583,18 +3583,39 @@ def place_strip_candidates(
         todo = todo + rejected_total
         solved.extend(placed)
 
-    if len(solved) > 1:
+    if solved:
         assert len({name for name, _dim_obj in solved}) == len(solved), (
             "strip survivor names must be unique before label selection"
         )
         natural_solved = dict(solved)
-        adjusted = prevent_dimension_label_ink(
-            solved,
+        # Local dimensions already participate through strip occupancy, and dimensions
+        # owned by another view retain their independent corridor/repair contract.  The
+        # missing class is non-dimension ink whose semantic owner is another view: an
+        # outboard leader can physically enter this corridor even though the projected
+        # view blocks are disjoint (#1781).
+        committed = []
+        for name, annotation in dwg.iter_annotations():
+            owner = dwg.view_of(name)
+            if (
+                owner is not None
+                and owner != view
+                and not isinstance(annotation, (Dimension, SafeDimension, AngularDimension))
+                and type(annotation).__name__ not in CROSSABLE_TYPES
+            ):
+                committed.append((name, annotation))
+        committed_names = {name for name, _annotation in committed}
+        adjusted_batch = prevent_dimension_label_ink(
+            [*committed, *solved],
             page=(
                 _drawing_bounds(dwg) if hasattr(dwg, "page_w") and hasattr(dwg, "page_h") else None
             ),
-            immutable={name for name, _dim_obj in solved if (anchored or {}).get(name, False)},
+            immutable={
+                *committed_names,
+                *(name for name, _dim_obj in solved if (anchored or {}).get(name, False)),
+            },
+            perpendicular_step=tier + strip.spacing,
         )
+        adjusted = adjusted_batch[len(committed) :]
         # A label shift normally stays inside the dimension's measured span and
         # therefore inside its already-validated footprint.  Keep the validation
         # contract explicit nevertheless: if an unusual helper/style grows the real

{draftwright-0.4.31 → draftwright-0.4.33}/src/draftwright/annotations/from_model.py

@@ -990,8 +990,10 @@ def _obround_radius_candidates(
 
 
 # Corridor-ladder ordering (ADR 2 (was 0009) end state, #346): feature-SIZE dims sit nearer the
-# view (inner run), datum-referenced LOCATION dims stack outward (a single ascending chain
-# by datum distance). Segregating the two runs keeps a slot length from landing mid-ladder.
+# view (inner run), while datum-referenced LOCATION dims form one ascending outer chain.
+# Step-height rungs measure from the common height datum, so they belong to that value-ordered
+# run rather than to producer registration order (#1779). Segregating ordinary feature sizes
+# keeps a slot length or local boss height from landing mid-ladder.
 _SIZE_SUBCHAIN = 0
 _LOC_SUBCHAIN = 1
 _OVERALL_SUBCHAIN = 2
@@ -2704,6 +2706,7 @@ def place_machined_leader_jobs(
     expand_lanes=True,
     region_policy=LeaderRegionPolicy.EXTERIOR,
     source_ids_by_name=None,
+    source_drop_severity="warning",
     priority=0.0,
 ) -> int:
     """Lower every machined callout to the one shared ``FeatureLeaderJob`` path.
@@ -2819,8 +2822,25 @@ def place_machined_leader_jobs(
                         feature,
                     )
 
-        def _build(tip, elbow, _feature, *, _label=label):
-            return Leader(tip=(tip[0], tip[1], 0), elbow=elbow, label=_label, draft=dwg.draft)
+        source_features = tuple(
+            {
+                id(getattr(identity, "feature", None)): getattr(identity, "feature", None)
+                for identity in measurement
+                if getattr(identity, "feature", None) is not None
+            }.values()
+        )
+
+        def _build(
+            tip,
+            elbow,
+            _feature,
+            *,
+            _label=label,
+            _source_features=source_features,
+        ):
+            leader = Leader(tip=(tip[0], tip[1], 0), elbow=elbow, label=_label, draft=dwg.draft)
+            leader.source_features = _source_features
+            return leader
 
         def _fallback_accept(
             candidate,
@@ -2883,8 +2903,13 @@ def place_machined_leader_jobs(
         ):
             validation = reason == "geometry_validation"
             detail = "rendered geometry validation failed" if validation else "no clear room"
+            severity = (
+                ("error" if _source_ids else "warning")
+                if source_drop_severity == "source"
+                else source_drop_severity
+            )
             ctx.record_issue(
-                "warning",
+                severity,
                 drop_code,
                 f"{noun} callout {_label} not placed ({detail})",
                 measurement=_measurement,
@@ -2934,7 +2959,7 @@ def place_machined_leader_jobs(
                 fallback_accept=_fallback_accept,
                 interior_label_clear=interior_label_clear,
                 allow_policy_b_fixed=True,
-                on_drop=_on_drop if source_ids else None,
+                on_drop=(_on_drop if source_ids or source_drop_severity == "source" else None),
             )
         )
 
@@ -2996,6 +3021,7 @@ def render_chamfers(dwg, plan, a, *, ctx, only=None) -> int:
         collapse.setdefault(spec, []).append((g, pd))
 
     jobs = []
+    source_ids_by_name = {}
     for gi, (_spec, members) in enumerate(sorted(collapse.items())):
         if only is not None:
             # Filter after enumerating the full collapse so a surviving group keeps the same
@@ -3023,9 +3049,17 @@ def render_chamfers(dwg, plan, a, *, ctx, only=None) -> int:
         label = _chamfer_label(representative_pd.value_text, representative_pd.value, ch)
         if len(members) > 1:
             label = f"{len(members)}× {label}"
+        name = f"m_chamfer_{axis}{gi}"
+        source_ids_by_name[name] = tuple(
+            dict.fromkeys(
+                source_id
+                for member, _pd in members
+                for source_id in getattr(member.facts, "source_ids", ())
+            )
+        )
         jobs.append(
             (
-                f"m_chamfer_{axis}{gi}",
+                name,
                 view,
                 vb,
                 label + _tol_suffix(representative_pd.tolerance, draft),
@@ -3050,6 +3084,8 @@ def render_chamfers(dwg, plan, a, *, ctx, only=None) -> int:
         ctx=ctx,
         joint=True,
         region_policy=LeaderRegionPolicy.AUTO,
+        source_ids_by_name=source_ids_by_name,
+        source_drop_severity="source",
     )
 
 
@@ -3695,6 +3731,37 @@ def _flat_label(across_text, sfx="") -> str:
     return f"{across_text}{sfx} A/F"
 
 
+def _flat_definition_count(members) -> int:
+    """Count repeated flat definitions without counting one centred opposed set as many.
+
+    ``axis_line`` and ``stock_span`` identify the provider's stock region, but interrupted
+    housing geometry can give several independent flat supports that share both.  A true
+    double-D or hex remains one definition because every support has its reflection through
+    that stock axis; a non-centred family retains one instance per source support.
+    """
+
+    if len(members) <= 1:
+        return 1
+    facts = members[0][0].facts
+    transverse = tuple(index for index, axis in enumerate("xyz") if axis != facts.axis)
+    centre = tuple(float(value) for value in facts.axis_line)
+    points = tuple(
+        tuple(float(member.facts.frame.origin[index]) for index in transverse)
+        for member, _dimension in members
+    )
+    centred = all(
+        any(
+            all(
+                abs(candidate[index] - (2.0 * centre[index] - point[index])) <= 0.01
+                for index in range(2)
+            )
+            for candidate in points
+        )
+        for point in points
+    )
+    return 1 if centred else len(members)
+
+
 def render_flats(dwg, plan, a, *, ctx, only=None) -> int:
     """Render physical stock definitions, counting identical ones without counting faces.
 
@@ -3757,8 +3824,9 @@ def render_flats(dwg, plan, a, *, ctx, only=None) -> int:
         vb = dwg.view_bounds(view)
         if vb is None:
             continue
-        if len(stocks) > 1:
-            label = f"{len(stocks)}× {label}"
+        definition_count = sum(_flat_definition_count(members) for _, members, _ in stocks)
+        if definition_count > 1:
+            label = f"{definition_count}× {label}"
         candidates = _flat_candidates(
             dwg,
             view,
@@ -6807,8 +6875,10 @@ def _render_height_ladder_in_view(dwg, plan, frame, *, ctx, detail_view, view) -
     # compiler's own number instead of re-deriving a convention from the rendered string,
     # which is the pattern ADR 4 (was 0016 Amendment 1) exists to stop.
     chain: list = []
+    order_values: dict[str, float] = {}
     if rung_set is not None and rung_set.representative:
         (rep,) = rungs
+        order_values["dim_step_typ"] = rep.value
         chain.append(
             (
                 "dim_step_typ",
@@ -6899,6 +6969,7 @@ def _render_height_ladder_in_view(dwg, plan, frame, *, ctx, detail_view, view) -
             if has_shoulders and rung.value * frame.scale < _MIN_STEP_DIM_MM:
                 short_rungs.append(rung)
                 continue
+            order_values[f"dim_step_{col}"] = rung.value
             chain.append(
                 (
                     f"dim_step_{col}",
@@ -7069,11 +7140,13 @@ def _render_height_ladder_in_view(dwg, plan, frame, *, ctx, detail_view, view) -
                 build=_build,
                 # Steps stack inner→outer in chain order; the overall height rides the
                 # OVERALL subchain so it lands outermost by construction (as the envelope
-                # dims do), replacing the old carve's outermost=True.
+                # dims do). Ordinary rungs join the same value-ordered baseline run as
+                # off-axis-hole heights (#1779), outside the inner boss-size run, instead
+                # of retaining producer registration order.
                 order=(
                     (_OVERALL_SUBCHAIN, 0, name)
                     if name == "dim_height"
-                    else (_SIZE_SUBCHAIN, k, name)
+                    else (_LOC_SUBCHAIN, order_values[name], name)
                 ),
                 on_place=lambda nm: None,
                 on_drop=_drop,
@@ -8872,6 +8945,26 @@ def _pmi_source_ids(item) -> tuple[str, ...]:
     return tuple(dict.fromkeys(((singular,) if singular else ()) + plural))
 
 
+def render_document_notes(dwg, model) -> int:
+    """Place source-proven drawing-wide requirements in one solver-owned notes block."""
+    notes = [feature for feature in model.features if feature.kind == "document_note"]
+    if not notes:
+        return 0
+    rows = [("GENERAL NOTES",)] + [
+        (f"{index}  {_font_safe_text(note.text)}",) for index, note in enumerate(notes, 1)
+    ]
+    placed = dwg.add_table(
+        rows,
+        prefer="tr",
+        name="general_notes",
+        _source_ids=tuple(source_id for note in notes for source_id in _pmi_source_ids(note)),
+        _features=tuple(notes),
+        _drop_code="pmi_dropped",
+        _drop_severity="error",
+    )
+    return len(notes) if placed is not None else 0
+
+
 def _gdt_glyph(item, draft):
     """Build the ISO 1101/5459/1302 glyph sketch for one GD&T IR item at the origin
     (the :class:`Leader` repositions it). A fresh sketch per call — the leader translate

{draftwright-0.4.31 → draftwright-0.4.33}/src/draftwright/annotations/orchestrator.py

@@ -24,6 +24,7 @@ from draftwright._core import (
     Analysis,
     DetailRequest,
     HoleRef,
+    _add_default_surface_finish,
     _add_projection_symbol,
     _add_scale_note,
     _add_sheet_frame,
@@ -124,16 +125,50 @@ from draftwright.model.planner import annotation_groups, internal_section_rows
 from draftwright.progress import stage
 from draftwright.registry import MeasurementCell
 from draftwright.repair import reconcile_witness_labels
-from draftwright.view_plan import ViewConstraints
+from draftwright.view_plan import (
+    DERIVED_VIEW_IDENTIFIERS,
+    DerivedViewIdentifierPool,
+    ViewConstraints,
+    derived_view_identifier,
+)
+
+
+def _derived_view_identifier_pool(a) -> DerivedViewIdentifierPool:
+    """Reserve every authored section/detail identity before any derived view renders."""
+
+    constraints = a.view_constraints
+    if not isinstance(constraints, ViewConstraints):
+        return DerivedViewIdentifierPool(candidates=DERIVED_VIEW_IDENTIFIERS)
+    requested = (
+        constraints.derived
+        if constraints.derived_source == "authored"
+        else constraints.added_derived
+    )
+    identifiers = []
+    for item in requested:
+        identifier = derived_view_identifier(item.spec.kind, item.spec.name)
+        if identifier is None:
+            raise ValueError(
+                f"derived view {item.spec.name!r} from {item.source} has no canonical identifier"
+            )
+        identifiers.append(identifier)
+    return DerivedViewIdentifierPool(identifiers, candidates=DERIVED_VIEW_IDENTIFIERS)
 
 
-def _planned_sections(a, model, feature_keys) -> tuple[SectionPlan, ...]:
+def _planned_sections(a, model, feature_keys, *, identifiers) -> tuple[SectionPlan, ...]:
     """Combine the automatic section candidate with ADR 2 (was 0018) authored/add requests."""
 
     constraints = a.view_constraints
     if not isinstance(constraints, ViewConstraints):
         automatic = plan_sections(model, feature_keys)
-        return (automatic,) if automatic is not None else ()
+        if automatic is None:
+            return ()
+        label = identifiers.allocate()
+        if label is None:
+            raise ValueError(
+                "automatic section cannot be named: derived-view identifiers exhausted"
+            )
+        return (replace(automatic, label=label),)
 
     requested = (
         constraints.derived
@@ -157,8 +192,9 @@ def _planned_sections(a, model, feature_keys) -> tuple[SectionPlan, ...]:
                 f"section {item.spec.name!r} from {item.source} cuts at y={cut_y:g}, outside "
                 f"the part interior ({a.bb.min.Y:g}, {a.bb.max.Y:g})"
             )
-        slug = item.spec.name.removeprefix("section_")
-        label = slug[0].upper() if len(set(slug)) == 1 else slug.upper()
+        label = derived_view_identifier(item.spec.kind, item.spec.name)
+        if label is None:
+            raise ValueError(f"section {item.spec.name!r} has no canonical identifier")
         plans.append(
             SectionPlan(
                 cut_y,
@@ -172,12 +208,13 @@ def _planned_sections(a, model, feature_keys) -> tuple[SectionPlan, ...]:
 
     if constraints.derived_source != "authored":
         automatic = plan_sections(model, feature_keys)
+        if automatic is not None and any(plan.cut_y == automatic.cut_y for plan in plans):
+            automatic = None
         if automatic is not None:
-            used = {plan.label for plan in plans}
-            auto_label = next((letter for letter in "ABCDEFGH" if letter not in used), None)
+            auto_label = identifiers.allocate()
             if auto_label is None:
                 raise ValueError(
-                    "automatic section cannot be named: authored labels A-H are exhausted"
+                    "automatic section cannot be named: derived-view identifiers exhausted"
                 )
             plans.insert(0, replace(automatic, label=auto_label))
     return tuple(plans)
@@ -201,7 +238,9 @@ def _queue_authored_details(dwg, a, ctx, plan) -> None:
         if not (isinstance(target, tuple) and len(target) == 2 and target[0] == "feature"):
             raise ValueError(f"detail {item.spec.name!r} has no semantic feature target")
         feature = target[1]
-        label = item.spec.name.removeprefix("detail_").upper()
+        label = derived_view_identifier(item.spec.kind, item.spec.name)
+        if label is None:
+            raise ValueError(f"detail {item.spec.name!r} has no canonical identifier")
         factor = item.spec.scale_factor or 2.0
         if queue_step_detail(
             dwg,
@@ -625,7 +664,8 @@ def _auto_annotate(dwg, a: Analysis, *, detail_view: bool = False):
     # Decide the section trigger + cut-plane row now (pure function of _model/
     # feature_keys, no placement dependency); the "reserve_section" stage reserves
     # its row and the "section" stage renders it.
-    _sections = _planned_sections(a, _model, feature_keys)
+    _derived_identifiers = _derived_view_identifier_pool(a)
+    _sections = _planned_sections(a, _model, feature_keys, identifiers=_derived_identifiers)
     ctx.dense_internal_section = any(section.internal_detail for section in _sections)
 
     # ── the stage thunks, run in _PASS_SEQUENCE order (#699 slice b) ─────────
@@ -901,7 +941,8 @@ def _auto_annotate(dwg, a: Analysis, *, detail_view: bool = False):
         # still render after it and avoid the section view.
         if _sections:
             for section in _sections:
-                _add_section_view(dwg, a, section, ctx=ctx)
+                if not _add_section_view(dwg, a, section, ctx=ctx):
+                    _derived_identifiers.release(section.label)
         else:
             # Recorded, not left at the initial `not_evaluated`: the planner DID run and
             # found no counterbore/spotface/blind Z-hole, which is a different fact from
@@ -916,7 +957,7 @@ def _auto_annotate(dwg, a: Analysis, *, detail_view: bool = False):
         # crowded turned heads alike — through the one generic detailer, now that all
         # views and main-view annotations are placed (so the detail avoids them).
         _queue_authored_details(dwg, a, ctx, _compiled)
-        _resolve_details(dwg, a, ctx=ctx)
+        _resolve_details(dwg, a, ctx=ctx, identifiers=_derived_identifiers)
 
     def _s_title_block():
         _add_title_block(dwg, a)
@@ -936,6 +977,7 @@ def _auto_annotate(dwg, a: Analysis, *, detail_view: bool = False):
         # ISO 5456-2 projection-method glyph (#769) in the reserved title-block band.
         _add_projection_symbol(dwg, a)
         _add_scale_note(dwg, a)
+        _add_default_surface_finish(dwg, a)
 
     def _s_tabulate():
         # Escalate to a hole table when the plan view is too dense to dimension