docs(plugin): enhance skills with game patterns and troubleshooting#604
Merged
JasonXuDeveloper merged 4 commits intomasterfrom Jan 30, 2026
Merged
docs(plugin): enhance skills with game patterns and troubleshooting#604JasonXuDeveloper merged 4 commits intomasterfrom
JasonXuDeveloper merged 4 commits intomasterfrom
Conversation
- Add Properties section and all state overloads to JAction skill - Add async game patterns (cooldown, DoT, wave spawner, health regen) - Add troubleshooting sections to JAction, JObjectPool, and editor-ui - Add zero-GC game patterns to JObjectPool (bullet pool, enemy spawner) - Add game dev examples to editor-ui (settings panel, build tool, asset browser) - Create new game-patterns skill with comprehensive zero-GC async patterns - Register new skill in plugin.json Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> Signed-off-by: JasonXuDeveloper - 傑 <jason@xgamedev.net>
github-actions
bot
added
the
documentation
There was a problem hiding this comment.
Pull request overview
This PR enhances the Claude plugin documentation with game-specific patterns and troubleshooting guidance, adding comprehensive examples that demonstrate zero-GC async patterns using JAction and JObjectPool with modern C# 9+ features.
Changes:
- Added new "Game Patterns" skill showcasing zero-allocation async patterns for combat, spawning, resources, projectiles, and UI
- Enhanced JAction skill with properties documentation, state overload examples, and troubleshooting section
- Enhanced JObjectPool skill with game-specific pooling patterns and troubleshooting guidance
- Enhanced Editor-UI skill with game development window examples and troubleshooting tips
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 11 comments.
Show a summary per file
| File | Description |
|---|---|
.claude-plugin/plugin.json |
Added game-patterns skill to the skills array |
.claude-plugin/skills/game-patterns/SKILL.md |
New comprehensive skill with zero-GC patterns for combat, spawning, resources, projectiles, and UI with best practices |
.claude-plugin/skills/jaction/SKILL.md |
Added Properties section, documented all state overloads, added game pattern examples (cooldown, DoT, waves, regen), and troubleshooting section |
.claude-plugin/skills/jobjectpool/SKILL.md |
Added game patterns section with bullet pool, enemy spawner, temp collections, StringBuilder examples, and troubleshooting guidance |
.claude-plugin/skills/editor-ui/SKILL.md |
Added game development examples (settings panel, build tool, asset browser) and comprehensive troubleshooting section |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Add null-conditional operator for TakeDamage in DoT pattern - Change ReadOnlySpan to array in async wave spawner (ref structs invalid) - Add Reset() methods to BulletLifetimeState and TypewriterState - Add Cleanup/Dispose methods to ComboSystem, RegenState, TooltipTrigger - Fix TypewriterState to use static lambda with state parameter - Ensure all JActions are properly disposed in long-running patterns Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> Signed-off-by: JasonXuDeveloper - 傑 <jason@xgamedev.net>
…ttern The code was already fixed to use static lambda with state parameter, so the comment suggesting it uses a closure was incorrect. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> Signed-off-by: JasonXuDeveloper - 傑 <jason@xgamedev.net>
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
- Add sync version of wave spawner using ReadOnlySpan<T> with ref readonly - Clarify that async methods cannot use ReadOnlySpan (ref struct limitation) - Note ReadOnlyMemory<T> as alternative for async with .Span access Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> Signed-off-by: JasonXuDeveloper - 傑 <jason@xgamedev.net>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 8 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Changes
JAction Skill
.Executing,.Cancelled,.IsParallel)Do,WaitUntil,WaitWhile,Repeat, etc.JObjectPool Skill
Editor-UI Skill
New Game Patterns Skill
Test plan
🤖 Generated with Claude Code