Repository navigation
fix(pstack): list every waiting item so each ketchup call is decidable - #692
Merged
Merged
Conversation
The 150-word cap and the two-line action item made the model fold a set of PRs into a count and send the rest of the calls to a linked file. Give each item its own bullet with its link, user change, evidence, risk, and pick, and let the reply grow with the item count. Keep structural diagrams and ban status and action-item diagrams.
The model named three PRs by title only, asked calls between options without the PR number, and left out the risk on low-risk merges. Ask for the number or link in every call, and for each item's risk even when it is low.
…stays decidable A call about one item is one bullet with the item named once. A call between options takes its risk from the options' costs. Say what waiting costs only when it is more than a delay, and size the reply by item count so a two-call chat stays well under 150 words.
The row still promised a few plain sentences. Ketchup now lists every waiting item, so a large batch makes a long reply.
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
/ketchupcapped the whole reply at 150 words and gave each action item two lines. When many items waited, the model folded a set of PRs into a count, such as "Merge the 12?", and moved the other calls to a linked file. The user could not decide from the reply.What changed
Scope
pstack/skills/ketchup, the matchingpoteto-helpline and README row, and their regenerated Claude Code copies./broor the screenshot rules.Blast Radius
Only
/ketchupreplies change. A large batch now makes a long reply, by design. On a small chat with two calls, replies stay near the old size, about 20% longer. The extra words are the evidence and risk that the old text left out.Verification
node scripts/validate-plugins.mjsandnode scripts/check-pstack-docs.mjspass, andnode scripts/generate-pstack-harness-trees.mjs --checkreports the harness tree up to date.Note
Low Risk
Prompt-only changes to the
/ketchupskill and help copy; behavior shifts to longer replies on large batches but no runtime code or security paths.Overview
/ketchupno longer squeezes the whole catch-up into ~150 words. The narrative recap is capped around 100 words (shorter when little happened), while each thing waiting on the user gets ~30 words, so replies grow with backlog size instead of collapsing many PRs into “merge the 12?” or pushing detail behind links.Action items are reframed as explicit calls with a shape that matches the decision: one bullet per single-item call (call, recommendation, user impact, evidence, risk); shared calls asked once, then one bullet per item (link/name, impact, evidence, risk, riskiest first). Counts can’t replace lists and links can’t replace items—the reply alone must be enough to decide. Option-style calls weigh gain/cost per option (that tradeoff is the risk); separate risk lines drop in that case.
Docs stay aligned:
ketchup/SKILL.mdfrontmatter and rules, plus the/ketchupblurbs inREADME.mdandpoteto-help. Visual rules move slightly—no diagrams of action-item lists or status lists; stack/flow/before-after diagrams remain.Reviewed by Cursor Bugbot for commit 55c6ce9. Bugbot is set up for automated code reviews on this repo. Configure here.