fix: shorten skill descriptions over the 1024-character operator limit - #158
Open
matallui wants to merge 1 commit into
Open
fix: shorten skill descriptions over the 1024-character operator limit#158matallui wants to merge 1 commit into
matallui wants to merge 1 commit into
Conversation
The pi coding agent rejects any skill whose description exceeds 1024 characters at startup — the base performance skill (1078 chars) fails with "[Skill conflicts] description exceeds 1024 characters (1078)" and never loads. - Trim the performance skill description from 1078 to 949 characters, keeping every trigger cue (slow/laggy/janky surfaces, re-renders, memoization, FlashList, Reanimated, TTI, bundle size, cpuprofile, render-regression tests, and the exclusions). - Trim the swaps-cpu-profile-audit description from 1411 to 893 characters, which the old budget allowed but pi users could not load. - Lower DESCRIPTION_MAX to 1024, the strictest operator ceiling, so the linter blocks descriptions at least one operator cannot load, and update the schema comment, README, CONTRIBUTING, and skill template to state the new number and cite the operator. Cite-the-operator evidence requested by the schema comment: pi coding agent v0.85.1, docs/skills.md — description field: "Max 1024 chars".
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.
Description
The pi coding agent rejects any skill whose
descriptionexceeds 1024 characters at startup. The baseperformanceskill is 1078 characters, so every pi user sees this on launch and the skill never loads:This PR:
performanceskill description from 1078 to 949 characters. Every trigger cue is kept — slow/laggy/janky surfaces, re-renders, memoization, FlashList, Reanimated, TTI, bundle size,.cpuprofile/Release Profiler analysis, render-regression tests, and the "not for" exclusions. The only cuts are filler ("at any stage", "and wanting to avoid perf pitfalls before building"), a redundant example filename, and a phrase duplicating the Release Profiler mention.swaps-cpu-profile-auditdescription from 1411 to 893 characters. It passed the old budget but was equally unloadable in pi.DESCRIPTION_MAXfrom 1536 to 1024 soyarn audit:skillsblocks descriptions at least one operator cannot load, and updates the schema comment, README, CONTRIBUTING, and skill template accordingly.On point 3: the schema comment asks contributors to "cite the operator and version before claiming any figure is externally imposed" — citation: pi coding agent v0.85.1,
docs/skills.md: thedescriptionfield is "Max 1024 chars", enforced with the error above. Claude Code still loads descriptions well over 1024 (as the old comment correctly noted), so 1024 is simply the strictest operator ceiling, exactly what CONTRIBUTING's "lowest limit across operators" already assumes.Type of Change
Skill Details (if adding a new skill)
N/A — existing skills only.
Checklist
Testing
yarn audit:skills→ 54 skill(s) checked, 0 errors (was 1 error after the budget change, 0 before — the swapped descriptions keep it at 0)yarn test→ 71/71 pass, including the docs-state/enforced-ceiling consistency test, which now asserts 1024yarn lint:changelog→ clean[Skill conflicts] description exceeds 1024 characters (1078)for the unmodified skill; after this change the description parses clean