Skip to content

Conversation

@sayomaki
Copy link
Contributor

@sayomaki sayomaki commented Aug 8, 2025

Description

It appears that the check for ensuring whether the evaluation output is for the stepper is missing (presumably removed as it is no longer in use), and the check falsely assumes an array with first element being an object (which unfortunately, arrays are objects as well) is stepper output, and then tries to render the stepper with invalid information

Some examples include:
pair(pair(1, 2), 3);
list(list(1), 2);
list(pair(1, 2), 3);
pair(list(1), 2);

Resolves #3279.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Code quality improvements

How to test

Checklist

  • I have tested this code
  • I have updated the documentation

@sayomaki sayomaki self-assigned this Aug 8, 2025
@sayomaki sayomaki requested a review from RichDom2185 August 8, 2025 08:19
Copy link
Member

@RichDom2185 RichDom2185 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks

@RichDom2185 RichDom2185 merged commit 6876b2f into master Aug 9, 2025
9 checks passed
@RichDom2185 RichDom2185 deleted the stepper-hotfix branch August 9, 2025 06:23
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.

Source §2: list program crashes frontend

4 participants