Skip to content

Route stdout and stderr through the broker - #1289

Merged
Weidong Cui (wdcui) merged 14 commits into
uliteboxfrom
wdcui/ulitebox/broker-stdio-output
Sep 4, 2026
Merged

Route stdout and stderr through the broker#1289
Weidong Cui (wdcui) merged 14 commits into
uliteboxfrom
wdcui/ulitebox/broker-stdio-output

Conversation

@wdcui

@wdcui Weidong Cui (wdcui) commented Sep 4, 2026

Copy link
Copy Markdown
Member

This PR routes /dev/stdout and /dev/stderr writes through a new shared-memory broker stdio operation, adds a userland broker provider for inherited output streams, requires an active broker for non-empty writes while preserving empty and bounded partial-write behavior, and updates runner tests, UnixBench integration, CI, and LVBS documentation for brokered output.

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>

Co-authored-by: Copilot CLI <copilot-cli@github.com>

Co-authored-by: Copilot Autofix <198982749+Copilot@users.noreply.github.com>

Co-authored-by: GitHub Copilot <198982749+Copilot@users.noreply.github.com>

Co-authored-by: OpenAI <noreply@openai.com>

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Co-authored-by: GitHub Copilot <copilot@github.com>

Co-authored-by: GitHub Copilot <107556219+Copilot@users.noreply.github.com>

Copilot-Session: 239ec5f7-870a-4259-bcae-4ca85fb913a0
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 239ec5f7-870a-4259-bcae-4ca85fb913a0
@wdcui
Weidong Cui (wdcui) force-pushed the wdcui/ulitebox/broker-stdio-output branch from 2ee2beb to acf810a Compare September 4, 2026 02:16
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 239ec5f7-870a-4259-bcae-4ca85fb913a0
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 239ec5f7-870a-4259-bcae-4ca85fb913a0
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 239ec5f7-870a-4259-bcae-4ca85fb913a0
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 239ec5f7-870a-4259-bcae-4ca85fb913a0
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 239ec5f7-870a-4259-bcae-4ca85fb913a0
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 239ec5f7-870a-4259-bcae-4ca85fb913a0
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 239ec5f7-870a-4259-bcae-4ca85fb913a0
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 239ec5f7-870a-4259-bcae-4ca85fb913a0
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 239ec5f7-870a-4259-bcae-4ca85fb913a0
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 239ec5f7-870a-4259-bcae-4ca85fb913a0
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 239ec5f7-870a-4259-bcae-4ca85fb913a0
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 239ec5f7-870a-4259-bcae-4ca85fb913a0
@wdcui Weidong Cui (wdcui) changed the title Route standard output through the broker Route stdout and stderr through the broker Sep 4, 2026
@wdcui
Weidong Cui (wdcui) added this pull request to the merge queue Sep 4, 2026
Merged via the queue into ulitebox with commit 52f18bf Sep 4, 2026
8 checks passed
@wdcui
Weidong Cui (wdcui) deleted the wdcui/ulitebox/broker-stdio-output branch September 4, 2026 04:21
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

🤖 SemverChecks 🤖 ⚠️ Potential breaking API changes detected ⚠️

Click for details
--- failure method_parameter_count_changed: pub method parameter count changed ---

Description:
A publicly-visible method now takes a different number of parameters, not counting the receiver (self) parameter.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#fn-change-arity
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.50.0/src/lints/method_parameter_count_changed.ron

Failed in:
  litebox_broker_core::BrokerCore::new takes 3 parameters in /home/runner/work/litebox/litebox/target/semver-checks/git-ulitebox/3dcf9d61088d0e4fe296ebc054568bf620f102d5/litebox_broker_core/src/lib.rs:130, but now takes 4 parameters in /home/runner/work/litebox/litebox/litebox_broker_core/src/lib.rs:133
  litebox_broker_core::BrokerCore::new_with_limits takes 4 parameters in /home/runner/work/litebox/litebox/target/semver-checks/git-ulitebox/3dcf9d61088d0e4fe296ebc054568bf620f102d5/litebox_broker_core/src/lib.rs:144, but now takes 5 parameters in /home/runner/work/litebox/litebox/litebox_broker_core/src/lib.rs:149

--- failure enum_variant_added: enum variant added on exhaustive enum ---

Description:
A publicly-visible enum without #[non_exhaustive] has a new variant.
        ref: https://doc.rust-lang.org/cargo/reference/semver.html#enum-variant-new
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.50.0/src/lints/enum_variant_added.ron

Failed in:
  variant BrokerResult:Stdio in /home/runner/work/litebox/litebox/litebox_broker_protocol/src/message.rs:155
  variant BrokerOperation:Stdio in /home/runner/work/litebox/litebox/litebox_broker_protocol/src/message.rs:50

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