Skip to content

Conversation

@VeVarunSharma
Copy link
Contributor

Pull Request Checklist

  • I have read and followed the CONTRIBUTING.md guidelines.
  • [] My contribution adds a new instruction, prompt, or chat mode file in the correct directory.
  • The file follows the required naming convention.
  • The content is clearly structured and follows the example format.
  • I have tested my instructions, prompt, or chat mode with GitHub Copilot.
  • I have run npm start and verified that README.md is up to date.

Description

This update ensures that this agent file works with the the Copilot CLI. After attempting to use it with the Copilot CLI it was initially broken but it now works on the Copilot CLI mode!

Copilot CLI still does not have Plan mode natively so having a proper agent file that folks can use is important for now until a native plan mode feature is released. There's an Issue for that here: github/copilot-cli#783


Type of Contribution

  • New instruction file.
  • New prompt file.
  • New chat mode file.
  • New collection file.
  • Update to existing instruction, prompt, chat mode, or collection.
  • Other (please specify):

Additional Notes


By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.

Copilot AI review requested due to automatic review settings December 14, 2025 15:09
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 updates the plan.agent.md file to use a YAML-based tools format compatible with Copilot CLI, replacing the previous JSON array format. The update includes migrating tool names to use namespace prefixes (e.g., search/codebase, vscode/extensions) and removing the deprecated findTestFiles tool and its associated documentation reference.

Key Changes:

  • Converted tools specification from JSON array to YAML list format
  • Added namespace prefixes to tool names for Copilot CLI compatibility
  • Removed findTestFiles tool from both front matter and content

Comment on lines 1 to 14
---
description: "Strategic planning and architecture assistant focused on thoughtful analysis before implementation. Helps developers understand codebases, clarify requirements, and develop comprehensive implementation strategies."
name: "Plan Mode - Strategic Planning & Architecture"
tools: ["codebase", "extensions", "fetch", "findTestFiles", "githubRepo", "problems", "search", "searchResults", "usages", "vscodeAPI"]
tools:
- search/codebase
- vscode/extensions
- web/fetch
- web/githubRepo
- read/problems
- azure-mcp/search
- search/searchResults
- search/usages
- vscode/vscodeAPI
---
Copy link

Copilot AI Dec 14, 2025

Choose a reason for hiding this comment

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

The front matter is missing the model field. According to AGENTS.md, it's strongly recommended to specify the model field for agent files. Consider adding a line like model: GPT-4.1 (or the appropriate model this agent is optimized for) after the name field.

Copilot generated this review using guidance from repository custom instructions.
### Information Gathering Tools

- **Codebase Exploration**: Use the `codebase` tool to examine existing code structure, patterns, and architecture
- **Search & Discovery**: Use `search` and `searchResults` tools to find specific patterns, functions, or implementations across the project
Copy link

Copilot AI Dec 14, 2025

Choose a reason for hiding this comment

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

The content references the search tool on line 33, but this tool is not listed in the front matter tools array. Looking at other agent files like janitor.agent.md, both search and namespaced search tools like search/searchResults should be included separately. Consider adding - search to the tools list in the front matter, or update the content to reference the specific search tool that's available.

Copilot uses AI. Check for mistakes.
@aaronpowell aaronpowell merged commit 8a1a571 into github:main Dec 14, 2025
8 checks passed
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.

2 participants