Skip to content

Conversation

@bpasero
Copy link
Member

@bpasero bpasero commented Nov 27, 2025

We have meanwhile added agent as built-in mode, so it seems counter intuitive to ask the user to use plan-fast or plan-deep

… setting

We have meanwhile added `agent` as built-in mode, so it seems counter intuitive to ask the user to use `plan-fast` or `plan-deep`
Copilot AI review requested due to automatic review settings November 27, 2025 13:04
@bpasero bpasero enabled auto-merge (squash) November 27, 2025 13:04
@bpasero bpasero self-assigned this Nov 27, 2025
@vs-code-engineering vs-code-engineering bot added this to the November 2025 milestone Nov 27, 2025
Copilot finished reviewing on behalf of bpasero November 27, 2025 13:07
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the chat.promptFilesRecommendations workspace setting configuration that was recommending plan-fast and plan-deep prompt files. The rationale is that with the addition of agent as a built-in mode, recommending these specific planning modes is now counterintuitive. The PR also includes minor formatting improvements by adding blank lines as section separators throughout the settings file.

  • Removes workspace-level configuration for chat prompt file recommendations
  • Moves a code comment inline for better organization
  • Adds consistent blank line separators between configuration sections

// Needed for kusto tool in data.prompt.md
"azureMcp.enabledServices": [
"kusto"
"kusto" // Needed for kusto tool in data.prompt.md
Copy link

Copilot AI Nov 27, 2025

Choose a reason for hiding this comment

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

[nitpick] The comment should be placed on its own line above the array for better readability and consistency with typical JSON formatting conventions.

Suggested change
"kusto" // Needed for kusto tool in data.prompt.md
// Needed for kusto tool in data.prompt.md
"kusto"

Copilot uses AI. Check for mistakes.
@bpasero bpasero merged commit 6a44fc1 into main Nov 27, 2025
34 checks passed
@bpasero bpasero deleted the ben/detailed-marmoset branch November 27, 2025 13:58
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