poe-code 3.0.283 → 3.0.284

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "poe-code",
-  "version": "3.0.283",
+  "version": "3.0.284",
   "description": "CLI tool to configure Poe API for developer workflows.",
   "type": "module",
   "main": "./dist/index.js",

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}".`);