@uoyo/mvtt 2.2.0 → 2.2.1

package/dist/scripts/plan-update.cjs

@@ -7397,12 +7397,15 @@ function parseArgs(argv) {
   }
   return args2;
 }
+function hasValue(value) {
+  return value !== void 0 && value !== true && String(value).trim() !== "";
+}
 function validateArgs(args2) {
   if (args2.validate) return null;
-  if (!args2.plan || args2.plan === true) return ERRORS.MISSING_PLAN();
-  if (!args2.task || args2.task === true) return ERRORS.MISSING_TASK();
-  const hasStatus = args2.status && args2.status !== true;
-  const hasMutation = hasStatus || args2.artifacts && args2.artifacts !== true || args2.notes && args2.notes !== true || args2["deliverables-pointer"] && args2["deliverables-pointer"] !== true || args2["mark-deliverable-stale"] && args2["mark-deliverable-stale"] !== true;
+  if (!hasValue(args2.plan)) return ERRORS.MISSING_PLAN();
+  if (!hasValue(args2.task)) return ERRORS.MISSING_TASK();
+  const hasStatus = hasValue(args2.status);
+  const hasMutation = hasStatus || hasValue(args2.artifacts) || hasValue(args2.notes) || hasValue(args2["deliverables-pointer"]) || hasValue(args2["mark-deliverable-stale"]);
   if (!hasMutation) return ERRORS.MISSING_STATUS();
   if (args2.status === true) return ERRORS.MISSING_STATUS();
   if (hasStatus && !VALID_STATUSES.includes(args2.status)) return ERRORS.INVALID_STATUS(args2.status);
@@ -7411,10 +7414,10 @@ function validateArgs(args2) {
 function applyUpdate(plan, args2, now) {
   const task = plan.tasks.find((t) => t.id === args2.task);
   const oldStatus = task.status;
-  if (args2.status && args2.status !== true) {
+  if (hasValue(args2.status)) {
     task.status = args2.status;
   }
-  if (args2.artifacts && args2.artifacts !== true) {
+  if (hasValue(args2.artifacts)) {
     const incoming = args2.artifacts.split(",").map((s) => s.trim()).filter(Boolean);
     if (incoming.length) {
       if (!task.artifacts || typeof task.artifacts !== "object") {
@@ -7432,23 +7435,23 @@ function applyUpdate(plan, args2, now) {
       }
     }
   }
-  if (args2.notes && args2.notes !== true) {
+  if (hasValue(args2.notes)) {
     task.notes = args2.notes;
   }
-  if (args2.status && args2.status !== true) {
+  if (hasValue(args2.status)) {
     if (args2.status === "done" && !task.completed_at) {
       task.completed_at = now;
     } else if (args2.status !== "done") {
       task.completed_at = null;
     }
   }
-  if (args2["deliverables-pointer"] && args2["deliverables-pointer"] !== true) {
+  if (hasValue(args2["deliverables-pointer"])) {
     if (args2["deliverables-pointer"] !== "current") {
       return { error: ERRORS.INVALID_DELIVERABLES_POINTER(args2["deliverables-pointer"]) };
     }
     task.deliverables = { freshness: "current" };
   }
-  if (args2["mark-deliverable-stale"] && args2["mark-deliverable-stale"] !== true) {
+  if (hasValue(args2["mark-deliverable-stale"])) {
     const staleIds = args2["mark-deliverable-stale"].split(",").map((s) => s.trim()).filter(Boolean);
     for (const staleTaskId of staleIds) {
       const staleTask = plan.tasks.find((t) => t.id === staleTaskId);
@@ -7706,7 +7709,7 @@ function main() {
     process.exit(1);
   }
   let projectList = null;
-  if (args2.projects && args2.projects !== true) {
+  if (hasValue(args2.projects)) {
     projectList = args2.projects.split(",").map((s) => s.trim()).filter(Boolean);
   } else {
     projectList = deriveProjectList(plan.tasks);

package/dist/scripts/session-update.cjs

@@ -7349,7 +7349,8 @@ var ERRORS = {
   CLOSE_NEW_EPIC_CONFLICT: () => "--close-epic and --new-epic are mutually exclusive",
   NO_ACTIVE_EPIC: (flag) => `${flag} requires an active epic (active_epic.id is empty)`,
   EPIC_ID_ORPHAN: () => "--epic-id (for sub-change) requires --new-change",
-  MISSING_REMOVE_VALUE: () => "--remove-change / --remove-epic requires a non-empty value"
+  MISSING_REMOVE_VALUE: () => "--remove-change / --remove-epic requires a non-empty value",
+  MISSING_FLAG_VALUE: (flag) => `${flag} requires a non-empty value`
 };
 var DEFAULT_LIMITS = {
   history: 20,
@@ -7388,6 +7389,9 @@ function parseIdList(value) {
   if (value == null) return [];
   return String(value).split(",").map((s) => s.trim()).filter(Boolean);
 }
+function hasValue(value) {
+  return value !== void 0 && value !== true && String(value).trim() !== "";
+}
 function loadHistoryLimits(configPath) {
   const limits = { ...DEFAULT_LIMITS };
   if (!(0, import_node_fs.existsSync)(configPath)) return limits;
@@ -7413,11 +7417,22 @@ function loadHistoryLimits(configPath) {
 }
 function validate(args) {
   if (!args.skill) return ERRORS.MISSING_SKILL();
+  if (args.skill === true) return ERRORS.MISSING_FLAG_VALUE("--skill");
   if (!args.summary) return ERRORS.MISSING_SUMMARY();
+  if (args.summary === true) return ERRORS.MISSING_FLAG_VALUE("--summary");
+  if (args["new-change"] !== void 0 && !hasValue(args["new-change"])) return ERRORS.MISSING_FLAG_VALUE("--new-change");
   if (args["new-change"] && !args["change-id"]) return ERRORS.CHANGE_ID_REQUIRED();
+  if (args["change-id"] !== void 0 && !hasValue(args["change-id"])) return ERRORS.MISSING_FLAG_VALUE("--change-id");
   if (args["new-epic"] && !args["epic-id"]) return ERRORS.EPIC_ID_REQUIRED();
+  if (args["new-epic"] !== void 0 && !hasValue(args["new-epic"])) return ERRORS.MISSING_FLAG_VALUE("--new-epic");
+  if (args["epic-id"] !== void 0 && !hasValue(args["epic-id"])) return ERRORS.MISSING_FLAG_VALUE("--epic-id");
   if (args["close-epic"] && args["new-epic"]) return ERRORS.CLOSE_NEW_EPIC_CONFLICT();
   if (args["epic-id"] && !args["new-change"] && !args["new-epic"]) return ERRORS.EPIC_ID_ORPHAN();
+  for (const flag of ["set-plan-path", "set-change-status", "truncate-history", "set-epic-path", "set-epic-status"]) {
+    if (args[flag] !== void 0 && !hasValue(args[flag])) {
+      return ERRORS.MISSING_FLAG_VALUE(`--${flag}`);
+    }
+  }
   if (args["remove-change"] !== void 0 && (args["remove-change"] === true || !String(args["remove-change"]).trim())) {
     return ERRORS.MISSING_REMOVE_VALUE();
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uoyo/mvtt",
-  "version": "2.2.0",
+  "version": "2.2.1",
   "description": "My Virtual Tech Team - AI-guided prompt orchestration framework",
   "type": "module",
   "bin": {
@@ -43,10 +43,10 @@
   },
   "devDependencies": {
     "@types/node": "^25.6.0",
-    "@vitest/coverage-v8": "^2.1.9",
+    "@vitest/coverage-v8": "^4.1.9",
     "esbuild": "^0.28.0",
     "typescript": "^5.4.0",
-    "vitest": "^2.0.0"
+    "vitest": "^4.1.9"
   },
   "dependencies": {
     "@clack/prompts": "^1.5.1",

package/sources/scripts/plan-update.js

@@ -116,16 +116,20 @@ function parseArgs(argv) {
   return args;
 }
 
+function hasValue(value) {
+  return value !== undefined && value !== true && String(value).trim() !== "";
+}
+
 function validateArgs(args) {
   if (args.validate) return null;
-  if (!args.plan || args.plan === true) return ERRORS.MISSING_PLAN();
-  if (!args.task || args.task === true) return ERRORS.MISSING_TASK();
-  const hasStatus = args.status && args.status !== true;
+  if (!hasValue(args.plan)) return ERRORS.MISSING_PLAN();
+  if (!hasValue(args.task)) return ERRORS.MISSING_TASK();
+  const hasStatus = hasValue(args.status);
   const hasMutation = hasStatus ||
-    (args.artifacts && args.artifacts !== true) ||
-    (args.notes && args.notes !== true) ||
-    (args["deliverables-pointer"] && args["deliverables-pointer"] !== true) ||
-    (args["mark-deliverable-stale"] && args["mark-deliverable-stale"] !== true);
+    hasValue(args.artifacts) ||
+    hasValue(args.notes) ||
+    hasValue(args["deliverables-pointer"]) ||
+    hasValue(args["mark-deliverable-stale"]);
   if (!hasMutation) return ERRORS.MISSING_STATUS();
   if (args.status === true) return ERRORS.MISSING_STATUS();
   if (hasStatus && !VALID_STATUSES.includes(args.status)) return ERRORS.INVALID_STATUS(args.status);
@@ -137,11 +141,11 @@ function applyUpdate(plan, args, now) {
   const task = plan.tasks.find((t) => t.id === args.task);
 
   const oldStatus = task.status;
-  if (args.status && args.status !== true) {
+  if (hasValue(args.status)) {
     task.status = args.status;
   }
 
-  if (args.artifacts && args.artifacts !== true) {
+  if (hasValue(args.artifacts)) {
     const incoming = args.artifacts
       .split(",")
       .map((s) => s.trim())
@@ -163,12 +167,12 @@ function applyUpdate(plan, args, now) {
     }
   }
 
-  if (args.notes && args.notes !== true) {
+  if (hasValue(args.notes)) {
     task.notes = args.notes;
   }
 
   // completed_at consistency: set only on status updates.
-  if (args.status && args.status !== true) {
+  if (hasValue(args.status)) {
     if (args.status === "done" && !task.completed_at) {
       task.completed_at = now;
     } else if (args.status !== "done") {
@@ -177,7 +181,7 @@ function applyUpdate(plan, args, now) {
   }
 
   // --deliverables-pointer current
-  if (args["deliverables-pointer"] && args["deliverables-pointer"] !== true) {
+  if (hasValue(args["deliverables-pointer"])) {
     if (args["deliverables-pointer"] !== "current") {
       return { error: ERRORS.INVALID_DELIVERABLES_POINTER(args["deliverables-pointer"]) };
     }
@@ -186,7 +190,7 @@ function applyUpdate(plan, args, now) {
 
   // --mark-deliverable-stale <task_id>[,task_id2,...]
   // Supports comma-separated list of downstream task ids.
-  if (args["mark-deliverable-stale"] && args["mark-deliverable-stale"] !== true) {
+  if (hasValue(args["mark-deliverable-stale"])) {
     const staleIds = args["mark-deliverable-stale"]
       .split(",")
       .map((s) => s.trim())
@@ -551,7 +555,7 @@ function main() {
 
   // Parse --projects; if not provided, derive from tasks
   let projectList = null;
-  if (args.projects && args.projects !== true) {
+  if (hasValue(args.projects)) {
     projectList = args.projects.split(",").map((s) => s.trim()).filter(Boolean);
   } else {
     // Derive project list from task.project arrays so that validation

package/sources/scripts/session-update.js

@@ -41,6 +41,7 @@ const ERRORS = {
   NO_ACTIVE_EPIC: (flag) => `${flag} requires an active epic (active_epic.id is empty)`,
   EPIC_ID_ORPHAN: () => "--epic-id (for sub-change) requires --new-change",
   MISSING_REMOVE_VALUE: () => "--remove-change / --remove-epic requires a non-empty value",
+  MISSING_FLAG_VALUE: (flag) => `${flag} requires a non-empty value`,
 };
 
 // ── Defaults ────────────────────────────────────────────────────────────────
@@ -94,6 +95,10 @@ function parseIdList(value) {
     .filter(Boolean);
 }
 
+function hasValue(value) {
+  return value !== undefined && value !== true && String(value).trim() !== "";
+}
+
 // ── Config Loading ──────────────────────────────────────────────────────────
 function loadHistoryLimits(configPath) {
   const limits = { ...DEFAULT_LIMITS };
@@ -126,14 +131,26 @@ function loadHistoryLimits(configPath) {
 // ── Validation ──────────────────────────────────────────────────────────────
 function validate(args) {
   if (!args.skill) return ERRORS.MISSING_SKILL();
+  if (args.skill === true) return ERRORS.MISSING_FLAG_VALUE("--skill");
   if (!args.summary) return ERRORS.MISSING_SUMMARY();
+  if (args.summary === true) return ERRORS.MISSING_FLAG_VALUE("--summary");
+  if (args["new-change"] !== undefined && !hasValue(args["new-change"])) return ERRORS.MISSING_FLAG_VALUE("--new-change");
   if (args["new-change"] && !args["change-id"]) return ERRORS.CHANGE_ID_REQUIRED();
+  if (args["change-id"] !== undefined && !hasValue(args["change-id"])) return ERRORS.MISSING_FLAG_VALUE("--change-id");
 
   // Epic combo validation
   if (args["new-epic"] && !args["epic-id"]) return ERRORS.EPIC_ID_REQUIRED();
+  if (args["new-epic"] !== undefined && !hasValue(args["new-epic"])) return ERRORS.MISSING_FLAG_VALUE("--new-epic");
+  if (args["epic-id"] !== undefined && !hasValue(args["epic-id"])) return ERRORS.MISSING_FLAG_VALUE("--epic-id");
   if (args["close-epic"] && args["new-epic"]) return ERRORS.CLOSE_NEW_EPIC_CONFLICT();
   if (args["epic-id"] && !args["new-change"] && !args["new-epic"]) return ERRORS.EPIC_ID_ORPHAN();
 
+  for (const flag of ["set-plan-path", "set-change-status", "truncate-history", "set-epic-path", "set-epic-status"]) {
+    if (args[flag] !== undefined && !hasValue(args[flag])) {
+      return ERRORS.MISSING_FLAG_VALUE(`--${flag}`);
+    }
+  }
+
   // Remove flags require non-empty values
   if (
     args["remove-change"] !== undefined

package/sources/sections/session-update.md

@@ -9,7 +9,7 @@ This skill is read-only and does NOT modify `.ai-agents/workspace/session.yaml`.
 After the skill's main task, run the session update script **exactly once**:
 
 ```bash
-node .ai-agents/scripts/session-update.cjs --skill {{current_skill}} --summary "<concise one-line summary>"{{#update_active_change}} --new-change "<active_change.title>" --change-id <active_change.id>{{#link_subchange_to_epic}} --epic-id <active_epic.id>{{/link_subchange_to_epic}}{{/update_active_change}}{{#set_plan_path}} --set-plan-path ".ai-agents/workspace/artifacts/{active_change.id}/plan.yaml"{{/set_plan_path}}{{#update_change}} --update-change{{/update_change}}{{#close_change}} --close-change{{/close_change}}{{#set_change_status}} --set-change-status <status>{{/set_change_status}}{{#new_epic}} --new-epic "<epic_title>" --epic-id <epic_id>{{/new_epic}}{{#set_epic_path}} --set-epic-path <epic_path>{{/set_epic_path}}{{#set_epic_status}} --set-epic-status <status>{{/set_epic_status}}{{#close_epic}} --close-epic{{/close_epic}}{{#no_change}} --no-change{{/no_change}}{{#set_synced}} --set-synced{{/set_synced}}{{#truncate_history}} --truncate-history <count>{{/truncate_history}}{{#update_initialized_at}} --set-initialized{{/update_initialized_at}}{{#remove_change}} --remove-change <ids>{{/remove_change}}{{#remove_epic}} --remove-epic <ids>{{/remove_epic}}
+node .ai-agents/scripts/session-update.cjs --skill {{current_skill}} --summary "<concise one-line summary>"{{#update_active_change}} --new-change "<active_change.title>" --change-id <active_change.id>{{#link_subchange_to_epic}} [--epic-id <active_epic.id>]{{/link_subchange_to_epic}}{{/update_active_change}}{{#set_plan_path}} --set-plan-path ".ai-agents/workspace/artifacts/{active_change.id}/plan.yaml"{{/set_plan_path}}{{#update_change}} --update-change{{/update_change}}{{#close_change}} --close-change{{/close_change}}{{#set_change_status}} --set-change-status <status>{{/set_change_status}}{{#new_epic}} --new-epic "<epic_title>" --epic-id <epic_id>{{/new_epic}}{{#set_epic_path}} --set-epic-path <epic_path>{{/set_epic_path}}{{#set_epic_status}} --set-epic-status <status>{{/set_epic_status}}{{#close_epic}} --close-epic{{/close_epic}}{{#no_change}} --no-change{{/no_change}}{{#set_synced}} --set-synced{{/set_synced}}{{#truncate_history}} --truncate-history <count>{{/truncate_history}}{{#update_initialized_at}} --set-initialized{{/update_initialized_at}}{{#remove_change}} --remove-change <ids>{{/remove_change}}{{#remove_epic}} --remove-epic <ids>{{/remove_epic}}
 
 ```
 
@@ -64,7 +64,7 @@ Write `--summary` as one concise line in the configured `interaction_language`.
 - `--remove-epic <ids>` removes entries with matching `id` from `session.epics[]` (comma-separated for multiple ids); does NOT touch `active_epic`. Unknown ids are silently skipped; if all ids are unknown, a warning is written to stderr (exit code remains 0).
 {{/remove_epic}}
 {{#link_subchange_to_epic}}
-- `--epic-id` with `--new-change` links the new active change to its parent epic; do not use it outside `--new-epic` or `--new-change`.
+- `--epic-id` with `--new-change` links the new active change to its parent epic; include it only when `active_epic.id` is non-empty. Do not pass `--epic-id` with an empty placeholder.
 {{/link_subchange_to_epic}}
 
 If the script exits with code 0, the state update was applied successfully; do not read or verify the session file.