Uh oh!
There was an error while loading. Please reload this page.
added requested changes - #11
Conversation
📝 WalkthroughWalkthroughAdds five new usage subsections to ChangesClaude Skill Documentation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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: 1
🤖 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 `@integrations/claude-skill.mdx`:
- Around line 170-188: The domain-classifier docs are inconsistent with the
actual response shape for zlm-v1-iab-domain-classifier: the prose mentions
categories, topics, keywords, and intent signals, but the example only shows
categories. Update the Claude skill section to match the real payload by either
expanding the example to include the missing fields or narrowing the description
so the documented output aligns with the returned object.
🪄 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
Run ID: 9fc54cc4-b4f3-4c18-992e-e14c21c6d563
📒 Files selected for processing (2)
integrations/claude-skill.mdxintegrations/index.mdx
| Map a bare domain to IAB categories, topics, and intent without fetching the page - the input is just the hostname, which keeps the payload tiny for bidstream and contextual targeting. | ||
| - **Model:** `zlm-v1-iab-domain-classifier` | ||
| - **Triggers on:** "what categories is this domain?", "classify indeed.com for targeting." | ||
| ```text | ||
| Classify this domain with ZeroGPU: | ||
| "indeed.com" | ||
| ``` | ||
| **Example output (returned after API call)** | ||
| ```json | ||
| { | ||
| "domain": "indeed.com", | ||
| "categories": [ | ||
| { "id": "IAB4", "name": "Careers", "confidence": 0.96 } | ||
| ] | ||
| } |
There was a problem hiding this comment.
🗄️ Data Integrity & Integration | 🟡 Minor | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/bin/bashset -euo pipefail
echo"== IAB/domain model references =="
rg -n -C2 'zlm-v1-iab-domain-classifier|zlm-v1-iab-classify-edge|zlm-v1-iab-classify-edge-enriched' \
integrations/claude-skill.mdx integrations/langchain.mdx SKILL.md 2>/dev/null ||trueechoecho"== IAB tool docs and documented return shapes =="
rg -n -C3 'ZeroGPUClassifyIAB|topics|intent|keywords|Returns:' \
integrations/langchain.mdx integrations/claude-skill.mdx SKILL.md 2>/dev/null ||trueechoecho"== Files that should advertise routed models =="
fd -i '(^SKILL\.md$|claude-skill\.mdx$|langchain\.mdx$|model-catalog)'.Repository: zerogpu/docs
Length of output: 17493
🏁 Script executed:
#!/bin/bashset -euo pipefail
echo"== model catalog references =="
rg -n -C2 'zlm-v1-iab-domain-classifier|zlm-v1-iab-classify-edge-enriched|zlm-v1-iab-classify-edge' docs/model-catalog.mdx integrations/claude-skill.mdx integrations/langchain.mdx 2>/dev/null ||trueechoecho"== claude-skill domain section =="
sed -n '168,190p' integrations/claude-skill.mdx
echoecho"== model catalog IAB-related entries =="
rg -n -C3 'IAB|domain classifier|classify-edge|classify_iab' docs/model-catalog.mdx 2>/dev/null ||trueRepository: zerogpu/docs
Length of output: 20188
Document the full domain-classifier response shape. The model catalog lists zlm-v1-iab-domain-classifier as returning categories, topics, keywords, and intent signals, but this section only promises categories/topics/intent and the example shows categories only. Either add the missing fields to the example or trim the description to the actual payload.
🤖 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 `@integrations/claude-skill.mdx` around lines 170 - 188, The domain-classifier
docs are inconsistent with the actual response shape for
zlm-v1-iab-domain-classifier: the prose mentions categories, topics, keywords,
and intent signals, but the example only shows categories. Update the Claude
skill section to match the real payload by either expanding the example to
include the missing fields or narrowing the description so the documented output
aligns with the returned object.
Uh oh!
There was an error while loading. Please reload this page.
In
integrations/claude-skill.mdx:In
integrations/index.mdx:Summary by CodeRabbit