Reporter overhaul (5/6): Heft protocol track (negotiated child descriptors) - #5869
Open
Sean Larkin (TheLarkInn) wants to merge 3 commits into
Open
Reporter overhaul (5/6): Heft protocol track (negotiated child descriptors)#5869Sean Larkin (TheLarkInn) wants to merge 3 commits into
Sean Larkin (TheLarkInn) wants to merge 3 commits into
Conversation
This was referenced Jul 15, 2026
Sean Larkin (TheLarkInn)
force-pushed
the
reporter/5-heft-protocol
branch
from
July 15, 2026 04:53
95b3891 to
be13212
Compare
Integrate Heft as a cross-process reporter producer for @rushstack/reporter (#5858). - Add allocateChildDescriptor and readChildDescriptorFd so Rush passes a dynamically allocated inherited descriptor to the child through a private environment variable while stdout and stderr stay normal streams - Add HeftChildEmitter, which emits structured NDJSON over the descriptor or falls back to raw stdout and stderr when negotiation is unavailable - Add HeftDescriptorHost, which negotiates the child hello and correlates each child event with the parent session and operation ids, surfacing an update-global-Rush diagnostic on rejection - Keep the raw-stream and problem-matcher path for older Heft versions - Cover both the new descriptor and old raw-stream paths with tests Assistant-model: GitHub Copilot Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 897dcf7e-e6e8-4a84-85ca-34b93fa29be3
Assistant-model: GitHub Copilot Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 897dcf7e-e6e8-4a84-85ca-34b93fa29be3
HeftDescriptorHost gains incremental record processing (processChildRecord) and createStreamProcessor(), which decodes NDJSON chunks as they arrive and forwards accepted events in receipt order. This continuously drains the child pipe (a chatty child no longer blocks on a full OS pipe buffer) and surfaces child progress live, per the design-review realignment. The batch processChildNdjson path is retained for completed streams and tests. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Sean Larkin (TheLarkInn)
force-pushed
the
reporter/5-heft-protocol
branch
from
August 19, 2026 14:14
be13212 to
a68db43
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📚 Reporter Overhaul PR stack (merge bottom-up)
mainEach PR's diff is scoped to its phase; review independently, merge from #5865 upward.
👉 This is PR 5 of 6.
Phase 5 — Heft protocol track (RFC §8.1)
Adds structured cross-process reporting for Heft child builds, with backward compatibility for older Heft.
What's included
hello+ events over it, correlated with the parent session/operation ids.Validation
rush build --to @rushstack/reporter✅rush test --only @rushstack/reporter✅ (adds HeftIntegration suite covering both the descriptor path and the raw-stream path)@beta.Scope note
Standalone package; not yet wired into the live CLI. Independently releasable/revertible per RFC §8.1.