replace execa by tinyexec

Author: zvictor

packages/build/package.json

@@ -71,8 +71,8 @@
   "dependencies": {
     "@trigger.dev/core": "workspace:3.3.13",
     "@trigger.dev/sdk": "workspace:3.3.13",
-    "execa": "^9.5.2",
     "pkg-types": "^1.1.3",
+    "tinyexec": "^0.3.2",
     "tinyglobby": "^0.2.2",
     "tsconfck": "3.1.3"
   },

packages/build/src/extensions/python.ts

@@ -1,12 +1,10 @@
 import fs from "node:fs";
 import assert from "node:assert";
-import { execa } from "execa";
 import { additionalFiles } from "./core/additionalFiles.js";
 import { BuildManifest } from "@trigger.dev/core/v3";
 import { BuildContext, BuildExtension } from "@trigger.dev/core/v3/build";
 import { logger } from "@trigger.dev/sdk/v3";
-
-import type { VerboseObject } from "execa";
+import { x, Options as XOptions, Result } from "tinyexec";
 
 export type PythonOptions = {
   requirements?: string[];
@@ -30,8 +28,6 @@ export type PythonOptions = {
   scripts?: string[];
 };
 
-type ExecaOptions = Parameters<typeof execa>[1];
-
 const splitAndCleanComments = (str: string) =>
   str
     .split("\n")
@@ -124,26 +120,24 @@ class PythonExtension implements BuildExtension {
 
 export const run = async (
   scriptArgs: string[] = [],
-  options: ExecaOptions = {}
-): Promise<ReturnType<typeof execa>> => {
+  options: Partial<XOptions> = {}
+): Promise<Result> => {
   const pythonBin = process.env.PYTHON_BIN_PATH || "python";
 
-  const result = await execa({
-    shell: true,
-    verbose: (line: string, obj: VerboseObject) => logger.debug(obj.message, obj),
+  const result = await x(pythonBin, scriptArgs, {
     ...options,
-  })(pythonBin, scriptArgs);
+    throwOnError: false, // Ensure errors are handled manually
+  });
 
   try {
-    assert(!result.failed, `Python command failed: ${result.stderr}\nCommand: ${result.command}`);
     assert(
       result.exitCode === 0,
       `Python command exited with non-zero code ${result.exitCode}\nStdout: ${result.stdout}\nStderr: ${result.stderr}`
     );
   } catch (error) {
     logger.error("Python command execution failed", {
       error: error instanceof Error ? error.message : error,
-      command: result.command,
+      command: `${pythonBin} ${scriptArgs.join(" ")}`,
       stdout: result.stdout,
       stderr: result.stderr,
       exitCode: result.exitCode,
@@ -157,25 +151,23 @@ export const run = async (
 export const runScript = (
   scriptPath: string,
   scriptArgs: string[] = [],
-  options: ExecaOptions = {}
+  options: Partial<XOptions> = {}
 ) => {
   assert(scriptPath, "Script path is required");
   assert(fs.existsSync(scriptPath), `Script does not exist: ${scriptPath}`);
 
   return run([scriptPath, ...scriptArgs], options);
 };
 
-export const runInline = async (scriptContent: string, options: ExecaOptions = {}) => {
+export const runInline = async (scriptContent: string, options: Partial<XOptions> = {}) => {
   assert(scriptContent, "Script content is required");
 
-  // Create a temporary file with restricted permissions
   const tmpFile = `/tmp/script_${Date.now()}.py`;
   await fs.promises.writeFile(tmpFile, scriptContent, { mode: 0o600 });
 
   try {
     return await runScript(tmpFile, [], options);
   } finally {
-    // Clean up temporary file
     await fs.promises.unlink(tmpFile);
   }
 };