Add VillageSQL Skills plugin#158
Conversation
AI=CLAUDE Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request registers the "VillageSQL Skills" plugin by adding it to the README list, creating its configuration file (plugin.json), and adding its icon asset. However, the skills directory referenced in the configuration is missing from the pull request, which would make the plugin non-functional.
| "brandColor": "#CC66FF", | ||
| "composerIcon": "./assets/icon.svg" | ||
| }, | ||
| "skills": "./skills/" |
There was a problem hiding this comment.
The skills property points to ./skills/, but the actual skills directory and its contents (such as SKILL.md) are missing from this pull request. Please include the necessary skill files in the plugin bundle so that the plugin is functional.
AI=CLAUDE Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
AI=CLAUDE Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
internet-dot
left a comment
internet-dot
left a comment
There was a problem hiding this comment.
This PR only adds a README entry and plugin bundle but is missing required index entries. A complete submission needs:
- Entry in
plugins.json(with name, url, owner, repo, description, category, source, install_url) - Entry in
.agents/plugins/marketplace.json
See CONTRIBUTING.md for the full spec.
Also: the repo uses an "Other" (non-standard) license. Please confirm this is intentional.
|
Thanks for the review! Both missing entries have been added:
On the license question: the repo has a full Apache 2.0 |
Rebase onto main, keeping VillageSQL Skills entry alongside newly-added Alcove, Epic Harness, Runtype Skills, and VidSeeds.ai. AI=CLAUDE
Code Review SummaryStatus: 1 Issue Found | Recommendation: Address before merge Overview
Issue Details (click to expand)WARNING
Files Reviewed (1 files)
Reviewed by laguna-m.1-20260312:free · 2,285,469 tokens |
Add Alcove, Epic Harness, Runtype Skills (merged from main), VillageSQL Skills entry, and bump total from 89 to 93. AI=CLAUDE
Set total to 92 to match main; VillageSQL entry is present making 93 actual entries. Bot will sync the count post-merge. AI=CLAUDE
![kilo-code-bot[bot]](https://avatars.githubusercontent.com/in/2193792?s=60&v=4){kind=small}
| "last_updated": "2026-05-25", | ||
| "total": 88, | ||
| "last_updated": "2026-05-27", | ||
| "total": 92, |
There was a problem hiding this comment.
WARNING: The total field value (92) does not match the actual number of plugins in the array (93). This field should accurately reflect the count of plugins for data integrity.
2 participants
Plugin submission: VillageSQL Skills
Repository: https://github.com/villagesql/villagesql-skills
Description: Skills for VillageSQL including building extensions from scratch and porting PostgreSQL extensions to VillageSQL.
Category: Development & Workflow
PR Checklist
plugins/villagesql/villagesql-skills/.codex-plugin/plugin.jsonexists and is valid JSONcomposerIconfield is set inplugin.jsoninterface sectionassets/icon.svg