Skip to content

Commit 31c8a30

Browse files
committed
test(sandbox): align cwd rebase test with v3 contract
1 parent 6c1ac27 commit 31c8a30

1 file changed

Lines changed: 3 additions & 15 deletions

File tree

packages/sandbox/daemon/routes/dispatch.test.ts

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -301,22 +301,17 @@ describe("POST /_sandbox/dispatch", () => {
301301
await reader.cancel();
302302
});
303303

304-
it("rebases symbolic workspace.cwd and codingWorkspace.cwd onto the daemon's sandbox root before the harness sees it", async () => {
304+
it("rebases symbolic workspace.cwd onto the daemon's sandbox root before the harness sees it", async () => {
305305
// The wire carries the symbolic value "/repo"; the daemon must rebase it
306306
// onto its own sandbox root (daemonAppRoot()) before handing the input to
307307
// the harness. The harness MUST receive the rebased absolute path, not the
308308
// wire symbol — so `effectiveCwd(input.workspace.cwd)` yields a real path.
309309
const runId = "run-cwd-rebase";
310-
let capturedInput:
311-
| { workspace?: { cwd: string }; codingWorkspace?: { cwd?: string } }
312-
| undefined;
310+
let capturedInput: { workspace?: { cwd: string | null } } | undefined;
313311

314312
const deps = makeDeps({
315313
lookupHarness: (_id, input) => {
316-
capturedInput = input as {
317-
workspace?: { cwd: string };
318-
codingWorkspace?: { cwd?: string };
319-
};
314+
capturedInput = input as { workspace?: { cwd: string | null } };
320315
return makeFakeHarness();
321316
},
322317
});
@@ -336,12 +331,6 @@ describe("POST /_sandbox/dispatch", () => {
336331
},
337332
branch: "main",
338333
},
339-
codingWorkspace: {
340-
repo: { owner: "deco", name: "site", connectedGithub: true },
341-
branch: "main",
342-
cwd: "/repo",
343-
workspaceKind: "github",
344-
},
345334
},
346335
});
347336
const res = await handleDispatchRequest(authedDispatch(body), deps);
@@ -356,7 +345,6 @@ describe("POST /_sandbox/dispatch", () => {
356345
expect(rebasedCwd).not.toBe("/repo");
357346
// Must end with /repo (rebased to the daemon's sandbox root)
358347
expect(rebasedCwd?.endsWith("/repo")).toBe(true);
359-
expect(capturedInput?.codingWorkspace?.cwd).toBe(rebasedCwd);
360348
});
361349

362350
it("passes null workspace.cwd through to the harness", async () => {

0 commit comments

Comments
 (0)