docs: document adding models not in the catalog - #896
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughThe documentation now covers Agent Network model pricing, cache rates, self-hosted pricing defaults, uncatalogued models, and AWS Bedrock cross-region inference profiles. ChangesAgent Network pricing documentation
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
src/pages/selfhosted/maintenance/configuration-files.mdxOops! Something went wrong! :( ESLint: 9.39.2 TypeError: Converting circular structure to JSON Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/pages/agent-network/providers.mdx`:
- Around line 135-137: Replace the sentence beginning “Self-hosters can seed”
with the grammatically correct wording “Self-hosters can use a pricing file to
seed the catalog defaults for these fields.” Preserve the existing
pricingDefaultsFile link.
In `@src/pages/selfhosted/maintenance/configuration-files.mdx`:
- Around line 408-410: Update the server.agentNetwork.pricingDefaultsFile
documentation to state that Bedrock pricing keys must use normalized model IDs
rather than pasted full IDs. Document removing ARN wrappers,
us./eu./apac./global. prefixes, and -vN[:N] or date-version suffixes, including
the example normalization from eu.anthropic.claude-sonnet-5 to
anthropic.claude-sonnet-5.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 4c6117e7-b4ca-400b-a152-4544a46c0405
⛔ Files ignored due to path filters (2)
public/docs-static/img/agent-network/providers/agent-network-custom-model.pngis excluded by!**/*.pngpublic/docs-static/img/agent-network/providers/agent-network-provider-metadata.pngis excluded by!**/*.png
📒 Files selected for processing (3)
src/pages/agent-network/integrations/bedrock.mdxsrc/pages/agent-network/providers.mdxsrc/pages/selfhosted/maintenance/configuration-files.mdx
| Self-hosters can seed the catalog defaults these fields prefill from with a | ||
| pricing file. See | ||
| [`server.agentNetwork.pricingDefaultsFile`](/selfhosted/maintenance/configuration-files#agent-network-settings). |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Rewrite the pricing-file sentence for clarity.
Line 53 uses pre-fills, while this line uses prefill. The current sentence is also ungrammatical. Use: “Self-hosters can use a pricing file to seed the catalog defaults for these fields.”
Static analysis flagged the inconsistent variants.
🧰 Tools
🪛 LanguageTool
[uncategorized] ~135-~135: Do not mix variants of the same word (‘prefill’ and ‘pre-fill’) within a single text.
Context: ... seed the catalog defaults these fields prefill from with a pricing file. See [`server....
(EN_WORD_COHERENCY)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@src/pages/agent-network/providers.mdx` around lines 135 - 137, Replace the
sentence beginning “Self-hosters can seed” with the grammatically correct
wording “Self-hosters can use a pricing file to seed the catalog defaults for
these fields.” Preserve the existing pricingDefaultsFile link.
Source: Linters/SAST tools
Summary by CodeRabbit