Repository navigation
Visualizer: no step for -x where it reads as its value; console check without buttons - #50
Merged
Merged
Conversation
…ck without buttons "-x" with x = 7 reads "-7" once x is replaced, so the minus records no step of its own; "--7" and "-True" keep theirs. The browser check's console text leaves out the entries' Visualize buttons, which land next to a ">>>" line or on a line of their own depending on the width. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
cedricvogel-leo
changed the base branch from
feature/literal-displays
to
main
November 4, 2026 23:54
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-ups from #48, as decided. This PR stacks on #48: merge #48 first. GitHub then retargets this PR to
main, or if the branch isn't deleted, I'll rebase it.What
-x(every step changes something, as in P40). Unary minus on a name no longer gets its own step when the line already reads as the value once the name is replaced:-xwithx = 7is one step:x → 7, so the line reads-7. Before, a second step-7 → -7followed that changed nothing.-xwithx = 2.5likewise.-xwithx = -7goes--7 → 7;-xwithx = Truegoes-True → -1.-(a + b)keeps its step-3 → -3; the rule is for names only;+3keeps its step+3 → 3.The P36
input()browser check reads the console's transcript without the entries' «Visualisieren» buttons. Depending on the width, a button sits next to its>>>line or wraps onto a line of its own. In CI it had wrapped, so>>> repl()\nVisualisieren\n> 1 + 2…didn't match.consoleTextnow hides the buttons while it reads the text. That covers every check that reads the console.Checks
scripts/check-runtime-lists.pycovers-xwith 7, 2.5, -7 and True, and-(a + b). The P40 case(n, -n, {n: [n]})now has no step for-n.lint,test:runtimeandtest:consolepass.How it goes live
maindeploys the embed, so the courses get it straight away.v0.1.25orv0.1.24if it's merged before you set that one, and then the IDE bump.Manual tests (desktop, and iPad with an external keyboard)
x = 7thenw = -x, in Ersetzung and Kasten. Line 2 has one substitution (w = -7), then the assignment.x = -7thenw = -x. Line 2 showsw = --7, thenw = 7.a = 1,b = 2,w = -(a + b). As before, the minus keeps its step.Follow-up (not fixed)
not xis labellednot7in the trace: the space afternotis missing. Shown in the step label and in Copy → chain. It's pre-existing.🤖 Generated with Claude Code