Skip to content

Comments

Removing force add of commands folder#811

Merged
aaronpowell merged 1 commit intostagedfrom
aaronpowell-patch-1
Feb 25, 2026
Merged

Removing force add of commands folder#811
aaronpowell merged 1 commit intostagedfrom
aaronpowell-patch-1

Conversation

@aaronpowell
Copy link
Contributor

Prompts aren't supported anymore so we don't need to try and add them.

Pull Request Checklist

  • I have read and followed the CONTRIBUTING.md guidelines.
  • 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, or workflow with GitHub Copilot.
  • I have run npm start and verified that README.md is up to date.

Description


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, 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.

Prompts aren't supported anymore so we don't need to try and add them.
Copilot AI review requested due to automatic review settings February 25, 2026 04:57
@aaronpowell aaronpowell merged commit a653f34 into staged Feb 25, 2026
4 checks passed
@aaronpowell aaronpowell deleted the aaronpowell-patch-1 branch February 25, 2026 04:58
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 force-add of plugins/*/commands/ from the publish workflow, reflecting the deprecation of commands/prompts in the plugin system. The repository now only supports agents and skills as valid plugin components.

Changes:

  • Removed plugins/*/commands/ from the force-add step in the publish workflow
Comments suppressed due to low confidence (1)

.github/workflows/publish.yml:52

  • While removing the force-add of plugins/*/commands/ is correct since commands/prompts are no longer supported, there are several other files in the codebase that still reference "commands" and should be cleaned up for consistency:
  1. eng/materialize-plugins.mjs:128 - Still includes "commands" in the rewrite loop
  2. eng/clean-materialized-plugins.mjs:8 - Still includes "commands" in MATERIALIZED_DIRS array
  3. eng/generate-website-data.mjs:528 - Still maps data.commands to prompt kind
  4. eng/update-readme.mjs:785,829 - Still includes (plugin.commands || []).length in item counts

These leftover references could cause confusion or unexpected behavior if someone later tries to use commands. Consider cleaning these up in this PR or creating a follow-up task to remove all commands-related code.

          git add -f plugins/*/agents/ plugins/*/skills/

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.

1 participant