Fix 6206: Cloudflare function path resolving one step above

withastro · Feb 10, 2023 · b9681c6 · b9681c6
1 parent 8b7cb64
commit b9681c6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/packages/integrations/cloudflare/src/index.ts b/packages/integrations/cloudflare/src/index.ts
@@ -205,8 +205,9 @@ export default function createIntegration(args?: Options): AstroIntegration {
 				if (isModeDirectory) {
 					const functionsUrl = new URL('functions', _config.root);
 					await fs.promises.mkdir(functionsUrl, { recursive: true });
-					const directoryUrl = new URL('[[path]].js', functionsUrl);
-					await fs.promises.rename(finalBuildUrl, directoryUrl);
+
+					const directoryUrl = new URL('functions/[[path]].js', _config.root);
+					await fs.promises.rename(finalBuildUrl, directoryUrl.pathname);
 				}
 			},
 		},

diff --git a/packages/integrations/cloudflare/test/directory.test.js b/packages/integrations/cloudflare/test/directory.test.js
@@ -17,5 +17,6 @@ describe('mode: "directory"', () => {
 
 	it('generates functions folder inside the project root', async () => {
 		expect(await fixture.pathExists('../functions')).to.be.true;
+		expect(await fixture.pathExists('../functions/[[path]].js')).to.be.true;
 	});
 });