feat: add the 2D and tilemap skills - #42
Open
ziyiunity wants to merge 3 commits into
Open
Conversation
ziyiunity
force-pushed
the
feat/migrate-2d-tilemap-skills
branch
from
August 7, 2026 19:39
046e3a7 to
0f77ead
Compare
ziyiunity
requested review from
ChuanXin-Unity,
MarekMarchlewicz,
andresbayon and
venkify
August 7, 2026 19:42
ChuanXin-Unity
previously approved these changes
Aug 11, 2026
ChuanXin-Unity
left a comment
There was a problem hiding this comment.
The Tilemap skills are editor-specific, so I believe it is fine if they make use of editor apis. Let me know if this is not the case, thanks!
This was referenced Aug 11, 2026
chris-addison
previously approved these changes
Aug 13, 2026
ziyiunity
dismissed stale reviews from chris-addison, venkify, and ChuanXin-Unity
via
August 13, 2026 16:39
b9bb8a8
chris-addison
previously approved these changes
Aug 13, 2026
chris-addison
approved these changes
Aug 13, 2026
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.
Second batch of the Unity skill migration, grouped by domain: 2D rendering and tilemaps. Six skills, 63 files, on top of the UI batch that landed in #41.
2d-pixel-perfectmanage-sprite-atlassprite-segment-3x3gridtilemap-palette-createtilemap-ruletile-createemptytilemap-ruletile-createfromsegmentsprite-segment-3x3gridoutputWhat was adapted
The
name:key was added to every skill, as this repo's convention requires, matching the folder in each case. Beyond that, the files landed byte-identical to the version these skills ship with elsewhere. Three of them have since changed on this branch; see Later fixes on this branch below.Checked before pushing: no
author:in any frontmatter, all 145 relativereferences/andresources/paths resolve to real files, every description is within the 1,536-character listing cap, and no skill references another as a slash command (they name each other instead, so the references work on hosts that don't expose skills as commands).Later fixes on this branch
The link check above originally passed only because it looked at
SKILL.md. Every link frommanage-sprite-atlas/references/*.mdintoresources/was broken: 38 of them, written asresources/foo.cs, which resolves relative toreferences/and so pointed at a path that does not exist. One also had a typo (handelatebinding.cs). Combined withSKILL.mdnot linking the resources at all, this skill shipped 38 working C# files that could not be opened from anywhere. All of them now use../resources/, andSKILL.mdcarries a path-to-files table at the top of its required-checks section so the resources get read before code is written.manage-sprite-atlas: the SpritePacker mode step is now verifiable. It said "ALWAYS enable Sprite Packer mode in Project Settings" and stopped there, with no API and no verification. That step decides whether the atlas is real:Disabledis the zero value ofSpritePackerMode, so any project nobody has configured carries it, and an atlas created while it is Disabled still imports, still looks finished, and can never pack. It now points atresources/enablespritepacking.csfor the call and requires reading the setting back and reporting the value.Two tilemap descriptions were tightened, both observed misfires.
tilemap-ruletile-createemptyfired even when the user had provided sprites, despite its description already saying not to, so the negative is now absolute and routes totilemap-ruletile-createfromsegmentby name.tilemap-ruletile-createfromsegmentmatched nothing for "set my sprites up to auto-tile when I paint", because the description was written from the converter's point of view and never contained the word autotile, so it now leads with the user-facing phrasing.One thing worth flagging before merge
Two of these skills describe creating an asset through an Editor-only API, and it is not obvious how an agent completes that step outside the Editor.
tilemap-palette-createinstructs the agent to useGridPaletteUtility.CreateNewPalette, and shipsscripts/CreatePaletteTemplate.cs, apublic static classwith no[MenuItem]and no entry point. An agent can write that file to disk, but nothing invokes it. For contrast,tilemap-ruletile-createfromsegment/RuleTileGenerator.csin this same batch does carry a[MenuItem].tilemap-ruletile-createemptypoints atresources/ruletile.md, which documents the rule patterns (the. . ./. * .neighbor tables) rather than the serialized asset format, so the step that actually writes the.assetis undefined.These skills came from an environment that had an Editor execution channel, so the gap is a property of the porting context, not something introduced here. I've deliberately left the content unmodified rather than guess at a fix. The owning team should decide whether to add an entry point, document the asset format, or scope the skills differently. Happy to split either skill out of this PR if you'd rather it land separately.