fix(skill): trim description to <1024 chars for Codex/Copilot (#12) - #13
Merged
Conversation
The SKILL.md frontmatter description: field was 1728 chars. Codex CLI and Copilot CLI both enforce a 1024-char hard cap on it and reject the manifest with 'invalid description: exceeds maximum length of 1024 characters'. Claude Code does not enforce the cap, which is why it shipped green for us but broke for downstream installs. - Trim description: to a focused 843-char pitch covering what + platforms + modes (Compose, CMP, TV, Paging 3, navigation, Review Mode, session_start) - Move the cut trigger keywords into a new ## When this skill applies section in the SKILL.md body. Body content is not subject to the 1024 cap; Claude reads it after the skill loads, so the trigger surface stays available for downstream context. - Add scripts/check-description-length.sh as a regression guard, wired into ci.yml (every PR) and release.yml (every tag push). - Bump to 2.3.1 with CHANGELOG entry. Closes #12
…rot at every release The first bats test was hardcoded to v2.1.2 and started failing on master when v2.3.0 shipped — the two failing master CI runs after v2.3.0 confirm. Read the current version from plugin.json in setup() so the tests verify the script's logic, not a fixed version string.
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
1 participant
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
Fixes #12. The
description:frontmatter inskills/compose-expert/SKILL.mdwas 1728 chars. Codex CLI and Copilot CLI both enforce a hard 1024-char cap on that field and reject the manifest:Claude Code does not enforce this cap, which is why it shipped green for us but broke for downstream Codex / Copilot installs (also confirmed by @HumboShot in the issue thread).
What changed
description:to a focused 843-char pitch covering what + platforms + modes (Compose, CMP, TV, Paging 3, navigation, Review Mode, session_start auto-detection).## When this skill appliessection in the SKILL.md body. Body content is not subject to the 1024 cap; Claude reads it after the skill loads, so the keyword list (Compose APIs, CMP symbols, TV phrases, Paging 3, design-system phrasing, casual phrasings, Review Mode triggers, session_start) stays available for downstream context.scripts/check-description-length.sh— regression guard that asserts the field stays ≤ 1024 chars..github/workflows/ci.yml(every PR) and.github/workflows/release.yml(every tag push) — so a future regression fails CI loudly instead of shipping green on Claude Code and breaking Codex / Copilot installs.2.3.1with CHANGELOG entry.Local validation
Test plan
v2.3.1, release workflow publishes the GH Releasealdefy/compose-skillno longer errors withinvalid descriptionaldefy/compose-skillno longer errors with the same message/plugin update compose-expertpicks up 2.3.1 and the skill still auto-loads on Compose-related promptsCloses #12