Skip to content

skill added for RNN android and iOS#8238

Open
markdevocht wants to merge 3 commits intomasterfrom
feat/skill-added-for-RNN
Open

skill added for RNN android and iOS#8238
markdevocht wants to merge 3 commits intomasterfrom
feat/skill-added-for-RNN

Conversation

@markdevocht
Copy link
Contributor

@markdevocht markdevocht commented Feb 25, 2026

The RNN skill gives the AI a mental map of the codebase so it doesn't have to discover the architecture from scratch every conversation. Specifically:

  • Cross-layer lookup table — When the AI needs to fix something (e.g. "topbar title is cut off"), it can immediately look up that the relevant files are TopBarPresenter.mm on iOS, views/stack/topbar/ on Android, and Options.ts on JS — instead of spending 10 tool calls searching.
  • Command flow — It knows the exact path a command takes from Navigation.push() in JS through the processing pipeline, across the TurboModule bridge, to RNNCommandsHandler (iOS) or Navigator (Android). Without this, the AI would trace it from scratch every time.
  • Pattern awareness — It knows things like "iOS uses the Presenter pattern for applying options", "Android is View-based not Fragment-based", "overlays use separate UIWindows on iOS". These are the kinds of things that would otherwise lead to wrong assumptions.
  • Options resolution order — It knows defaults → parent → component → mergeOptions. Without this it might look in the wrong place for why an option isn't applying.
  • Gotchas — Things like passProps never crossing the bridge, lib/ being generated, splitView being iOS-only. Prevents common mistakes.

@markdevocht markdevocht requested a review from yedidyak February 25, 2026 18:11
yedidyak
yedidyak previously approved these changes Feb 26, 2026
Copy link
Contributor

@yedidyak yedidyak left a comment

Choose a reason for hiding this comment

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

LGTM! All three factual issues from the initial review have been fixed, and the skill is now available across Cursor, Claude Code, and GitHub Copilot.

One minor note: the relative ARCHITECTURE.md links (../../ARCHITECTURE.md) resolve correctly from .cursor/ and .claude/ but are off by one level from .github/skills/rnn-codebase/ (needs ../../../). Not a blocker.

🤖 Generated with Claude Code

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.

2 participants