Reserve top-level .files http route (#39281)

This might be used for files in the future in order to move file hosting to the HTTP router domain.

GitOrigin-RevId: 208123e0ec4ef2b7e00253003cefd0ce8d282a23

Author: thomasballinger

npm-packages/convex/src/server/router.test.ts

@@ -58,6 +58,19 @@ test("HttpRouter", () => {
     });
   }).toThrow();
 
+  expect(() => {
+    http.route({ pathPrefix: "/.files/", method: "GET", handler: action1 });
+  }).toThrow("pathPrefix '/.files/' is reserved");
+  expect(() => {
+    http.route({ path: "/.files", method: "GET", handler: action1 });
+  }).toThrow("is reserved");
+  expect(() => {
+    http.route({ path: "/.files/", method: "GET", handler: action1 });
+  }).toThrow("is reserved");
+  expect(() => {
+    http.route({ path: "/.files/foo/bar", method: "GET", handler: action1 });
+  }).toThrow("is reserved");
+
   expect(http.getRoutes()).toEqual([
     ["/path1", "GET", action1],
     ["/path1", "POST", action2],

npm-packages/convex/src/server/router.ts

@@ -177,6 +177,9 @@ export class HttpRouter {
       if (!spec.path.startsWith("/")) {
         throw new Error(`path '${spec.path}' does not start with a /`);
       }
+      if (spec.path.startsWith("/.files/") || spec.path === "/.files") {
+        throw new Error(`path '${spec.path}' is reserved`);
+      }
       const methods: Map<RoutableMethod, PublicHttpAction> =
         this.exactRoutes.has(spec.path)
           ? this.exactRoutes.get(spec.path)!
@@ -197,6 +200,9 @@ export class HttpRouter {
       if (!spec.pathPrefix.endsWith("/")) {
         throw new Error(`pathPrefix ${spec.pathPrefix} must end with a /`);
       }
+      if (spec.pathPrefix.startsWith("/.files/")) {
+        throw new Error(`pathPrefix '${spec.pathPrefix}' is reserved`);
+      }
       const prefixes =
         this.prefixRoutes.get(method) || new Map<string, PublicHttpAction>();
       if (prefixes.has(spec.pathPrefix)) {