fix: extract container name when docker call is wrapped in shell logic

[ChiragAgg5k]

Problem

extractContainerName() only recognized the docker call when it appeared at the start of the command (after skipping leading KEY=value tokens). When a workflow wraps the docker call in shell logic — variable assignments, conditionals, && chains — the parser hit a non-docker token first and returned null:

command: |
  SERVICE_PATH="/usr/src/code/tests/e2e/Services/Account"
  FLAG=""
  if [ "${{ matrix.functional }}" = "true" ]; then FLAG="--functional"; fi
  docker compose exec -T -e _APP_VAR="x" appwrite vendor/bin/paratest "$SERVICE_PATH" --log-junit ...

This produced:

Warning: Could not extract container name from command, JUnit XML extraction skipped
Warning: Tests failed but no specific failures in JUnit XML

JUnit extraction was skipped, so per-test failure detection (and therefore targeted retry) silently degraded to retrying the whole command — even though the command is a valid docker command.

Root cause: internal inconsistency

The action already detects the docker pattern anywhere in two places, but extractContainerName was the odd one out:

Function	Detection	Wrapped command
isDockerCommand / isDockerCompose	command.includes('docker compose exec')	✅
extractJUnitPath	command.match(/--log-junit\s+(\S+)/)	✅
extractContainerName	positional, must be at start	❌

So isDockerCommand returns true (action takes the docker path) but extractContainerName returns null — the extraction is skipped.

Fix

Scan the token stream for the docker exec / docker compose exec / docker-compose exec segment anywhere, then keep the existing flag-skipping logic from there. This matches the substring-based detection used elsewhere, and the leading KEY=value skip is now subsumed by the scan (existing env-var tests still pass unchanged).

Tests

• All existing extractContainerName tests pass unchanged (including the leading-env-var cases).
• Added regression tests for a shell-wrapped command and an &&-chained command.
• bun test tests/unit/ → 119 pass / 0 fail. bun run check (prettier + tsc) clean.
• dist/index.js rebuilt via bun run build.

No runtime/dependency changes; only the parser logic and the rebuilt bundle.