chore: guard start-work skill to require an issue before coding#864
Merged
Conversation
Add a Precondition that routes untracked work (a conversational request, a review finding, an idea) through webjs-file-issue before any branch or code, and state the no-code-before-a-tracked-issue rule in when_to_use. Closes the gap where a feature shipped in a PR with no Closes #N and never appeared on the board.
vivek7405
force-pushed
the
chore/start-work-issue-guard
branch
from
4d5a3c2 to
80706db
Compare
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.
Closes #863
What
Adds a Precondition to
webjs-start-work(.claude/skills/webjs-start-work/SKILL.md): if a task has no tracked issue yet, STOP and file one viawebjs-file-issuebefore any branch or code. Also states the "no code before a tracked issue + a branch from it" rule inwhen_to_useso the skill is the funnel for ALL new work, not only when the user names an existing issue.This closes the gap where #855 was implemented and merged before any issue existed (filed retroactively as #858 only after the user noticed).
Note: home-vs-repo skill drift
The repo copy (
.claude/skills/webjs-start-work/SKILL.md, the versioned source thattest/hooks/route-skills.test.mjsasserts) has drifted from a machine-local~/.claude/skills/copy that actually executes for this user. This PR updates the repo source only. The local copy should be re-synced separately so the guard is live.Tests
node --test test/hooks/route-skills.test.mjs: 13/13.https://claude.ai/code/session_016RNAZ8EfZYE8ZBpKUCzUao