Skip to content

Clarify directory review context and remove redundant CLAUDE.md - #62

Merged
georgeatparallel merged 2 commits into
mainfrom
clarify-directory-review
Oct 5, 2026
Merged

georgeatparallel merged 2 commits into
mainfrom
clarify-directory-review

Conversation

@georgeatparallel

@georgeatparallel georgeatparallel commented Oct 5, 2026 •

Copy link
Copy Markdown
Collaborator

Claude's directory flags the root CLAUDE.md because it is not loaded when the plugin is installed:

CLAUDE.md at the plugin root is not loaded as project context. To ship context with your plugin, use a skill (skills/<name>/SKILL.md) instead.

The file only repeats that limitation and points to the skills and agents. Remove it; it contains no development instructions or installed plugin behavior.

Also clarify the README's credential disclosure: the bundled MCP is anonymous, CLI skills authenticate separately to Parallel, and migration examples and synthetic scanner fixtures do not run when the plugin loads. Authentication and onboarding behavior are unchanged. This explanation helps reviewers assess the remaining credential holds; it does not claim to clear them.

Set documentationUrl to https://docs.parallel.ai/home so the directory links to Parallel documentation instead of inferring Claude's generic plugin guide from the README.

The documentation URL update passes Claude plugin validation and git diff --check.

Earlier validation: 37 Python tests pass, git diff --check passes, and Claude plugin validation passes without the root-file warning.

Portal warning before this change:

claude-root-file-warning

@georgeatparallel
georgeatparallel requested review from a team and sergei1152 October 5, 2026 17:51
@georgeatparallel
georgeatparallel merged commit 92694a2 into main Oct 5, 2026
1 check passed
@georgeatparallel
georgeatparallel deleted the clarify-directory-review branch October 5, 2026 21:44
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