fix(ui): sync worker busy/idle count with actual worker state #344

Merged

review.bot merged 1 commit from fix/341-worker-count-desync into develop

2026-03-02 13:50:55 +00:00

jack commented

2026-03-02 13:49:48 +00:00

Owner

Summary

The header worker metrics (working/idle counts) only checked SSE workerTasks state, while individual worker cards also checked worker.currentTaskId from the API. This caused the header to show fewer working workers than the cards displayed (e.g. "2 working, 1 idle" when all 3 cards showed "Working").

Fix: Added worker.currentTaskId fallback to the busy count calculation, matching the logic already used in WorkerItem.

Closes #341

## Summary The header worker metrics (working/idle counts) only checked SSE `workerTasks` state, while individual worker cards also checked `worker.currentTaskId` from the API. This caused the header to show fewer working workers than the cards displayed (e.g. "2 working, 1 idle" when all 3 cards showed "Working"). **Fix:** Added `worker.currentTaskId` fallback to the busy count calculation, matching the logic already used in `WorkerItem`. Closes #341

jack added 1 commit

2026-03-02 13:49:48 +00:00

fix(ui): sync worker busy/idle count with actual worker state (#341 ) c3400c899f

The header metrics (working/idle counts) only checked SSE workerTasks
state, while individual worker cards also checked worker.currentTaskId
from the API. This caused the header to show fewer working workers
than the cards displayed.

Now both sources are checked consistently, matching the logic already
used in WorkerItem.

review.bot

2026-03-02 13:49:48 +00:00

added the
component
api

type
bug
labels
requested review from review.bot

review.bot reviewed

2026-03-02 13:50:54 +00:00

review.bot left a comment

Owner

AI Code Review (Devstral)

The fix correctly addresses the inconsistency between worker status in the header and individual worker cards by checking both SSE state and API state for busy workers.

Automated review by pr-reviewer

## AI Code Review (Devstral) The fix correctly addresses the inconsistency between worker status in the header and individual worker cards by checking both SSE state and API state for busy workers. --- *Automated review by [pr-reviewer](https://github.com/customable/pr-reviewer)*

review.bot added

and removed

labels

2026-03-02 13:50:54 +00:00

review.bot merged commit c2fd3fd576 into develop

2026-03-02 13:50:55 +00:00