poe-code 3.0.282 → 3.0.284

package/packages/toolcraft/dist/cli.js

@@ -588,6 +588,20 @@ function parseArrayValue(value, schema, label) {
     }
     return splitArrayInput(value).map((item) => parseScalarValue(item, itemSchema, label));
 }
+function isNegativeNumericArrayToken(token, schema) {
+    if (!token.startsWith("-") || token.startsWith("--")) {
+        return false;
+    }
+    const itemSchema = unwrapOptional(schema.item);
+    if (itemSchema.kind !== "number") {
+        return false;
+    }
+    const items = splitArrayInput(token);
+    return (items.length > 0 && items.every((item) => isValidNumberSchemaValue(Number(item), itemSchema)));
+}
+function isNextArrayOptionToken(token, schema) {
+    return token.startsWith("-") && !isNegativeNumericArrayToken(token, schema);
+}
 function validateArrayBounds(value, schema, label) {
     if (schema.minItems !== undefined && value.length < schema.minItems) {
         throw new UserError(`Invalid value for "${label}". Expected an array with at least ${schema.minItems} items, got array(${value.length}).`);
@@ -2044,7 +2058,7 @@ function consumeFieldValue(args, index, schema, label, inlineValue) {
         let cursor = index + 1;
         while (cursor < args.length) {
             const token = args[cursor] ?? "";
-            if (token.startsWith("-")) {
+            if (isNextArrayOptionToken(token, schema)) {
                 break;
             }
             const parsed = parseArrayValue(token, schema, label);
@@ -2817,6 +2831,12 @@ function handleRunError(error, options) {
             }));
             return;
         }
+        logger.error(appendUsagePointer(formatCommanderErrorMessage(error), {
+            rootUsageName: options.rootUsageName,
+            commandPath: options.commandPath.length > 0
+                ? options.commandPath
+                : findCurrentCommanderCommandPath(options.program, options.argv ?? process.argv)
+        }));
         return;
     }
     if (isHttpErrorLike(error)) {
@@ -2831,6 +2851,9 @@ function handleRunError(error, options) {
     }
     process.exitCode = 1;
 }
+function formatCommanderErrorMessage(error) {
+    return error.message.startsWith("error:") ? error.message : `error: ${error.message}`;
+}
 function formatInvalidEnumMessage(label, value, values, opts = {}) {
     const suggestions = suggest(value, opts.candidates ?? values.map((candidate) => String(candidate)), opts);
     const suggestionLine = suggestions.length > 0 ? ` Did you mean: ${suggestions.join(", ")}?\n` : " ";

package/packages/toolcraft-codemode/dist/execute.js

@@ -1,10 +1,14 @@
 import { Budget, lint, run } from "@poe-code/safejs/core";
-import { defineCommand } from "toolcraft";
+import { defineCommand, UserError } from "toolcraft";
 import { S } from "toolcraft-schema";
 import { getOwnErrorCode } from "./error-codes.js";
 import { buildHostModules } from "./host-modules.js";
 const executeParams = S.Object({
-    source: S.String({ description: "SafeJS source to execute." })
+    source: S.String({
+        description: "SafeJS source to execute.",
+        minLength: 1,
+        pattern: "\\S"
+    })
 });
 function createBudget(options) {
     return new Budget({
@@ -90,6 +94,9 @@ export function makeExecuteCommand({ root, sdk, entries, budget, scope = ["mcp",
         scope,
         params: executeParams,
         handler: async ({ params }) => {
+            if (params.source.trim().length === 0) {
+                throw new UserError("source must not be empty or whitespace");
+            }
             const { lintModules, modules } = await buildHostModules(root, sdk, entries);
             let diagnostics;
             try {

package/packages/toolcraft-codemode/dist/host-modules.js

@@ -16,6 +16,9 @@ function resolveSdkMember(sdk, path) {
         if (typeof current !== "object" && typeof current !== "function") {
             return undefined;
         }
+        if (!Object.prototype.hasOwnProperty.call(current, segment)) {
+            return undefined;
+        }
         current = current[segment];
     }
     return current;

package/packages/toolcraft-codemode/dist/search.js

@@ -4,18 +4,21 @@ import { resolveCommandEntries } from "./tree.js";
 const K1 = 1.5;
 const B = 0.75;
 const FALLBACK_LIMIT = 10;
+const SEARCH_DETAILS = ["brief", "detailed", "full"];
 const searchParams = S.Object({
     query: S.String({ description: "Search query." }),
-    limit: S.Optional(S.Number({ description: "Maximum result count." })),
-    detail: S.Optional(S.Enum(["brief", "detailed", "full"], {
+    limit: S.Optional(S.Number({
+        description: "Maximum result count.",
+        jsonType: "integer",
+        minimum: 0
+    })),
+    detail: S.Optional(S.Enum(SEARCH_DETAILS, {
         description: "Result detail level."
     }))
 });
 function isWordCharacter(character) {
     const code = character.charCodeAt(0);
-    return ((code >= 97 && code <= 122) ||
-        (code >= 48 && code <= 57) ||
-        character === "_");
+    return (code >= 97 && code <= 122) || (code >= 48 && code <= 57) || character === "_";
 }
 function tokenize(value) {
     const tokens = [];
@@ -89,6 +92,12 @@ function normalizeLimit(limit) {
     }
     return limit;
 }
+function normalizeDetail(detail) {
+    if (SEARCH_DETAILS.includes(detail)) {
+        return detail;
+    }
+    throw new UserError(`detail must be one of: ${SEARCH_DETAILS.join(", ")}, received ${JSON.stringify(detail)}`);
+}
 function toSearchResult(entry, detail) {
     const result = {
         path: entry.path,
@@ -115,7 +124,7 @@ export function makeSearchCommand({ entries, defaults = {}, scope = ["mcp", "sdk
             if (limit === 0) {
                 return [];
             }
-            const detail = params.detail ?? defaults.detail ?? "brief";
+            const detail = normalizeDetail(params.detail ?? defaults.detail ?? "brief");
             const indexedEntries = resolvedEntries.map(indexEntry);
             const frequencies = documentFrequencies(indexedEntries);
             const averageLength = averageDocumentLength(indexedEntries);

package/packages/toolcraft-codemode/dist/tree.js

@@ -22,7 +22,9 @@ function splitWords(value) {
         const isUppercase = character !== lower && character === upper;
         const previous = value[index - 1];
         const next = value[index + 1];
-        const previousIsLowercase = previous !== undefined && previous === previous.toLowerCase() && previous !== previous.toUpperCase();
+        const previousIsLowercase = previous !== undefined &&
+            previous === previous.toLowerCase() &&
+            previous !== previous.toUpperCase();
         const nextIsLowercase = next !== undefined && next === next.toLowerCase() && next !== next.toUpperCase();
         if (isUppercase && current.length > 0 && (previousIsLowercase || nextIsLowercase)) {
             words.push(current.toLowerCase());
@@ -44,6 +46,13 @@ export function formatSdkSegment(segment) {
 export function formatModuleSegment(segment) {
     return splitWords(segment).join("_");
 }
+function formatNamedModuleSegment(segment, kind) {
+    const formatted = formatModuleSegment(segment);
+    if (formatted.length === 0) {
+        throw new Error(`Codemode ${kind} name "${segment}" must include at least one non-separator character.`);
+    }
+    return formatted;
+}
 function commandIsExecutable(command) {
     return command.scope.includes("sdk");
 }
@@ -61,7 +70,9 @@ export async function resolveCommandTree(root, options = {}) {
     const exportsByGroupPath = new Map();
     const paths = new Set();
     function visit(group, groupSegments) {
-        const groupPath = groupSegments.map(formatModuleSegment).join(".");
+        const groupPath = groupSegments
+            .map((segment) => formatNamedModuleSegment(segment, "group"))
+            .join(".");
         for (const child of group.children) {
             if (child.kind === "group") {
                 visit(child, [...groupSegments, child.name]);
@@ -70,7 +81,7 @@ export async function resolveCommandTree(root, options = {}) {
             if (!commandIsExecutable(child)) {
                 continue;
             }
-            const name = formatModuleSegment(child.name);
+            const name = formatNamedModuleSegment(child.name, "command");
             const path = groupPath.length === 0 ? name : `${groupPath}.${name}`;
             if (paths.has(path)) {
                 throw new Error(`Duplicate codemode command path "${path}".`);