refactor(aidd-context): conform 09-mermaid to the skill contract#286
Merged
Conversation
Make the skill follow the same conventions as the reworked context skills. - SKILL.md becomes a real router: an Actions table and transversal rules, instead of embedding the action file. Tight description with when and not-for. Title is "Mermaid", not "Skill: mermaid". - the action uses the Input/Output/Process/Test anatomy (singular) and drops the frozen yaml input block and the frozen mermaid output block. - move the LR default and the minimum version into the conventions reference, where the per-format data belongs. - README: fix the stale "04" heading to "09" and the framework link. Validated via headless claude -p: the skill plans, generates a valid flowchart from the confirmed plan, and applies the conventions reference.
Verified the rules against the official Mermaid syntax docs and fixed three that were wrong or misleading: - `==>` is a thick link, not a self-loop. A self-loop is `A --> A`. - `<br>` is supported, so drop the absolute ban; keep it a style preference (a literal `\n` does render wrong, that ban stays). - a `:` in a state id collides with the description delimiter, so keep it out of the id rather than swapping it for `$`. Also reword the styling rule to its real intent (do not style unless asked) and state that the global sections apply to every diagram type.
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
1 participant
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.
🎯 What & why
Bring
aidd-context:09-mermaidin line with the skill contract the other context skills now follow. The oldSKILL.mdembedded its action file instead of routing, the action used pluralInputs/Outputswith frozenyaml/mermaiddata blocks, and the README still carried a stale04heading.🛠️ How it works
SKILL.mdis a real router: an## Actionstable plus transversal rules (plan, confirm, generate only the confirmed plan, follow the conventions), and a tight description with when and not-for.Input/Output/Process/Testanatomy and drops the frozen blocks.LRdefault and the minimum Mermaid version move intoreferences/mermaid-conventions.md, where the per-format data belongs.04to09, framework link corrected.🧪 How to verify
grep -rnE '—|->|;' plugins/aidd-context/skills/09-mermaid/SKILL.md plugins/aidd-context/skills/09-mermaid/actionsis clean; the action carries the four anatomy sections;SKILL.mdhas Actions + transversal rules + References.claude -p: the skill plans the diagram, generates a validflowchart LRfrom the confirmed plan, and applies the conventions reference (title via---, declare-first/link-last, datastore cylinder).✅ I certify