Skip to content

Pin openscad_cpp_evaluator 0.26.1: step-into-child works from children() - #202

Merged
revarbat merged 1 commit into
mainfrom
bump-evaluator-0.26.1
Aug 9, 2026
Merged

Pin openscad_cpp_evaluator 0.26.1: step-into-child works from children()#202
revarbat merged 1 commit into
mainfrom
bump-evaluator-0.26.1

Conversation

@revarbat

@revarbat revarbat commented Aug 9, 2026

Copy link
Copy Markdown
Member

Picks up openscad_cpp_evaluator#83.

Stepping to a child from inside a module's own children(); found no target and degraded into a continue — running the script to completion and ending the debug session. childStatementPositions returned a node's own children, and a children() call has none; it forwards the enclosing invocation's, which live on the context.

Verified against the published wheel, both child kinds:

--- builtin child (cube) ---      continue -> 2   to_child -> paused 5
--- user module child (leaf) ---  continue -> 5   to_child -> paused 8

The built-in was a red herring in the original report — a user module failed identically. Where the pause was is what mattered.

Also

The run-off-the-end message stays as the honest answer for whatever else reaches no child, but its comment is corrected: it previously said the cause was not understood, and now it is.

Verification

Editable install removed, uv lock --refresh-package + uv sync pulled 0.26.1 from PyPI (all six files confirmed present first), and openscad_cpp_evaluator.__file__ verified in site-packages.

653 Python tests and all six GUI verifiers pass.

🤖 Generated with Claude Code

0.26.1 aims step-into-child at the children a children() call forwards.
Stepping to a child from inside a module's own children() previously found
no target and degraded into a continue, running the script to completion
and ending the session.

Verified against the published wheel for both child kinds: from
children(), the step now lands on a forwarded cube() and on a forwarded
user module alike. The built-in was a red herring in the original report --
where the pause was is what mattered.

The run-off-the-end message stays as the honest answer for whatever else
reaches no child, with its comment corrected: it previously said the cause
was not understood.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@revarbat
revarbat merged commit 475318e into main Aug 9, 2026
1 check passed
@revarbat
revarbat deleted the bump-evaluator-0.26.1 branch August 9, 2026 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant