Rewrite output processors from Agent side used by Platform

- Introduced a separate output processor, that accepts a result handler and gives the result of the Output data container to this handler.
- removed dependency to symfony/ai-agent from composer file within platform subtree
- added deptrac check

Author: glady

.github/workflows/deptrac.yaml

@@ -0,0 +1,59 @@
+name: deptrac
+
+on:
+    push:
+        paths-ignore:
+            - 'src/*/doc/**'
+            - 'src/**/*.md'
+    pull_request:
+        paths-ignore:
+            - 'src/*/doc/**'
+            - 'src/**/*.md'
+
+concurrency:
+    group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
+    cancel-in-progress: true
+
+jobs:
+    deptrac:
+        name: deptrac
+        runs-on: ubuntu-latest
+        strategy:
+            fail-fast: false
+            matrix:
+                php-version: [ '8.4' ]
+        steps:
+            - name: Checkout
+              uses: actions/checkout@v5
+
+            - name: Configure environment
+              run: |
+                  echo COLUMNS=120 >> $GITHUB_ENV
+                  echo COMPOSER_UP='composer update --no-progress --no-interaction --ansi --ignore-platform-req=ext-mongodb' >> $GITHUB_ENV
+
+            - name: Setup PHP
+              uses: shivammathur/setup-php@v2
+              with:
+                  php-version: ${{ matrix.php-version }}
+
+            - name: Get composer cache directory
+              id: composer-cache
+              run: |
+                  echo "dir=$(composer config cache-files-dir)" >> $GITHUB_OUTPUT
+
+            - name: Cache packages dependencies
+              uses: actions/cache@v4
+              with:
+                  path: ${{ steps.composer-cache.outputs.dir }}
+                  key: ${{ runner.os }}-composer-packages-${{ matrix.php-version }}-${{ hashFiles('src/**/composer.json') }}
+                  restore-keys: |
+                      ${{ runner.os }}-composer-packages-${{ matrix.php-version }}
+
+            - name: Install root dependencies
+              uses: ramsey/composer-install@v3
+
+            - name: Build root packages
+              run: php .github/build-packages.php
+
+            - name: Run DEPTRAC
+              run: $COMPOSER_UP && vendor/bin/deptrac

composer.json

@@ -7,6 +7,7 @@
     ],
     "require-dev": {
         "php": ">=8.2",
+        "deptrac/deptrac": "^4.2",
         "php-cs-fixer/shim": "^3.75",
         "phpstan/phpstan-strict-rules": "^2.0",
         "symfony/filesystem": "^7.3|^8.0",

deptrac.yaml

@@ -0,0 +1,26 @@
+deptrac:
+  paths:
+    - ./src
+  exclude_files:
+    - '#.*test.*#'
+  layers:
+    - name: Agent
+      collectors:
+        - type: classLike
+          value: Symfony\\AI\\Agent.*
+    - name: Platform
+      collectors:
+        - type: classLike
+          value: Symfony\\AI\\Platform.*
+    - name: Store
+      collectors:
+        - type: classLike
+          value: Symfony\\AI\\Store.*
+  ruleset:
+    Platform: ~
+    Agent:
+        - Platform
+        - orm
+        - Store
+    Store:
+        - Platform

src/agent/src/OutputProcessor/ResultOutputProcessor.php

@@ -0,0 +1,29 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\AI\Agent\OutputProcessor;
+
+use Symfony\AI\Agent\Output;
+use Symfony\AI\Agent\OutputProcessorInterface;
+use Symfony\AI\Platform\Result\ResultHandlerInterface;
+
+readonly class ResultOutputProcessor implements OutputProcessorInterface
+{
+    public function __construct(
+        private ResultHandlerInterface $resultProcessor,
+    ) {
+    }
+
+    public function processOutput(Output $output): void
+    {
+        $this->resultProcessor->handleResult($output->result);
+    }
+}

src/ai-bundle/tests/DependencyInjection/AiBundleTest.php

@@ -683,9 +683,23 @@ public function testTokenUsageProcessorTags()
 
         $agentId = 'ai.agent.tracked_agent';
 
+        // Token usage determiner must exist for OpenAI platform
+        $tokenUsageDeterminer = $container->getDefinition('ai.platform.token_usage_result_handler.openai');
+        $outputTags = $tokenUsageDeterminer->getTag('ai.agent.token_usage_result_handler');
+
+        $foundTag = false;
+        foreach ($outputTags as $tag) {
+            if (($tag['agent'] ?? '') === $agentId) {
+                $foundTag = true;
+                break;
+            }
+        }
+
+        $this->assertTrue($foundTag, 'Token usage determiner should have output tag with full agent ID');
+
         // Token usage processor must exist for OpenAI platform
-        $tokenUsageProcessor = $container->getDefinition('ai.platform.token_usage_processor.openai');
-        $outputTags = $tokenUsageProcessor->getTag('ai.agent.output_processor');
+        $tokenOutputProcessor = $container->getDefinition('ai.platform.token_usage_processor.openai');
+        $outputTags = $tokenOutputProcessor->getTag('ai.platform.token_usage_processor');
 
         $foundTag = false;
         foreach ($outputTags as $tag) {