Restructure tile palette skills - #85
markg-unity wants to merge 2 commits into
Conversation
6e54d84 to
d1b4156
Compare
markg-unity
changed the title
csantayanaUnity
left a comment
csantayanaUnity
left a comment
There was a problem hiding this comment.
Also need to wait for PR#75 to land before submitting this one
| name: create-tile-palette | ||
| description: Create a tile palette from sprites or textures. Make sure to use this skill when the user wants to create tiles for 2D level design, even if they don't explicitly ask about tile palettes or tile assets. | ||
| --- | ||
|
|
There was a problem hiding this comment.
question: I noticed that we don't have any required packages on the routing skill, is that ok? Before when flat the skill knew what packages it needed, not this skill does not expose what packages are needed for it's reference skills.
There was a problem hiding this comment.
I think agents would ignore any required-packages fields in the metadata of these Unity CLI-specific skills, because that's an AI Assistant-specific field? So this PR moves any package requirements into the skill body instead.
Let me know if that's not the case though
| # Tilemap Palette Creation | ||
|
|
||
| Requires package: `com.unity.2d.tilemap` >=1.0.0 | ||
|
|
||
| ## Workflow | ||
|
|
There was a problem hiding this comment.
other references seem to keep most of the description before workflow. Here we don't have any, is that intended and why?
There was a problem hiding this comment.
Oh that's just because the new SKILL.md already has that description, and routes to this reference file. So it would just be unnecessary duplication in this file.
|
@leoyaikUnity @ChuanXin-Unity I've updated the skill name and description from our Slack discussion now |
cf3423a to
ef35248
Compare
|
|
||
| # Tilemap RuleTile Create From Segment | ||
|
|
||
| Use when the user wants tiles that auto-tile (autotile) as they paint, wants a RuleTile built from existing terrain or edge sprites, or asks to make sprites "tile correctly" or "connect properly". Also converts sprite-segment-3x3grid output patterns into Unity RuleTile TilingRules: 3x3 grid text patterns (X, ., *) become TilingRule neighbor configurations, mapping '.' to 'This' rules and 'X' to 'DontCare', sorted by specificity (more 'This' rules first). Sprites must be provided as input. |
There was a problem hiding this comment.
Would it be possible use something other than "autotile"? I wonder if this would conflict with another Tile asset we have called "AutoTile" (similar to RuleTile).
There was a problem hiding this comment.
Hi @ChuanXin-Unity - this is a verbatim copy from the original skill (https://github.com/Unity-Technologies/skills/blob/main/skills/tilemap-ruletile-createfromsegment/SKILL.md), it just moved out of the description into the body. In the interests of keeping this PR quick, we can probably leave it as is for now?
ChuanXin-Unity
left a comment
ChuanXin-Unity
left a comment
There was a problem hiding this comment.
Looks good, would be great if the "autotile" term could be replaced if possible.
|
Thanks for this! These skills are now maintained in Unity's own skills repo and published here automatically, so your change has been moved there with your commits kept. Closing it here. |
4 participants
Restructure tile palette skills into one routing SKILL.md plus reference files to help with Codex limits and other agent limitations.