Skip to content

Commit 55f14b6

Browse files
committed
fix(ci): verify managed-settings.json without runner-shell quote stripping
The verify-command added in #99 embedded literal " around the expected path inside a single-quoted jq filter. After GHA expansion into bash -c "...", the inner " terminated and reopened the outer "..." rather than reaching jq as a string delimiter, so jq received .hooks.SessionStart[0].hooks[0].command == /usr/local/bin/patch-playwright-mcp and failed with "unexpected '/'". Switch to the printenv | grep -qx idiom used since #93: jq -r prints the bare string value, grep -qx asserts an exact match. No literal " inside the matrix value, so the value survives both the YAML decode and the bash -c wrapping unchanged.
1 parent 65ef170 commit 55f14b6

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

‎.github/workflows/build-claude-code.yml‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -77,19 +77,19 @@ jobs:
7777
matrix:
7878
include:
7979
- image-suffix: claude-code
80-
verify-command: "bun --version || true && claude --version && mise --version && fish --version && rtk --version && ralphex --version && test -x /usr/local/bin/patch-playwright-mcp && jq -e '.hooks.SessionStart[0].hooks[0].command == \"/usr/local/bin/patch-playwright-mcp\"' /etc/claude-code/managed-settings.json >/dev/null && printenv AGENT_BROWSER_EXECUTABLE_PATH | grep -qx /usr/bin/chromium"
80+
verify-command: "bun --version || true && claude --version && mise --version && fish --version && rtk --version && ralphex --version && test -x /usr/local/bin/patch-playwright-mcp && jq -r '.hooks.SessionStart[0].hooks[0].command' /etc/claude-code/managed-settings.json | grep -qx /usr/local/bin/patch-playwright-mcp && printenv AGENT_BROWSER_EXECUTABLE_PATH | grep -qx /usr/bin/chromium"
8181
runner: ubuntu-24.04
8282
arch: amd64
8383
- image-suffix: claude-code
84-
verify-command: "bun --version || true && claude --version && mise --version && fish --version && rtk --version && ralphex --version && test -x /usr/local/bin/patch-playwright-mcp && jq -e '.hooks.SessionStart[0].hooks[0].command == \"/usr/local/bin/patch-playwright-mcp\"' /etc/claude-code/managed-settings.json >/dev/null && printenv AGENT_BROWSER_EXECUTABLE_PATH | grep -qx /usr/bin/chromium"
84+
verify-command: "bun --version || true && claude --version && mise --version && fish --version && rtk --version && ralphex --version && test -x /usr/local/bin/patch-playwright-mcp && jq -r '.hooks.SessionStart[0].hooks[0].command' /etc/claude-code/managed-settings.json | grep -qx /usr/local/bin/patch-playwright-mcp && printenv AGENT_BROWSER_EXECUTABLE_PATH | grep -qx /usr/bin/chromium"
8585
runner: ubuntu-24.04-arm
8686
arch: arm64
8787
- image-suffix: claude-code-sandbox
88-
verify-command: "claude --version && mise --version && fish --version && which iptables && rtk --version && ralphex --version && test -x /usr/local/bin/patch-playwright-mcp && jq -e '.hooks.SessionStart[0].hooks[0].command == \"/usr/local/bin/patch-playwright-mcp\"' /etc/claude-code/managed-settings.json >/dev/null"
88+
verify-command: "claude --version && mise --version && fish --version && which iptables && rtk --version && ralphex --version && test -x /usr/local/bin/patch-playwright-mcp && jq -r '.hooks.SessionStart[0].hooks[0].command' /etc/claude-code/managed-settings.json | grep -qx /usr/local/bin/patch-playwright-mcp"
8989
runner: ubuntu-24.04
9090
arch: amd64
9191
- image-suffix: claude-code-sandbox
92-
verify-command: "claude --version && mise --version && fish --version && which iptables && rtk --version && ralphex --version && test -x /usr/local/bin/patch-playwright-mcp && jq -e '.hooks.SessionStart[0].hooks[0].command == \"/usr/local/bin/patch-playwright-mcp\"' /etc/claude-code/managed-settings.json >/dev/null"
92+
verify-command: "claude --version && mise --version && fish --version && which iptables && rtk --version && ralphex --version && test -x /usr/local/bin/patch-playwright-mcp && jq -r '.hooks.SessionStart[0].hooks[0].command' /etc/claude-code/managed-settings.json | grep -qx /usr/local/bin/patch-playwright-mcp"
9393
runner: ubuntu-24.04-arm
9494
arch: arm64
9595
runs-on: ${{ matrix.runner }}

0 commit comments

Comments
 (0)