Skip to content

Add and update AI coding skills#8246

Open
yedidyak wants to merge 4 commits intomasterfrom
update-skills
Open

Add and update AI coding skills#8246
yedidyak wants to merge 4 commits intomasterfrom
update-skills

Conversation

@yedidyak
Copy link
Contributor

Summary

  • Add RNN codebase navigation skill (.claude/skills/rnn-codebase/SKILL.md + copies for Cursor/GitHub)
  • Add RN version upgrade skill (.claude/skills/rn-version-upgrade.md)
  • Update upgrade skill with lessons from RN 0.84:
    • Prefer __has_include for conditional imports (forward-compatible)
    • Use #ifndef RCT_REMOVE_LEGACY_ARCH for legacy code blocks (RN 0.84+)
    • Added "Research Before Fixing" phase: investigate better approaches from the target RN version before pattern-matching on existing code
    • New pitfall: don't blindly follow existing codebase patterns when better alternatives exist

Test plan

  • Verify skill files are well-formed markdown
  • Verify no non-skill files are included in this PR

🤖 Generated with Claude Code

markdevocht and others added 4 commits March 12, 2026 11:32
* skill added for RNN android and iOS

* update for Skills and added to multiple agent folders

* fixing relative path for github copilot
Formalises the process of adding new React Native version support,
validated against PRs for RN 0.80 through 0.84. Covers CI setup,
dependency analysis, native compilation fixes, and 10 common pitfalls.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Learned from PR #8244's approach to iOS arch guards:
- Use __has_include for conditional imports (forward-compatible)
- Use #ifndef RCT_REMOVE_LEGACY_ARCH for legacy code blocks (RN 0.84+)
- Added "Research Before Fixing" phase: investigate better approaches
  from the target RN version before pattern-matching on existing code
- Updated all fix patterns and pitfalls to reflect these techniques

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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