Restructure Repository as a Skills Marketplace#5
Conversation
Moves the chart-visualization tool into a skills/ directory and adds a .claude-plugin/marketplace.json file to configure the repository as a multi-skill marketplace.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
Updates the `name` field in `.claude-plugin/marketplace.json` to "visualization skills" as requested.
Updates the Usage section of the README.md file to include instructions for adding the entire marketplace to Claude Code.
|
Thank you for the documentation link. I have reviewed it and can confirm that the marketplace setup aligns with the guidelines and schema described. |
- Corrects the `name` in `marketplace.json` to use kebab-case. - Updates the installation instructions in `README.md`.
There was a problem hiding this comment.
Pull request overview
This pull request restructures the repository into a skills marketplace by moving the chart-visualization skill into a skills/ directory and adding a .claude-plugin/marketplace.json configuration file. The skill provides 26 chart types for data visualization powered by AntV, with comprehensive reference documentation and a Python script for chart generation.
Changes:
- Restructured repository to support a skills marketplace format with a new
.claude-plugin/marketplace.jsonconfiguration - Added complete chart-visualization skill with 26 chart type reference documents in Chinese
- Added Python generation script that interfaces with the AntV visualization API
- Updated README with marketplace and skill installation instructions
Reviewed changes
Copilot reviewed 2 out of 30 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
.claude-plugin/marketplace.json |
Defines marketplace metadata and plugin configuration for the visualization skills |
skills/chart-visualization/SKILL.md |
Main skill documentation describing workflow, chart selection guidelines, and usage instructions |
skills/chart-visualization/scripts/generate.py |
Python script that handles chart type mapping and API communication for chart generation |
skills/chart-visualization/references/*.md |
26 reference documentation files (in Chinese) describing each chart type's parameters and usage |
README.md |
Updated with marketplace installation commands and skill overview |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "source": "./", | ||
| "strict": false, | ||
| "skills": [ | ||
| "./skills/chart-visualization" |
There was a problem hiding this comment.
The plugin configuration has "source" pointing to "./" (root) while "skills" points to "./skills/chart-visualization". This may cause path resolution issues. Consider either setting "source" to "./skills/chart-visualization" and removing the skills array prefix, or clarifying the intended directory structure. Verify this configuration works correctly with the Claude plugin system.
| "source": "./", | |
| "strict": false, | |
| "skills": [ | |
| "./skills/chart-visualization" | |
| "source": "./skills/chart-visualization", | |
| "strict": false, | |
| "skills": [ | |
| "./" |
This change restructures the repository into a skills marketplace by moving the
chart-visualizationskill into askills/directory and adding a.claude-plugin/marketplace.jsonconfiguration file.PR created automatically by Jules for task 13813595913882550266 started by @wang1212