docs: update dev note with TL;DR tips and install instructions#461
Merged
johnnygreco merged 4 commits intomainfrom Mar 25, 2026
Merged
docs: update dev note with TL;DR tips and install instructions#461johnnygreco merged 4 commits intomainfrom
johnnygreco merged 4 commits intomainfrom
Conversation
Add a tip block with four key lessons for building agent skills to the "Data Designer Got Skills" dev note. Remove the Claude Code marketplace install option from both the blog and README, keeping only the skills.sh method. Update skill mode descriptions and clarify Claude Code testing scope.
Contributor
Greptile SummaryThis is a documentation-only PR that streamlines the agent skill install story and enriches the "Data Designer Got Skills" dev note with actionable TL;DR tips. Key changes:
|
| Filename | Overview |
|---|---|
| docs/devnotes/posts/data-designer-got-skills.md | Adds a TL;DR admonition tip block with four lessons for building agent skills, appends a sentence about CLI curated context to the "Agents as First-Class Users" section, fixes punctuation in the "Coding best practices" paragraph, clarifies interactive mode as the default, and expands the autopilot mode description with concrete trigger phrases. No issues found. |
| README.md | Removes the Claude Code marketplace install block, consolidates to skills.sh only, and fixes the devnote URL from devnotes/posts/data-designer-got-skills/ to devnotes/data-designer-got-skills/ — the correct path per the MkDocs blog plugin's post_url_format: "{slug}" setting. Also adds context about Claude Code being the primary tested agent. |
| .claude-plugin/marketplace.json | Deleted entirely — this file defined the Claude Code marketplace entry that pointed to ./claude-plugin. Removal is intentional as the marketplace install path is being dropped in favour of skills.sh. |
| claude-plugin/.claude-plugin/plugin.json | Deleted — this was the plugin metadata used exclusively by the marketplace install path. The canonical plugin.json at skills/data-designer/.claude-plugin/plugin.json remains and is the correct location for the skills.sh workflow. |
| claude-plugin/skills | Deleted symlink (../skills) — this provided the skills/ tree to the marketplace plugin bundle. No longer needed now that the marketplace path is removed; the actual skill files under skills/data-designer/ remain intact. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A([User wants to install the skill]) --> B{Install method}
B -->|skills.sh — only supported path| C["npx skills add NVIDIA-NeMo/DataDesigner\n(select Claude Code as agent)"]
B -->|~~Claude Code marketplace~~ removed| X[❌ Removed in this PR]
C --> D[Skill available in Claude Code]
D --> E{Skill mode}
E -->|Default| F["Interactive mode\n— agent asks clarifying questions,\n user reviews samples & iterates"]
E -->|'be opinionated' / 'surprise me'| G["Autopilot mode\n— agent makes all design decisions,\n validates & generates without waiting"]
F --> H["Output: standalone Python script\ncalling Data Designer's public API"]
G --> H
Reviews (1): Last reviewed commit: "docs: remove claude-plugin marketplace d..." | Re-trigger Greptile
andreatgretel
approved these changes
Mar 25, 2026
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.
Summary