fix(backend): use workerId in error logs and fix worker linking (#164, #162) #227

Merged
jack merged 4 commits from fix/164-162-unused-code into main 2026-01-24 18:00:05 +00:00
Owner

Summary

Issue #164 - Unused workerId parameter

  • Add workerId to task error log messages for better debugging
  • Include workerId in SSE task:failed broadcast data

Issue #162 - Unused elements in WorkerProcessManager

  • Remove unused loadSettings() call in detectWorkerBinaryPath()
  • Add onWorkerLinked callback to TaskDispatcher options
  • Call linkToHubWorker when worker connects with spawnedId in metadata
  • Remove redundant onWorkerConnected/onWorkerDisconnected handlers from backend-service.ts (TaskDispatcher handles these)

Test plan

  • Verify error logs include workerId
  • Verify spawned workers are linked to hub workers on connect
  • Check findByHubWorkerId() works after worker connection

Closes #164
Closes #162

🤖 Generated with Claude Code

## Summary ### Issue #164 - Unused workerId parameter - Add `workerId` to task error log messages for better debugging - Include `workerId` in SSE `task:failed` broadcast data ### Issue #162 - Unused elements in WorkerProcessManager - Remove unused `loadSettings()` call in `detectWorkerBinaryPath()` - Add `onWorkerLinked` callback to TaskDispatcher options - Call `linkToHubWorker` when worker connects with `spawnedId` in metadata - Remove redundant `onWorkerConnected`/`onWorkerDisconnected` handlers from backend-service.ts (TaskDispatcher handles these) ## Test plan - [ ] Verify error logs include workerId - [ ] Verify spawned workers are linked to hub workers on connect - [ ] Check `findByHubWorkerId()` works after worker connection Closes #164 Closes #162 🤖 Generated with [Claude Code](https://claude.com/claude-code)
fix(backend): use workerId in error logs and fix worker linking (#164, #162)
All checks were successful
CI / build (pull_request) Successful in 38s
632793fb3a
Issue #164:
- Add workerId to task error log messages and SSE broadcast

Issue #162:
- Remove unused loadSettings call in detectWorkerBinaryPath
- Add onWorkerLinked callback to TaskDispatcher to link spawned workers
- Call linkToHubWorker when worker connects with spawnedId metadata
- Remove redundant onWorkerConnected/onWorkerDisconnected handlers from
  backend-service.ts (now handled by TaskDispatcher)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Merge branch 'main' into fix/164-162-unused-code
All checks were successful
CI / build (pull_request) Successful in 1m14s
bd697c10ee
Merge branch 'main' into fix/164-162-unused-code
All checks were successful
CI / build (pull_request) Successful in 1m12s
b5e3b1b4a6
feat(hooks): return cached context immediately if available (#154)
All checks were successful
CI / build (pull_request) Successful in 41s
5653e3ee35
When a cached CLAUDE.md context exists for a project, return it
immediately instead of just queueing a generation task. This provides
faster context injection at session start.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
jack merged commit 4222284e5a into main 2026-01-24 18:00:05 +00:00
Sign in to join this conversation.
No description provided.