crewly 1.6.3 → 1.6.4

package/dist/backend/backend/src/services/agent/crewly-agent/in-process-runtime-registry.d.ts

@@ -0,0 +1,91 @@
+/**
+ * In-Process Runtime Registry
+ *
+ * Module-level registry that tracks active in-process Crewly Agent runtimes
+ * by session name. Provides a lightweight lookup mechanism for callers (such
+ * as `orchestrator-status.service.ts`) that need to determine whether an
+ * in-process runtime is alive without holding a reference to the
+ * `AgentRegistrationService` instance.
+ *
+ * Why this exists (B0 hot-fix):
+ * The PTY-based `sessionExists()` check returns `false` for in-process
+ * Crewly Agent runtimes because they have no PTY session. Before this
+ * registry, the orchestrator-status service had no runtime-aware fallback,
+ * so `isActive` would report `false` for healthy in-process orchestrators
+ * — causing SlackBridge to incorrectly route messages to the offline path.
+ *
+ * The `AgentRegistrationService` already maintains an internal
+ * `inProcessRuntimes` map for in-process runtime routing. This module
+ * mirrors that knowledge at module scope so any caller can probe runtime
+ * health without a service-locator dance.
+ *
+ * @module services/agent/crewly-agent/in-process-runtime-registry
+ */
+import type { CrewlyAgentRuntimeService } from './crewly-agent-runtime.service.js';
+/**
+ * Register an in-process Crewly Agent runtime.
+ *
+ * Called by `AgentRegistrationService` after `initializeInProcess()`
+ * succeeds. If a runtime is already registered for the same session, it is
+ * replaced — the old reference is discarded but not shut down (the caller
+ * is responsible for the lifecycle).
+ *
+ * @param sessionName - Session name owning the runtime (e.g. 'crewly-orc')
+ * @param runtime - The initialized CrewlyAgentRuntimeService instance
+ *
+ * @example
+ * ```typescript
+ * registerInProcessRuntime('crewly-orc', crewlyRuntime);
+ * ```
+ */
+export declare function registerInProcessRuntime(sessionName: string, runtime: CrewlyAgentRuntimeService): void;
+/**
+ * Unregister an in-process runtime by session name.
+ *
+ * Called when a runtime is shut down or the session is destroyed.
+ * Idempotent — safe to call when the session is not registered.
+ *
+ * @param sessionName - Session name to remove
+ * @returns True if a runtime was removed, false if none was registered
+ */
+export declare function unregisterInProcessRuntime(sessionName: string): boolean;
+/**
+ * Look up an in-process runtime by session name.
+ *
+ * @param sessionName - Session name to look up
+ * @returns The CrewlyAgentRuntimeService if registered, undefined otherwise
+ */
+export declare function getInProcessRuntime(sessionName: string): CrewlyAgentRuntimeService | undefined;
+/**
+ * Check whether an in-process runtime exists and is ready to handle
+ * messages for the given session.
+ *
+ * Returns `true` only when both:
+ * - A runtime is registered for the session
+ * - The runtime's `isReady()` returns `true`
+ *
+ * This is the function callers should use when answering "is this
+ * session alive?" — it correctly handles both the worker-process and
+ * direct in-process modes of `CrewlyAgentRuntimeService`.
+ *
+ * @param sessionName - Session name to check
+ * @returns True when the runtime exists and is ready
+ */
+export declare function isInProcessRuntimeActive(sessionName: string): boolean;
+/**
+ * Reset the registry. Test-only helper.
+ *
+ * Leaves any actually-running runtimes untouched — only clears the
+ * registry's bookkeeping. Production code must never call this.
+ *
+ * @internal
+ */
+export declare function _resetInProcessRuntimeRegistryForTesting(): void;
+/**
+ * Get the current registry size. Test-only helper.
+ *
+ * @returns Number of registered runtimes
+ * @internal
+ */
+export declare function _getInProcessRuntimeRegistrySize(): number;
+//# sourceMappingURL=in-process-runtime-registry.d.ts.map

package/dist/backend/backend/src/services/agent/crewly-agent/in-process-runtime-registry.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"in-process-runtime-registry.d.ts","sourceRoot":"","sources":["../../../../../../../backend/src/services/agent/crewly-agent/in-process-runtime-registry.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;GAsBG;AAEH,OAAO,KAAK,EAAE,yBAAyB,EAAE,MAAM,mCAAmC,CAAC;AAWnF;;;;;;;;;;;;;;;GAeG;AACH,wBAAgB,wBAAwB,CACtC,WAAW,EAAE,MAAM,EACnB,OAAO,EAAE,yBAAyB,GACjC,IAAI,CAEN;AAED;;;;;;;;GAQG;AACH,wBAAgB,0BAA0B,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAEvE;AAED;;;;;GAKG;AACH,wBAAgB,mBAAmB,CACjC,WAAW,EAAE,MAAM,GAClB,yBAAyB,GAAG,SAAS,CAEvC;AAED;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,wBAAwB,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAWrE;AAED;;;;;;;GAOG;AACH,wBAAgB,wCAAwC,IAAI,IAAI,CAE/D;AAED;;;;;GAKG;AACH,wBAAgB,gCAAgC,IAAI,MAAM,CAEzD"}

package/dist/backend/backend/src/services/agent/crewly-agent/in-process-runtime-registry.js

