Skip to content

Commit c95e735

Browse files
authored
Fix subprocess isolation install step never running (#1148)
env context isn't available in composite-action if: conditions. Move opt-out check into run: body. :house: Remote-Dev: homespace
1 parent 58dbe8e commit c95e735

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

action.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -198,10 +198,14 @@ runs:
198198
- name: Install subprocess isolation dependencies
199199
# Install subprocess isolation dependencies when processing content from non-write users.
200200
# Best-effort: skips on non-Linux or when sudo/apt unavailable (self-hosted runners).
201-
if: ${{ inputs.allowed_non_write_users != '' && env.CLAUDE_CODE_SUBPROCESS_ENV_SCRUB != '0' && runner.os == 'Linux' }}
201+
if: ${{ inputs.allowed_non_write_users != '' && runner.os == 'Linux' }}
202202
continue-on-error: true
203203
shell: bash
204204
run: |
205+
if [ "${CLAUDE_CODE_SUBPROCESS_ENV_SCRUB:-}" = "0" ]; then
206+
echo "Subprocess isolation opted out via CLAUDE_CODE_SUBPROCESS_ENV_SCRUB=0"
207+
exit 0
208+
fi
205209
if command -v apt-get >/dev/null && command -v sudo >/dev/null; then
206210
for i in 1 2 3; do
207211
sudo apt-get update -qq && sudo apt-get install -y --no-install-recommends bubblewrap socat && break

0 commit comments

Comments
 (0)