Skip to content

Add Rush reporter repository configuration - #5987

Open
Sean Larkin (TheLarkInn) wants to merge 1 commit into
copilot/reporter-r1b-bootstrap-generationfrom
copilot/reporter-r2a-experiment-config
Open

Add Rush reporter repository configuration#5987
Sean Larkin (TheLarkInn) wants to merge 1 commit into
copilot/reporter-r1b-bootstrap-generationfrom
copilot/reporter-r2a-experiment-config

Conversation

@TheLarkInn

@TheLarkInn Sean Larkin (TheLarkInn) commented Aug 28, 2026

Copy link
Copy Markdown
Member

Part of #5975.

Stack

This is the R2A child of #5986. Keep auto-merge disabled while either ancestor is open. After #5985 and #5986 merge in order, retarget this PR to main, verify its slice and CI, then merge it before #5989.

Changes

  • Add the experimental useRushReporter opt-in to experiments types, schema, and rush init template.
  • Add typed rush.json loading for reporting.agentEnvironmentVariables, including schema and template support.
  • Add focused default, parsing, and invalid-input coverage plus the Rush API report and changefile.

Validation

  • rush test --only @microsoft/rush-lib
  • rush check
  • rush change --verify --no-fetch
  • Generated rush init files verified to omit reporting and useRushReporter by default.
  • gh pr diff 5987 contains only the 11-file R2A configuration/schema slice.

Compatibility and non-goals

Absent or false useRushReporter preserves legacy reporting behavior, and absent reporting configuration yields an empty agent-variable list. This PR only exposes typed repository configuration; it does not activate agent auto-selection, add frontend/CLI controls, or produce visible reporter output.

@TheLarkInn

Copy link
Copy Markdown
Member Author

Independent stack-base review found no actionable defects in the repository configuration/schema/API slice. Local rush-lib tests, rush check, and changefile verification passed. Combined validation with #5989 is clean. Stack gate: keep this PR based on copilot/reporter-r1a-package-wiring with auto-merge disabled until #5985 merges; then retarget to main and run full CI. Merge #5987 before #5989 so the public useRushReporter schema/API lands before the frontend consumes it.

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

Copilot-Session: d6318e80-5da9-4858-a147-817e8692f10e
@TheLarkInn
Sean Larkin (TheLarkInn) force-pushed the copilot/reporter-r2a-experiment-config branch from da8e9ca to c647c7f 比较 August 28, 2026 04:37
@TheLarkInn
Sean Larkin (TheLarkInn) changed the base branch from copilot/reporter-r1a-package-wiring to copilot/reporter-r1b-bootstrap-generation August 28, 2026 04:37
@TheLarkInn

Copy link
Copy Markdown
Member Author

Linear stack update: R2A is now head c647c7f based on #5986. Its GitHub diff is exactly the 11-file configuration/schema slice. Auto-merge remains disabled while #5985/#5986 are open.

@TheLarkInn

Copy link
Copy Markdown
Member Author

Combined deep review of current head c647c7f967.

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

注册 for free to join this conversation on GitHub. Already have an account? 登录 to comment

标签

None yet

项目

Status: Needs triage

Development

Successfully merging this pull request may close these issues.

1 participant