@zudojs/config 1.0.0 → 1.0.1

package/dist/configSchema/configSchema.validator.js

@@ -124,8 +124,26 @@ function appendCustomValidationResult(result, path, issues) {
     }
     issues.push(result);
 }
+/**
+ * Compiles a string schema pattern for a one-shot `test`.
+ *
+ * A RegExp carrying the `g` or `y` flag keeps `lastIndex` between calls, so
+ * the same schema object alternated between accepting and rejecting an
+ * identical value on successive validations. Both flags are dropped; they
+ * have no meaning for a whole-value match.
+ */
+function compilePattern(pattern) {
+    if (typeof pattern === "string") {
+        return new RegExp(pattern);
+    }
+    const flags = pattern.flags.replace(/[gy]/g, "");
+    return flags === pattern.flags ? pattern : new RegExp(pattern.source, flags);
+}
 /**
  * Built-in schema validation rules.
+ *
+ * Returns a replacement value when a rule rewrote it (an array whose item
+ * schema applied `transform` or `default`), otherwise undefined.
  */
 function validateBuiltInRules(value, schema, context, issues) {
     if (typeof value === "string") {
@@ -139,9 +157,7 @@ function validateBuiltInRules(value, schema, context, issues) {
             issues.push(createConfigValidationIssue(context.path, `Value must contain at most ${stringSchema.maxLength} characters.`, "MAX_LENGTH"));
         }
         if (stringSchema.pattern) {
-            const pattern = typeof stringSchema.pattern === "string"
-                ? new RegExp(stringSchema.pattern)
-                : stringSchema.pattern;
+            const pattern = compilePattern(stringSchema.pattern);
             if (!pattern.test(value)) {
                 issues.push(createConfigValidationIssue(context.path, "Value does not match the required pattern.", "PATTERN"));
             }
@@ -176,6 +192,11 @@ function validateBuiltInRules(value, schema, context, issues) {
             issues.push(createConfigValidationIssue(context.path, `Array must contain at most ${arraySchema.maxItems} items.`, "MAX_ITEMS"));
         }
         if (arraySchema.items) {
+            // Item results used to be consulted for issues only, so an item
+            // schema's `transform` (or `default`) ran and its output was thrown
+            // away: the caller got the untransformed array back as "valid".
+            const rebuilt = [];
+            let allValid = true;
             value.forEach((item, index) => {
                 const result = validateConfigValue(item, arraySchema.items, {
                     path: `${context.path}[${index}]`,
@@ -184,9 +205,15 @@ function validateBuiltInRules(value, schema, context, issues) {
                     key: index,
                 });
                 issues.push(...result.issues);
+                if (!result.valid) {
+                    allValid = false;
+                }
+                rebuilt.push(result.value === undefined ? item : result.value);
             });
+            return allValid ? rebuilt : undefined;
         }
     }
+    return undefined;
 }
 /**
  * Validates a value against a schema.
@@ -239,7 +266,7 @@ export function validateConfigValue(value, schema, context) {
             issues,
         };
     }
-    validateBuiltInRules(value, schema, validationContext, issues);
+    const rewritten = validateBuiltInRules(value, schema, validationContext, issues);
     // An object schema carries `properties` / `additionalProperties`.
     // Those were declared on ConfigObjectSchema but read nowhere in this
     // function, so every nested object schema — including the ones used
@@ -247,7 +274,7 @@ export function validateConfigValue(value, schema, context) {
     // validation unconditionally. Delegate to validateConfigObject so
     // nested constraints are actually enforced.
     const objectSchema = schema;
-    let base = value;
+    let base = rewritten ?? value;
     if (matchesConfigType(value, ConfigValueType.OBJECT) &&
         (objectSchema.properties !== undefined ||
             objectSchema.additionalProperties !== undefined)) {

package/package.json

@@ -1,8 +1,12 @@
 {
   "name": "@zudojs/config",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "Layered configuration management with multiple sources, validation, and environment-specific overrides.",
   "license": "MIT",
+  "author": {
+    "name": "Oluwayemi Oyinlola",
+    "url": "https://github.com/oyinlola-tech"
+  },
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.js",
@@ -24,7 +28,7 @@
     "node": ">=24.0.0"
   },
   "dependencies": {
-    "@zudojs/errors": "1.0.0"
+    "@zudojs/errors": "1.0.1"
   },
   "devDependencies": {
     "typescript": "7.0.2",