Add direct Rush reporter demo path by TheLarkInn · Pull Request #5997 · microsoft/rushstack

Sean Larkin (TheLarkInn) · 2026-08-28T07:55:44Z

Part of #5978

Stack

Parent: #5996 (copilot/reporter-r5a-operation-adapter)

This PR is based directly on the R5A operation adapter and intentionally excludes the parallel R6 #5993 bootstrap work. Keep auto-merge disabled while stack ancestors remain open.

Demo path

The direct current-version rush build path now transfers visible operation presentation to the frontend-selected reporters when either an explicit non-legacy --reporter is present or the repository useRushReporter experiment is enabled.

the selected primary reporter owns its visible destination;
the always-on file reporter owns a complete invocation log under common/temp/rush-logs;
the retained StreamCollator writes to a no-op destination, preserving scheduling/writer completion semantics without duplicate terminal output;
non-operation Rush terminal messages are converted into semantic reporter messages, while operation summaries come from authoritative operationCompleted events;
RUSH_REPORTER=legacy removes reporter controls before launching Rush and returns immediately to the existing legacy path.

The file reporter spools active operation output to owner-only temporary files, then appends each operation as a grouped block in completion order. This preserves stdout/stderr chunk order without unbounded heap buffering.

Reproducible demo

rush build --to @microsoft/rush
node apps/rush/src/test/sandbox/reporter-demo/run.mjs

The script exercises:

node apps/rush/bin/rush build --only @rushstack/rush-reporter
node apps/rush/bin/rush build --only @rushstack/rush-reporter --reporter=plaintext
node apps/rush/bin/rush build --only @rushstack/rush-reporter --reporter=json --log-level=debug
node apps/rush/bin/rush build --only @rushstack/rush-reporter --reporter=ai
RUSH_REPORTER=legacy node apps/rush/bin/rush build --only @rushstack/rush-reporter --reporter=json

Observed result:

plaintext reconstructs phase-aware grouped operation output and prints an absolute full-log path;
JSON stdout is valid payload-only NDJSON with no blank/control lines;
AI emits bounded status/final records, suppresses warning details on failure, and includes the exact full-log path;
legacy rollback matches the feature-off transcript after normalizing runtime durations;
failed parser/selection paths retain their actionable message and flush through sessionCompleted.

Repository opt-in remains "useRushReporter": true in common/config/rush/experiments.json; removing it or setting RUSH_REPORTER=legacy provides immediate rollback.

Validation

rush test --only @rushstack/rush-reporter --only @microsoft/rush-lib --only @microsoft/rush --verbose
- reporter: 301 tests
- rush-lib: 770 tests
- apps/rush: 25 tests
rush build --only @rushstack/rush-reporter --only @microsoft/rush-lib --only @microsoft/rush --verbose
reporter snapshots/goldens in the targeted suite
rush check
rush change --verify --no-fetch
direct demo matrix above
direct failed JSON invocation confirmed final commandResult, commandCompleted, and sessionCompleted, with the actionable error in the full log

Coverage includes success, cache hit, warnings, operation failure, blocked and aborted operations, silent operations, parallel grouping, stdout/stderr ordering, TTY/non-TTY behavior, quiet/verbose/debug controls, watch-cycle completion, and legacy flag-off parity.

Non-goals

