Skip to content

test(claude-plugin): add structural parity tests mirroring test_codex_plugin#963

Merged
RaghavChamadiya merged 1 commit into
repowise-dev:mainfrom
Ayush7614:test/claude-plugin-structure
Jul 22, 2026
Merged

test(claude-plugin): add structural parity tests mirroring test_codex_plugin#963
RaghavChamadiya merged 1 commit into
repowise-dev:mainfrom
Ayush7614:test/claude-plugin-structure

Conversation

@Ayush7614

Copy link
Copy Markdown
Contributor

Summary

  • Add tests/unit/cli/test_claude_plugin.py with five structural tests mirroring test_codex_plugin.py.
  • Cover manifest sanity, MCP config, hooks vs installer matchers, all six skills with valid frontmatter, and marketplace version sync.

Motivation

The Codex plugin already has structural regression tests; the Claude Code plugin had none. These tests catch drift in bundled hooks, skills, and marketplace metadata before it ships.

Test plan

  • pytest tests/unit/cli/test_claude_plugin.py — all 5 tests pass

…_plugin

Lock in manifest, MCP, hooks, skills, and marketplace version sync for the
Claude Code plugin so drift is caught the same way as the Codex plugin.

@RaghavChamadiya RaghavChamadiya left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, this is a nice fill for a real gap. What I like is that it isn't a blind copy of the Codex test: it tracks the Claude plugin's actual shape, the mcpServers-nested .mcp.json, the PostToolUse/SessionStart hook rows checked against claude_config._AUGMENT_MATCHER / _SESSION_START_MATCHER, and the marketplace version-sync assertion.

Pulled it and confirmed 5/5 pass locally, and the skill-set assertion matches what's actually on disk. Good regression net for catching plugin drift before it ships. Merging.

@RaghavChamadiya
RaghavChamadiya merged commit 7678c57 into repowise-dev:main Jul 22, 2026
7 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