@opennextjs/aws 3.9.14 → 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/package.json

@@ -3,7 +3,7 @@
     "access": "public"
   },
   "name": "@opennextjs/aws",
-  "version": "3.9.14",
+  "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",