Skip to content

Skills plugin#6

Closed
Kronos11 wants to merge 3 commits into
masterfrom
skills-plugin
Closed

Skills plugin#6
Kronos11 wants to merge 3 commits into
masterfrom
skills-plugin

Conversation

@Kronos11
Copy link
Copy Markdown

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/agents directory.

Plugin support and metadata:

  • Added .claude-plugin/plugin.json and .claude-plugin/marketplace.json files 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:

  • Updated README.md to document plugin installation steps, clarify the plugin's capabilities, and adjust all skill and agent links to reflect their new locations under the skills/ and agents/ directories. [1] [2] [3]

File and reference reorganization:

  • Changed all skill and agent documentation links throughout README.md and related files to use the new skills/ and agents/ paths, improving consistency and discoverability. [1] [2] [3]

Cleanup of agent documentation and files:

  • Removed the now-obsolete dispatch/agents/README.md, dispatch/agents/vscode-orchestration.txt, and dispatch/agents/vscode-orchestrator.agent.md files since agent references and documentation have been centralized and updated. [1] [2] [3]

Copilot AI review requested due to automatic review settings April 23, 2026 23:40
Copy link
Copy Markdown

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

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 a skills/skill.index* sub-index.
  • Updated docs and dispatch-skill references for the new top-level agents/ location; removed legacy dispatch/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.

Comment thread skill.index.sha256
Comment thread skills/skill.index.sha256
Comment thread skills/markdown-hygiene/spec.md.sha256
Comment thread README.md Outdated
Comment thread README.md
Comment thread skills/skill-auditing/instructions.txt Outdated
Copy link
Copy Markdown
Contributor

@electricessence electricessence left a comment

Choose a reason for hiding this comment

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

Please re-review — 3 SHA256 threads resolved (deprecated), 2 Worker fixes pushed (README + instructions.txt).

@electricessence
Copy link
Copy Markdown
Contributor

@copilot review

Copy link
Copy Markdown

Copilot AI commented May 3, 2026

@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 owner/repo and name@source-alias syntax, which resolves the copy-paste ambiguity concern. ✅

skills/skill-auditing/instructions.txt line 32 (Phase 3, item 4): The contradiction is resolved. Placing (standard mode only; --uncompressed mode: skip) immediately after the check name makes it unambiguous that all named finding patterns ("too much why", "essay not reference card", "prose conditionals") apply exclusively to standard mode — the --uncompressed path skips the entire item. The structure now matches items 7 and 10, which use the same -- mode: SKIP. pattern. ✅

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.

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.

4 participants