Skip to content

feat: add ComfyUI plugin#1

Open
williamxie1989 wants to merge 2 commits intomainfrom
feat/comfyui-plugin
Open

feat: add ComfyUI plugin#1
williamxie1989 wants to merge 2 commits intomainfrom
feat/comfyui-plugin

Conversation

@williamxie1989
Copy link
Copy Markdown
Owner

Summary

  • Add comfyui/ plugin to manage ComfyUI servers from the CLI
  • nodes — list all registered node types (700+)
  • node-info — detailed definition of a node (inputs, types, defaults, constraints)
  • search-node — search nodes by keyword
  • models — list available model files
  • system-stats — server status and system information
  • run — execute workflows via JSON prompt or file
  • queue — view running/pending queue status
  • history — view execution history
  • Configurable via COMFYUI_HOST environment variable (default: http://127.0.0.1:8188)

ComfyUI API Endpoints Used

Endpoint Purpose
/api/object_info List all node definitions
/api/models List model directories
/api/models/<type> List models of a type
/api/system_stats Server information
/prompt Queue a workflow for execution
/api/queue Queue status
/api/history Execution history

Test plan

  • All 8 commands tested against a live ComfyUI server (macOS, Apple Silicon)
  • Error handling: invalid node name, non-existent prompt_id, no-match search
  • COMFYUI_HOST env var correctly configures server address

…execution

Add a complete plugin for interacting with ComfyUI servers:
- nodes: list all registered node types
- node-info: detailed definition of a node (inputs, types, defaults)
- search-node: search nodes by keyword
- models: list available model files
- system-stats: server status and system information
- run: execute workflows via JSON prompt
- queue: view running/pending queue status
- history: view execution history

Configurable via COMFYUI_HOST environment variable.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4d2fcc29df

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@@ -0,0 +1,26 @@
import { cli, Strategy } from '../../registry.js';
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Import registry via package export path

This adapter imports ../../registry.js, but that path does not exist in either the repo root or built package layout (the registry is exported as @jackwener/opencli/registry), so module loading fails with ERR_MODULE_NOT_FOUND when discovery imports the file and the comfyui commands are never registered.

Useful? React with 👍 / 👎.

@@ -0,0 +1,48 @@
import { cli, Strategy } from '../../registry.js';
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Keep built-in adapters as TS sources

This new built-in command is added as a .js file under clis/, but the build pipeline compiles clis/**/*.ts and publishes from dist/clis; without a copy step for source .js adapters, the generated manifest can reference dist/clis/comfyui/*.js files that are not emitted, causing opencli comfyui ... commands to fail in installed builds.

Useful? React with 👍 / 👎.

Add browser adapter doc for ComfyUI with commands table, usage examples,
and configuration instructions. Register in VitePress sidebar.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

1 participant