Skip to content

docs: reorganize README and soften framing#35

Merged
hybridindie merged 1 commit intomainfrom
worktree-plugin-packaging
Mar 13, 2026
Merged

docs: reorganize README and soften framing#35
hybridindie merged 1 commit intomainfrom
worktree-plugin-packaging

Conversation

@hybridindie
Copy link
Owner

Summary

  • Reorganize README to highlight Tools and Security upfront, push Setup/Configuration/Docker toward the end
  • Slim Quick Start from ~65 to ~20 lines
  • Consolidate scattered security sections into one cohesive section
  • Soften "Why this exists" to focus on ComfyUI's API gaps rather than criticizing other servers
  • Fix security hook false positives and align Docker image references (from PR feat: Claude Code plugin packaging and multi-app setup docs #34 review)

Test plan

  • Verify README renders correctly on GitHub
  • All internal anchor links resolve

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings March 13, 2026 15:26
Package the MCP server as a Claude Code plugin with slash commands,
contextual skills, and a security hook. Add configuration snippets
for 8 client environments (Claude Code, Claude Desktop, VS Code,
Cursor, Windsurf, Continue.dev, OpenCode, Open WebUI).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Reorganizes end-user documentation and companion Claude Code assets to present ComfyUI-MCP’s tools/security posture more clearly, while adding/adjusting hook + skill files used by the Claude Code plugin workflow.

Changes:

  • Restructured README.md (shorter Quick Start, consolidated Security section, expanded multi-client Setup snippets, aligned GHCR image refs).
  • Added Claude/skill documentation under skills/ (slash-command skills + knowledge guides).
  • Added a PostToolUse security-warning hook (hooks/) plus default MCP/client configs (.mcp.json, .claude-plugin/plugin.json).

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
README.md Reorganized docs; moved Setup later; consolidated Security details; updated Docker image references/snippets.
hooks/security-warning.sh New post-tool hook to surface security warnings based on tool output.
hooks/hooks.json Registers PostToolUse hook matcher for selected ComfyUI MCP tools.
skills/gen/SKILL.md Skill doc for /comfy:gen flow.
skills/workflow/SKILL.md Skill doc for /comfy:workflow flow.
skills/status/SKILL.md Skill doc for /comfy:status output formatting.
skills/progress/SKILL.md Skill doc for /comfy:progress output formatting.
skills/models/SKILL.md Skill doc for /comfy:models behavior/formatting.
skills/history/SKILL.md Skill doc for /comfy:history behavior/formatting.
skills/workflows/SKILL.md Knowledge guide on ComfyUI workflow JSON/node patterns.
skills/troubleshooting/SKILL.md Troubleshooting knowledge guide.
.mcp.json Example stdio MCP server config using uvx.
.claude-plugin/plugin.json Claude Code plugin metadata.

You can also share your feedback on Copilot code review. Take the survey.

fi

# Check for node auditor results with actual findings (count > 0)
if echo "$input" | grep -qE '"dangerous":\s*\{\s*"count":\s*[1-9]'; then

- **Current node**: which node is executing
- **Progress**: step X of Y (percentage)
- **Status**: running, completed, or failed
@hybridindie hybridindie force-pushed the worktree-plugin-packaging branch from 1ee173f to 8c3e301 Compare March 13, 2026 15:31
@hybridindie hybridindie merged commit 268592b into main Mar 13, 2026
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