docs: Add comprehensive LLM-friendly API documentation#2126
Closed
askdevai-bot wants to merge 2 commits intoBoundaryML:canaryfrom
Closed
docs: Add comprehensive LLM-friendly API documentation#2126askdevai-bot wants to merge 2 commits intoBoundaryML:canaryfrom
askdevai-bot wants to merge 2 commits intoBoundaryML:canaryfrom
Conversation
Add LLM.md file containing complete API reference and usage examples for BAML (Boundary AI Markup Language). This documentation provides: - Complete type system reference - Client configuration examples - Function definition patterns - Testing framework usage - Multi-language client examples - Streaming and retry mechanisms - Provider configuration details This documentation is designed to be LLM-friendly for better AI-assisted development workflows.
|
@askdevai-bot is attempting to deploy a commit to the Gloo Team on Vercel. A member of the Team first needs to authorize it. |
Contributor
There was a problem hiding this comment.
Caution
Changes requested ❌
Reviewed everything up to 6974697 in 2 minutes and 33 seconds. Click for details.
- Reviewed
3027lines of code in1files - Skipped
0files when reviewing. - Skipped posting
2draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. LLM.md:3
- Draft comment:
Consider adding a Table of Contents at the top to help navigate this very extensive document. - Reason this comment was not posted:
Decided after close inspection that this draft comment was likely wrong and/or not actionable: usefulness confidence = 0% vs. threshold = 50% This is a very long technical document with many sections. A ToC would help readers navigate it. However, this is a Markdown file and adding a ToC is more of a documentation improvement suggestion rather than a code issue that needs fixing. The rules state we should not make purely informative comments or comments that don't require clear code changes. The suggestion would genuinely improve document usability. Many documentation systems can auto-generate ToCs from Markdown headings, so this might not even require manual maintenance. While useful, the rules explicitly state not to make purely informative comments. Documentation improvements that don't affect functionality should be handled through other channels. The comment should be deleted as it suggests a documentation improvement rather than a required code change, violating the rule about not making purely informative comments.
2. LLM.md:76
- Draft comment:
Ensure consistent quoting for string literals; for example, model names are unquoted here but quoted in later examples. - Reason this comment was not posted:
Decided after close inspection that this draft comment was likely wrong and/or not actionable: usefulness confidence = 20% vs. threshold = 50% While this is technically a valid observation about inconsistent style, it's a very minor formatting issue. The code will work either way. The rules say to only keep comments that clearly require code changes, and to not make purely informative comments. This seems more like a style suggestion than a critical issue requiring change. The inconsistency could potentially cause confusion for users copying examples. Consistent style does improve code readability and maintainability. While consistency is good, this is too minor of an issue to warrant a PR comment. The code works correctly either way, and this is more of a style preference than a functional issue. Delete the comment. It points out a real but very minor style inconsistency that doesn't affect functionality. The rules specifically say not to make purely informative comments.
Workflow ID: wflow_Bl6GdihodmaIGdgF
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
| ### Caching and Memoization | ||
| ```python | ||
| from functools import lru_cache | ||
| import hashlib |
Contributor
There was a problem hiding this comment.
The caching example uses time.time() but does not import the time module. Please add 'import time'.
| @@ -0,0 +1,3021 @@ | |||
| # BAML (Boundary AI Markup Language) - Complete API Reference | |||
Contributor
There was a problem hiding this comment.
Consider splitting this one massive documentation file into smaller, modular files or sections to improve maintainability.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request: Add Comprehensive LLM-Friendly API Documentation
Thanks for taking the time to review this pull request!
Issue Reference
Changes
This PR adds a comprehensive
LLM.mdfile containing complete API reference and usage examples for BAML (Boundary AI Markup Language). The documentation provides:Key Features Documented:
Documentation Structure:
Testing
Screenshots
N/A - This is a documentation-only change
PR Checklist
Additional Notes
This documentation is specifically designed to be LLM-friendly, meaning it provides comprehensive context and examples that AI assistants can use to help developers work more effectively with BAML. The documentation includes:
This will significantly improve the developer experience when using AI assistants for BAML development, as the AI will have access to comprehensive, well-structured documentation about all BAML features and capabilities.
A developer on Askdev.AI requested this update
Important
Adds
LLM.mdwith comprehensive BAML API documentation, covering type system, client configuration, function definitions, testing, and advanced features for LLM providers.LLM.mdwith comprehensive API reference and usage examples for BAML.This description was created by
for 6974697. You can customize this summary. It will automatically update as commits are pushed.