Skip to content

Add v-development skill 🤖🤖🤖 - #3296

Open
ulises-jeremias wants to merge 1 commit into
github:mainfrom
ulises-jeremias:add-v-development-skill
Open

ulises-jeremias wants to merge 1 commit into
github:mainfrom
ulises-jeremias:add-v-development-skill

Conversation

@ulises-jeremias

Copy link
Copy Markdown

Adds a new v-development skill that teaches Copilot how to work with the V language (vlang): toolchain setup, v.mod project 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:validate passes (419/419) and npm run build regenerated docs/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.

Teach Copilot V language development: toolchain, v.mod layout, build/test/fmt, and Option/Result error handling.
Copilot AI balanced review requested due to automatic review settings September 17, 2026 21:01
@github-actions

Copy link
Copy Markdown
Contributor

🔒 PR Risk Scan Results

Scanned 1 changed file(s).

Severity Count
🔴 High 0
🟠 Medium 0
ℹ️ Info 0

✅ No matching risk patterns were detected in changed files.

This is an automated soft-gate report. Findings indicate review targets and do not block merge by themselves.

@github-actions github-actions Bot added new-submission PR adds at least one new contribution skills PR touches skills labels Sep 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔴 Contributor Reputation Check: HIGH risk

Check Risk
Profile HIGH
Credential audit NONE

Maintainers: please review this contributor before merging.
See the workflow run for full details.
Automated check powered by AGT.

@github-actions github-actions Bot added the needs-review:HIGH Contributor reputation check flagged HIGH risk label Sep 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔍 Vally Lint Results

✅ All checks passed

Scope Checked
Skills 1
Agents 0
Total 1
Severity Count
❌ Errors 0
⚠️ Warnings 0
ℹ️ Advisories 0

Summary

Level Finding
ℹ️ ✅ v-development (2/2 checks passed)
ℹ️ ✓ [spec-compliance] All 1 skill(s) are spec-compliant.
ℹ️ ✓ spec-compliance: All spec checks passed.
ℹ️ ✓ [valid-refs] All file references across 1 skill(s) are valid.
ℹ️ ✓ valid-refs: All file references resolve to existing files within the skill directory.
ℹ️ 1 skill(s) linted, 1 passed
Full linter output
### Linting skills/v-development
✅ v-development (2/2 checks passed)
    ✓ [spec-compliance] All 1 skill(s) are spec-compliant.
        ✓ spec-compliance: All spec checks passed.
    ✓ [valid-refs] All file references across 1 skill(s) are valid.
        ✓ valid-refs: All file references resolve to existing files within the skill directory.

1 skill(s) linted, 1 passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 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-development skill.
  • 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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review:HIGH Contributor reputation check flagged HIGH risk new-submission PR adds at least one new contribution skills PR touches skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants