@lizard-build/cli 4.0.37 → 4.0.38

package/test/unit/login.test.ts

@@ -186,20 +186,17 @@ describe("account state", () => {
 
   test("the closing note leads with sandboxes, aligned in two columns", () => {
     const plain = (t: string) => t.replace(/\x1b\[[0-9;]*m/g, "");
-    expect(plain(nextStepsNote(false)).split("\n")).toEqual([
-      "lizard init             Create a project and link this folder",
-      "lizard sandbox create   Start a sandbox in it",
-      "lizard up               Or deploy this folder",
-    ]);
-    expect(plain(nextStepsNote(true)).split("\n")).toEqual([
-      "lizard sandbox create   Start a sandbox in this project",
+    expect(plain(nextStepsNote()).split("\n")).toEqual([
+      "lizard sandbox create   Start a sandbox",
       "lizard up               Or deploy this folder",
     ]);
   });
 
-  test("an unlinked folder gets lizard init before lizard up", () => {
-    expect(nextSteps(false).map((s) => s.command)).toEqual(["lizard skills get core", "lizard init", "lizard sandbox create", "lizard up"]);
+  test("no lizard init needed: an unlinked folder uses the default project", () => {
+    expect(nextSteps(false).map((s) => s.command)).toEqual(["lizard skills get core", "lizard sandbox create", "lizard up"]);
+    expect(nextSteps(false)[1].why).toContain("default project");
     expect(nextSteps(true).map((s) => s.command)).toEqual(["lizard skills get core", "lizard sandbox create", "lizard up"]);
+    expect(nextSteps(true)[1].why).toContain("linked project");
   });
 });
 
@@ -239,7 +236,7 @@ describe("lizard login --json", () => {
     expect(out).toMatchObject({ status: "complete", username: "ada", email: "ada@example.com", source: "browser" });
     expect(out.account.state).toBe("trial_available");
     expect(out.account.billingUrl).toBe("https://lizard.build/profile/account-billing");
-    expect(out.next.map((s: { command: string }) => s.command)).toContain("lizard init");
+    expect(out.next.map((s: { command: string }) => s.command)).toContain("lizard sandbox create");
     expect(loadConfig().pendingAuth).toBeUndefined();
     expect(loadConfig().credentials?.username).toBe("ada");
   });

package/test/unit/sandbox.test.ts

@@ -3,6 +3,7 @@ import { Command } from "commander";
 import { registerSandbox } from "../../src/commands/sandbox.js";
 import { api, APIError } from "../../src/lib/api.js";
 import { printJSON } from "../../src/lib/format.js";
+import { getProjectLink, resolveProjectId } from "../../src/lib/config.js";
 
 vi.mock("../../src/lib/api.js", () => {
   class APIError extends Error {
@@ -13,8 +14,8 @@ vi.mock("../../src/lib/api.js", () => {
 vi.mock("open", () => ({ default: vi.fn().mockResolvedValue(undefined) }));
 vi.mock("../../src/lib/config.js", () => ({
   resolveProjectId: vi.fn().mockResolvedValue("project-test"),
+  getProjectLink: vi.fn(() => ({ projectId: "project-test" })),
 }));
-vi.mock("../../src/lib/config.js", () => ({ resolveProjectId: vi.fn().mockResolvedValue("project-test") }));
 vi.mock("../../src/lib/format.js", () => ({ isJSONMode: () => true, printJSON: vi.fn() }));
 
 function create(args: string[]) {
@@ -46,6 +47,41 @@ describe("sandbox create timeout", () => {
 });
 
 
+describe("sandbox create without a project", () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+    vi.mocked(api.post).mockResolvedValue({ id: "sandbox-test" });
+  });
+
+  it("an unlinked folder sends no project and leaves the choice to the platform", async () => {
+    vi.mocked(getProjectLink).mockReturnValueOnce(null);
+    await create([]);
+    expect(resolveProjectId).not.toHaveBeenCalled();
+    expect(api.post).toHaveBeenCalledWith("/api/sandboxes", expect.objectContaining({ projectId: undefined }));
+  });
+
+  it("a linked folder or --project still sends its project", async () => {
+    await create([]);
+    expect(api.post).toHaveBeenCalledWith("/api/sandboxes", expect.objectContaining({ projectId: "project-test" }));
+    vi.mocked(getProjectLink).mockReturnValueOnce(null);
+    await create(["--project", "web"]);
+    expect(resolveProjectId).toHaveBeenLastCalledWith("web");
+  });
+
+  it("an unlinked --volume still resolves a project, since a volume name lives in one", async () => {
+    vi.mocked(getProjectLink).mockReturnValueOnce(null);
+    await create(["--volume", "data"]);
+    expect(resolveProjectId).toHaveBeenCalledWith(undefined);
+    expect(api.post).toHaveBeenCalledWith("/api/sandboxes", expect.objectContaining({ projectId: "project-test", volumeName: "data" }));
+  });
+
+  it("a platform that still wants a project gets the old message", async () => {
+    vi.mocked(getProjectLink).mockReturnValueOnce(null);
+    vi.mocked(api.post).mockRejectedValueOnce(new APIError(400, "A project is required", "PROJECT_REQUIRED"));
+    await expect(create([])).rejects.toThrow("No project linked. Run `lizard init` or pass --project <id>.");
+  });
+});
+
 describe("unsupported sandbox commands", () => {
   it.each(["fork", "snapshot-fork", "logs"])(
     "rejects %s without making an API request", async (name) => {