Skip to content

Conversation

@Nyholm
Copy link
Member

@Nyholm Nyholm commented May 29, 2025

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

This PR will also include fixes to make the CI green

@Nyholm Nyholm closed this May 29, 2025
@Nyholm Nyholm deleted the link branch May 29, 2025 10:57
@Nyholm Nyholm restored the link branch May 29, 2025 10:57
@Nyholm Nyholm reopened this May 29, 2025
@Nyholm Nyholm merged commit 1741c85 into main May 29, 2025
24 checks passed
@Nyholm Nyholm deleted the link branch May 29, 2025 17:52
chr-hertel added a commit that referenced this pull request Jun 6, 2025
ci: introduce github actions
chr-hertel added a commit that referenced this pull request Jun 13, 2025
chr-hertel added a commit that referenced this pull request Jun 13, 2025
feat: add support for client transports
chr-hertel pushed a commit that referenced this pull request Jun 22, 2025
As the demo is mainly configured to be utilized with OpenAI GPT and Embeddings, what is later in the Readme explicit mentioned, it should also be mentioned in the Requirements-Section of the Readme.
chr-hertel added a commit that referenced this pull request Jul 20, 2025
This PR was merged into the main branch.

Discussion
----------

[AIBundle] Fix profiler with non-string tool results

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| Docs?         | no
| License       | MIT

Fix the following issue in Symfony AI profiler view which appears when using `blog` demo which has a `clock` tool returning a `DatePoint` instance:

```
An exception has been thrown during the rendering of a template ("nl2br(): Argument #1 ($string) must be of type string, Symfony\Component\Clock\DatePoint given") in `@AI`/data_collector.html.twig at line 242
```

Commits
-------

5f9bcc4 [AIBundle] Fix profiler with non-string tool results
chr-hertel added a commit that referenced this pull request Jul 20, 2025
This PR was merged into the main branch.

Discussion
----------

[AIBundle] Fix dependency injection

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| Docs?         | no
| License       | MIT

Fix DI config after #163 / #154. Should've rebased before merge – now we're getting a bunch of errors on `main`:

```
Symfony\AI\Agent\Toolbox\Toolbox::__construct(): Argument #1 ($tools) must be of type Traversable|array, Symfony\AI\Agent\Toolbox\ToolFactory\ReflectionToolFactory given
```

etc

Commits
-------

3c6290e [AIBundle] Fix dependency injection config after #163
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants