Skip to content

feat: add Excalidraw diagram tooling#1070

Merged
ericgregory merged 4 commits into
wasmCloud:mainfrom
ericgregory:add-diagram-tooling
May 20, 2026
Merged

feat: add Excalidraw diagram tooling#1070
ericgregory merged 4 commits into
wasmCloud:mainfrom
ericgregory:add-diagram-tooling

Conversation

@ericgregory
Copy link
Copy Markdown
Contributor

This PR adds tooling and documentation for consistent generation of diagrams with Excalidraw and optionally Claude Code

@ericgregory ericgregory requested a review from a team as a code owner February 18, 2026 15:52
@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 18, 2026

Deploy Preview for dreamy-golick-5f201e ready!

Name Link
🔨 Latest commit ec3486c
🔍 Latest deploy log https://app.netlify.com/projects/dreamy-golick-5f201e/deploys/6a0cb75f7b05fd000807e518
😎 Deploy Preview https://deploy-preview-1070--dreamy-golick-5f201e.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@vados-cosmonic
Copy link
Copy Markdown
Contributor

Hey @ericgregory can we push this in? Though I like the idea of suggesting people install the skill instead of explicitly including it as a submodule (e.g. on the chance that the skill is unnecessary in the future)

@lachieh
Copy link
Copy Markdown
Contributor

lachieh commented May 19, 2026

Suggestion: the skills cli has support for a lockfile and the ability to support many different agents

TL;DR:
Run npx skills add robtaylor/excalidraw-diagrams --skill excalidraw-diagrams --agent claude-code
Commit the skills-lock.json file
.gitignore: .agents/skills and .claude/skills

Signed-off-by: Eric Gregory <[email protected]>
Replaces the git-submodule excalidraw-diagrams skill with the
brand-guidelines skill from cosmonic-labs/skills, installed via the
skills CLI lockfile (skills-lock.json). Removes .gitmodules and the
submodule pointer. Adds .agents/skills/ to .gitignore so other agents'
install paths are also ignored (.claude/ was already covered).

The brand-guidelines skill bundles both Cosmonic and wasmCloud
materials; CLAUDE.md and the README point readers at the wasmCloud
sections and keep a short palette inline as quick reference, with the
skill called out as the canonical source.

Addresses review feedback from @vados-cosmonic and @lachieh.

Signed-off-by: Eric Gregory <[email protected]>
@ericgregory ericgregory force-pushed the add-diagram-tooling branch from 1041ec0 to 767417e Compare May 19, 2026 19:16
Clarify the ownership of the brand guidelines skill. Mirrors the same
phrasing change to README.md so the two references stay in sync.

Signed-off-by: Eric Gregory <[email protected]>
@ericgregory
Copy link
Copy Markdown
Contributor Author

Thanks @lachieh and @vados-cosmonic -- I've updated this and it should be ready for review.

@ericgregory ericgregory enabled auto-merge (rebase) May 19, 2026 19:21
Copy link
Copy Markdown
Contributor

@vados-cosmonic vados-cosmonic left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

Thanks for helping with this (and other things recently!) @lachieh

@ericgregory ericgregory merged commit 143ecbf into wasmCloud:main May 20, 2026
7 checks passed
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.

3 participants