Skip to content

Card Builder + Agentic Card Messages#411

Open
mathews-cometchat wants to merge 68 commits into
mainfrom
feature/card-builder
Open

Card Builder + Agentic Card Messages#411
mathews-cometchat wants to merge 68 commits into
mainfrom
feature/card-builder

Conversation

@mathews-cometchat

@mathews-cometchat mathews-cometchat commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

Description

Related Issue(s)

Type of Change

  • Documentation correction/update
  • New documentation
  • Improvement to existing documentation
  • Typo fix
  • Other (please specify)

Checklist

  • I have read the CONTRIBUTING document
  • My branch name follows the naming convention
  • My changes follow the documentation style guide
  • I have checked for spelling and grammar errors
  • All links in my changes are valid and working
  • My changes are accurately described in this pull request

Additional Information

Screenshots (if applicable)

@mintlify

mintlify Bot commented Jun 28, 2026

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
cometchat 🟢 Ready View Preview Jun 28, 2026, 7:44 AM

@jitvarpatil

jitvarpatil commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

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)

  • Build / nav: 0 unresolved nav refs (build-safe). All 5 card-builder + 8 ai-agents card-message pages are wired into docs.json. The leading-slash format on the ai-agents entries matches the ai-agents section's established convention (the whole tree uses leading slashes; 70 such entries repo-wide), while card-builder uses the no-slash form of its neighbors — both internally consistent.
  • Orphans: 0 new orphans (the analyzer's 38 ai-agents orphans are all pre-existing API pages; none of this PR's 13 new pages are orphaned).
  • Links & anchors: all 12 internal cross-links resolve (incl. /ai-agents/cometchat-ag-ui-byoa, /ai-agents/agent-builder/tools/overview); all 5 anchor fragments map to real headings (#6-use-variables, #8-save, #actions, #components, #variables).
  • Assets: all 9 added images/videos resolve; the 5 build-*.mp4 walkthrough videos appear in order in the worked example.
  • Shared-snippet pattern: the 8 framework pages are correctly near-identical — each imports /snippets/ai-agents/card-messages.mdx and differs only by the framework name in description.
  • Formal schema: the "Full JSON Schema" block is a valid 1,563-line JSON Schema (draft-07, parses cleanly with proper $schema/required/properties/definitions). Element-type vocabulary is consistent between elements.mdx and schema.mdx.
  • No placeholders/TODOs/lorem.

🔧 Nits to fix before merge

  • card-builder/schema.mdx (~L24 and ~L41): two ```json blocks aren't copy-paste-valid JSON — one uses a /* one or more elements */ comment, the other is a single-field fragment ("background": { … }). They're clearly illustrative, but please retag them ```jsonc (or plain ```) so they don't read as literal, runnable JSON.
  • docs.json — "Card Builder" tab is "hidden": true: reachable only via deep links (from the card-message pages and the <CardGroup> cards), not the top nav bar. Please confirm this is intentional (linked-only launch) and not an accidental leftover.

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>
aaryan-rawat
aaryan-rawat previously approved these changes Jul 2, 2026
Afroz-Cometchat
Afroz-Cometchat previously approved these changes Jul 2, 2026
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>

@jitvarpatil jitvarpatil left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Re-review — ✅ Approve

Both nits from the earlier review are fixed:

  • Illustrative JSON retagged: the two non-literal snippets in card-builder/schema.mdx are 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 /docs prefix). 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.

jitvarpatil
jitvarpatil previously approved these changes Jul 2, 2026
hritika-cometchat and others added 13 commits July 24, 2026 16:00
- Add prop to AI Quick Reference JSON
- Add prop entry in Props section (boolean, default: false)
- Enables automatic loading of the most recent agent conversation
…ageList

- Add loadLastAgentConversation prop documentation to props table with type, default, and description
- Remove duplicate loadLastAgentConversation entry from optional props section
- Add new code example demonstrating how to use loadLastAgentConversation to resume recent agent conversations
- Consolidate prop definitions to eliminate redundancy and improve documentation clarity
…pages to nav

- Replace /ui-kit/react-native/getting-started (404) with /ui-kit/react-native/overview in campaigns.mdx and property-changes.mdx
- Add calling-integration to Features group in docs.json (before call-features)
- Add property-changes to Migration Guide group in docs.json

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
MDX does not allow <!-- --> syntax, causing parse failures on deploy.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
aanshisingh-cometchat and others added 24 commits July 24, 2026 16:03
Point the React row in the render note at /ui-kit/react/v6/card-messages,
matching the Angular row's /ui-kit/angular/guides/card-messages link so both
frameworks reference their own UI Kit Card Messages guide.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Simplify the render note to reference the React and Angular UI Kit Card
Messages guides only. Removes the npm package links and the vanilla
@cometchat/cards bullet (no docs guide to point to).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Update the React row in the render note to /ui-kit/react/v7/card-messages
(current React UI Kit major) instead of v6.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Add documentation for the v7 React UI Kit card message feature (category
"card"), rendered by the CometChatCardBubble plugin over the prebuilt
CometChatCardView renderer:

- components/card-bubble.mdx: new page for CometChatCardBubble (props,
  card actions, fallback behavior, CSS selectors) with the message
  hierarchy diagram
- event-system.mdx: add the message/card-received SDK event and the
  ui:card/action UI event (Card Actions section + quick reference)
- plugins/overview.mdx: add the built-in Card plugin row
- components-overview.mdx: add the CometChatCardBubble row
- docs.json: register the card-bubble page in the sidebar nav

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Add React v6 documentation for render-only Card Message support:
- card-bubble.mdx: CometChatCardBubble component reference
- card-messages.mdx: guide covering developer, persisted agent, and
  streaming agent card delivery paths and card action handling
- events.mdx: add onCardMessageReceived and ccCardActionClicked
- docs.json: register both pages under React v6 nav

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…nt chat

- Remove outdated limitation note about AI agents only responding in one-on-one conversations
- Remove note about group support being on the roadmap
- Delete picture-in-picture.mdx page
- Remove PiP actions from actions.mdx
- Drop PiP method renames and layout events from v5 migration guide
- Remove PiP entry from docs.json navigation

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- Rename unMuteAudio() to unmuteAudio() across all integration guides
- Replace login() with loginWithAuthToken() in v5 migration guide
- Rename showSwitchModeButton() to showModeButton() in builder/mapping
- Replace onRecordingToggled with onRecordingStarted/onRecordingStopped events
- Remove onAudioModeChanged listener and hideSwitchLayoutButton setting
- Drop the join-by-session-ID option, leaving token-based joinSession
- Update idleTimeoutPeriodAfterPrompt default from 120000 to 180000
- Update hideRecordingButton and hideNetworkIndicator example defaults
- Use onParticipantJoined in troubleshooting guidance
- Unpin @cometchat/calls-sdk-react-native version in setup install steps

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- Document setAudioMode() and participant-list panel methods
- Fix onAudioModeChanged payload as a string, not an object
- Correct Android foreground-service class, types, and permissions
- Add onConnectionFailed event listener
- Fix idle-timeout default values and descriptions
- Update migration guide: loginWithAuthToken(), setCallEventListener(),
  showSwitchCameraButton(), recording/audio-mode listener renames,
  switchToVideoCall() now a deprecated no-op
- Document header, status-indicator, identity, and behavior session settings
- Clarify join flow uses component rendering, not joinSession()

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- Add Install Required Dependencies section to RN Calls setup guide
- List async-storage, background-timer, performance, svg, url-polyfill, and webrtc with pinned versions
- Provide npm and Yarn install commands

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…-37141]

Add nav entries and OpenAPI-backed MDX pages for three thread endpoints
under Chat & Messaging → Messages:
- GET  /threads                              (List Threads)
- POST /messages/{id}/thread/subscription    (Subscribe Thread)
- DELETE /messages/{id}/thread/subscription  (Unsubscribe Thread)
…} + operationId typo fixes [ENG-37141]

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.