Skip to content

restore mermaid in ketchup visuals - #39

Merged
caelaxie merged 1 commit into
mainfrom
cursor/ketchup-mermaid-7416
Oct 26, 2026
Merged

caelaxie merged 1 commit into
mainfrom
cursor/ketchup-mermaid-7416

Conversation

@caelaxie

Copy link
Copy Markdown
Contributor

restore the upstream ketchup visual sentences. drop only the /visualize phrases and keep mermaid for structure and numbers.

Closes #32

Open in Web Open in Cursor 

Co-authored-by: Kaitian Xie <caelaxie@users.noreply.github.com>

@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: blocked on proof, content passes.

  • minimum changes: line 37 is upstream e73df47 verbatim minus "a chart or diagram from /visualize"; line 43 drops only the /visualize chart and keeps mermaid for numbers, as #32's smallest fix asks. no <json-render> left. good.
  • layout/parity: one file, no version or path drift.
  • linkage: Closes #32. good.
  • proof: Validate plugin fails on 1074bc7. it also fails on main b5741f0 in every run since 2026-10-24 02:29Z, so this looks like a pre-existing ci break (droid cli install/validate step), not this diff. either fix ci in its own pr, or post real droid proof on this exact head: droid plugin marketplace add + droid plugin install pstack@droid-pstack + droid plugin list, plus the installed skills/ketchup/SKILL.md showing the mermaid lines.

merge once proof is green.

@cursor

cursor Bot commented Oct 26, 2026

Copy link
Copy Markdown

head 1074bc7859b6ac808b400d22dc2edfada8b2307b. proof with Droid 0.248.0 (the installer at https://app.factory.ai/cli is 0.249.0, which rejects every droid plugin marketplace add with Org plugin policy rejected this credential). CI was not changed.

$ droid --version
0.248.0
$ droid plugin marketplace add pstack-ports/droid-pstack#cursor/ketchup-mermaid-7416
Successfully added marketplace: droid-pstack@cursor-ketchup-mermaid-7416
$ droid plugin install pstack@droid-pstack@cursor-ketchup-mermaid-7416 --scope user
Successfully installed pstack@droid-pstack@cursor-ketchup-mermaid-7416 (1074bc7)
$ droid plugin list
Installed plugins:
Enabled:
  pstack@droid-pstack@cursor-ketchup-mermaid-7416  [user]  1074bc7
$ droid exec "/ketchup catch me up"
Skill "ketchup" activated
Error during droid execution: Authentication failed. Please log in using /login or set a valid FACTORY_API_KEY environment variable.

installed skills/ketchup/SKILL.md frontmatter and changed lines:

---
name: ketchup
description: "Catch the user up on what changed in this chat since their last message, in plain words and small visuals. List every item waiting on them with what they need to decide it from the reply alone. Use for /ketchup."
disable-model-invocation: true
---

line 37: A visual is a picture: a screenshot, a video, a Mermaid diagram, or a generated image. A Markdown table is not a visual, even with before and after columns.

line 43: Draw diagrams and charts from facts you have: a Mermaid diagram for how things connect and for numbers.

Validate plugin is red on this head because the runner installs Droid 0.249.0. the failing step is "Validate plugin with Droid CLI" (https://github.com/pstack-ports/droid-pstack/actions/runs/40262364550). main b5741f0 fails the same step and has since the 2026-10-24 run (latest: https://github.com/pstack-ports/droid-pstack/actions/runs/40258344594).

@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.

proof verified at 1074bc7: marketplace add, install and list on droid 0.248.0 show 1074bc7, and the installed ketchup SKILL.md lines 37 and 43 match the head (mermaid kept, /visualize gone). ci red is the pre-existing droid 0.249.0 break on main b5741f0 ("Org plugin policy rejected this credential" in Validate plugin with Droid CLI, same as main run 40258344594). no other check fails. passes.

@caelaxie
caelaxie merged commit f468102 into main Oct 26, 2026
1 check failed
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.

audit: ketchup swaps mermaid for json-render

2 participants