@wrongstack/requirement-intake 1.0.27 → 1.0.28

package/dist/index.js

@@ -386,9 +386,7 @@ function pendingQuestions(record) {
   return record.questions.filter((question) => question.status === "unanswered").sort((a, b) => a.order - b.order);
 }
 function upsertQuestion(record, template) {
-  const existing = record.questions.find(
-    (question) => question.field === template.field && question.status !== "skipped"
-  );
+  const existing = record.questions.find((question) => question.field === template.field);
   if (existing) return false;
   record.questions.push(makeQuestion(template, record.questions.length, false));
   return true;
@@ -460,8 +458,11 @@ var RequirementIntakeStore = class {
     try {
       const raw = await fsp.readFile(this.recordPath(id), "utf8");
       return JSON.parse(raw);
-    } catch {
-      return null;
+    } catch (error) {
+      if (error instanceof IntakeValidationError || error.code === "ENOENT") {
+        return null;
+      }
+      throw error;
     }
   }
   async exists(id) {
@@ -1704,22 +1705,22 @@ var RequirementIntakeService = class {
   async cancelIntake(id, ctx, reason, expectedVersion) {
     const record = await this.requireRecord(id, ctx, "cancel");
     assertTransition(record.status, "cancelled");
-    const updated = await this.store.update(
-      id,
-      {
-        actorId: ctx.id,
-        actorType: ctx.type,
-        action: "cancelled",
-        from: record.status,
-        to: "cancelled",
-        expectedVersion
-      },
-      (next) => {
-        next.status = "cancelled";
-        next.cancelledAt = Date.now();
-        next.cancelledReason = reason?.trim() || void 0;
-      }
-    );
+    let previousStatus = record.status;
+    const meta = {
+      actorId: ctx.id,
+      actorType: ctx.type,
+      action: "cancelled",
+      from: record.status,
+      to: "cancelled",
+      expectedVersion
+    };
+    const updated = await this.store.update(id, meta, (next) => {
+      assertTransition(next.status, "cancelled");
+      previousStatus = meta.from = next.status;
+      next.status = "cancelled";
+      next.cancelledAt = Date.now();
+      next.cancelledReason = reason?.trim() || void 0;
+    });
     this.metrics.increment("intake.cancelled");
     this.logger.info("intake", "intake.cancelled", {
       intakeId: updated.id,
@@ -1731,7 +1732,7 @@ var RequirementIntakeService = class {
       projectId: updated.projectId,
       actorId: ctx.id,
       actorType: ctx.type,
-      previousStatus: record.status,
+      previousStatus,
       status: "cancelled"
     });
     return updated;
@@ -1739,21 +1740,21 @@ var RequirementIntakeService = class {
   async archiveIntake(id, ctx, expectedVersion) {
     const record = await this.requireRecord(id, ctx, "archive");
     assertTransition(record.status, "archived");
-    const updated = await this.store.update(
-      id,
-      {
-        actorId: ctx.id,
-        actorType: ctx.type,
-        action: "archived",
-        from: record.status,
-        to: "archived",
-        expectedVersion
-      },
-      (next) => {
-        next.status = "archived";
-        next.archivedAt = Date.now();
-      }
-    );
+    let previousStatus = record.status;
+    const meta = {
+      actorId: ctx.id,
+      actorType: ctx.type,
+      action: "archived",
+      from: record.status,
+      to: "archived",
+      expectedVersion
+    };
+    const updated = await this.store.update(id, meta, (next) => {
+      assertTransition(next.status, "archived");
+      previousStatus = meta.from = next.status;
+      next.status = "archived";
+      next.archivedAt = Date.now();
+    });
     this.metrics.increment("intake.archived");
     this.logger.info("intake", "intake.archived", {
       intakeId: updated.id,
@@ -1765,7 +1766,7 @@ var RequirementIntakeService = class {
       projectId: updated.projectId,
       actorId: ctx.id,
       actorType: ctx.type,
-      previousStatus: record.status,
+      previousStatus,
       status: "archived"
     });
     return updated;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wrongstack/requirement-intake",
-  "version": "1.0.27",
+  "version": "1.0.28",
   "license": "MIT",
   "description": "WrongStack Requirements Intake — collect, preserve, validate, normalize, and submit unstructured software development requests as structured intake records. Upstream of spec-driven development: it never plans, specifies, or implements.",
   "repository": {
@@ -27,7 +27,7 @@
     "!dist/**/*.map"
   ],
   "dependencies": {
-    "@wrongstack/core": "1.0.27",
+    "@wrongstack/core": "1.0.28",
     "zod": "4.6.2"
   },
   "devDependencies": {