@zq-silk/yui 0.7.1 → 0.8.2

package/dist/cli/completion.js

@@ -55,7 +55,7 @@ ${bashDynamicOptionCases(entries)}
   if [[ "$dynamic" == true ]]; then
     while IFS= read -r dynamic_candidate; do
       [[ -n "$dynamic_candidate" ]] && dynamic_candidates+=("$dynamic_candidate")
-    done < <(command ${identity} completion candidates "$current" -- "\${COMP_WORDS[@]:1:COMP_CWORD-1}" 2>/dev/null)
+    done < <(command ${identity} config completion candidates "$current" -- "\${COMP_WORDS[@]:1:COMP_CWORD-1}" 2>/dev/null)
     candidates=("\${dynamic_candidates[@]}")
   elif [[ "$current" == -* ]]; then
     case "$path" in
@@ -95,7 +95,7 @@ ${zshDynamicOptionCases(entries)}
   esac
 fi
 if [[ "$dynamic" == true ]]; then
-  dynamic_output="$(command ${identity} completion candidates "$current" -- "\${(@)words[2,CURRENT-1]}" 2>/dev/null)"
+  dynamic_output="$(command ${identity} config completion candidates "$current" -- "\${(@)words[2,CURRENT-1]}" 2>/dev/null)"
   candidates=("\${(@f)dynamic_output}")
 elif [[ "$current" == -* ]]; then
   case "$command_path" in
