@kisev/agentomatic 12.0.1-dev.118.geda2dc538b36 → 12.0.1-dev.120.g392563c8f5d9

package/dist/agent-profiles.d.ts

@@ -106,6 +106,7 @@ type BuiltPlan = {
     legacyTransferred: string[];
 };
 export declare function validateAgentName(name: string): string;
+export declare function suggestCriticName(input: string): string | null;
 export declare function validateModel(model: string): string;
 export declare function validateVariant(variant: string | null | undefined): string | undefined;
 export declare function renderAgentProfile(name: string, config: AgentProfileConfig, canonical: Record<FixedAgentRole, Buffer>): Buffer;

package/dist/agent-profiles.js

@@ -60,7 +60,19 @@ function emptyConfig() {
 export function validateAgentName(name) {
     if (NAME_PATTERN.test(name) || CRITIC_PATTERN.test(name))
         return name;
-    throw new AgentProfileError("invalid_name", "Agent must be a fixed role or critic-<safe-suffix>");
+    throw new AgentProfileError("invalid_name", "Agent must be a fixed role (manager, architect, mapper, worker, review, critic) or critic-<suffix> where suffix uses lowercase letters, digits, and hyphens between parts (for example critic-sonnet-5-5)");
+}
+// Convert free-form critic input such as "Sonnet 5.5" or "critic-sonnet-5.5" into a
+// safe critic name; null when no safe form exists.
+export function suggestCriticName(input) {
+    const cleaned = input.trim().toLowerCase();
+    if (CRITIC_PATTERN.test(cleaned))
+        return cleaned;
+    const body = cleaned
+        .replace(/^critic-/, "")
+        .replace(/[^a-z0-9]+/g, "-")
+        .replace(/^-+|-+$/g, "");
+    return body ? `critic-${body}` : null;
 }
 export function validateModel(model) {
     if (!MODEL_PATTERN.test(model))

package/dist/cli-output.d.ts

@@ -16,6 +16,7 @@ export declare function renderConfigSetup(plan: ConfigSetupPlan, options: {
     applyHint?: string;
     hint?: string;
 }): string;
+export declare function renderCriticsTable(inventory: AgentInventory): string;
 export declare function renderInventory(inventory: AgentInventory): string;
 export declare function shellCommand(arguments_: readonly string[]): string;
 export declare function scopeArguments(scope: Scope): [] | ["--global"];

package/dist/cli-output.js

@@ -165,6 +165,21 @@ function table(rows) {
         .join("  ")
         .trimEnd());
 }
+export function renderCriticsTable(inventory) {
+    const critics = inventory.profiles.filter((profile) => profile.name.startsWith("critic-") && profile.ownership !== "user-owned");
+    if (!critics.length)
+        return "Critics:\n  No critics\n";
+    const rows = [
+        ["NAME", "MODEL", "VARIANT", "PROVIDER"],
+        ...critics.map((profile) => [
+            terminalSafe(profile.name),
+            terminalSafe(profile.model ?? "default"),
+            terminalSafe(profile.variant ?? "-"),
+            terminalSafe(profile.model?.split("/", 1)[0] ?? "-"),
+        ]),
+    ];
+    return `Critics:\n${table(rows).join("\n")}\n`;
+}
 export function renderInventory(inventory) {
     const rows = [
         ["NAME", "MODEL", "VARIANT", "OWNER", "STATE"],

package/dist/command-cli.js

@@ -1,5 +1,5 @@
 import { Command, Option, common, applyConfig, configFileSync, reporter } from "./generated/cli.js";
-import { applyAgentProfileChange, previewAgentProfileChange, listAgentProfiles, availableModels, availableModelVariants, validateAgentName, validateModel, validateVariant, FIXED_AGENT_ROLES, } from "./agent-profiles.js";
+import { applyAgentProfileChange, previewAgentProfileChange, listAgentProfiles, availableModels, availableModelVariants, validateAgentName, validateModel, validateVariant, suggestCriticName, FIXED_AGENT_ROLES, } from "./agent-profiles.js";
 import { apply, preview, installedSelection, defaultSelection, normalizeSelection, InstallerError, SKILL_COMMANDS, SELECTABLE_PLUGINS, } from "./installer.js";
 import { applyConfigSetup, previewConfigSetup, recoverConfigSetup, normalizeConfigSelection, defaultConfigSelection, CONFIG_TARGETS, CONFIG_FRAGMENTS, inspectIntegration, } from "./config-setup.js";
 import { collectDoctorFacts, doctorExitCode } from "./doctor.js";
@@ -9,7 +9,7 @@ import { skillsInstallerSpec } from "./package-metadata.js";
 import { LifecycleError } from "./lifecycle.js";
 import { previewDependencyRemoval, removeDependency, planDependency } from "./self-install.js";
 import { selectOption, selectOptions, promptText, confirmQuestion } from "./terminal-wizard.js";
-import { renderPlan, renderInventory, renderDoctor, renderConfigSetup, renderReconcile, terminalSafe, shellCommand, scopeArguments, } from "./cli-output.js";
+import { renderPlan, renderInventory, renderCriticsTable, renderDoctor, renderConfigSetup, renderReconcile, terminalSafe, shellCommand, scopeArguments, } from "./cli-output.js";
 let diagnostics;
 let machineOutput = false;
 let diagnosticJSON = false;
@@ -27,12 +27,12 @@ const commands = {
         options: ["commands", "agents", "plugins", "core", "dependency", "targets", "fragments"],
     },
     "configure agent": {
-        description: "Choose a model and variant, or supply --model provider/model directly.",
+        description: "Open a staged editor for every agent, or set one model with a name and --model provider/model.",
         options: ["provider", "model", "variant", "clearVariant"],
         name: true,
     },
     "configure critics": {
-        description: "Add, configure, or remove additional critics interactively.",
+        description: "Alias of configure agent: staged agent models and critic management.",
         options: [],
     },
     "configure integration": {
@@ -104,6 +104,26 @@ function json(value) {
 function applyHint(args) {
     return `${shellCommand(args)} (confirm interactively; add --yes outside a terminal)`;
 }
+// Prompt for a critic name, auto-converting unsafe input such as "sonnet-5.5" to
+// "critic-sonnet-5-5" after explicit confirmation; exact safe names pass untouched.
+async function promptCriticName() {
+    for (;;) {
+        const input = value(await promptText("Critic name (critic-<suffix>)"));
+        const suggestion = suggestCriticName(input);
+        if (suggestion !== null && suggestion === input.trim().toLowerCase())
+            return suggestion;
+        if (suggestion) {
+            const accept = value(await selectOption(`"${terminalSafe(input)}" is not a safe critic name`, [
+                `Use ${suggestion}`,
+                "Enter a different name",
+            ]));
+            if (accept === 0)
+                return suggestion;
+            continue;
+        }
+        process.stderr.write("Invalid critic name: use a fixed role or critic-<suffix> with lowercase letters, digits, and hyphens between parts (for example critic-sonnet-5-5).\n");
+    }
+}
 function help(topic = "") {
     if (topic && !commands[topic])
         fail("invalid_input", `Unknown command: ${topic}`);
@@ -348,15 +368,18 @@ async function modelSelection(options, name, current) {
         const candidates = models.filter((model) => model.startsWith(`${provider}/`));
         const model = candidates[value(await selectOption("Model", candidates))];
         const variants = await availableModelVariants(model);
-        const variant = variants.length
-            ? [undefined, ...variants][value(await selectOption("Variant", ["(none)", ...variants]))]
-            : undefined;
-        return { action: "model-set", name, model, variant: variant ?? null };
+        let variant = null;
+        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);
+        }
+        return { action: "model-set", name, model, variant };
     }
     catch (error) {
         if (!(error instanceof LifecycleError && error.code === "catalog_unavailable"))
             throw error;
-        process.stderr.write("Model catalog unavailable; enter an explicit model instead of a catalog pick.\n");
+        process.stderr.write("Start opencode in another terminal to browse models, or enter provider/model manually\n");
         const model = validateModel(value(await promptText("Model (provider/model)")));
         const answer = await promptText(current?.variant ? "Variant (empty keeps current)" : "Variant (optional)");
         const variant = (answer ? validateVariant(answer) : current?.variant) ?? null;
@@ -375,8 +398,7 @@ async function profileDraft(options, names) {
         if (action === "Done")
             return changes;
         if (action === "Add critic") {
-            const suffix = value(await promptText("Critic name (critic-<suffix>)"));
-            const name = validateAgentName(suffix.startsWith("critic-") ? suffix : `critic-${suffix}`);
+            const name = await promptCriticName();
             if (records.has(name))
                 fail("profile_exists", `Profile already exists: ${name}`);
             const request = { ...(await modelSelection(options, name)), action: "critic-add" };
@@ -821,16 +843,10 @@ async function run(args) {
     const options = parse(topic, args);
     if (topic === "configure") {
         requireApply(options);
-        const routes = [
-            "configure components",
-            "configure agent",
-            "configure critics",
-            "configure integration",
-        ];
+        const routes = ["configure components", "configure agent", "configure integration"];
         const selected = value(await selectOption("What would you like to configure?", [
             "Installed components",
-            "Agent models",
-            "Additional critics",
+            "Agent models and critics",
             "Application connection and presets",
         ]));
         topic = routes[selected];
@@ -888,12 +904,32 @@ async function run(args) {
         process.exitCode = doctorExitCode(report);
         return;
     }
-    if (topic === "configure agent" || topic === "agent add-critic") {
+    if (topic === "configure critics" ||
+        topic === "configure agent" ||
+        topic === "agent add-critic") {
         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)) {
+            requireApply(options);
+            if (!options.json)
+                process.stdout.write(renderCriticsTable(inventory));
+            const changes = await profileDraft(options, inventory.profiles
+                .filter((profile) => profile.ownership !== "user-owned")
+                .map((profile) => profile.name));
+            if (!changes.length) {
+                if (options.json)
+                    json({ status: "ok", applied: false });
+                else
+                    process.stdout.write("No profile changes selected.\n");
+                return;
+            }
+            await profileChange(options, { action: "model-set", changes });
+            return;
+        }
         let name = options.name;
         if (!name) {
             if (topic === "agent add-critic")
-                name = value(await promptText("Critic name (critic-<suffix>)"));
+                name = await promptCriticName();
             else {
                 const names = inventory.profiles
                     .filter((profile) => profile.ownership !== "user-owned")
@@ -916,23 +952,6 @@ async function run(args) {
         await profileChange(options, { action: "critic-remove", name: options.name });
         return;
     }
-    if (topic === "configure critics") {
-        requireApply(options);
-        const inventory = await listAgentProfiles(options.scope);
-        const changes = await profileDraft(options, inventory.profiles
-            .filter((profile) => (profile.name === "critic" || profile.name.startsWith("critic-")) &&
-            profile.ownership !== "user-owned")
-            .map((profile) => profile.name));
-        if (!changes.length) {
-            if (options.json)
-                json({ status: "ok", applied: false });
-            else
-                process.stdout.write("No profile changes selected.\n");
-            return;
-        }
-        await profileChange(options, { action: "model-set", changes });
-        return;
-    }
     if (topic === "maintenance recover") {
         requireApply(options);
         const plan = await recoverConfigSetup(options.scope, true);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kisev/agentomatic",
-  "version": "12.0.1-dev.118.geda2dc538b36",
+  "version": "12.0.1-dev.120.g392563c8f5d9",
   "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.118.geda2dc538b36",
+    "@kisev/safe-fs": "12.0.1-dev.120.g392563c8f5d9",
     "commander": "15.0.0",
     "zod": "4.1.8"
   },