Vaultspec code review
Skill nevenincs/vaultspec-core/.vaultspec/skills/vaultspec-code-review
A spec-driven harness for coding agents (and, humans)
npx -y skills add nevenincs/vaultspec-core --skill vaultspec-code-reviewAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 1 stars1 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
What its author says it does
Copied from the file, not written here
Run a formal code review for safety, intent, and quality. Use to verify completed work before marking it done.
SKILL.md
2.1 KB, as published. Nobody here has run it
Code review skill (vaultspec-code-review)
When to use this skill:
- Mandatory: At the end of every
vaultspec-executecycle, before marking a feature as "Done", and before publishing a PR. - After major feature implementation work.
- When you need a second pair of eyes on a specific module or PR.
- Safety Check: When you suspect a safety violation (e.g.,
unsafeusage) or testing framework issue in complex projects.
Workflow
-
Announce at start: "I'm using the
vaultspec-code-reviewskill to audit the implementation." -
Identify relevant docs, the plan (
.vault/plan/...), ADR and research documents -
Identify files modified
-
Scaffold the audit document with
vaultspec-core vault add audit --feature {feature}; the CLI owns the filename and frontmatter. Log discovered issues to its body as triagedLOW->CRITICALtask entries. -
Use a
vaultspec-code-revieweragent persona, or other code-review specialists. -
Use parallel subagents to comprehensively comb through codebase.
-
Instruct agents to always read grounding docs, ADRs, and plans.
-
Instruct agents to log findings as triaged issue entries into the single shared scaffolded audit document's body.
-
Code review is not a code fixer skill - do NOT modify the codebase.
Important
-
Template: You MUST read and use the template at
.vaultspec/templates/audit.md; its embedded hint blocks govern the body structure, and its## Findingssection carries the rolling per-finding log format. -
Location: the scaffold creates
.vault/audit/yyyy-mm-dd-{feature}-audit.md; never hand-write the filename or frontmatter. When the feature already carries an audit, disambiguate with the optional narrative infix:yyyy-mm-dd-{feature}-{topic}-audit.md. -
Tags: the scaffold tags the audit document with
#auditand#{feature}; verify viavaultspec-core vault check allrather than hand-editing. -
Issues must be continuously appended to the audit document as a rolling log of open tasks.