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

Add OpenAI Assistants API integration #568

Closed

Conversation

devin-ai-integration[bot]
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Dec 11, 2024

This PR adds OpenAI Assistants API integration to AgentOps SDK.

Changes:

  • Add AssistantAgent class for OpenAI Assistants API integration
  • Map Assistant → Agent, Thread → Session, Run → Event
  • Add example usage in assistant_example.py
  • Add comprehensive test suite
  • Fix session initialization in tests
  • Fix string formatting

Link to Devin run: https://app.devin.ai/sessions/badb89f0ffd941cebcf631a7e6e593c1

Testing:

  • Added unit tests
  • Tests passing locally
  • CI checks pending

- Add AssistantAgent wrapper class for OpenAI integration
- Implement Thread->Session mapping with thread_id tracking
- Add Run->Event tracking with LLMEvent and ToolEvent support
- Create example usage and comprehensive tests

The integration allows AgentOps to track and monitor AI assistants
created through OpenAI's Assistants API, mapping:
- Assistant → Agent
- Thread → Session
- Run → Event

Co-Authored-By: Alex Reibman <[email protected]>
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Add "(aside)" to your comment to have me ignore it.

@teocns
Copy link
Contributor

teocns commented Dec 11, 2024

haha noob

devin-ai-integration bot and others added 6 commits December 11, 2024 19:49
- Fix JSON serialization of UUID objects in session.py
- Remove duplicate update_session requests
- Refactor get_analytics to use _update_session response
- Update test_pre_init.py to verify request count

Co-Authored-By: Alex Reibman <[email protected]>
@the-praxs
Copy link
Member

Closing this because - well it sucks.

@the-praxs the-praxs closed this Dec 16, 2024
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