Skip to content

Adopt Agent Plugins v1 package layout - #757

Closed
matt-ramotar wants to merge 2 commits into
MobileNativeFoundation:store6from
matt-ramotar:plugins/canonical-agent-plugins
Closed

Adopt Agent Plugins v1 package layout#757
matt-ramotar wants to merge 2 commits into
MobileNativeFoundation:store6from
matt-ramotar:plugins/canonical-agent-plugins

Conversation

@matt-ramotar

Copy link
Copy Markdown
Collaborator

Summary

  • Convert plugins/store to an Agent Plugins v1.0.0 package with root plugin.json (0.3.0) and remove the Claude marketplace plus .claude-plugin manifests.
  • Move contributor skills from .claude/skills/ into plugins/internal/documentation as a second portable plugin (0.1.0).
  • Point AGENTS.md and the store plugin README at the new paths. Claude Code /plugin marketplace add for this repo no longer applies.

Test plan

  • Confirm plugins/store/plugin.json and plugins/internal/documentation/plugin.json use https://agent-plugins.org/schemas/1.0.0/plugin.schema.json and only permitted top-level fields.
  • Confirm each skill is an immediate child of skills/ whose SKILL.md frontmatter name matches the directory name.
  • Confirm .claude-plugin/ and .claude/skills/ are gone, and .claude/launch.json is unchanged.
  • Load plugins/store in an Agent Plugins client and confirm building-a-store6-data-layer and migrating-to-store6 are discovered.
  • Load plugins/internal/documentation and confirm documentation-discipline and code-documentation are discovered.

Made with Cursor

matt-ramotar and others added 2 commits August 16, 2026 17:05
Root plugin.json is the portable manifest. Claude marketplace and
.claude/skills paths are client-specific and block other clients.

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-project-automation github-project-automation Bot moved this to 🆕 Triage in Store Roadmap Aug 16, 2026
@github-project-automation github-project-automation Bot moved this from 🆕 Triage to ✅ Done in Store Roadmap Aug 16, 2026

@cursor cursor Bot 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.

Cursor Bugbot has reviewed your changes using default effort and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 5512c45. Configure here.

comm -12 \
<(LC_ALL=C sort "${source_list}") \
<(git diff --name-only origin/main...HEAD | LC_ALL=C sort)
)"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Docs sync checks wrong base branch

High Severity

The docs-sync guard always diffs against origin/main, but this workflow also runs on pull requests targeting store6. Relative to the wrong base, the guard can demand docs-sync-ack for docs the PR did not change, or miss docs it did change against store6.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 5512c45. Configure here.

Comment thread .github/workflows/ci.yml

- name: Set up JDK 17
uses: actions/setup-java@v5
uses: actions/setup-java@v4

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

CI downgrades to deprecated Node 20

Medium Severity

This commit replaces actions/checkout@v6 and actions/setup-java@v5 with the v4 actions that still run on Node 20. GitHub is removing Node 20 from hosted runners on 2026-09-16, so these jobs are headed for runner breakage shortly after merge.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 5512c45. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

1 participant