Docs/android llms indexes - #481

Merged
raj-dubey1 merged 5 commits into
docs/skills-v5-tempfrom
docs/android-llms-indexes
Aug 25, 2026
Merged

Docs/android llms indexes#481
raj-dubey1 merged 5 commits into
docs/skills-v5-tempfrom
docs/android-llms-indexes

Conversation

@hritika-cometchat

@hritika-cometchathritika-cometchat commented Aug 24, 2026•
edited
Loading

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

mintlifyBot commented Aug 24, 2026•
edited
Loading

Copy link
Copy Markdown

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

ProjectStatusPreviewUpdated (UTC)
cometchat🟢 ReadyView PreviewAug 24, 2026, 6:22 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

raj-dubey1
raj-dubey1 reviewed Aug 25, 2026
View reviewed changes

@raj-dubey1raj-dubey1 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.

Docs review — Android LLM indexes (ENG-38205 / PR #175 companion)

Reviewed from the skills pack perspective: does this PR give the Android skills the docs surface they depend on?

✅ What's correct and solid

llms-android-v6.mdx — "unlisted not hidden" pattern correct (explicit rationale comment, no hidden: true, not in docs.json). Both cohorts covered prominently: "Two UI flavors of ONE kit: chatuikit-kotlin-android:6.+ (XML Views) and chatuikit-compose-android:6.+ (Jetpack Compose)". The hot-path section links both cohort entry points. The "How to use this index" section tells the agent to read the matching tab (Views vs Compose) on every component page. This is the right approach — keeping a single index that reflects the single-kit architecture.

Both-cohort cross-links complete: v6 UI Kit index → v5 SDK index, and v5 SDK index → v6 UI Kit index. getting-started-kotlin.mdx and getting-started-jetpack.mdx are both linked from the hot path.

Pending pages documented: The three features gated on feature/android-pin_save_thread (pinned-messages, saved-messages, guide-thread-subscription) are listed in a JSX comment block with the branch name and the instruction to uncomment when that branch merges. This matches the PENDING section in features.android-v6.json — the two files are in sync.

llms-android-v5.mdx (SDK) — present and cross-linked to the UI Kit index.

Docs corrections across 14+ pages: The ANDROID-DOCS-AUDIT.md methodology (mechanical diff of docs vs 6.0.5 source) produced 5 verified wrong method names in the live docs (A1–A5: setOnAcceptClick vs setOnAcceptClickListener, etc.). These align with the STOPGAP blocks in the skills pack core skill — once these corrections merge, those stopgaps can be removed.

⚠️ Items to address

1. PR description is very thin for the scope of changes
This PR touches 14+ existing docs pages, adds two index files, includes ANDROID-DOCS-AUDIT.md (documenting 5 wrong method names in production docs), and adds REPRO-enableCalling-launch-crash.md. The PR body is the default template with checkboxes but almost no description of what was corrected or why. For a PR this size, reviewers have no way to know which method names were fixed, which pages were corrected, or what the REPRO file documents.

Please add a PR body summary similar to PR #477's format — at minimum: a list of corrected pages, the method name corrections (equivalent to the A1–A5 list in ANDROID-DOCS-AUDIT.md), and a note about the enableCalling crash repro. This is a review-process issue, not a correctness issue, but it makes the PR significantly harder to review.

2. Confirm STOPGAP removal from skills pack after merge
The cometchat-android-v6-core/SKILL.md in PR #175 has two STOPGAP blocks that are explicitly tied to this PR merging. Merge order: PR #481 first → then remove the STOPGAP blocks from PR #175. If both merge simultaneously or in reverse order, the stopgap blocks will remain in the published skills indefinitely. Coordinate the merge order.

3. AUDIT-084 in the index
The index's hot-path section correctly points to getting-started-kotlin / getting-started-jetpack where AUDIT-084 wiring lives. Confirm both getting-started pages explicitly use CometChatUIKit.initFromSettings(this, callback) (not the classic UIKitSettingsBuilder + init()) — this is what the skills pack's contracts.android-v6.json verifies.

Summary

The two LLM indexes are well-structured, correctly unlisted, and cover both cohorts in the right way (single index, per-cohort tabs in linked pages). The docs corrections align with the oracle findings in the skills pack. Two procedural items before this lands cleanly: add a meaningful PR description, and coordinate merge order with PR #175 (docs first so the skill STOPGAP blocks can be removed).

APPROVE — with the PR description and merge-order notes above.

@raj-dubey1
raj-dubey1 changed the base branch from main to docs/skills-v5-tempAugust 25, 2026 08:39
@raj-dubey1
raj-dubey1 merged commit 6e09c78 into docs/skills-v5-tempAug 25, 2026
3 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@raj-dubey1raj-dubey1raj-dubey1 left review comments
@jitvarpatiljitvarpatilAwaiting requested review from jitvarpatiljitvarpatil is a code owner

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@hritika-cometchat@raj-dubey1
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content
Docs/android llms indexes - #481
Merged
raj-dubey1 merged 5 commits into
docs/skills-v5-tempfrom 
docs/android-llms-indexes
Aug 25, 2026
Merged
Docs/android llms indexes#481
raj-dubey1 merged 5 commits into
docs/skills-v5-tempfrom 
docs/android-llms-indexes
Conversation
@hritika-cometchat
@hritika-cometchathritika-cometchat
commented
Aug 24, 2026•

edited

Loading

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
mintlifyBot
commented
Aug 24, 2026•

edited

Loading

Copy link


Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.
ProjectStatusPreviewUpdated (UTC)
cometchat🟢 ReadyView PreviewAug 24, 2026, 6:22 AM
💡 Tip: Enable Workflows to automatically generate PRs for you.
raj-dubey1
raj-dubey1
reviewed
Aug 25, 2026
View reviewed changes
@raj-dubey1raj-dubey1
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.

Docs review — Android LLM indexes (ENG-38205 / PR #175 companion)
Reviewed from the skills pack perspective: does this PR give the Android skills the docs surface they depend on?
✅ What's correct and solid
llms-android-v6.mdx — "unlisted not hidden" pattern correct (explicit rationale comment, no hidden: true, not in docs.json). Both cohorts covered prominently: "Two UI flavors of ONE kit: chatuikit-kotlin-android:6.+ (XML Views) and chatuikit-compose-android:6.+ (Jetpack Compose)". The hot-path section links both cohort entry points. The "How to use this index" section tells the agent to read the matching tab (Views vs Compose) on every component page. This is the right approach — keeping a single index that reflects the single-kit architecture.
Both-cohort cross-links complete: v6 UI Kit index → v5 SDK index, and v5 SDK index → v6 UI Kit index. getting-started-kotlin.mdx and getting-started-jetpack.mdx are both linked from the hot path.
Pending pages documented: The three features gated on feature/android-pin_save_thread (pinned-messages, saved-messages, guide-thread-subscription) are listed in a JSX comment block with the branch name and the instruction to uncomment when that branch merges. This matches the PENDING section in features.android-v6.json — the two files are in sync.
llms-android-v5.mdx (SDK) — present and cross-linked to the UI Kit index.
Docs corrections across 14+ pages: The ANDROID-DOCS-AUDIT.md methodology (mechanical diff of docs vs 6.0.5 source) produced 5 verified wrong method names in the live docs (A1–A5: setOnAcceptClick vs setOnAcceptClickListener, etc.). These align with the STOPGAP blocks in the skills pack core skill — once these corrections merge, those stopgaps can be removed.
⚠️ Items to address
1. PR description is very thin for the scope of changes

This PR touches 14+ existing docs pages, adds two index files, includes ANDROID-DOCS-AUDIT.md (documenting 5 wrong method names in production docs), and adds REPRO-enableCalling-launch-crash.md. The PR body is the default template with checkboxes but almost no description of what was corrected or why. For a PR this size, reviewers have no way to know which method names were fixed, which pages were corrected, or what the REPRO file documents.
Please add a PR body summary similar to PR #477's format — at minimum: a list of corrected pages, the method name corrections (equivalent to the A1–A5 list in ANDROID-DOCS-AUDIT.md), and a note about the enableCalling crash repro. This is a review-process issue, not a correctness issue, but it makes the PR significantly harder to review.
2. Confirm STOPGAP removal from skills pack after merge

The cometchat-android-v6-core/SKILL.md in PR #175 has two STOPGAP blocks that are explicitly tied to this PR merging. Merge order: PR #481 first → then remove the STOPGAP blocks from PR #175. If both merge simultaneously or in reverse order, the stopgap blocks will remain in the published skills indefinitely. Coordinate the merge order.
3. AUDIT-084 in the index

The index's hot-path section correctly points to getting-started-kotlin / getting-started-jetpack where AUDIT-084 wiring lives. Confirm both getting-started pages explicitly use CometChatUIKit.initFromSettings(this, callback) (not the classic UIKitSettingsBuilder + init()) — this is what the skills pack's contracts.android-v6.json verifies.
Summary
The two LLM indexes are well-structured, correctly unlisted, and cover both cohorts in the right way (single index, per-cohort tabs in linked pages). The docs corrections align with the oracle findings in the skills pack. Two procedural items before this lands cleanly: add a meaningful PR description, and coordinate merge order with PR #175 (docs first so the skill STOPGAP blocks can be removed).
APPROVE — with the PR description and merge-order notes above.
@raj-dubey1
raj-dubey1
changed the base branch from
main
to
docs/skills-v5-tempAugust 25, 2026 08:39
@raj-dubey1
raj-dubey1
merged commit 6e09c78
into
docs/skills-v5-tempAug 25, 2026

3 checks passed

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@raj-dubey1raj-dubey1raj-dubey1 left review comments
@jitvarpatiljitvarpatilAwaiting requested review from jitvarpatiljitvarpatil is a code owner

Assignees


No one assigned


Labels


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@hritika-cometchat@raj-dubey1

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Docs/android llms indexes - #481
Merged
raj-dubey1 merged 5 commits into
docs/skills-v5-tempfrom 
docs/android-llms-indexes
Aug 25, 2026
Merged
Docs/android llms indexes#481
raj-dubey1 merged 5 commits into
docs/skills-v5-tempfrom 
docs/android-llms-indexes
Conversation
@hritika-cometchat
@hritika-cometchathritika-cometchat
commented
Aug 24, 2026•

edited

Loading

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
mintlifyBot
commented
Aug 24, 2026•

edited

Loading

Copy link


Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.
ProjectStatusPreviewUpdated (UTC)
cometchat🟢 ReadyView PreviewAug 24, 2026, 6:22 AM
💡 Tip: Enable Workflows to automatically generate PRs for you.
raj-dubey1
raj-dubey1
reviewed
Aug 25, 2026
View reviewed changes
@raj-dubey1raj-dubey1
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.

Docs review — Android LLM indexes (ENG-38205 / PR #175 companion)
Reviewed from the skills pack perspective: does this PR give the Android skills the docs surface they depend on?
✅ What's correct and solid
llms-android-v6.mdx — "unlisted not hidden" pattern correct (explicit rationale comment, no hidden: true, not in docs.json). Both cohorts covered prominently: "Two UI flavors of ONE kit: chatuikit-kotlin-android:6.+ (XML Views) and chatuikit-compose-android:6.+ (Jetpack Compose)". The hot-path section links both cohort entry points. The "How to use this index" section tells the agent to read the matching tab (Views vs Compose) on every component page. This is the right approach — keeping a single index that reflects the single-kit architecture.
Both-cohort cross-links complete: v6 UI Kit index → v5 SDK index, and v5 SDK index → v6 UI Kit index. getting-started-kotlin.mdx and getting-started-jetpack.mdx are both linked from the hot path.
Pending pages documented: The three features gated on feature/android-pin_save_thread (pinned-messages, saved-messages, guide-thread-subscription) are listed in a JSX comment block with the branch name and the instruction to uncomment when that branch merges. This matches the PENDING section in features.android-v6.json — the two files are in sync.
llms-android-v5.mdx (SDK) — present and cross-linked to the UI Kit index.
Docs corrections across 14+ pages: The ANDROID-DOCS-AUDIT.md methodology (mechanical diff of docs vs 6.0.5 source) produced 5 verified wrong method names in the live docs (A1–A5: setOnAcceptClick vs setOnAcceptClickListener, etc.). These align with the STOPGAP blocks in the skills pack core skill — once these corrections merge, those stopgaps can be removed.
⚠️ Items to address
1. PR description is very thin for the scope of changes

This PR touches 14+ existing docs pages, adds two index files, includes ANDROID-DOCS-AUDIT.md (documenting 5 wrong method names in production docs), and adds REPRO-enableCalling-launch-crash.md. The PR body is the default template with checkboxes but almost no description of what was corrected or why. For a PR this size, reviewers have no way to know which method names were fixed, which pages were corrected, or what the REPRO file documents.
Please add a PR body summary similar to PR #477's format — at minimum: a list of corrected pages, the method name corrections (equivalent to the A1–A5 list in ANDROID-DOCS-AUDIT.md), and a note about the enableCalling crash repro. This is a review-process issue, not a correctness issue, but it makes the PR significantly harder to review.
2. Confirm STOPGAP removal from skills pack after merge

The cometchat-android-v6-core/SKILL.md in PR #175 has two STOPGAP blocks that are explicitly tied to this PR merging. Merge order: PR #481 first → then remove the STOPGAP blocks from PR #175. If both merge simultaneously or in reverse order, the stopgap blocks will remain in the published skills indefinitely. Coordinate the merge order.
3. AUDIT-084 in the index

The index's hot-path section correctly points to getting-started-kotlin / getting-started-jetpack where AUDIT-084 wiring lives. Confirm both getting-started pages explicitly use CometChatUIKit.initFromSettings(this, callback) (not the classic UIKitSettingsBuilder + init()) — this is what the skills pack's contracts.android-v6.json verifies.
Summary
The two LLM indexes are well-structured, correctly unlisted, and cover both cohorts in the right way (single index, per-cohort tabs in linked pages). The docs corrections align with the oracle findings in the skills pack. Two procedural items before this lands cleanly: add a meaningful PR description, and coordinate merge order with PR #175 (docs first so the skill STOPGAP blocks can be removed).
APPROVE — with the PR description and merge-order notes above.
@raj-dubey1
raj-dubey1
changed the base branch from
main
to
docs/skills-v5-tempAugust 25, 2026 08:39
@raj-dubey1
raj-dubey1
merged commit 6e09c78
into
docs/skills-v5-tempAug 25, 2026

3 checks passed

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@raj-dubey1raj-dubey1raj-dubey1 left review comments
@jitvarpatiljitvarpatilAwaiting requested review from jitvarpatiljitvarpatil is a code owner

Assignees


No one assigned


Labels


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@hritika-cometchat@raj-dubey1

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Docs/android llms indexes - #481
Merged
raj-dubey1 merged 5 commits into
docs/skills-v5-tempfrom 
docs/android-llms-indexes
Aug 25, 2026
Merged
Docs/android llms indexes#481
raj-dubey1 merged 5 commits into
docs/skills-v5-tempfrom 
docs/android-llms-indexes
Conversation
@hritika-cometchat
@hritika-cometchathritika-cometchat
commented
Aug 24, 2026•

edited

Loading

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
mintlifyBot
commented
Aug 24, 2026•

edited

Loading

Copy link


Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.
ProjectStatusPreviewUpdated (UTC)
cometchat🟢 ReadyView PreviewAug 24, 2026, 6:22 AM
💡 Tip: Enable Workflows to automatically generate PRs for you.
raj-dubey1
raj-dubey1
reviewed
Aug 25, 2026
View reviewed changes
@raj-dubey1raj-dubey1
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.

Docs review — Android LLM indexes (ENG-38205 / PR #175 companion)
Reviewed from the skills pack perspective: does this PR give the Android skills the docs surface they depend on?
✅ What's correct and solid
llms-android-v6.mdx — "unlisted not hidden" pattern correct (explicit rationale comment, no hidden: true, not in docs.json). Both cohorts covered prominently: "Two UI flavors of ONE kit: chatuikit-kotlin-android:6.+ (XML Views) and chatuikit-compose-android:6.+ (Jetpack Compose)". The hot-path section links both cohort entry points. The "How to use this index" section tells the agent to read the matching tab (Views vs Compose) on every component page. This is the right approach — keeping a single index that reflects the single-kit architecture.
Both-cohort cross-links complete: v6 UI Kit index → v5 SDK index, and v5 SDK index → v6 UI Kit index. getting-started-kotlin.mdx and getting-started-jetpack.mdx are both linked from the hot path.
Pending pages documented: The three features gated on feature/android-pin_save_thread (pinned-messages, saved-messages, guide-thread-subscription) are listed in a JSX comment block with the branch name and the instruction to uncomment when that branch merges. This matches the PENDING section in features.android-v6.json — the two files are in sync.
llms-android-v5.mdx (SDK) — present and cross-linked to the UI Kit index.
Docs corrections across 14+ pages: The ANDROID-DOCS-AUDIT.md methodology (mechanical diff of docs vs 6.0.5 source) produced 5 verified wrong method names in the live docs (A1–A5: setOnAcceptClick vs setOnAcceptClickListener, etc.). These align with the STOPGAP blocks in the skills pack core skill — once these corrections merge, those stopgaps can be removed.
⚠️ Items to address
1. PR description is very thin for the scope of changes

This PR touches 14+ existing docs pages, adds two index files, includes ANDROID-DOCS-AUDIT.md (documenting 5 wrong method names in production docs), and adds REPRO-enableCalling-launch-crash.md. The PR body is the default template with checkboxes but almost no description of what was corrected or why. For a PR this size, reviewers have no way to know which method names were fixed, which pages were corrected, or what the REPRO file documents.
Please add a PR body summary similar to PR #477's format — at minimum: a list of corrected pages, the method name corrections (equivalent to the A1–A5 list in ANDROID-DOCS-AUDIT.md), and a note about the enableCalling crash repro. This is a review-process issue, not a correctness issue, but it makes the PR significantly harder to review.
2. Confirm STOPGAP removal from skills pack after merge

The cometchat-android-v6-core/SKILL.md in PR #175 has two STOPGAP blocks that are explicitly tied to this PR merging. Merge order: PR #481 first → then remove the STOPGAP blocks from PR #175. If both merge simultaneously or in reverse order, the stopgap blocks will remain in the published skills indefinitely. Coordinate the merge order.
3. AUDIT-084 in the index

The index's hot-path section correctly points to getting-started-kotlin / getting-started-jetpack where AUDIT-084 wiring lives. Confirm both getting-started pages explicitly use CometChatUIKit.initFromSettings(this, callback) (not the classic UIKitSettingsBuilder + init()) — this is what the skills pack's contracts.android-v6.json verifies.
Summary
The two LLM indexes are well-structured, correctly unlisted, and cover both cohorts in the right way (single index, per-cohort tabs in linked pages). The docs corrections align with the oracle findings in the skills pack. Two procedural items before this lands cleanly: add a meaningful PR description, and coordinate merge order with PR #175 (docs first so the skill STOPGAP blocks can be removed).
APPROVE — with the PR description and merge-order notes above.
@raj-dubey1
raj-dubey1
changed the base branch from
main
to
docs/skills-v5-tempAugust 25, 2026 08:39
@raj-dubey1
raj-dubey1
merged commit 6e09c78
into
docs/skills-v5-tempAug 25, 2026

3 checks passed

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@raj-dubey1raj-dubey1raj-dubey1 left review comments
@jitvarpatiljitvarpatilAwaiting requested review from jitvarpatiljitvarpatil is a code owner

Assignees


No one assigned


Labels


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@hritika-cometchat@raj-dubey1

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content
Docs/android llms indexes - #481
Merged
raj-dubey1 merged 5 commits into
docs/skills-v5-tempfrom 
docs/android-llms-indexes
Aug 25, 2026
Merged
Docs/android llms indexes#481
raj-dubey1 merged 5 commits into
docs/skills-v5-tempfrom 
docs/android-llms-indexes
Conversation
@hritika-cometchat
@hritika-cometchathritika-cometchat
commented
Aug 24, 2026•

edited

Loading

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
mintlifyBot
commented
Aug 24, 2026•

edited

Loading

Copy link


Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.
ProjectStatusPreviewUpdated (UTC)
cometchat🟢 ReadyView PreviewAug 24, 2026, 6:22 AM
💡 Tip: Enable Workflows to automatically generate PRs for you.
raj-dubey1
raj-dubey1
reviewed
Aug 25, 2026
View reviewed changes
@raj-dubey1raj-dubey1
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.

Docs review — Android LLM indexes (ENG-38205 / PR #175 companion)
Reviewed from the skills pack perspective: does this PR give the Android skills the docs surface they depend on?
✅ What's correct and solid
llms-android-v6.mdx — "unlisted not hidden" pattern correct (explicit rationale comment, no hidden: true, not in docs.json). Both cohorts covered prominently: "Two UI flavors of ONE kit: chatuikit-kotlin-android:6.+ (XML Views) and chatuikit-compose-android:6.+ (Jetpack Compose)". The hot-path section links both cohort entry points. The "How to use this index" section tells the agent to read the matching tab (Views vs Compose) on every component page. This is the right approach — keeping a single index that reflects the single-kit architecture.
Both-cohort cross-links complete: v6 UI Kit index → v5 SDK index, and v5 SDK index → v6 UI Kit index. getting-started-kotlin.mdx and getting-started-jetpack.mdx are both linked from the hot path.
Pending pages documented: The three features gated on feature/android-pin_save_thread (pinned-messages, saved-messages, guide-thread-subscription) are listed in a JSX comment block with the branch name and the instruction to uncomment when that branch merges. This matches the PENDING section in features.android-v6.json — the two files are in sync.
llms-android-v5.mdx (SDK) — present and cross-linked to the UI Kit index.
Docs corrections across 14+ pages: The ANDROID-DOCS-AUDIT.md methodology (mechanical diff of docs vs 6.0.5 source) produced 5 verified wrong method names in the live docs (A1–A5: setOnAcceptClick vs setOnAcceptClickListener, etc.). These align with the STOPGAP blocks in the skills pack core skill — once these corrections merge, those stopgaps can be removed.
⚠️ Items to address
1. PR description is very thin for the scope of changes

This PR touches 14+ existing docs pages, adds two index files, includes ANDROID-DOCS-AUDIT.md (documenting 5 wrong method names in production docs), and adds REPRO-enableCalling-launch-crash.md. The PR body is the default template with checkboxes but almost no description of what was corrected or why. For a PR this size, reviewers have no way to know which method names were fixed, which pages were corrected, or what the REPRO file documents.
Please add a PR body summary similar to PR #477's format — at minimum: a list of corrected pages, the method name corrections (equivalent to the A1–A5 list in ANDROID-DOCS-AUDIT.md), and a note about the enableCalling crash repro. This is a review-process issue, not a correctness issue, but it makes the PR significantly harder to review.
2. Confirm STOPGAP removal from skills pack after merge

The cometchat-android-v6-core/SKILL.md in PR #175 has two STOPGAP blocks that are explicitly tied to this PR merging. Merge order: PR #481 first → then remove the STOPGAP blocks from PR #175. If both merge simultaneously or in reverse order, the stopgap blocks will remain in the published skills indefinitely. Coordinate the merge order.
3. AUDIT-084 in the index

The index's hot-path section correctly points to getting-started-kotlin / getting-started-jetpack where AUDIT-084 wiring lives. Confirm both getting-started pages explicitly use CometChatUIKit.initFromSettings(this, callback) (not the classic UIKitSettingsBuilder + init()) — this is what the skills pack's contracts.android-v6.json verifies.
Summary
The two LLM indexes are well-structured, correctly unlisted, and cover both cohorts in the right way (single index, per-cohort tabs in linked pages). The docs corrections align with the oracle findings in the skills pack. Two procedural items before this lands cleanly: add a meaningful PR description, and coordinate merge order with PR #175 (docs first so the skill STOPGAP blocks can be removed).
APPROVE — with the PR description and merge-order notes above.
@raj-dubey1
raj-dubey1
changed the base branch from
main
to
docs/skills-v5-tempAugust 25, 2026 08:39
@raj-dubey1
raj-dubey1
merged commit 6e09c78
into
docs/skills-v5-tempAug 25, 2026

3 checks passed

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@raj-dubey1raj-dubey1raj-dubey1 left review comments
@jitvarpatiljitvarpatilAwaiting requested review from jitvarpatiljitvarpatil is a code owner

Assignees


No one assigned


Labels


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@hritika-cometchat@raj-dubey1

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Docs/android llms indexes - #481
Merged
raj-dubey1 merged 5 commits into
docs/skills-v5-tempfrom 
docs/android-llms-indexes
Aug 25, 2026
Merged
Docs/android llms indexes#481
raj-dubey1 merged 5 commits into
docs/skills-v5-tempfrom 
docs/android-llms-indexes
Conversation
@hritika-cometchat
@hritika-cometchathritika-cometchat
commented
Aug 24, 2026•

edited

Loading

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
mintlifyBot
commented
Aug 24, 2026•

edited

Loading

Copy link


Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.
ProjectStatusPreviewUpdated (UTC)
cometchat🟢 ReadyView PreviewAug 24, 2026, 6:22 AM
💡 Tip: Enable Workflows to automatically generate PRs for you.
raj-dubey1
raj-dubey1
reviewed
Aug 25, 2026
View reviewed changes
@raj-dubey1raj-dubey1
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.

Docs review — Android LLM indexes (ENG-38205 / PR #175 companion)
Reviewed from the skills pack perspective: does this PR give the Android skills the docs surface they depend on?
✅ What's correct and solid
llms-android-v6.mdx — "unlisted not hidden" pattern correct (explicit rationale comment, no hidden: true, not in docs.json). Both cohorts covered prominently: "Two UI flavors of ONE kit: chatuikit-kotlin-android:6.+ (XML Views) and chatuikit-compose-android:6.+ (Jetpack Compose)". The hot-path section links both cohort entry points. The "How to use this index" section tells the agent to read the matching tab (Views vs Compose) on every component page. This is the right approach — keeping a single index that reflects the single-kit architecture.
Both-cohort cross-links complete: v6 UI Kit index → v5 SDK index, and v5 SDK index → v6 UI Kit index. getting-started-kotlin.mdx and getting-started-jetpack.mdx are both linked from the hot path.
Pending pages documented: The three features gated on feature/android-pin_save_thread (pinned-messages, saved-messages, guide-thread-subscription) are listed in a JSX comment block with the branch name and the instruction to uncomment when that branch merges. This matches the PENDING section in features.android-v6.json — the two files are in sync.
llms-android-v5.mdx (SDK) — present and cross-linked to the UI Kit index.
Docs corrections across 14+ pages: The ANDROID-DOCS-AUDIT.md methodology (mechanical diff of docs vs 6.0.5 source) produced 5 verified wrong method names in the live docs (A1–A5: setOnAcceptClick vs setOnAcceptClickListener, etc.). These align with the STOPGAP blocks in the skills pack core skill — once these corrections merge, those stopgaps can be removed.
⚠️ Items to address
1. PR description is very thin for the scope of changes

This PR touches 14+ existing docs pages, adds two index files, includes ANDROID-DOCS-AUDIT.md (documenting 5 wrong method names in production docs), and adds REPRO-enableCalling-launch-crash.md. The PR body is the default template with checkboxes but almost no description of what was corrected or why. For a PR this size, reviewers have no way to know which method names were fixed, which pages were corrected, or what the REPRO file documents.
Please add a PR body summary similar to PR #477's format — at minimum: a list of corrected pages, the method name corrections (equivalent to the A1–A5 list in ANDROID-DOCS-AUDIT.md), and a note about the enableCalling crash repro. This is a review-process issue, not a correctness issue, but it makes the PR significantly harder to review.
2. Confirm STOPGAP removal from skills pack after merge

The cometchat-android-v6-core/SKILL.md in PR #175 has two STOPGAP blocks that are explicitly tied to this PR merging. Merge order: PR #481 first → then remove the STOPGAP blocks from PR #175. If both merge simultaneously or in reverse order, the stopgap blocks will remain in the published skills indefinitely. Coordinate the merge order.
3. AUDIT-084 in the index

The index's hot-path section correctly points to getting-started-kotlin / getting-started-jetpack where AUDIT-084 wiring lives. Confirm both getting-started pages explicitly use CometChatUIKit.initFromSettings(this, callback) (not the classic UIKitSettingsBuilder + init()) — this is what the skills pack's contracts.android-v6.json verifies.
Summary
The two LLM indexes are well-structured, correctly unlisted, and cover both cohorts in the right way (single index, per-cohort tabs in linked pages). The docs corrections align with the oracle findings in the skills pack. Two procedural items before this lands cleanly: add a meaningful PR description, and coordinate merge order with PR #175 (docs first so the skill STOPGAP blocks can be removed).
APPROVE — with the PR description and merge-order notes above.
@raj-dubey1
raj-dubey1
changed the base branch from
main
to
docs/skills-v5-tempAugust 25, 2026 08:39
@raj-dubey1
raj-dubey1
merged commit 6e09c78
into
docs/skills-v5-tempAug 25, 2026

3 checks passed

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@raj-dubey1raj-dubey1raj-dubey1 left review comments
@jitvarpatiljitvarpatilAwaiting requested review from jitvarpatiljitvarpatil is a code owner

Assignees


No one assigned


Labels


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@hritika-cometchat@raj-dubey1

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Docs/android llms indexes - #481
Merged
raj-dubey1 merged 5 commits into
docs/skills-v5-tempfrom 
docs/android-llms-indexes
Aug 25, 2026
Merged
Docs/android llms indexes#481
raj-dubey1 merged 5 commits into
docs/skills-v5-tempfrom 
docs/android-llms-indexes
Conversation
@hritika-cometchat
@hritika-cometchathritika-cometchat
commented
Aug 24, 2026•

edited

Loading

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
mintlifyBot
commented
Aug 24, 2026•

edited

Loading

Copy link


Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.
ProjectStatusPreviewUpdated (UTC)
cometchat🟢 ReadyView PreviewAug 24, 2026, 6:22 AM
💡 Tip: Enable Workflows to automatically generate PRs for you.
raj-dubey1
raj-dubey1
reviewed
Aug 25, 2026
View reviewed changes
@raj-dubey1raj-dubey1
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.

Docs review — Android LLM indexes (ENG-38205 / PR #175 companion)
Reviewed from the skills pack perspective: does this PR give the Android skills the docs surface they depend on?
✅ What's correct and solid
llms-android-v6.mdx — "unlisted not hidden" pattern correct (explicit rationale comment, no hidden: true, not in docs.json). Both cohorts covered prominently: "Two UI flavors of ONE kit: chatuikit-kotlin-android:6.+ (XML Views) and chatuikit-compose-android:6.+ (Jetpack Compose)". The hot-path section links both cohort entry points. The "How to use this index" section tells the agent to read the matching tab (Views vs Compose) on every component page. This is the right approach — keeping a single index that reflects the single-kit architecture.
Both-cohort cross-links complete: v6 UI Kit index → v5 SDK index, and v5 SDK index → v6 UI Kit index. getting-started-kotlin.mdx and getting-started-jetpack.mdx are both linked from the hot path.
Pending pages documented: The three features gated on feature/android-pin_save_thread (pinned-messages, saved-messages, guide-thread-subscription) are listed in a JSX comment block with the branch name and the instruction to uncomment when that branch merges. This matches the PENDING section in features.android-v6.json — the two files are in sync.
llms-android-v5.mdx (SDK) — present and cross-linked to the UI Kit index.
Docs corrections across 14+ pages: The ANDROID-DOCS-AUDIT.md methodology (mechanical diff of docs vs 6.0.5 source) produced 5 verified wrong method names in the live docs (A1–A5: setOnAcceptClick vs setOnAcceptClickListener, etc.). These align with the STOPGAP blocks in the skills pack core skill — once these corrections merge, those stopgaps can be removed.
⚠️ Items to address
1. PR description is very thin for the scope of changes

This PR touches 14+ existing docs pages, adds two index files, includes ANDROID-DOCS-AUDIT.md (documenting 5 wrong method names in production docs), and adds REPRO-enableCalling-launch-crash.md. The PR body is the default template with checkboxes but almost no description of what was corrected or why. For a PR this size, reviewers have no way to know which method names were fixed, which pages were corrected, or what the REPRO file documents.
Please add a PR body summary similar to PR #477's format — at minimum: a list of corrected pages, the method name corrections (equivalent to the A1–A5 list in ANDROID-DOCS-AUDIT.md), and a note about the enableCalling crash repro. This is a review-process issue, not a correctness issue, but it makes the PR significantly harder to review.
2. Confirm STOPGAP removal from skills pack after merge

The cometchat-android-v6-core/SKILL.md in PR #175 has two STOPGAP blocks that are explicitly tied to this PR merging. Merge order: PR #481 first → then remove the STOPGAP blocks from PR #175. If both merge simultaneously or in reverse order, the stopgap blocks will remain in the published skills indefinitely. Coordinate the merge order.
3. AUDIT-084 in the index

The index's hot-path section correctly points to getting-started-kotlin / getting-started-jetpack where AUDIT-084 wiring lives. Confirm both getting-started pages explicitly use CometChatUIKit.initFromSettings(this, callback) (not the classic UIKitSettingsBuilder + init()) — this is what the skills pack's contracts.android-v6.json verifies.
Summary
The two LLM indexes are well-structured, correctly unlisted, and cover both cohorts in the right way (single index, per-cohort tabs in linked pages). The docs corrections align with the oracle findings in the skills pack. Two procedural items before this lands cleanly: add a meaningful PR description, and coordinate merge order with PR #175 (docs first so the skill STOPGAP blocks can be removed).
APPROVE — with the PR description and merge-order notes above.
@raj-dubey1
raj-dubey1
changed the base branch from
main
to
docs/skills-v5-tempAugust 25, 2026 08:39
@raj-dubey1
raj-dubey1
merged commit 6e09c78
into
docs/skills-v5-tempAug 25, 2026

3 checks passed

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@raj-dubey1raj-dubey1raj-dubey1 left review comments
@jitvarpatiljitvarpatilAwaiting requested review from jitvarpatiljitvarpatil is a code owner

Assignees


No one assigned


Labels


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@hritika-cometchat@raj-dubey1

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content
Docs/android llms indexes - #481
Merged
raj-dubey1 merged 5 commits into
docs/skills-v5-tempfrom 
docs/android-llms-indexes
Aug 25, 2026
Merged
Docs/android llms indexes#481
raj-dubey1 merged 5 commits into
docs/skills-v5-tempfrom 
docs/android-llms-indexes
Conversation
@hritika-cometchat
@hritika-cometchathritika-cometchat
commented
Aug 24, 2026•

edited

Loading

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
mintlifyBot
commented
Aug 24, 2026•

edited

Loading

Copy link


Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.
ProjectStatusPreviewUpdated (UTC)
cometchat🟢 ReadyView PreviewAug 24, 2026, 6:22 AM
💡 Tip: Enable Workflows to automatically generate PRs for you.
raj-dubey1
raj-dubey1
reviewed
Aug 25, 2026
View reviewed changes
@raj-dubey1raj-dubey1
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.

Docs review — Android LLM indexes (ENG-38205 / PR #175 companion)
Reviewed from the skills pack perspective: does this PR give the Android skills the docs surface they depend on?
✅ What's correct and solid
llms-android-v6.mdx — "unlisted not hidden" pattern correct (explicit rationale comment, no hidden: true, not in docs.json). Both cohorts covered prominently: "Two UI flavors of ONE kit: chatuikit-kotlin-android:6.+ (XML Views) and chatuikit-compose-android:6.+ (Jetpack Compose)". The hot-path section links both cohort entry points. The "How to use this index" section tells the agent to read the matching tab (Views vs Compose) on every component page. This is the right approach — keeping a single index that reflects the single-kit architecture.
Both-cohort cross-links complete: v6 UI Kit index → v5 SDK index, and v5 SDK index → v6 UI Kit index. getting-started-kotlin.mdx and getting-started-jetpack.mdx are both linked from the hot path.
Pending pages documented: The three features gated on feature/android-pin_save_thread (pinned-messages, saved-messages, guide-thread-subscription) are listed in a JSX comment block with the branch name and the instruction to uncomment when that branch merges. This matches the PENDING section in features.android-v6.json — the two files are in sync.
llms-android-v5.mdx (SDK) — present and cross-linked to the UI Kit index.
Docs corrections across 14+ pages: The ANDROID-DOCS-AUDIT.md methodology (mechanical diff of docs vs 6.0.5 source) produced 5 verified wrong method names in the live docs (A1–A5: setOnAcceptClick vs setOnAcceptClickListener, etc.). These align with the STOPGAP blocks in the skills pack core skill — once these corrections merge, those stopgaps can be removed.
⚠️ Items to address
1. PR description is very thin for the scope of changes

This PR touches 14+ existing docs pages, adds two index files, includes ANDROID-DOCS-AUDIT.md (documenting 5 wrong method names in production docs), and adds REPRO-enableCalling-launch-crash.md. The PR body is the default template with checkboxes but almost no description of what was corrected or why. For a PR this size, reviewers have no way to know which method names were fixed, which pages were corrected, or what the REPRO file documents.
Please add a PR body summary similar to PR #477's format — at minimum: a list of corrected pages, the method name corrections (equivalent to the A1–A5 list in ANDROID-DOCS-AUDIT.md), and a note about the enableCalling crash repro. This is a review-process issue, not a correctness issue, but it makes the PR significantly harder to review.
2. Confirm STOPGAP removal from skills pack after merge

The cometchat-android-v6-core/SKILL.md in PR #175 has two STOPGAP blocks that are explicitly tied to this PR merging. Merge order: PR #481 first → then remove the STOPGAP blocks from PR #175. If both merge simultaneously or in reverse order, the stopgap blocks will remain in the published skills indefinitely. Coordinate the merge order.
3. AUDIT-084 in the index

The index's hot-path section correctly points to getting-started-kotlin / getting-started-jetpack where AUDIT-084 wiring lives. Confirm both getting-started pages explicitly use CometChatUIKit.initFromSettings(this, callback) (not the classic UIKitSettingsBuilder + init()) — this is what the skills pack's contracts.android-v6.json verifies.
Summary
The two LLM indexes are well-structured, correctly unlisted, and cover both cohorts in the right way (single index, per-cohort tabs in linked pages). The docs corrections align with the oracle findings in the skills pack. Two procedural items before this lands cleanly: add a meaningful PR description, and coordinate merge order with PR #175 (docs first so the skill STOPGAP blocks can be removed).
APPROVE — with the PR description and merge-order notes above.
@raj-dubey1
raj-dubey1
changed the base branch from
main
to
docs/skills-v5-tempAugust 25, 2026 08:39
@raj-dubey1
raj-dubey1
merged commit 6e09c78
into
docs/skills-v5-tempAug 25, 2026

3 checks passed

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@raj-dubey1raj-dubey1raj-dubey1 left review comments
@jitvarpatiljitvarpatilAwaiting requested review from jitvarpatiljitvarpatil is a code owner

Assignees


No one assigned


Labels


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@hritika-cometchat@raj-dubey1