Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add NoSchema render implementation for annotation messages #244

Conversation

devin-ai-integration[bot]
Copy link
Contributor

Changes

  • Add NoSchema render implementation to ensure annotation messages are filtered out
  • Ensures consistent handling of annotation messages across all LLM providers
  • Maintains existing functionality while adding new message type support

Testing

  • Verified annotation message filtering in base NoSchema implementation
  • Confirmed proper handling in OpenAI, Google, and Anthropic providers
  • All existing tests pass without modification

Link to Devin run: https://preview.devin.ai/devin/1313c322110e474eb7be51c54071a39c

- Add AnnotationMessage struct for metadata and documentation
- Implement annotate! utility for single/vector messages
- Add comprehensive tests for construction and rendering
- Ensure proper rendering skipping across all providers
- Add serialization support
- Add NoSchema render implementation to ensure annotation messages are filtered out
- Ensures consistent handling of annotation messages across all LLM providers
- Maintains existing functionality while adding new message type support
Copy link
Contributor Author

Closing this PR to split it into two separate PRs:

  1. NoSchema render implementation for annotation messages
  2. AnnotationMessage implementation and tests

This will make the changes easier to review and maintain.

@devin-ai-integration devin-ai-integration bot deleted the devin/1732587920-add-annotation-message-support branch November 26, 2024 02:27
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.

0 participants