alphard/ComfyUI-Lora-Manager
1
0
Fork 0
mirror of https://github.com/willmiao/ComfyUI-Lora-Manager.git synced 2026-07-13 20:21:16 -03:00
Code Issues Packages Projects Releases Wiki Activity

fix(registry): force re-registration on WS refresh to prevent timeout, demote empty-registry log to debug

Browse Source 
- workflow_registry.js: add force param to refreshRegistry(), bypass fingerprint
  dedup when responding to lora_registry_refresh WS message. Without this, the
  backend's wait_for_all() times out after 0.5s because the frontend skips the
  register-nodes POST when the workflow fingerprint hasn't changed (common after
  ComfyUI restart with an empty or unchanged workflow).
- misc_handlers.py: demote 'No nodes registered after refresh' from WARNING to
  DEBUG — empty workflows are a normal operational state, not a warning-worthy
  condition.
This commit is contained in:
Will Miao
2026-07-13 19:10:48 +08:00
parent 2018722cc8
commit 966024e534
2 changed files with 10 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
py/routes/handlers/misc_handlers.py
View File

@@ -3314,7 +3314,10 @@ class NodeRegistryHandler:
self._last_slow_path_ts = time.monotonic() self._last_slow_path_ts = time.monotonic()
if registry_info["node_count"] == 0: if registry_info["node_count"] == 0:
logger.warning("No nodes registered after refresh") logger.debug(
"[LM:Registry] refresh OK — %s connected tab(s) but 0 compatible nodes found",
registry_info["tab_count"],
)
return web.json_response( return web.json_response(
{ {
"success": False, "success": False,

10
web/comfyui/workflow_registry.js
View File

@@ -84,7 +84,7 @@ app.registerExtension({
this._log("extension initialized, clientId=%s", api.clientId ?? api.initialClientId ?? "(pending)"); this._log("extension initialized, clientId=%s", api.clientId ?? api.initialClientId ?? "(pending)");
api.addEventListener("lora_registry_refresh", () => { api.addEventListener("lora_registry_refresh", () => {
this.refreshRegistry(); this.refreshRegistry(true);
}); });
api.addEventListener("lm_widget_update", (event) => { api.addEventListener("lm_widget_update", (event) => {
@@ -151,7 +151,7 @@ app.registerExtension({
console.debug(`[LM:Registry ${ts}] ${msg}`); console.debug(`[LM:Registry ${ts}] ${msg}`);
}, },
async refreshRegistry() { async refreshRegistry(force = false) {
try { try {
const workflowNodes = []; const workflowNodes = [];
const nodeEntries = getAllGraphNodes(app.graph); const nodeEntries = getAllGraphNodes(app.graph);
@@ -204,11 +204,13 @@ app.registerExtension({
const clientId = api.clientId ?? api.initialClientId ?? ""; const clientId = api.clientId ?? api.initialClientId ?? "";
// Content-based dedup: skip POST if identical to last sent payload. // Content-based dedup: skip POST if identical to last sent payload,
// unless forced (e.g. responding to a lora_registry_refresh WS message
// where the backend explicitly requests a re-registration).
const fingerprint = JSON.stringify( const fingerprint = JSON.stringify(
workflowNodes.map(n => `${n.graph_id}:${n.node_id}|${n.marker_role ?? ""}|${n.mode ?? 0}`).sort() workflowNodes.map(n => `${n.graph_id}:${n.node_id}|${n.marker_role ?? ""}|${n.mode ?? 0}`).sort()
); );
if (fingerprint === this._lastFingerprint) { if (!force && fingerprint === this._lastFingerprint) {
return; return;
} }
this._lastFingerprint = fingerprint; this._lastFingerprint = fingerprint;