toolcraft 0.0.41 → 0.0.43

package/dist/cli.js

@@ -402,10 +402,13 @@ function parseEnumValue(value, values, label) {
     return match;
 }
 function validateStringPattern(value, schema, label) {
-    if (schema.pattern === undefined) {
-        return value;
+    if (schema.minLength !== undefined && value.length < schema.minLength) {
+        throw new UserError(`Invalid value for "${label}". Expected a string with length at least ${schema.minLength}, got string with length ${value.length}.`);
+    }
+    if (schema.maxLength !== undefined && value.length > schema.maxLength) {
+        throw new UserError(`Invalid value for "${label}". Expected a string with length at most ${schema.maxLength}, got string with length ${value.length}.`);
     }
-    if (!matchesStringPattern(value, schema.pattern)) {
+    if (schema.pattern !== undefined && !matchesStringPattern(value, schema.pattern)) {
         throw new UserError(`Invalid value for "${label}": "${value}" does not match pattern "${schema.pattern}".`);
     }
     return value;
@@ -585,6 +588,14 @@ function parseArrayValue(value, schema, label) {
     }
     return splitArrayInput(value).map((item) => parseScalarValue(item, itemSchema, label));
 }
+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}).`);
+    }
+    if (schema.maxItems !== undefined && value.length > schema.maxItems) {
+        throw new UserError(`Invalid value for "${label}". Expected an array with at most ${schema.maxItems} items, got array(${value.length}).`);
+    }
+}
 function createOption(field, globalLongOptionFlags) {
     const flags = formatOptionFlags(field, globalLongOptionFlags);
     const collidesWithGlobalFlag = globalLongOptionFlags.has(field.optionFlag);
@@ -979,7 +990,11 @@ function formatExampleCommand(breadcrumb, rootUsageName, params) {
     const commandPath = buildUsageLine(breadcrumb, rootUsageName, "");
     const flags = Object.entries(params).map(([key, value]) => {
         const flag = `--${key}`;
-        return typeof value === "boolean" ? (value ? flag : `--no-${key}`) : `${flag} ${formatExampleValue(value)}`;
+        return typeof value === "boolean"
+            ? value
+                ? flag
+                : `--no-${key}`
+            : `${flag} ${formatExampleValue(value)}`;
     });
     return [commandPath, ...flags].filter((token) => token.length > 0).join(" ");
 }
@@ -1493,6 +1508,9 @@ function describeExpectedPresetValue(schema) {
     if (schema.kind === "array") {
         return "an array";
     }
+    if (schema.kind === "number") {
+        return getExpectedNumberDescription(schema);
+    }
     if (schema.kind === "json") {
         return "valid JSON";
     }
@@ -1510,6 +1528,12 @@ function validatePresetScalarValue(value, schema, fieldPath, presetPath) {
             if (typeof value !== "string") {
                 break;
             }
+            if (schema.minLength !== undefined && value.length < schema.minLength) {
+                throw new UserError(`Preset file "${presetPath}" has an invalid value for "${fieldPath}". Expected a string with length at least ${schema.minLength}, got string with length ${value.length}.`);
+            }
+            if (schema.maxLength !== undefined && value.length > schema.maxLength) {
+                throw new UserError(`Preset file "${presetPath}" has an invalid value for "${fieldPath}". Expected a string with length at most ${schema.maxLength}, got string with length ${value.length}.`);
+            }
             if (schema.pattern !== undefined && !matchesStringPattern(value, schema.pattern)) {
                 throw new UserError(`Preset file "${presetPath}" has an invalid value for "${fieldPath}": "${value}" does not match pattern "${schema.pattern}".`);
             }
@@ -1548,6 +1572,12 @@ function validatePresetFieldValue(value, field, presetPath) {
     if (!Array.isArray(value)) {
         throw new UserError(`Preset file "${presetPath}" has an invalid value for "${field.displayPath}". Expected an array, got ${describeReceived(value)}.`);
     }
+    if (field.schema.minItems !== undefined && value.length < field.schema.minItems) {
+        throw new UserError(`Preset file "${presetPath}" has an invalid value for "${field.displayPath}". Expected an array with at least ${field.schema.minItems} items, got array(${value.length}).`);
+    }
+    if (field.schema.maxItems !== undefined && value.length > field.schema.maxItems) {
+        throw new UserError(`Preset file "${presetPath}" has an invalid value for "${field.displayPath}". Expected an array with at most ${field.schema.maxItems} items, got array(${value.length}).`);
+    }
     return value.map((item) => validatePresetScalarValue(item, itemSchema, field.displayPath, presetPath));
 }
 async function loadPresetValues(fields, presetPath) {
@@ -1959,12 +1989,17 @@ function parseOptionFieldValue(field, value, errors) {
                 }
                 parsedValues.push(...parsed);
             }
+            validateArrayBounds(parsedValues, field.schema, field.displayPath);
             return { ok: true, value: parsedValues };
         }
         if (typeof value !== "string") {
             return { ok: true, value };
         }
-        return { ok: true, value: parseFieldInputValue(value, field.schema, field.displayPath) };
+        const parsedValue = parseFieldInputValue(value, field.schema, field.displayPath);
+        if (field.schema.kind === "array" && Array.isArray(parsedValue)) {
+            validateArrayBounds(parsedValue, field.schema, field.displayPath);
+        }
+        return { ok: true, value: parsedValue };
     }
     catch (error) {
         if (error instanceof UserError || error instanceof InvalidArgumentError) {
@@ -2026,6 +2061,7 @@ function consumeFieldValue(args, index, schema, label, inlineValue) {
         if (values.length === 0) {
             throw new InvalidArgumentError(`option '${label}' argument missing`);
         }
+        validateArrayBounds(values, schema, label);
         return {
             nextIndex,
             value: values

package/dist/mcp.js

@@ -360,6 +360,9 @@ function validateSchemaValue(schema, value, casing, label, errors) {
                     message: `Invalid value for "${label}". Expected a string, got ${describeReceived(value)}.`
                 });
             }
+            else {
+                validateStringConstraints(unwrappedSchema, value, label, errors);
+            }
             return value;
         case "number":
             if (!isValidNumberSchemaValue(value, unwrappedSchema)) {
@@ -390,6 +393,7 @@ function validateSchemaValue(schema, value, casing, label, errors) {
                 });
                 return value;
             }
+            validateArrayConstraints(unwrappedSchema, value, label, errors);
             return value.map((item, index) => validateSchemaValue(unwrappedSchema.item, item, casing, `${label}[${index}]`, errors));
         case "object":
             return validateObjectSchema(unwrappedSchema, value, casing, label, errors);
@@ -437,6 +441,40 @@ function validateSchemaValue(schema, value, casing, label, errors) {
         }
     }
 }
+function validateStringConstraints(schema, value, label, errors) {
+    if (schema.minLength !== undefined && value.length < schema.minLength) {
+        errors.push({
+            path: label,
+            message: `Invalid value for "${label}". Expected a string with length at least ${schema.minLength}, got string with length ${value.length}.`
+        });
+    }
+    if (schema.maxLength !== undefined && value.length > schema.maxLength) {
+        errors.push({
+            path: label,
+            message: `Invalid value for "${label}". Expected a string with length at most ${schema.maxLength}, got string with length ${value.length}.`
+        });
+    }
+    if (schema.pattern !== undefined && !new RegExp(schema.pattern).test(value)) {
+        errors.push({
+            path: label,
+            message: `Invalid value for "${label}": "${value}" does not match pattern "${schema.pattern}".`
+        });
+    }
+}
+function validateArrayConstraints(schema, value, label, errors) {
+    if (schema.minItems !== undefined && value.length < schema.minItems) {
+        errors.push({
+            path: label,
+            message: `Invalid value for "${label}". Expected an array with at least ${schema.minItems} items, got array(${value.length}).`
+        });
+    }
+    if (schema.maxItems !== undefined && value.length > schema.maxItems) {
+        errors.push({
+            path: label,
+            message: `Invalid value for "${label}". Expected an array with at most ${schema.maxItems} items, got array(${value.length}).`
+        });
+    }
+}
 function validateObjectSchema(schema, value, casing, label, errors) {
     if (!isPlainObject(value)) {
         errors.push({
@@ -545,7 +583,9 @@ function serializeResultValue(schema, value, casing, label, errors) {
             if (branch === undefined) {
                 const branchNames = Object.keys(unwrappedSchema.branches);
                 errors.push({
-                    path: label.length === 0 ? unwrappedSchema.discriminator : `${label}.${unwrappedSchema.discriminator}`,
+                    path: label.length === 0
+                        ? unwrappedSchema.discriminator
+                        : `${label}.${unwrappedSchema.discriminator}`,
                     message: `Invalid value for "${label.length === 0 ? unwrappedSchema.discriminator : `${label}.${unwrappedSchema.discriminator}`}". Expected one of: ${branchNames.join(", ")}, got ${describeReceived(discriminator)}.`
                 });
                 return value;
@@ -654,9 +694,7 @@ function throwResultValidationErrors(errors) {
     if (errors.length === 1) {
         throw new ToolError(JSON_RPC_ERROR_CODES.INTERNAL_ERROR, errors[0]?.message ?? "Invalid command result.");
     }
-    const rendered = errors
-        .slice(0, 10)
-        .map((error) => `  - ${error.path}: ${error.message}`);
+    const rendered = errors.slice(0, 10).map((error) => `  - ${error.path}: ${error.message}`);
     const remaining = errors.length - rendered.length;
     if (remaining > 0) {
         rendered.push(`  ... and ${remaining} more`);

package/dist/sdk.js

@@ -146,6 +146,9 @@ function validateSchemaValue(schema, value, label, errors) {
                     message: `Invalid value for "${label}". Expected a string, got ${describeReceived(value)}.`
                 });
             }
+            else {
+                validateStringConstraints(unwrappedSchema, value, label, errors);
+            }
             return value;
         case "number":
             if (!isValidNumberSchemaValue(value, unwrappedSchema)) {
@@ -176,6 +179,7 @@ function validateSchemaValue(schema, value, label, errors) {
                 });
                 return value;
             }
+            validateArrayConstraints(unwrappedSchema, value, label, errors);
             return value.map((item, index) => validateSchemaValue(unwrappedSchema.item, item, `${label}[${index}]`, errors));
         case "object":
             return validateObjectSchema(unwrappedSchema, value, label, errors);
@@ -220,6 +224,40 @@ function validateSchemaValue(schema, value, label, errors) {
         }
     }
 }
+function validateStringConstraints(schema, value, label, errors) {
+    if (schema.minLength !== undefined && value.length < schema.minLength) {
+        errors.push({
+            path: label,
+            message: `Invalid value for "${label}". Expected a string with length at least ${schema.minLength}, got string with length ${value.length}.`
+        });
+    }
+    if (schema.maxLength !== undefined && value.length > schema.maxLength) {
+        errors.push({
+            path: label,
+            message: `Invalid value for "${label}". Expected a string with length at most ${schema.maxLength}, got string with length ${value.length}.`
+        });
+    }
+    if (schema.pattern !== undefined && !new RegExp(schema.pattern).test(value)) {
+        errors.push({
+            path: label,
+            message: `Invalid value for "${label}": "${value}" does not match pattern "${schema.pattern}".`
+        });
+    }
+}
+function validateArrayConstraints(schema, value, label, errors) {
+    if (schema.minItems !== undefined && value.length < schema.minItems) {
+        errors.push({
+            path: label,
+            message: `Invalid value for "${label}". Expected an array with at least ${schema.minItems} items, got array(${value.length}).`
+        });
+    }
+    if (schema.maxItems !== undefined && value.length > schema.maxItems) {
+        errors.push({
+            path: label,
+            message: `Invalid value for "${label}". Expected an array with at most ${schema.maxItems} items, got array(${value.length}).`
+        });
+    }
+}
 function validateObjectSchema(schema, value, label, errors) {
     if (!isPlainObject(value)) {
         errors.push({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "toolcraft",
-  "version": "0.0.41",
+  "version": "0.0.43",
   "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.41",
+    "toolcraft-schema": "0.0.43",
     "commander": "^14.0.3",
     "fast-string-width": "^3.0.2",
     "fast-wrap-ansi": "^0.2.0",