@@ -0,0 +1,120 @@
+/**
+ * In-Process Runtime Registry
+ *
+ * Module-level registry that tracks active in-process Crewly Agent runtimes
+ * by session name. Provides a lightweight lookup mechanism for callers (such
+ * as `orchestrator-status.service.ts`) that need to determine whether an
+ * in-process runtime is alive without holding a reference to the
+ * `AgentRegistrationService` instance.
+ *
+ * Why this exists (B0 hot-fix):
+ * The PTY-based `sessionExists()` check returns `false` for in-process
+ * Crewly Agent runtimes because they have no PTY session. Before this
+ * registry, the orchestrator-status service had no runtime-aware fallback,
+ * so `isActive` would report `false` for healthy in-process orchestrators
+ * — causing SlackBridge to incorrectly route messages to the offline path.
+ *
+ * The `AgentRegistrationService` already maintains an internal
+ * `inProcessRuntimes` map for in-process runtime routing. This module
+ * mirrors that knowledge at module scope so any caller can probe runtime
+ * health without a service-locator dance.
+ *
+ * @module services/agent/crewly-agent/in-process-runtime-registry
+ */
+/**
+ * Module-level registry of active in-process runtimes keyed by session name.
+ *
+ * Mirrors `AgentRegistrationService.inProcessRuntimes`. Both maps must stay
+ * in sync — `AgentRegistrationService` writes to this registry whenever it
+ * registers/unregisters a runtime.
+ */
+const registry = new Map();
+/**
+ * Register an in-process Crewly Agent runtime.
+ *
+ * Called by `AgentRegistrationService` after `initializeInProcess()`
+ * succeeds. If a runtime is already registered for the same session, it is
+ * replaced — the old reference is discarded but not shut down (the caller
+ * is responsible for the lifecycle).
+ *
+ * @param sessionName - Session name owning the runtime (e.g. 'crewly-orc')
+ * @param runtime - The initialized CrewlyAgentRuntimeService instance
+ *
+ * @example
+ * ```typescript
+ * registerInProcessRuntime('crewly-orc', crewlyRuntime);
+ * ```
+ */
+export function registerInProcessRuntime(sessionName, runtime) {
+    registry.set(sessionName, runtime);
+}
+/**
+ * Unregister an in-process runtime by session name.
+ *
+ * Called when a runtime is shut down or the session is destroyed.
+ * Idempotent — safe to call when the session is not registered.
+ *
+ * @param sessionName - Session name to remove
+ * @returns True if a runtime was removed, false if none was registered
+ */
+export function unregisterInProcessRuntime(sessionName) {
+    return registry.delete(sessionName);
+}
+/**
+ * Look up an in-process runtime by session name.
+ *
+ * @param sessionName - Session name to look up
+ * @returns The CrewlyAgentRuntimeService if registered, undefined otherwise
+ */
+export function getInProcessRuntime(sessionName) {
+    return registry.get(sessionName);
+}
+/**
+ * Check whether an in-process runtime exists and is ready to handle
+ * messages for the given session.
+ *
+ * Returns `true` only when both:
+ * - A runtime is registered for the session
+ * - The runtime's `isReady()` returns `true`
+ *
+ * This is the function callers should use when answering "is this
+ * session alive?" — it correctly handles both the worker-process and
+ * direct in-process modes of `CrewlyAgentRuntimeService`.
+ *
+ * @param sessionName - Session name to check
+ * @returns True when the runtime exists and is ready
+ */
+export function isInProcessRuntimeActive(sessionName) {
+    const runtime = registry.get(sessionName);
+    if (!runtime) {
+        return false;
+    }
+    try {
+        return runtime.isReady();
+    }
+    catch {
+        // Defensive: a misbehaving isReady() must not break callers.
+        return false;
+    }
+}
+/**
+ * Reset the registry. Test-only helper.
+ *
+ * Leaves any actually-running runtimes untouched — only clears the
+ * registry's bookkeeping. Production code must never call this.
+ *
+ * @internal
+ */
+export function _resetInProcessRuntimeRegistryForTesting() {
+    registry.clear();
+}
+/**
+ * Get the current registry size. Test-only helper.
+ *
+ * @returns Number of registered runtimes
+ * @internal
+ */
+export function _getInProcessRuntimeRegistrySize() {
+    return registry.size;
+}
+//# sourceMappingURL=in-process-runtime-registry.js.map

package/dist/backend/backend/src/services/agent/crewly-agent/in-process-runtime-registry.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"in-process-runtime-registry.js","sourceRoot":"","sources":["../../../../../../../backend/src/services/agent/crewly-agent/in-process-runtime-registry.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;GAsBG;AAIH;;;;;;GAMG;AACH,MAAM,QAAQ,GAA2C,IAAI,GAAG,EAAE,CAAC;AAEnE;;;;;;;;;;;;;;;GAeG;AACH,MAAM,UAAU,wBAAwB,CACtC,WAAmB,EACnB,OAAkC;IAElC,QAAQ,CAAC,GAAG,CAAC,WAAW,EAAE,OAAO,CAAC,CAAC;AACrC,CAAC;AAED;;;;;;;;GAQG;AACH,MAAM,UAAU,0BAA0B,CAAC,WAAmB;IAC5D,OAAO,QAAQ,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;AACtC,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,mBAAmB,CACjC,WAAmB;IAEnB,OAAO,QAAQ,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;AACnC,CAAC;AAED;;;;;;;;;;;;;;GAcG;AACH,MAAM,UAAU,wBAAwB,CAAC,WAAmB;IAC1D,MAAM,OAAO,GAAG,QAAQ,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;IAC1C,IAAI,CAAC,OAAO,EAAE,CAAC;QACb,OAAO,KAAK,CAAC;IACf,CAAC;IACD,IAAI,CAAC;QACH,OAAO,OAAO,CAAC,OAAO,EAAE,CAAC;IAC3B,CAAC;IAAC,MAAM,CAAC;QACP,6DAA6D;QAC7D,OAAO,KAAK,CAAC;IACf,CAAC;AACH,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,wCAAwC;IACtD,QAAQ,CAAC,KAAK,EAAE,CAAC;AACnB,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,gCAAgC;IAC9C,OAAO,QAAQ,CAAC,IAAI,CAAC;AACvB,CAAC"}

package/dist/backend/backend/src/services/orchestrator/index.d.ts

