Skip to content

Update copilot-instructions: clarify rush install/update workflow and fix version management example#5782

Draft
Copilot wants to merge 3 commits intomainfrom
copilot/update-copilot-instructions
Draft

Update copilot-instructions: clarify rush install/update workflow and fix version management example#5782
Copilot wants to merge 3 commits intomainfrom
copilot/update-copilot-instructions

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 17, 2026

Corrects misleading guidance around rush install vs rush update usage, adds commit hygiene rules for lockfile changes, and fixes a bad common-versions.json example.

Changes

  • Command order and guidance: Reordered commands so rush install is listed first as the default day-to-day command (first clone, pulling changes, switching branches, CI). rush update is demoted to a secondary command only to be run when rush install explicitly emits an error requiring it.
  • rush update --recheck: Added as a specific use case for when pnpm-config.json or .pnpmfile.cjs has been modified and bare rush update fails to produce expected lockfile changes.
  • Commit hygiene: Added guidance that files modified by rush update must be isolated in a dedicated (chore) rush update commit. Rebase workflow: drop prior (chore) rush update commits, rerun rush update, recreate the commit from rebased state.
  • Version management example: Fixed common-versions.json examples — a version listed in preferredVersions was incorrectly also listed in allowedAlternativeVersions. Removed the duplicate.
  • Troubleshooting: Replaced the broad "run rush update --recheck" advice with the correct heuristic: rush install first, rush update only when explicitly prompted.

Copilot AI and others added 3 commits April 17, 2026 21:10
Agent-Logs-Url: https://github.com/microsoft/rushstack/sessions/5a9639e9-0f43-402c-aa26-b6303dd01505

Co-authored-by: dmichon-msft <26827560+dmichon-msft@users.noreply.github.com>
Agent-Logs-Url: https://github.com/microsoft/rushstack/sessions/5a9639e9-0f43-402c-aa26-b6303dd01505

Co-authored-by: dmichon-msft <26827560+dmichon-msft@users.noreply.github.com>
… changes

Agent-Logs-Url: https://github.com/microsoft/rushstack/sessions/63c96529-dda7-4592-993f-1e319c628d47

Co-authored-by: dmichon-msft <26827560+dmichon-msft@users.noreply.github.com>
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.

3 participants