sysprom 1.24.0 → 1.25.1

package/dist/src/cli/commands/update.js

@@ -51,10 +51,15 @@ const updateNodeArgs = z.object({
     id: z.string().describe("node ID to update"),
 });
 const updateNodeOpts = mutationOpts.extend({
+    name: z.string().optional().describe("update node name"),
     description: z.string().optional().describe("update node description"),
     status: NodeStatus.optional().describe("set lifecycle state to true"),
     context: z.string().optional().describe("update node context"),
     rationale: z.string().optional().describe("update node rationale"),
+    selected: z
+        .string()
+        .optional()
+        .describe("set selected option on decision nodes"),
     lifecycle: z
         .array(z.string())
         .optional()
@@ -95,12 +100,16 @@ const nodeSubcommand = {
             process.exit(1);
         }
         const fields = {};
+        if (opts.name !== undefined)
+            fields.name = opts.name;
         if (opts.description !== undefined)
             fields.description = opts.description;
         if (opts.context !== undefined)
             fields.context = opts.context;
         if (opts.rationale !== undefined)
             fields.rationale = opts.rationale;
+        if (opts.selected !== undefined)
+            fields.selected = opts.selected;
         const lifecycleArgs = [...(opts.lifecycle ?? [])];
         if (opts.status !== undefined) {
             lifecycleArgs.push(`${opts.status}=true`);
@@ -111,7 +120,7 @@ const nodeSubcommand = {
         }
         if (Object.keys(fields).length === 0) {
             console.error("No fields specified to update.");
-            console.error("Use --description, --status, --context, --rationale, or --lifecycle.");
+            console.error("Use --name, --description, --status, --context, --rationale, --selected, or --lifecycle.");
             process.exit(1);
         }
         const newDoc = updateNodeOp({ doc, id: args.id, fields });

package/dist/src/cli/define-command.js

@@ -160,11 +160,11 @@ function registerOption(cmd, flagName, field, desc) {
             opt.makeOptionMandatory(true);
         cmd.addOption(opt);
     }
-    else if (optional) {
-        cmd.addOption(new Option(`--${flagName} <value>`).hideHelp());
-    }
     else {
-        cmd.addOption(new Option(`--${flagName} <value>`).makeOptionMandatory(true).hideHelp());
+        const opt = new Option(`--${flagName} <value>`, desc);
+        if (!optional)
+            opt.makeOptionMandatory(true);
+        cmd.addOption(opt);
     }
 }
 function registerOptions(cmd, optsSchema) {

package/dist/src/operations/validate.js

@@ -118,12 +118,18 @@ export const validateOp = defineOperation({
                 issues.push(`${n.id} (${n.name}): affects domain nodes but has no must_preserve relationship`);
             }
         }
-        // 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`);
             }
         }

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "sysprom",
-	"version": "1.24.0",
+	"version": "1.25.1",
 	"description": "SysProM — System Provenance Model CLI and library",
 	"author": "ExaDev",
 	"homepage": "https://exadev.github.io/SysProM",