agentsw 0.10.1 → 0.11.0

package/dist/menu.js

@@ -1,6 +1,6 @@
 import pc from "picocolors";
 import prompts from "prompts";
-import { cmdAdd, cmdQuickAdd, cmdApps, cmdInstall, cmdDiscover, cmdImport, cmdRefreshMeta, cmdList, cmdStatus, cmdSync, cmdUpgrade, cmdUse, } from "./commands.js";
+import { cmdAdd, cmdQuickAdd, cmdApps, cmdInstall, cmdImport, cmdRefreshMeta, cmdList, cmdStatus, cmdSync, cmdUpgrade, cmdUse, } from "./commands.js";
 import { getLocale, setLocale, t } from "./i18n.js";
 import { getMetadataMode } from "./metadata.js";
 import { loadStore, saveStore } from "./store.js";
@@ -66,6 +66,41 @@ async function pickProvider(message) {
     const p = store.providers[id];
     return { id, defaultModel: p.defaultModel, models: p.models.map((m) => m.id) };
 }
+/**
+ * Multi-select providers for one sync. The first row takes every configured provider and maps to
+ * the CLI's `--all`; toggling everything off means "back", not "all", because a sync writes files.
+ */
+async function pickProviders(message) {
+    const store = loadStore();
+    const ids = Object.keys(store.providers);
+    if (ids.length === 0) {
+        out(pc.yellow(t("menu.noProvidersHint")));
+        return undefined;
+    }
+    const { providers } = await prompts({
+        type: "multiselect",
+        name: "providers",
+        message,
+        hint: t("menu.appsHint"),
+        instructions: false,
+        choices: [
+            { title: t("menu.providersAll", { count: ids.length }), value: "__all__" },
+            ...ids.map((pid) => {
+                const p = store.providers[pid];
+                return {
+                    title: `${pid} · ${p.protocol} · ${t("menu.defaultModel")} ${p.defaultModel}${store.active === pid ? `  (${t("menu.active")})` : ""}`,
+                    value: pid,
+                    selected: store.active === pid,
+                };
+            }),
+        ],
+    }, cancel);
+    const chosen = Array.isArray(providers) ? providers : [];
+    if (chosen.includes("__all__"))
+        return { all: true, ids };
+    const selected = chosen.filter((value) => value !== "__all__");
+    return selected.length ? { all: false, ids: selected } : undefined;
+}
 /**
  * Multi-select target apps, remembering the previous choice. The first row selects
  * every detected app; picking it (or toggling everything off) returns undefined,
@@ -212,7 +247,6 @@ export async function cmdMenu() {
                 { title: t("menu.status"), description: t("menu.statusHelp"), value: "status" },
                 { title: t("menu.list"), description: t("menu.listHelp"), value: "list" },
                 { title: t("menu.sync"), description: t("menu.syncHelp"), value: "sync" },
-                { title: t("menu.discover"), description: t("menu.discoverHelp"), value: "discover" },
                 { title: t("menu.metadata"), description: t("menu.metadataHelp"), value: "metadata" },
                 { title: t("menu.rename"), description: t("menu.renameHelp"), value: "rename" },
                 { title: t("menu.remove"), description: t("menu.removeHelp"), value: "remove" },
@@ -271,14 +305,11 @@ export async function cmdMenu() {
                 cmdList();
             }
             else if (action === "sync") {
-                await cmdSync({ apps: await pickApps(t("menu.pickApps")) });
-            }
-            else if (action === "discover") {
-                const picked = await pickProvider(t("menu.discoverFor"));
+                const picked = await pickProviders(t("menu.syncFor"));
                 if (!picked)
                     continue;
-                const sync = await askToggle(t("menu.pushRefresh"));
-                await cmdDiscover(picked.id, { sync, apps: sync ? await pickApps(t("menu.pickApps")) : undefined });
+                const apps = await pickApps(t("menu.pickApps"));
+                await cmdSync(picked.all ? { all: true, apps } : { provider: picked.ids.join(","), apps });
             }
             else if (action === "metadata") {
                 const picked = await pickProvider(t("menu.metadataProvider"));

package/dist/targets/claudecode.js

@@ -14,6 +14,8 @@ export const claudecode = transactionalTarget({
     id: "claude",
     name: "Claude Code",
     protocols: ["anthropic"],
+    // The env block names one endpoint: a second provider replaces the first rather than joining it.
+    singleProvider: true,
     get configPaths() {
         return [claudeSettingsFile()];
     },

package/dist/targets/codex.js

@@ -82,7 +82,11 @@ export const codex = transactionalTarget({
             wire_api: "responses",
             requires_openai_auth: true,
         };
-        notes.push("codex requires an OpenAI Responses-compatible endpoint (/v1/responses); chat-completions-only endpoints will not work");
+        // Only worth saying when the endpoint has not declared itself Responses-compatible:
+        // a provider the user already marked `responses` needs no caveat.
+        if (provider.openaiApi !== "responses") {
+            notes.push("codex requires an OpenAI Responses-compatible endpoint (/v1/responses); chat-completions-only endpoints will not work");
+        }
         config.model_providers = providers;
         const configBackup = backupFile(configFile);
         if (configBackup)

package/dist/targets/dsh.js

@@ -5,7 +5,7 @@ import { dshCredentialsFile, dshDir, dshSettingsFile } from "../app-paths.js";
 import { isManagedCredentialRef, legacyManagedCredentialRef, managedCredentialRef } from "../provider-identity.js";
 import { transactionalTarget } from "../target-transaction.js";
 import { parseYamlMapping, serializeYamlMapping } from "../yaml.js";
-import { apiValue, classifyApi, entryApi, mergeModels, sdkBaseUrl } from "./wire.js";
+import { apiValue, classifyApi, entryApi, hasModelMetadata, mergeModels, sdkBaseUrl } from "./wire.js";
 /** Layout version of `.credentials.yaml` this build reads and writes. */
 const CREDENTIALS_VERSION = 1;
 /** Thinking levels a `reasoningEfforts` entry may offer (llm-pi-ai THINKING_LEVEL_GATE). */
