@goodandready/dsh-cron 0.3.16 → 0.3.18

package/lib/cron-tool.js

@@ -152,6 +152,10 @@ export const cronToolParameters = {
     enum: ['default', 'read-only', 'workspace-write', 'full'],
     description: 'Permission preset applied to the task session (only used for agent session tasks: llm, skill, workflow)',
   },
+  sessionTitle: {
+    type: 'string',
+    description: 'Custom session title template for isolated agent runs (supports {{date}} placeholder, e.g. "{{date}} [Daily] Report")',
+  },
   requireApproval: {
     type: 'boolean',
     description: 'Require explicit human approval before execution (#318)',

package/lib/http-utils.js

@@ -78,10 +78,16 @@ export function parseJsonBody(req, limit = MAX_BODY_BYTES) {
 
 /**
  * Safe helper to get the DSH connection service from context or request (#357).
+ * Defensively degrades when context is a Cordis proxy throwing without inject (#366, GH #13).
  */
 export function getConnection(ctx) {
   if (!ctx) return null;
-  return Reflect.get(ctx, 'connection') || ctx.connection || null;
+  try {
+    return Reflect.get(ctx, 'connection') || ctx.connection || null;
+  } catch (_err) {
+    // Degrade gracefully if context is a Cordis proxy throwing without inject (GH #13)
+    return null;
+  }
 }
 
 export function getDshConnection(req, options = {}) {

package/lib/runner.js

@@ -38,15 +38,19 @@ export {
  * Resolves the effective target session ID taking into account rotation policies (#143).
  * Supports {{date}} interpolation, 'daily' rotation (adds -YYYY-MM-DD), and 'weekly' rotation (-YYYY-Www).
  */
+export function formatDateYmd(date = new Date()) {
+  const yyyy = date.getFullYear();
+  const mm = String(date.getMonth() + 1).padStart(2, '0');
+  const dd = String(date.getDate()).padStart(2, '0');
+  return `${yyyy}-${mm}-${dd}`;
+}
+
 export function resolveTargetSessionId(targetSessionId, resetPolicy = 'never', date = new Date()) {
   if (!targetSessionId || typeof targetSessionId !== 'string') return null;
   let baseId = targetSessionId.trim();
   if (!baseId) return null;
 
-  const yyyy = date.getFullYear();
-  const mm = String(date.getMonth() + 1).padStart(2, '0');
-  const dd = String(date.getDate()).padStart(2, '0');
-  const yyyymmdd = `${yyyy}-${mm}-${dd}`;
+  const yyyymmdd = formatDateYmd(date);
 
   if (baseId.includes('{{date}}')) {
     return baseId.replaceAll('{{date}}', yyyymmdd);
@@ -282,8 +286,11 @@ export class SessionRunner {
   /** Legacy session service path; falls back to the bare report. */
   async _runLegacySession(task) {
     try {
+      const ymd = formatDateYmd();
+      const explicit = task.sessionTitle ? String(task.sessionTitle).trim().replaceAll('{{date}}', ymd) : '';
+      const legacyTitle = explicit || `[Cron] ${ymd} ${task.title}`;
       const session = await this.ctx.session.create({
-        title: `[Cron] ${task.title}`,
+        title: legacyTitle,
         ephemeral: task.delivery === 'isolated',
       });
       if (session && typeof session.send === 'function') {
@@ -659,13 +666,22 @@ export class SessionRunner {
 
       const explicitTitle = task.sessionTitle ? String(task.sessionTitle).trim() : '';
       const taskTitle = task.title ? String(task.title).trim() : '';
-      const label = explicitTitle || (taskTitle.startsWith('[') ? taskTitle : '');
+      const ymd = formatDateYmd();
+
+      let label = '';
+      if (explicitTitle) {
+        label = explicitTitle.replaceAll('{{date}}', ymd);
+      } else if (taskTitle.startsWith('[')) {
+        label = taskTitle.replaceAll('{{date}}', ymd);
+      } else if (taskTitle) {
+        label = `${ymd} ${taskTitle}`;
+      }
 
-      if (!label) return;
+      if (!label.trim()) return;
       if (isResumed && !explicitTitle) return;
 
-      sessionTitle.rename(session, label);
-      logger.info(`[dsh-cron] assigned session title "${label}" for session ${session.id}`);
+      sessionTitle.rename(session, label.trim());
+      logger.info(`[dsh-cron] assigned session title "${label.trim()}" for session ${session.id}`);
     } catch (titleErr) {
       logger.debug?.(`[dsh-cron] could not rename session ${session?.id}: ${titleErr.message}`);
     }

package/package.json

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