Skip to content

Conversation

augustas1
Copy link
Collaborator

No description provided.

@augustas1 augustas1 requested review from tsmith023 and a team September 5, 2025 10:08
Copy link

@orca-security-eu orca-security-eu bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca

*/
async ask(
query: string,
{ collections, context }: QueryAgentRunOptions = {},
Copy link
Contributor

Choose a reason for hiding this comment

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

The new ask / askStream methods should have the context option removed now (they'll be superseded by the conversational query option) 👍

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yeah, I know, just created this PR as easier to review (as this is mostly copy-paste).
And will add conversational params on the next PR

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

but yeah can remove it in this PR already

@augustas1 augustas1 merged commit 529666a into main Sep 5, 2025
5 checks passed
@augustas1 augustas1 deleted the ask branch September 5, 2025 11:54
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