Conversation
![ellipsis-dev[bot]](https://avatars.githubusercontent.com/in/64358?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Important
Looks good to me! 👍
Reviewed everything up to c044440 in 12 seconds. Click for details.
- Reviewed
996lines of code in4files - Skipped
0files when reviewing. - Skipped posting
0draft 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.
Workflow ID: wflow_cREXtvCoTYytmTGt
You can customize 
Contributor
📝 WalkthroughWalkthroughThis PR introduces comprehensive guardrails documentation to the Traceloop Hub by adding a new "Guardrails" navigation section with three pages (overview, configuration, evaluators), updating existing guardrails documentation with implementation modes and YAML examples, and establishing detailed evaluator specifications. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~18 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In `@hub/guardrails-configuration.mdx`:
- Around line 75-109: Update all broken evaluator links that point to
/hub/guardrails-evaluators to the correct slug /hub/evaluators; specifically
replace the link target on the "Evaluators Reference" anchor used in the YAML
examples and any other occurrences (e.g., the inline reference after
evaluator_slug and other docs blocks) so that references like "Evaluators
Reference" resolve to /hub/evaluators across the document (also fix the same
links in the repeated section covering lines 206-315).
In `@hub/overview.mdx`:
- Around line 213-219: The "Evaluators Reference" Card within the CardGroup uses
the wrong href; update the Card with title "Evaluators Reference" (the Card
component instance in the snippet) to use href="/hub/evaluators" instead of
href="/hub/guardrails-evaluators" so the link matches the navigation slug;
verify the CardGroup/Card markup is unchanged and only the href value is
corrected.
doronkopit5
reviewed
Feb 16, 2026
Member
There was a problem hiding this comment.
this is the overview of the whole hub???
doronkopit5
reviewed
Feb 16, 2026
Member
There was a problem hiding this comment.
should be part of the existing configuration page
Contributor
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@evaluators/guardrails.mdx`:
- Around line 68-74: The term "Database Mode" is misleading and should be
renamed to "Dashboard Mode": update the list item text and the section heading
"Database Mode" to "Dashboard Mode" in the content referencing evaluators
configured in the Traceloop dashboard, and update any nearby references (e.g.,
the paragraph that starts "Evaluators configured in Traceloop dashboard, applied
via SDK decorators..." and the "### Database Mode" heading) so they consistently
use "Dashboard Mode".
In `@hub/guardrails/overview.mdx`:
- Around line 226-229: Update the Card component's href to the new navigation
slug: change the href prop value on the Card inside CardGroup (the Card with
title "Configuration Guide") from "/hub/guardrails-configuration" to
"/hub/guardrails/configuration" so the link points to the correct page.
---
Duplicate comments:
In `@hub/guardrails/overview.mdx`:
- Around line 230-231: Update the Card component with title "Evaluators
Reference" to point to the new guardrails slug by changing its href value from
"/hub/evaluators" to "/hub/guardrails/evaluators" (locate the Card element in
overview.mdx and update the href attribute).
Comment on lines
+68
to
+74
| Guardrails can be implemented in two modes: | ||
|
|
||
| 1. **Database Mode** - Evaluators configured in Traceloop dashboard, applied via SDK decorators in your application code (shown below) | ||
| 2. **Config Mode** - Available in Traceloop Hub v1, guardrails and evaluators fully defined in YAML (see [Config Mode Guardrails](#config-mode-guardrails-v1)) | ||
|
|
||
| ### Database Mode | ||
|
|
Contributor
There was a problem hiding this comment.
Rename “Database Mode” to “Dashboard Mode” for clarity.
Line 70 and Line 73 describe dashboard-configured evaluators, so “Database Mode” reads like a typo.
🛠️ Proposed fix
-1. **Database Mode** - Evaluators configured in Traceloop dashboard, applied via SDK decorators in your application code (shown below)
+1. **Dashboard Mode** - Evaluators configured in Traceloop dashboard, applied via SDK decorators in your application code (shown below)
-### Database Mode
+### Dashboard Mode🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@evaluators/guardrails.mdx` around lines 68 - 74, The term "Database Mode" is
misleading and should be renamed to "Dashboard Mode": update the list item text
and the section heading "Database Mode" to "Dashboard Mode" in the content
referencing evaluators configured in the Traceloop dashboard, and update any
nearby references (e.g., the paragraph that starts "Evaluators configured in
Traceloop dashboard, applied via SDK decorators..." and the "### Database Mode"
heading) so they consistently use "Dashboard Mode".
Comment on lines
+226
to
+229
| <CardGroup cols={2}> | ||
| <Card title="Configuration Guide" icon="gear" href="/hub/guardrails-configuration"> | ||
| Learn how to configure guardrails with complete YAML reference | ||
| </Card> |
Contributor
There was a problem hiding this comment.
Fix Configuration Guide link path.
Line 227 points to /hub/guardrails-configuration, but the new navigation slug is /hub/guardrails/configuration. This will 404.
🛠️ Proposed fix
- <Card title="Configuration Guide" icon="gear" href="/hub/guardrails-configuration">
+ <Card title="Configuration Guide" icon="gear" href="/hub/guardrails/configuration">🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@hub/guardrails/overview.mdx` around lines 226 - 229, Update the Card
component's href to the new navigation slug: change the href prop value on the
Card inside CardGroup (the Card with title "Configuration Guide") from
"/hub/guardrails-configuration" to "/hub/guardrails/configuration" so the link
points to the correct page.
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
2 participants
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.
Important
Adds documentation for Traceloop Hub guardrails, including an overview, configuration guide, and evaluators reference, with updates to navigation in
mint.json.hub/overview.mdxfor real-time safety and quality checks overview.hub/guardrails-configuration.mdxfor configuring guardrails in Traceloop Hub.hub/evaluators.mdxfor a reference of 12 built-in evaluators.pre_call,post_call, or both modes.mint.jsonto include new documentation pages under "Guardrails" group.This description was created by
for c044440. You can customize this summary. It will automatically update as commits are pushed.
Summary by CodeRabbit
New Features
Documentation