Skip to content

docs(migration): explain the "No default base_url" config-load error - #1881

Merged
miyoungc merged 3 commits into
developfrom
fix/vdr-format-b-error-message
May 13, 2026
Merged

docs(migration): explain the "No default base_url" config-load error#1881
miyoungc merged 3 commits into
developfrom
fix/vdr-format-b-error-message

Conversation

@Pouyanpi

@Pouyanpi Pouyanpi commented May 13, 2026

Copy link
Copy Markdown
Collaborator

Description

VDR feedback: developers hitting No default base_url for provider 'X'. on v0.22 couldn't find the verbatim error in the migration guide and had nothing to search for. Add a new "Unsupported Engine on the Default Framework" section that quotes the error verbatim and points at the two real remediations (LangChain opt-in or OpenAI-compatible base_url).

Related Issue(s)

Checklist

  • I've read the CONTRIBUTING guidelines.
  • I've updated the documentation if applicable.
  • I've added tests if applicable.
  • @mentions of the person or team responsible for reviewing proposed changes.

Summary by CodeRabbit

  • Documentation
    • Added migration guide section documenting unsupported engine configuration scenarios in version 0.22, including error details and remediation paths.

Review Change Stack

A v0.21 configuration that uses an engine whose HTTP API is not
OpenAI-compatible (anthropic, cohere, vertexai, google_genai,
huggingface_endpoint, huggingface_pipeline, trt_llm, self_hosted, ...)
fails to load on v0.22 unless the user has opted into LangChain. The
default framework raises:

    No default base_url for provider 'X'. If your endpoint is
    OpenAI-compatible, set parameters.base_url. Otherwise, set
    NEMOGUARDRAILS_LLM_FRAMEWORK=langchain and install the matching
    langchain-<provider> package (see migration guide).

The migration guide already covered the parameter-rename path
(`openai_api_base` to `base_url`) but did not contain the verbatim
string above. A user pasting the error into the docs search landed
on nothing.

Add a new "Unsupported Engine on the Default Framework" section after
"Mixed-Shape Configs" that quotes the error verbatim with a
placeholder provider name, names the engines that typically trigger
it, and links the two existing remediation sections: "Using LangChain"
for the opt-in route and "OpenAI-Compatible Providers" for the
endpoint-swap route.

Doc-only; the error message itself is unchanged.
@coderabbitai

coderabbitai Bot commented May 13, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: b4ea438c-02d3-40c1-b1b4-77e0d626427c

📥 Commits

Reviewing files that changed from the base of the PR and between ef69220 and 275265d.

📒 Files selected for processing (1)
  • docs/migration/0.22.md

📝 Walkthrough

Walkthrough

This PR adds a migration guidance section to the 0.22 documentation, explaining a configuration-load failure that occurs when loading non-OpenAI-compatible LLM engines without explicitly opting into LangChain. The section describes the error message and provides two concrete remediation paths for affected users.

Changes

0.22 Migration Guide Documentation

Layer / File(s) Summary
Unsupported Engine Configuration Guidance
docs/migration/0.22.md
New section explaining the configuration-load failure for non-OpenAI-compatible engines on the default framework, including the error message format and two remediation options: keeping LangChain with NEMOGUARDRAILS_LLM_FRAMEWORK=langchain, or switching to an OpenAI-compatible endpoint with engine: openai and a base URL.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 6
✅ Passed checks (6 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding documentation that explains the 'No default base_url' config-load error in the migration guide. It directly corresponds to the new section added to the migration documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Test Results For Major Changes ✅ Passed Documentation-only change with no code modifications. Per check instructions, minor changes pass without requiring test results.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/vdr-format-b-error-message

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

Copy link
Copy Markdown
Contributor

Documentation preview

https://nvidia-nemo.github.io/Guardrails/review/pr-1881

@greptile-apps

greptile-apps Bot commented May 13, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds a new "Unsupported Engine on the Default Framework" section to the v0.22 migration guide and updates the summary table to reference both remediation paths (LangChain opt-in or OpenAI-compatible base_url), improving discoverability for users hitting the ValueError: No default base_url for provider '...' error.

  • New section quotes the ValueError verbatim and explains two remediation paths: keeping the LangChain provider or switching to an OpenAI-compatible endpoint.
  • Summary table updated to link to the new section instead of only pointing at the LangChain section, surfacing the OpenAI-compatible path at a glance.

Confidence Score: 5/5

Documentation-only change that adds a new section and updates a table row — no executable code is touched.

The change is confined to a single Markdown file. The new section quotes the error message that already exists in troubleshooting.md (with the ValueError: prefix now present and consistent), anchor links resolve correctly against the existing headings, and both remediation paths are accurate and already described elsewhere in the guide. No logic, configuration, or runtime behaviour is affected.

No files require special attention.

Important Files Changed

Filename Overview
docs/migration/0.22.md Adds a new "Unsupported Engine on the Default Framework" section with the verbatim error message and two remediation bullets; updates the summary table row for unsupported engines to reference the new section. No inconsistencies found.

Reviews (3): Last reviewed commit: "docs: edit pass" | Re-trigger Greptile

Comment thread docs/migration/0.22.md Outdated
Apply two reviewer suggestions on the Unsupported-Engine section:

- Prefix the verbatim error with `ValueError:` to match how
  troubleshooting.md (line 76) renders the same error, so a traceback-
  paste search hits both reference docs.
- Update the migration-summary table row for non-OpenAI-compatible
  engines to point at the new section title and to mention the
  "switch to OpenAI-compatible" remediation alongside the LangChain
  opt-in. The new section already cross-links both fix paths
  internal
@Pouyanpi Pouyanpi added the VDR label May 13, 2026
@Pouyanpi Pouyanpi added this to the v0.22.0 milestone May 13, 2026
@Pouyanpi Pouyanpi self-assigned this May 13, 2026
@miyoungc
miyoungc merged commit 16be99a into develop May 13, 2026
4 checks passed
@miyoungc
miyoungc deleted the fix/vdr-format-b-error-message branch May 13, 2026 23:07
@miyoungc miyoungc mentioned this pull request May 20, 2026
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants