Skills plugin#6
Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Packages the repo for Claude Code plugin/marketplace installation by adding manifests, moving skill/agent references to the new skills/ + agents/ layout, and rebuilding the skill-index cascade alongside some documentation/spec cleanups.
Changes:
- Added Claude Code plugin + marketplace manifests under
.claude-plugin/. - Rebuilt skill index artifacts to make
skills/the root node and added askills/skill.index*sub-index. - Updated docs and dispatch-skill references for the new top-level
agents/location; removed legacydispatch/agents/*docs/files.
Reviewed changes
Copilot reviewed 28 out of 320 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| skills/skill.index.sha256 | Adds SHA stamp for the new skills/ index node |
| skills/skill.index.md | Adds human-readable index for skills directory |
| skills/skill.index | Adds searchable index keywords for skills directory |
| skills/skill-auditing/instructions.uncompressed.md.sha256 | Adds stamp after updating uncompressed instructions |
| skills/skill-auditing/instructions.uncompressed.md | Removes a named finding pattern from conciseness checks |
| skills/skill-auditing/instructions.txt.sha256 | Adds stamp after updating compressed instructions |
| skills/skill-auditing/instructions.txt | Updates dispatch/uncompressed-mode rules and conciseness criteria |
| skills/markdown-hygiene/spec.md.sha256 | Adds stamp for moved/updated markdown-hygiene spec |
| skills/markdown-hygiene/spec.md | Updates constraints language around external tooling |
| skills/dispatch/uncompressed.md | Updates agent-file references to new agents/ paths |
| skills/dispatch/spec.md | Updates dispatch skill spec references to new agent file locations |
| skills/dispatch/installation.md | Updates installation paths/source references for moved agent files |
| skills/dispatch/agents/README.md | Adds a pointer README explaining moved agent file locations |
| skills/dispatch/SKILL.md | Updates runtime “Related” link to new agent file locations |
| skill.index.sha256 | Updates root index stamp after restructuring |
| skill.index.md | Changes root index to point to the new skills/ subtree conceptually |
| skill.index | Changes root index keywords to reference skills/ node |
| skill-auditing/instructions.uncompressed.md.sha256 | Removes old root-level stamp after move to skills/ |
| skill-auditing/instructions.txt.sha256 | Removes old root-level stamp after move to skills/ |
| markdown-hygiene/spec.md.sha256 | Removes old root-level stamp after move to skills/ |
| docs/claude-code-marketplace.md | Adds packaging spec for Claude Code marketplace/plugin layout |
| docs/claude-code-marketplace-plan.md | Adds detailed implementation plan for the packaging migration |
| dispatch/agents/vscode-orchestrator.agent.md | Removes obsolete orchestrator agent definition |
| dispatch/agents/vscode-orchestration.txt | Removes obsolete orchestration notes file |
| dispatch/agents/README.md | Removes legacy dispatch/agents README after centralizing agent docs |
| README.md | Updates links for new skills/ layout and adds plugin install instructions |
| .claude-plugin/plugin.json | Adds Claude Code plugin manifest |
| .claude-plugin/marketplace.json | Adds Claude Code marketplace manifest |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…pressed conciseness contradiction
electricessence
left a comment
There was a problem hiding this comment.
Please re-review — 3 SHA256 threads resolved (deprecated), 2 Worker fixes pushed (README + instructions.txt).
|
@copilot review |
Review complete. All three previously unresolved threads are addressed: README.md Quick Start (threads on lines 33–35 and 45–47): The Quick Start now clearly distinguishes two paths — direct-reference clone and plugin install — so users won't look for skills at the repo root. The install code block includes comments explaining the
Three SHA256 format threads: Appropriately deprecated/resolved by the author; checksum regeneration is a post-merge governance gate, not a PR artifact. No new issues found. LGTM. |
This pull request introduces plugin metadata files to support installation as a Claude Code plugin, updates the documentation to reflect the new plugin structure, and reorganizes agent and skill references for improved clarity and maintainability. It also removes now-redundant agent documentation and files from the
dispatch/agentsdirectory.Plugin support and metadata:
.claude-plugin/plugin.jsonand.claude-plugin/marketplace.jsonfiles to define plugin metadata, enabling the repository to be installed as a Claude Code plugin and registered in the plugin marketplace. [1] [2]Documentation updates:
README.mdto document plugin installation steps, clarify the plugin's capabilities, and adjust all skill and agent links to reflect their new locations under theskills/andagents/directories. [1] [2] [3]File and reference reorganization:
README.mdand related files to use the newskills/andagents/paths, improving consistency and discoverability. [1] [2] [3]Cleanup of agent documentation and files:
dispatch/agents/README.md,dispatch/agents/vscode-orchestration.txt, anddispatch/agents/vscode-orchestrator.agent.mdfiles since agent references and documentation have been centralized and updated. [1] [2] [3]