b3dkit 0.4.0__tar.gz → 0.5.2__tar.gz

b3dkit-0.5.2/.coveragerc

@@ -0,0 +1,20 @@
+[run]
+source = src/b3dkit
+# `*test*` also matched any source path containing "test", so it was scoped to
+# the test directory instead.
+omit =
+    tests/*
+
+[report]
+show_missing = True
+# CONTRIBUTING.md declares a 90% floor with no ratchet. Enforced here so the
+# number is a gate rather than a suggestion.
+fail_under = 90
+exclude_lines =
+    pragma: no cover
+    # The __main__ blocks are demos, not library code. They were previously
+    # executed by ten tests that asserted nothing, purely so these lines would
+    # count toward coverage; deleting those tests is what this excludes for.
+    if __name__ == .__main__.:
+    # Raised only when the package is not installed, e.g. run from a source tree.
+    except PackageNotFoundError:

{b3dkit-0.4.0 → b3dkit-0.5.2}/CHANGELOG.md

@@ -7,6 +7,70 @@ ship in minor releases.
 
 ## [Unreleased]
 
+### Fixed
+
+- **build123d 0.12 and 0.13 support.** `TeardropBoltCutSinkhole`,
+  `BoltCutSinkhole`, `SquareNutSinkhole`, `TwistSnapSocket` and the
+  `high_top_slide_box` parts built the wrong geometry on build123d 0.12+, which
+  places geometry inside located builders differently. Every public object now
+  builds identically on 0.11, 0.12 and 0.13; 0.11 geometry is unchanged.
+- **`Teardrop`'s `rotation` is a float**, defaulting to `0`. The old `(0, 0)`
+  default raised `TypeError` on build123d 0.12+.
+
+## [0.5.2] - 2026-10-03
+
+### Changed
+
+- **`slide_box` sliding edges are filleted** by `wall_thickness * 0.025` on
+  both the slot and the lid.
+
+## [0.5.1] - 2026-10-01
+
+### Changed
+
+- **`slide_box` lids no longer intersect the box.** The lid was
+  `wall_thickness + tolerance` thick in a `wall_thickness` slot. `tolerance` is
+  now the total gap on every mating face, split evenly: the slot grows by half
+  and the lid shrinks by half, so the lid is `wall_thickness - tolerance / 2`
+  thick and its top stays flush. Box, lid and `slide_lid` geometry all change.
+- **`slide_box` divots now line up.** The box detents and lid bumps were placed
+  from different tolerances and sat up to `tolerance` apart.
+
+## [0.5.0] - 2026-09-09
+
+### Changed
+
+- **`NutCut` now sits on the origin rather than straddling it.** The recess ran
+  from `-head_depth` to `0` while the shaft began at `+head_depth`, so the two
+  never met: the cutter came back as two solids with a `head_depth` gap, and
+  subtracting it left a membrane the bolt could not pass through. It is now one
+  solid spanning `0` to `head_depth + shaft_length`.
+
+  **This moves the cutter.** It previously had to be positioned *below* a
+  surface to work; it can now be placed *on* one. Anything calling `NutCut` and
+  compensating for the old placement needs that compensation removed.
+
+- **`dovetail_subpart` no longer adds each click-fit divot twice.** One copy
+  landed outside the part as a free-floating sliver. Only geometry using
+  `click_fit_radius` is affected; the tail's fused solid is unchanged and the
+  socket's moves by 0.02%.
+
+- **A positive `vertical_offset` no longer cuts the dovetail socket in two.**
+  Tails, `vertical_offset=0` and negative offsets are all unchanged.
+
+### Known issues
+
+Found while adding solid-count assertions, all pre-existing and all invisible to
+the `is_valid` checks that used to stand in for them:
+
+- `TwistSnapConnector` returns five solids: a base and four snapfits that meet
+  only on a shared plane and never fuse.
+- `HexWall` can return partial hexagons clipped at the bounds as separate
+  slivers, depending on how the grid divides the given width.
+- `SNUGTAIL` splits its own tail into disconnected fins on parts wider than
+  about 30mm. Pinned by `TestSnugtailWidthLimit`.
+
+
 ## [0.4.0] - 2026-09-08
 
 ### Removed
@@ -111,6 +175,9 @@ the Python 3.11 floor.
 See the git history. b3dkit began as `fb-library`; the rename accompanied a
 rewrite that reworked names and usage to feel closer to native build123d.
 
-[Unreleased]: https://github.com/x0pherl/b3dkit/compare/v0.4.0...HEAD
+[Unreleased]: https://github.com/x0pherl/b3dkit/compare/v0.5.2...HEAD
+[0.5.2]: https://github.com/x0pherl/b3dkit/releases/tag/v0.5.2
+[0.5.1]: https://github.com/x0pherl/b3dkit/releases/tag/v0.5.1
+[0.5.0]: https://github.com/x0pherl/b3dkit/releases/tag/v0.5.0
 [0.4.0]: https://github.com/x0pherl/b3dkit/releases/tag/v0.4.0
 [0.3.2]: https://github.com/x0pherl/b3dkit/releases/tag/v0.3.2

{b3dkit-0.4.0 → b3dkit-0.5.2}/CONTRIBUTING.md

@@ -84,6 +84,11 @@ release, raise the floor in `pyproject.toml` in the same change. b3dkit 0.1.5
 shipped code that could not run against its own advertised minimum, which is
 what that CI job exists to prevent.
 
+Inside a `BuildPart` that has a location, do not open another builder or
+sketch, and give operations geometry from the builder (`builder.faces()`), not
+from `builder.part`. build123d 0.11 and 0.12+ place geometry differently in
+both cases.
+
 Do not add an upper bound. Caps propagate into every downstream resolution and
 convert a runtime failure into an install failure for everyone.
 

{b3dkit-0.4.0 → b3dkit-0.5.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: b3dkit
-Version: 0.4.0
+Version: 0.5.2
 Summary: build123d libraries and utilities
 Project-URL: Homepage, https://github.com/x0pherl/b3dkit
 Project-URL: Issues, https://github.com/x0pherl/b3dkit/issues

{b3dkit-0.4.0 → b3dkit-0.5.2}/docs/slide_box.md

@@ -45,7 +45,9 @@ The `x_straighten_distance` parameter controls how much of the sliding mechanism
 ## Design Considerations
 
 ### Tolerances
-The `tolerance` parameter controls the fit between the lid and base:
+The `tolerance` parameter is the total gap between the lid and base on every mating face, split evenly between the two: the slot grows by half of it and the lid shrinks by half of it. The lid is `wall_thickness - tolerance / 2` thick, with its top flush with the box.
+
+Typical values:
 - **0.1mm**: Very tight fit, may require force to operate
 - **0.15mm**: Standard fit (default) - smooth operation with minimal play
 - **0.2-0.25mm**: Loose fit for rough printing or materials that swell

{b3dkit-0.4.0 → b3dkit-0.5.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "b3dkit"
-version = "0.4.0"
+version = "0.5.2"
 authors = [
   { name="x0pherl"},
 ]
@@ -63,8 +63,17 @@ build-backend = "hatchling.build"
 [tool.pytest.ini_options]
 pythonpath = ["src"]
 addopts = "--strict-markers"
-markers = [
-    "manual: marks tests that can only be executed manually",
+filterwarnings = [
+    # b3dkit's own warnings are failures; we control those.
+    "error",
+    # Everything below is raised inside the dependency stack, not by b3dkit.
+    # ezdxf, reached through build123d, calls pyparsing's pre-snake_case API.
+    "ignore::DeprecationWarning:ezdxf.*",
+    "ignore::DeprecationWarning:pyparsing.*",
+    "default::DeprecationWarning:build123d.*",
+    "default::PendingDeprecationWarning:build123d.*",
+    # build123d 0.13 renamed add() to insert(); 0.11 has no insert().
+    "ignore:add\\(\\) is deprecated:DeprecationWarning",
 ]
 
 [tool.black]

{b3dkit-0.4.0 → b3dkit-0.5.2}/src/b3dkit/antichamfer.py

@@ -11,7 +11,6 @@ from build123d import (
     Face,
     Iterable,
     Location,
-    Mode,
     Part,
     add,
     extrude,
@@ -78,18 +77,19 @@ def anti_chamfer(
     if length == 0 or length2 == 0:
         return target
 
+    taper = -degrees(atan(length2 / length))
+    if context is not None:
+        # extrude straight into the caller's builder: a builder opened here would
+        # nest inside it, and build123d 0.12 changed where nested builders place
+        # their geometry
+        for f in faces_list:
+            extrude(f.offset(-length), amount=length, taper=taper)
+        return Part(Compound([context.part]).wrapped)
+
     with BuildPart() as new_part:
         add(target)
         for f in faces_list:
-            extrude(
-                f.offset(-length),
-                amount=length,
-                taper=-degrees(atan(length2 / length)),
-            )
-    if context is not None:
-        context._add_to_context(
-            Part(Compound([new_part.part]).wrapped), mode=Mode.REPLACE
-        )
+            extrude(f.offset(-length), amount=length, taper=taper)
     return Part(Compound([new_part.part]).wrapped)
 
 

{b3dkit-0.4.0 → b3dkit-0.5.2}/src/b3dkit/basic_shapes.py

@@ -313,7 +313,7 @@ class Teardrop(BaseSketchObject):
         self,
         radius: float,
         peak_distance: float,
-        rotation: RotationLike = (0, 0),
+        rotation: float = 0,
         align: None | Align | tuple[Align, Align] = None,
         mode: Mode = Mode.ADD,
     ):

{b3dkit-0.4.0 → b3dkit-0.5.2}/src/b3dkit/bolt_fittings.py

@@ -92,7 +92,7 @@ class TeardropBoltCutSinkhole(BasePartObject):
                     amount=extension_distance,
                 )
             anti_chamfer(
-                sinkhole.part.faces().sort_by(Axis.Z)[-1],
+                sinkhole.faces().sort_by(Axis.Z)[-1],
                 chamfer_radius,
             )
         super().__init__(
@@ -205,7 +205,7 @@ class SquareNutSinkhole(BasePartObject):
                     nut_length,
                     align=(Align.CENTER, Align.MIN, Align.CENTER),
                 )
-                extrude(nut.part.faces().sort_by(Axis.Z)[-1], amount=nut_depth)
+                extrude(nut.faces().sort_by(Axis.Z)[-1], amount=nut_depth)
             if bolt_extension > 0:
                 with BuildPart(Location((0, bolt_depth + nut_height, 0))) as nut:
                     TeardropCylinder(
@@ -251,13 +251,13 @@ class NutCut(BasePartObject):
         context: BuildPart = BuildPart._get_context()
         validate_inputs(context, self)
 
-        with BuildPart(Location((0, 0, head_depth))) as cut:
+        with BuildPart() as cut:
             with BuildSketch():
                 RegularPolygon(radius=head_radius, side_count=6)
-            extrude(amount=-head_depth)
+            extrude(amount=head_depth)
             Cylinder(
                 radius=shaft_radius,
-                height=shaft_length,
+                height=shaft_length + head_depth,
                 align=(Align.CENTER, Align.CENTER, Align.MIN),
             )
 

{b3dkit-0.4.0 → b3dkit-0.5.2}/src/b3dkit/dovetail.py

@@ -528,18 +528,10 @@ def _traditional_subpart_divots(
             ),
             mode=topmode,
         ):
-            add(
-                Divot(
-                    click_fit_radius,
-                    positive=topmode == Mode.ADD,
-                    extend_base=True,
-                )
-                .rotate(
-                    Axis.X,
-                    (90 * (-1 if vertical_offset < 0 else 1))
-                    + adjusted_top_divot_angle,
-                )
-                .rotate(Axis.Y, cut_angle),
+            Divot(
+                click_fit_radius,
+                positive=topmode == Mode.ADD,
+                extend_base=True,
             )
         #####################################
         # Bottom divots
@@ -562,29 +554,20 @@ def _traditional_subpart_divots(
             ),
             mode=bottommode,
         ):
-            add(
-                Divot(
-                    click_fit_radius,
-                    positive=bottommode == Mode.ADD,
-                    extend_base=True,
-                )
-                .rotate(
-                    Axis.X,
-                    (90 * (1 if vertical_offset < 0 else -1)) + scarf_angle,
-                )
-                .rotate(Axis.Y, cut_angle),
+            Divot(
+                click_fit_radius,
+                positive=bottommode == Mode.ADD,
+                extend_base=True,
             )
         with BuildPart(
             Location((end_side.x, end_side.y, click_fit_radius * 2)),
             mode=bottommode,
         ):
-            add(
-                Divot(click_fit_radius, positive=True, extend_base=True)
-                .rotate(
-                    Axis.X,
-                    (90 * (1 if vertical_offset < 0 else -1)) + scarf_angle,
-                )
-                .rotate(Axis.Y, cut_angle),
+            Divot(
+                click_fit_radius,
+                # was hardcoded True while the sibling divot above derives it
+                positive=bottommode == Mode.ADD,
+                extend_base=True,
             )
 
     return divotedpart.part
@@ -954,7 +937,16 @@ def dovetail_subpart(
         current_floor = (
             part.bounding_box().min.Z
             if vertical_offset <= 0
-            else part.bounding_box().min.Z + abs(vertical_offset) + vertical_tolerance
+            # Must match the straightened slab's top_z above, which uses
+            # vertical_tolerance_adjustment. Using the raw vertical_tolerance
+            # here agreed for the tail, where the adjustment is +tolerance, and
+            # disagreed for the socket, where it is -tolerance: the two slabs
+            # then missed each other by 2 * vertical_tolerance, cutting a slot
+            # clean through the socket and leaving it in two pieces. The
+            # vertical_offset < 0 branch below already uses the adjustment.
+            else part.bounding_box().min.Z
+            + abs(vertical_offset)
+            + vertical_tolerance_adjustment
         )
         add(
             _subpart_slab(

{b3dkit-0.4.0 → b3dkit-0.5.2}/src/b3dkit/high_top_slide_box.py

@@ -83,10 +83,11 @@ def _slide_top_rail_cut(
         -(part_depth - wall_thickness) / 2,
     )
 
-    with BuildPart(
-        Location((0, wall_thickness / 2, 0)),
-        mode=Mode.SUBTRACT,
-    ) as rail_cut:
+    # every child below is a nested builder or sketch, so this builder takes no
+    # location of its own: build123d 0.11 ignores a parent's location in nested
+    # builders and 0.12 composes it, so only an unlocated parent means the same
+    # thing to both
+    with BuildPart() as rail_cut:
         with BuildSketch(Location((0, wall_thickness / 2))):
             with BuildLine():
 
@@ -224,13 +225,15 @@ def _high_top_slide_box_top(
             )
         ):
             add(base_part)
-            with BuildPart(mode=Mode.INTERSECT):
-                Box(
-                    part_max_dimension,
-                    part_max_dimension,
-                    part_max_dimension,
-                    align=(Align.CENTER, Align.CENTER, Align.MIN),
-                )
+        # outside the located builder above, so it sits at z=0 on every
+        # build123d version
+        Box(
+            part_max_dimension,
+            part_max_dimension,
+            part_max_dimension,
+            align=(Align.CENTER, Align.CENTER, Align.MIN),
+            mode=Mode.INTERSECT,
+        )
         with BuildPart(
             Location((0, wall_thickness / 2, rail_height)),
             mode=Mode.SUBTRACT,
@@ -389,6 +392,19 @@ def high_top_slide_box_base(
     part_depth = base_part.bounding_box().size.Y
     part_height = base_part.bounding_box().size.Z
 
+    # built before any builder opens: inside one it would be a nested builder,
+    # and build123d 0.12 changed where nested builders place their geometry
+    top_cut = _high_top_slide_box_top(
+        base_part,
+        top_height,
+        rail_height,
+        wall_thickness,
+        rail_angle,
+        divot_radius,
+        tolerance,
+        cut_template=True,
+    )
+
     with BuildPart() as boxbottom:
         add(base_part)
         with BuildPart(
@@ -423,18 +439,7 @@ def high_top_slide_box_base(
             ),
             mode=Mode.SUBTRACT,
         ):
-            add(
-                _high_top_slide_box_top(
-                    base_part,
-                    top_height,
-                    rail_height,
-                    wall_thickness,
-                    rail_angle,
-                    divot_radius,
-                    tolerance,
-                    cut_template=True,
-                )
-            )
+            add(top_cut)
         with BuildPart(
             Location(
                 (

{b3dkit-0.4.0 → b3dkit-0.5.2}/src/b3dkit/slide_box.py

@@ -7,7 +7,7 @@ They are exempt from the compatibility promise in ``__all__`` until that is
 settled.
 """
 
-from math import radians, tan
+from math import cos, radians, tan
 
 from build123d import (
     Align,
@@ -43,6 +43,13 @@ __all__ = [
 ]
 
 
+def _tapered_inset(clearance: float) -> float:
+    """
+    The horizontal shift that moves a tapered face ``clearance`` along its normal.
+    """
+    return clearance / cos(radians(_SLIDER_TAPER_ANGLE))
+
+
 def _divot_spacing(
     sketch_width: float,
     wall_thickness: float,
@@ -59,14 +66,18 @@ def _divot_spacing(
     tapered face. A grazing intersection leaves needle-thin slivers behind the
     boolean, so pull the pair far enough inboard that a divot only ever meets the
     flat underside.
+
+    The spacing is measured against the lid's tapered face, the narrower of the
+    two, so the box and lid templates place their divots identically.
     """
     half_width = sketch_width / 2
+    lid_clearance = -abs(tolerance) / 2
     # depth below the top of the template at the widest point of the divot
-    widest_depth = max(wall_thickness - divot_radius, 0)
+    widest_depth = max(wall_thickness + lid_clearance - divot_radius, 0)
     tapered_face = (
         half_width
         - wall_thickness
-        - abs(tolerance)
+        + _tapered_inset(lid_clearance)
         + widest_depth * tan(radians(_SLIDER_TAPER_ANGLE))
     )
     return (
@@ -89,13 +100,21 @@ def _slider_template(
 ) -> Part:
     """
     Create a slider part based on a sketch.
+
+    ``tolerance`` is the total gap between lid and box; the cut template grows
+    by half of it and the lid template shrinks by half of it. The top of the
+    template stays at the top of the sketch either way.
     """
+    clearance = abs(tolerance) / 2 * (1 if cut_template else -1)
     with BuildPart() as slider_part:
         with BuildSketch() as top_sketch:
-            offset(sketch, amount=-abs(tolerance) - (abs(wall_thickness)))
+            offset(
+                sketch,
+                amount=-abs(wall_thickness) + _tapered_inset(clearance),
+            )
         extrude(
             top_sketch.sketch,
-            amount=-wall_thickness - abs(tolerance),
+            amount=-wall_thickness - clearance,
             taper=-_SLIDER_TAPER_ANGLE,
         )
         cross_section = section(
@@ -111,15 +130,20 @@ def _slider_template(
                 cross_section, amount=x_straighten_distance * 2 + wall_thickness * 2
             )
         )
+        fillet(
+            slider_part.edges() - slider_part.faces().filter_by(Plane.XY).edges(),
+            wall_thickness * 0.025,
+        )
+        fillet(slider_part.faces().sort_by(Axis.Z)[0].edges(), wall_thickness * 0.025)
+
         if divot_radius > 0:
             # a divot wider than half the wall would overhang the open front of
-            # the template; keep it far enough back to meet only the underside
-            divot_y = max(
-                sketch.bounding_box().min.Y + wall_thickness / 2,
-                slider_part.part.bounding_box().min.Y + divot_radius + divot_radius / 5,
+            # the part; keep it far enough back to meet only the underside
+            divot_y = sketch.bounding_box().min.Y + max(
+                wall_thickness / 2, divot_radius + divot_radius / 5
             )
             with BuildPart(
-                Location((0, divot_y, -wall_thickness), (180, 0, 0)),
+                Location((0, divot_y, -wall_thickness - clearance), (180, 0, 0)),
                 mode=Mode.ADD,
             ):
                 with GridLocations(
@@ -161,7 +185,7 @@ def slide_lid(
     args:
         - part: the part defining the box's outer dimensions
         - wall_thickness: the thickness of the box walls
-        - tolerance: the clearance between the lid and the base; 0.1 is a tight
+        - tolerance: the total gap between the lid and the base; 0.1 is a tight
             fit, 0.15 the default, 0.2 and above loose
         - top_offset: how far below the top of the part the sliding section sits,
             for parts with features at the top worth preserving
@@ -247,7 +271,8 @@ def slide_box(
         - thumb_radius: the radius of the thumb-grip cutout; 0 disables it
         - x_straighten_distance: how much of the slider stays straight at the
             edges, which stops wide lids binding at the corners
-        - tolerance: the clearance between the lid and the base
+        - tolerance: the total gap between the lid and the base, split evenly
+            between the slot and the lid
         - divot_radius: the radius of the click-fit positioning divots; 0
             disables them
 
@@ -260,7 +285,7 @@ def slide_box(
     lid_cut_template = _slider_template(
         cross_section,
         wall_thickness,
-        tolerance=0,
+        tolerance=tolerance,
         top_offset=top_offset,
         x_straighten_distance=x_straighten_distance,
         divot_radius=divot_radius,
@@ -303,11 +328,20 @@ def slide_box(
 
 
 if __name__ == "__main__":
-    from ocp_vscode import Camera, show
+    from ocp_vscode import show
 
     with BuildPart() as base_box:
         Box(20, 44, 14, align=(Align.CENTER, Align.CENTER, Align.MIN))
         fillet(base_box.part.edges().filter_by(Axis.Z), radius=1.5)
 
-    sb = slide_box(base_box.part, wall_thickness=2, thumb_radius=3.5, divot_radius=0.5)
-    show(sb, reset_camera=Camera.KEEP)
+    sb = slide_box(
+        base_box.part,
+        wall_thickness=2,
+        thumb_radius=3.5,
+        divot_radius=0.5,
+        tolerance=0.3,
+    )
+
+    show(
+        sb,
+    )

{b3dkit-0.4.0 → b3dkit-0.5.2}/src/b3dkit/twist_snap.py

@@ -204,8 +204,11 @@ class TwistSnapSocket(BasePartObject):
                     align=(Align.CENTER, Align.CENTER, Align.MIN),
                     mode=Mode.SUBTRACT,
                 )
+            # snap_socket.part, not snap_socket.edges(): build123d 0.12 reports a
+            # located builder's edges in its local frame, and this path is used
+            # outside it
             trace_path = (
-                snap_socket.edges()
+                snap_socket.part.edges()
                 .filter_by(GeomType.CIRCLE)
                 .sort_by(Axis.Z, reverse=True)
                 .sort_by(SortBy.RADIUS, reverse=True)[-1]

{b3dkit-0.4.0 → b3dkit-0.5.2}/tests/test_antichamfer.py

@@ -1,7 +1,3 @@
-from importlib.machinery import SourceFileLoader
-from importlib.util import module_from_spec, spec_from_loader
-from unittest.mock import patch
-
 import pytest
 from build123d import (
     Align,
@@ -16,7 +12,6 @@ from build123d import (
 )
 
 from b3dkit.antichamfer import anti_chamfer
-from conftest import module_path
 
 
 class TestAntiChamfer:
@@ -122,7 +117,10 @@ class TestAntiChamfer:
             Box(10, 10, 10)
 
         ac_part = anti_chamfer(bp.part.faces().filter_by(Axis.Z)[-1], -1.0, -0.5)
-        assert ac_part.is_valid
+        assert len(ac_part.solids()) == 1
+        # negative lengths still extend the face outward rather than cutting in
+        assert ac_part.volume == pytest.approx(1110.3333, rel=1e-4)
+        assert ac_part.volume > bp.part.volume
 
     def test_anti_chamfer_zero_length_values(self):
         """Test anti_chamfer with negative length values"""
@@ -136,19 +134,6 @@ class TestAntiChamfer:
         assert ac2.is_valid
         assert bp.part.volume == ac1.volume == ac2.volume
 
-    def test_direct_run(self):
-
-        with (
-            patch("build123d.export_stl"),
-            patch("pathlib.Path.mkdir"),
-            patch("pathlib.Path.exists"),
-            patch("pathlib.Path.is_dir"),
-            patch("ocp_vscode.show"),
-            patch("ocp_vscode.save_screenshot"),
-        ):
-            loader = SourceFileLoader("__main__", module_path("antichamfer"))
-            loader.exec_module(module_from_spec(spec_from_loader(loader.name, loader)))
-
 
 class TestAntiChamferBuilderContract:
     """anti_chamfer applies to BuildPart only, and must say so.

{b3dkit-0.4.0 → b3dkit-0.5.2}/tests/test_ball_socket.py

@@ -1,13 +1,9 @@
 import math
-from importlib.machinery import SourceFileLoader
-from importlib.util import module_from_spec, spec_from_loader
-from unittest.mock import patch
 
 import pytest
 from build123d import Part
 
 from b3dkit.ball_socket import BallMount, BallSocket
-from conftest import module_path
 
 
 # ---------- Helpers ----------
@@ -45,8 +41,11 @@ class TestBallSocket:
     @pytest.mark.parametrize("r,w", [(3, 2), (5, 1), (10, 2), (20, 4), (12.5, 3.5)])
     def test_ball_socket_param_dimensions(self, r, w):
         socket = BallSocket(r, wall_thickness=w)
-        assert socket.is_valid
+        assert len(socket.solids()) == 1
         bbox = socket.bounding_box()
+        # the socket encloses the ball plus a wall on each side
+        assert bbox.size.X == pytest.approx(2 * r + 2 * w)
+        assert socket.volume > 0
 
     def test_ball_socket_custom_wall_thickness(self):
         r, w = 10.0, 3.0
@@ -97,11 +96,3 @@ class TestEdgeCases:
         assert tight.is_valid
         assert loose.is_valid
         assert loose.volume < BallSocket(10.0, tolerance=0.0).volume
-
-
-# ---------- Direct Run ----------
-class TestDirectRun:
-    def test_direct_run(self):
-        with patch("ocp_vscode.show"):
-            loader = SourceFileLoader("__main__", module_path("ball_socket"))
-            loader.exec_module(module_from_spec(spec_from_loader(loader.name, loader)))