no install-run-rush or bootstrap handoff ownership (R6 / [rush reporter][R6A] Add bootstrap handoff and compatibility adapters #5993);
no Heft child descriptors or problem-matcher protocol changes (R7);
no agent auto-selection (R8 / [rush reporter][R8] Qualify AI reporter auto-selection #5981);
no terminal API removal, Rush 6 default flip, or later-major cleanup (R9+).

Sean Larkin (TheLarkInn)

Independent R5B review completed against #5978 and RFC phase 4.

Fixed in e61332b:

command-specific --json remains the sole stdout owner; incompatible reporter combinations fail before launch;
older repository engines force the legacy path and reporter teardown runs on selector failure;
file/plaintext grouping is disk-backed without per-graph descriptor growth or unbounded heap retention; stale spools are retained/cleaned safely;
watch errors use the semantic terminal, final artifact completeness precedes machine results, zero-width TTYs fall back to 80 columns, and AI parser failures retain actionable context;
the demo now self-checks legacy rollback, plaintext/JSON/AI/file/quiet/failure/help/command-JSON output, one visible writer, chunk ordering, final flush, and owner-only logs.

Validation: build succeeded for @rushstack/rush-reporter, @microsoft/rush-lib, and @microsoft/rush; tests 304/773/28; rush check; rush change --verify --no-fetch; expanded demo and real TTY run passed. No unresolved review threads. Auto-merge remains disabled while #5996 and its ancestors are open.

Sean Larkin (TheLarkInn) · 2026-08-28T08:44:26Z

Final restack correction: #5997 was force-replayed onto the exact current remote #5996 tip ac92280377966dc0da67ec6972ea086219abfebb and pushed with --force-with-lease=refs/heads/copilot/reporter-r5b-demo-reporters:e61332b35ccc0964a1f3f240c4a328975b880c24.

Final #5997 tip: e30aa365e5527213bf91ef64b8f8171e3357cc07 (R5B commits 45e43a810d, e30aa365e5). The old and new parent-to-head binary diffs have the identical SHA-256 84086932b410faa5239a08d091c1bf7a92e874bedd84834abbd76fc7af5c4718; the slice remains exactly 31 files with no R5A conflict-resolution delta. GitHub now reports MERGEABLE/CLEAN and auto-merge remains disabled.

Post-restack validation: reporter 304, rush-lib 773, apps/rush 28; expanded reporter demo; rush check; rush change --verify --no-fetch.

Sean Larkin (TheLarkInn) · 2026-08-28T14:43:26Z

Combined deep review of current head e30aa365e5.

HIGH old-engine forceLegacy silently strips explicit non-legacy reporter controls. Reject unsupported explicit requests instead of changing user intent.
MEDIUM MinimalRushConfiguration.commonTempFolder ignores RUSH_TEMP_FOLDER, so logs and purge target different locations.
MEDIUM legacy verbose configuration discovery output is dropped at the repository root. Restore the original predicate and separator.
MEDIUM artifactAvailable.complete is a live getter, so one event can serialize differently across reporter queues. Snapshot the value.
MEDIUM file spooling opens and closes a file for every output chunk on the main thread. Keep bounded persistent descriptors and close them at operation finalization.

Sean Larkin (TheLarkInn) · 2026-08-28T20:09:17Z

Review fixes published at 715990145a68fd8e9b03ce4ecc01b9612c062d87 on exact #5996 head 11a7f8462dd40b1d3c0d83ffafc5f0b748a80394.

Fixed: incompatible-engine controls remain explicit and are never silently stripped; RUSH_TEMP_FOLDER uses Rush normalization for logs and purge; legacy discovery text and blank lines are restored; artifact completeness is a frozen boolean emitted after file closure; file and plaintext spools use persistent bounded descriptors with exact cleanup. Also fixed output-control ownership, purge-log survival, action-name sanitization, TTY multiline activity, verbose versus debug filtering, AI fallback counts, and file-only parser failures.

Validation: reporter 314, rush-lib 786, apps/rush 70; all affected builds; direct demo matrix at /tmp/rush-reporter-demo-5RtqS3; TTY, CI, cache, watch coverage; rush check; stacked change verification. PR is CLEAN and auto-merge remains disabled pending fresh review.

Sean Larkin (TheLarkInn) · 2026-08-28T20:28:06Z

Four-review gate on 715990145a confirms the prior fixes. Two actionable findings remain:

HIGH The lock-conflict reason is emitted as informational text before AlreadyReportedError. The default reporter can erase it, leaving only a generic command failure. Preserve the actionable reason as an error or structured diagnostic.
MEDIUM Watch iterations re-register and re-complete the same operation IDs, but default, plaintext, and AI reporters accumulate lifetime totals. Later cycles can report 20/20 for a 10-operation graph. Reset or replace per-cycle counts on re-registration or watch-cycle completion.

Dismissible low follow-on: configuration discovery should use the effective RUSH_PREVIEW_VERSION when deciding legacy output. Auto-merge remains disabled.

Sean Larkin (TheLarkInn) · 2026-08-28T20:55:20Z

Round-two findings fixed at 523124198d221cef9ec72c81dd1cfb26eb67023f.

Lock contention now emits the actionable reason at error severity in reporter mode while preserving legacy output without duplication.
Default, plaintext, and AI watch totals reset per graph cycle, including repeated IDs, silent operations, failure, and recovery.
Discovery now uses the effective RUSH_PREVIEW_VERSION, keeping machine stdout clean when the preview matches the frontend.

Validation: reporter 317, rush-lib 787, apps/rush 71; all affected builds; direct demo /tmp/rush-reporter-demo-2ckBQl; preview /tmp/r5b-r2-preview-2ktFGu; real lock contention /tmp/r5b-r2-lock-TtwdV3; TTY/CI/watch coverage; rush check; stacked change verification. PR is CLEAN and auto-merge remains disabled pending fresh review.

Sean Larkin (TheLarkInn) · 2026-08-28T21:01:00Z

Four-review rerun on 523124198d confirms the prior runtime fixes. One MEDIUM concurrency issue remains: a queued watch iteration can register known operation IDs before the prior watchCycleCompleted event. Reporter state can then mix abort completions and next-cycle completions, producing totals such as 20/10.

Required fix: use an explicit cycle boundary or cycle identity that handles overlapping scheduling. Do not assume re-registration occurs after cycle completion.

One dismissible low follow-on remains: make the AI final record consistently per-cycle or session-cumulative after fail-then-success watch recovery. The repeated command-name concern remains dismissed because no supported failing entry point was reproduced.

Sean Larkin (TheLarkInn) · 2026-08-28T21:44:59Z

Round-three findings fixed at 97f807ed6549387b6aab2eccec197123196b725d.

Operation lifecycle events now carry an explicit graph iteration ID, and default, plaintext, AI, and file reporters isolate overlapping generations.
Superseded scheduled iterations no longer register before execution, completed lifecycle-only cycles are released, and watch state remains bounded.
AI final counts, failed projects, diagnostics, and error totals now use the same recovered cycle scope.
Grouped output remains ordered and isolated, active file spools retain the real operation ID, and lock contention renders the actionable reason exactly once.

Validation: reporter 317, rush-lib 788, apps/rush 71; affected builds and daemon dependency chain; full demo /tmp/rush-reporter-demo-Iiphvo; lock contention /tmp/r5b-r3-lock-IxxlNG; preview JSON /tmp/r5b-r3-preview-mYC7Pr; rush check; stacked change verification. #5997 remains on exact #5996 head 11a7f8462dd40b1d3c0d83ffafc5f0b748a80394, with auto-merge disabled pending fresh review.

Sean Larkin (TheLarkInn) · 2026-08-28T21:51:53Z

Final combined review of 97f807ed65 found no critical, high, or medium actionable issue. All lock, watch concurrency, AI recovery scope, preview, temp, artifact, spool, compatibility, and output findings are fixed.

One dismissible low follow-on remains: the exported regroupOperationOutput helper still groups by operation ID without iteration ID for third-party watch reporters. The daemon completion concern was dismissed because the current multiplexer forwards completion, and the repeated command-name concern remains unreproduced. This head is low-only.

Sean Larkin (TheLarkInn)

R5B acceptance review against #5978: live diagnostics and lossless full-detail logs.

Address #5997 review 5154313870: display bounded errors immediately, summarize warnings once per cycle, and handle short and zero-progress writes with truthful artifact completeness. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Preserve the published diagnostics, UTF-8 writes, spooling, privacy, preview selection and operation-cycle fixes while reconciling native parser, watcher, and graph members. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Copilot

Copilot review overview

🟡 Changes recommended

Watch-cycle completion, artifact finalization, and legacy help/discovery parity have unresolved correctness issues.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review tier: Balanced
Findings: 3

New issues introduced by this change (3)

Severity	Finding
	`apps/rush/src/MinimalRushConfiguration.ts` — For legacy/feature-off invocations, discovery output is now printed only after…
	`apps/rush/src/RushReporterHost.ts` — When help forces the legacy path, only `--reporter` is stripped. Reporter-owned…
	`apps/rush/src/RushReporterHost.ts` — The full-detail reporter is closed before `publishIfChanged()` enqueues the final `complete: true`…

What changed in this PR

Adds the opt-in direct Rush reporter path, transferring presentation ownership from legacy collation to frontend-selected reporters while retaining complete file logging and rollback support.

Changes:

Adds iteration-aware operation and watch reporting.
Implements reporter-owned terminal output, diagnostics, redaction, and log finalization.
Adds direct demo coverage and legacy compatibility checks.

File	Description
`libraries/rush-lib/src/pluginFramework/RushSession.ts`	Adds reporter state and flushing helpers.
`libraries/rush-lib/src/logic/ProjectWatcher.ts`	Makes in-place watch rendering optional.
`libraries/rush-lib/src/logic/operations/test/OperationGraphEventSink.test.ts`	Expands iteration and outcome coverage.
`libraries/rush-lib/src/logic/operations/ReporterOperationEventSink.ts`	Adapts iteration-aware operation events.
`libraries/rush-lib/src/logic/operations/OperationGraph.ts`	Assigns iteration IDs and delays registration.
`libraries/rush-lib/src/logic/operations/OperationExecutionRecord.ts`	Propagates iteration identity through streams.
`libraries/rush-lib/src/logic/operations/OperationEventSink.ts`	Extends sink callbacks with iteration IDs.
`libraries/rush-lib/src/logic/operations/IOperationExecutionResult.ts`	Exposes owning iteration identity.
`libraries/rush-lib/src/cli/test/RushCommandLineParserReporterLifecycle.test.ts`	Tests initialization failure flushing.
`libraries/rush-lib/src/cli/test/RushCommandLineParser.test.ts`	Tests sole-writer and lock errors.
`libraries/rush-lib/src/cli/scriptActions/PhasedScriptAction.ts`	Routes presentation through reporters.
`libraries/rush-lib/src/cli/RushCommandLineParser.ts`	Bridges terminal and failure output.
`libraries/rush-lib/src/cli/actions/BaseRushAction.ts`	Reports lock contention semantically.
`libraries/rush-lib/src/api/Rush.ts`	Suppresses the legacy startup banner.
`libraries/rush-lib/src/api/EnvironmentConfiguration.ts`	Adds early temp-folder resolution.
`libraries/rush-daemon/src/test/PhasedRequestEventMultiplexer.test.ts`	Updates multiplexer coverage.
`libraries/rush-daemon/src/PhasedRequestEventMultiplexer.ts`	Forwards iteration identity.
`libraries/reporter/src/utilities/writeAllSync.ts`	Adds complete synchronous writes.
`libraries/reporter/src/test/WriteAllSync.test.ts`	Tests short writes and failures.
`libraries/reporter/src/test/PlaintextReporter.test.ts`	Expands grouping and privacy coverage.
`libraries/reporter/src/test/OperationStreamEmitter.test.ts`	Tests iteration propagation.
`libraries/reporter/src/test/Manager.test.ts`	Tests confirmed flush timeouts.
`libraries/reporter/src/test/LogLevelFilter.test.ts`	Tests message log-level overrides.
`libraries/reporter/src/test/JsonAiReporter.test.ts`	Expands redaction and AI results.
`libraries/reporter/src/test/HumanReadableDiagnostic.test.ts`	Tests safe diagnostic formatting.
`libraries/reporter/src/test/DefaultInteractiveReporter.test.ts`	Expands watch and diagnostic coverage.
`libraries/reporter/src/session/ScopedReporterFactory.ts`	Emits message log-level metadata.
`libraries/reporter/src/scheduler/OperationStreamEmitter.ts`	Adds iteration IDs to operation events.
`libraries/reporter/src/scheduler/OperationOutputGrouping.ts`	Exposes output iteration identity.
`libraries/reporter/src/reporters/ReporterRedaction.ts`	Strengthens event redaction.
`libraries/reporter/src/reporters/LegacyReporter.ts`	Recognizes aborted operations.
`libraries/reporter/src/reporters/JsonReporter.ts`	Redacts machine-readable messages.
`libraries/reporter/src/reporters/HumanReadableDiagnostic.ts`	Adds privacy-aware formatting.
`libraries/reporter/src/reporters/DefaultInteractiveReporter.ts`	Adds bounded iteration-aware presentation.
`libraries/reporter/src/reporters/AiReporter.ts`	Adds watch-aware final records.
`libraries/reporter/src/producers/IScopedReporter.ts`	Adds minimum message log levels.
`libraries/reporter/src/manager/ReporterManager.ts`	Adds confirmed flush support.
`libraries/reporter/src/lifecycle/LifecycleEvents.ts`	Extends lifecycle payloads with iterations.
`libraries/reporter/src/events/IMessageEmittedPayload.ts`	Adds log-level metadata.
`libraries/reporter/src/diagnostics/IRushDiagnostic.ts`	Associates diagnostics with iterations.
`libraries/reporter/src/config/LogLevelFilter.ts`	Honors per-message minimum levels.
`common/reviews/api/rush-reporter.api.md`	Updates reporter API baselines.
`common/reviews/api/rush-lib.api.md`	Updates Rush library API baselines.
`common/changes/@rushstack/rush-reporter/reporter-watch-and-short-writes_2026-09-09.json`	Records watch and write fixes.
`common/changes/@rushstack/rush-reporter/reporter-secret-source-aliases_2026-09-09.json`	Records privacy fixes.
`common/changes/@rushstack/rush-reporter/reporter-heft-diagnostics_2026-09-09.json`	Records diagnostic rendering changes.
`common/changes/@rushstack/rush-reporter/copilot-reporter-r5b-demo-reporters_2026-08-28-07-10.json`	Records reporter presentation support.
`common/changes/@microsoft/rush/copilot-reporter-r5b-demo-reporters_2026-08-28-07-10.json`	Records Rush integration support.
`apps/rush/src/test/sandbox/reporter-demo/run.mjs`	Adds the self-checking demo matrix.
`apps/rush/src/test/sandbox/reporter-demo/README.md`	Documents direct reporter usage.
`apps/rush/src/test/RushReporterHost.test.ts`	Expands host selection and artifact tests.
`apps/rush/src/test/RushFrontend.test.ts`	Tests frontend log placement.
`apps/rush/src/test/MinimalRushConfiguration.test.ts`	Tests discovery and temp overrides.
`apps/rush/src/start.ts`	Centralizes preview-version lookup.
`apps/rush/src/RushReporterHost.ts`	Creates and finalizes frontend reporters.
`apps/rush/src/RushPreviewVersion.ts`	Adds preview-version resolution.
`apps/rush/src/RushFrontend.ts`	Connects host lifecycle to Rush.
`apps/rush/src/MinimalRushConfiguration.ts`	Adds early reporter-aware configuration.

Suppressed comments (1)

libraries/rush-lib/src/logic/operations/OperationGraph.ts:738

The all-silent branch now registers and finalizes its records, then returns without executing #executeInnerAsync. Consequently afterExecuteIterationAsync never runs, so an opted-in watch whose operations are all silent emits operation completions but no watchCycleCompleted; reporters cannot summarize or close that cycle. Emit an equivalent watch boundary for this short-circuit, or route it through the normal after-iteration hook path.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Address #5997 comment 3981283816 without emitting legacy discovery into explicitly owned reporter output. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Address #5997 comment 3981283879 with selective help ownership, preserving custom values, rollback, and following flags. Exercise the real legacy help parser. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Document and guard the deliberately post-close completeness notification raised by #5997 comment 3981283923. Preserve production ordering and prove fsync/close failures never announce a complete artifact. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Keep R5B terminal ownership and its registration/lifetime helpers. Adapt inherited unscoped-output assertions to expect native operation presentation without resurrecting the R5A deferral wrapper. Preserve native initialization and iteration regressions plus post-close artifact completeness. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Reproduce repository-opted-in custom/global and action help through the real frontend/parser. Use command definitions before interpreting reporter-shaped values, preserve mixed and unknown namespaces, and retain malformed configuration discovery guards. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Retain R5B command-JSON stdout ownership, environment sanitization, quiet handling, and artifact lifetime while replacing ordinary value-shape inference with foundation command ownership. Preserve the shared separated-value guard and optional flag list. Apply primary file levels through the existing wrapper without filtering automatic full-detail logs; retain both regression families. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Preserve the final owning help commit and reuse foundation command ownership rather than duplicate metadata loading. Carry frontend-owned standalone flags through help while retaining custom/action verbose aliases. Align frontend and native parser fixture lookup, preserving unknown/plugin conservatism and all close-before-complete contracts. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Remove only the propagation-added primary FileReporter filter: R5B's automatic full-detail log remains unfiltered at every selected level. Assert selected normal/debug independently from retained debug content. Distinguish no-action help from unknown-command ownership and cover both real root-help paths without changing owning refs. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Address #5997 comment 3981283816 without emitting legacy discovery into explicitly owned reporter output. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Document and guard the deliberately post-close completeness notification raised by #5997 comment 3981283923. Preserve production ordering and prove fsync/close failures never announce a complete artifact. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Address #5997 comment 3981283879 with selective help ownership, preserving custom values, rollback, and following flags. Exercise the real legacy help parser. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Retain the exact reviewed R5B slice, including unfiltered canonical full-detail logs and declaration-aware help. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Preserve the existing slice and current parent histories; only capture successful fixture setup diagnostics, without changing production behavior, warning policy, or watch assertions. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Preserve the existing slice and parent histories while carrying the validated test-helper correction. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

The newly landed parent tree is byte-identical to the already integrated parent. Preserve the entire reviewed slice tree and both parent histories while correcting ancestry after the protected stack-prefix squash landing. For R3C, exclude the 300 lines of duplicated old graph tests introduced by automatic merge; the final source, test, dependency and API trees are exactly unchanged. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Land the four remaining reviewed Reporter core slices (#5992, #5996, #5997, #5998) together after exact-head approval, zero unresolved discussions, and all six current CI runs with 36 complete platform workflows were verified. Preserve opt-in behavior and existing quality gates. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Sean Larkin (TheLarkInn) requested review from Apostolis Haitalis (apostolisms), Bharat Middha (bmiddha), David Michon (dmichon-msft), Ian Clanton-Thuon (iclanton), Jonathon Anthony (jxanthony) and Pete Gonzalez (octogonz) as code owners August 28, 2026 07:55

github-project-automation Bot added this to Bug Triage Aug 28, 2026

github-project-automation Bot moved this to Needs triage in Bug Triage Aug 28, 2026

Sean Larkin (TheLarkInn) force-pushed the copilot/reporter-r5b-demo-reporters branch from 25891f4 to e61332b Compare August 28, 2026 08:35

This was referenced Aug 28, 2026

Add feature-flagged operation event adapter #5996

Merged

[rush reporter][R5] Ship the feature-flagged operation stream demo #5978

Open

Sean Larkin (TheLarkInn) commented Aug 28, 2026

View reviewed changes

Sean Larkin (TheLarkInn) force-pushed the copilot/reporter-r5b-demo-reporters branch from e61332b to e30aa36 Compare August 28, 2026 08:40

Sean Larkin (TheLarkInn) force-pushed the copilot/reporter-r5b-demo-reporters branch from e30aa36 to 7159901 Compare August 28, 2026 20:08

Sean Larkin (TheLarkInn) commented Sep 9, 2026

View reviewed changes

Comment thread libraries/reporter/src/reporters/DefaultInteractiveReporter.ts Outdated

Comment thread libraries/reporter/src/reporters/FileReporter.ts Outdated

Sean Larkin (TheLarkInn) force-pushed the copilot/reporter-r5b-demo-reporters branch from 26c2ac0 to 26a469f Compare September 9, 2026 16:03

Sean Larkin (TheLarkInn) force-pushed the copilot/reporter-r5b-demo-reporters branch from 26a469f to 8ae2115 Compare September 10, 2026 01:30

Mo Jazayeri (mojaza) requested a balanced review from Copilot September 10, 2026 16:21

Copilot started reviewing on behalf of Mo Jazayeri (mojaza) September 10, 2026 16:22 View session

Copilot AI reviewed Sep 10, 2026

View reviewed changes

Comment thread apps/rush/src/MinimalRushConfiguration.ts

Comment thread apps/rush/src/RushReporterHost.ts Outdated

Comment thread apps/rush/src/RushReporterHost.ts

Sean Larkin (TheLarkInn) and others added 8 commits September 10, 2026 18:18

Sean Larkin (TheLarkInn) and others added 2 commits September 10, 2026 20:51

Merge native physical-cwd watch fix into R5B

0548584

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Mo Jazayeri (mojaza) approved these changes Sep 10, 2026

View reviewed changes

Sean Larkin (TheLarkInn) and others added 3 commits September 10, 2026 22:37

Merge physical package-link assertions into R5B

74a269d

Preserve the existing slice and parent histories while carrying the validated test-helper correction. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10

Sean Larkin (TheLarkInn) merged commit f443bb5 into main Sep 11, 2026
16 checks passed

github-project-automation Bot moved this from Needs triage to Closed in Bug Triage Sep 11, 2026

Uh oh!

Conversation

Sean Larkin (TheLarkInn) commented Aug 28, 2026

Stack

Demo path

Reproducible demo

Validation

Non-goals

Uh oh!

Sean Larkin (TheLarkInn) left a comment

Choose a reason for hiding this comment

Uh oh!

Sean Larkin (TheLarkInn) commented Aug 28, 2026

Uh oh!

Sean Larkin (TheLarkInn) commented Aug 28, 2026

Uh oh!

Sean Larkin (TheLarkInn) commented Aug 28, 2026

Uh oh!

Sean Larkin (TheLarkInn) commented Aug 28, 2026

Uh oh!

Sean Larkin (TheLarkInn) commented Aug 28, 2026

Uh oh!

Sean Larkin (TheLarkInn) commented Aug 28, 2026

Uh oh!

Sean Larkin (TheLarkInn) commented Aug 28, 2026

Uh oh!

Sean Larkin (TheLarkInn) commented Aug 28, 2026

Uh oh!

Sean Larkin (TheLarkInn) left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Copilot review overview

🟡 Changes recommended

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants