@deksden-com/dd-flow-cli 0.9.0-beta.14 → 0.9.0-beta.141

package/dist/services/cli-operation-classifier.js

@@ -1,3 +1,8 @@
+import { parseCommandArgs } from "./lifecycle-command.js";
+import { commandKeyForPositions, routerNativeRoute } from "../cli/command-metadata.js";
+import { hasHelpIntent } from "../cli/help.js";
+import { isUpgradeAllowlisted } from "./cli-upgrade-allowlist.js";
+export { isUpgradeAllowlisted } from "./cli-upgrade-allowlist.js";
 export function classifyCliOperation(args, env = process.env) {
     const positional = positionalArgs(args);
     const [family = null, command = null, action = null] = positional;
@@ -5,6 +10,8 @@ export function classifyCliOperation(args, env = process.env) {
     if (isMbUpgradeMode(args, env)) {
         return { mode: "mb_upgrade", operation, family, command, action };
     }
+    if (family === "run" && command === "reconcile" && !args.includes("--apply"))
+        return { mode: "read_only_diagnostics", operation, family, command, action };
     if (isRouterNative(args)) {
         return { mode: "router_native", operation, family, command, action };
     }
@@ -14,19 +21,7 @@ export function classifyCliOperation(args, env = process.env) {
     return { mode: "normal_write", operation, family, command, action };
 }
 function positionalArgs(args) {
-    const positional = [];
-    for (let index = 0; index < args.length; index += 1) {
-        const value = args[index];
-        if (!value)
-            continue;
-        if (value.startsWith("--")) {
-            if (!value.includes("=") && args[index + 1] && !args[index + 1]?.startsWith("--"))
-                index += 1;
-            continue;
-        }
-        positional.push(value);
-    }
-    return positional;
+    return parseCommandArgs(args).positional;
 }
 export function isReadOnlyCliOperation(args, env = process.env) {
     return classifyCliOperation(args, env).mode === "read_only_diagnostics";
@@ -57,59 +52,19 @@ function isMbUpgradeMode(args, env) {
     }
     return marked && isUpgradeAllowlisted(args);
 }
-function isUpgradeAllowlisted(args) {
-    const positional = positionalArgs(args);
-    const [family, command] = positional;
-    if (family === "status")
-        return true;
-    if (family === "engine" && command === "resolve")
-        return true;
-    if (family === "migration" && ["impact", "plan", "report", "verify", "apply"].includes(command ?? ""))
-        return true;
-    if (family === "memory" && command === "permissions" && positional[2] === "preflight")
-        return true;
-    if (family === "run" && command === "start")
-        return optionValue(args, "flow-kind") === "mb-upgrade";
-    if (family === "stage" && ["start", "pause", "resume", "finish"].includes(command ?? ""))
-        return hasRunBinding(args);
-    if (family === "session" && ["status", "register", "stop", "usage"].includes(command ?? ""))
-        return hasRunBinding(args);
-    if (family === "dashboard" && ["data", "render", "refresh", "open"].includes(command ?? ""))
-        return true;
-    return false;
-}
-function hasRunBinding(args) {
-    return Boolean(optionValue(args, "run") || positionalArgs(args).some((value) => value.startsWith("RUN-")));
-}
 function optionValue(args, name) {
-    for (let index = args.length - 1; index >= 0; index -= 1) {
-        const arg = args[index];
-        if (arg === `--${name}`)
-            return args[index + 1] ?? null;
-        if (arg?.startsWith(`--${name}=`))
-            return arg.slice(name.length + 3);
-    }
-    return null;
+    const values = parseCommandArgs(args).options.get(name);
+    const value = values?.[values.length - 1];
+    return value && value !== "__dd_flow_flag__" ? value : null;
 }
-function isRouterNative(args) {
+export function isRouterNative(args) {
     const first = args[0];
     if (!first || first === "--version")
         return true;
-    if (first === "--help" || first === "-h")
+    if (hasHelpIntent(args))
         return true;
-    if (args.includes("--help") || args.includes("-h"))
-        return true;
-    if (["engine", "version", "schema"].includes(first))
-        return true;
-    return (first === "codex" && args[1] === "hook" && args[2] === "handle")
-        || (first === "zcode" && args[1] === "event" && args[2] === "handle")
-        || (first === "zcode" && args[1] === "usage" && args[2] === "ingest")
-        || (first === "grok" && args[1] === "event" && args[2] === "handle")
-        || (first === "grok" && args[1] === "usage" && args[2] === "ingest")
-        || (first === "opencode" && args[1] === "event" && args[2] === "handle")
-        || (first === "opencode" && args[1] === "usage" && args[2] === "ingest")
-        || (first === "agy" && args[1] === "event" && args[2] === "handle")
-        || (first === "agy" && args[1] === "usage" && args[2] === "ingest");
+    const key = commandKeyForPositions(positionalArgs(args));
+    return key !== undefined && routerNativeRoute(key);
 }
 function isReadOnlyDiagnostic(family, command, action) {
     if (!family)
@@ -119,13 +74,21 @@ function isReadOnlyDiagnostic(family, command, action) {
     if (family === "canon")
         return command === "status" || command === "resolve";
     if (family === "project")
-        return command === "status" || command === "resolve";
+        return command === "status" || command === "resolve" || (command === "config" && action === "status");
     if (family === "protocol")
         return ["status", "branch-status", "ready", "blockers", "implement"].includes(command ?? "");
     if (family === "plan")
         return command === "status";
     if (family === "run")
-        return command === "status" || command === "list";
+        return ["status", "list", "timeline"].includes(command ?? "") || (["config", "flags", "drive", "control"].includes(command ?? "") && action === "status") || (command === "vars" && ["ls", "get"].includes(action ?? "")) || (command === "recovery" && action === "inspect");
+    if (family === "runtime" && command === "process" && ["status", "check-admission"].includes(action ?? ""))
+        return true;
+    if (family === "runtime" && command === "scope" && action === "status")
+        return true;
+    if (family === "work")
+        return ["status", "ls", "show"].includes(command ?? "");
+    if (family === "stage")
+        return command === "fanout" && action === "status";
     if (family === "lane") {
         return (command === "status" ||
             command === "waiters" ||

package/dist/services/cli-upgrade-allowlist.js

@@ -0,0 +1,27 @@
+import { parseCommandArgs } from "./lifecycle-command.js";
+/** Syntax eligibility only. Storage/project/RUN authority is checked separately. */
+export function isUpgradeAllowlisted(args) {
+    const parsed = parseCommandArgs(args), [family, command] = parsed.positional;
+    const option = (name) => {
+        const value = parsed.options.get(name)?.at(-1);
+        return value && value !== "__dd_flow_flag__" ? value : null;
+    };
+    if (family === "status")
+        return true;
+    if (family === "engine" && command === "resolve")
+        return true;
+    if (family === "migration" && ["impact", "plan", "report", "verify", "apply"].includes(command ?? ""))
+        return true;
+    if (family === "memory" && command === "permissions" && parsed.positional[2] === "preflight")
+        return true;
+    if (family === "run" && command === "start")
+        return option("flow-kind") === "mb-upgrade";
+    const bound = Boolean(option("run") || parsed.positional.some(value => value.startsWith("RUN-")));
+    if (family === "stage" && ["start", "pause", "resume", "finish"].includes(command ?? ""))
+        return bound;
+    if (family === "session" && ["status", "register", "stop", "usage"].includes(command ?? ""))
+        return bound;
+    if (family === "dashboard" && ["data", "render", "refresh", "open"].includes(command ?? ""))
+        return true;
+    return false;
+}