Skip to content

fix: include assignedWorker in session create/get response#1872

Open
gergesh wants to merge 1 commit intodevlikeapro:corefrom
gergesh:feat/add-worker-id-to-created-sessions
Open

fix: include assignedWorker in session create/get response#1872
gergesh wants to merge 1 commit intodevlikeapro:corefrom
gergesh:feat/add-worker-id-to-created-sessions

Conversation

@gergesh
Copy link

@gergesh gergesh commented Jan 29, 2026

Issue: POST /api/sessions (and GET /api/sessions/{name}) don't return assignedWorker, even though SessionInfo DTO already declares it and GET /api/sessions (list) does return it.
Root cause: getSessions() in manager.core.ts never sets assignedWorker. The plus edition's listSessions() adds it after the fact from the worker repository, but getSessionInfo() doesn't.

Fix — two files:

  1. src/core/manager.core.ts — add assignedWorker: this.workerId to both return paths in getSessions() (STOPPED path and active path). (Already patched in this PR.)
  2. src/plus/manager.plus.ts — in getSessionInfo(), add assignedWorker: this.workerId to the return object. It currently returns { ...session, engine } — change it to { ...session, engine, assignedWorker: this.workerId }.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant