@vercel/sandbox 0.0.14 → 0.0.15

package/.turbo/turbo-build.log

@@ -1,4 +1,4 @@
 
-> @vercel/sandbox@0.0.14 build /home/runner/work/sandbox-sdk/sandbox-sdk/packages/sandbox
+> @vercel/sandbox@0.0.15 build /home/runner/work/sandbox-sdk/sandbox-sdk/packages/sandbox
 > tsc
 

package/.turbo/turbo-typecheck.log

@@ -1,4 +1,4 @@
 
-> @vercel/sandbox@0.0.14 typecheck /home/runner/work/sandbox-sdk/sandbox-sdk/packages/sandbox
+> @vercel/sandbox@0.0.15 typecheck /home/runner/work/sandbox-sdk/sandbox-sdk/packages/sandbox
 > tsc --noEmit
 

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # @vercel/sandbox
 
+## 0.0.15
+
+### Patch Changes
+
+- Remove warning when consuming logs more than once ([#91](https://github.com/vercel/sandbox-sdk/pull/91))
+
+- Improve future compatibility of runtime parameter ([#88](https://github.com/vercel/sandbox-sdk/pull/88))
+
 ## 0.0.14
 
 ### Patch Changes

package/dist/api-client/api-client.d.ts

@@ -53,7 +53,7 @@ export declare class APIClient extends BaseClient {
         resources?: {
             vcpus: number;
         };
-        runtime?: "node22" | "python3.13";
+        runtime?: "node22" | "python3.13" | (string & {});
     }): Promise<Parsed<{
         sandbox: {
             region: string;

package/dist/api-client/api-client.js

@@ -8,20 +8,10 @@ const base_client_1 = require("./base-client");
 const validators_1 = require("./validators");
 const api_error_1 = require("./api-error");
 const file_writer_1 = require("./file-writer");
-const lru_cache_1 = require("lru-cache");
 const version_1 = require("../version");
 const consume_readable_1 = require("../utils/consume-readable");
 const jsonlines_1 = __importDefault(require("jsonlines"));
 const os_1 = __importDefault(require("os"));
-const ms_1 = __importDefault(require("ms"));
-/**
- * Allows to track the logs hits for a command un a to maximum of items and
- * TTL so that we don't incur in memory leaks in a log running process.
- */
-const logHits = new lru_cache_1.LRUCache({
-    ttl: (0, ms_1.default)("45m"),
-    max: 1000,
-});
 class APIClient extends base_client_1.BaseClient {
     constructor(params) {
         super({
@@ -128,18 +118,6 @@ class APIClient extends base_client_1.BaseClient {
                 message: "Expected a stream of logs",
             });
         }
-        /**
-         * Currently, once we consume logs in the backend we cannot read them
-         * again. This logic writes a warning when the endpoint for a command
-         * logs is consumed more than once to alert the user about this.
-         *
-         * This is a temporary solution, we should be able to handle this in
-         * the backend in the future.
-         */
-        if (logHits.get(url)) {
-            console.warn(`Multiple consumers for logs of command \`${params.cmdId}\`. This may lead to unexpected behavior.`);
-        }
-        logHits.set(url, true);
         for await (const chunk of response.body.pipe(jsonlines_1.default.parse())) {
             yield validators_1.LogLine.parse(chunk);
         }

package/dist/sandbox.d.ts

@@ -51,7 +51,7 @@ export interface CreateSandboxParams {
      * The runtime of the sandbox, currently only `node22` and `python3.13` are supported.
      * If not specified, the default runtime `node22` will be used.
      */
-    runtime?: "node22" | "python3.13";
+    runtime?: "node22" | "python3.13" | (string & {});
 }
 /** @inline */
 interface GetSandboxParams {

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const VERSION = "0.0.14";
+export declare const VERSION = "0.0.15";

package/dist/version.js

@@ -2,4 +2,4 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.VERSION = void 0;
 // Autogenerated by inject-version.ts
-exports.VERSION = "0.0.14";
+exports.VERSION = "0.0.15";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vercel/sandbox",
-  "version": "0.0.14",
+  "version": "0.0.15",
   "description": "",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -13,7 +13,6 @@
     "async-retry": "1.3.3",
     "form-data": "3.0.0",
     "jsonlines": "0.1.1",
-    "lru-cache": "11.1.0",
     "ms": "2.1.3",
     "node-fetch": "2.6.11",
     "tar-stream": "3.1.7",

package/src/api-client/api-client.ts

@@ -14,22 +14,11 @@ import {
 } from "./validators";
 import { APIError } from "./api-error";
 import { FileWriter } from "./file-writer";
-import { LRUCache } from "lru-cache";
 import { VERSION } from "../version";
 import { consumeReadable } from "../utils/consume-readable";
 import { z } from "zod";
 import jsonlines from "jsonlines";
 import os from "os";
-import ms from "ms";
-
-/**
- * Allows to track the logs hits for a command un a to maximum of items and
- * TTL so that we don't incur in memory leaks in a log running process.
- */
-const logHits = new LRUCache<string, boolean>({
-  ttl: ms("45m"),
-  max: 1000,
-});
 
 export class APIClient extends BaseClient {
   private teamId: string;
@@ -78,7 +67,7 @@ export class APIClient extends BaseClient {
       | { type: "tarball"; url: string };
     timeout?: number;
     resources?: { vcpus: number };
-    runtime?: "node22" | "python3.13";
+    runtime?: "node22" | "python3.13" | (string & {});
   }) {
     return parseOrThrow(
       SandboxAndRoutesResponse,
@@ -239,21 +228,6 @@ export class APIClient extends BaseClient {
       });
     }
 
-    /**
-     * Currently, once we consume logs in the backend we cannot read them
-     * again. This logic writes a warning when the endpoint for a command
-     * logs is consumed more than once to alert the user about this.
-     *
-     * This is a temporary solution, we should be able to handle this in
-     * the backend in the future.
-     */
-    if (logHits.get(url)) {
-      console.warn(
-        `Multiple consumers for logs of command \`${params.cmdId}\`. This may lead to unexpected behavior.`,
-      );
-    }
-
-    logHits.set(url, true);
     for await (const chunk of response.body.pipe(jsonlines.parse())) {
       yield LogLine.parse(chunk);
     }

package/src/command.test.ts

@@ -11,8 +11,7 @@ afterEach(async () => {
   await sandbox.stop();
 });
 
-it("warns when there is more than one logs consumer", async () => {
-  const warnSpy = vi.spyOn(console, "warn").mockImplementation(() => {});
+it("supports more than one logs consumer", async () => {
   const stdoutSpy = vi
     .spyOn(process.stdout, "write")
     .mockImplementation(() => true);
@@ -23,13 +22,8 @@ it("warns when there is more than one logs consumer", async () => {
     stdout: process.stdout,
   });
 
-  expect(await cmd.stdout()).toEqual("");
+  expect(await cmd.stdout()).toEqual("Hello World!\n");
   expect(stdoutSpy).toHaveBeenCalledWith("Hello World!\n");
-  expect(warnSpy).toHaveBeenCalledWith(
-    expect.stringMatching(
-      /Multiple consumers for logs of command `[^`]+`\.\sThis may lead to unexpected behavior\./,
-    ),
-  );
 });
 
 it("does not warn when there is only one logs consumer", async () => {
@@ -85,9 +79,12 @@ it("can execute commands with sudo", async () => {
   expect(output).toContain("USER=root\n");
   expect(output).toContain("SUDO_USER=vercel-sandbox\n");
 
-  // The actual path contains more, but this is enough
-  // to verify that we do not reset it to a default path.
-  expect(output).toContain("PATH=/vercel/bin");
+  const pathLine = output.split("\n").find((line) => line.startsWith("PATH="));
+  expect(pathLine).toBeDefined();
+
+  const pathSegments = pathLine!.slice(5).split(":");
+  expect(pathSegments).toContain("/vercel/bin");
+  expect(pathSegments).toContain("/vercel/runtimes/node22/bin");
 
   const dnf = await sandbox.runCommand({
     cmd: "dnf",
@@ -98,5 +95,5 @@ it("can execute commands with sudo", async () => {
   expect(dnf.exitCode).toBe(0);
 
   const which = await sandbox.runCommand("which", ["go"]);
-  expect(which.output()).resolves.toContain("/usr/bin/go");
+  expect(await which.output()).toContain("/usr/bin/go");
 });

package/src/sandbox.ts

@@ -51,7 +51,7 @@ export interface CreateSandboxParams {
    * The runtime of the sandbox, currently only `node22` and `python3.13` are supported.
    * If not specified, the default runtime `node22` will be used.
    */
-  runtime?: "node22" | "python3.13";
+  runtime?: "node22" | "python3.13" | (string & {});
 }
 
 /** @inline */

package/src/version.ts

@@ -1,2 +1,2 @@
 // Autogenerated by inject-version.ts
-export const VERSION = "0.0.14";
+export const VERSION = "0.0.15";