fix(validate): allow undecided decisions without selected option

Only require selected option on decisions with decided lifecycle states
(accepted, implemented, adopted). Allows expressing intentionally
undecided decisions in proposed, experimental, or deferred states
without validation errors.

Fixes issue #24

Author: Mearman

src/operations/validate.ts

@@ -161,12 +161,19 @@ export const validateOp = defineOperation({
 			}
 		}
 
-		// INV13: Decisions must have options and selected
+		// INV13: Decisions must have options and selected (if decided)
 		for (const n of input.doc.nodes.filter((n) => n.type === "decision")) {
 			if (!n.options || n.options.length === 0) {
 				issues.push(`${n.id} (${n.name}): decision has no options`);
 			}
-			if (!n.selected) {
+			// Only require selected option if the decision is in a "decided" state
+			// Decided states: accepted, implemented, adopted
+			// Undecided states allowed: proposed, experimental, deferred
+			const isDecided =
+				hasLifecycleState(n, "accepted") ||
+				hasLifecycleState(n, "implemented") ||
+				hasLifecycleState(n, "adopted");
+			if (isDecided && !n.selected) {
 				issues.push(`${n.id} (${n.name}): decision has no selected option`);
 			}
 		}

tests/validate.unit.test.ts

@@ -78,20 +78,36 @@ describe("validate", () => {
 		assert.ok(result.issues.some((i) => i.includes("has no options")));
 	});
 
-	it("decisions without selected detected", () => {
+	it("decisions without selected detected (when decided)", () => {
 		const doc = makeDoc([
 			{
 				id: "DEC1",
 				type: "decision",
 				name: "Dec",
 				options: [{ id: "a", description: "A" }],
+				lifecycle: { accepted: true },
 			},
 		]);
 		const result = validateOp({ doc });
 		assert.equal(result.valid, false);
 		assert.ok(result.issues.some((i) => i.includes("has no selected")));
 	});
 
+	it("undecided decisions without selected allowed", () => {
+		const doc = makeDoc([
+			{
+				id: "DEC1",
+				type: "decision",
+				name: "Dec",
+				options: [{ id: "a", description: "A" }],
+				// No lifecycle state means undecided (like proposed)
+			},
+		]);
+		const result = validateOp({ doc });
+		assert.equal(result.valid, true);
+		assert.ok(!result.issues.some((i) => i.includes("has no selected")));
+	});
+
 	it("realises accepts capability → stage", () => {
 		const doc = makeDoc(
 			[