Skip to content

Commit 28822b2

Browse files
ellismgCopilot
andcommitted
Run sandbox bypass E2E where supported
Use the macOS sandbox backend available in CI and prove the approved bypass by observing the denied-path result in the successful tool completion. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
1 parent 54a6ded commit 28822b2

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

nodejs/test/e2e/sandbox_bypass.e2e.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@ const TEST_TIMEOUT_MS = 180_000;
1313
const TEST_NAME = "approves a blocked search and executes it outside the sandbox";
1414

1515
describe("Sandbox bypass", async () => {
16-
if (process.platform === "win32") {
17-
// The Windows backend requires BaseContainer, which is unavailable on the SDK's runners.
16+
if (process.platform !== "darwin") {
17+
// SDK runners provide a sandbox backend only on macOS (no bwrap/BaseContainer elsewhere).
1818
it.skip(TEST_NAME, () => undefined);
1919
return;
2020
}
@@ -54,7 +54,7 @@ describe("Sandbox bypass", async () => {
5454
event.type === "tool.execution_complete" &&
5555
event.data.toolName === "grep" &&
5656
event.data.success &&
57-
event.data.sandboxed === false
57+
event.data.result?.content.includes("OUTSIDE_MATCH_LINE bypass-approved")
5858
) {
5959
bypassedSearchCompleted = true;
6060
}

0 commit comments

Comments
 (0)