refactor: ♻️ remove dotEnvPath from options and worker

microsoft · Dec 13, 2024 · bad8797 · bad8797
1 parent 1977b7e
commit bad8797
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 11 deletions.
diff --git a/packages/cli/src/api.ts b/packages/cli/src/api.ts
@@ -25,14 +25,10 @@ export async function run(
      * GenAIScript generation options.
      */
     options?: Partial<PromptScriptRunOptions> & {
-        /**
-         * Path to the .env file
-         */
-        dotEnvPath?: string
         /**
          * Environment variables to use for the operation.
          */
-        env?: Record<string, string>
+        envVars?: Record<string, string>
         /**
          * The signal to use for aborting the operation. Terminates the worker thread.
          */
@@ -45,10 +41,9 @@ export async function run(
     if (!scriptId) throw new Error("scriptId is required")
     if (typeof files === "string") files = [files]
 
-    const { dotEnvPath, env, signal, ...rest } = options || {}
+    const { envVars, signal, ...rest } = options || {}
     const workerData = {
         type: "run",
-        dotEnvPath,
         scriptId,
         files: files || [],
         options: rest,

diff --git a/packages/cli/src/worker.ts b/packages/cli/src/worker.ts
@@ -3,11 +3,10 @@ import { runScriptInternal } from "./run"
 import { NodeHost } from "./nodehost"
 
 export async function worker() {
-    const { type, dotEnvPath, ...data } = workerData as {
+    const { type, ...data } = workerData as {
         type: string
-        dotEnvPath: string
-    } & object
-    await NodeHost.install(dotEnvPath) // Install NodeHost with environment options
+    }
+    await NodeHost.install(undefined) // Install NodeHost with environment options
     switch (type) {
         case "run": {
             const { scriptId, files, options } = data as {