Skip to content

port upstream ketchup #692 and the #708 sentence - #6

Merged
caelaxie merged 1 commit into
mainfrom
cursor/port-ketchup-692-5f7f
Oct 18, 2026
Merged

caelaxie merged 1 commit into
mainfrom
cursor/port-ketchup-692-5f7f

Conversation

@caelaxie

Copy link
Copy Markdown
Contributor

ports upstream #692 (16541c6) + #708 sentence verbatim that #5 missed, per portreview's review on #5 (#5 (review)). adaptations kept: triggers: [user], /pstack:, xychart-beta. harness install/list is blocked on missing devin auth.

Open in Web Open in Cursor 

Copy the 0.15.26 ketchup skill and its poteto-help row verbatim, keeping the port's /pstack: prefixes, triggers: [user], and xychart-beta wording.

Co-authored-by: Kaitian Xie <[email protected]>

@caelaxie caelaxie left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

verdict: changes requested (posted as COMMENT since author == reviewer).

  1. minimum changes: pass. skills/ketchup/SKILL.md now matches upstream main (16541c6 #692 + bc14437 #708) line for line, #708 sentence verbatim. only diffs left are the existing harness adaptations: triggers: [user] for disable-model-invocation, /pstack: prefixes, and mermaid xychart-beta for /visualize. poteto-help ketchup row matches upstream except the prefix.
  2. parity: pass. closes the #692 gap from #5; README already says 0.15.26. Validate plugin is green on 408c303.
  3. harness proof: missing. the body says devin install/list is blocked on devin auth. need devin plugins install + devin plugins list output on this head, plus ketchup run once (or its installed SKILL.md shown with the new action-item section). won't merge without it.
  4. issue linkage: n/a, no issue.

once proof lands (comment or body edit), this merges as is.

@caelaxie
caelaxie merged commit fa4e874 into main Oct 18, 2026
1 check passed
@caelaxie
caelaxie deleted the cursor/port-ketchup-692-5f7f branch October 18, 2026 23:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants