[deniskern]

Supersedes #611. A fresh take against today's main, as @colbymchenry suggested when closing #611 (close comment). It carries @jorgerobles's Symfony resolver (credited with Co-authored-by on the commits that port his code) plus the AST-based route work from jorgerobles#1, and resolves every point raised when #611 was closed.

What it adds

• Route nodes from three sources, each linked to the controller action that serves it:
  • PHP 8 #[Route] attributes, read from the PHP AST: class-level prefixes, stacked routes on one action, multi-line attributes, methods:, name:, defaults:, requirements:, host:, schemes:, and Request::METHOD_GET-style constants.
  • config/routes.yaml (path, controller, methods, prefix; a resource: import is not a route).
  • The compiled router in the kernel's build dir (var/cache/<env>/…).
• Service definitions: a services.yaml / compiled-container service links to its class.
• PHP attributes as decorates edges on classes, methods and functions (#[AsCommand], #[ORM\Entity], #[AsEventListener], …) for every PHP project, so attribute classes show who uses them.
• Detection: symfony/framework-bundle, symfony/symfony, shopware/core (Shopware 6 shops and plugins are Symfony apps without bin/console), or bin/console + config/.

The close comment, point by point

1. (?:\s*\n\s*)* before class backtracks exponentially. The regex is gone; routes are read from the attribute_list → attribute_group → attribute nodes of each class and method declaration. On the old pattern, #[Route('/x')] followed by 16 whitespace-only lines took 3.3 s and 18 lines took 17.5 s. symfony-resolver.test.ts > pathological input feeds 20,000 such lines plus 2,000 unclosed nested [ and requires the extraction to finish in under 1 s.
2. Controller resolution scanned every class per reference. resolve() now looks the class up through the name index (getNodesByName(shortName)), takes only the class in the namespace the FQCN names (another namespace's class of the same short name stays unlinked), and asks only for the method by name in that file (getNodesInFileNamed), limited to the class's line range. The unit test's context throws if getNodesByKind is called; an integration test with two BlogControllers in different namespaces pins the FQCN choice. claimsReference is narrowed to names containing \, and resolve() answers only route and service references.
3. PHP attributes don't reach the native kernel. The PHP walker in codegraph-kernel/src/php.rs (the default PHP path) dropped attributes by design, and the wasm extractor never descended into attribute_list. Both now emit the same decorates refs; kernel-php-parity.test.ts checks kernel/wasm equality and the expected refs, and the end-to-end test passes on both paths (CODEGRAPH_KERNEL=0 and default).
4. The #[ change in strip-comments.ts affects every PHP project. Dropped. Route extraction reads the AST, so commented-out attributes are skipped without touching the comment stripper.

Also fixed from the review of #611

• A path-less class-level #[Route(defaults: …)] (Shopware's _routeScope) no longer swallows the first method route.
• Every stacked #[Route] on one action becomes a route.
• No duplicate class nodes for entities, repositories, subscribers and commands (those extractors were removed; the attribute edges cover them).
• symfony/polyfill-* is excluded by prefix.
• Only var/cache/… and .symfony/… count as the compiled container, not any src/Cache/…Container.php.
• A compiled route's HTTP methods are read from the seventh Route argument (the sixth is the schemes), and an empty list means ANY.
• #[Route('')] under a class prefix is that prefix.
• The extra PHP syntax trees are deleted after reading.
• defaults: / requirements: arrays keep their own pairs when both are present, and unquoted scalars ('page' => 1, => true) are kept. Fixtures that used YAML-style { key: value } (not valid PHP) now use PHP arrays.

Notes for review

• Route extraction does a second PHP parse in the framework extractor, only for files containing both #[ and Route. The kernel's tables don't carry attribute arguments, so reading them from the kernel would mean widening the kernel ABI. I kept that out of this PR.
• EXTRACTION_VERSION bumped to 29 (new framework extractor and new PHP edges), so existing indexes get the re-index hint.
• XML service definitions (services.xml, common in Shopware) are not parsed.

Testing

• __tests__/symfony-resolver.test.ts (unit, 48 tests) and __tests__/symfony-integration.test.ts (end to end, 10 tests).
• __tests__/kernel-php-parity.test.ts, with the kernel built (npm run build:kernel).
• npm test with the kernel built: 9,815 passed, 46 skipped. The only failure, ui-server-api.test.ts > the engine's own busiest symbol (a 100 ms timing bar, 114–128 ms here), runs only on a machine with a local .codegraph/ index of this repository and does not touch PHP or Symfony code.
• npx tsc --noEmit: clean.

Credits

• @jorgerobles wrote the original Symfony resolver in Feat/symfony resolver #611; the commits that port his code carry him as Co-authored-by.
• The AST route extraction and the fixes listed above come from fix(symfony): AST-based #[Route] extraction, no duplicate class nodes, PHP attribute edges jorgerobles/codegraph#1.

🤖 Generated with Claude Code and onedot-devkit