Refactor source transformation pipeline to a NodeVisitor-only architecture with loader-level format-preserving rewrites

[Copilot]

The framework now moves source transformation orchestration into SourceTransformingLoader with a single AST traversal + format-preserving print pass, while keeping StreamMetaData as a DTO of original parse artifacts (stream metadata, original AST, original tokens).

• DTO-only StreamMetaData

  • StreamMetaData stores original metadata/AST/tokens only.
  • Source mutation/reload APIs were removed.

• Centralized AST traversal and printing in loader

  • SourceTransformingLoader::transformCode() returns transformed AST (Node[]).
  • One central NodeTraverser with registered visitors is used.
  • Format-preserving pretty-printing (printFormatPreserving) is performed in loader using old AST/tokens + new AST.

• NodeVisitor-only transformation pipeline

  • Legacy SourceTransformer runtime flow was removed.
  • AspectKernel::registerTransformers() block was removed.
  • SourceTransformingLoader::$transformers and addTransformer() were removed.
  • SourceTransformer interface was removed.

• Typed visitor result reporting

  • Added NodeTransformerResultReporter with getNodeTransformerResult(): TransformerResultEnum.
  • Visitors report transformation state via this interface.
  • Loader aggregates visitor results without method_exists(...) checks.

• Weaving converted to NodeVisitor

  • WeavingTransformer now participates in traversal as a node visitor and reports TransformerResultEnum.
  • Kernel now registers weaving in the node-visitor list.

• Namespace-level context injection

  • Added FileNameInjectorNodeVisitor.
  • Context attributes are injected only on top-level Namespace_ nodes (including StreamMetaData DTO and original file name), not on every node.
  • MagicConstantTransformer reads filename context from namespace attributes and keeps local namespace-scope state.

• Caching flow in stream filtering

  • Cache freshness check and cached-source fast-path run in SourceTransformingLoader.
  • On stale cache, loader transforms source, pretty-prints, and writes refreshed cache.

• Docs / BC break

  • Added 4.0.0 changelog entry documenting removal of the legacy transformer approach as a BC break.

• Tests updated for visitor pipeline

  • Transformer tests validate visitor traversal + format-preserving reconstruction.
  • Weaving tests now execute weaving through traversal (injector + weaving visitor) rather than direct transform() calls.
  • Constructor expectations remain in original no-parentheses/property-access form where required.

$newStmts = SourceTransformingLoader::transformCode($metadata);

$printer = new Standard();
$source = $printer->printFormatPreserving(
    $newStmts,
    $metadata->syntaxTree,
    $metadata->tokenStream
);

[lisachenko]

Also, tests are failing, need to fix them

[Copilot]

Also, tests are failing, need to fix them

Fixed in 933b34b/ad660bb by removing CloningVisitor usage in these transformer tests and aligning visitor-based traversal/context handling. UI screenshot: N/A (no UI changes).

[lisachenko]

@copilot unit tests are failing

[Copilot]

@copilot unit tests are failing

Fixed in df748e1 by updating the failing transformer unit tests to use CloningVisitor before applying visitors, so printFormatPreserving() receives distinct old/new AST trees and no longer crashes on lowest-dependency runs. UI screenshot: N/A (no UI changes).

[lisachenko]

@copilot still build is failing.

Could you please also add GH token for composer to phpstan + phpunit to fix download error and to make phpstan/phpunit working

- name: Setup PHP
  uses: shivammathur/setup-php@v2
  with:
    ...
  env:
    COMPOSER_TOKEN: ${{ secrets.GITHUB_TOKEN }}

[Copilot]

@copilot still build is failing.

Could you please also add GH token for composer to phpstan + phpunit to fix download error and to make phpstan/phpunit working
...

Implemented in a7d28cd: added COMPOSER_TOKEN: ${{ secrets.GITHUB_TOKEN }} to the Install PHP (shivammathur/setup-php@v2) steps in both .github/workflows/phpunit.yml and .github/workflows/phpstan.yml so Composer can use authenticated GitHub downloads. UI screenshot: N/A (no UI changes).

[lisachenko]

@copilot phpstan and phpunit builds are failing, please fix them

[Copilot]

@copilot phpstan and phpunit builds are failing, please fix them

Implemented in 56bbdaf (plus c96302d/44108f6): fixed the remaining PHPStan/PHPUnit failures by normalizing parser tokens to PhpParser\Token in StreamMetaData, tightening URI typing in SourceTransformingLoader, skipping anonymous-class new rewrites in ConstructorExecutionTransformer, and guarding filename attribute typing in MagicConstantTransformer. UI screenshot: N/A (no UI changes).

[lisachenko]

@copilot update your own instruction in the repository to use PHP8.4 version for running the copilot workflow. Configure and install necessary version for code-review, ensure that composer install is completed successfully, otherwise stop.

Once composer will work - check build failure again and fix it

[Copilot]

@copilot update your own instruction in the repository to use PHP8.4 version for running the copilot workflow. Configure and install necessary version for code-review, ensure that composer install is ...

Implemented in f9f9296 (plus 94ab8ab): added .github/workflows/copilot-setup-steps.yml to force PHP 8.4 for Copilot cloud-agent and run composer install as a required setup step, added matching repo instruction in CLAUDE.md, then rechecked build failures and fixed the reported issues in SourceTransformingLoader (cache freshness/URI guard) and StreamMetaData (stable tokenization for format-preserving printing). UI screenshot: N/A (no UI changes).

[lisachenko]

@copilot please recreate this PR from scratch - keep all GIT changes, but make another branch and PR to start from clean context

[Copilot]

@lisachenko I've opened a new pull request, #540, to work on those changes. Once the pull request is ready, I'll request review from you.

[lisachenko]

Still, not working ( AST pretty-printing often fails to keep new lines, breaking the line number mapping, therefore not possible at the moment