Repository navigation
feat(skills): add jaimintf iOS skill pack (SHUN-5788) - #211
Conversation
Add expo/skills (24), Appllama/appllama-skills (2), and twostraws/SwiftUI-Agent-Skill (swiftui-pro); expand emilkowalski/skills to keep-all except prototype (collides with mattpocock); add vercel-react-native-skills + vercel-optimize to vercel-labs/agent-skills. Source: https://x.com/jaimintf/status/2101017883870667052 Linear: https://linear.app/shunkakinoki/issue/SHUN-5788
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 SummarySummary by CodeRabbit
WalkthroughChangesSkills catalog refresh
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~12 minutes Change: Feature Merge Risk: ⚪ Minimal · up to The catalog and lock updates have no identified actionable merge risk. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. A rabbit checks each skill by moonlight Comment |
There was a problem hiding this comment.
1 issue found across 2 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="SKILLS.txt">
<violation number="1" location="SKILLS.txt:206">
P3: The `twostraws/SwiftUI-Agent-Skill` block is inserted between `tobi/qmd` and `trailofbits/skills`, but the file is otherwise maintained in case-insensitive alphabetical order and 'trailofbits' < 'twostraws' (t-r vs t-w). The block should sit after the `trailofbits/skills` entry (before `vercel/ai`), matching the alphabetical-order requirement stated in the PR's testing notes.</violation>
</file>
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
| tobi/qmd qmd,release | ||
|
|
||
| # twostraws/SwiftUI-Agent-Skill (1 total) - keep all (SHUN-5788 / jaimintf iOS pack) | ||
| twostraws/SwiftUI-Agent-Skill swiftui-pro |
There was a problem hiding this comment.
P3: The twostraws/SwiftUI-Agent-Skill block is inserted between tobi/qmd and trailofbits/skills, but the file is otherwise maintained in case-insensitive alphabetical order and 'trailofbits' < 'twostraws' (t-r vs t-w). The block should sit after the trailofbits/skills entry (before vercel/ai), matching the alphabetical-order requirement stated in the PR's testing notes.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At SKILLS.txt, line 206:
<comment>The `twostraws/SwiftUI-Agent-Skill` block is inserted between `tobi/qmd` and `trailofbits/skills`, but the file is otherwise maintained in case-insensitive alphabetical order and 'trailofbits' < 'twostraws' (t-r vs t-w). The block should sit after the `trailofbits/skills` entry (before `vercel/ai`), matching the alphabetical-order requirement stated in the PR's testing notes.</comment>
<file context>
@@ -195,6 +202,9 @@ subsy/ralph-tui ralph-tui-create-beads,ralph-tui-create-beads-rust,ralph-tui-cre
tobi/qmd qmd,release
+# twostraws/SwiftUI-Agent-Skill (1 total) - keep all (SHUN-5788 / jaimintf iOS pack)
+twostraws/SwiftUI-Agent-Skill swiftui-pro
+
# trailofbits/skills (61 total) - keep security essentials
</file context>
Summary
Adds the five iOS agent skill sources from jaimintf's tweet to
dotagentsviaSKILLS.txt+skills-lock.json(same pipeline as BetterWright).Changes
expo/skills(24) — keepplugins/expo/skills/*(not eval / experiments)Appllama/appllama-skills(2) — keep-all:appllama-app-design-skill,appllama-usageemilkowalski/skills— keep-all upstream exceptprototype(skill id collides with existingmattpocock/skillsprototype); 6 → 12 locked idsvercel-labs/agent-skills— keep existing 4; ADDvercel-react-native-skills+vercel-optimize(exact lock ids fromskills add --list/name:fields)twostraws/SwiftUI-Agent-Skill—swiftui-proLinear
SHUN-5788
Notes
prototypefrom emilkowalski omitted from the lock because the flatskills-lock.jsonkey already maps to mattpocock; called out inSKILLS.txtcomment.skillPath+skillFolderHashfrom a globalskills addon this box; existing entries preserved frommain.Test plan
SKILLS.txtalphabetical inserts + comments reference SHUN-5788 / jaimintfskills-lock.jsoncontains all new ids with hashes (expo 24, Appllama 2, emil 12, vercel 6, swiftui-pro 1)eas-app-stores,appllama-usage,write-swift,vercel-react-native-skills,swiftui-promake skills-installidempotent after mergeSummary by cubic
Adds five iOS skill sources from jaimintf's tweet to
dotagents, expanding the available iOS development skills.Changes
swiftui-profromtwostraws/SwiftUI-Agent-Skill.emilkowalski/skillsfrom 6 to 12 skills, omittingprototypeto avoid collision withmattpocock/skills.vercel-react-native-skillsandvercel-optimizetovercel-labs/agent-skills.skills-lock.jsonfor new and upstream-updated entries.Notes
prototypeskill is intentionally left out of the lock; the existingmattpocockentry remains.Written for commit fe5569b. Summary will update on new commits.