@try-works/dsh-memory 0.1.12 → 0.1.13

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@try-works/dsh-memory",
-  "version": "0.1.12",
+  "version": "0.1.13",
   "description": "Git-backed, workspace-scoped memory for the DeepSeek Harness (global layer + per-workspace layers)",
   "type": "module",
   "main": "src/index.js",

package/src/tasks.js

@@ -151,7 +151,7 @@ const STATE_MEANING = {
   [TASK_STATES.STOPPED]: 'was armed, and its owning session stopped, so the host deleted it',
   [TASK_STATES.FAILED]: 'was due and has still not delivered; the host records no failure of its own',
   [TASK_STATES.ORPHANED]: 'armed, but the session that owns it cannot be resolved, so it can never fire',
-  [TASK_STATES.SKIPPED]: 'seen in the live store but not captured, because its kind is not one we can read',
+  [TASK_STATES.SKIPPED]: 'captured, but its kind is not one we can re-arm; a restore refuses it rather than guessing',
   [TASK_STATES.DONE]: 'delivered, or its recurrence is exhausted',
 }
 
@@ -397,8 +397,8 @@ export function readScheduleStore(path) {
  * deleted on purpose must stay visible as gone rather than vanish from the record of what this
  * machine used to run.
  *
- * A task whose kind we do not recognise is SKIPPED, not written: storing it would mean restoring
- * it later without understanding it. It is reported, so the omission is visible.
+ * A task whose kind we do not recognise is stored and flagged SKIPPED: dropping it would mean losing
+ * a task from a backup, and flagging it means a restore refuses it rather than guessing.
  */
 export function backupTasks(root, { schedulePath, now = Date.now(), nowIso, sessions } = {}) {
   const at = nowIso ?? new Date(now).toISOString()
@@ -424,7 +424,13 @@ export function backupTasks(root, { schedulePath, now = Date.now(), nowIso, sess
     const record = task.record
     if (!record || typeof record !== 'object') { skipped.push({ id: task.id, reason: 'no record' }); continue }
     if (!KNOWN_KINDS.includes(String(record.kind))) {
+      // CAPTURED, not dropped. An earlier version skipped these entirely, which meant a task whose
+      // kind we did not recognise silently vanished from a BACKUP - the one job this layer has. It
+      // is written and flagged 'skipped' instead, and planRestore refuses that state, so it can
+      // never be blindly re-armed. Preserved, visible, and unrestorable: all three on purpose.
       skipped.push({ id: task.id, reason: 'unrecognised kind: ' + String(record.kind) })
+      writeTask(root, record, TASK_STATES.SKIPPED, byKey.get(contentKey(record))?.backedUpAt ?? at)
+      seen.add(contentKey(record))
       continue
     }
     const key = contentKey(record)