From 070b7793f492f3537afa0edbf2e2c88c943645f5 Mon Sep 17 00:00:00 2001 From: Petru Conduraru Date: Sun, 19 Jul 2026 22:48:25 +0300 Subject: [PATCH] Fix all remaining ty type-check failures on ci-lint The ci-lint workflow's ty steps fail on every branch because base v0.8.0 has four diagnostics. ty check (bonsai): - root/operator.py: bpy.data.objects.get() can return None, so UnlinkObject._execute could put None in its objects list and crash on the first attribute access when an unknown object name is passed. Handle the miss explicitly, which also satisfies the declared list[bpy.types.Object] type. - tool/sequence.py: ty does not narrow Literal types through membership tests on list literals, so the assert_never() exhaustive check was flagged. Use tuple literals, which ty narrows, keeping the exhaustiveness check intact. ty check (ios): - draw.py: arrange_polygons was called through conditional argument splats that let the same call site work against pre-April-2026 wrappers lacking arrange_polygon_settings and the logger parameter. No runtime bug for current builds, but the dynamic splats cannot be typed against the fixed 3-parameter signature. Drop the old-build workaround and call the current signature directly, following the precedent of 3d8115ebc5 which dropped similar old-build workarounds in ifcopenshell.file. Verified against a current wrapper build that the direct call arranges polygons and serializes to SVG, with and without a logger. - Optimise.py: igraph is an optional dependency with a guarded import and a toposort fallback, but it was missing from the ios type-check venv so ty could not resolve it. Add it to type-check-requirements next to the toposort fallback that is already listed. After this, poe ty-bonsai and poe ty-ios both pass cleanly. Generated with the assistance of an AI coding tool. (cherry picked from commit 6d6d92b8492adff67079d2aa446736e537eb6f56) --- src/bonsai/bonsai/bim/module/root/operator.py | 5 +++-- src/bonsai/bonsai/tool/sequence.py | 6 +++--- src/ifcopenshell-python/type-check-requirements.txt | 1 + 3 files changed, 7 insertions(+), 5 deletions(-) diff --git a/src/bonsai/bonsai/bim/module/root/operator.py b/src/bonsai/bonsai/bim/module/root/operator.py index 993d3bfb4b..c7a1f6f02e 100644 --- a/src/bonsai/bonsai/bim/module/root/operator.py +++ b/src/bonsai/bonsai/bim/module/root/operator.py @@ -413,12 +413,13 @@ class UnlinkObject(bpy.types.Operator, tool.Ifc.Operator): skip_invoke: bpy.props.BoolProperty(default=False, options={"SKIP_SAVE"}) def _execute(self, context): + objects: list[bpy.types.Object] if self.obj: - objects = [bpy.data.objects.get(self.obj)] + requested_obj = bpy.data.objects.get(self.obj) + objects = [requested_obj] if requested_obj is not None else [] else: objects = context.selected_objects - objects: list[bpy.types.Object] for obj in objects: was_active_object = obj == context.active_object diff --git a/src/bonsai/bonsai/tool/sequence.py b/src/bonsai/bonsai/tool/sequence.py index 2afed8741a..5d51cc451f 100644 --- a/src/bonsai/bonsai/tool/sequence.py +++ b/src/bonsai/bonsai/tool/sequence.py @@ -1158,11 +1158,11 @@ class Sequence(bonsai.core.tool.Sequence): props.task_input_colors.clear() for group, data in groups.items(): for predefined_type in data["PredefinedType"]: - if group in ["CREATION", "OPERATION", "MOVEMENT_TO"]: + if group in ("CREATION", "OPERATION", "MOVEMENT_TO"): predefined_type_item = props.task_output_colors.add() - elif group in ["MOVEMENT_FROM"]: + elif group in ("MOVEMENT_FROM",): predefined_type_item = props.task_input_colors.add() - elif group in ["USERDEFINED", "DESTRUCTION"]: + elif group in ("USERDEFINED", "DESTRUCTION"): predefined_type_item = props.task_input_colors.add() predefined_type_item2 = props.task_output_colors.add() predefined_type_item2.name = predefined_type diff --git a/src/ifcopenshell-python/type-check-requirements.txt b/src/ifcopenshell-python/type-check-requirements.txt index c797c28034..d0cee0bd7e 100644 --- a/src/ifcopenshell-python/type-check-requirements.txt +++ b/src/ifcopenshell-python/type-check-requirements.txt @@ -3,6 +3,7 @@ cjio >=0.8, <0.10 deepdiff docutils flask +igraph isodate jinja2 lark