@@ -29,7 +29,8 @@ function modelEntry(m) {
         ...(m.name ? { name: m.name } : {}),
         ...(m.contextWindow ? { contextWindow: m.contextWindow } : {}),
         ...(m.maxOutput ? { maxTokens: m.maxOutput } : {}),
-        input: m.imageInput ? ["text", "image"] : ["text"],
+        // `input` is a guess for a spec that says nothing; see hasModelMetadata.
+        ...(hasModelMetadata(m) ? { input: m.imageInput ? ["text", "image"] : ["text"] } : {}),
         ...(m.reasoning === false
             ? { reasoningEfforts: false }
             : m.reasoning && m.reasoningEfforts?.length

package/dist/targets/omp.js

@@ -5,9 +5,7 @@ import { ompAgentDir, ompModelsFiles } from "../app-paths.js";
 import { looksLikeEnvName } from "../slug.js";
 import { transactionalTarget } from "../target-transaction.js";
 import { parseYamlMapping, serializeYamlMapping } from "../yaml.js";
-import { apiValue, classifyApi, entryApi, mergeModels, sdkBaseUrl, stripConflictingOverrides } from "./wire.js";
-/** Per-model keys this adapter writes; one that stops being emitted is cleared, not inherited. */
-const OWNED_MODEL_KEYS = ["id", "name", "reasoning", "input", "contextWindow", "maxTokens", "cost"];
+import { apiValue, applyDeepseekCompat, classifyApi, entryApi, mergeModels, OWNED_MODEL_METADATA_KEYS, ownedModelMetadata, sdkBaseUrl, stripConflictingOverrides, } from "./wire.js";
 /** omp has no directory override; the file set is shared with remove/rename. */
 function modelFiles() {
     const [yml, yaml] = ompModelsFiles();
@@ -43,27 +41,12 @@ export const omp = transactionalTarget({
         const anthropic = provider.protocol === "anthropic";
         const api = apiValue(provider.protocol, provider.openaiApi, prev?.api ?? entryApi(prev ?? {}));
         const baseUrl = sdkBaseUrl(provider.protocol, provider.baseUrl);
-        const models = mergeModels(prev?.models, provider.models.map((m) => ({
-            id: m.id,
-            ...(m.name ? { name: m.name } : {}),
-            ...(m.reasoning !== undefined ? { reasoning: m.reasoning } : {}),
-            input: m.imageInput ? ["text", "image"] : ["text"],
-            ...(m.contextWindow ? { contextWindow: m.contextWindow } : {}),
-            ...(m.maxOutput ? { maxTokens: m.maxOutput } : {}),
-            ...(m.cost
-                ? {
-                    cost: {
-                        input: m.cost.input ?? 0,
-                        output: m.cost.output ?? 0,
-                        cacheRead: m.cost.cacheRead ?? 0,
-                        cacheWrite: m.cost.cacheWrite ?? 0,
-                    },
-                }
-                : {}),
-        })), OWNED_MODEL_KEYS);
+        const models = mergeModels(prev?.models, provider.models.map(ownedModelMetadata), OWNED_MODEL_METADATA_KEYS);
         const conflicts = stripConflictingOverrides(models, api, baseUrl);
         if (conflicts.length)
             notes.push(`dropped model overrides pointing elsewhere: ${conflicts.join(", ")}`);
+        if (!anthropic)
+            applyDeepseekCompat(models, { requiresReasoningContentForToolCalls: true });
         const entry = {
             baseUrl,
             apiKey: provider.apiKey, // omp treats value as env-var name first, then literal

package/dist/targets/pistyle.js

@@ -5,18 +5,9 @@ import { piDir, piModelsFile, piSettingsFile } from "../app-paths.js";
 import { editJsoncObject, isJsonObject, readJsoncObject } from "../jsonc.js";
 import { looksLikeEnvName } from "../slug.js";
 import { transactionalTarget } from "../target-transaction.js";
-import { apiValue, classifyApi, entryApi, mergeModels, sdkBaseUrl, stripConflictingOverrides } from "./wire.js";
-/** Per-model keys this adapter writes; one a metadata-bearing entry stops emitting is cleared, not inherited. */
-const OWNED_MODEL_KEYS = [
-    "id",
-    "name",
-    "reasoning",
-    "thinkingLevelMap",
-    "input",
-    "contextWindow",
-    "maxTokens",
-    "cost",
-];
+import { apiValue, applyDeepseekCompat, classifyApi, entryApi, mergeModels, OWNED_MODEL_METADATA_KEYS, ownedModelMetadata, sdkBaseUrl, stripConflictingOverrides, } from "./wire.js";
+/** pi writes the shared metadata plus its own thinking-level map. */
+const OWNED_MODEL_KEYS = [...OWNED_MODEL_METADATA_KEYS, "thinkingLevelMap"];
 function readModels(file) {
     const document = readJsoncObject(file);
     const providers = document.value.providers;
@@ -84,26 +75,17 @@ export function piStyleTarget(opts) {
             const api = apiValue(provider.protocol, provider.openaiApi, prev.api ?? entryApi(prev));
             const baseUrl = sdkBaseUrl(provider.protocol, provider.baseUrl);
             const models = mergeModels(prev.models, provider.models.map((m) => ({
-                id: m.id,
-                ...(m.name ? { name: m.name } : {}),
-                ...(m.reasoning !== undefined ? { reasoning: m.reasoning } : {}),
+                ...ownedModelMetadata(m),
                 ...(m.reasoning && m.reasoningEfforts?.length
                     ? { thinkingLevelMap: thinkingLevelMap(m.reasoningEfforts) }
                     : {}),
-                input: m.imageInput ? ["text", "image"] : ["text"],
-                ...(m.contextWindow ? { contextWindow: m.contextWindow } : {}),
-                ...(m.maxOutput ? { maxTokens: m.maxOutput } : {}),
-                ...(m.cost
-                    ? {
-                        cost: {
-                            input: m.cost.input ?? 0,
-                            output: m.cost.output ?? 0,
-                            cacheRead: m.cost.cacheRead ?? 0,
-                            cacheWrite: m.cost.cacheWrite ?? 0,
-                        },
-                    }
-                    : {}),
             })), OWNED_MODEL_KEYS);
+            // pi/prime key their DeepSeek wire rules by provider name, so a gateway
+            // entry never picks them up; the flags below are what their own deepseek
+            // catalog entries carry. Anthropic-wire entries replay differently.
+            if (provider.protocol === "openai") {
+                applyDeepseekCompat(models, { requiresReasoningContentOnAssistantMessages: true, thinkingFormat: "deepseek" });
+            }
             const conflicts = stripConflictingOverrides(models, api, baseUrl);
             if (conflicts.length)
                 notes.push(`dropped model overrides pointing elsewhere: ${conflicts.join(", ")}`);

package/dist/targets/wire.js

@@ -99,6 +99,11 @@ derived = ["input"]) {
         const old = prev.get(m.id);
         if (!old)
             return m;
+        // An entry that carries nothing but its id says nothing about the model: it
+        // is no evidence that the config's metadata went stale, so clearing owned
+        // keys on its behalf would delete what a richer sync or the user put there.
+        if (Object.keys(m).length === 1)
+            return { ...m, ...old };
         const kept = { ...old };
         const known = informative.some((key) => key in m);
         if (known)
@@ -108,6 +113,66 @@ derived = ["input"]) {
         return { ...kept, ...m };
     });
 }
+/** Per-model keys every pi-family adapter owns; see `ownedModelMetadata`. */
+export const OWNED_MODEL_METADATA_KEYS = ["id", "name", "reasoning", "input", "contextWindow", "maxTokens", "cost"];
+const DEEPSEEK_MODEL_ID = /deepseek/i;
+/**
+ * omp/pi/prime key their DeepSeek wire rules by provider name ("deepseek",
+ * "opencode-go", ...), so a gateway or reseller entry matches none of them: its
+ * reasoning replay then breaks and DeepSeek answers 400. Match on the model id
+ * instead and write the adapter's own flags per model; an explicit user value wins.
+ */
+export function applyDeepseekCompat(models, flags) {
+    for (const model of models) {
+        if (!DEEPSEEK_MODEL_ID.test(String(model.id)))
+            continue;
+        const compat = (model.compat ?? {});
+        model.compat = { ...flags, ...compat };
+    }
+}
+/**
+ * Whether a store entry describes the model at all. A bare id is ignorance,
+ * not a catalog statement, so adapters must not clear what the app config
+ * already holds on its behalf.
+ */
+export function hasModelMetadata(m) {
+    return (m.name !== undefined ||
+        m.reasoning !== undefined ||
+        m.reasoningEfforts !== undefined ||
+        m.imageInput !== undefined ||
+        m.contextWindow !== undefined ||
+        m.maxInput !== undefined ||
+        m.maxOutput !== undefined ||
+        m.cost !== undefined);
+}
+/**
+ * The per-model metadata agentsw writes into a pi-family models config. omp and
+ * the pi/prime adapters share the shape; only the extras around it differ, so
+ * keeping one builder is what stops the two from drifting apart.
+ */
+export function ownedModelMetadata(m) {
+    // `input` is a guess for a spec that says nothing; writing it would make the
+    // entry look authoritative to mergeModels and clear the config's own values.
+    const known = hasModelMetadata(m);
+    return {
+        id: m.id,
+        ...(m.name ? { name: m.name } : {}),
+        ...(m.reasoning !== undefined ? { reasoning: m.reasoning } : {}),
+        ...(known ? { input: m.imageInput ? ["text", "image"] : ["text"] } : {}),
+        ...(m.contextWindow ? { contextWindow: m.contextWindow } : {}),
+        ...(m.maxOutput ? { maxTokens: m.maxOutput } : {}),
+        ...(m.cost
+            ? {
+                cost: {
+                    input: m.cost.input ?? 0,
+                    output: m.cost.output ?? 0,
+                    cacheRead: m.cost.cacheRead ?? 0,
+                    cacheWrite: m.cost.cacheWrite ?? 0,
+                },
+            }
+            : {}),
+    };
+}
 /**
  * Drop per-model `api`/`baseUrl` overrides that contradict the route being
  * written. A model-level override wins over the provider entry in omp and pi,

package/dist/targets/workbuddy.js

@@ -6,6 +6,7 @@ import { workbuddyDir } from "../app-paths.js";
 import { localProviderId } from "../provider-identity.js";
 import { providerIdFromBaseUrl, providerNameFromBaseUrl } from "../slug.js";
 import { transactionalTarget } from "../target-transaction.js";
+import { hasModelMetadata } from "./wire.js";
 function configDir() {
     return workbuddyDir();
 }
@@ -101,12 +102,15 @@ export const workbuddy = transactionalTarget({
         const previous = new Map(owned.map((row) => [row.id, row]));
         const ours = provider.models.map((model) => {
             const custom = { ...previous.get(model.id) };
-            for (const key of OWNED_MODEL_KEYS)
-                delete custom[key];
+            // A bare id says nothing about the model; the row keeps what it already has.
+            const known = hasModelMetadata(model);
+            if (known)
+                for (const key of OWNED_MODEL_KEYS)
+                    delete custom[key];
             return {
                 ...custom,
                 id: model.id,
-                name: model.name ?? model.id,
+                name: model.name ?? (known ? undefined : custom.name) ?? model.id,
                 vendor: provider.name,
                 url,
                 apiKey: provider.apiKey,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentsw",
-  "version": "0.10.1",
+  "version": "0.11.0",
   "description": "CLI provider switcher for coding agents: manage OpenAI/Anthropic-protocol providers in one place and sync them, with models.dev metadata, into Claude Code, Codex, omp, pi, prime-agent, opencode, Hermes, WorkBuddy and DeepSeek Harness",
   "type": "module",
   "repository": {