@kisev/agentomatic 12.0.1-dev.126.g020f34744898 → 12.0.1-dev.128.g273b5ff20349

package/dist/agent-profiles.d.ts

@@ -117,8 +117,9 @@ export declare function previewAgentProfileChange(request: AgentProfileRequest,
 export declare function applyAgentProfileChange(request: AgentProfileRequest, scope: Scope, cwd?: string, home?: string, options?: TransactionOptions & {
     expectedDigest?: string;
 }): Promise<AgentProfileResult>;
+export declare function parseModelsOutput(stdout: string): string[];
 export declare function availableModels(): Promise<string[]>;
-export declare function availableModelVariants(model: string): Promise<string[]>;
+export declare function availableModelVariants(model: string): Promise<string[] | null>;
 export declare function inspectFileMode(path: string): Promise<number | undefined>;
 export declare function readLegacyManifest(root: string): Promise<LegacyAgentOwnership | undefined>;
 export declare function applyBuiltAgentPlan(built: BuiltPlan, stateRoot: string, options?: TransactionOptions): Promise<void>;

package/dist/agent-profiles.js

@@ -752,8 +752,44 @@ export async function applyAgentProfileChange(request, scope, cwd = process.cwd(
     });
 }
 async function modelCatalog() {
-    const query = new URLSearchParams({ "location[directory]": process.cwd() });
-    const { stdout } = await execFileAsync("opencode", ["api", "get", `/api/model?${query}`], {
+    // The CLI catalog is the primary source: it works in a clean terminal, needs
+    // no background service, and never depends on the per-directory location
+    // filter that returns an empty list for some checkouts. If the background
+    // service is down, retry standalone before giving up with a remedy.
+    const models = await cliModels().catch((error) => cliModels(["--standalone"]).catch(() => {
+        throw new AgentProfileError("catalog_unavailable", `The opencode models CLI is unavailable (${error instanceof Error ? error.message : String(error)}); start opencode in a clean terminal or run 'opencode models' yourself to verify, then retry - or pass an explicit provider/model`);
+    }));
+    return models.map((model) => {
+        const [providerID, ...rest] = model.split("/");
+        return { providerID, id: rest.join("/") };
+    });
+}
+// Line-by-line `provider/model` output of `opencode models`; anything else in
+// the stream is ignored. Exported for the parser test.
+export function parseModelsOutput(stdout) {
+    return [
+        ...new Set(stdout
+            .split("\n")
+            .map((line) => line.trim())
+            .filter((line) => MODEL_PATTERN.test(line))),
+    ].sort();
+}
+async function cliModels(extra = []) {
+    const { stdout } = await execFileAsync("opencode", ["models", ...extra], {
+        timeout: 10_000,
+        encoding: "utf8",
+        maxBuffer: 10 * 1024 * 1024,
+    });
+    const models = parseModelsOutput(stdout);
+    if (!models.length)
+        throw new Error("opencode models printed no provider/model lines");
+    return models;
+}
+// The variants (effort) metadata still comes from the API - without the
+// location filter: some directories answer an empty list to a located query,
+// and that must never zero the cascade.
+async function modelApiEntries() {
+    const { stdout } = await execFileAsync("opencode", ["api", "get", "/api/model"], {
         timeout: 10_000,
         encoding: "utf8",
         maxBuffer: 10 * 1024 * 1024,
@@ -783,7 +819,7 @@ export async function availableModels() {
 export async function availableModelVariants(model) {
     const selected = validateModel(model);
     try {
-        const metadata = (await modelCatalog()).find((entry) => `${entry.providerID}/${entry.id}` === selected);
+        const metadata = (await modelApiEntries()).find((entry) => `${entry.providerID}/${entry.id}` === selected);
         if (!metadata)
             throw new Error("selected model is absent");
         if (metadata.variants === undefined)
@@ -793,8 +829,9 @@ export async function availableModelVariants(model) {
             throw new Error("invalid variants metadata");
         return metadata.variants.map((variant) => variant.id);
     }
-    catch (error) {
-        throw new AgentProfileError("catalog_unavailable", `OpenCode V2 model variants are unavailable: ${error instanceof Error ? error.message : String(error)}`);
+    catch {
+        // Unknown is honest: the cascade says so and allows a manual variant.
+        return null;
     }
 }
 export async function inspectFileMode(path) {

package/dist/command-cli.js

@@ -51,20 +51,13 @@ const commands = {
         description: "Remove owned components; disconnect the plugin by default; retain models and npm dependency unless selected.",
         options: ["disconnect", "removeDependency"],
     },
-    agent: { description: "List agents or manage additional critics.", options: [] },
-    "agent list": {
-        description: "Show installed and saved profiles, models, ownership, collisions, and drift.",
+    agent: {
+        description: "Read-only agent overview: profiles, models, ownership, collisions, and drift (mutations: configure agent).",
         options: [],
     },
-    "agent add-critic": {
-        description: "Add critic-<name> interactively or with an explicit model.",
-        options: ["provider", "model", "variant"],
-        name: true,
-    },
-    "agent remove": {
-        description: "Remove an additional critic; fixed roles cannot be removed here.",
+    "agent list": {
+        description: "Show installed and saved profiles, models, ownership, collisions, and drift.",
         options: [],
-        name: true,
     },
     maintenance: {
         description: "Clean up retired files, repair selected components, or recover an interrupted transaction.",
@@ -89,6 +82,15 @@ const commands = {
 };
 const mutationOptions = new Set(["dryRun", "yes"]);
 const observations = new Set(["status", "doctor", "agent list", "catalog"]);
+// Mutations were unified into configure agent; removed commands answer with a
+// one-line pointer to it.
+const removedCommands = {
+    "agent add-critic": "Removed command; agentomatic configure agent is the single mutation point for agent models and critics",
+    "agent remove": "Removed command; agentomatic configure agent is the single mutation point for agent models and critics",
+};
+function failRemoved(topic) {
+    return fail("invalid_input", removedCommands[topic]);
+}
 function fail(code, message) {
     throw new InstallerError(code, message);
 }
@@ -125,6 +127,8 @@ async function promptCriticName() {
     }
 }
 function help(topic = "") {
+    if (topic && removedCommands[topic])
+        failRemoved(topic);
     if (topic && !commands[topic])
         fail("invalid_input", `Unknown command: ${topic}`);
     const visible = topic
@@ -177,7 +181,7 @@ function help(topic = "") {
         "Use status to inspect the installation, doctor to diagnose problems, and maintenance for explicit recovery.",
         "",
         "Examples:",
-        `  ${shellCommand([...(topic ? topic.split(" ") : ["install"]), ...(commands[topic]?.name ? [topic === "agent remove" ? "critic-security" : "critic"] : []), ...(observations.has(topic) || ["agent", "maintenance"].includes(topic) ? [] : ["--dry-run"])])}`,
+        `  ${shellCommand([...(topic ? topic.split(" ") : ["install"]), ...(commands[topic]?.name ? ["critic"] : []), ...(observations.has(topic) || ["agent", "maintenance"].includes(topic) ? [] : ["--dry-run"])])}`,
         "Documentation: https://github.com/kisev/skills/blob/main/docs/how-to/opencode-integration.md",
         "",
     ].join("\n");
@@ -369,7 +373,14 @@ async function modelSelection(options, name, current) {
         const model = candidates[value(await selectOption("Model", candidates))];
         const variants = await availableModelVariants(model);
         let variant = null;
-        if (variants.length) {
+        if (variants === null) {
+            // The variants metadata is unavailable: say so honestly and accept a
+            // manual variant instead of degrading the whole model selection.
+            process.stderr.write("Model variants are unknown (variants metadata unavailable); enter one manually or leave it empty.\n");
+            const answer = await promptText(current?.variant ? "Variant (empty keeps current)" : "Variant (optional)");
+            variant = (answer ? validateVariant(answer) : current?.variant) ?? null;
+        }
+        else if (variants.length) {
             const labels = variants.map((item) => item === current?.variant ? `${item} (default)` : item);
             const choice = value(await selectOption("Variant", ["(none)", ...labels], undefined, undefined, current?.variant ? variants.indexOf(current.variant) + 1 : 0));
             variant = choice === 0 ? null : (variants[choice - 1] ?? null);
@@ -506,18 +517,9 @@ async function deploy(options, repair = false) {
             ],
         };
     }
-    let changes = [];
-    if (!repair &&
-        tty() &&
-        value(await confirmQuestion("Configure agent models or additional critics now?"))) {
-        const inventory = await listAgentProfiles(options.scope);
-        changes = await profileDraft(options, [
-            ...selection.agents,
-            ...inventory.profiles
-                .filter((profile) => profile.name.startsWith("critic-") && profile.ownership !== "user-owned")
-                .map((profile) => profile.name),
-        ]);
-    }
+    // Install never mutates the critic panel: the summary ends with the panel
+    // table and the pointer to configure agent, the single mutation point.
+    const changes = [];
     const plan = await preview("install", options.scope, undefined, undefined, selection, !options.noDependency, changes);
     const config = await previewConfigSetup(integration, options.scope, undefined, undefined, false, false);
     const report = {
@@ -567,6 +569,7 @@ async function deploy(options, repair = false) {
         if (changes.length)
             process.stdout.write(`Staged profiles: ${terminalSafe(JSON.stringify(changes))}\n`);
         process.stdout.write("Components, npm, and application configuration are separate stages; a later failure does not roll back completed stages.\n");
+        await criticsEpilogue(options);
     }
     if (options.dryRun) {
         if (options.json)
@@ -605,8 +608,10 @@ async function deploy(options, repair = false) {
         };
         if (options.json)
             json(result);
-        else
+        else {
             process.stdout.write(renderPlan(applied, { applied: true }) + renderConfigSetup(connected, { applied: true }));
+            await criticsEpilogue(options);
+        }
     }
     catch (error) {
         if (error instanceof LifecycleError &&
@@ -616,6 +621,16 @@ async function deploy(options, repair = false) {
         partial(options, error, completed, `Run status, then ${repair ? "maintenance repair" : "install"} with the same selected components; use configure integration for configuration-only continuation.`, completed.includes("owned-components-and-profiles") && plan.requires_restart);
     }
 }
+// The critic panel epilogue: the table plus the canonical view and mutation
+// pointers, printed wherever the panel matters (install, staged applies,
+// configure integration).
+async function criticsEpilogue(options) {
+    if (options.json)
+        return;
+    const inventory = await listAgentProfiles(options.scope);
+    process.stdout.write(renderCriticsTable(inventory));
+    process.stdout.write("Inspect any time: agentomatic agent list | Manage critics: agentomatic configure agent\n");
+}
 function partial(options, error, completed, next, requiresRestart = false) {
     const code = error instanceof LifecycleError ? error.code : "internal_error";
     const message = error instanceof Error ? error.message : String(error);
@@ -672,8 +687,10 @@ async function configureIntegration(options) {
                 plan: result,
                 requires_restart: result.requires_restart,
             });
-        else
+        else {
             process.stdout.write(renderConfigSetup(result, { applied: true }));
+            await criticsEpilogue(options);
+        }
     }
     catch (error) {
         partial(options, error, error instanceof LifecycleError && error.code === "npm_dependency_failed"
@@ -689,20 +706,18 @@ async function profileChange(options, request) {
     if (!options.json) {
         const args = request.changes
             ? ["configure", "critics"]
-            : request.action === "critic-remove"
-                ? ["agent", "remove", request.name]
-                : [
-                    request.action === "critic-add" ? "agent" : "configure",
-                    request.action === "critic-add" ? "add-critic" : "agent",
-                    request.name,
-                    "--model",
-                    request.model,
-                    ...(request.variant
-                        ? ["--variant", request.variant]
-                        : request.action === "model-set"
-                            ? ["--clear-variant"]
-                            : []),
-                ];
+            : [
+                "configure",
+                "agent",
+                request.name,
+                "--model",
+                request.model,
+                ...(request.variant
+                    ? ["--variant", request.variant]
+                    : request.action === "model-set"
+                        ? ["--clear-variant"]
+                        : []),
+            ];
         process.stdout.write(renderPlan(plan, {
             applied: false,
             applyHint: applyHint([...args, ...scopeArguments(options.scope)]),
@@ -720,8 +735,11 @@ async function profileChange(options, request) {
     });
     if (options.json)
         json(result);
-    else
+    else {
         process.stdout.write(renderPlan(result.plan, { applied: true }));
+        // After a panel mutation the fresh table closes the loop.
+        await criticsEpilogue(options);
+    }
 }
 async function uninstall(options) {
     requireApply(options);
@@ -830,12 +848,16 @@ async function run(args) {
         return;
     }
     if (args.includes("--help") || args.includes("-h")) {
+        if (removedCommands[topic])
+            failRemoved(topic);
         if (!commands[topic])
             fail("invalid_input", `Unknown command: ${topic}`);
         parse(topic, args.filter((arg) => !["--help", "-h"].includes(arg)));
         process.stdout.write(help(topic));
         return;
     }
+    if (removedCommands[topic])
+        failRemoved(topic);
     if (!commands[topic]) {
         const match = [...new Set(Object.keys(commands).map((key) => key.split(" ", 1)[0]))].find((root) => root.startsWith(topic) || topic.startsWith(root));
         fail("invalid_input", `Unknown command: ${topic}.${match ? ` Did you mean '${match}'?` : ""} Use install, configure, status, doctor, uninstall, agent, maintenance, or catalog.`);
@@ -904,12 +926,11 @@ async function run(args) {
         process.exitCode = doctorExitCode(report);
         return;
     }
-    if (topic === "configure critics" ||
-        topic === "configure agent" ||
-        topic === "agent add-critic") {
+    if (topic === "configure critics" || topic === "configure agent") {
         const inventory = await listAgentProfiles(options.scope);
-        // configure critics is an alias; a nameless configure agent opens the same staged editor.
-        if (topic === "configure critics" || (topic === "configure agent" && !options.name)) {
+        // configure critics is an alias; a nameless configure agent opens the staged
+        // editor - the single mutation point for models, additions, and removals.
+        if (!options.name) {
             requireApply(options);
             if (!options.json)
                 process.stdout.write(renderCriticsTable(inventory));
@@ -926,30 +947,20 @@ async function run(args) {
             await profileChange(options, { action: "model-set", changes });
             return;
         }
-        let name = options.name;
-        if (!name) {
-            if (topic === "agent add-critic")
-                name = await promptCriticName();
-            else {
-                const names = inventory.profiles
-                    .filter((profile) => profile.ownership !== "user-owned")
-                    .map((profile) => profile.name);
-                name = names[value(await selectOption("Agent", names))];
-            }
+        const name = options.name;
+        const existing = inventory.profiles.find((profile) => profile.name === name);
+        if (!existing && !name.startsWith("critic-")) {
+            fail("invalid_input", `Unknown agent profile: ${name}`);
         }
-        if (topic === "agent add-critic" && !name.startsWith("critic-"))
-            name = `critic-${name}`;
-        const request = await modelSelection(options, name, inventory.profiles.find((profile) => profile.name === name));
-        await profileChange(options, {
-            ...request,
-            action: topic === "agent add-critic" ? "critic-add" : "model-set",
-        });
-        return;
-    }
-    if (topic === "agent remove") {
-        if (!options.name)
-            fail("invalid_input", "agent remove requires an additional critic name");
-        await profileChange(options, { action: "critic-remove", name: options.name });
+        if (!existing) {
+            // configure agent on a fresh critic-<name> adds the critic: one command
+            // covers model-set for existing profiles and critic-add for new critics.
+            const request = await modelSelection(options, name, undefined);
+            await profileChange(options, { ...request, action: "critic-add" });
+            return;
+        }
+        const request = await modelSelection(options, name, existing);
+        await profileChange(options, { ...request, action: "model-set" });
         return;
     }
     if (topic === "maintenance recover") {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kisev/agentomatic",
-  "version": "12.0.1-dev.126.g020f34744898",
+  "version": "12.0.1-dev.128.g273b5ff20349",
   "description": "OpenCode integration and opt-in installer for portable Agent Skills.",
   "homepage": "https://github.com/kisev/skills#readme",
   "bugs": {
@@ -50,7 +50,7 @@
   },
   "dependencies": {
     "@clack/prompts": "^1.8.1",
-    "@kisev/safe-fs": "12.0.1-dev.126.g020f34744898",
+    "@kisev/safe-fs": "12.0.1-dev.128.g273b5ff20349",
     "commander": "15.0.0",
     "zod": "4.1.8"
   },