@real-router/validation-plugin 0.5.1 → 0.6.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@real-router/validation-plugin",
-  "version": "0.5.1",
+  "version": "0.6.1",
   "type": "commonjs",
   "description": "Route validation plugin with type guards and parameter validation",
   "main": "./dist/cjs/index.js",
@@ -44,23 +44,23 @@
   },
   "sideEffects": false,
   "dependencies": {
-    "@real-router/core": "^0.48.0",
+    "@real-router/core": "^0.50.0",
     "@real-router/logger": "^0.3.0"
   },
   "peerDependencies": {
-    "@real-router/core": "^0.48.0"
+    "@real-router/core": "^0.50.0"
   },
   "devDependencies": {
     "route-tree": "^0.3.4",
     "type-guards": "^0.4.8"
   },
   "scripts": {
-    "build": "tsdown --config-loader unrun",
     "test": "vitest",
     "type-check": "tsc --noEmit",
     "lint": "eslint --cache --ext .ts src/ tests/ --fix --max-warnings 0",
     "lint:package": "publint",
     "lint:types": "attw --pack .",
-    "test:properties": "vitest run --config vitest.config.properties.mts"
+    "test:properties": "vitest run --config vitest.config.properties.mts",
+    "bundle": "tsdown --config-loader unrun"
   }
 }

package/src/validationPlugin.ts

@@ -57,6 +57,7 @@ import {
   validateForwardToTargetsStore,
   validateDependenciesStructure,
   validateLimitsConsistency,
+  validateResolvedDefaultRoute,
 } from "./validators/retrospective";
 import {
   validateBuildPathArgs,
@@ -186,6 +187,7 @@ function buildValidatorObject(ctx: RouterInternals): RouterValidator {
         validateLimits(limits, "validate");
       },
       validateOptions,
+      validateResolvedDefaultRoute,
     },
     dependencies: {
       validateDependencyName,
@@ -315,6 +317,10 @@ export function validationPlugin(): PluginFactory {
         options as unknown,
         "constructor (retrospective)",
       );
+
+      if (typeof options.defaultRoute === "string") {
+        validateResolvedDefaultRoute(options.defaultRoute, store);
+      }
     } catch (error) {
       ctx.validator = null;
 

package/src/validators/options.ts

@@ -1,5 +1,6 @@
 // packages/validation-plugin/src/validators/options.ts
 
+import { logger } from "@real-router/logger";
 import { isObjKey } from "type-guards";
 
 const VALID_OPTION_VALUES = {
@@ -98,6 +99,19 @@ export function validateLimits(
 
     validateLimitValue(key as keyof LimitsConfig, value, methodName);
   }
+
+  const { warnListeners, maxListeners } = limits as Partial<LimitsConfig>;
+
+  if (
+    typeof warnListeners === "number" &&
+    typeof maxListeners === "number" &&
+    maxListeners > 0 &&
+    warnListeners > maxListeners
+  ) {
+    throw new RangeError(
+      `[router.${methodName}] "limits.warnListeners" (${warnListeners}) must not exceed "limits.maxListeners" (${maxListeners}) — the warning channel would be unreachable`,
+    );
+  }
 }
 
 function validateStringEnum(
@@ -241,6 +255,16 @@ function validateLoggerOption(loggerOpt: unknown, methodName: string): void {
       `[router.${methodName}] Invalid "logger.callbackIgnoresLevel": expected boolean`,
     );
   }
+
+  if (
+    loggerOptions.callbackIgnoresLevel === true &&
+    loggerOptions.callback === undefined
+  ) {
+    logger.error(
+      `router.${methodName}`,
+      `"logger.callbackIgnoresLevel: true" has no effect without "logger.callback" — the option is ignored`,
+    );
+  }
 }
 
 export function validateOptions(options: unknown, methodName: string): void {

package/src/validators/retrospective.ts

@@ -538,3 +538,36 @@ export function validateLimitsConsistency(
   checkRouteCountLimit(store, configuredLimits);
   checkDepCountLimit(deps, configuredLimits);
 }
+
+// =============================================================================
+// 7. validateResolvedDefaultRoute
+// =============================================================================
+
+/**
+ * Validates that a resolved defaultRoute name points to a route that exists
+ * in the tree. Called at two places:
+ *
+ *   1. At plugin registration (retrospective) — with the static string value
+ *      of options.defaultRoute, if any.
+ *   2. At runtime inside resolveDefault() — with the return value of a
+ *      DefaultRouteCallback, on every navigateToDefault() / start() fallback.
+ *
+ * No-op for empty string (means "no default configured" — handled upstream by
+ * NavigationNamespace.navigateToDefault).
+ */
+export function validateResolvedDefaultRoute(
+  routeName: unknown,
+  store: unknown,
+): void {
+  if (typeof routeName !== "string" || !routeName) {
+    return;
+  }
+
+  const routesStore = assertRoutesStore(store, "validateResolvedDefaultRoute");
+
+  if (!routeExistsInTree(routesStore.tree, routeName)) {
+    throw new Error(
+      `[validation-plugin] defaultRoute resolved to non-existent route: "${routeName}"`,
+    );
+  }
+}