docs: show output path override example#1624
Open
YOMXXX wants to merge 1 commit into
Open
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.
What problem are you trying to solve?
Issue #939 reports that project-level output path preferences can be missed when an agent follows the concrete Superpowers defaults for design specs and implementation plans.
The issue's working workaround is to put an explicit imperative instruction next to the
Output Pathstable. The maintainer direction on #939 was that the right fix is a README example showing how to override the default.What does this PR change?
Adds a README-only
Customizing Output Pathssection that shows users how to configure alternate spec and plan locations in their project instruction file.The example includes both the table and the direct MUST/NOT instruction from the working workaround, without changing any skill behavior.
Is this change appropriate for the core library?
Yes. This is general Superpowers user documentation for where core workflow artifacts are written. It is not project-specific, does not add a service or dependency, and does not change behavior-shaping skill content.
What alternatives did you consider?
I considered editing
skills/brainstorming/SKILL.mdandskills/writing-plans/SKILL.md, but that repeats the risky part of #1020: behavior-shaping skill edits without multi-session eval evidence.I also considered adding a README regression test, but #1593 was closed partly because that test only grepped for strings the PR itself added. This PR intentionally keeps the change README-only, with no decorative test.
Does this PR contain multiple unrelated changes?
No. It changes one README section for one documentation gap: how to override the default output paths.
Existing PRs
#1020 bundled skill edits with README documentation and did not follow the PR template. #1593 was closer, but it was part of a batch and added a tautological grep test. This PR is the narrower shape requested in the #1593 closure comment: README-only, no decorative regression test.
Environment tested
New harness support (required if this PR adds a new harness)
Not applicable. This PR does not add support for a new harness.
Clean-session transcript for "Let's make a react todo list"
Evaluation
Output Pathstable plus a direct MUST/NOT instruction next to it.Verification run locally:
Not run as a PR signal:
node --test tests/pi/test-pi-extension.mjs. In this shell it cannot import.pi/extensions/superpowers.tsdirectly and failed the same way before the README edit.Rigor
superpowers:writing-skillsand completed adversarial pressure testing (paste results below)This is not a skills change. It deliberately avoids behavior-shaping skill edits and does not add a tautological docs test.
Human review
Fixes #939.