[PR #1676] [MERGED] feat&fix: ComfyUI 0.2.6 #2024

Closed
opened 2026-02-16 11:15:53 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langgenius/dify-official-plugins/pull/1676
Author: @yt-koike
Created: 9/7/2025
Status: Merged
Merged: 9/7/2025
Merged by: @crazywoola

Base: mainHead: main


📝 Commits (10+)

📊 Changes

22 files changed (+183 additions, -163 deletions)

View changed files

📝 tools/comfyui/main.py (+1 -1)
📝 tools/comfyui/manifest.yaml (+1 -1)
📝 tools/comfyui/provider/comfyui.py (+3 -2)
📝 tools/comfyui/tools/comfyui_client.py (+4 -19)
📝 tools/comfyui/tools/comfyui_model_manager.py (+28 -33)
📝 tools/comfyui/tools/comfyui_workflow.py (+32 -28)
📝 tools/comfyui/tools/convert_to_api.py (+6 -3)
📝 tools/comfyui/tools/download_by_json.py (+6 -4)
📝 tools/comfyui/tools/download_by_url.py (+7 -11)
📝 tools/comfyui/tools/download_civitai.py (+6 -4)
📝 tools/comfyui/tools/download_huggingface.py (+6 -3)
📝 tools/comfyui/tools/image_info.py (+6 -3)
📝 tools/comfyui/tools/img2any.py (+8 -5)
📝 tools/comfyui/tools/img2img.py (+7 -4)
📝 tools/comfyui/tools/img2vid.py (+6 -3)
📝 tools/comfyui/tools/list_models.py (+5 -2)
📝 tools/comfyui/tools/list_samplers.py (+4 -2)
📝 tools/comfyui/tools/quickstart.py (+13 -10)
📝 tools/comfyui/tools/quickstart.yaml (+14 -14)
📝 tools/comfyui/tools/run_workflow.py (+6 -3)

...and 2 more files

📄 Description

Related Issues or Context

New feature:

  • Download public models without API Keys: This plugin used to need API keys to download public models on CivitAI and Hugging Face. Now it firstly tries to download them without keys.

Fix: https://github.com/langgenius/dify-official-plugins/issues/1627

Refactor:

This PR does NOT contain Changes to Non-Plugin

This PR contains Changes to Non-LLM Models Plugin

  • I have Run Comprehensive Tests Relevant to My Changes

This PR does NOT contain Changes to LLM Models Plugin

Version Control (Any Changes to the Plugin Will Require Bumping the Version)

  • I have Bumped Up the Version in Manifest.yaml (Top-Level Version Field, Not in Meta Section)

Dify Plugin SDK Version

  • I have Ensured dify_plugin>=0.3.0,<0.5.0 is in requirements.txt (SDK docs)

Environment Verification (If Any Code Changes)

Local Deployment Environment

  • Dify Version is: 1.8.1, I have Tested My Changes on Local Deployment Dify with a Clean Environment That Matches the Production Configuration.

SaaS Environment

  • I have Tested My Changes on cloud.dify.ai with a Clean Environment That Matches the Production Configuration

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langgenius/dify-official-plugins/pull/1676 **Author:** [@yt-koike](https://github.com/yt-koike) **Created:** 9/7/2025 **Status:** ✅ Merged **Merged:** 9/7/2025 **Merged by:** [@crazywoola](https://github.com/crazywoola) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (10+) - [`6b7a08a`](https://github.com/langgenius/dify-official-plugins/commit/6b7a08ad8c2cb2692d661568d30559c062295bde) rm replace("'",'"') on json file - [`1158487`](https://github.com/langgenius/dify-official-plugins/commit/1158487194e1555cbc597ff16af080f5e58a2fea) introduce ruff the format - [`bb797e5`](https://github.com/langgenius/dify-official-plugins/commit/bb797e57a99b081e5c51d46df08ded601c268c93) Bump up the version - [`467d45b`](https://github.com/langgenius/dify-official-plugins/commit/467d45bced6f1cfa542bf1eaa21e38713260030e) rm print() according to https://github.com/langgenius/dify-official-plugins/issues/1673 - [`18146ba`](https://github.com/langgenius/dify-official-plugins/commit/18146ba764741d95e11f17c0320c8ef2953c9494) introduce ComfyUIModel - [`c1a81a0`](https://github.com/langgenius/dify-official-plugins/commit/c1a81a0ae1ccccab3d508a2bbcf79e0a5f7e5f68) rename model_manager.py - [`7442410`](https://github.com/langgenius/dify-official-plugins/commit/7442410b0cebe288b0fb2df2cf71e916a5329eaa) bump up - [`11c48d0`](https://github.com/langgenius/dify-official-plugins/commit/11c48d0c35a3ba982aa0bb01ab892cca131432d3) first try download without tokens - [`6f32b72`](https://github.com/langgenius/dify-official-plugins/commit/6f32b727cf3a05d7b5e28a4e154c01d6cff044ae) bugfix - [`3feab06`](https://github.com/langgenius/dify-official-plugins/commit/3feab06a19f3dadff10ec355c54aa95c20956acc) try downloading models without tokens first ### 📊 Changes **22 files changed** (+183 additions, -163 deletions) <details> <summary>View changed files</summary> 📝 `tools/comfyui/main.py` (+1 -1) 📝 `tools/comfyui/manifest.yaml` (+1 -1) 📝 `tools/comfyui/provider/comfyui.py` (+3 -2) 📝 `tools/comfyui/tools/comfyui_client.py` (+4 -19) 📝 `tools/comfyui/tools/comfyui_model_manager.py` (+28 -33) 📝 `tools/comfyui/tools/comfyui_workflow.py` (+32 -28) 📝 `tools/comfyui/tools/convert_to_api.py` (+6 -3) 📝 `tools/comfyui/tools/download_by_json.py` (+6 -4) 📝 `tools/comfyui/tools/download_by_url.py` (+7 -11) 📝 `tools/comfyui/tools/download_civitai.py` (+6 -4) 📝 `tools/comfyui/tools/download_huggingface.py` (+6 -3) 📝 `tools/comfyui/tools/image_info.py` (+6 -3) 📝 `tools/comfyui/tools/img2any.py` (+8 -5) 📝 `tools/comfyui/tools/img2img.py` (+7 -4) 📝 `tools/comfyui/tools/img2vid.py` (+6 -3) 📝 `tools/comfyui/tools/list_models.py` (+5 -2) 📝 `tools/comfyui/tools/list_samplers.py` (+4 -2) 📝 `tools/comfyui/tools/quickstart.py` (+13 -10) 📝 `tools/comfyui/tools/quickstart.yaml` (+14 -14) 📝 `tools/comfyui/tools/run_workflow.py` (+6 -3) _...and 2 more files_ </details> ### 📄 Description ## Related Issues or Context <!-- ⚠️ NOTE: This repository is for Dify Official Plugins only. For community contributions, please submit to https://github.com/langgenius/dify-plugins instead. - Link Related Issues if Applicable: #issue_number - Or Provide Context about Why this Change is Needed --> New feature: - Download public models without API Keys: This plugin used to need API keys to download public models on CivitAI and Hugging Face. Now it firstly tries to download them without keys. Fix: https://github.com/langgenius/dify-official-plugins/issues/1627 Refactor: - Delete all print() according to https://github.com/langgenius/dify-official-plugins/issues/1673 - [Executed ruff check --fix](https://github.com/langgenius/dify-official-plugins/commit/747a29844314ce0b84feba9d0c9e27384e9cd80c) - Introduce download_models_autotoken() to automatically select a token to use ## This PR does NOT contain Changes to *Non-Plugin* ## This PR contains Changes to *Non-LLM Models Plugin* - [x] I have Run Comprehensive Tests Relevant to My Changes <!-- 📷 Include Screenshots/Videos Demonstrating the Fix, New Feature, or the Behavior Before/After Breaking Changes. --> ## This PR does NOT contain Changes to *LLM Models Plugin* ## Version Control (Any Changes to the Plugin Will Require Bumping the Version) - [x] I have Bumped Up the Version in Manifest.yaml (Top-Level `Version` Field, Not in Meta Section) <!-- ⚠️ NOTE: Version Format: MAJOR.MINOR.PATCH - MAJOR (0.x.x): Reserved for Significant architectural changes or incompatible API modifications - MINOR (x.0.x): For New feature additions while maintaining backward compatibility - PATCH (x.x.0): For Backward-compatible bug fixes and minor improvements - Note: Each Version Component (MAJOR, MINOR, PATCH) Can Be 2 Digits, e.g., 10.11.22 --> ## Dify Plugin SDK Version - [x] I have Ensured `dify_plugin>=0.3.0,<0.5.0` is in requirements.txt ([SDK docs](https://github.com/langgenius/dify-plugin-sdks/blob/main/python/README.md)) ## Environment Verification (If Any Code Changes) <!-- ⚠️ NOTE: At Least One Environment Must Be Tested. --> ### Local Deployment Environment - [x] Dify Version is: 1.8.1, I have Tested My Changes on Local Deployment Dify with a Clean Environment That Matches the Production Configuration. <!-- - Python Virtual Env Matching Manifest.yaml & requirements.txt - No Breaking Changes in Dify That May Affect the Testing Result --> ### SaaS Environment - [ ] I have Tested My Changes on cloud.dify.ai with a Clean Environment That Matches the Production Configuration <!-- - Python Virtual Env Matching Manifest.yaml & requirements.txt --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 11:15:53 -05:00
yindo closed this issue 2026-02-16 11:15:53 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify-official-plugins#2024