From 992ae37aa9c002ef8bb1b3ecfbc361a26f7685ba Mon Sep 17 00:00:00 2001 From: Lily Shen <115414357+lilyshen0722@users.noreply.github.com> Date: Sun, 25 Oct 2026 15:02:37 -0700 Subject: [PATCH] ci: the egress job copies packages/seats, which cli has linked since #2739 (TASK-295) #2739 made cli depend on file:../packages/seats, and opencode.js loads it through runtime-observation.js. The job copied cli, commonly-mcp and scripts into /work but not packages/seats, so the link had no target and scripts/verify-opencode-bwrap-mcp.mjs died with ERR_MODULE_NOT_FOUND after both jest suites had passed. #2746 was the first PR to trigger the job since #2739 (run 40237115248). packages/seats/** also becomes a trigger path, since a change there now reaches the spawn path this job exercises. Co-Authored-By: Claude Opus 5.5 Co-authored-by: wren Agent-Model: claude/claude-opus-5-5/max --- .github/workflows/opencode-bwrap-egress.yml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.github/workflows/opencode-bwrap-egress.yml b/.github/workflows/opencode-bwrap-egress.yml index 06ed235d94..d477299a79 100644 --- a/.github/workflows/opencode-bwrap-egress.yml +++ b/.github/workflows/opencode-bwrap-egress.yml @@ -9,6 +9,7 @@ on: - 'cli/src/lib/sandbox/bwrap.js' - 'cli/src/lib/sandbox/opencode-loopback.mjs' - 'commonly-mcp/**' + - 'packages/seats/**' # opencode.js loads it (cli depends on file:../packages/seats) - 'cli/__tests__/opencode.bwrap-egress.test.mjs' - 'cli/__tests__/instance-proxy.test.mjs' - 'scripts/fixtures/opencode-mcp-probe.mjs' @@ -32,8 +33,10 @@ jobs: set -euo pipefail apt-get update -qq apt-get install -y -qq bubblewrap iproute2 >/dev/null - mkdir -p /work + mkdir -p /work/packages cp -R /source/cli /source/commonly-mcp /source/scripts /work/ + # cli links file:../packages/seats, so npm ci needs the link target. + cp -R /source/packages/seats /work/packages/ rm -rf /work/cli/node_modules /work/commonly-mcp/node_modules npm ci --prefix /work/cli --silent npm ci --prefix /work/commonly-mcp --silent