Card Builder + Agentic Card Messages#411
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
Docs review — ✅ Approve (two nits to fix)A large all-new docs PR (24 files, +2192/-0): a new Card Builder section plus agentic card-message pages across 8 agent frameworks, 9 image/video assets, and nav wiring. Nothing is moved or deleted, so there's no redirect/404 risk. I verified nav integrity, every cross-link and anchor, all asset refs, the shared-snippet pattern, and the formal JSON Schema — everything checks out. ✅ What passed (verified)
🔧 Nits to fix before merge
Otherwise this is excellent, well-structured work and is ready to merge once the two nits are addressed. 🤖 Automated docs review (Mintlify link/redirect/content checks + JSON-Schema validation). |
The root-structure example and the colors fragment aren't copy-paste-valid JSON (one uses a /* ... */ comment, the other is a bare key-value fragment). Retag their fences from json to jsonc so they don't read as literal, runnable JSON. Addresses review nit on PR #411; Full JSON Schema block stays json. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Internal links used a root-relative /docs/ prefix that 404s on the Mintlify site (pages live at /ai-agents/...). Strip the prefix on 7 hrefs across ai-agents.mdx and ai-agents/agent-builder/overview.mdx (5 user-facing on /ai-agents and /ai-agents/agent-builder/overview, plus 2 in a commented-out Step). Canonical and full https://.../docs/ URLs left intact. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
674966a
jitvarpatil
left a comment
There was a problem hiding this comment.
Re-review — ✅ Approve
Both nits from the earlier review are fixed:
- Illustrative JSON retagged: the two non-literal snippets in
card-builder/schema.mdxare now```jsonc, while the valid 1,563-line "Full JSON Schema" block stays```json(correct). - Links fixed: the ai-agents card-message internal links now resolve (dropped the stray
/docsprefix). Analyzer: 0 build breaks, 0 orphans, 0 broken links across both card-builder and ai-agents scopes.
Structure, nav wiring, all 9 assets, the shared-snippet pattern, and the formal JSON Schema were already verified. Ready to merge.
🤖 Automated docs review — re-check after fixes.
Description
Related Issue(s)
Type of Change
Checklist
Additional Information
Screenshots (if applicable)