-
Notifications
You must be signed in to change notification settings - Fork 12
update tools list in various prompt and chatmodes files #9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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 invalid tool keys from the tools arrays across multiple prompt and chatmode files. The removed tools were flagged as unknown and are not available in agent mode according to GitHub Copilot's current tool list.
Key changes:
- Removed deprecated/invalid tool names from tools arrays in prompt files
- Removed deprecated/invalid tool names from tools arrays in chatmode files
- Maintained consistent tool arrays across similar file types
Reviewed Changes
Copilot reviewed 46 out of 46 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
prompts/update-markdown-file-index.prompt.md | Removed 'runTests' tool from tools array |
prompts/update-avm-modules-in-bicep.prompt.md | Removed 'think', 'todos', 'bicepschema', 'azure_get_schema_for_Bicep' tools |
prompts/suggest-github-copilot-prompts.prompt.md | Removed 'runTests', 'github' tools |
prompts/suggest-github-copilot-chatmodes.prompt.md | Removed 'runTests', 'github' tools |
prompts/suggest-awesome-github-copilot-prompts.prompt.md | Removed 'runTests', 'github' tools |
prompts/suggest-awesome-github-copilot-chatmodes.prompt.md | Removed 'runTests', 'github' tools |
prompts/review-aac-multitenant-service-specific-guidance.prompt.md | Removed 'think', 'todos', 'Microsoft Docs' tools |
prompts/next-intl-add-language.prompt.md | Removed 'writeTest' tool |
prompts/my-pull-requests.prompt.md | Removed GitHub-specific API tools like 'github', 'get_me', 'get_pull_request' etc. |
prompts/my-issues.prompt.md | Removed GitHub-specific API tools like 'github', 'get_issue', 'get_me' etc. |
prompts/create-spring-boot-kotlin-project.prompt.md | Removed 'runTests' tool |
prompts/create-spring-boot-java-project.prompt.md | Removed 'runTests' tool |
prompts/create-github-issues-for-unmet-specification-requirements.prompt.md | Removed GitHub API tools like 'github', 'create_issue', 'search_issues', 'update_issue' |
prompts/create-github-issues-feature-from-implementation-plan.prompt.md | Removed GitHub API tools like 'github', 'create_issue', 'search_issues', 'update_issue' |
prompts/create-github-issue-feature-from-specification.prompt.md | Removed GitHub API tools like 'github', 'create_issue', 'search_issues', 'update_issue' |
prompts/create-github-action-workflow-specification.prompt.md | Removed deprecated tools like 'runInTerminal2', 'microsoft.docs.mcp', 'github', 'Microsoft Docs' |
Multiple chatmode files | Removed various deprecated tools including 'github', 'microsoft.docs.mcp', 'Microsoft Docs', 'think', 'todos', database-specific tools, etc. |
mode: 'agent' | ||
description: 'Update Azure Verified Modules (AVM) to latest versions in Bicep files.' | ||
tools: ['codebase', 'think', 'changes', 'fetch', 'searchResults', 'todos', 'editFiles', 'search', 'runCommands', 'bicepschema', 'azure_get_schema_for_Bicep'] | ||
tools: ['codebase', 'changes', 'fetch', 'searchResults','editFiles', 'search', 'runCommands'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing space after comma between 'searchResults' and 'editFiles' in the tools array. This should be consistent with other spacing in the array.
tools: ['codebase', 'changes', 'fetch', 'searchResults','editFiles', 'search', 'runCommands'] | |
tools: ['codebase', 'changes', 'fetch', 'searchResults', 'editFiles', 'search', 'runCommands'] |
Copilot uses AI. Check for mistakes.
description: 'Suggest relevant GitHub Copilot prompt files from a GitHub repository based on current repository context and chat history, avoiding duplicates with existing prompts in this repository. | ||
'tools: ['changes', 'codebase', 'editFiles', 'fetch', 'findTestFiles', 'githubRepo', 'new', 'openSimpleBrowser', 'problems', 'runCommands', 'runTasks', 'runTests', 'search', 'searchResults', 'terminalLastCommand', 'terminalSelection', 'testFailure', 'usages', 'vscodeAPI', 'github'] | ||
'tools: ['changes', 'codebase', 'editFiles', 'fetch', 'findTestFiles', 'githubRepo', 'new', 'openSimpleBrowser', 'problems', 'runCommands', 'runTasks', 'search', 'searchResults', 'terminalLastCommand', 'terminalSelection', 'testFailure', 'usages', 'vscodeAPI'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The description field spans multiple lines and ends with a newline character before the closing quote, which is inconsistent with single-line descriptions in other files. The description should be contained on a single line.
Copilot uses AI. Check for mistakes.
…available in tools options in agent mode
beaadc4
to
b21929a
Compare
Removed tool keys from tools array. These tool keys were flagged unknown as these aren't available in tools list in agent mode.
Pull Request
Type of Change
Asset Details
For New/Updated Chat Modes
For New/Updated Prompt Files
Description
Testing
Examples
Sample Usage
Expected Behavior
Quality Checklist
${input:VariableName}
)Compatibility and Standards
Community Value
Related Issues
Additional Notes
This change is