Skip to content

Conversation

@augustas1
Copy link
Collaborator

No description provided.

@augustas1 augustas1 requested review from a team and tsmith023 September 5, 2025 13:00
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

agentsHost?: string;
};

export type QueryAgentQuery = string | ConversationContext;
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it possible to make this string | ChatMessage[] and then wrap in ConversationContext just for the request, so the user doesn't need to include the {messages: ...} (to match the python client)?

Copy link
Contributor

@danmichaeljones danmichaeljones left a comment

Choose a reason for hiding this comment

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

Looks good! 👍

@augustas1 augustas1 merged commit a04bd6a into main Sep 8, 2025
5 checks passed
@augustas1 augustas1 deleted the conversation branch September 8, 2025 09:10
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.

3 participants