Repository navigation
Conversation
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.
Summary
Fixes #367.
Four pstack skills set
disable-model-invocation: true(slash-only) but theirdescriptionstill reads as agent-selection guidance (Use only when the user explicitly asks...,Use when reading or editing...,Use when the user says reflect). That phrasing only makes sense for model auto-invocation; on a slash-only skill it lands in context on every/tddand tells the model when to pick a skill it cannot pick. This rewrites those four descriptions as a short capability summary plus the slash trigger, and leaves the flag in place.tdd: describe the workflow and when it applies; dropUse only when the user...typescript-best-practices: capability summary; keeppathsand the flagmake-bot-ui: capability summary plus/make-bot-uireflect:Run it with /reflectplugin.json: 0.15.15 -> 0.15.16Same scope as #392, which has been conflicting with
mainsince Sep 25 with no update; this is the same change rebased on the current tree.unslopis covered separately by #514.Test plan
npm install --no-save ajv ajv-formats && node scripts/validate-plugins.mjs: All plugins validated successfullygit diff --checkcleandisable-model-invocation: truestill present on all four skills