@goodandready/dsh-cron 0.3.7 → 0.3.9

package/README.md

@@ -607,6 +607,14 @@ Save LLM tokens and API quotas on watcher tasks by dynamically slowing down exec
 - Multiplies interval by `backoffFactor` up to `maxInterval` after `idleRunsThreshold` silent or unchanged runs.
 - Instantly resets to `minInterval` as soon as real changes or non-silent output are detected.
 
+### 🔄 Session Resumption & GUI Session Naming (#346, GH #9, #354, GH #10, Added in v0.3.8, Patched in v0.3.9)
+
+- **HTTP API `sessionTitle` Persistence (v0.3.9, #354, GH #10, PR #11)**: Tasks created or updated via HTTP endpoints (`POST /dsh-cron/tasks` and `POST /dsh-cron/api/tasks`) now reliably forward and persist `sessionTitle` into the task store and runner, mirroring the agent-tool (`cron_create_task`) and PATCH pathways.
+
+- **Universal Host Resumption (`resumeSessionId`)**: DeepSeek Harness host `0.2.0-rc.2+` specifies `readonly resumeSessionId: SessionId` on `agents.resume()`. The runner now passes both `resumeSessionId` and `sessionId`, guaranteeing seamless session resumption across all DSH core versions without falling back to recreating sessions that already exist.
+- **Enhanced Diagnostics on Failure**: When an existing session cannot be resumed, `dsh-cron` logs an actionable warning and, if recreating the session fails, surfaces the underlying resume error in the task run output.
+- **Automatic GUI Session Naming (`sessionTitle`)**: Executed or resumed sessions now integrate with the host's `sessionTitle` service. Tasks can specify an explicit `sessionTitle` (or begin `task.title` with `[...]`) to automatically label the session in the DeepSeek Harness navigation sidebar instead of leaving it untitled (`cron-exec-...`).
+
 ### 📺 Real-Time Streaming Logs via SSE (#336)
 
 Connect to `GET /dsh-cron/tasks/:id/stream` using Server-Sent Events (SSE) to receive live stdout/stderr chunks and execution completion events with an automatic 500-line ring buffer.

package/lib/api.js

@@ -639,6 +639,7 @@ function mergeExecutionFields(current, body, parsed) {
     agentPreset: body.agentPreset !== undefined ? String(body.agentPreset).trim() : (current ? (current.agentPreset || '') : ''),
     targetSessionId: body.targetSessionId !== undefined ? String(body.targetSessionId).trim() : (current ? (current.targetSessionId || '') : ''),
     targetSessionReset: body.targetSessionReset !== undefined ? String(body.targetSessionReset).trim() : (current ? (current.targetSessionReset || 'never') : 'never'),
+    sessionTitle: body.sessionTitle !== undefined ? String(body.sessionTitle).trim() : (current ? (current.sessionTitle || '') : ''),
     onSuccess: body.onSuccess !== undefined ? String(body.onSuccess).trim() : (current ? (current.onSuccess || '') : ''),
     onFailure: body.onFailure !== undefined ? String(body.onFailure).trim() : (current ? (current.onFailure || '') : ''),
     heartbeatIntervalSeconds: body.heartbeatIntervalSeconds !== undefined ? Number(body.heartbeatIntervalSeconds) : (current ? (Number(current.heartbeatIntervalSeconds) || 0) : 0),

package/lib/runner.js

@@ -475,11 +475,13 @@ export class SessionRunner {
         }
       } : {};
 
-      // If targetSessionId is set, attempt to resume the existing session (#143)
+      // If targetSessionId is set, attempt to resume the existing session (#143, #346, GH #9)
+      let resumeFailure = null;
       if (prep.targetSessionId && this.ctx.agents && typeof this.ctx.agents.resume === 'function') {
         try {
           handle = await this.ctx.agents.resume({
             sessionId: prep.targetSessionId,
+            resumeSessionId: prep.targetSessionId,
             meta: {
               cwd: prep.cwd,
               ...(preset ? { agentPreset: preset.id } : {})
@@ -490,27 +492,36 @@ export class SessionRunner {
           isResumed = true;
           logger.info(`[dsh-cron] resumed existing persistent session: ${prep.targetSessionId}`);
         } catch (resumeErr) {
-          logger.info(`[dsh-cron] could not resume session ${prep.targetSessionId} (${resumeErr.message}), creating new persistent session`);
+          resumeFailure = resumeErr;
+          logger.warn(`[dsh-cron] could not resume session ${prep.targetSessionId} (${resumeErr.message}), fallback creating new session`);
           handle = null;
         }
       }
 
       // Create new session if not resumed
       if (!handle) {
-        handle = await this.ctx.agents.create({
-          sessionId: prep.mintSessionId(),
-          meta: {
-            cwd: prep.cwd,
-            ephemeral: !prep.targetSessionId,
-            internal: !prep.targetSessionId,
-            ...(preset ? { agentPreset: preset.id } : {})
-          },
-          agentOptions: prep.provider && prep.model ? { provider: prep.provider, model: prep.model } : undefined,
-          ...presetSetup,
-        });
+        try {
+          handle = await this.ctx.agents.create({
+            sessionId: prep.mintSessionId(),
+            meta: {
+              cwd: prep.cwd,
+              ephemeral: !prep.targetSessionId,
+              internal: !prep.targetSessionId,
+              ...(preset ? { agentPreset: preset.id } : {})
+            },
+            agentOptions: prep.provider && prep.model ? { provider: prep.provider, model: prep.model } : undefined,
+            ...presetSetup,
+          });
+        } catch (createErr) {
+          if (resumeFailure) {
+            throw new Error(`could not resume session "${prep.targetSessionId}" (${resumeFailure.message}): ${createErr.message}`);
+          }
+          throw createErr;
+        }
       }
 
       this._applyPermissionPreset(task, handle);
+      this._nameSession(task, handle, isResumed);
       await prep.raceAbort(handle.agent.whenIdle());
 
       // 1. Capture starting sequence before sending user message (#227, #228)
@@ -639,6 +650,27 @@ export class SessionRunner {
     }
   }
 
+  _nameSession(task, handle, isResumed) {
+    if (!handle?.agent?.session) return;
+    const session = handle.agent.session;
+    try {
+      const sessionTitle = this.ctx.get ? this.ctx.get('sessionTitle') : this.ctx.sessionTitle;
+      if (!sessionTitle || typeof sessionTitle.rename !== 'function') return;
+
+      const explicitTitle = task.sessionTitle ? String(task.sessionTitle).trim() : '';
+      const taskTitle = task.title ? String(task.title).trim() : '';
+      const label = explicitTitle || (taskTitle.startsWith('[') ? taskTitle : '');
+
+      if (!label) return;
+      if (isResumed && !explicitTitle) return;
+
+      sessionTitle.rename(session, label);
+      logger.info(`[dsh-cron] assigned session title "${label}" for session ${session.id}`);
+    } catch (titleErr) {
+      logger.debug?.(`[dsh-cron] could not rename session ${session?.id}: ${titleErr.message}`);
+    }
+  }
+
   /** Token usage reported by session events or the handle (#228). */
   _extractUsage(handle, turnEvents = []) {
     const usage = { inputTokens: 0, outputTokens: 0, cacheReadTokens: 0 };

package/lib/store.js

@@ -628,6 +628,7 @@ export class TaskStore {
       agentPreset: task.agentPreset !== undefined ? String(task.agentPreset).trim() : (prev.agentPreset || ''),
       targetSessionId: task.targetSessionId !== undefined ? String(task.targetSessionId).trim() : (prev.targetSessionId || ''),
       targetSessionReset: task.targetSessionReset !== undefined ? String(task.targetSessionReset).trim() : (prev.targetSessionReset || 'never'),
+      sessionTitle: task.sessionTitle !== undefined ? String(task.sessionTitle).trim() : (prev.sessionTitle || ''),
       totalTokens: task.totalTokens !== undefined ? Number(task.totalTokens) : (prev.totalTokens || 0),
       totalCostUsd: task.totalCostUsd !== undefined ? Number(task.totalCostUsd) : (prev.totalCostUsd || 0),
       createdAt: prev.createdAt || task.createdAt || now,

package/lib/task-create.js

@@ -48,6 +48,7 @@ export function executeCreateTask(store, scheduler, args) {
     agentPreset: args.agentPreset ? String(args.agentPreset).trim() : '',
     targetSessionId: args.targetSessionId ? String(args.targetSessionId).trim() : '',
     targetSessionReset: args.targetSessionReset ? String(args.targetSessionReset).trim() : 'never',
+    sessionTitle: args.sessionTitle ? String(args.sessionTitle).trim() : '',
     onSuccess: args.onSuccess ? String(args.onSuccess).trim() : '',
     onFailure: args.onFailure ? String(args.onFailure).trim() : '',
     dependsOn: args.dependsOn !== undefined ? sanitizeDependsOn(args.dependsOn) : undefined,

package/lib/task-transfer.js

@@ -33,6 +33,7 @@ export const PATCHABLE_TASK_FIELDS = [
   'agentPreset',
   'targetSessionId',
   'targetSessionReset',
+  'sessionTitle',
   'env',
   'cwd',
   'workspaceId',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-cron",
-  "version": "0.3.7",
+  "version": "0.3.9",
   "description": "Background automation runner for DSH: isolated agent runs, script/HTTP/SSH/Docker runtimes, cost guard, notifications, heartbeats.",
   "type": "module",
   "main": "lib/index.js",