Support static paths with spaces (#14)

Author: afonsograca

src/index.ts

@@ -1,7 +1,7 @@
+import { type DeployConfig } from "@deno/experimental-route-config";
 import type { Adapter } from "@sveltejs/kit";
-import path from "node:path";
 import fsp from "node:fs/promises";
-import { type DeployConfig } from "@deno/experimental-route-config";
+import path from "node:path";
 
 const OUT_DIR = ".deno-deploy";
 
@@ -111,8 +111,9 @@ export default function denoAdapter(): Adapter {
       await walk(assetDir, assets);
       for (const asset of assets) {
         const rel = path.relative(assetDir, asset);
+        const encodedRel = encodeAssetRelativePath(rel);
         staticFiles.push({
-          source: `/${rel.replace(/\\+/, "/")}`,
+          source: `/${encodedRel}`,
           destination: path.join(dirs.static, rel),
         });
       }
@@ -162,3 +163,11 @@ async function walk(dir: string, result: string[]): Promise<void> {
     }
   }
 }
+
+export function encodeAssetRelativePath(rel: string): string {
+  return rel
+    .replace(/\\+/g, "/")
+    .split("/")
+    .map(encodeURIComponent)
+    .join("/");
+}

test/adapter.test.ts

@@ -93,6 +93,14 @@ Deno.test("Adapter - serve static files from root", async () => {
   });
 });
 
+Deno.test("Adapter - serve static files with space in path (URL-encoded)", async () => {
+  await withServer(async (origin) => {
+    const res = await fetch(`${origin}/foo%20bar/baz.txt`);
+    expect(res.status).toEqual(200);
+    expect(await res.text()).toContain("quux");
+  });
+});
+
 Deno.test("Adapter - serve static files with cache headers", async () => {
   const immutableDir = path.join(
     cwd,

test/fixture/static/foo bar/baz.txt

@@ -0,0 +1 @@
+quux

test/utils.test.ts

@@ -0,0 +1,15 @@
+import { expect } from "expect";
+import { encodeAssetRelativePath } from "../src/index.ts";
+
+Deno.test("Utils - encodeAssetRelativePath encodes path segments", () => {
+  expect(encodeAssetRelativePath("foo bar/baz qux.txt")).toEqual(
+    "foo%20bar/baz%20qux.txt",
+  );
+});
+
+Deno.test("Utils - encodeAssetRelativePath normalizes Windows separators before URL-encoding", () => {
+  expect(encodeAssetRelativePath("foo bar\\baz\\qux.txt")).toEqual(
+    "foo%20bar/baz/qux.txt",
+  );
+  expect(encodeAssetRelativePath("a\\b\\c.txt")).toEqual("a/b/c.txt");
+});