Skip to content

Wire rush-reporter package dependencies - #5985

Open
Sean Larkin (TheLarkInn) wants to merge 2 commits into
mainfrom
copilot/reporter-r1a-package-wiring
Open

Wire rush-reporter package dependencies#5985
Sean Larkin (TheLarkInn) wants to merge 2 commits into
mainfrom
copilot/reporter-r1a-package-wiring

Conversation

@TheLarkInn

Copy link
Copy Markdown
Member

Summary

  • add @rushstack/rush-reporter as a workspace dependency of @microsoft/rush-lib and @microsoft/rush
  • update the default-subspace PNPM importers and approved-package metadata through the Rush update workflow
  • add a focused package-boundary test that keeps the reporter independent from rush-lib

Validation

  • rush build --to @microsoft/rush --verbose
  • rush test --only @rushstack/rush-reporter --verbose (293 tests passed)
  • rush check
  • rush change --verify --no-fetch

Non-goals

  • no bootstrap-envelope encoder or protocol-major generation (owned by R1B)
  • no runtime integration, reporter opt-in, or default output changes

Part of #5974

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

Copilot-Session: d6318e80-5da9-4858-a147-817e8692f10e
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: d6318e80-5da9-4858-a147-817e8692f10e
@TheLarkInn

Copy link
Copy Markdown
Member Author

All required CI is green and independent integration review is complete. This PR is now the first gate for the validated reporter stack: #5986#5987#5989#5988#5991#5992#5996#5997, with #5993/#6000/#5999/#5998 branching from the appropriate tips.

The slice remains limited to package dependency/lockfile/boundary wiring and preserves runtime/default output behavior. Squash auto-merge is enabled; requesting maintainer approval so the fleet can begin sequential retargeting and full-CI validation of each child.

@TheLarkInn

Copy link
Copy Markdown
Member Author

Combined deep review of current head 2b89273bbb.

No critical, high, or medium actionable findings. No useful low follow-ons.

@TheLarkInn

Copy link
Copy Markdown
Member Author

Ian Clanton-Thuon (@iclanton) David Michon (@dmichon-msft) Maintainer review is now the only gate on this root stack. #5985 is clean with full CI green and auto-merge enabled. Every descendant current head has completed repeated deep review with no actionable issue above low. Could one of you review the root so sequential retargeting can begin?

The independent privacy root #5990 is also full-CI green, low-only, and auto-merge enabled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Needs triage

Development

Successfully merging this pull request may close these issues.

1 participant