fix(zig,php parsers): attribute generic-container & anonymous-class methods correctly

[gadievron]

Zig + PHP: attribute generic-container & anonymous-class methods correctly

Fixes the class-key collapse bug family in the Zig and PHP parsers (cross-parser siblings of the Go generic-receiver fix).

---

⚠️ MERGE TARGET — this PR targets staging/parser-fix-stack, NOT master (humans and agents)

This change is part of the parser-fix stack and must merge into the integration branch staging/parser-fix-stack, which already contains every prerequisite it needs. It reaches master later, together with the rest of the stack — not on its own.

Do NOT retarget this at master. master is missing all three prerequisites below — the two parser rewrites AND the grammar pin:

Depends-on: #110   (Zig parser: fix 6 finder bugs — realigns the parser to current tree-sitter-zig node names)
Depends-on: #111   (PHP parser: fix 8 finder bugs — reworks _extract_functions_from_tree)
Depends-on: the tree-sitter-zig>=1.1.2 grammar pin — carried by the stack; absent from master's requirements.txt

On master the fix is inert and its regression tests fail (Zig) / methods aren't extracted (PHP) — see "Why" below. Because the stack carries all three prerequisites, on staging/parser-fix-stack this PR is exactly 4 files and fully green (73 tests pass).

Why the order matters (verify it yourself)

master's Zig parser matches stale tree-sitter node names (VarDecl, container_decl, FnProto); the installed tree-sitter-zig (>=1.1.2) emits variable_declaration / struct_declaration / function_declaration. #110 realigns the parser to the new names. Until then, Zig struct/method extraction is dead, and this fix (written against the new names) cannot work:

git show origin/master:libs/openant-core/parsers/zig/function_extractor.py | grep -c 'struct_declaration'
# -> 0   (the node names this fix depends on don't exist on master; #110 adds them)

PHP similarly needs #111's reworked _extract_functions_from_tree traversal.

Self-guard: the Zig regression test skips with this exact explanation when run on a base that lacks the prerequisite, so an agent/human running the suite on the wrong base sees why instead of a cryptic failure (it does not silently pass — it reports SKIPPED with the reason).

---

Summary

• Zig (parsers/zig/function_extractor.py): methods inside the generic-container idiom fn List(comptime T: type) type { return struct { fn push() ... }; } were emitted as bare top-level functions (class_name=None) and distinct containers' same-named methods collided. Fix: _returns_type() + thread the container fn name as struct context → List.push.
• PHP (parsers/php/function_extractor.py): methods inside new class {} anonymous classes had class_name=None and collided across distinct anon classes (silent overwrite / data loss). Fix: an anonymous_class branch synthesizing a stable class@anonymous:<line>:<col> identity (line+col so multiple anon classes on one line stay distinct).

Tests

5 new (qualified-name + collision regressions, incl. same-line anon classes). On the prerequisite base (staging/parser-fix-stack) the full zig+php suites are green — 73 passed. Independent + judge reviewed.

Related

Companion Go PR (fix(go-parser): key generic-type methods on base type) fixes the same bug family for the Go parser. It is independent — Go uses the stdlib go/ast (stable node types), so it lands on master on its own with no dependency in either direction.

[gadievron]

Heads-up for reviewers/agents: two follow-up fix PRs are stacked on this branch (pr/zig-php-generic-anon) and should merge after #127 (or be rebased onto master once this lands):

• fix(parsers/c): guard macro-alias resolution against cyclic #defines #130 fix(parsers/c): guard macro-alias resolution against cyclic #defines — this branch's C-parser rework still carries the unguarded macro-alias recursion in _resolve_call; fix(parsers/c): guard macro-alias resolution against cyclic #defines #130 adds the cycle guard on top of it.
• fix(parsers/php): resolve $this->/self:: calls into used traits #131 fix(parsers/php): resolve $this->/self:: calls into used traits — builds on the <?php tagless-parse and relative_scope fixes already in this branch, adding only the net-new class→trait composition index.

Both are draft until #127 merges.

[gadievron]

Rebased onto origin/master (post-release #133 / Wave 1). Conflicts resolved by UNION — both this PR's fix and the Wave-1 parser fixes are preserved; no Wave-1 change reverted (independently judge-verified). Force-pushed with --force-with-lease.

Base retargeted staging→master. PHP+Zig tests green (45). Clean rebase — master already carries the #110/#111 prerequisites.

[gadievron]

Closing as already resolved on current master. This fix's behavior shipped via the parser-fix-stack release (#133/#134); confirmed by independent reproduction — the original bug (reconstructed from its bug-pipeline investigation) no longer manifests on master, not merely by line-presence. Part of a full validated sweep of the open-PR corpus.