Repository navigation
restore mermaid in ketchup visuals - #39
Conversation
Co-authored-by: Kaitian Xie <caelaxie@users.noreply.github.com>
caelaxie
left a comment
There was a problem hiding this comment.
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/visualizechart 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 pluginfails 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 installedskills/ketchup/SKILL.mdshowing the mermaid lines.
merge once proof is green.
|
head installed line 37: line 43: 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 |
caelaxie
left a comment
There was a problem hiding this comment.
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.
restore the upstream ketchup visual sentences. drop only the /visualize phrases and keep mermaid for structure and numbers.
Closes #32