heshengtao / comfyui_LLM_party

LLM Agent Framework in ComfyUI includes Omost,GPT-sovits, ChatTTS,GOT-OCR2.0, and FLUX prompt nodes,access to Feishu,discord,and adapts to all llms with similar openai/gemini interfaces, such as o1,ollama, grok, qwen, GLM, deepseek, moonshot,doubao. Adapted to local llms, vlm, gguf such as llama-3.2, Linkage neo4j KG, graphRAG / RAG / html 2 img
GNU Affero General Public License v3.0
1.06k stars 94 forks source link

Error: Extension named 'party.ShowText' already registered. #94

Open snoche opened 1 month ago

snoche commented 1 month ago

Describe the bug ShowText stop working for me after the confyui ui update a few weeks ago

image

To Reproduce when opening Confyui in the browser console I got this error: app.ts:1725 Error loading extension /extensions/party/showtext_party.js Error: Extension named 'party.ShowText' already registered. at ComfyApp.registerExtension (app.ts:2856:13) at showtext_party.js:5:5

This file is been copied in the web/extensions/party but looks like is already registed

image

On executing error: showText.js:45 Uncaught TypeError: Cannot read properties of undefined (reading 'text') at nodeType.onExecuted (showText.js:45:33) at nodeType.onExecuted (easy_videooutput_node.js:240:41) at ComfyApi. (app.ts:1629:16) at os.invokeTarget (AdGuard Popup Blocker.user.js:455:270074) at gt (AdGuard Popup Blocker.user.js:455:287512) at eval (AdGuard Popup Blocker.user.js:455:271601) at eval (AdGuard Popup Blocker.user.js:455:270275) at ComfyApi.dispatchEvent (AdGuard Popup Blocker.user.js:455:269497) at WebSocket. (api.ts:202:20)

If I delete the file from comfyui_LLM_party\web\js I don't get the already registered error and the showText from customScripts works again. It could be a conflict between the other package?

image

By the other hand if I delete showText from ComfyUI-Custom-Scripts nothing is fixed https://github.com/pythongosssss/ComfyUI-Custom-Scripts

image

image

image

I tried deleting full ComfyUI-Custom-Scripts folder and nothing is fix neither, so looks like it could be LLM party registering twice or something? and breaks also the other node? I am so confuse, no one else has this issue?

heshengtao commented 1 month ago

I tried to fix this issue. Please update to the latest LLM party to see if it has been fixed.

snoche commented 1 month ago

That fixed the error of redeclaration thanks

FYI: The issue I was having was different, it was Adguard, even not enabled for local host and disabling Adguard was still blocking something, I had to allow "Unsafe content" con the extension to allow this too work.