KarpelesLab / goro / 26095678956

12%

push

github

vm: lower \$arr[\$k1][\$k2] ?? default coalesce LHS natively

The coalesce \`??\` LHS used to AST-delegate the whole expression
when the LHS was an array-access chain. emit_expr now uses the
same permissive chain reads as nested isset
(emitIssetContainerRead + OP_ARRAY_GET_SAFE) — they return null on
missing intermediate / final keys without warning, which is exactly
what \`??\` needs.

Only array-access chains whose innermost container is a simple
variable are lowered (matching emitIssetContainerRead's supported
shapes). Object props, nullsafe chains, and other LHS shapes still
AST-delegate.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

4 of 20 new or added lines in 1 file covered. (20.0%)

14496 of 117566 relevant lines covered (12.33%)

2553.1 hits per line