add functionality to omit scripts section in package.json

README.md

@@ -413,10 +413,10 @@ excluded for PNPM.
 
 ### A Partial Workaround
 
-If you can't use a lockfile, and you are worried about things breaking, 
-a partial workaround would be to declare 
-dependencies using exact versions in your manifest file. This doesn't prevent 
-your dependencies dependencies from installing newer versions, like a lockfile 
+If you can't use a lockfile, and you are worried about things breaking,
+a partial workaround would be to declare
+dependencies using exact versions in your manifest file. This doesn't prevent
+your dependencies dependencies from installing newer versions, like a lockfile
 would, but at least you minimize the risk of things breaking.
 
 ## Different Package Managers

src/helpers/config.ts

@@ -14,6 +14,7 @@ export type IsolateConfigResolved = {
   workspaceRoot: string;
   excludeLockfile: boolean;
   avoidPnpmPack: boolean;
+  omitScripts?: boolean;
 };
 
 export type IsolateConfig = Partial<IsolateConfigResolved>;
@@ -29,6 +30,7 @@ const configDefaults: IsolateConfigResolved = {
   workspaceRoot: "../..",
   excludeLockfile: false,
   avoidPnpmPack: false,
+  omitScripts: false,
 };
 
 /**

src/helpers/index.ts

@@ -8,6 +8,7 @@ export * from "./find-packages-globs";
 export * from "./get-build-output-dir";
 export * from "./list-local-dependencies";
 export * from "./manifest";
+export * from "./omit-package-scripts";
 export * from "./pack-dependencies";
 export * from "./patch-workspace-entries";
 export * from "./process-build-output-files";

src/helpers/omit-package-scripts.ts

@@ -0,0 +1,19 @@
+import { omit } from "lodash-es";
+import path from "node:path";
+import { readTypedJsonSync } from "~/utils";
+import { PackageManifest } from "./create-packages-registry";
+import fs from "fs-extra";
+
+export async function omitPackageScripts(isolateDir: string) {
+  const isolatePackageJsonPath = path.join(isolateDir, "package.json");
+  const packageManifest = readTypedJsonSync<PackageManifest>(
+    isolatePackageJsonPath
+  );
+
+  const outputManifest = omit(packageManifest, ["scripts"]);
+
+  await fs.writeFile(
+    isolatePackageJsonPath,
+    JSON.stringify(outputManifest, null, 2)
+  );
+}

src/index.ts

@@ -25,6 +25,7 @@ import {
   getBuildOutputDir,
   getConfig,
   listLocalDependencies,
+  omitPackageScripts,
   packDependencies,
   processBuildOutputFiles,
   processLockfile,
@@ -199,6 +200,20 @@ async function start() {
     log.debug("Copied .npmrc file to the isolate output");
   }
 
+  /**
+   * Firebase Functions relies on the build script from package.json.
+   * However, Cloud Build only recognizes npm, not pnpm, leading to issues with
+   * commands like pnpm nest build. To resolve this, the script section from package.json
+   * is removed, as there is no need to rerun the script commands when the file, which
+   * should already be built and ready for deployment, is being deployed.
+   *
+   * Ensure smooth deployments in a pnpm environment by utilizing the omitScripts option to prevent issues during deployment.
+   */
+  if (config.omitScripts) {
+    await omitPackageScripts(isolateDir);
+    log.debug("Removed the scripts section from package.json in isolate");
+  }
+
   /**
    * Clean up. Only so this in the happy path, so we can look at the temp folder
    * when thing go wrong.