[mattmillerai]

ELI5

A product listing has a few blanks for the fine print: terms, a support contact, where the source lives, and search keywords. The Claude plugin directory reads those from the plugin's manifest, and ours left four of them empty. This fills them in.

Motivation

The comfy-cloud plugin is being prepared for Anthropic's Claude plugin directory. The directory builds its listing from plugin.json, and a validation run in the developer portal showed terms of service, support, repository, and keywords as not set. #46 — feat(comfy-cloud): add Agent Skills for the Claude directory (BE-25899) already added the README, LICENSE, icon, and the documentation and privacy fields, so only these four remain.

Provenance

• Authored by: interactive session
• Verified:
  • Plugin manifest validation: passed on the branch head
  • Terms URL and privacy URL both return 200 (/privacy redirects to /privacy-policy/)
  • Diff against main: plugin.json only
  • Not run: the directory portal's own Validate step against this branch

Reviewer context

• Type: hygiene, listing metadata only. No skill, command, or MCP behavior changes.
• Slots into: follows feat(comfy-cloud): add Agent Skills for the Claude directory (BE-25899) #46 — feat(comfy-cloud): add Agent Skills for the Claude directory (BE-25899)
• Not in scope: MCP tool annotations and tool-description wording, which live in the MCP server, not this repo

Summary

• plugin.json: add termsOfServiceUrl, supportUrl (mailto:support@comfy.org), repository, and keywords; bump version to 0.2.1
• This branch originally also added a README, LICENSE, and icon. feat(comfy-cloud): add Agent Skills for the Claude directory (BE-25899) #46 landed its own versions first, so the merge keeps main's and drops the duplicates.

Test plan

• Re-run Validate in the directory portal and confirm the Listing details step shows terms, support, repository, and keywords
• /plugin install comfy-cloud@comfy-skills still installs and /mcp still signs in

[coderabbitai]

Warning

Review limit reached

• Run on-demand review

_{This review includes 1 billable file and costs up to $0.25.}

• Ask an admin to make reviews automatic

Open in CodeRabbit

Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing.

Or wait 36 minutes for your next included review.

Check out review usage here.

View limit details

Limit details: You’ve used all 2 included reviews currently available. Your 83 included PR review attempts over the past 7 days set your current allowance at 2 reviews per hour.

Learn how review limits work.

Review configuration:

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: ASSERTIVE
• Plan: Team
• Run ID: 90a8b667-a245-4013-9c18-5371ac7a433b

Commits

Reviewing files that changed from the base of the PR and between 8c3bb20 and 55addd2.

Files selected for processing: 1

• claude-code/.claude-plugin/plugin.json

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

[vqt123]

Starting review at 2f54d27.

[vqt123]

Approving at 2f54d27.

Should fix

• Words: People assessing local execution are misled by "runs no local code, scripts, or hooks" in claude-code/README.md:41. Evidence (read, not run): commands/generate-image.md:33 directs Claude to execute curl and open the downloaded file; commands/clone-voice.md:36 directs local ffmpeg processing. These commands also exist at the merge base, which made no such assurance. Say there are no bundled executables or automatic hooks, and disclose command-directed shell actions.
• Contract: Users expecting to retrieve every output later from their account are misled by claude-code/README.md:42. Evidence (read, not run): an Ideogram request with a negative prompt declines library storage in comfy-cloud-mcp-server/src/partner/client/registry/image.ts:1995; src/tools-execution.ts:6667 takes direct dispatch, whose response explicitly says it was not saved to the asset library at line 4388. The merge base made no storage guarantee. Qualify this sentence and explain that some partner outputs require downloading.

Clean: Correctness, Resilience, Access, Vendor, Rollout, Rules, Cause.
Not covered: Contract/Vendor acceptance: claude plugin validate ./claude-code, then the directory portal's Validate and Listing details steps against this branch; Access/Rollout installation and OAuth: /plugin install comfy-cloud@comfy-skills, then /mcp.
Checked: README claims against existing commands and server source; marketplace source and sibling manifest; license against the root MIT license. Source and referenced tests were read, not run.

_{~632k effective tokens for this review (3.1M raw; cache reads weighted 0.1x, cache writes 1.25-2x)}

[deepme987]

LGTM

[vqt123]

Reviewed at 55addd2 (since 2f54d27): kept main's listing files, added four metadata fields, and bumped the version.

Fixed: Both earlier README findings. The replacement README removes the no-local-execution assurance and describes short-lived output links instead of promising account storage.
Still open: none.

Clean: Correctness, Resilience, Access.
Not covered: Contract/Vendor manifest acceptance: claude plugin validate ./claude-code, then the directory portal's Validate and Listing details steps; Access/Rollout installation and OAuth: /plugin install comfy-cloud@comfy-skills, then /mcp.
Checked: Read the manifest against the merge base, marketplace registration, sibling plugin, and replacement README. Read, not run.

_{~171k effective tokens for this review (660k raw; cache reads weighted 0.1x, cache writes 1.25-2x)}