@astrojs/cloudflare 10.2.2 → 10.2.4

package/dist/index.js

@@ -274,8 +274,13 @@ export default function createIntegration(args) {
                 // Those modules are build only for prerendering routes.
                 const chunksToDelete = chunkAnalyzer.getNonServerChunks();
                 for (const chunk of chunksToDelete) {
-                    // Chunks are located on `./_worker.js` directory inside of the output directory
-                    await unlink(new URL(`./_worker.js/${chunk}`, _config.outDir));
+                    try {
+                        // Chunks are located on `./_worker.js` directory inside of the output directory
+                        await unlink(new URL(`./_worker.js/${chunk}`, _config.outDir));
+                    }
+                    catch (error) {
+                        logger.warn(`Issue while trying to delete unused file from server bundle: ${new URL(`./_worker.js/${chunk}`, _config.outDir).toString()}`);
+                    }
                 }
             },
         },

package/dist/utils/generate-routes-json.js

@@ -114,6 +114,9 @@ export async function createRoutesFile(_config, logger, routes, pages, redirects
         [{ content: '', dynamic: true, spread: false }],
     ], _config);
     excludePaths.push(assetsPath);
+    for (const redirect of redirects) {
+        excludePaths.push(segmentsToCfSyntax(redirect, _config));
+    }
     if (existsSync(fileURLToPath(_config.publicDir))) {
         const staticFiles = await glob(`${fileURLToPath(_config.publicDir)}/**/*`, {
             cwd: fileURLToPath(_config.publicDir),
@@ -133,9 +136,6 @@ export async function createRoutesFile(_config, logger, routes, pages, redirects
             excludePaths.push(segmentsToCfSyntax(segments, _config));
         }
     }
-    for (const redirect of redirects) {
-        excludePaths.push(segmentsToCfSyntax(redirect, _config));
-    }
     let hasPrerendered404 = false;
     for (const route of routes) {
         const convertedPath = segmentsToCfSyntax(route.segments, _config);

package/dist/utils/index.js

@@ -18,6 +18,13 @@ export function mutatePageMapInPlace(node, constsToRemove, s) {
             if (arrayExpression.start && arrayExpression.end) {
                 // @ts-expect-error - @types/estree seem to be wrong
                 s.remove(arrayExpression.start, arrayExpression.end);
+                // We need to check if there are any leftover commas, which are not part of the `ArrayExpression` node
+                // @ts-expect-error - @types/estree seem to be wrong
+                const endChar = s.slice(arrayExpression.end, arrayExpression.end + 1);
+                if (endChar.includes(',')) {
+                    // @ts-expect-error - @types/estree seem to be wrong
+                    s.remove(arrayExpression.end, arrayExpression.end + 1);
+                }
             }
         }
     }

package/dist/utils/non-server-chunk-detector.js

@@ -11,6 +11,9 @@ export class NonServerChunkDetector {
         return {
             name: 'non-server-chunk-detector',
             generateBundle: (_, bundle) => {
+                // Skip if we bundle for client
+                if (!bundle['index.js'])
+                    return;
                 this.processBundle(bundle);
             },
         };

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@astrojs/cloudflare",
   "description": "Deploy your site to Cloudflare Workers/Pages",
-  "version": "10.2.2",
+  "version": "10.2.4",
   "type": "module",
   "types": "./dist/index.d.ts",
   "author": "withastro",