Skip to content

repl: Pluck project off of editor directly#48762

Merged
MrSubidubi merged 1 commit intozed-industries:mainfrom
rgbkrk:rgbkrk/investigate-repl-run
Feb 9, 2026
Merged

repl: Pluck project off of editor directly#48762
MrSubidubi merged 1 commit intozed-industries:mainfrom
rgbkrk:rgbkrk/investigate-repl-run

Conversation

@rgbkrk
Copy link
Collaborator

@rgbkrk rgbkrk commented Feb 9, 2026

The new multi workspace introduced in #47795 changed the window root from Workspace to MultiWorkspace, which broke Workspace::for_window() (assuming that was meant to). That returns None now. The REPL action registration in repl_sessions_ui.rs used this to check if the project was local, so when it got None, it silently skipped registering repl::Run and repl::RunInPlace on every editor.

Luckily we can just get the project directly from the editor in order to register actions.

Release Notes:

  • N/A

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 9, 2026
@rgbkrk rgbkrk added the area:repl repl, jupyter, notebooks, etc label Feb 9, 2026
Copy link
Member

@MrSubidubi MrSubidubi left a comment

Choose a reason for hiding this comment

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

Thanks for the quick fix!

@MrSubidubi MrSubidubi self-assigned this Feb 9, 2026
@MrSubidubi MrSubidubi merged commit db6c006 into zed-industries:main Feb 9, 2026
33 checks passed
@rgbkrk
Copy link
Collaborator Author

rgbkrk commented Feb 9, 2026

Thanks for the quick fix!

It was driving me crazy while I was working on other PRs! Thank you for the quick merge.

@rgbkrk rgbkrk deleted the rgbkrk/investigate-repl-run branch February 9, 2026 09:12
MrSubidubi added a commit that referenced this pull request Feb 9, 2026
MrSubidubi added a commit that referenced this pull request Feb 9, 2026
MrSubidubi pushed a commit that referenced this pull request Feb 9, 2026
Clean up code by pulling the project off the editor directly.

Repeat of #48762 since
reversion happened in #48776.

Release Notes:

- N/A
Serophots pushed a commit to Serophots/zed that referenced this pull request Feb 9, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 10, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 10, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 10, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 10, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
oscarvarto pushed a commit to oscarvarto/zed that referenced this pull request Feb 12, 2026
Clean up code by pulling the project off the editor directly.

Repeat of zed-industries#48762 since
reversion happened in zed-industries#48776.

Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:repl repl, jupyter, notebooks, etc cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants