Skip to content

docs(readme): list Kotlin Multiplatform (9 platforms) - #100

Open
krassx wants to merge 2 commits into
mainfrom
docs/readme-nine-platforms
Open

krassx wants to merge 2 commits into
mainfrom
docs/readme-nine-platforms

Conversation

@krassx

@krassx krassx commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

The README said 8 platforms and omitted Kotlin Multiplatform, which has had its own section under docs/sdk/kmp/ since the beta. The docs/sdk/ folders are: android, cordova, dotnet, flutter, ios, kmp, react_native, unity, xamarin.

🤖 Generated with Claude Code

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Change-Id: Id778c165ed094095cbf75cf6940e0cbae5913982
@coderabbitai

coderabbitai Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: edc78a60-ca62-4184-8588-28c527c37641
📥 Commits

Reviewing files that changed from the base of the PR and between bccf67b and 6dc6d4c.

📒 Files selected for processing: 1
  • .claude/CLAUDE.md

Limit details: You’ve used all 10 included reviews currently available.

📝 Summary

Summary by CodeRabbit

  • Documentation
    • Updated the README to state that SDK documentation covers 9 platforms instead of 8.

The README changes the documented SDK platform count from eight to nine. The project overview adds Kotlin Multiplatform to the platform list and updates the SDK content directory description to report nine platform SDKs.

Priority: ⬇️ Low

Change: Other

Merge Risk: ⚪ Minimal · up to 6dc6d

The documentation consistently includes Kotlin Multiplatform in the nine-platform SDK inventory, which matches the existing SDK directories. No actionable merge risk remains.

### Architecture Summary

Architecture risk: 🔵 Low · up to 6dc6d

The change affects 1 system.

Changed systems: README.md

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — README.md (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in README.md: The README changes the documented SDK platform count from 8 to 9; the platform list remains the same.
  • observed — Modified behavior in .claude/CLAUDE.md: The overview now lists Kotlin Multiplatform as a ninth documented SDK platform; the previous list contained eight platforms.
  • observed — Modified behavior in .claude/CLAUDE.md: The SDK content directory description now states that it contains 9 platform SDKs instead of 8.
Pre-merge checks: Passed 5
Check name Status Explanation
Title check Passed The title clearly identifies the documentation change: adding Kotlin Multiplatform and updating the platform count to 9.
Description check Passed The description directly explains the README correction and lists the nine SDK platforms, including Kotlin Multiplatform.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Usage-based review receipt

  • Mode: Continue automatically
  • Reviewed files: 1
  • Waived: $0.25 (charged $0.00)
  • View usage details

Note

This review exceeded your plan’s limits and used usage-based reviews—free during trial, billed after paid activation unless disabled. Manage usage-based reviews.

Comment @coderabbitai help to get the list of available commands.

@claude

claude Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Claude finished @krassx's task in 29s —— View job


PR review: README lists 9 platforms

  • Read the diff against origin/main
  • Checked it against docs/sdk/, sidebars.ts and the rest of the repo for other platform counts
  • Ran cspell on README.md: 0 issues

Findings

  • P3 Low: .claude/CLAUDE.md still says "8 platforms" (line 7) and "8 platform SDKs" (line 44), and has no Kotlin Multiplatform. This PR doesn't touch that file, so I couldn't attach an inline comment to it (GitHub rejected it because the file isn't in the diff). The README and CLAUDE.md now disagree on the count.

I did not run the full npx cspell "docs/**/*.md" ... command from CLAUDE.md. This PR doesn't touch docs/.

Summary

  1. Overall risk: Low. This is a documentation-only change to one file.
  2. Merge recommendation: Approve. I can't approve formally, so this is a recommendation. It is fine to merge as is, or after the optional CLAUDE.md fix.
  3. Most important issue to fix: Update the two stale "8" references in .claude/CLAUDE.md. This is optional.
  4. Positives:
    • The count of 9 matches the 9 folders in docs/sdk/ (android, cordova, dotnet, flutter, ios, kmp, react_native, unity, xamarin).
    • sidebars.ts already has the KMP section.
    • The PR description lists the folders, which made the claim easy to check.
      · branch docs/readme-nine-platforms

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Change-Id: I38e6d92515daaf804696d987469182bed673240f
@claude

claude Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Claude finished @krassx's task in 25s —— View job


PR review: README and CLAUDE.md list 9 platforms

  • Read the diff against origin/main (2 commits)
  • Checked the count against docs/sdk/, sidebars.ts and the rest of the repo
  • Ran cspell on README.md: 0 issues

Earlier finding is fixed. My previous review noted that .claude/CLAUDE.md still said "8 platforms" (line 7) and "8 platform SDKs" (line 44). Commit 6dc6d4c updates both to 9 and adds Kotlin Multiplatform to the platform list. A repo-wide search finds no other "8 platforms" or "8 SDKs" references.

New findings: none.

  • The count of 9 matches the 9 folders in docs/sdk/: android, cordova, dotnet, flutter, ios, kmp, react_native, unity, xamarin.
  • sidebars.ts already has the sdk/kmp/* pages.
  • The README and CLAUDE.md platform lists now match.

Checks run: I ran cspell on README.md and it was clean. cspell also flags "Tawk" and "easyops" on lines 55 and 77 of .claude/CLAUDE.md. Those lines are not in this PR's diff, and cspell's docs/** scope doesn't cover that file, so they don't affect this change. I did not run the full docs/** spell check or a build, because this PR doesn't touch docs/.

Summary

  1. Overall risk: Low. It is a documentation-only change to two files.
  2. Merge recommendation: Approve. I can't approve formally, so this is a recommendation. It is fine to merge as is.
  3. Most important issues to fix: None.
  4. Positives:
    • The PR description lists the docs/sdk/ folders, which made the count easy to verify.
    • The follow-up commit fixed the README and CLAUDE.md mismatch.
      · branch docs/readme-nine-platforms

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant