[alexwrite]

Bug / Context

Closes #17. PHP lets any keyword be a class constant name (public const string DEFAULT = ...). tree-sitter-php up to 0.25.1 rejects 61 keywords once the constant is typed (PHP 8.3) and 7 even untyped (VOID, TRUE, ...). Depending on error recovery, the whole class declaration becomes an ERROR node: the class vanishes from the graph and its importers land on a sourceless stub.

Typed constants themselves were never the problem: the grammar supports them since tree-sitter/tree-sitter-php#205. The keyword bug is tree-sitter/tree-sitter-php#295, fixed by the open upstream PR tree-sitter/tree-sitter-php#307 (CI green, awaiting review, not released).

Fix

• pyproject.toml: tree-sitter-php pinned to commit 9678351 of alexwrite/tree-sitter-php, branch fix/keyword-constant-names = the Lack of NoneType handling in serve.py - causes AttributeError: 'NoneType' object has no attribute 'lower' Graphify-Labs/graphify#307 commits (author kept) + one corpus test for typed keyword-named constants (fails on upstream master, passes on Lack of NoneType handling in serve.py - causes AttributeError: 'NoneType' object has no attribute 'lower' Graphify-Labs/graphify#307; their suite: 157/157).
• uv.lock: only the two tree-sitter-php hunks (the rest of what uv lock rewrites is pre-existing drift, uv lock --check already fails on main).
• tests/test_php_keyword_constant_names.py: 7 keyword declarations parse without parse_errors; a constants-only class keeps its node and its references_constant fan-in.

pip builds the binding from the git URL (C compiler required, no subdirectory), checked in a clean venv with the same pip install "... @ git+..." form the daily provisioning uses.

Quality

• Regression test: 8 failed with tree-sitter-php 0.24.1, 8 passed with the pin
• Full suite: 7536 passed, 14 skipped, 0 failed (main baseline 7528 + the 8 new tests)
• ruff: All checks passed; pyright on the new file: 0 errors

Measure (private projects, code-only, before/after, exhaustive node/edge diff)

• Project A: syntax-error warnings 1 -> 0; the missing class is back (1 node, contains + 8 imports moved off a sourceless stub); one constructor location corrected. Nothing else changed.
• Project B: identical graph (0 node, 0 edge difference).

Plan post-merge

• Drop the pin back to a PyPI range once tree-sitter-php ships Lack of NoneType handling in serve.py - causes AttributeError: 'NoneType' object has no attribute 'lower' Graphify-Labs/graphify#307 (0.25.2 or later).
• Follow-up found while testing: PHP : un accès Classe::CONSTANTE vers un autre fichier ne crée aucune arête sans autre mention de la classe #28 (cross-file Foo::BAR edge needs another mention of Foo).

[github-actions]

Thanks for the pull request, @alexwrite. A maintainer will review it soon.

Want to talk it through while it is in review? Come join us on our Discord server. For longer-form discussion there is also GitHub Discussions.

A couple of things that speed up review: make sure the test suite passes on Python 3.10 and 3.13, and that the change keeps extraction deterministic.