Skip to content

perf(desktop): talk to the main process without the Effect runtime in the renderer - #50112

Merged
Hona merged 1 commit into
v2from
renderer-plain-rpc
Sep 20, 2026
Merged

Hona merged 1 commit into
v2from
renderer-plain-rpc

Conversation

@Hona

@Hona Hona commented Sep 20, 2026

Copy link
Copy Markdown
Member

Issue for this PR

Closes #

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Please provide a description of the issue, the changes you made to fix it, and why they work. It is expected that you understand why your changes work and if you do not understand why at least say as much so a maintainer knows how much to value the PR.

If you paste a large clearly AI generated description here your PR may be IGNORED or CLOSED!

How did you verify your code works?

Screenshots / recordings

If this is a UI change, please include a screenshot or recording.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

If you do not follow this template your PR will be automatically rejected.

Copilot AI lite review requested due to automatic review settings September 20, 2026 06:13
@Hona
Hona requested a review from Brendonovich as a code owner September 20, 2026 06:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Hona
Hona added this pull request to stack #50115 September 20, 2026 06:13
@Hona
Hona force-pushed the renderer-plain-rpc branch from af88b1c to f53732f Compare September 20, 2026 06:17
@Hona
Hona force-pushed the renderer-plain-rpc branch from f53732f to c55cd39 Compare September 20, 2026 06:22
Base automatically changed from startup-deferrals to v2 September 20, 2026 06:23
… the renderer

The renderer speaks Effect's RPC wire format over the MessagePort directly: messages are structured clones, every payload is JSON-native or a Uint8Array, and the main process is trusted. That keeps Effect's runtime, Stream, Layer, RPC client and its timer-based fiber scheduler out of the renderer's initial module graph. Also adds a source-map attributed summary for renderer CPU profiles.
@Hona
Hona force-pushed the renderer-plain-rpc branch from c55cd39 to f2b9d2d Compare September 20, 2026 06:25
@Hona
Hona merged commit 0530c8e into v2 Sep 20, 2026
6 checks passed
@Hona
Hona deleted the renderer-plain-rpc branch September 20, 2026 06:26
jinhuang712 pushed a commit to jinhuang712/opencode that referenced this pull request Sep 26, 2026
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.

2 participants