Skip to content

Conversation

@chr-hertel
Copy link
Member

Q A
Bug fix? no
New feature? no
Docs? no
Issues
License MIT

Basically cherry picking from #573 - I hope that'S alright @glady - thanks for sure! 🙏

- 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
@carsonbot carsonbot changed the title Bringing in Deptrac for Component Dependencies Bringing in Deptrac for Component Dependencies Oct 21, 2025
@chr-hertel
Copy link
Member Author

chr-hertel commented Oct 21, 2025

@chr-hertel chr-hertel added the Blocked This PR is currently blocked by another topic. label Oct 23, 2025
@OskarStark
Copy link
Contributor

Related

@OskarStark OskarStark closed this Nov 22, 2025
@OskarStark OskarStark reopened this Nov 22, 2025
@OskarStark
Copy link
Contributor

OskarStark commented Nov 24, 2025

The --prefer-lowest does the trick right now:

CleanShot 2025-11-24 at 14 31 52@2x

with a fix here:

OskarStark added a commit that referenced this pull request Nov 24, 2025
…ions (OskarStark)

This PR was merged into the main branch.

Discussion
----------

[Platform] Fix Platform component dependency on Agent exceptions

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| Docs?         | no
| Issues        | Refs #798
| License       | MIT

Remove incorrect import of RuntimeException from Agent component in MissingModelSupportException. The Platform component should use its own exception hierarchy and not depend on Agent exceptions.

Commits
-------

046df40 Fix Platform component dependency on Agent exceptions
@OskarStark
Copy link
Contributor

Please rebase and squash my commit with the prefer-lowest manually, afterwards good to merge

@OskarStark OskarStark changed the title Bringing in Deptrac for Component Dependencies Use deptrac/deptrac to ensure component dependencies Nov 24, 2025
- name: Build root packages
run: php .github/build-packages.php

- name: Run DEPTRAC
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- name: Run DEPTRAC
- name: Run deptrac/deptrac

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Blocked This PR is currently blocked by another topic. Status: Needs Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants