Skip to content

test(codex-cloud): budget the shell-policy case for Windows Git Bash spawns#2255

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
BigSimmo merged 2 commits into from
Aug 21, 2026
+8 −1
Merged

test(codex-cloud): budget the shell-policy case for Windows Git Bash spawns #2255

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
c4851ee
test(codex-cloud): budget the shell-policy case for Windows Git Bash …
claude Aug 21, 2026
c962da7
Merge branch 'main' into claude/clever-edison-167y82
BigSimmo Aug 21, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion tests/codex-cloud-setup.test.ts
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -788,7 +788,14 @@ describe("Codex Cloud environment contract", () => {
expect(rewritten).toContain("[mcp_servers.example]");
expect(rewritten.match(/^\[shell_environment_policy\]$/gm)).toHaveLength(1);
expect(rewritten.match(/BEGIN clinical-kb-codex-cloud shell policy/g)).toHaveLength(1);
});
// Two full `bash scripts/setup-codex-cloud.sh` runs. That is cheap on Linux
// and is not on Windows, where each spawn goes through Git Bash: measured at
// 24.96s on a Windows workstation running this file ALONE — 83% of the 30s
// default in vitest.config.mts. Under a full `npm run test` the four workers
// contend and it tips over, which is how this failed there while passing in
// isolation and passing everywhere on Linux. The sibling case below already
// carries this budget for the same reason; this one was missed.
}, 120_000);

it("pins connected retrieval mode and rejects unsafe shell-policy configs", () => {
const connectedHome = temporaryDirectory("codex-cloud-connected-");
Expand Down
Loading