fix(sdk): use nameOrRequestOptions in envvars.update outside a task

The 4-arg management form assigned $name from a binding that is not a
parameter. TypeScript accepted it via the DOM global. Node throws
ReferenceError before any request.

Repro: tsx --eval update("proj_x","dev","FOO",{value:"bar"}) on 0a23814.

Guard the string, assign nameOrRequestOptions, add envvars.test.ts.
Tests failed with ReferenceError without the assignment, 2 passed after.

Author: claxman

.changeset/fix-envvars-update-name.md

@@ -0,0 +1,5 @@
+---
+"@trigger.dev/sdk": patch
+---
+
+`envvars.update(projectRef, slug, name, params)` no longer throws `ReferenceError: name is not defined` when called from a Node script outside a task run.

packages/trigger-sdk/src/v3/envvars.test.ts

@@ -0,0 +1,84 @@
+import { createServer, type IncomingMessage, type Server, type ServerResponse } from "node:http";
+import type { AddressInfo } from "node:net";
+import { apiClientManager, taskContext } from "@trigger.dev/core/v3";
+import { afterEach, beforeEach, describe, expect, it } from "vitest";
+import { update } from "./envvars.js";
+
+type ReceivedRequest = {
+  method: string;
+  url: string;
+  authorization?: string;
+  body: unknown;
+};
+
+describe("envvars.update outside a task", () => {
+  let server: Server;
+  let baseUrl: string;
+  let requests: ReceivedRequest[];
+
+  beforeEach(async () => {
+    requests = [];
+    apiClientManager.disable();
+    taskContext.disable();
+    server = createServer((request, response) => {
+      void handleRequest(request, response, requests);
+    });
+    await new Promise<void>((resolve) => {
+      server.listen(0, "127.0.0.1", () => {
+        const address = server.address() as AddressInfo;
+        baseUrl = `http://127.0.0.1:${address.port}`;
+        resolve();
+      });
+    });
+  });
+
+  afterEach(async () => {
+    await new Promise<void>((resolve) => server.close(() => resolve()));
+    apiClientManager.disable();
+    taskContext.disable();
+  });
+
+  it("PUTs the 4-arg form to /projects/{ref}/envvars/{slug}/{name}", async () => {
+    const key = "tr_dev_sk_0123456789abcdefghijklmn";
+    await apiClientManager.runWithConfig({ baseURL: baseUrl, accessToken: key }, () =>
+      update("proj_x", "dev", "FOO", { value: "bar" })
+    );
+
+    expect(requests).toEqual([
+      {
+        method: "PUT",
+        url: "/api/v1/projects/proj_x/envvars/dev/FOO",
+        authorization: `Bearer ${key}`,
+        body: { value: "bar" },
+      },
+    ]);
+  });
+
+  it("throws name is required when the 4-arg name is missing", () => {
+    expect(() => update("proj_x", "dev", undefined as unknown as string, { value: "bar" })).toThrow(
+      "name is required"
+    );
+    expect(requests).toEqual([]);
+  });
+});
+
+async function handleRequest(
+  request: IncomingMessage,
+  response: ServerResponse,
+  requests: ReceivedRequest[]
+) {
+  const chunks: Buffer[] = [];
+  for await (const chunk of request) {
+    chunks.push(Buffer.from(chunk));
+  }
+  const rawBody = Buffer.concat(chunks).toString();
+  requests.push({
+    method: request.method ?? "",
+    url: request.url ?? "",
+    authorization: request.headers.authorization,
+    body: rawBody ? JSON.parse(rawBody) : undefined,
+  });
+
+  response.writeHead(200, { "content-type": "application/json" });
+  response.end(JSON.stringify({ success: true }));
+}

packages/trigger-sdk/src/v3/envvars.ts

@@ -332,13 +332,17 @@ export function update(
       throw new Error("projectRef is required");
     }
 
+    if (typeof nameOrRequestOptions !== "string") {
+      throw new Error("name is required");
+    }
+
     if (!params) {
       throw new Error("params is required");
     }
 
     $projectRef = projectRefOrName;
     $slug = slugOrParams;
-    $name = name!;
+    $name = nameOrRequestOptions;
     $params = params;
   }