-
Notifications
You must be signed in to change notification settings - Fork 254
Update docs for trace and bug fixes #131
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
Conversation
There was a problem hiding this 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 updates documentation for POML trace functionality and fixes bugs related to OpenAI API format handling. The changes focus on improving POML's integration capabilities and adding comprehensive documentation for Python integrations.
- Fixes OpenAI Chat Completion API format issues in tool definitions and response format schemas
- Adds extensive documentation for Python integrations (OpenAI, LangChain, MCP, MLflow, AgentOps, Weave)
- Enhances tracing functionality with better context handling and local variable scoping
Reviewed Changes
Copilot reviewed 37 out of 46 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
python/tests/test_poml_formats.py |
Fixes OpenAI format structure by adding missing wrapper objects for tools and response_format |
python/tests/test_examples.py |
Adds context file loading support for example tests |
python/tests/integration/ |
Adds comprehensive integration test files for OpenAI, MCP server functionality |
python/tests/assets/ |
Adds POML template files for testing tool calls, response formats, and dynamic tools |
python/poml/api.py |
Fixes OpenAI format generation to properly wrap tools and response_format schemas |
packages/poml/file.tsx |
Improves let variable handling with better local vs global context separation |
docs/python/ |
Adds comprehensive integration documentation for multiple platforms |
examples/ |
Adds math calculation example files with context support |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
This pull request adds documentation and CI support for new integrations (MCP and OpenAI), and improves test workflows to handle context files and additional integration scenarios. The main changes are grouped into documentation additions for new integrations, and enhancements to the GitHub Actions workflows for broader integration testing and context-aware execution.
Documentation for new integrations:
docs/python/integration/agentops.md)docs/python/integration/langchain.md)docs/python/integration/mcp.md)GitHub Actions workflow enhancements:
mcpandopenaiin addition to existing integrations, enabling automated testing for these new integrations. (.github/workflows/test-integration.yml,.github/workflows/shared-test-integration.yml) [1] [2] [3]openai. (.github/workflows/shared-test-integration.yml) [1] [2] [3]Test execution improvements:
.context.jsonfile and use it if available, both forpomlandpoml-clicommands. (.github/workflows/test-after-publish.yml) [1] [2]