Expose scoped RushSession reporter producers - #5988
Expose scoped RushSession reporter producers#5988Sean Larkin (TheLarkInn) wants to merge 1 commit into
Conversation
|
Independent R3 review against #5976/RFC phase 3: the slice is correctly limited to the optional frontend-to-engine sink plus trusted Rush/action/plugin producer facades. I found no source-identity spoofing, privacy, no-sink compatibility, error-handling, API, or visible-output defect. Stack-only diff is one commit over #5985; no unresolved review threads are present. Full local stack validation is recorded on child #5992. |
277edae to
eb6ae44
Compare
|
Combined deep review of current head
|
eb6ae44 to
07f17e4
Compare
|
Rebased the single R3A commit onto #5989 head |
|
Final combined review of |
9325875 to
39827fa
Compare
257d502 to
8a9585c
Compare
8a9585c to
8a54622
Compare
8a54622 to
007b113
Compare
007b113 to
23de51a
Compare
c8637e5 to
e3bf1e7
Compare
d57e66b to
fe52c2d
Compare
Retain the exact scoped producer API and WeakMap-backed plugin facades while preserving native-private parser/plugin members and real launch-boundary coverage. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10
fe52c2d to
9c40269
Compare
Part of #5976
Stack parent: #5985
Summary
RushSessionand command-bound reporters to Rush actionsValidation
rush installrush test --only @rushstack/rush-reporter --only @microsoft/rush-lib --only @rushstack/rush-sdkrush build --to @microsoft/rush-lib --to @rushstack/rush-sdkrush checkrush change --verifyCompatibility guarantee
No reporter is created and no structured event is emitted unless the frontend explicitly supplies a sink. Existing
RushSession.getLogger(),RushSession.terminalProvider, and visible terminal output remain authoritative and unchanged.Non-goals