Skip to content

e-stepper: "Clear dead frames" is now "Clear unreachable objects" - #182

Merged
martin-henz merged 1 commit into
mainfrom
clear-unreachable-objects
Oct 18, 2026
Merged

martin-henz merged 1 commit into
mainfrom
clear-unreachable-objects

Conversation

@martin-henz

Copy link
Copy Markdown
Member

Plugins side of source-academy/py-slang#585 (docs and comments: source-academy/py-slang#593). The button clears unreachable lists as well as frames, so the old name was a misnomer.

  • Button label, prop (clearDead → clearUnreachable), comments, READMEs and tests.
  • One changeset, for the ignored web-e-stepper only. The common-cse-machine change is a comment and a README line.
  • Function objects are a separate matter (py-slang#586).

Tests: the e-stepper suite passes except EStepperHostPlugin.test.tsx, which needs the sibling package's build output that a fresh worktree lacks; I did not touch that file.

🤖 Generated with Claude Code

…-slang#585)

Lists are cleared too, not only frames.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@changeset-bot

changeset-bot Bot commented Oct 18, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 816165d

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@martin-henz

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-18T08:00:05.469980Z 816165d Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Bravo.

Reviewed commit: 816165d529

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@martin-henz
martin-henz merged commit c3616ac into main Oct 18, 2026
2 checks passed
@martin-henz
martin-henz deleted the clear-unreachable-objects branch October 18, 2026 08:47
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