Uh oh!
There was an error while loading. Please reload this page.
docs(ui-kit/angular): add scoped Angular v5 LLM docs index (ENG-38204) - #471
Closed
shagundeogharkar wants to merge 1 commit into
Closed
docs(ui-kit/angular): add scoped Angular v5 LLM docs index (ENG-38204)#471shagundeogharkar wants to merge 1 commit into
shagundeogharkar wants to merge 1 commit into
Conversation
Adds ui-kit/angular/llms-angular-v5.mdx — a machine-readable, Angular-v5-scoped routing index of all 91 v5 pages as .md twins, for AI coding agents. Mirrors the shape of ui-kit/react/llms-react-v7.mdx (branch docs/react-v7-feature-guides). - Unlisted, NOT hidden: omitted from docs.json navigation so it never shows in the human sidebar, but still built, served as a .md twin, and indexed. Using `hidden: true` would auto-apply noindex and drop it from search + the global llms.txt, which defeats the purpose. docs.json is deliberately untouched. - Scoped to v5 only; the 2.0/, 3.0/ and v4/ trees are excluded so agents are never routed at dead API surfaces. - Angular-specific framing the React index has no equivalent for: kebab-case selectors, @input() rather than props, content-projection/TemplateRef rather than render props, and env config in src/environments/environment.ts. Also fixes four content defects surfaced while building the index: - api-reference/formatter-config-service.mdx, api-reference/ rich-text-editor-service.mdx and guides/rich-text-formatting.mdx shipped with NO frontmatter at all despite being in docs.json navigation, so they rendered untitled. Adds title/description per house style (see api-reference/chat-state-service.mdx) and drops the two leading H1s that would now duplicate the frontmatter title. - overview.mdx "AI Integration Quick Reference" listed peer deps as @cometchat/chat-sdk-javascript + dompurify, missing @cometchat/cards-angular@^1.0.0 which @cometchat/chat-uikit-angular@5.1.0 added. Verified against the published package. NOT fixed here, needs an owner decision: the same accordion claims Angular "v18, v19, v20, v21, v22" but the published peer range at 5.1.0 is @angular/core ">=17.0.0 <22.0.0" — v22 is excluded (install hard-fails with ERESOLVE) and v17 is supported but undocumented. Either the docs or the peer range is wrong; that is a support-policy call, not a typo. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
6 tasks
shagundeogharkar
commented
Aug 19, 2026
ContributorAuthor
Superseded by #475 — this branch is merged into |
anshuman-cometchat added a commit
to anshuman-cometchat/docs
that referenced
this pull request
Aug 25, 2026
Flutter was the only platform with none. On the skills-v5-temp base, react has 128 Quick References and 3 llms indexes, android 112/2, angular 81/1, react-native 56/2, ios 34/2 - and flutter 0 and 0. This closes the index half. Structure and conventions follow the reference PRs for the same feature: cometchat#446 (React v7), cometchat#466 (JS SDK), cometchat#471 (Angular v5), cometchat#476 (React Native). Unlisted rather than hidden, for the reason those PRs give: in Mintlify hidden auto-applies noindex, which would drop the page from search and from the auto global llms.txt - and the whole point is that an agent can discover it. Not registered in docs.json, same as every prior index. ui-kit/flutter/llms-flutter-v6.mdx 70 links, all 54 UI Kit pages sdk/flutter/llms-flutter-v5.mdx 58 links, all 52 SDK pages Both are 100 percent page coverage with zero dead links, verified by resolving every href against the tree. The Platform rules section is the part that carries real weight, and every claim in it was verified against cometchat_chat_uikit 6.1.0 rather than recalled: - TWO barrels with different surfaces. Chat widgets do not resolve from the calls barrel and vice versa, so a screen showing both imports both. This is the most common Flutter-specific compile failure. - Lists need a bounded box or layout throws at render, not at build. - Kit widgets paint their own surface; the app ThemeData does not reach inside. - Theming is ThemeExtension, and registering on only one of light/dark silently leaves the other on kit defaults. - v5's CometChatUIKit.getDataSource() is gone; v6 uses MessageTemplateUtils. - Custom message types need addTemplate, not templates - templates only registers the bubble and the message is filtered out before it can render, with no error. A hand-rolled MessagesRequestBuilder does not help because the list always overrides uid/guid/types/categories on it. - Messages sent with CometChat.send*Message must emit ccMessageSent or a mounted list never shows them. - SDK: onSuccess AND onError are both required - compile-checked, omitting onError is missing_required_argument. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
anshuman-cometchat added a commit
to anshuman-cometchat/docs
that referenced
this pull request
Aug 25, 2026
…ent pages Flutter had 0 of these while react has 128, android 112, angular 81, react-native 56 and ios 34. This closes the UI Kit half. Format follows the reference PRs (cometchat#446, cometchat#466, cometchat#471, cometchat#476): an Accordion straight after the frontmatter, a Field/Value table, and rows that let an agent decide whether the page is worth opening at all. Generated from the compiler-verified prop tables rather than hand-written, so every prop named here provably exists on the widget and the row cannot drift from the kit on the next release. 35 in-page anchors, all resolving. Two rows are Flutter-specific and are the reason a generic template would not have done: - Import carries the RIGHT BARREL per widget. Calling widgets resolve only from cometchat_calls_uikit.dart, so call-buttons, incoming-call, outgoing-call and call-logs also get an explicit Barrel row saying so. Importing a calling widget from the chat barrel is the most common Flutter compile failure and no other platform has this split. - Layout warns that list widgets fill their parent and need an Expanded or a sized box, because the failure is an unbounded-height throw at render rather than a build error. Plus the two traps found by building on the kit: message-list carries the addTemplate-not-templates rule, and message-composer carries the ccMessageSent requirement for messages sent outside it. Classification is by TYPE, not name, after three passes got it wrong: messagesRequestBuilder ends in Builder but is data, hideThreadView ends in View but is a bool toggle, headerView is a HeaderFooterBuilder typedef with neither Widget nor Function in its name, and onError is typed OnError? so only the on-plus-capital convention identifies it. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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 freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
What & why
Part of ENG-38204 (Angular skills — UI Kit + SDK). That ticket's blocking prerequisite is a scoped
llms.txt-style index for Angular, because the skills pack is docs-maximal: it builds features from the official task guides and compares its output against them. Without a scoped index there is no way for an agent to discover which guides exist, so it silently falls back to component-wiring and loses the guide-first completeness check.Adds
ui-kit/angular/llms-angular-v5.mdx— a machine-readable, Angular-v5-scoped routing index covering all 91 v5 pages as.mdtwins, in 13 sections. It mirrors the shape ofui-kit/react/llms-react-v7.mdx(currently on branchdocs/react-v7-feature-guides, not yet onmain).No Angular SDK index is needed. The ticket asks for two indexes, but Angular consumes the same
@cometchat/chat-sdk-javascript@4as React, and the existingsdk/javascript/llms-javascript-v4.mdxis framework-agnostic — Angular inherits it unchanged.Changes
New —
ui-kit/angular/llms-angular-v5.mdxdocs.jsonnavigation so it never appears in the human sidebar, but it still builds, serves its.mdtwin, and stays indexed.hidden: truewas not used because Mintlify auto-appliesnoindexto hidden pages, which would drop this page from search and from the auto-generated globalllms.txt— defeating the purpose.docs.jsonis untouched, so there are no navigation refs to break and no redirects required.2.0/,3.0/andv4/trees are excluded so agents are never routed at dead API surfaces.<cometchat-conversations>),@Input()rather than props, content-projection /TemplateRefrather than render props, and environment config insrc/environments/environment.tsrather than.env.Fixes — four content defects surfaced while building the index
Three pages shipped with no frontmatter at all, despite being present in
docs.jsonnavigation, so they render untitled on the live site. Addedtitle/descriptionper house style (cf.api-reference/chat-state-service.mdx), and removed the two leadingH1s that would now duplicate the frontmatter title:api-reference/formatter-config-service.mdxtitle: "Formatter Config Service"api-reference/rich-text-editor-service.mdx# RichTextEditorService API Referencetitle: "Rich Text Editor Service"guides/rich-text-formatting.mdx# Rich Text Formatting Guidetitle: "Rich Text Formatting"overview.mdx— "AI Integration Quick Reference" listed incomplete peer deps.@cometchat/chat-sdk-javascript,dompurify@cometchat/chat-sdk-javascript,@cometchat/cards-angular,dompurifyGround truth (verified against the shipped package)
The peer-dep fix was confirmed against the published package, not inferred:
@cometchat/cards-angularwas added between5.0.3and5.1.0(latestis5.1.0; there is no v6 line). Anyone following the docs today gets an incomplete install.Not fixed here — needs an owner decision
The same
overview.mdxaccordion states:But the published peer range at
5.1.0is@angular/core ">=17.0.0 <22.0.0":npm install @cometchat/chat-uikit-angular@^5on an Angular 22 project hard-fails withERESOLVE. The docs currently advertise a version where install is impossible.So either the docs or the kit's peer range is wrong. That is a support-policy call for the UI Kit owners, not a typo, so this PR leaves the line untouched and flags it instead. It also blocks the Angular
coreskill, which has to state a supported range.Checklist
docs/angular-v5-llms-index)docs.jsonuntouched — no nav refs changed, no redirects neededui-kit/angularfile tree, with page titles read from each file's frontmatternpx mintlify devnot yet run — before merge, please confirm the page builds (a "not in navigation" warning is expected and intended) and that/ui-kit/angular/llms-angular-v5.mdresolves as raw Markdown. The design depends on that twin being fetchable.Follow-ups (not in this PR)
components/cometchat-*.mdxpages. The index tells agents each.mdtwin carries input/output names, types and defaults. That holds forguides/andintegration.mdx, but the component pages have no accordion yet, so the promise is currently partial. The React equivalent added these across 56 files in the same branch; for Angular it is deliberately split out as a separate, mechanical PR.docs/react-v7-feature-guides,docs/js-sdk-llms-index). Merging this one puts Angular's index in production while React's stays on a preview deploy — worth deciding whether those two should land as well, since the React skills pack currently documents.mintlify.sitepreview URLs as its docs base.