Stop SIGPIPE from failing the macOS PHP verification step

[wojtekn]

Related issues

• Follow-up to Use the runner's preinstalled PHP to run static-php-cli on macOS #4889
• Unblocks the SOAP binary rebuild for Ship the SOAP extension with the native PHP runtime #4888

How AI was used in this PR

Claude Code diagnosed the failure and wrote the fix. I reviewed the diff.

Proposed Changes

The Verify preinstalled PHP for static-php-cli step added in #4889 fails on macos-x86_64 with:

Required PHP extension missing: mbstring.

That message is wrong — mbstring is present on the runner image. The check itself was broken.

php -m | grep -qix mbstring lets grep exit the moment it matches, which closes the pipe while php -m is still writing. PHP is killed by SIGPIPE and exits 255. Because the step runs under set -o pipefail, the pipeline inherits that failure even though grep succeeded. Reproduced locally on a PHP that definitely has mbstring:

PIPESTATUS=255 0    # php killed by SIGPIPE, grep matched fine

It tripped on mbstring and not zlib purely because of ordering: mbstring appears near the top of php -m and zlib near the bottom, so only the mbstring match leaves enough unwritten output to trigger SIGPIPE. That asymmetry is what made it read like a genuine missing-extension result.

The module list is now captured once and matched against in the loop, so nothing is piped out of php and there is no early-closed pipe.

Testing Instructions

Verified locally by extracting the step from the workflow and running it under the same shell GitHub uses (/bin/bash --noprofile --norc -e -o pipefail):

• with the real mbstring zlib list → exits 0 and prints php -v / composer --version
• with a bogus extension name → still exits 1 with Required PHP extension missing: …, so the check hasn't been defanged

End to end, dispatch Build PHP CLI Binaries from this branch with apps_cdn_visibility: none:

gh workflow run build-php-cli-binaries.yml --ref fix-php-verify-sigpipe \
  -f php_version=8.4.25 -f package_version=studio-2 \
  -f xdebug_version=3.5.3 -f apps_cdn_visibility=none

Expect all five targets green.

Pre-merge Checklist

• Have you checked for TypeScript, React or other console errors?

🤖 Generated with Claude Code