fix(poteto-mode): resolve principle leaves by sibling path - #3
Merged
Conversation
Agents applying a principle could not find its leaf skill. pi hides skills flagged disable-model-invocation from the system prompt entirely, and no artifact text mapped principle names to paths, so agents guessed directories and failed outside a checkout root. State the sibling-directory rule in the Principles section so any holder of this skill resolves leaves from its own anchor directory.
Owner
Author
|
Independent pre-merge verdict: PASS.
Not merged; no files modified. |
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.
Why
Agents working in poteto mode could not find principle leaf skills. They searched wrong directories and concluded the principles only exist inside the poteto-mode skill file.
Scope
The Principles section of
skills/poteto-mode/SKILL.mdnow states the resolution rule. All principle skills carrydisable-model-invocation: true, which pi honors by hiding them from the system prompt entirely. The original Cursor pstack uses the same flag and the same sibling layout, so the port dropped nothing. The missing piece was a name-to-path mapping inside the artifact. The section now says each bold name below is also the name of a sibling directory of the skill, with../principle-laziness-protocol/SKILL.mdas the example.Tradeoffs
Registering the leaves would make them discoverable by name but would add 24 descriptions to every session prompt across all projects. The path rule keeps that context cost at zero and works from any cwd because it anchors on the loaded skill's own directory.
Blast Radius
One sentence block in one file. Readers of poteto-mode in any harness get the same rule. Subagents still receive resolved file pointers from the parent, unchanged.
Verification
Extracted every
principle-*reference across poteto-mode, its playbooks, its references, no-comments, and docs, and resolved each against the anchor directory with the stated rule. 21 of 21 referenced names map to existing leafSKILL.mdfiles. An earlier draft of the line used a placeholder token that a mechanical resolver tripped over; reworded to a real example.node scripts/validate.mjspasses on all 49 skills.