toolcraft 0.0.40 → 0.0.41

package/dist/number-schema.js

@@ -1,8 +1,15 @@
 export function isValidNumberSchemaValue(value, schema) {
     return (typeof value === "number" &&
         Number.isFinite(value) &&
-        (schema.jsonType !== "integer" || Number.isInteger(value)));
+        (schema.jsonType !== "integer" || Number.isInteger(value)) &&
+        (schema.minimum === undefined || value >= schema.minimum) &&
+        (schema.maximum === undefined || value <= schema.maximum));
 }
 export function getExpectedNumberDescription(schema) {
-    return schema.jsonType === "integer" ? "an integer" : "a number";
+    const type = schema.jsonType === "integer" ? "an integer" : "a number";
+    const bounds = [
+        schema.minimum === undefined ? undefined : `greater than or equal to ${schema.minimum}`,
+        schema.maximum === undefined ? undefined : `less than or equal to ${schema.maximum}`
+    ].filter((bound) => bound !== undefined);
+    return bounds.length === 0 ? type : `${type} ${bounds.join(" and ")}`;
 }

package/node_modules/@poe-code/agent-defs/dist/agents/claude-code.js

@@ -11,7 +11,7 @@ export const claudeCodeAgent = {
             CLAUDE_CODE_ENABLE_TELEMETRY: "1"
         }
     },
