Point stable-skill instructions at resources/bundled/skills/#9406
Point stable-skill instructions at resources/bundled/skills/#9406vorporeal merged 1 commit intowarpdotdev:masterfrom
Conversation
The channel-gated skills README and the matching error message in \`script/copy_conditional_skills\` both told contributors to put stable-ready skills in \`resources/skills/\`, but that directory does not exist — the always-bundled skills source is \`resources/bundled/skills/\`, which is what \`script/prepare_bundled_resources\` actually copies. Concretely the error message a contributor would hit if they happened to add a \`stable/\` gate directory was pointing them at the wrong fix. Updated both the README copy and the runtime echo so they match the real path.
|
I'm starting a first review of this pull request. You can view the conversation on Warp. I approved this pull request. No matching stakeholder was found for the changed files, so no human reviewers were requested. Comment Powered by Oz |
![oz-for-oss[bot]](https://avatars.githubusercontent.com/in/3450139?s=60&v=4){kind=small}
There was a problem hiding this comment.
Overview
This PR corrects stale stable-skill guidance in resources/channel-gated-skills/README.md and script/copy_conditional_skills to point contributors at resources/bundled/skills/, matching the existing bundled resource layout and copy path.
Concerns
- No blocking correctness, security, or maintainability concerns found in the changed lines.
Verdict
Found: 0 critical, 0 important, 0 suggestions
Approve
Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).
Powered by Oz
|
thanks for fixing the incorrect paths! |
captainsafia
added
the
external-contributor
Warp Dev Github Integration
3 participants
Description
The channel-gated skills README and the runtime error message in `script/copy_conditional_skills` both pointed contributors at `resources/skills/` for stable-ready skills, but that directory doesn't exist in the repo — `script/prepare_bundled_resources` copies the always-bundled skills from `resources/bundled/`, so the right destination is `resources/bundled/skills/`.
The result: if anyone bumped into the existing `stable/` guard (`copy_conditional_skills` line 39), the printed remediation pointed them at a path that doesn't exist.
Three updates:
No behavior change in the script itself; only the comment and the error string contents.
Testing
Doc/script-comment only — nothing to run. The actual copy logic is unchanged.
Server API
No server changes.
Agent Mode
Not applicable.
Changelog Entries
None.