Skip to content

Conversation

@35C4n0r
Copy link
Collaborator

@35C4n0r 35C4n0r commented Jul 19, 2025

@35C4n0r
Copy link
Collaborator Author

35C4n0r commented Jul 19, 2025

@matifali @bpmct (or anyone maintaining this repo). Let me know if this needs any changes.

@matifali matifali requested review from bpmct and hugodutka July 19, 2025 07:33
@35C4n0r
Copy link
Collaborator Author

35C4n0r commented Jul 19, 2025

fixing tests in a while

@35C4n0r
Copy link
Collaborator Author

35C4n0r commented Jul 19, 2025

the tests should ideally pass now.
they do 👍🏻

Copy link
Collaborator

@hugodutka hugodutka left a comment

Choose a reason for hiding this comment

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

Thank you for contributing @35C4n0r! The PR looks good - it's only missing tests. Could you add a couple of test cases for gemini to https://github.com/coder/agentapi/tree/main/lib/msgfmt/testdata/format? They will be run by TestFormatAgentMessage after you add AgentTypeGemini to this array.

@35C4n0r
Copy link
Collaborator Author

35C4n0r commented Jul 21, 2025

@hugodutka thanks for the review, will add the tests :)
If you're on discord can you also take a look at this thread related to agentapi.

https://discord.com/channels/747933592273027093/1143265221545316445/1396545560768417792

@35C4n0r 35C4n0r requested a review from hugodutka July 21, 2025 19:19
@35C4n0r
Copy link
Collaborator Author

35C4n0r commented Jul 21, 2025

@hugodutka added the tests

Copy link
Collaborator

@hugodutka hugodutka left a comment

Choose a reason for hiding this comment

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

Thank you!

@hugodutka hugodutka merged commit dde7836 into coder:main Jul 22, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants