fix(skills): quote v4→v5 migration skill description so it parses as YAML - #3779
Conversation
🦋 Changeset detectedLatest commit: e909e0c The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
@SergioDiez is attempting to deploy a commit to the Vercel Labs Team on Vercel. A member of the Team first needs to authorize it. |
…YAML Signed-off-by: Sergio Diez <sergiodiez@users.noreply.github.com>
bc1ec73 to
e909e0c
Compare
|
No backport to The only substantive change is to To override, re-run the Backport to stable workflow manually via |
#3779 fixed the app migration skill, whose description ended with `mode: 'client'`. The colon-space made the plain scalar parse as a nested mapping, and the skills CLI dropped the file: `Found 5 skills`, and `--skill migrating-workflow-v4-to-v5` answered `No matching skills found`, which is the command whats-new.mdx tells readers to run. The World skill's description has no colon-space today, so it parses. It is also 590 characters of backticked identifiers, and any future edit naming a field with its value reintroduces the break. Fold it the same way now, while the value is known good. Verified with the yaml package that the folded scalar is byte-identical, and with `skills add . --skill migrating-world-v4-to-v5` that the CLI still finds all six. Signed-off-by: Peter Wielander <peter.wielander@vercel.com>
2 participants
Description
npx skills add vercel/workflowfails for every skill in the repo:The
descriptioninskills/migrating-workflow-v4-to-v5/SKILL.mdis a plain YAML scalar that ends with`mode: 'client'`. The:inside it is parsed as a nested mapping, so the frontmatter is invalid. The skills CLI parses everySKILL.md, blocking all of 6 skills from being installed.This PR turns the description into a folded block scalar (
>-).Fixes #3776.
How did you test your changes?
npx skills add <local clone> --listnow enumerates all six skills; onmainit aborts on this file.PR Checklist - Required to merge
pnpm changesetwas run to create a changelog for this PRpatchfor bug fixes,minorfor new features,majorfor breaking changes.pnpm changeset --emptyif you are changing documentation or workbench appsgit commit --signoffon your commits)@vercel/workflowin a comment once the PR is ready, and the above checklist is complete