Comfy-Org/ComfyUI_frontend
1
0
Fork 0
mirror of https://github.com/Comfy-Org/ComfyUI_frontend.git synced 2026-04-20 06:20:11 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: use prompt_id to update correct job in execution handlers

Browse Source 
handleExecutionCached and handleExecuted were updating activeJob, which
breaks when the focused job differs from the active job. Look up the job
by e.detail.prompt_id in queuedJobs instead.

Addresses review feedback:
https://github.com/Comfy-Org/ComfyUI_frontend/pull/9786#discussion_r2926758475
This commit is contained in:
bymyself
2026-03-13 09:30:53 -07:00
parent 638f1b9938
commit c2c4f84d24
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/stores/executionStore.ts
View File

@@ -274,9 +274,10 @@ export const useExecutionStore = defineStore('execution', () => {
}
function handleExecutionCached(e: CustomEvent<ExecutionCachedWsMessage>) {
if (!activeJob.value) return
const job = queuedJobs.value[e.detail.prompt_id]
if (!job) return
for (const n of e.detail.nodes) {
activeJob.value.nodes[n] = true
job.nodes[n] = true
}
}
@@ -289,8 +290,9 @@ export const useExecutionStore = defineStore('execution', () => {
}
function handleExecuted(e: CustomEvent<ExecutedWsMessage>) {
if (!activeJob.value) return
activeJob.value.nodes[e.detail.node] = true
const job = queuedJobs.value[e.detail.prompt_id]
if (!job) return
job.nodes[e.detail.node] = true
}
function handleExecutionSuccess(e: CustomEvent<ExecutionSuccessWsMessage>) {