@try-works/dsh-memory 0.1.13 → 0.1.14

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@try-works/dsh-memory",
-  "version": "0.1.13",
+  "version": "0.1.14",
   "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

@@ -56,14 +56,22 @@ export const TASK_STATES = {
  * without one.
  */
 export const TASK_STATE_EXITS = {
-  active: ['stopped', 'done', 'failed'],
-  pending: ['active'],
+  // Two entries were missing, and NOTHING noticed until writeTask began reading this table.
+  //
+  // 'orphaned': the session holding a live task can disappear while the task itself does not
+  // change, so an armed task can become one that can never fire.
+  //
+  // 'paused' in every row: holding a task back is a DECISION the user makes about a task in any
+  // condition, not a lifecycle step. Writing the table without it left the entire pause path
+  // undeclared - the feature worked, and the declaration that was supposed to describe it did not.
+  active: ['stopped', 'done', 'failed', 'orphaned', 'paused'],
+  pending: ['active', 'paused'],
   paused: ['pending'],
-  stopped: ['pending'],
-  failed: ['active'],
-  orphaned: ['pending'],
-  skipped: ['active'],
-  done: ['active'],
+  stopped: ['pending', 'paused'],
+  failed: ['active', 'paused'],
+  orphaned: ['pending', 'paused'],
+  skipped: ['active', 'paused'],
+  done: ['active', 'paused'],
 }
 
 const STATES = Object.values(TASK_STATES)
@@ -183,6 +191,18 @@ function ensureTasksDir(root) {
 export function writeTask(root, record, state, nowIso) {
   ensureTasksDir(root)
   const file = taskFilePath(root, record)
+  // ENFORCED, not documented. This table sat in the file unread since it was written, so a state
+  // could move anywhere and nothing objected - the same shape as the vocabulary states that had a
+  // test and no producer. Refusing here means a task cannot silently leave a state by a route
+  // nobody declared, which is what makes 'every resting state has a working exit' checkable
+  // rather than aspirational.
+  const previous = readTaskFile(file)
+  if (previous && previous.stateKnown && previous.state !== state) {
+    const allowed = TASK_STATE_EXITS[previous.state] ?? []
+    if (!allowed.includes(state)) {
+      throw new Error('tasks: refusing the transition ' + previous.state + ' -> ' + state + ' for "' + String(record.title ?? '') + '"; allowed from ' + previous.state + ': ' + allowed.join(', '))
+    }
+  }
   writeFileSync(file, JSON.stringify({ backedUpAt: nowIso, state, record }, null, 2) + '\n', 'utf8')
   return file;
 }