Add event triggering for streaming [#3489846]

Problem/Motivation

Add the methods that makes logging possilbe of responses on streaming events from here: https://www.drupal.org/project/ai/issues/3489690

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comment	File	Size	Author
#3	openai_provider_iterator.patch	562 bytes	seogow

Issue fork ai_provider_openai-3489846

Show commands

Start within a Git clone of the project using the version control instructions.

Add & fetch this issue fork’s repository

Or, if you do not have SSH keys set up on git.drupalcode.org:

Add & fetch this issue fork’s repository

3489846-add-event-trigger changes, plain diff MR !48
Check out this branch for the first time

Check out existing branch, if you already have it locally

About issue forks

Comments

Comment #1

26 November 2024 at 11:00

marcus_johansson created an issue. See original summary.

Comment #2

26 November 2024 at 11:35

marcus_johansson committed 57f616b2 on 3489846-add-event-triggering
```
Issue #3489846: Add event triggering for streaming
```

Comment #3

seogow commented 4 February 2025 at 11:54

Status	File	Size
new	openai_provider_iterator.patch	562 bytes

I have uploaded a patch which uses a suggested fix in a AI module.

Each AI provider supporting stream should be able to iterate any way it needs to, so I suggest this change: https://git.drupalcode.org/issue/ai-3493277/-/blob/3493277-extend-loggin...

This way the getIterator() works as a wrapper template, ensuring compatibility, which at the end of stream triggers the event. If a provider requires different implementation of the iterator, it can override overrideGetIterator().