Skip to content

refactor(helper): drop stderr from response - #616

Draft
Alex Shabalin (alex-sparus) wants to merge 1 commit into
alex/fix-helper-c2from
alex/fix-helper-c2.1
Draft

refactor(helper): drop stderr from response#616
Alex Shabalin (alex-sparus) wants to merge 1 commit into
alex/fix-helper-c2from
alex/fix-helper-c2.1

Conversation

@alex-sparus

@alex-sparus Alex Shabalin (alex-sparus) commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • removing unused field from the helper protocol

internal code C2.1

Test Plan

  • Manual testing of a CI-build artifact

Docs

  • Docs updated (companion PR in darkmatter/nixmac-web: #___)
  • No docs update needed

@alex-sparus

Alex Shabalin (alex-sparus) commented Jul 28, 2026

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by jj-ryu.

@darkmatter

darkmatter Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

🎨 Storybook preview

Open Storybook preview

Updated for f2cab13

@github-actions

github-actions Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor
Warnings
⚠️

No Linear issue ID found in this PR's title, description, or branch name (expected something like ENG-123). Add one so this work is traceable in Linear, or add #no-linear to the PR description to acknowledge it's intentionally untracked.

⚠️ PR is marked WIP / draft — do not merge until ready for review.

📋 PR Overview

Lines changed 48 (+19 / -29)
Files 0 added, 2 modified, 0 deleted
Draft / WIP yes
Has Test Plan yes
Linear issue no
No Test Plan Needed no
New UI components no
New Storybook stories no
New Rust modules no
New TS source files no
New tests no
package.json touched no
Cargo.toml touched no
Infra / CI touched no

🔬 Coverage

Report Lines Statements Functions Branches
apps/native/coverage/coverage-summary.json 35.5% 35.1% 30.6% 29.4%

Generated by 🚫 dangerJS against c73bf3d

The daemon never populates it: activation runs with stderr merged
into stdout (2>&1), so the field only implied a separate error
channel that does not exist. Removing it is free while protocol v2
already breaks the wire shape.
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