Skip to content

Add Sonnet 5 prompt setting - #328884

Merged
bhavyaus merged 2 commits into
microsoft:mainfrom
bhavyaus:bhavya/sonnet-5-prompt
Aug 4, 2026
Merged

Add Sonnet 5 prompt setting#328884
bhavyaus merged 2 commits into
microsoft:mainfrom
bhavyaus:bhavya/sonnet-5-prompt

Conversation

@bhavyaus

@bhavyaus bhavyaus commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Adds an experiment-gated Sonnet 5 system prompt and model-routing coverage.

Tests: Copilot typecheck, lint, and agent prompt tests.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 4, 2026 02:33

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an experiment-gated system prompt customization and routing for Claude Sonnet 5.

Changes:

  • Registers the Sonnet 5 experimental setting.
  • Adds Sonnet 5 prompt instructions and resolver routing.
  • Tests enabled, disabled, and non-Sonnet behavior.
Show a summary per file
File Description
configurationService.ts Defines the experiment-backed setting.
anthropicPrompts.spec.tsx Tests prompt selection and isolation.
anthropicPrompts.tsx Implements and routes the Sonnet 5 prompt.
package.nls.json Adds the localized setting description.
package.json Contributes the experimental setting.

Review details

  • Files reviewed: 5/5 changed files
  • Comments generated: 0
  • Review effort level: Balanced

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@bhavyaus
bhavyaus marked this pull request as ready for review August 4, 2026 05:28
@bhavyaus
bhavyaus enabled auto-merge (squash) August 4, 2026 05:28
@bhavyaus
bhavyaus requested a balanced review from Copilot August 4, 2026 05:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review details

Suppressed comments (1)

extensions/copilot/src/extension/prompts/node/agent/anthropicPrompts.tsx:583

  • The new experiment-enabled routing branch is not exercised by the agent prompt snapshots: test/agentPrompt.spec.tsx:34-50 has no Sonnet 5 family, and the flag defaults to false. Please add coverage that enables ClaudeSonnet5PromptEnabled for a Sonnet 5 endpoint and verifies the new prompt (plus the disabled fallback), otherwise model matching or prompt rendering can regress unnoticed.
		if (this.isSonnet5(endpoint) && this.configurationService.getExperimentBasedConfig(ConfigKey.ClaudeSonnet5PromptEnabled, this.experimentationService)) {
			return ClaudeSonnet5Prompt;
  • Files reviewed: 4/4 changed files
  • Comments generated: 0 new
  • Review effort level: Balanced

@bhavyaus
bhavyaus merged commit cc2c348 into microsoft:main Aug 4, 2026
30 checks passed
@vs-code-engineering vs-code-engineering Bot added this to the 1.133.0 milestone Aug 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants