Skip to content

Conversation

@Smidge
Copy link
Contributor

@Smidge Smidge commented Jul 29, 2025

Add sendHumanMessage() method for sending human text messages
Method checks streaming status and active session
Formats message with timestamp, session_id, and message_type
Add JSDoc documentation for talk() and sendDataMessage() methods

Smidge added 2 commits July 29, 2025 09:49
- Add sendHumanMessage() method for sending human text messages
- Method checks streaming status and active session
- Formats message with timestamp, session_id, and message_type
- Add JSDoc documentation for talk() and sendDataMessage() methods
- Rename method from sendHumanMessage() to sendUserMessage()
- Update JSDoc documentation to use 'user' instead of 'human'
- Update error messages and console warnings accordingly
Copilot AI review requested due to automatic review settings July 29, 2025 09:39
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new sendUserMessage method to the AnamClient class for sending human text messages during streaming sessions. The changes focus on enhancing the client's messaging capabilities while improving documentation.

  • Added sendUserMessage() method with validation for streaming status and active sessions
  • Enhanced JSDoc documentation for existing talk() and sendDataMessage() methods
  • Minor code reorganization of imports
Comments suppressed due to low confidence (1)

src/AnamClient.ts:427

  • [nitpick] The method name sendUserMessage conflicts with the PR title which mentions sendHumanMessage. Consider using consistent naming between the PR description and implementation.
  public sendUserMessage(content: string): void {

@Smidge Smidge merged commit f8857e1 into main Jul 29, 2025
5 checks passed
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