Add v-development skill 🤖🤖🤖 - #3296
ulises-jeremias wants to merge 1 commit into
Conversation
Teach Copilot V language development: toolchain, v.mod layout, build/test/fmt, and Option/Result error handling.
🔒 PR Risk Scan ResultsScanned 1 changed file(s).
✅ No matching risk patterns were detected in changed files.
|
|
🔴 Contributor Reputation Check: HIGH risk
Maintainers: please review this contributor before merging. |
🔍 Vally Lint Results✅ All checks passed
Summary
Full linter output |
There was a problem hiding this comment.
🟡 Changes recommended
The skill contains incorrect v.mod dependency fields and misleading project entry-point guidance.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
Adds V language development guidance covering tooling, project structure, and idiomatic error handling.
Changes:
- Adds the
v-developmentskill. - Registers it in the generated skills catalog.
File summaries
| File | Description |
|---|---|
skills/v-development/SKILL.md |
Defines V development guidance. |
docs/README.skills.md |
Adds the skill to the catalog. |
Review details
Suppressed comments (1)
skills/v-development/SKILL.md:86
- This repeats the invalid manifest key from the example. Dependencies must be recorded under
dependencies, otherwise the skill will direct users to edit a field that V does not consume.
- When adding a dependency, record it in `v.mod` under `deps` and mention the `v install` command.
- Files reviewed: 2/2 changed files
- Comments generated: 3
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
|
||
| ## Project layout | ||
|
|
||
| Every project has a `v.mod` file at its root: |
| Module { | ||
| name: 'myapp' | ||
| version: '0.1.0' | ||
| deps: [] |
|
|
||
| Conventions: | ||
|
|
||
| - `main.v` (or `module main`) is the entry point; `fn main()` starts execution. |
Adds a new
v-developmentskill that teaches Copilot how to work with the V language (vlang): toolchain setup,v.modproject layout, build/test/format/vet commands, and idiomatic V including Option/Result error handling, immutability, and no-null rules.There is currently no V coverage among the 419 skills, and frontier models frequently emit invalid V (Go/Rust idioms translated literally, invented null checks). This skill addresses that gap with stable, checkable toolchain facts grounded in the official V docs.
Validation performed:
npm run skill:validatepasses (419/419) andnpm run buildregenerateddocs/README.skills.md, both included in this PR. Content was cross-checked against https://docs.vlang.io and the V toolchain; I could not live-test against Copilot itself in this environment, so reviewer testing is welcome.Disclosure: I maintain V ecosystem projects (RxV, Create Awesome V App) and the Agent Toolkit, whose V practices informed this skill. The skill text itself is original and vendor-neutral.