fix(using-superpowers): re-check skills mid-workflow, not only at turn start - #2068
Open
arimu1 wants to merge 1 commit into
Open
fix(using-superpowers): re-check skills mid-workflow, not only at turn start#2068arimu1 wants to merge 1 commit into
arimu1 wants to merge 1 commit into
Conversation
…n start The bootstrap already required skill invocation before any action, but agents treated that as a once-per-turn-start check. Once a process skill (e.g. brainstorming) was active, other skills whose triggers appeared mid-conversation were never re-scanned. Make continuous re-check explicit and close the observed rationalizations in Red Flags. Fixes obra#2051.
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.
Who is submitting this PR? (required)
What problem are you trying to solve?
Real session reported in #2051 (Claude Code + Superpowers 6.2.0, Claude Sonnet 5).
using-superpowersstates that a relevant skill should be checked before any action, with even a 1% chance of relevance enough to invoke it. In the reported session:brainstormingengaged correctly for a rate-limiting design.When asked why, the agent explained (verbatim from the issue):
Root instruction gap: the bootstrap forbids skipping skill checks before action, but does not say the check is continuous. Agents rationalize "I already checked this turn" / "I'm mid-workflow so skill check is done."
A third-party skill cannot fix this from its own body: trigger decisions run against short frontmatter descriptions; body text only loads after the skill is already chosen. Only
using-superpowerscan restate continuous re-check.Issue author noted prompt-based enforcement may not hard-guarantee compliance; this PR closes the documented wording gap and the exact rationalizations observed.
What does this PR change?
Skill-content only in
skills/using-superpowers/SKILL.md:No harness/hook/code changes. No third-party skill content.
Is this change appropriate for the core library?
Yes. Continuous skill discovery is core bootstrap behavior for every Superpowers user on every project. It is not domain-specific, not a third-party integration, and not a project-local workflow. Third-party skills cannot fix this from their own files (as #2051 demonstrated).
What alternatives did you consider?
descriptionfrontmatter — Rejected: description is discovery/when-to-load; this skill is session bootstrap. Body is where continuous behavior is enforced. Description already says "before ANY response."Does this PR contain multiple unrelated changes?
No. Single file, single failure mode: skill check treated as turn-start only while mid-workflow.
Existing PRs
Searched open and closed PRs for:
2051,using-superpowers,mid-workflow,check continuously,before every action. Nearby using-superpowers PRs (#2010 dangling anchor, #1894 Claude Code platform adaptation, harness PRs) do not address continuous re-check. Related issues cited by #2051 (#1007 composition, #1812 memory) are different problems.Environment tested
Skill-content-only change; no runtime harness integration. Grounding session is the #2051 transcript (Claude Code / Claude Sonnet 5).
New harness support (required if this PR adds a new harness)
N/A — this PR does not add a new harness.
Clean-session transcript for "Let's make a react todo list"
Evaluation
Rigor
Human review
Fixes #2051