@@ -151,7 +151,7 @@ function renderFish(entries, identity) {
         "end",
         `function ${prefix}_dynamic`,
         "  set -l words (commandline -opc)",
-        `  ${identity} completion candidates (commandline -ct) -- $words[2..-1] 2>/dev/null`,
+        `  ${identity} config completion candidates (commandline -ct) -- $words[2..-1] 2>/dev/null`,
         "end"
     ];
     for (const entry of entries) {

package/dist/cli/helpRenderer.js

@@ -15,6 +15,9 @@ export function renderCommandHelp(node, version) {
         const width = Math.max(...node.options.map((option) => option.length));
         lines.push("", "Options:", ...node.options.map((option) => `  ${option.padEnd(width)}  ${describeOption(option)}`));
     }
+    if (node.examples.length > 0) {
+        lines.push("", "Examples:", ...node.examples.map((example) => `  ${example}`));
+    }
     return `${lines.join("\n")}\n`;
 }
 function describeOption(option) {

package/dist/cli/interactionPolicy.js

@@ -1,3 +1,4 @@
+import { findCommandNode } from "./commandCatalog.js";
 const taskTarget = (command, argumentIndex = 2, statuses) => ({
     commandPath: ["task", command],
     selectors: [{
@@ -47,7 +48,7 @@ export const INTERACTION_POLICIES = Object.freeze([
             "--development": "value"
         }
     },
-    ...["add", "update", "retire", "list", "show"].map((command) => ({
+    ...["add", "retire", "list", "show"].map((command) => ({
         commandPath: ["project", "knowledge", command],
         selectors: [{
                 argumentIndex: 3,
@@ -57,26 +58,24 @@ export const INTERACTION_POLICIES = Object.freeze([
             }],
         ...(command === "add"
             ? { trailingOptions: { "--body": "value" } }
-            : command === "update"
-                ? { trailingOptions: { "--title": "value", "--body": "value" } }
-                : command === "list"
-                    ? { trailingOptions: { "--all": "flag" } }
-                    : {})
+            : command === "list"
+                ? { trailingOptions: { "--all": "flag" } }
+                : {})
     })),
     ...["show", "capabilities", "update", "remove"].map((command) => ({
-        commandPath: ["agent", command],
+        commandPath: ["config", "agent", command],
         selectors: [{
-                argumentIndex: 2,
+                argumentIndex: 3,
                 entity: "agent",
                 provider: "configured-agents",
                 actionTarget: true
             }],
         ...(command === "remove"
-            ? { confirmation: { action: "Remove Agent", targetArgumentIndex: 2 } }
+            ? { confirmation: { action: "Remove Agent", targetArgumentIndex: 3 } }
             : {})
     })),
     {
-        commandPath: ["role", "add"],
+        commandPath: ["config", "role", "add"],
         selectors: [{
                 option: "--agent",
                 requiredOption: true,
@@ -93,29 +92,31 @@ export const INTERACTION_POLICIES = Object.freeze([
             "--search": "value"
         }
     },
-    ...["show", "update", "remove", "enter"].map((command) => ({
-        commandPath: ["role", command],
+    ...["show", "update", "remove", "enter", "context"].map((command) => ({
+        commandPath: command === "enter" || command === "context"
+            ? ["session", command]
+            : ["config", "role", command],
         selectors: [{
-                argumentIndex: 2,
+                argumentIndex: command === "enter" || command === "context" ? 2 : 3,
                 entity: "global-role",
                 provider: "global-roles",
                 actionTarget: true
             }],
         ...(command === "remove"
-            ? { confirmation: { action: "Remove Role", targetArgumentIndex: 2 } }
+            ? { confirmation: { action: "Remove Role", targetArgumentIndex: 3 } }
             : {})
     })),
     ...["bind", "unbind"].map((command) => ({
-        commandPath: ["role", command],
+        commandPath: ["config", "role", command],
         selectors: [
-            { argumentIndex: 2, entity: "global-role", provider: "global-roles", actionTarget: true },
-            { argumentIndex: 3, entity: "agent", provider: "configured-agents", actionTarget: false }
+            { argumentIndex: 3, entity: "global-role", provider: "global-roles", actionTarget: true },
+            { argumentIndex: 4, entity: "agent", provider: "configured-agents", actionTarget: false }
         ]
     })),
     ...["record", "replace"].map((command) => ({
-        commandPath: ["role", "session", command],
+        commandPath: ["session", command],
         selectors: [{
-                argumentIndex: 3,
+                argumentIndex: 2,
                 entity: "global-role",
                 provider: "global-roles",
                 actionTarget: true
@@ -125,7 +126,7 @@ export const INTERACTION_POLICIES = Object.freeze([
             : { "--native-id": "value" }
     })),
     {
-        commandPath: ["profile", "add"],
+        commandPath: ["config", "profile", "add"],
         selectors: [],
         trailingOptions: {
             "--description": "value", "--instructions": "value",
@@ -133,9 +134,9 @@ export const INTERACTION_POLICIES = Object.freeze([
         }
     },
     ...["show", "update", "remove"].map((command) => ({
-        commandPath: ["profile", command],
+        commandPath: ["config", "profile", command],
         selectors: [{
-                argumentIndex: 2,
+                argumentIndex: 3,
                 entity: "agent-profile",
                 provider: "agent-profiles",
                 actionTarget: true
@@ -151,7 +152,7 @@ export const INTERACTION_POLICIES = Object.freeze([
             }
             : {}),
         ...(command === "remove"
-            ? { confirmation: { action: "Remove Agent Profile", targetArgumentIndex: 2 } }
+            ? { confirmation: { action: "Remove Agent Profile", targetArgumentIndex: 3 } }
             : {})
     })),
     taskTarget("show"),
@@ -360,8 +361,8 @@ export const INTERACTION_POLICIES = Object.freeze([
         trailingOptions: { "--reason": "value" },
         confirmation: { action: "Reset Task Role Session", targetArgumentIndex: 4 }
     },
-    {
-        commandPath: ["task", "role", "enter"],
+    ...["view", "takeover", "release"].map((command) => ({
+        commandPath: ["task", "role", command],
         selectors: [
             { argumentIndex: 3, entity: "task", provider: "tasks", actionTarget: true },
             {
@@ -371,9 +372,8 @@ export const INTERACTION_POLICIES = Object.freeze([
                 dependsOn: 3,
                 actionTarget: true
             }
-        ],
-        trailingOptions: { "--read-only": "flag", "--read-write": "flag" }
-    },
+        ]
+    })),
     {
         commandPath: ["task", "work", "create"],
         selectors: [
@@ -533,11 +533,6 @@ export const INTERACTION_POLICIES = Object.freeze([
                 ? { trailingOptions: { "--reason": "value" } }
                 : {})
     })),
-    {
-        commandPath: ["task", "enter"],
-        selectors: [{ argumentIndex: 2, entity: "task", provider: "tasks", actionTarget: true }],
-        trailingOptions: { "--read-only": "flag", "--read-write": "flag" }
-    },
     {
         commandPath: ["jobs", "retry"],
         selectors: [{ argumentIndex: 2, entity: "job", provider: "jobs", actionTarget: true }]
@@ -630,10 +625,30 @@ export function validateInteractionPolicies(policies = INTERACTION_POLICIES) {
         if (paths.has(path))
             throw new Error(`Duplicate interaction policy: ${path}`);
         paths.add(path);
+        const node = findCommandNode(policy.commandPath);
+        if (node === undefined || node.hidden) {
+            throw new Error(`Interaction policy references an unknown command: ${path}`);
+        }
         for (const selector of policy.selectors) {
             if ((selector.argumentIndex === undefined) === (selector.option === undefined)) {
                 throw new Error(`Interaction selector must declare exactly one slot: ${path}`);
             }
+            if (selector.argumentIndex !== undefined && (!Number.isInteger(selector.argumentIndex)
+                || selector.argumentIndex < node.path.length - 1)) {
+                throw new Error(`Interaction selector has an invalid argument index: ${path}`);
+            }
+            if (selector.option !== undefined && !node.options.includes(selector.option)) {
+                throw new Error(`Interaction selector references an unknown option: ${path} ${selector.option}`);
+            }
+        }
+        for (const option of Object.keys(policy.trailingOptions ?? {})) {
+            if (!node.options.includes(option)) {
+                throw new Error(`Interaction policy references an unknown trailing option: ${path} ${option}`);
+            }
+        }
+        if (policy.confirmation !== undefined && !policy.selectors.some((selector) => selector.argumentIndex === policy.confirmation?.targetArgumentIndex
+            && selector.actionTarget)) {
+            throw new Error(`Interaction confirmation has no action target selector: ${path}`);
         }
     }
 }

package/dist/cli/interactiveSelection.js

@@ -53,7 +53,7 @@ export async function resolveInteractiveArguments(args, node, ports, io) {
 }
 export function allowsInteractiveSelection(args, globalJson) {
     return !globalJson && !args.includes("--json")
-        && !(args[0] === "completion" && args[1] === "candidates");
+        && !(args[0] === "config" && args[1] === "completion" && args[2] === "candidates");
 }
 function missingSlot(args, selector, node) {
     if (selector.argumentIndex !== undefined) {

package/dist/cli/invocationRouter.js

@@ -31,8 +31,9 @@ function resolveExecutionPath(args) {
             break;
         const child = findChild(node, args[index] ?? "");
         const internalExecutable = child !== undefined && ((node === ROOT_COMMAND && child.name === "internal")
-            || (node.path.join(" ") === "yui completion" && child.name === "candidates")
-            || (node.path.join(" ") === "yui task run" && child.name === "checkpoint")
+            || (node.path.join(" ") === "yui config completion" && child.name === "candidates")
+            || (node.path.join(" ") === "yui task run"
+                && (child.name === "checkpoint" || child.name === "context"))
             || (node.path.join(" ") === "yui controller"
                 && (child.name === "identity" || child.name === "live-identity")));
         if (child === undefined || (child.hidden && !internalExecutable)) {

package/dist/cli/roleWizard.js

@@ -20,7 +20,7 @@ export async function resolveRoleWizardArguments(commandArgs, ports, io) {
     return { kind: "unchanged", args };
 }
 export async function resolveGlobalRoleAgentConfigurationArguments(roleName, agentId, ports, io) {
-    const args = ["role", "update", roleName];
+    const args = ["config", "role", "update", roleName];
     if (!io.interactive || io.json)
         return { kind: "unchanged", args };
     const role = asRole(await ports.call("role.show", { name: roleName }));
@@ -31,7 +31,7 @@ export async function resolveGlobalRoleAgentConfigurationArguments(roleName, age
     return updateAgentBindingSettings(args, role, binding, ports, io);
 }
 function isGlobalRoleAdd(args) {
-    return args[0] === "role" && args[1] === "add";
+    return args[0] === "config" && args[1] === "role" && args[2] === "add";
 }
 function isTaskRoleAdd(args) {
     return args[0] === "task" && args[1] === "role" && args[2] === "add";
@@ -39,18 +39,18 @@ function isTaskRoleAdd(args) {
 // Explicit update options always win. This keeps scripts and deliberate
 // non-interactive invocations out of the wizard even when run from a TTY.
 function isGlobalRoleUpdate(args) {
-    return args.length === 3 && args[0] === "role" && args[1] === "update"
-        && isValue(args[2]);
+    return args.length === 4 && args[0] === "config" && args[1] === "role"
+        && args[2] === "update" && isValue(args[3]);
 }
 function isTaskRoleUpdate(args) {
     return args.length === 5 && args[0] === "task" && args[1] === "role"
         && args[2] === "update" && isValue(args[3]) && isValue(args[4]);
 }
 async function addGlobalRole(args, ports, io) {
-    const named = await ensureRoleName(args, 2, io);
+    const named = await ensureRoleName(args, 3, io);
     if (named === undefined)
         return { kind: "cancelled", args };
-    const explicitSettings = hasExplicitAddSettings(named, 3);
+    const explicitSettings = hasExplicitAddSettings(named, 4);
     const agents = await loadAgentSelection(ports);
     const explicitAgent = optionValue(named, "--agent");
     const selected = explicitAgent ?? await selectConfiguredAgent(ports, io, "Create Global Role from Agent", agents);
@@ -236,7 +236,7 @@ async function configureNewAgentField(args, agent, ports, io) {
         : { kind: "resolved", args: [...args, selected.set, value] };
 }
 async function updateGlobalRole(args, ports, io) {
-    const role = asRole(await ports.call("role.show", { name: args[2] }));
+    const role = asRole(await ports.call("role.show", { name: args[3] }));
     if (role === undefined)
         return { kind: "unchanged", args };
     return updateRole(args, role, ports, io, false);
@@ -286,7 +286,7 @@ async function updateRoleSettings(args, role, ports, io, taskRole) {
             return { kind: "cancelled", args };
         return taskRole
             ? { kind: "resolved", args: ["task", "role", "bind", args[3] ?? "", args[4] ?? "", target] }
-            : { kind: "resolved", args: ["role", "bind", args[2] ?? "", target] };
+            : { kind: "resolved", args: ["config", "role", "bind", args[3] ?? "", target] };
     }
     if (field === "workspace") {
         const value = (await io.question(`Workspace [${roleDisplay(role.workspace)}]: `))?.trim();