@astrojs/cloudflare 10.2.1 → 10.2.3

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/index.js

@@ -14,11 +14,17 @@ export function mutatePageMapInPlace(node, constsToRemove, s) {
         if (!arrayExpression.elements[1] || arrayExpression.elements[1].type !== 'Identifier')
             continue;
         if (constsToRemove.includes(arrayExpression.elements[1].name)) {
-            console.log(arrayExpression);
             // @ts-expect-error - @types/estree seem to be wrong
             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.1",
+  "version": "10.2.3",
   "type": "module",
   "types": "./dist/index.d.ts",
   "author": "withastro",
@@ -49,8 +49,8 @@
     "rollup": "^4.14.0",
     "strip-ansi": "^7.1.0",
     "vite": "^5.2.6",
-    "@astrojs/test-utils": "0.0.1",
-    "astro-scripts": "0.0.14"
+    "astro-scripts": "0.0.14",
+    "@astrojs/test-utils": "0.0.1"
   },
   "publishConfig": {
     "provenance": true