skills: vendor marimo skills, add wandb-specific skill#637
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR introduces an .agents/skills/ skill set to guide AI agents in authoring marimo notebooks in this repository, including vendored upstream marimo skills plus a repo-specific “example notebook” skill.
Changes:
- Added a skills index README describing the local vs. vendored skills and vendoring provenance.
- Vendored
marimo-notebookandjupyter-to-marimoskills (plus references and Apache-2.0 licenses) frommarimo-team/skills. - Added a repo-specific
marimo-example-notebookskill encoding W&B / repo conventions for marimo examples.
Reviewed changes
Copilot reviewed 21 out of 21 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
| .agents/skills/README.md | Documents skill structure, lists skills, and documents vendoring details. |
| .agents/skills/marimo-notebook/SKILL.md | Vendored general guidance for writing marimo notebooks, including running, reactivity, and testing notes. |
| .agents/skills/marimo-notebook/references/WATCHING.md | Vendored reference on --watch and external editing behavior. |
| .agents/skills/marimo-notebook/references/UI.md | Vendored reference on common marimo UI components and form patterns. |
| .agents/skills/marimo-notebook/references/TOP-LEVEL-IMPORTS.md | Vendored reference on making notebook-defined symbols importable. |
| .agents/skills/marimo-notebook/references/STATE.md | Vendored reference on state/reacitivity patterns and mo.state(). |
| .agents/skills/marimo-notebook/references/SQL.md | Vendored reference on mo.sql usage and engines. |
| .agents/skills/marimo-notebook/references/REACTIVITY.md | Vendored reference on DAG/reactivity rules and mutation pitfalls. |
| .agents/skills/marimo-notebook/references/PYTEST.md | Vendored reference on pytest discovery/execution in marimo notebooks. |
| .agents/skills/marimo-notebook/references/EXPORTS.md | Vendored reference on exporting marimo notebooks via CLI. |
| .agents/skills/marimo-notebook/references/EXPENSIVE.md | Vendored reference on performance patterns (mo.stop, caching, lazy). |
| .agents/skills/marimo-notebook/references/DEPLOYMENT.md | Vendored reference on deploying notebooks / thumbnails. |
| .agents/skills/marimo-notebook/references/CONFIGURATION.md | Vendored reference on marimo configuration scopes and discovery. |
| .agents/skills/marimo-notebook/references/COLUMNS.md | Vendored reference on multi-column notebook layout. |
| .agents/skills/marimo-notebook/references/ANYWIDGET.md | Vendored reference/skill content for anywidget usage in marimo. |
| .agents/skills/marimo-notebook/LICENSE | Apache-2.0 license for vendored marimo-notebook skill content. |
| .agents/skills/marimo-example-notebook/SKILL.md | Repo-specific marimo example notebook guidance (structure + W&B integration patterns). |
| .agents/skills/jupyter-to-marimo/SKILL.md | Vendored guidance for converting Jupyter notebooks to marimo notebooks. |
| .agents/skills/jupyter-to-marimo/references/widgets.md | Vendored mapping and patterns for porting ipywidgets → marimo UI/anywidget. |
| .agents/skills/jupyter-to-marimo/references/latex.md | Vendored guidance for migrating MathJax LaTeX → KaTeX-friendly marimo strings. |
| .agents/skills/jupyter-to-marimo/LICENSE | Apache-2.0 license for vendored jupyter-to-marimo skill content. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| ## Responding to other files | ||
|
|
||
| marimo has `mo.watch.file` and `mo.watch.file` utilities that can cause cells to update when a file/folder updates. |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 21 out of 21 changed files in this pull request and generated 4 comments.
Comments suppressed due to low confidence (2)
.agents/skills/marimo-notebook/references/WATCHING.md:55
- This sentence lists
mo.watch.filetwice; it looks like a copy/paste typo and also makes the guidance unclear.
marimo has `mo.watch.file` and `mo.watch.file` utilities that can cause cells to update when a file/folder updates.
.agents/skills/marimo-example-notebook/SKILL.md:16
- This claims a “canonical exemplar” notebook at
examples/marimo/mnist-registry/mnist_registry.py, but there is noexamples/marimo/directory in the repo right now. Pointing to a non-existent file will confuse agents following this skill.
The canonical exemplar is
`examples/marimo/mnist-registry/mnist_registry.py` — when in doubt, match
its structure.
Comment on lines
+1
to
+4
| --- | ||
| name: anywidget-generator | ||
| description: Generate anywidget components for marimo notebooks. | ||
| --- |
johndmulhausen
approved these changes
Jul 21, 2026
johndmulhausen
self-requested a review
July 21, 2026 18:55
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 21 out of 21 changed files in this pull request and generated no new comments.
Comments suppressed due to low confidence (1)
.agents/skills/marimo-example-notebook/SKILL.md:16
- This skill points to
examples/marimo/mnist-registry/mnist_registry.pyas the canonical exemplar, but there is noexamples/marimo/directory (or that file) in the repo. This makes the guidance immediately dead-ending for users/agents; either add the exemplar in this PR or update the text to not reference a non-existent path.
The canonical exemplar is
`examples/marimo/mnist-registry/mnist_registry.py` — when in doubt, match
its structure.
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
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.
This PR adds a few agent skills for authoring marimo notebooks.
marimo-example-notebook: this adds a skill for creating or refactoring example marimo notebooks, encoding best practices.marimo-notebook: a general skill for writing marimo notebooks; vendored frommarimo-team/skillsjupyter-to-marimo: a general skill for converting Jupyter notebooks to marimo; vendored frommarimo-team/skills.