@adhdev/daemon-core 0.9.82-rc.254 → 0.9.82-rc.256

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adhdev/daemon-core",
-  "version": "0.9.82-rc.254",
+  "version": "0.9.82-rc.256",
   "description": "ADHDev daemon core — CDP, IDE detection, providers, command execution",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/commands/router.ts

@@ -4309,6 +4309,27 @@ export class DaemonCommandRouter {
             }
 
             case 'launch_cli': {
+                // Worker launch envelope hardening: a mesh worker session must carry
+                // meshCoordinatorDaemonId so completion events route back to the
+                // coordinator (the cloud relay and the daemon-core forwarder both key on
+                // it). mesh_launch_session resolves coordinatorNode.daemonId || ctx.localDaemonId
+                // upstream, but for a freshly-cloned local worktree both can be empty. When the
+                // launch is a coordinator-driven worker on this machine and the id is missing,
+                // stamp this daemon's own id — worker and coordinator are co-located here.
+                {
+                    const launchSettings = (args?.settings && typeof args.settings === 'object')
+                        ? args.settings as Record<string, unknown>
+                        : undefined;
+                    const isMeshWorkerLaunch = !!launchSettings
+                        && (readStringValue(launchSettings.meshNodeFor) || launchSettings.launchedByCoordinator === true);
+                    const hasCoordinatorDaemonId = !!launchSettings && !!readStringValue(launchSettings.meshCoordinatorDaemonId);
+                    if (launchSettings && isMeshWorkerLaunch && !hasCoordinatorDaemonId) {
+                        try {
+                            const localDaemonId = readStringValue(loadConfig().machineId);
+                            if (localDaemonId) launchSettings.meshCoordinatorDaemonId = localDaemonId;
+                        } catch { /* best-effort — launch proceeds without the stamp */ }
+                    }
+                }
                 const launchResult = await this.deps.cliManager.handleCliCommand(cmd, args);
                 // Bug C fix (part 1): when launching a mesh node worker session, surface
                 // bootstrapPending:true if the node's worktree bootstrap is still running.

package/src/providers/cli-provider-instance.ts

@@ -1088,8 +1088,31 @@ export class CliProviderInstance implements ProviderInstance {
     }
 
     private completionFinalSummary(parsedMessages: unknown): string | undefined {
-        const evidence = this.completionFinalAssistantEvidence(parsedMessages);
-        return extractFinalSummaryFromMessages(evidence.messages as any);
+        // For native-source providers (claude-cli: chatMessagesOwnedExternally), the PTY
+        // screen parse is NOT the source of truth for the final summary — the terminal
+        // wraps/scrolls/clips text, so a screen-parsed assistant message is often a partial
+        // prefix (e.g. 76 chars of a 112-char turn). The append-only native transcript holds
+        // the complete turn. Prefer it whenever it yields a longer/complete summary; fall back
+        // to the parsed screen only when the transcript is unavailable. This is the real cause
+        // of the truncated finalSummary — independent of cloud vs standalone (it surfaces on
+        // any short, fast-completing task where screen parse wins the race).
+        const adapterOwnsMessagesElsewhere = (this.adapter as any)?.chatMessagesOwnedExternally === true;
+        const parsedSummary = extractFinalSummaryFromMessages(
+            (this.completionHasFinalAssistantMessage(parsedMessages)
+                ? (Array.isArray(parsedMessages) ? parsedMessages : [])
+                : []) as any,
+        );
+        if (adapterOwnsMessagesElsewhere) {
+            const externalMessages = this.readExternalCompletionMessages();
+            const externalSummary = externalMessages
+                ? extractFinalSummaryFromMessages(externalMessages as any)
+                : '';
+            // The transcript is authoritative for native-source providers. Use it unless it is
+            // empty (not yet written) — only then fall back to whatever the screen parsed.
+            if (externalSummary) return externalSummary;
+            return parsedSummary || undefined;
+        }
+        return parsedSummary || undefined;
     }
 
     private buildCompletedFinalizationDiagnostic(args: {