Skip to content

Conversation

jakemac53
Copy link
Contributor

Initially created this just for testing Gemini CLI support for progress tokens, but it makes sense to land.

Copy link

PR Health

Changelog Entry ✔️
Package Changed Files

Changes to files need to be accounted for in their respective changelogs.

@jakemac53
Copy link
Contributor Author

We might eventually want to make this a bit more ergonomic too - such as including a factory for creating unique progress tokens.

),
const delayMs = 2000;
print(
'Calling `${tool.name}` tool, with an artificial ${delayMs}ms second '
Copy link
Member

Choose a reason for hiding this comment

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

nit: Remove "second".

meta: MetaWithProgressToken(progressToken: ProgressToken(1)),
);
// Make sure to listen before awaiting the response - you could listen
// after sending the request but before awaiting the result as well.
Copy link
Member

Choose a reason for hiding this comment

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

If both scenarios should work then we should probably test both.

@auto-submit auto-submit bot merged commit a9eb685 into main Aug 22, 2025
28 checks passed
@auto-submit auto-submit bot deleted the progress-example branch August 22, 2025 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants