Skip to content

ci: validate plugins in CI, and land the Codex plugin - #4

Merged
minhthanhdang merged 6 commits into
mainfrom
minh/rm-18445
Sep 17, 2026
Merged

minhthanhdang merged 6 commits into
mainfrom
minh/rm-18445

Conversation

@minhthanhdang

@minhthanhdang minhthanhdang commented Sep 14, 2026 •

Copy link
Copy Markdown
Collaborator
🚥 Resolves RM-18445
🚥 Resolves RM-18419

🧰 Changes

Adds a GitHub Actions workflow that runs Claude Code's built-in plugin validation on every PR and push to main, covering both the plugin folder and the root marketplace. It runs in strict mode, so warnings fail the check too. A broken manifest, marketplace entry, or skill frontmatter now blocks the merge instead of surfacing at install time.

One gap worth knowing: the validator does not check that a marketplace entry's source path exists, so a marketplace pointing at a missing folder still passes.

Also carries the Codex plugin and the root marketplace, so codex plugin marketplace add readmeio/agent-plugins then codex plugin add readme@readme installs it. That work was reviewed and approved in #3, which landed on the wrong base and never reached main — this re-targets it rather than reopening.

🧬 QA & Testing

  • Summary Report: n/a. Validation fails on a skill with no frontmatter and passes once it is restored. Codex install verified in an isolated CODEX_HOME on 0.154.0: plugin shows installed, enabled at 1.0.0 with its skills, MCP config, and icons.

(I merged #3 into wrong branch so this PR carries #3 changes as well)

@minhthanhdang
minhthanhdang changed the base branch from minh/rm-18349 to minh/rm-18419 September 14, 2026 21:47
@minhthanhdang
minhthanhdang added this pull request to stack #5 September 14, 2026 21:47
@minhthanhdang
minhthanhdang removed this pull request from stack #5 September 14, 2026 21:47
@minhthanhdang
minhthanhdang changed the base branch from minh/rm-18419 to minh/rm-18349 September 14, 2026 21:47
@greptile-apps

greptile-apps Bot commented Sep 14, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

The PR appears safe to merge; the incremental change fully fixes the remaining invalid skill handoff and introduces no new actionable issue.

Reviews (8) · Last reviewed commit: "docs(onboarding): drop the pointer to a ..."

Comment thread .github/workflows/validate-claude-plugin.yml Outdated
greptile-apps[bot]
greptile-apps Bot previously approved these changes Sep 14, 2026
@greptile-apps
greptile-apps Bot dismissed their stale review September 14, 2026 22:58

Dismissed because a newer commit was pushed; Greptile will re-review the current head.

greptile-apps[bot]
greptile-apps Bot previously approved these changes Sep 14, 2026
@greptile-apps
greptile-apps Bot dismissed their stale review September 16, 2026 04:35

Dismissed because a newer commit was pushed; Greptile will re-review the current head.

greptile-apps[bot]
greptile-apps Bot previously approved these changes Sep 16, 2026
@minhthanhdang
minhthanhdang force-pushed the minh/rm-18445 branch 2 times, most recently from d8e30bc to a1feeef Compare September 17, 2026 03:03
@minhthanhdang
minhthanhdang changed the base branch from minh/rm-18349 to main September 17, 2026 03:13
@greptile-apps
greptile-apps Bot dismissed their stale review September 17, 2026 03:13

Dismissed because a newer commit was pushed; Greptile will re-review the current head.

@minhthanhdang minhthanhdang changed the title ci: validate the Claude plugin on pull requests and main ci: validate plugins in CI, and land the Codex plugin Sep 17, 2026
Comment thread codex/skills/onboarding/SKILL.md Outdated
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@minhthanhdang
minhthanhdang merged commit 38eebbe into main Sep 17, 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.

2 participants