-    configPath: "~/.claude/settings.json",
+    configPath: "~/.claude.json",
     branding: {
         colors: {
             dark: "#C15F3C",

package/node_modules/@poe-code/agent-defs/dist/agents/claude-desktop.js

@@ -3,7 +3,12 @@ export const claudeDesktopAgent = {
     name: "claude-desktop",
     label: "Claude Desktop",
     summary: "Anthropic's official desktop application for Claude",
-    configPath: "~/.claude/settings.json",
+    configPath: "~/.config/Claude/claude_desktop_config.json",
+    configPaths: {
+        darwin: "~/Library/Application Support/Claude/claude_desktop_config.json",
+        linux: "~/.config/Claude/claude_desktop_config.json",
+        win32: "~/AppData/Roaming/Claude/claude_desktop_config.json"
+    },
     branding: {
         colors: {
             dark: "#D97757",

package/node_modules/@poe-code/agent-defs/dist/agents/cursor.js

@@ -5,7 +5,7 @@ export const cursorAgent = {
     label: "Cursor",
     summary: "Cursor's CLI coding agent.",
     binaryName: "cursor-agent",
-    configPath: "~/.cursor/cli-config.json",
+    configPath: "~/.cursor/mcp.json",
     branding: {
         colors: {
             dark: "#FFFFFF",

package/node_modules/@poe-code/agent-defs/dist/agents/kimi.js

@@ -6,7 +6,7 @@ export const kimiAgent = {
     aliases: ["kimi-cli"],
     binaryName: "kimi",
     apiShapes: ["openai-chat-completions"],
-    configPath: "~/.kimi/config.toml",
+    configPath: "~/.kimi/mcp.json",
     branding: {
         colors: {
             dark: "#7B68EE",

package/node_modules/@poe-code/agent-defs/dist/agents/opencode.js

@@ -10,7 +10,7 @@ export const openCodeAgent = {
             OPENCODE_CONFIG_CONTENT: '{"experimental":{"openTelemetry":true}}'
         }
     },
-    configPath: "~/.config/opencode/config.json",
+    configPath: "~/.config/opencode/opencode.json",
     branding: {
         colors: {
             dark: "#4A4F55",

package/node_modules/@poe-code/agent-defs/dist/specifier.js

@@ -2,24 +2,33 @@ import { resolveAgentId } from "./registry.js";
 function getOwnModel(specifier) {
     return Object.prototype.hasOwnProperty.call(specifier, "model") ? specifier.model : undefined;
 }
+function requireNonBlank(value, field) {
+    const trimmed = value.trim();
+    if (trimmed.length === 0) {
+        throw new TypeError(`${field} must not be empty`);
+    }
+    return trimmed;
+}
 export function parseAgentSpecifier(input) {
     const colonIndex = input.indexOf(":");
     if (colonIndex === -1) {
-        return { agent: input.trim() };
+        return { agent: requireNonBlank(input, "agent") };
     }
-    const agent = input.slice(0, colonIndex).trim();
+    const agent = requireNonBlank(input.slice(0, colonIndex), "agent");
     const model = input.slice(colonIndex + 1).trim();
     return {
         agent,
-        ...(model.length > 0 ? { model } : {}),
+        ...(model.length > 0 ? { model } : {})
     };
 }
 export function formatAgentSpecifier(specifier) {
+    const agent = requireNonBlank(specifier.agent, "agent");
     const model = getOwnModel(specifier);
-    if (model) {
-        return `${specifier.agent}:${model}`;
+    const normalizedModel = model?.trim();
+    if (normalizedModel) {
+        return `${agent}:${normalizedModel}`;
     }
-    return specifier.agent;
+    return agent;
 }
 export function normalizeAgentId(input) {
     const specifier = parseAgentSpecifier(input.trim());

package/node_modules/@poe-code/agent-defs/dist/types.d.ts

@@ -14,6 +14,11 @@ export interface AgentDefinition {
     readonly apiShapes?: readonly ApiShapeId[];
     readonly otelCapture?: OtelCaptureDefinition;
     configPath: string;
+    readonly configPaths?: {
+        readonly darwin: string;
+        readonly linux: string;
+        readonly win32: string;
+    };
     branding: {
         colors: {
             dark: string;

package/node_modules/@poe-code/agent-mcp-config/dist/apply.js

@@ -15,6 +15,32 @@ export class UnsupportedAgentError extends Error {
 function isConfigObject(value) {
     return Boolean(value) && typeof value === "object" && !Array.isArray(value);
 }
+function assertNonEmptyString(value, message) {
+    if (typeof value !== "string" || value.trim().length === 0) {
+        throw new Error(message);
+    }
+}
+function assertHttpUrl(value) {
+    assertNonEmptyString(value, "MCP HTTP URL must be a valid http or https URL.");
+    let parsed;
+    try {
+        parsed = new URL(value);
+    }
+    catch {
+        throw new Error("MCP HTTP URL must be a valid http or https URL.");
+    }
+    if (parsed.protocol !== "http:" && parsed.protocol !== "https:") {
+        throw new Error("MCP HTTP URL must be a valid http or https URL.");
+    }
+}
+function validateServerEntry(server) {
+    assertNonEmptyString(server.name, "MCP server name must be a non-empty string.");
+    if (server.config.transport === "stdio") {
+        assertNonEmptyString(server.config.command, "MCP stdio command must be a non-empty string.");
+        return;
+    }
+    assertHttpUrl(server.config.url);
+}
 function resolveServerMap(document, configKey) {
     const value = document[configKey];
     if (value === undefined) {
@@ -32,12 +58,15 @@ export async function configure(agentId, server, options) {
     if (!isSupported(agentId)) {
         throw new UnsupportedAgentError(agentId);
     }
+    validateServerEntry(server);
     const config = getAgentConfig(agentId);
     const configPath = resolveConfigPath(config, options.platform);
     const shapeTransformer = getShapeTransformer(config.shape);
     const shaped = shapeTransformer(server);
+    const enabledServer = { ...server, enabled: true };
+    const enabledShaped = shapeTransformer(enabledServer);
     if (shaped === undefined) {
-        await unconfigure(agentId, server.name, options);
+        await unconfigure(agentId, enabledServer, options);
         return;
     }
     const configDir = getConfigDirectory(configPath);
@@ -57,10 +86,13 @@ export async function configure(agentId, server, options) {
                     ? servers[server.name]
                     : undefined;
                 const shapedServer = shaped;
+                const enabledShapedServer = enabledShaped;
                 if (existingServer !== undefined && isDeepStrictEqual(existingServer, shapedServer)) {
                     return { changed: false, content: document };
                 }
-                if (existingServer !== undefined && server.enabled !== false) {
+                if (existingServer !== undefined &&
+                    (enabledShapedServer === undefined ||
+                        !isDeepStrictEqual(existingServer, enabledShapedServer))) {
                     throw new Error(`MCP server "${server.name}" already exists with different configuration in ${configPath}.`);
                 }
                 const newServers = {
@@ -98,7 +130,8 @@ export async function unconfigure(agentId, server, options) {
                 if (!Object.hasOwn(servers, serverName)) {
                     return { changed: false, content: document };
                 }
-                if (expectedServer !== undefined && !isDeepStrictEqual(servers[serverName], expectedServer)) {
+                if (expectedServer !== undefined &&
+                    !isDeepStrictEqual(servers[serverName], expectedServer)) {
                     return { changed: false, content: document };
                 }
                 const newServers = { ...servers };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "toolcraft",
-  "version": "0.0.40",
+  "version": "0.0.41",
   "type": "module",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -47,7 +47,7 @@
     "postpack": "node ../../scripts/manage-bundled-workspace-deps.mjs cleanup . toolcraft-design @poe-code/frontmatter @poe-code/agent-mcp-config @poe-code/agent-human-in-loop @poe-code/task-list @poe-code/agent-defs @poe-code/config-mutations @poe-code/process-runner tiny-mcp-client mcp-oauth auth-store"
   },
   "dependencies": {
-    "toolcraft-schema": "0.0.40",
+    "toolcraft-schema": "0.0.41",
     "commander": "^14.0.3",
     "fast-string-width": "^3.0.2",
     "fast-wrap-ansi": "^0.2.0",