feat(pstack): add /correct skill - #494
Merged
Merged
Conversation
/correct turns mistakes the operator keeps correcting into repo changes that make each one impossible. It mines history for mistake classes, fixes each at the highest level that works, and keeps a table that pairs each rule with what enforces it. Co-authored-by: lauren <poteto@users.noreply.github.com>
Co-authored-by: lauren <poteto@users.noreply.github.com>
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
Operators correct agents for the same mistakes again and again.
/correctchanges the repo so the next agent cannot make the same mistake again. It finds mistake classes in history and fixes each class at the highest level that works. The order is architecture, then types and lint, then tests, and docs last.What changed
pstack/skills/correct/SKILL.md. It setsdisable-model-invocation: true, the same as the other operator-invoked pstack skills. The body keeps the source prompt's content and order. The edits are only for skill format: second person to the agent, "the operator" for the human, and sentence-case headings. The description is the operator's short form in pstack style./correctrow to the README skills table and a line to the examples block. The guide and thepoteto-moderouting do not list every skill, so they do not change.Verification
node scripts/validate-plugins.mjspasses.source,metadata,credit,adapted,based on,notes from, author names, URLs), and the mechanism names from the operator's notes. The only hits are thedisable-model-invocationkey and the prompt's own words "internals" and "source of truth".make checkin both runs, and each error names what to use instead. The runs differ only in design choices. The raw-prompt run added a money type and a check for tests that assert nothing. The skill run used a lint for money and added fewer lines (542 against 1143). The skill version lost nothing.