Skip to content

Clarify and de-duplicate Copilot instructions across the repo#11357

Draft
timotheeguerin with Copilot wants to merge 5 commits into
mainfrom
copilot/review-copilot-instructions
Draft

Clarify and de-duplicate Copilot instructions across the repo#11357
timotheeguerin with Copilot wants to merge 5 commits into
mainfrom
copilot/review-copilot-instructions

Conversation

Copilot AI commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

This cleans up Copilot guidance so it matches the current repository workflows and tool surface. The main changes remove inaccurate commands, eliminate stale tool references, and consolidate repeated guidance where the same rule was expressed multiple ways.

  • Shared monorepo guidance

    • Replaced invalid or misleading command examples in .github/copilot-instructions.md
    • Standardized changelog guidance on pnpm chronus add
    • Clarified watch-mode guidance so it points to package-local test/watch commands instead of a nonexistent root script
  • Emitter-specific instruction cleanup

    • Consolidated repeated changelog instructions in http-client-java.instructions.md
    • Simplified Java release branch guidance to align with the repo-level publish/ convention
    • Kept emitter-specific setup intact while removing redundant wording
  • Prompt accuracy fixes

    • Updated testserver-generation.md to use the explicit Chronus command for @typespec/http-specs
    • Removed references to unavailable agent/tool APIs from upgrade-tcgc.instructions.md
    • Reframed the TCGC upgrade flow around a plain checklist instead of nonexistent task-management tools
  • Noise reduction

    • Removed out-of-band behavioral instructions from packages/http-client-js/.github/copilot-instructions.md that were not about repository development
    • Tightened wording where the same intent was repeated across files

Example of the command normalization applied in the docs:

- Add change description: `pnpm change add`
+ Add a change description with `pnpm chronus add`

Copilot AI and others added 2 commits July 22, 2026 17:17
Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
1 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
@microsoft-github-policy-service microsoft-github-policy-service Bot added the emitter:client:java Issue for the Java client emitter: @typespec/http-client-java label Jul 22, 2026
Copilot AI and others added 2 commits July 22, 2026 17:41
Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

emitter:client:java Issue for the Java client emitter: @typespec/http-client-java emitter:client:js Issue for the JS client emitter: @typespec/http-client-js eng

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants