@opennextjs/aws 3.9.13 → 3.9.15

package/dist/build/copyTracedFiles.d.ts

@@ -1,3 +1,10 @@
+/**
+ * Copies a file and ensures the destination is writable.
+ * This is necessary because copyFileSync preserves file permissions,
+ * and source files may be read-only (e.g., in Bazel's node_modules).
+ * Without this, subsequent patches would fail with EACCES errors.
+ */
+export declare function copyFileAndMakeOwnerWritable(src: string, dest: string): void;
 export declare function isExcluded(srcPath: string): boolean;
 interface CopyTracedFilesOptions {
     buildOutputPath: string;

package/dist/build/copyTracedFiles.js

@@ -1,11 +1,25 @@
 import url from "node:url";
-import { copyFileSync, cpSync, existsSync, mkdirSync, readFileSync, readdirSync, readlinkSync, statSync, symlinkSync, writeFileSync, } from "node:fs";
+import { chmodSync, copyFileSync, cpSync, existsSync, mkdirSync, readFileSync, readdirSync, readlinkSync, statSync, symlinkSync, writeFileSync, } from "node:fs";
 import path from "node:path";
 import { loadAppPathsManifest, loadBuildId, loadConfig, loadFunctionsConfigManifest, loadMiddlewareManifest, loadPagesManifest, loadPrerenderManifest, } from "../adapters/config/util.js";
 import { getCrossPlatformPathRegex } from "../utils/regex.js";
 import logger from "../logger.js";
 import { INSTRUMENTATION_TRACE_FILE, MIDDLEWARE_TRACE_FILE, } from "./constant.js";
 const __dirname = url.fileURLToPath(new URL(".", import.meta.url));
+/**
+ * Copies a file and ensures the destination is writable.
+ * This is necessary because copyFileSync preserves file permissions,
+ * and source files may be read-only (e.g., in Bazel's node_modules).
+ * Without this, subsequent patches would fail with EACCES errors.
+ */
+export function copyFileAndMakeOwnerWritable(src, dest) {
+    copyFileSync(src, dest);
+    // Ensure the copied file is writable (add owner write permission)
+    const stats = statSync(dest);
+    if (!(stats.mode & 0o200)) {
+        chmodSync(dest, stats.mode | 0o200);
+    }
+}
 //TODO: we need to figure which packages we could safely remove
 const EXCLUDED_PACKAGES = [
     "caniuse-lite",
@@ -29,7 +43,7 @@ export function isExcluded(srcPath) {
 function copyPatchFile(outputDir) {
     const patchFile = path.join(__dirname, "patch", "patchedAsyncStorage.js");
     const outputPatchFile = path.join(outputDir, "patchedAsyncStorage.cjs");
-    copyFileSync(patchFile, outputPatchFile);
+    copyFileAndMakeOwnerWritable(patchFile, outputPatchFile);
 }
 export function getManifests(nextDir) {
     return {
@@ -138,13 +152,13 @@ File ${serverPath} does not exist
     // Check for instrumentation trace file
     if (existsSync(path.join(dotNextDir, INSTRUMENTATION_TRACE_FILE))) {
         // We still need to copy the nft.json file so that computeCopyFilesForPage doesn't throw
-        copyFileSync(path.join(dotNextDir, INSTRUMENTATION_TRACE_FILE), path.join(standaloneNextDir, INSTRUMENTATION_TRACE_FILE));
+        copyFileAndMakeOwnerWritable(path.join(dotNextDir, INSTRUMENTATION_TRACE_FILE), path.join(standaloneNextDir, INSTRUMENTATION_TRACE_FILE));
         computeCopyFilesForPage("instrumentation");
         logger.debug("Adding instrumentation trace files");
     }
     if (existsSync(path.join(dotNextDir, MIDDLEWARE_TRACE_FILE))) {
         // We still need to copy the nft.json file so that computeCopyFilesForPage doesn't throw
-        copyFileSync(path.join(dotNextDir, MIDDLEWARE_TRACE_FILE), path.join(standaloneNextDir, MIDDLEWARE_TRACE_FILE));
+        copyFileAndMakeOwnerWritable(path.join(dotNextDir, MIDDLEWARE_TRACE_FILE), path.join(standaloneNextDir, MIDDLEWARE_TRACE_FILE));
         computeCopyFilesForPage("middleware");
         logger.debug("Adding node middleware trace files");
     }
@@ -207,7 +221,7 @@ File ${serverPath} does not exist
             // where some files listed in the .nft.json might not be present in the standalone folder
             // TODO: investigate that further - is it expected?
             try {
-                copyFileSync(from, to);
+                copyFileAndMakeOwnerWritable(from, to);
             }
             catch (e) {
                 logger.debug("Error copying file:", e);
@@ -218,7 +232,7 @@ File ${serverPath} does not exist
     readdirSync(standaloneNextDir)
         .filter((fileOrDir) => !statSync(path.join(standaloneNextDir, fileOrDir)).isDirectory())
         .forEach((file) => {
-        copyFileSync(path.join(standaloneNextDir, file), path.join(outputNextDir, file));
+        copyFileAndMakeOwnerWritable(path.join(standaloneNextDir, file), path.join(outputNextDir, file));
         tracedFiles.push(path.join(outputNextDir, file));
     });
     // We then need to copy all the files at the root of server
@@ -227,7 +241,7 @@ File ${serverPath} does not exist
         .filter((fileOrDir) => !statSync(path.join(standaloneServerDir, fileOrDir)).isDirectory())
         .filter((file) => file !== "server.js")
         .forEach((file) => {
-        copyFileSync(path.join(standaloneServerDir, file), path.join(path.join(outputNextDir, "server"), file));
+        copyFileAndMakeOwnerWritable(path.join(standaloneServerDir, file), path.join(path.join(outputNextDir, "server"), file));
         tracedFiles.push(path.join(outputNextDir, "server", file));
     });
     // Copy patch file
@@ -245,7 +259,7 @@ File ${serverPath} does not exist
             mkdirSync(path.dirname(path.join(outputNextDir, filePath)), {
                 recursive: true,
             });
-            copyFileSync(path.join(standaloneNextDir, filePath), path.join(outputNextDir, filePath));
+            copyFileAndMakeOwnerWritable(path.join(standaloneNextDir, filePath), path.join(outputNextDir, filePath));
         }
     };
     const manifests = getManifests(standaloneNextDir);

package/dist/build/patch/patches/patchNextServer.d.ts

@@ -1,6 +1,7 @@
 import type { CodePatcher } from "../codePatcher.js";
 export declare const disablePreloadingRule = "\nrule:\n  kind: statement_block\n  inside:\n    kind: if_statement\n    any:\n      - has:\n          kind: member_expression\n          pattern: this.nextConfig.experimental.preloadEntriesOnStart\n          stopBy: end\n      - has:\n          kind: binary_expression\n          pattern: appDocumentPreloading === true\n          stopBy: end\nfix:\n  '{}'\n";
 export declare const removeMiddlewareManifestRule = "\nrule:\n  kind: statement_block\n  inside:\n    kind: method_definition\n    has:\n      kind: property_identifier\n      regex: ^getMiddlewareManifest$\nfix:\n  '{return null;}'\n";
+export declare const emptyHandleNextImageRequestRule = "\nrule:\n  kind: assignment_expression\n  pattern: this.handleNextImageRequest = $VALUE\n  inside:\n    kind: method_definition\n    stopBy: end\n    has:\n      kind: property_identifier\n      regex: ^constructor$\n    inside:\n      kind: class_body\nfix:\n  this.handleNextImageRequest = async (req, res, parsedUrl) => false\n";
 /**
  * Swaps the body for a throwing implementation
  *

package/dist/build/patch/patches/patchNextServer.js

@@ -30,6 +30,23 @@ rule:
 fix:
   '{return null;}'
 `;
+// Make `handleNextImageRequest` a no-op to avoid pulling `sharp`
+// Apply to Next 14, 15, and 16
+export const emptyHandleNextImageRequestRule = `
+rule:
+  kind: assignment_expression
+  pattern: this.handleNextImageRequest = $VALUE
+  inside:
+    kind: method_definition
+    stopBy: end
+    has:
+      kind: property_identifier
+      regex: ^constructor$
+    inside:
+      kind: class_body
+fix:
+  this.handleNextImageRequest = async (req, res, parsedUrl) => false
+`;
 /**
  * Swaps the body for a throwing implementation
  *
@@ -81,6 +98,12 @@ export const patchNextServer = {
             contentFilter: /imageOptimizer\(/,
             patchCode: createPatchCode(createEmptyBodyRule("imageOptimizer")),
         },
+        // Make `handleNextImageRequest` a no-op to avoid pulling `sharp` - unused in OpenNext
+        {
+            pathFilter,
+            contentFilter: /handleNextImageRequest/,
+            patchCode: createPatchCode(emptyHandleNextImageRequestRule),
+        },
         // Disable Next background preloading done at creation of `NextServer`
         {
             versions: ">=14.0.0",

package/package.json

@@ -3,7 +3,7 @@
     "access": "public"
   },
   "name": "@opennextjs/aws",
-  "version": "3.9.13",
+  "version": "3.9.15",
   "bin": {
     "open-next": "./dist/index.js"
   },
@@ -33,7 +33,7 @@
     "README.md"
   ],
   "dependencies": {
-    "@ast-grep/napi": "^0.40.0",
+    "@ast-grep/napi": "^0.40.5",
     "@aws-sdk/client-cloudfront": "3.398.0",
     "@aws-sdk/client-dynamodb": "^3.398.0",
     "@aws-sdk/client-lambda": "^3.398.0",