arcway 0.4.9 → 0.4.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "arcway",
-  "version": "0.4.9",
+  "version": "0.4.10",
   "description": "A convention-based framework for building modular monoliths with strict domain boundaries.",
   "license": "MIT",
   "type": "module",

package/server/router/routes.js

@@ -46,6 +46,13 @@ function prefixRoutePattern(pattern, prefix) {
   if (pattern === '/') return normalized;
   return normalized + pattern;
 }
+function defaultRateLimitKey(relativePath, method, prefix) {
+  let endpoint = relativePath.replace(/\\/g, '/').replace(/\.jsx?$/, '');
+  endpoint = endpoint.replace(/\/index$/, '');
+  if (endpoint === 'index') endpoint = '';
+  endpoint = prefixRoutePattern('/' + endpoint, prefix);
+  return `${endpoint}:${method.toLowerCase()}`;
+}
 function buildRoutesFromModules(entries, { prefix = '', plugin } = {}) {
   const routes = [];
   for (const { filePath, relativePath, module } of entries) {
@@ -65,13 +72,22 @@ function buildRoutesFromModules(entries, { prefix = '', plugin } = {}) {
           throw new Error(`Route ${method} ${urlPattern} (${filePath}) ratelimit cannot be empty`);
         }
         config._parsedRateLimits = policies.map((policy, index) => {
-          if (!policy?.key || typeof policy.key !== 'string') {
+          if (!policy || typeof policy !== 'object') {
+            const suffix = policies.length > 1 ? `[${index}]` : '';
+            throw new Error(
+              `Route ${method} ${urlPattern} (${filePath}) ratelimit${suffix} must be an object`,
+            );
+          }
+          if (policy.key !== undefined && (typeof policy.key !== 'string' || !policy.key)) {
             const suffix = policies.length > 1 ? `[${index}]` : '';
             throw new Error(
               `Route ${method} ${urlPattern} (${filePath}) ratelimit${suffix}.key must be a non-empty string`,
             );
           }
-          return { key: policy.key, ...parseRateLimit(policy.limit) };
+          return {
+            key: policy.key ?? defaultRateLimitKey(relativePath, method, prefix),
+            ...parseRateLimit(policy.limit),
+          };
         });
         // Preserve the existing internal field for integrations that inspect a
         // route with one rate-limit policy.