Skip to content

catalog: list Matt Pocock skills as a whole plugin - #95

Merged
trent-j merged 3 commits into
mainfrom
trent-j/catalog-mattpocock-plugin
Oct 28, 2026
Merged

trent-j merged 3 commits into
mainfrom
trent-j/catalog-mattpocock-plugin

Conversation

@trent-j

@trent-j trent-j commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Summary

List Matt Pocock Skills as the whole Claude-layout plugin rather than exposing its bundled skills as standalone catalog resources.

  • Add the parent plugin with application/vnd.anthropic.claude-plugin+json, a distinct identifier, and the full .claude-plugin/plugin.json URL plus matching source metadata.
  • Remove grill-me and grilling, which the plugin explicitly declares.
  • Remove the obsolete skills/in-progress/ references for to-questionnaire and wizard; their current locations are also declared by the plugin.
  • Move the existing reviewed project-logo declaration to the parent identifier. The asset, immutable URL, digest, and approval provenance are unchanged.
  • Leave the other 19 Matt Pocock source records unchanged, regenerate the ingestion catalog, and add five regression tests.

Evidence and scope

Verified against the upstream manifest at 59db609e: plugin mattpocock-skills, version 1.3.1, declares 27 skills. The catalog reference follows main, consistent with the existing source-reference workflow; no fixed version is introduced.

This follows #93 and preserves the approved logo from #94. No service, schema, ingestion-allowlist, or artwork changes are included. The existing ARD compatibility envelope is unchanged; ARD v0.91 (Proposal) was consulted.

Validation

Run in an isolated virtual environment with the repository's pinned requirements:

  • python3 -m unittest discover -s tests -p test_mattpocock_plugin_refs.py -v — 5 passed.
  • python3 -m unittest discover -s tests — 51 passed.
  • Python compilation of all 7 script/test modules — passed.
  • python3 scripts/generate_ai_catalog.py and python3 scripts/generate_ai_catalog.py --check — passed, 1,705 entries at the authored commit.
  • git diff --check — passed.
  • Exact baseline comparison of authored commit 97b1ffc: four generated entries removed, one plugin added, all 1,704 other generated entries unchanged; all 19 retained source records are byte-identical, and icon provenance is unchanged.

Generated-catalog automation

The normal Catalog workflow subsequently added bot commit 28dc38f, refreshing imported MCP Registry timestamps, star counts, and three server-version references. Verified that all of that commit's changes are registry imports: it does not alter contributor-managed resources, the new plugin, the retired skills, or the icon mapping. The authored change remains independently reviewable in 97b1ffc.

Catalog CI passed on the updated head, including the full test suite and catalog generation.

Publication

After merge, ingestion, enrichment, and reconciliation must complete before the plugin appears in production and the retired skill listings are withdrawn. Verify publication under the plugin media type rather than treating ingestion alone as completion. No production jobs were triggered or production configuration changed as part of this PR.

Publish the root Claude plugin instead of its bundled skill listings, including the obsolete questionnaire and wizard references. Preserve the other catalog records and move the approved project icon to the parent plugin.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI auto review requested due to automatic review settings October 28, 2026 18:34

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The generated ai-catalog.json includes unrelated MCP registry version, timestamp, and star-count changes, making the PR noisy and non-reproducible instead of matching the stated exact baseline.

1 open issue
What changed in this PR

This pull request consolidates Matt Pocock’s bundled skills into a single Claude-layout plugin catalog entry, removes superseded standalone entries, preserves the approved icon metadata, and adds regression coverage.

Changes:

  • Adds the mattpocock-skills plugin entry targeting .claude-plugin/plugin.json.
  • Removes four standalone bundled or obsolete skill entries.
  • Moves the reviewed icon declaration to the parent plugin identifier.
  • Adds five regression tests.
  • The generated ai-catalog.json also includes unrelated MCP registry refreshes; these should be removed or regenerated from the intended cached snapshot.
File Summary
tests/​test_mattpocock_plugin_refs.py Adds regression coverage for plugin metadata, retired and retained skills, icon mapping, and generated output.
catalog/​mattpocock/​wizard.json Removes the obsolete standalone wizard entry.
catalog/​mattpocock/​to-questionnaire.json Removes the obsolete standalone to-questionnaire entry.
catalog/​mattpocock/​mattpocock-skills.json Adds the parent Claude plugin catalog entry.
catalog/​mattpocock/​grilling.json Removes the bundled grilling entry.
catalog/​mattpocock/​grill-me.json Removes the bundled grill-me entry.
catalog-icons.json Reassigns the approved icon declaration to the parent plugin identifier.
ai-catalog.json Publishes the plugin replacement and removals, but also contains unrelated MCP metadata drift that makes the generated diff noisy and non-reproducible.

Review effort: Lite (auto)

Comment thread ai-catalog.json
@trent-j
trent-j merged commit 129ead2 into main Oct 28, 2026
2 checks passed
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