PHP : les constantes de classe typées disparaissent du graphe #17

Activity

1. alexwrite

   added

   bugSomething isn't workingSomething isn't working

   p2Défaut visible par l'utilisateurDéfaut visible par l'utilisateur

   on Oct 11, 2026
2. github-actions commented on Oct 11, 2026

   

   github-actionsBot

   on Oct 11, 2026 – with GitHub Actions

   More actions

   Thanks for opening this issue, @alexwrite. A maintainer will take a look soon.

   If you would like to discuss it in real time, come say hi on our Discord server. For longer-form questions and ideas there is also GitHub Discussions.

   To help us triage, please make sure the report includes what you expected, what actually happened, and the steps (and a small sample) to reproduce it.
3. alexwrite commented on Oct 11, 2026

   

   alexwrite

   on Oct 11, 2026

   OwnerAuthor

   More actions

   Prise en charge.

   Le diagnostic initial est à corriger, constaté par test isolé (tree-sitter-php 0.24.1) :

   Affirmation de l'issue	Constat
   La grammaire ne connaît pas le type des constantes	Faux : supporté depuis tree-sitter/tree-sitter-php#205 (2024). public const string X, ?string, union, classe, enum, interface, trait : 0 erreur
   Toute constante typée est absente du graphe	Vrai, mais aussi pour les non typées : Graphify ne crée pas de nœud de constante PHP, un accès Foo::BAR devient une arête references_constant vers la classe. Choix de conception, hors de cette issue
   427 et 157 fichiers touchés	Ce sont les fichiers qui contiennent une constante typée, pas ceux en erreur

   Vrai défaut : une constante de classe dont le nom est un mot réservé (DEFAULT, PRIVATE, CLASS, NEW...). 61 mots cassent quand la constante est typée, 7 même sans type (VOID, TRUE...). Selon la récupération d'erreur, la classe entière devient un nœud ERROR et disparaît du graphe. Projet A : 2 fichiers en erreur sur 2 943, dont une classe perdue. Projet B : 0 sur 476.

   Amont : tree-sitter/tree-sitter-php#295 (issue) et Graphify-Labs#307 (PR ouverte le 2026-10-08, CI verte, en attente de relecture) corrigent exactement ce cas. Pas de correctif maison ni de PR amont en doublon.

   Périmètre de fichiers visé :

   • pyproject.toml, uv.lock : tree-sitter-php figé sur un commit du fork alexwrite/tree-sitter-php (commits de Lack of NoneType handling in serve.py - causes AttributeError: 'NoneType' object has no attribute 'lower' Graphify-Labs/graphify#307, auteur conservé, plus un test de corpus)
   • tests/test_php_keyword_constant_names.py : test de régression
4. alexwrite commented on Oct 11, 2026

   

   alexwrite

   on Oct 11, 2026

   OwnerAuthor

   More actions

   Fait

   • Une classe PHP dont les constantes portent un nom de mot réservé (DEFAULT, PRIVATE, CLASS, VOID...), typées ou non, s'analyse sans erreur : la classe reste dans le graphe, avec ses arêtes contains, imports et references_constant.
   • PR : fix(php): parse class constants named after reserved keywords #46 (fermera cette issue à la fusion).

   Comment

   • pyproject.toml:40 : tree-sitter-php figé sur alexwrite/tree-sitter-php@9678351 (branche fix/keyword-constant-names = commits de fix: parse keyword identifiers in expressions and constants tree-sitter/tree-sitter-php#307, auteur conservé, plus un test de corpus).
   • uv.lock : seulement les deux blocs tree-sitter-php.
   • tests/test_php_keyword_constant_names.py : 7 déclarations à nom réservé sans parse_errors, et une classe ne contenant que des constantes qui garde son nœud et son arête references_constant.
   • Fork de la grammaire : test/corpus/bugs.txt, cas « Reserved keywords as typed class constant names » (final, optionnel, union, type nommé ; classe, interface, enum).

   Pourquoi ce choix

   • Reprendre Lack of NoneType handling in serve.py - causes AttributeError: 'NoneType' object has no attribute 'lower' Graphify-Labs/graphify#307 plutôt qu'écrire un correctif : la PR amont (CI verte, en attente de relecture) corrige exactement le défaut. Un correctif maison ou une PR amont aurait fait doublon. Soutien chiffré posté sur Lack of NoneType handling in serve.py - causes AttributeError: 'NoneType' object has no attribute 'lower' Graphify-Labs/graphify#307, avec le cas de corpus en plus.
   • Commit figé par référence directe PEP 508 plutôt que [tool.uv.sources] : l'installation quotidienne passe par pip install "graphifyy @ git+...", et pip ignore tool.uv.sources. Vérifié dans un venv propre : pip construit le binding depuis la racine du dépôt, sans sous-répertoire.
   • Alternative écartée : attendre la publication de 0.25.2 sur PyPI, sans date connue.

   Apprentissages

   • Le diagnostic de départ était faux : les constantes typées sont supportées depuis feat: support typed class constants tree-sitter/tree-sitter-php#205 (2024). Seul le nom réservé casse : 61 des 83 mots-clés testés en typé, 7 en non typé.
   • Graphify ne crée pas de nœud de constante PHP, typée ou non. L'objectif « de 0 à toutes les constantes typées » ne pouvait donc pas être atteint par la grammaire.
   • Selon le contenu des chaînes (;, parenthèses), la récupération d'erreur avale soit la constante, soit toute la classe. La fixture reproduit le second cas.
   • Les 2 fichiers en erreur sur le projet A étaient l'un « 0 symbole » (avertissement affiché), l'autre silencieux parce que le seuil d'avertissement n'a rien vu disparaître.

   Faiblesses restantes

   • Référence directe en Git : bloque la publication PyPI depuis ce fork (sans objet aujourd'hui) et exige un compilateur C à l'installation. À ramener à une plage PyPI dès que Lack of NoneType handling in serve.py - causes AttributeError: 'NoneType' object has no attribute 'lower' Graphify-Labs/graphify#307 sera publiée.
   • uv lock --check échoue, mais il échouait déjà sur main (dérive du verrou antérieure, sans lien avec ce lot).
   • PHP : un accès Classe::CONSTANTE vers un autre fichier ne crée aucune arête sans autre mention de la classe #28 (p2) : un accès Foo::BAR vers un autre fichier ne crée aucune arête si Foo n'est pas aussi utilisé comme type dans le fichier appelant. Repéré en écrivant le test.

   Quality

   • Test de régression : 8 échecs avec tree-sitter-php 0.24.1, 8 réussites avec le commit figé
   • Suite complète : 7536 passed, 14 skipped, 0 failed (référence main : 7528 + 8 nouveaux)
   • ruff : All checks passed ; pyright sur le nouveau fichier : 0 erreur
   • Grammaire, tree-sitter test (CLI 0.27.0) : 157/157 ; le nouveau cas échoue sur le master amont
   • Mesure, avant puis après, différentiel complet des nœuds et des arêtes : projet A, 2 fichiers en erreur de syntaxe deviennent 0 ; la classe disparue revient (1 nœud, contains + 8 imports qui visaient un nœud sans source), une position de constructeur corrigée, rien d'autre ne change. Projet B : 0 fichier en erreur avant comme après, graphe identique.
5. alexwrite

   added a commit that references this issue on Oct 11, 2026

   Merge fix/php-typed-class-constants (#17)

   462e694
6. alexwrite

   closed this as completedin #46on Oct 11, 2026