@@ -12,4 +12,5 @@ export { ImprovementMarkerService, getImprovementMarkerService, resetImprovement
 export { ImprovementStartupService, getImprovementStartupService, resetImprovementStartupService, type StartupResult, } from './improvement-startup.service.js';
 export { isOrchestratorActive, isAgentActive, getOrchestratorStatus, getOrchestratorOfflineMessage, type OrchestratorStatusResult, } from './orchestrator-status.service.js';
 export { OrchestratorRestartService, type RestartStats, } from './orchestrator-restart.service.js';
+export { triggerOrchestratorSetup, setOrchestratorSetupDependencies, type OrchestratorSetupResult, type AgentRegistrationServiceLike, } from './orchestrator-setup.service.js';
 //# sourceMappingURL=index.d.ts.map

package/dist/backend/backend/src/services/orchestrator/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../../backend/src/services/orchestrator/index.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EACL,uBAAuB,EACvB,0BAA0B,EAC1B,4BAA4B,GAC7B,MAAM,gCAAgC,CAAC;AAExC,OAAO,EACL,kBAAkB,EAClB,qBAAqB,EACrB,uBAAuB,EACvB,KAAK,aAAa,EAClB,KAAK,aAAa,GACnB,MAAM,2BAA2B,CAAC;AAEnC,OAAO,EACL,sBAAsB,EACtB,yBAAyB,EACzB,2BAA2B,EAC3B,KAAK,kBAAkB,EACvB,KAAK,iBAAiB,EACtB,KAAK,eAAe,GACrB,MAAM,+BAA+B,CAAC;AAEvC,OAAO,EACL,wBAAwB,EACxB,2BAA2B,EAC3B,6BAA6B,EAC7B,KAAK,iBAAiB,EACtB,KAAK,gBAAgB,EACrB,KAAK,gBAAgB,EACrB,KAAK,gBAAgB,GACtB,MAAM,iCAAiC,CAAC;AAEzC,OAAO,EACL,yBAAyB,EACzB,4BAA4B,EAC5B,8BAA8B,EAC9B,KAAK,aAAa,GACnB,MAAM,kCAAkC,CAAC;AAE1C,OAAO,EACL,oBAAoB,EACpB,aAAa,EACb,qBAAqB,EACrB,6BAA6B,EAC7B,KAAK,wBAAwB,GAC9B,MAAM,kCAAkC,CAAC;AAE1C,OAAO,EACL,0BAA0B,EAC1B,KAAK,YAAY,GAClB,MAAM,mCAAmC,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../../backend/src/services/orchestrator/index.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EACL,uBAAuB,EACvB,0BAA0B,EAC1B,4BAA4B,GAC7B,MAAM,gCAAgC,CAAC;AAExC,OAAO,EACL,kBAAkB,EAClB,qBAAqB,EACrB,uBAAuB,EACvB,KAAK,aAAa,EAClB,KAAK,aAAa,GACnB,MAAM,2BAA2B,CAAC;AAEnC,OAAO,EACL,sBAAsB,EACtB,yBAAyB,EACzB,2BAA2B,EAC3B,KAAK,kBAAkB,EACvB,KAAK,iBAAiB,EACtB,KAAK,eAAe,GACrB,MAAM,+BAA+B,CAAC;AAEvC,OAAO,EACL,wBAAwB,EACxB,2BAA2B,EAC3B,6BAA6B,EAC7B,KAAK,iBAAiB,EACtB,KAAK,gBAAgB,EACrB,KAAK,gBAAgB,EACrB,KAAK,gBAAgB,GACtB,MAAM,iCAAiC,CAAC;AAEzC,OAAO,EACL,yBAAyB,EACzB,4BAA4B,EAC5B,8BAA8B,EAC9B,KAAK,aAAa,GACnB,MAAM,kCAAkC,CAAC;AAE1C,OAAO,EACL,oBAAoB,EACpB,aAAa,EACb,qBAAqB,EACrB,6BAA6B,EAC7B,KAAK,wBAAwB,GAC9B,MAAM,kCAAkC,CAAC;AAE1C,OAAO,EACL,0BAA0B,EAC1B,KAAK,YAAY,GAClB,MAAM,mCAAmC,CAAC;AAE3C,OAAO,EACL,wBAAwB,EACxB,gCAAgC,EAChC,KAAK,uBAAuB,EAC5B,KAAK,4BAA4B,GAClC,MAAM,iCAAiC,CAAC"}

package/dist/backend/backend/src/services/orchestrator/index.js

@@ -12,4 +12,5 @@ export { ImprovementMarkerService, getImprovementMarkerService, resetImprovement
 export { ImprovementStartupService, getImprovementStartupService, resetImprovementStartupService, } from './improvement-startup.service.js';
 export { isOrchestratorActive, isAgentActive, getOrchestratorStatus, getOrchestratorOfflineMessage, } from './orchestrator-status.service.js';
 export { OrchestratorRestartService, } from './orchestrator-restart.service.js';
+export { triggerOrchestratorSetup, setOrchestratorSetupDependencies, } from './orchestrator-setup.service.js';
 //# sourceMappingURL=index.js.map

package/dist/backend/backend/src/services/orchestrator/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../../../../backend/src/services/orchestrator/index.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EACL,uBAAuB,EACvB,0BAA0B,EAC1B,4BAA4B,GAC7B,MAAM,gCAAgC,CAAC;AAExC,OAAO,EACL,kBAAkB,EAClB,qBAAqB,EACrB,uBAAuB,GAGxB,MAAM,2BAA2B,CAAC;AAEnC,OAAO,EACL,sBAAsB,EACtB,yBAAyB,EACzB,2BAA2B,GAI5B,MAAM,+BAA+B,CAAC;AAEvC,OAAO,EACL,wBAAwB,EACxB,2BAA2B,EAC3B,6BAA6B,GAK9B,MAAM,iCAAiC,CAAC;AAEzC,OAAO,EACL,yBAAyB,EACzB,4BAA4B,EAC5B,8BAA8B,GAE/B,MAAM,kCAAkC,CAAC;AAE1C,OAAO,EACL,oBAAoB,EACpB,aAAa,EACb,qBAAqB,EACrB,6BAA6B,GAE9B,MAAM,kCAAkC,CAAC;AAE1C,OAAO,EACL,0BAA0B,GAE3B,MAAM,mCAAmC,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../../../../backend/src/services/orchestrator/index.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EACL,uBAAuB,EACvB,0BAA0B,EAC1B,4BAA4B,GAC7B,MAAM,gCAAgC,CAAC;AAExC,OAAO,EACL,kBAAkB,EAClB,qBAAqB,EACrB,uBAAuB,GAGxB,MAAM,2BAA2B,CAAC;AAEnC,OAAO,EACL,sBAAsB,EACtB,yBAAyB,EACzB,2BAA2B,GAI5B,MAAM,+BAA+B,CAAC;AAEvC,OAAO,EACL,wBAAwB,EACxB,2BAA2B,EAC3B,6BAA6B,GAK9B,MAAM,iCAAiC,CAAC;AAEzC,OAAO,EACL,yBAAyB,EACzB,4BAA4B,EAC5B,8BAA8B,GAE/B,MAAM,kCAAkC,CAAC;AAE1C,OAAO,EACL,oBAAoB,EACpB,aAAa,EACb,qBAAqB,EACrB,6BAA6B,GAE9B,MAAM,kCAAkC,CAAC;AAE1C,OAAO,EACL,0BAA0B,GAE3B,MAAM,mCAAmC,CAAC;AAE3C,OAAO,EACL,wBAAwB,EACxB,gCAAgC,GAGjC,MAAM,iCAAiC,CAAC"}

package/dist/backend/backend/src/services/orchestrator/orchestrator-setup.service.d.ts

@@ -0,0 +1,114 @@
+/**
+ * Orchestrator Setup Service
+ *
+ * Provides a service-level entrypoint for triggering orchestrator setup,
+ * mirroring the logic behind `POST /api/orchestrator/setup`. Used by the
+ * SlackBridge auto-recovery path so it can reattempt setup without going
+ * through HTTP.
+ *
+ * Design notes:
+ * - Dependencies are injected via `setOrchestratorSetupDependencies()` from
+ *   the bootstrap layer (`backend/src/index.ts`), keeping this service
+ *   decoupled from the API context shape.
+ * - Concurrent calls are deduplicated via an in-flight promise — if a
+ *   setup is already running, the second caller awaits the same result.
+ * - Callers should impose their own timeout on the returned promise; this
+ *   service does not enforce a timeout because setup duration varies per
+ *   runtime type.
+ *
+ * @module services/orchestrator/orchestrator-setup.service
+ */
+import { type RuntimeType } from '../../constants.js';
+import type { StorageService } from '../core/storage.service.js';
+/**
+ * Minimal contract expected from the agent registration service.
+ *
+ * Defined as an inline structural type rather than imported to avoid a
+ * circular module graph (agent-registration → orchestrator → bridge →
+ * agent-registration). Matches `AgentRegistrationService.createAgentSession`.
+ */
+export interface AgentRegistrationServiceLike {
+    createAgentSession(config: {
+        sessionName: string;
+        role: string;
+        projectPath: string;
+        windowName: string;
+        runtimeType: RuntimeType;
+        forceRecreate?: boolean;
+        additionalAllowlistPaths?: string[];
+    }): Promise<{
+        success: boolean;
+        sessionName?: string;
+        message?: string;
+        error?: string;
+    }>;
+}
+/**
+ * Result of `triggerOrchestratorSetup()`.
+ */
+export interface OrchestratorSetupResult {
+    /** Whether the setup attempt succeeded (or the orchestrator was already healthy). */
+    success: boolean;
+    /** Session name if setup succeeded. */
+    sessionName?: string;
+    /** Human-readable status message. */
+    message?: string;
+    /** Error message if setup failed. */
+    error?: string;
+    /** True when setup was skipped because the orchestrator was already healthy. */
+    skipped?: boolean;
+}
+/**
+ * Injected dependencies wired by the bootstrap layer.
+ */
+interface SetupDependencies {
+    agentRegistrationService: AgentRegistrationServiceLike;
+    storageService: StorageService;
+}
+/**
+ * Wire the dependencies needed to trigger setup from any caller.
+ *
+ * Must be called once during bootstrap, after the agent registration service
+ * is created. Passing the same dependencies twice is safe (last write wins).
+ *
+ * @param deps - Required service dependencies
+ *
+ * @example
+ * ```typescript
+ * setOrchestratorSetupDependencies({
+ *   agentRegistrationService,
+ *   storageService,
+ * });
+ * ```
+ */
+export declare function setOrchestratorSetupDependencies(deps: SetupDependencies): void;
+/**
+ * Reset injected dependencies. Test-only helper.
+ *
+ * @internal
+ */
+export declare function _resetOrchestratorSetupDependenciesForTesting(): void;
+/**
+ * Trigger orchestrator setup if it is not already healthy.
+ *
+ * Steps:
+ * 1. Health-check via `getOrchestratorStatus()` — if active, return early.
+ * 2. Resolve runtime type from persisted orchestrator status (default: claude-code).
+ * 3. Call `createAgentSession()` with `forceRecreate: true`.
+ * 4. Initialize memory and start chat monitoring (best-effort).
+ *
+ * Concurrent callers receive the same in-flight promise — at most one setup
+ * runs at a time across the process.
+ *
+ * The orchestrator's conversation history (`AgentRunner.state.messages`) is
+ * left empty after this call. The registration path skips the activation
+ * kickoff message for the orchestrator session — this preserves the B1
+ * `messageCount === 0` cold-start invariant.
+ *
+ * @returns Setup result. `skipped: true` if the orchestrator was already healthy.
+ *
+ * @throws Never — errors are returned in the result object instead.
+ */
+export declare function triggerOrchestratorSetup(): Promise<OrchestratorSetupResult>;
+export {};
+//# sourceMappingURL=orchestrator-setup.service.d.ts.map

package/dist/backend/backend/src/services/orchestrator/orchestrator-setup.service.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"orchestrator-setup.service.d.ts","sourceRoot":"","sources":["../../../../../../backend/src/services/orchestrator/orchestrator-setup.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;GAmBG;AAEH,OAAO,EAKN,KAAK,WAAW,EAChB,MAAM,oBAAoB,CAAC;AAG5B,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,4BAA4B,CAAC;AAKjE;;;;;;GAMG;AACH,MAAM,WAAW,4BAA4B;IAC5C,kBAAkB,CAAC,MAAM,EAAE;QAC1B,WAAW,EAAE,MAAM,CAAC;QACpB,IAAI,EAAE,MAAM,CAAC;QACb,WAAW,EAAE,MAAM,CAAC;QACpB,UAAU,EAAE,MAAM,CAAC;QACnB,WAAW,EAAE,WAAW,CAAC;QACzB,aAAa,CAAC,EAAE,OAAO,CAAC;QACxB,wBAAwB,CAAC,EAAE,MAAM,EAAE,CAAC;KACpC,GAAG,OAAO,CAAC;QAAE,OAAO,EAAE,OAAO,CAAC;QAAC,WAAW,CAAC,EAAE,MAAM,CAAC;QAAC,OAAO,CAAC,EAAE,MAAM,CAAC;QAAC,KAAK,CAAC,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;CAC1F;AAED;;GAEG;AACH,MAAM,WAAW,uBAAuB;IACvC,qFAAqF;IACrF,OAAO,EAAE,OAAO,CAAC;IACjB,uCAAuC;IACvC,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,qCAAqC;IACrC,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,qCAAqC;IACrC,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,gFAAgF;IAChF,OAAO,CAAC,EAAE,OAAO,CAAC;CAClB;AAED;;GAEG;AACH,UAAU,iBAAiB;IAC1B,wBAAwB,EAAE,4BAA4B,CAAC;IACvD,cAAc,EAAE,cAAc,CAAC;CAC/B;AAQD;;;;;;;;;;;;;;;GAeG;AACH,wBAAgB,gCAAgC,CAAC,IAAI,EAAE,iBAAiB,GAAG,IAAI,CAE9E;AAED;;;;GAIG;AACH,wBAAgB,6CAA6C,IAAI,IAAI,CAGpE;AAED;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,wBAAsB,wBAAwB,IAAI,OAAO,CAAC,uBAAuB,CAAC,CAiBjF"}

package/dist/backend/backend/src/services/orchestrator/orchestrator-setup.service.js

@@ -0,0 +1,195 @@
+/**
+ * Orchestrator Setup Service
+ *
+ * Provides a service-level entrypoint for triggering orchestrator setup,
+ * mirroring the logic behind `POST /api/orchestrator/setup`. Used by the
+ * SlackBridge auto-recovery path so it can reattempt setup without going
+ * through HTTP.
+ *
+ * Design notes:
+ * - Dependencies are injected via `setOrchestratorSetupDependencies()` from
+ *   the bootstrap layer (`backend/src/index.ts`), keeping this service
+ *   decoupled from the API context shape.
+ * - Concurrent calls are deduplicated via an in-flight promise — if a
+ *   setup is already running, the second caller awaits the same result.
+ * - Callers should impose their own timeout on the returned promise; this
+ *   service does not enforce a timeout because setup duration varies per
+ *   runtime type.
+ *
+ * @module services/orchestrator/orchestrator-setup.service
+ */
+import { ORCHESTRATOR_SESSION_NAME, ORCHESTRATOR_WINDOW_NAME, ORCHESTRATOR_ROLE, RUNTIME_TYPES, } from '../../constants.js';
+import { LoggerService } from '../core/logger.service.js';
+import { formatError } from '../../utils/format-error.js';
+import { MemoryService } from '../memory/memory.service.js';
+import { getTerminalGateway } from '../../websocket/terminal.gateway.js';
+import { getOrchestratorStatus } from './orchestrator-status.service.js';
+let dependencies = null;
+let setupInFlight = null;
+const getLogger = () => LoggerService.getInstance().createComponentLogger('OrchestratorSetup');
+/**
+ * Wire the dependencies needed to trigger setup from any caller.
+ *
+ * Must be called once during bootstrap, after the agent registration service
+ * is created. Passing the same dependencies twice is safe (last write wins).
+ *
+ * @param deps - Required service dependencies
+ *
+ * @example
+ * ```typescript
+ * setOrchestratorSetupDependencies({
+ *   agentRegistrationService,
+ *   storageService,
+ * });
+ * ```
+ */
+export function setOrchestratorSetupDependencies(deps) {
+    dependencies = deps;
+}
+/**
+ * Reset injected dependencies. Test-only helper.
+ *
+ * @internal
+ */
+export function _resetOrchestratorSetupDependenciesForTesting() {
+    dependencies = null;
+    setupInFlight = null;
+}
+/**
+ * Trigger orchestrator setup if it is not already healthy.
+ *
+ * Steps:
+ * 1. Health-check via `getOrchestratorStatus()` — if active, return early.
+ * 2. Resolve runtime type from persisted orchestrator status (default: claude-code).
+ * 3. Call `createAgentSession()` with `forceRecreate: true`.
+ * 4. Initialize memory and start chat monitoring (best-effort).
+ *
+ * Concurrent callers receive the same in-flight promise — at most one setup
+ * runs at a time across the process.
+ *
+ * The orchestrator's conversation history (`AgentRunner.state.messages`) is
+ * left empty after this call. The registration path skips the activation
+ * kickoff message for the orchestrator session — this preserves the B1
+ * `messageCount === 0` cold-start invariant.
+ *
+ * @returns Setup result. `skipped: true` if the orchestrator was already healthy.
+ *
+ * @throws Never — errors are returned in the result object instead.
+ */
+export async function triggerOrchestratorSetup() {
+    if (setupInFlight) {
+        // Concurrent call — caller waits on the existing setup.
+        return setupInFlight;
+    }
+    if (!dependencies) {
+        const error = 'Orchestrator setup dependencies not initialized';
+        getLogger().error(error);
+        return { success: false, error };
+    }
+    setupInFlight = _runSetup(dependencies).finally(() => {
+        setupInFlight = null;
+    });
+    return setupInFlight;
+}
+/**
+ * Internal setup runner. Extracted for clarity — `triggerOrchestratorSetup`
+ * just guards concurrency.
+ */
+async function _runSetup(deps) {
+    const logger = getLogger();
+    try {
+        // 1. Skip when already healthy. Avoids hammering setup when SlackBridge's
+        // auto-recovery races with another caller.
+        try {
+            const currentStatus = await getOrchestratorStatus();
+            if (currentStatus.isActive) {
+                logger.info('Orchestrator is already healthy — skipping setup', {
+                    agentStatus: currentStatus.agentStatus,
+                });
+                return {
+                    success: true,
+                    skipped: true,
+                    message: 'Orchestrator is already running (setup skipped)',
+                    sessionName: ORCHESTRATOR_SESSION_NAME,
+                };
+            }
+        }
+        catch (healthErr) {
+            logger.warn('Health check failed, proceeding with setup', {
+                error: formatError(healthErr),
+            });
+        }
+        // 2. Resolve runtime type. Falls back to claude-code if unknown.
+        let runtimeType = RUNTIME_TYPES.CLAUDE_CODE;
+        try {
+            const orchestratorStatus = await deps.storageService.getOrchestratorStatus();
+            const stored = orchestratorStatus?.runtimeType;
+            if (stored && Object.values(RUNTIME_TYPES).includes(stored)) {
+                runtimeType = stored;
+            }
+        }
+        catch (lookupErr) {
+            logger.warn('Failed to read runtime type from storage; defaulting to claude-code', {
+                error: formatError(lookupErr),
+            });
+        }
+        // 3. For Gemini CLI we'd normally collect project paths for the
+        // allowlist. The auto-recovery path is best-effort and the controller
+        // already does this on the explicit setup endpoint, so we keep it
+        // simple here. The createAgentSession contract treats the parameter
+        // as optional.
+        logger.info('Triggering createAgentSession from auto-recovery', {
+            sessionName: ORCHESTRATOR_SESSION_NAME,
+            runtimeType,
+        });
+        const result = await deps.agentRegistrationService.createAgentSession({
+            sessionName: ORCHESTRATOR_SESSION_NAME,
+            role: ORCHESTRATOR_ROLE,
+            projectPath: process.cwd(),
+            windowName: ORCHESTRATOR_WINDOW_NAME,
+            runtimeType,
+            forceRecreate: true,
+        });
+        if (!result.success) {
+            logger.error('createAgentSession returned failure', { error: result.error });
+            return {
+                success: false,
+                error: result.error || 'Failed to create orchestrator session',
+            };
+        }
+        // 4. Best-effort memory init + chat monitoring. Failures here do not
+        // block setup success — the session is created, these are auxiliary.
+        try {
+            await MemoryService.getInstance().initializeForSession(ORCHESTRATOR_SESSION_NAME, ORCHESTRATOR_ROLE, process.cwd());
+        }
+        catch (memErr) {
+            logger.warn('Memory initialization failed (non-fatal)', {
+                error: formatError(memErr),
+            });
+        }
+        try {
+            const terminalGateway = getTerminalGateway();
+            if (terminalGateway) {
+                terminalGateway.startOrchestratorChatMonitoring(ORCHESTRATOR_SESSION_NAME);
+            }
+        }
+        catch (monitorErr) {
+            logger.warn('Failed to start chat monitoring (non-fatal)', {
+                error: formatError(monitorErr),
+            });
+        }
+        return {
+            success: true,
+            sessionName: result.sessionName ?? ORCHESTRATOR_SESSION_NAME,
+            message: result.message ?? 'Orchestrator session created and registered',
+        };
+    }
+    catch (error) {
+        logger.error('Auto-recovery setup failed', { error: formatError(error) });
+        return {
+            success: false,
+            error: error instanceof Error ? error.message : 'Failed to setup orchestrator session',
+        };
+    }
+}
+//# sourceMappingURL=orchestrator-setup.service.js.map

package/dist/backend/backend/src/services/orchestrator/orchestrator-setup.service.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"orchestrator-setup.service.js","sourceRoot":"","sources":["../../../../../../backend/src/services/orchestrator/orchestrator-setup.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;GAmBG;AAEH,OAAO,EACN,yBAAyB,EACzB,wBAAwB,EACxB,iBAAiB,EACjB,aAAa,GAEb,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EAAE,aAAa,EAAmB,MAAM,2BAA2B,CAAC;AAC3E,OAAO,EAAE,WAAW,EAAE,MAAM,6BAA6B,CAAC;AAE1D,OAAO,EAAE,aAAa,EAAE,MAAM,6BAA6B,CAAC;AAC5D,OAAO,EAAE,kBAAkB,EAAE,MAAM,qCAAqC,CAAC;AACzE,OAAO,EAAE,qBAAqB,EAAE,MAAM,kCAAkC,CAAC;AA6CzE,IAAI,YAAY,GAA6B,IAAI,CAAC;AAClD,IAAI,aAAa,GAA4C,IAAI,CAAC;AAElE,MAAM,SAAS,GAAG,GAAoB,EAAE,CACvC,aAAa,CAAC,WAAW,EAAE,CAAC,qBAAqB,CAAC,mBAAmB,CAAC,CAAC;AAExE;;;;;;;;;;;;;;;GAeG;AACH,MAAM,UAAU,gCAAgC,CAAC,IAAuB;IACvE,YAAY,GAAG,IAAI,CAAC;AACrB,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,6CAA6C;IAC5D,YAAY,GAAG,IAAI,CAAC;IACpB,aAAa,GAAG,IAAI,CAAC;AACtB,CAAC;AAED;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,MAAM,CAAC,KAAK,UAAU,wBAAwB;IAC7C,IAAI,aAAa,EAAE,CAAC;QACnB,wDAAwD;QACxD,OAAO,aAAa,CAAC;IACtB,CAAC;IAED,IAAI,CAAC,YAAY,EAAE,CAAC;QACnB,MAAM,KAAK,GAAG,iDAAiD,CAAC;QAChE,SAAS,EAAE,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;QACzB,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,KAAK,EAAE,CAAC;IAClC,CAAC;IAED,aAAa,GAAG,SAAS,CAAC,YAAY,CAAC,CAAC,OAAO,CAAC,GAAG,EAAE;QACpD,aAAa,GAAG,IAAI,CAAC;IACtB,CAAC,CAAC,CAAC;IAEH,OAAO,aAAa,CAAC;AACtB,CAAC;AAED;;;GAGG;AACH,KAAK,UAAU,SAAS,CAAC,IAAuB;IAC/C,MAAM,MAAM,GAAG,SAAS,EAAE,CAAC;IAC3B,IAAI,CAAC;QACJ,0EAA0E;QAC1E,2CAA2C;QAC3C,IAAI,CAAC;YACJ,MAAM,aAAa,GAAG,MAAM,qBAAqB,EAAE,CAAC;YACpD,IAAI,aAAa,CAAC,QAAQ,EAAE,CAAC;gBAC5B,MAAM,CAAC,IAAI,CAAC,kDAAkD,EAAE;oBAC/D,WAAW,EAAE,aAAa,CAAC,WAAW;iBACtC,CAAC,CAAC;gBACH,OAAO;oBACN,OAAO,EAAE,IAAI;oBACb,OAAO,EAAE,IAAI;oBACb,OAAO,EAAE,iDAAiD;oBAC1D,WAAW,EAAE,yBAAyB;iBACtC,CAAC;YACH,CAAC;QACF,CAAC;QAAC,OAAO,SAAS,EAAE,CAAC;YACpB,MAAM,CAAC,IAAI,CAAC,4CAA4C,EAAE;gBACzD,KAAK,EAAE,WAAW,CAAC,SAAS,CAAC;aAC7B,CAAC,CAAC;QACJ,CAAC;QAED,iEAAiE;QACjE,IAAI,WAAW,GAAgB,aAAa,CAAC,WAA0B,CAAC;QACxE,IAAI,CAAC;YACJ,MAAM,kBAAkB,GAAG,MAAM,IAAI,CAAC,cAAc,CAAC,qBAAqB,EAAE,CAAC;YAC7E,MAAM,MAAM,GAAG,kBAAkB,EAAE,WAAW,CAAC;YAC/C,IAAI,MAAM,IAAI,MAAM,CAAC,MAAM,CAAC,aAAa,CAAC,CAAC,QAAQ,CAAC,MAAqB,CAAC,EAAE,CAAC;gBAC5E,WAAW,GAAG,MAAqB,CAAC;YACrC,CAAC;QACF,CAAC;QAAC,OAAO,SAAS,EAAE,CAAC;YACpB,MAAM,CAAC,IAAI,CAAC,qEAAqE,EAAE;gBAClF,KAAK,EAAE,WAAW,CAAC,SAAS,CAAC;aAC7B,CAAC,CAAC;QACJ,CAAC;QAED,gEAAgE;QAChE,sEAAsE;QACtE,kEAAkE;QAClE,oEAAoE;QACpE,eAAe;QAEf,MAAM,CAAC,IAAI,CAAC,kDAAkD,EAAE;YAC/D,WAAW,EAAE,yBAAyB;YACtC,WAAW;SACX,CAAC,CAAC;QAEH,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,wBAAwB,CAAC,kBAAkB,CAAC;YACrE,WAAW,EAAE,yBAAyB;YACtC,IAAI,EAAE,iBAAiB;YACvB,WAAW,EAAE,OAAO,CAAC,GAAG,EAAE;YAC1B,UAAU,EAAE,wBAAwB;YACpC,WAAW;YACX,aAAa,EAAE,IAAI;SACnB,CAAC,CAAC;QAEH,IAAI,CAAC,MAAM,CAAC,OAAO,EAAE,CAAC;YACrB,MAAM,CAAC,KAAK,CAAC,qCAAqC,EAAE,EAAE,KAAK,EAAE,MAAM,CAAC,KAAK,EAAE,CAAC,CAAC;YAC7E,OAAO;gBACN,OAAO,EAAE,KAAK;gBACd,KAAK,EAAE,MAAM,CAAC,KAAK,IAAI,uCAAuC;aAC9D,CAAC;QACH,CAAC;QAED,qEAAqE;QACrE,qEAAqE;QACrE,IAAI,CAAC;YACJ,MAAM,aAAa,CAAC,WAAW,EAAE,CAAC,oBAAoB,CACrD,yBAAyB,EACzB,iBAAiB,EACjB,OAAO,CAAC,GAAG,EAAE,CACb,CAAC;QACH,CAAC;QAAC,OAAO,MAAM,EAAE,CAAC;YACjB,MAAM,CAAC,IAAI,CAAC,0CAA0C,EAAE;gBACvD,KAAK,EAAE,WAAW,CAAC,MAAM,CAAC;aAC1B,CAAC,CAAC;QACJ,CAAC;QAED,IAAI,CAAC;YACJ,MAAM,eAAe,GAAG,kBAAkB,EAAE,CAAC;YAC7C,IAAI,eAAe,EAAE,CAAC;gBACrB,eAAe,CAAC,+BAA+B,CAAC,yBAAyB,CAAC,CAAC;YAC5E,CAAC;QACF,CAAC;QAAC,OAAO,UAAU,EAAE,CAAC;YACrB,MAAM,CAAC,IAAI,CAAC,6CAA6C,EAAE;gBAC1D,KAAK,EAAE,WAAW,CAAC,UAAU,CAAC;aAC9B,CAAC,CAAC;QACJ,CAAC;QAED,OAAO;YACN,OAAO,EAAE,IAAI;YACb,WAAW,EAAE,MAAM,CAAC,WAAW,IAAI,yBAAyB;YAC5D,OAAO,EAAE,MAAM,CAAC,OAAO,IAAI,6CAA6C;SACxE,CAAC;IACH,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QAChB,MAAM,CAAC,KAAK,CAAC,4BAA4B,EAAE,EAAE,KAAK,EAAE,WAAW,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;QAC1E,OAAO;YACN,OAAO,EAAE,KAAK;YACd,KAAK,EAAE,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,sCAAsC;SACtF,CAAC;IACH,CAAC;AACF,CAAC"}

package/dist/backend/backend/src/services/orchestrator/orchestrator-status.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"orchestrator-status.service.d.ts","sourceRoot":"","sources":["../../../../../../backend/src/services/orchestrator/orchestrator-status.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AASH;;GAEG;AACH,MAAM,WAAW,wBAAwB;IACvC,uEAAuE;IACvE,QAAQ,EAAE,OAAO,CAAC;IAClB,mDAAmD;IACnD,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,oCAAoC;IACpC,OAAO,EAAE,MAAM,CAAC;CACjB;AAED;;;;;;;;;;;;;GAaG;AACH,wBAAsB,oBAAoB,IAAI,OAAO,CAAC,OAAO,CAAC,CAG7D;AAED;;;;;;;;;;;;;;;GAeG;AACH,wBAAsB,qBAAqB,IAAI,OAAO,CAAC,wBAAwB,CAAC,CAyI/E;AAED;;;;;;GAMG;AACH,wBAAsB,aAAa,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,CAczE;AAED;;;;;;GAMG;AACH,wBAAgB,6BAA6B,CAAC,UAAU,UAAO,GAAG,MAAM,CAMvE"}
+{"version":3,"file":"orchestrator-status.service.d.ts","sourceRoot":"","sources":["../../../../../../backend/src/services/orchestrator/orchestrator-status.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAWH;;GAEG;AACH,MAAM,WAAW,wBAAwB;IACvC,uEAAuE;IACvE,QAAQ,EAAE,OAAO,CAAC;IAClB,mDAAmD;IACnD,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,oCAAoC;IACpC,OAAO,EAAE,MAAM,CAAC;CACjB;AAED;;;;;;;;;;;;;GAaG;AACH,wBAAsB,oBAAoB,IAAI,OAAO,CAAC,OAAO,CAAC,CAG7D;AAED;;;;;;;;;;;;;;;GAeG;AACH,wBAAsB,qBAAqB,IAAI,OAAO,CAAC,wBAAwB,CAAC,CAqJ/E;AAED;;;;;;GAMG;AACH,wBAAsB,aAAa,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC,CAczE;AAED;;;;;;GAMG;AACH,wBAAgB,6BAA6B,CAAC,UAAU,UAAO,GAAG,MAAM,CAMvE"}

package/dist/backend/backend/src/services/orchestrator/orchestrator-status.service.js

@@ -10,6 +10,8 @@
 import { StorageService } from '../core/storage.service.js';
 import { CREWLY_CONSTANTS, WEB_CONSTANTS } from '../../../../config/index.js';
 import { getSessionBackendSync } from '../session/index.js';
+import { isInProcessRuntimeActive } from '../agent/crewly-agent/in-process-runtime-registry.js';
+import { RUNTIME_TYPES } from '../../constants.js';
 /** Dashboard URL for user-facing messages */
 const DASHBOARD_URL = `http://localhost:${WEB_CONSTANTS.PORTS.FRONTEND}`;
 /**
@@ -58,9 +60,9 @@ export async function getOrchestratorStatus() {
         // aligning with how the teams controller checks session existence.
         let sessionExists = false;
         let sessionCheckPerformed = false;
+        const sessionName = orchestratorStatus?.sessionName || CREWLY_CONSTANTS.SESSIONS.ORCHESTRATOR_NAME;
         try {
             const sessionBackend = getSessionBackendSync();
-            const sessionName = orchestratorStatus?.sessionName || CREWLY_CONSTANTS.SESSIONS.ORCHESTRATOR_NAME;
             if (sessionBackend && sessionName) {
                 sessionExists = sessionBackend.sessionExists(sessionName);
                 sessionCheckPerformed = true;
@@ -69,6 +71,19 @@ export async function getOrchestratorStatus() {
         catch {
             // Ignore session check errors - fall back to storage-based status
         }
+        // Runtime-aware fallback (B0 hot-fix):
+        // The PTY-based `sessionExists()` returns false for in-process Crewly
+        // Agent runtimes because they have no PTY session. Treat the session as
+        // alive when the runtime type is `crewly-agent` AND the in-process
+        // registry confirms a ready runtime is registered. The PTY path is
+        // untouched for `claude-code` / other runtimes.
+        if (!sessionExists
+            && orchestratorStatus?.runtimeType === RUNTIME_TYPES.CREWLY_AGENT
+            && sessionName
+            && isInProcessRuntimeActive(sessionName)) {
+            sessionExists = true;
+            sessionCheckPerformed = true;
+        }
         if (!orchestratorStatus) {
             return {
                 isActive: false,
@@ -99,8 +114,7 @@ export async function getOrchestratorStatus() {
         if (sessionCheckPerformed && !sessionExists && agentStatus === CREWLY_CONSTANTS.AGENT_STATUSES.ACTIVE) {
             try {
                 const storageServiceForCleanup = StorageService.getInstance();
-                const cleanupSessionName = orchestratorStatus?.sessionName || CREWLY_CONSTANTS.SESSIONS.ORCHESTRATOR_NAME;
-                await storageServiceForCleanup.updateAgentStatus(cleanupSessionName, CREWLY_CONSTANTS.AGENT_STATUSES.INACTIVE);
+                await storageServiceForCleanup.updateAgentStatus(sessionName, CREWLY_CONSTANTS.AGENT_STATUSES.INACTIVE);
             }
             catch {
                 // Best-effort cleanup — don't let it break the status check
@@ -125,7 +139,6 @@ export async function getOrchestratorStatus() {
             let childProcessAlive = false;
             try {
                 const sessionBackend = getSessionBackendSync();
-                const sessionName = orchestratorStatus?.sessionName || CREWLY_CONSTANTS.SESSIONS.ORCHESTRATOR_NAME;
                 childProcessAlive = !!sessionBackend?.isChildProcessAlive?.(sessionName);
             }
             catch {
@@ -135,8 +148,7 @@ export async function getOrchestratorStatus() {
                 // Best-effort: persist the recovered status but return active regardless
                 try {
                     const storageServiceForRecovery = StorageService.getInstance();
-                    const recoverySessionName = orchestratorStatus?.sessionName || CREWLY_CONSTANTS.SESSIONS.ORCHESTRATOR_NAME;
-                    await storageServiceForRecovery.updateAgentStatus(recoverySessionName, CREWLY_CONSTANTS.AGENT_STATUSES.ACTIVE);
+                    await storageServiceForRecovery.updateAgentStatus(sessionName, CREWLY_CONSTANTS.AGENT_STATUSES.ACTIVE);
                 }
                 catch {
                     // Best-effort — don't let persist failure block recovery