Skip to content

fix: Changed 'bash' to 'command' in hooks.json#1019

Draft
parveen-dotnet wants to merge 1 commit intogithub:stagedfrom
parveen-dotnet:fix/hooks-bash-to-command
Draft

fix: Changed 'bash' to 'command' in hooks.json#1019
parveen-dotnet wants to merge 1 commit intogithub:stagedfrom
parveen-dotnet:fix/hooks-bash-to-command

Conversation

@parveen-dotnet
Copy link

@parveen-dotnet parveen-dotnet commented Mar 16, 2026

As per VS code docs, I don't find "bash" as a property in hooks, the best/suitable property is "command". Reference:
https://code.visualstudio.com/docs/copilot/customization/hooks#_hook-command-properties

Pull Request Checklist

  • I have read and followed the CONTRIBUTING.md guidelines.
  • I have read and followed the Guidance for submissions involving paid services.
  • My contribution adds a new instruction, prompt, agent, skill, or workflow 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, agent, skill, hooks or workflow with GitHub Copilot.
  • I have run npm start and verified that README.md is up to date.

Description

Changed 'bash' to 'command' in hooks.json
As per VS code docs, I don't find "bash" as a property in hooks, the best/suitable property is "command". Reference:
https://code.visualstudio.com/docs/copilot/customization/hooks#_hook-command-properties


Type of Contribution

  • New instruction file.
  • New prompt file.
  • New agent file.
  • New plugin.
  • New skill file.
  • New agentic workflow.
  • Update to existing instruction, prompt, agent, plugin, skill, hooks or workflow.
  • 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.

As per VS code docs, I don't find "bash" as a property in hooks, the best/suitable property is "command".
Reference:
https://code.visualstudio.com/docs/copilot/customization/hooks#_hook-command-properties
@aaronpowell
Copy link
Contributor

This is, unfortunately, a case where we have deviation between implementations, as Copilot CLI uses a slightly different schema to what VS Code uses.

Before we can action on this PR I'm going to have to work with the teams to get it aligned.

@parveen-dotnet
Copy link
Author

Thanks @aaronpowell for the clarification. I wasn't aware of the schema deviation between the Copilot CLI and VS Code implementations. I'll hold off on this change for now until the schemas are aligned. Please let me know once there's an update and I’ll be happy to revisit the PR.

@aaronpowell
Copy link
Contributor

We're actively tracking the changes internally on CLI so hopefully it won't be too long

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