feat(release): narrow plugin vendor to hooks+skills, idempotent, marketplace-owned manifest #453

Merged

hexajon merged 1 commit from feat/443-plugin-assemble-narrow into main 2026-08-25 21:39:31 +00:00

Conversation 0 Commits 1 Files changed 7 +360 -327

hexajon commented 2026-08-25 21:39:30 +00:00

Owner

Copy link

Summary

The shipped Claude Code plugin is now hooks + skills only, and the marketplace repo (sixfold-space/claude-plugins) owns the plugin metadata. This reworks the madtea side to match.

• assemble-plugin-artifact.sh vendors only the madtea-tracked content (hooks, skills, LICENSE, NOTICE) into plugins/madtea/, replacing only those paths. It no longer copies, stamps, or version-bumps plugin.json, marketplace.json, or README - those belong to the marketplace repo. It refuses to scaffold the plugin dir if it is absent.
• No plugin version field. Claude Code keys update detection off the marketplace commit hash, so the vendor step is idempotent: it branches, commits, and opens a PR only when the content actually changed. An unchanged release is a clean no-op, so no phantom plugin update reaches users.
• Removed stamp_plugin_json_version, bump_marketplace_version, the versioned README generation, and the now-unused artifact semver.
• Deleted the source .claude-plugin/ directory.
• Updated the assemble drift-guard test: metadata-not-written assertions, the idempotent no-op path (via a local fake-marketplace seam), and the change-detected path. Refreshed release.sh and the contributing docs.
• ADR 0010: dated inline amendment recording marketplace-owned manifest, no version (hash-based updates), no bundled server, and the idempotent assemble. Body preserved.

The marketplace side is already merged (the claude-plugins narrowing). Full gate green.

Refs #450, #437.

Closes #[443]

## Summary The shipped Claude Code plugin is now hooks + skills only, and the marketplace repo (sixfold-space/claude-plugins) owns the plugin metadata. This reworks the madtea side to match. - `assemble-plugin-artifact.sh` vendors only the madtea-tracked content (hooks, skills, LICENSE, NOTICE) into `plugins/madtea/`, replacing only those paths. It no longer copies, stamps, or version-bumps `plugin.json`, `marketplace.json`, or `README` - those belong to the marketplace repo. It refuses to scaffold the plugin dir if it is absent. - No plugin `version` field. Claude Code keys update detection off the marketplace commit hash, so the vendor step is idempotent: it branches, commits, and opens a PR only when the content actually changed. An unchanged release is a clean no-op, so no phantom plugin update reaches users. - Removed `stamp_plugin_json_version`, `bump_marketplace_version`, the versioned README generation, and the now-unused artifact `semver`. - Deleted the source `.claude-plugin/` directory. - Updated the assemble drift-guard test: metadata-not-written assertions, the idempotent no-op path (via a local fake-marketplace seam), and the change-detected path. Refreshed `release.sh` and the contributing docs. - ADR 0010: dated inline amendment recording marketplace-owned manifest, no version (hash-based updates), no bundled server, and the idempotent assemble. Body preserved. The marketplace side is already merged (the claude-plugins narrowing). Full gate green. Refs #450, #437. Closes #[443]

hexajon added 1 commit 2026-08-25 21:39:30 +00:00

feat(release): narrow plugin vendor to hooks+skills, idempotent, marketplace-owned manifest ... 90c68c2a0a

Closes #[443]

hexajon merged commit 90c68c2a0a into main 2026-08-25 21:39:31 +00:00

hexajon referenced this pull request 2026-08-25 21:39:58 +00:00

decision: plugin distribution contradicts the official docs (plugin.json version is authoritative; a separate marketplace repo is not needed for one plugin) #437

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

breaking

Breaking change - flag for release notes

  

bug

Something is not working

  

documentation

Documentation only

  

enhancement

New feature

  

epic

Tracking epic with dependency children

  

good first issue

Interested in contributing? Get started here.

  

help wanted

Maintainer would welcome outside help

  

refactoring

Code restructuring without behavior changes

  

resolution/duplicate

Closed as duplicate of another issue

  

resolution/invalid

Closed as invalid or not reproducible

  

resolution/wontfix

Closed as won't fix

  

security

Security issue or hardening

  

severity/critical

Security severity: critical

  

severity/high

Security severity: high

  

severity/low

Security severity: low

  

severity/medium

Security severity: medium

  

status/abandoned

Someone started, then walked away - up for grabs

  

status/blocked

Blocked by another issue or an external dependency

  

status/needs-decision

Waiting on a maintainer decision; workable once decided

  

status/needs-info

Waiting on reporter / more information before work can start

  

status/needs-verification

Done except verification in a live interactive session

  

testing

Test suite / CI infrastructure work

  

upstream

Related to an upstream repository, already reported there

No labels

breaking

bug

documentation

enhancement

epic

good first issue

help wanted

refactoring

resolution/duplicate

resolution/invalid

resolution/wontfix

security

severity/critical

severity/high

severity/low

severity/medium

status/abandoned

status/blocked

status/needs-decision

status/needs-info

status/needs-verification

testing

upstream

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

sixfold-space/madtea!453

No description provided.