Regression: zip_open Reflection filename OK but named filename rejected (ext/zip/php_zip.stub.php)

[PurHur]

Category

bug · php-src-strict · named arguments · ext/zip · PHP_COMPILER_PROFILE=8.4

Problem

Under PROFILE=8.4, zip_open() Reflection already reports Zend's filename, and positional zip_open("/no/such.zip") returns false, but zip_open(filename: …) still throws Unknown named parameter $filename — classic BuiltinParamNames gap (same class as #23786 get_html_translation_table). Probed 2026-07-29, VM @ acd38db4c.

Repro (PHP_COMPILER_PROFILE=8.4)	Zend / php-src	VM (2026-07-29)
Reflection params	filename	filename
zip_open("/no/such.zip")	false / resource	false
zip_open(filename: "/no/such.zip")	false / resource	Unknown named parameter $filename

php-src reference

• php/php-src ext/zip/php_zip.stub.php — function zip_open(string $filename)

PHP implementation target

• lib/BuiltinParamNames.php — zip_open → ['filename'] (and siblings zip_read / zip_entry_* if same gap)
• ext/zip/*.php only if registration bypasses the map
• PHP-in-PHP only; no new runtime/ C

Repro

./script/docker-exec.sh -- bash -lc 'PHP_COMPILER_PROFILE=8.4 php bin/vm.php -r '\''
$n=[]; foreach ((new ReflectionFunction("zip_open"))->getParameters() as $p) $n[]=$p->getName();
echo "params=", implode(",", $n), "\n";
echo "pos=", var_export(@zip_open("/no/such.zip"), true), "\n";
try { var_export(zip_open(filename: "/no/such.zip")); }
catch (Throwable $e) { echo $e->getMessage(), "\n"; }
'\'''

Done when

• zip_open(filename: …) matches positional semantics
• Unit guard covering named + Reflection
• php-src-strict; no php-compiler-strict shortcut

[PurHur]

claim: PHP Compiler Ext — I/O & Crypto — starting this run

[PurHur]

unclaim: PHP Compiler Ext — I/O & Crypto — zip_open/ZipArchive not registered in current VM profile (function_exists false); pivoting to #27853.

[PurHur]

claim: PHP Compiler Ext — I/O & Crypto — starting this run

Previous claim pivoted because zip is withheld unless host ext/zip or PHP_COMPILER_ENABLE_ZIP=1. Reproducing with that gate, then wiring BuiltinParamNames for zip_open (+ siblings) to php-src php_zip.stub.php names.

[PurHur]

Merged #32169

Verification (Docker, zip enabled — host Zend has no ext/zip so PHP_COMPILER_ENABLE_ZIP=1 is required, #25010):

./script/docker-exec.sh -- bash -lc 'PHP_COMPILER_ENABLE_ZIP=1 php bin/vm.php test/repro/issue_24666_zip_open_named.php'

params=filename
pos=false
named=false
close_params=zip_entry
zip_entry=zip_entry_close(): Argument #1 ($zip_entry) must be of type resource, bool given
zip_ent=Unknown named parameter $zip_ent

zip_open(filename:) already bound via InternalArgInfo; the residual gap was zip_entry_close (zip_ent vs Zend zip_entry) plus $filename TypeError labels on zip_close/zip_read. Named + Reflection guarded in ZipModuleTest / BuiltinParamNamesAliasTest.