Merged
Conversation
Co-Authored-by Kin Chong wongkinchong75@gmail.com
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
This pull request refactors the tracer to support debugger statements by introducing the StepperDebuggerStatement class and removing the previous logic that stripped these statements. Review feedback suggests improving code consistency by adding a space in an import statement and standardizing indentation and semicolon usage in the new file.
Pull Request Test Coverage Report for Build 23892138051Details
💛 - Coveralls |
7 tasks
martin-henz
approved these changes
Apr 2, 2026
Member
martin-henz
left a comment
martin-henz
left a comment
There was a problem hiding this comment.
Looks good to me.
Let's merge to make progress in the frontend.
Let's add test cases in a separate PR.
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
3 participants
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.
Co-Authored-by Kin Chong wongkinchong75@gmail.com
Description
This PR is related to source-academy/frontend#3702
Now, debugger statements are no longer removed from the program. We have added support for the DebuggerStatement node type in the js-slang tracer. This allows the Source Academy Stepper to correctly identify, display, and evaluate the debugger statement in the programs.
Type of change
How to test
Upon adding a breakpoint at a certain line,
You should be able to see the debugger; statement in the stepper.
Checklist
yarn testjs-slangexecutable is verified to work