@vibefoundry/vibe-shield 0.0.0-stage → 1.1.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/LICENSE +21 -0
- package/README.md +571 -3
- package/SECURITY.md +110 -0
- package/SKILL.md +124 -0
- package/dist/bin/vibe-shield.d.ts +59 -0
- package/dist/bin/vibe-shield.js +541 -0
- package/dist/src/index.d.ts +25 -0
- package/dist/src/index.js +108 -0
- package/dist/src/init/scaffold.d.ts +122 -0
- package/dist/src/init/scaffold.js +573 -0
- package/dist/src/mcp/server.d.ts +127 -0
- package/dist/src/mcp/server.js +479 -0
- package/dist/src/mcp-guard/skill-auditor.d.ts +105 -0
- package/dist/src/mcp-guard/skill-auditor.js +1129 -0
- package/dist/src/reporter/console-reporter.d.ts +58 -0
- package/dist/src/reporter/console-reporter.js +254 -0
- package/dist/src/reporter/formatters.d.ts +38 -0
- package/dist/src/reporter/formatters.js +262 -0
- package/dist/src/reporter/patch-prompt-writer.d.ts +35 -0
- package/dist/src/reporter/patch-prompt-writer.js +238 -0
- package/dist/src/scanner/context.d.ts +51 -0
- package/dist/src/scanner/context.js +568 -0
- package/dist/src/scanner/engine.d.ts +130 -0
- package/dist/src/scanner/engine.js +722 -0
- package/dist/src/scanner/gitignore.d.ts +49 -0
- package/dist/src/scanner/gitignore.js +195 -0
- package/dist/src/scanner/lexicon.d.ts +56 -0
- package/dist/src/scanner/lexicon.js +357 -0
- package/dist/src/scanner/rules/ai-vulnerabilities.d.ts +15 -0
- package/dist/src/scanner/rules/ai-vulnerabilities.js +859 -0
- package/dist/src/scanner/rules/client-exposure.d.ts +18 -0
- package/dist/src/scanner/rules/client-exposure.js +410 -0
- package/dist/src/scanner/rules/database-exposure.d.ts +15 -0
- package/dist/src/scanner/rules/database-exposure.js +626 -0
- package/dist/src/scanner/rules/helpers.d.ts +31 -0
- package/dist/src/scanner/rules/helpers.js +85 -0
- package/dist/src/scanner/rules/index.d.ts +30 -0
- package/dist/src/scanner/rules/index.js +95 -0
- package/dist/src/scanner/rules/secret-entropy.d.ts +17 -0
- package/dist/src/scanner/rules/secret-entropy.js +338 -0
- package/dist/src/scanner/rules/shared.d.ts +100 -0
- package/dist/src/scanner/rules/shared.js +161 -0
- package/dist/src/scanner/rules/unguarded-routes.d.ts +16 -0
- package/dist/src/scanner/rules/unguarded-routes.js +526 -0
- package/dist/src/scanner/text.d.ts +90 -0
- package/dist/src/scanner/text.js +472 -0
- package/dist/src/scanner/types.d.ts +340 -0
- package/dist/src/scanner/types.js +195 -0
- package/dist/src/secrets/patterns.d.ts +63 -0
- package/dist/src/secrets/patterns.js +392 -0
- package/package.json +94 -4
package/LICENSE
ADDED
|
@@ -0,0 +1,21 @@
|
|
|
1
|
+
MIT License
|
|
2
|
+
|
|
3
|
+
Copyright (c) 2026 VibeFoundry
|
|
4
|
+
|
|
5
|
+
Permission is hereby granted, free of charge, to any person obtaining a copy
|
|
6
|
+
of this software and associated documentation files (the "Software"), to deal
|
|
7
|
+
in the Software without restriction, including without limitation the rights
|
|
8
|
+
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
|
|
9
|
+
copies of the Software, and to permit persons to whom the Software is
|
|
10
|
+
furnished to do so, subject to the following conditions:
|
|
11
|
+
|
|
12
|
+
The above copyright notice and this permission notice shall be included in all
|
|
13
|
+
copies or substantial portions of the Software.
|
|
14
|
+
|
|
15
|
+
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
16
|
+
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
17
|
+
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
|
|
18
|
+
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
19
|
+
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
|
|
20
|
+
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
|
|
21
|
+
SOFTWARE.
|
package/README.md
CHANGED
|
@@ -1,3 +1,571 @@
|
|
|
1
|
-
|
|
2
|
-
|
|
3
|
-
|
|
1
|
+
<div align="center">
|
|
2
|
+
|
|
3
|
+
# 🛡️ vibe-shield
|
|
4
|
+
|
|
5
|
+
**The security gate for AI-written code.**
|
|
6
|
+
|
|
7
|
+
Your agent ships fast. It also ships `NEXT_PUBLIC_SUPABASE_SERVICE_ROLE_KEY`, a `DELETE` route with no session check, and a `CREATE TABLE` with no Row Level Security — because it was optimising for "it works".
|
|
8
|
+
|
|
9
|
+
vibe-shield is a **static security scanner built for that failure mode**, plus an **Agent Skill** that makes Claude Code, Codex, Cursor and Windsurf audit before they say *"done"*.
|
|
10
|
+
|
|
11
|
+
[](https://github.com/avivgerz-hue/vibe-shield/blob/main/package.json)
|
|
12
|
+
[](#-quick-start)
|
|
13
|
+
[](https://github.com/avivgerz-hue/vibe-shield/actions/workflows/ci.yml)
|
|
14
|
+
[](#-requirements)
|
|
15
|
+
[](./LICENSE)
|
|
16
|
+
[](./package.json)
|
|
17
|
+
[](./src/scanner/rules)
|
|
18
|
+
[](#-it-scans-itself)
|
|
19
|
+
|
|
20
|
+
```bash
|
|
21
|
+
npx github:avivgerz-hue/vibe-shield scan
|
|
22
|
+
```
|
|
23
|
+
|
|
24
|
+

|
|
25
|
+
|
|
26
|
+
</div>
|
|
27
|
+
|
|
28
|
+
---
|
|
29
|
+
|
|
30
|
+
## The problem, in one file
|
|
31
|
+
|
|
32
|
+
This is the shape of code an agent writes when you ask for "a delete endpoint":
|
|
33
|
+
|
|
34
|
+
```ts
|
|
35
|
+
// app/api/users/route.ts
|
|
36
|
+
export async function DELETE(req: Request) {
|
|
37
|
+
const { id, userId } = await req.json();
|
|
38
|
+
await db.query(`DELETE FROM users WHERE id = ${id} AND owner = ${userId}`);
|
|
39
|
+
return new Response('ok');
|
|
40
|
+
}
|
|
41
|
+
```
|
|
42
|
+
|
|
43
|
+
It works. It also has **three critical vulnerabilities** — no authentication, SQL injection, and an IDOR that lets the caller choose whose data they delete. Nothing in a normal review loop catches all three, and the agent reported the task as complete.
|
|
44
|
+
|
|
45
|
+
`npx github:avivgerz-hue/vibe-shield scan` catches all three in **17 milliseconds**:
|
|
46
|
+
|
|
47
|
+
```
|
|
48
|
+
🛡️ vibe-shield v1.0.0 · security scan
|
|
49
|
+
target /home/you/app
|
|
50
|
+
|
|
51
|
+
💀 GRADE F — 0/100 · 3 critical
|
|
52
|
+
|
|
53
|
+
CRITICAL 3 ███
|
|
54
|
+
|
|
55
|
+
Findings
|
|
56
|
+
|
|
57
|
+
app/api/users/route.ts
|
|
58
|
+
CRITICAL 2:1 VSB-AUTH-001 Next.js App Router mutation handler has no authentication check
|
|
59
|
+
export async function DELETE(req: Request) {
|
|
60
|
+
CRITICAL 4:22 VSB-DB-007 SQL statement built by concatenating request data
|
|
61
|
+
await db.query(`DELETE FROM users WHERE id = ${id} AND owner = ${userId}`);
|
|
62
|
+
CRITICAL 4:22 VSB-AUTH-004 Record ownership taken from the request instead of the session (IDOR)
|
|
63
|
+
|
|
64
|
+
7 files · 1 KiB · 412 files/s · 17 ms · 48 rules
|
|
65
|
+
|
|
66
|
+
⚠ 3 critical findings — rotate any exposed credential before anything else.
|
|
67
|
+
Fix it npx github:avivgerz-hue/vibe-shield scan --fix-prompt # writes SECURITY_PATCH_PROMPT.md for your agent
|
|
68
|
+
Explain npx github:avivgerz-hue/vibe-shield explain VSB-AUTH-001
|
|
69
|
+
Gate CI npx github:avivgerz-hue/vibe-shield scan --fail-on critical --format github
|
|
70
|
+
```
|
|
71
|
+
|
|
72
|
+
Then hand the fix list to the agent that wrote the bug:
|
|
73
|
+
|
|
74
|
+
```bash
|
|
75
|
+
npx github:avivgerz-hue/vibe-shield scan --fix-prompt # → SECURITY_PATCH_PROMPT.md
|
|
76
|
+
```
|
|
77
|
+
|
|
78
|
+
That file contains, for every finding: the severity, the CWE id and link, the exact `file:line`, the vulnerable snippet, the ordered remediation steps, the target code shape, and a verification command. It is written for a machine to execute and a human to audit.
|
|
79
|
+
|
|
80
|
+
---
|
|
81
|
+
|
|
82
|
+
## ⚡ Quick start
|
|
83
|
+
|
|
84
|
+
```bash
|
|
85
|
+
# 1. Scan (no install, no config, no network calls)
|
|
86
|
+
npx github:avivgerz-hue/vibe-shield scan
|
|
87
|
+
|
|
88
|
+
# 2. Give your agent the fix list
|
|
89
|
+
npx github:avivgerz-hue/vibe-shield scan --fix-prompt
|
|
90
|
+
|
|
91
|
+
# 3. Install the gate permanently — skill + CI + ignore file
|
|
92
|
+
npx github:avivgerz-hue/vibe-shield init
|
|
93
|
+
```
|
|
94
|
+
|
|
95
|
+
> **Why GitHub and not `npx vibe-shield`?** The name `vibe-shield` on the npm
|
|
96
|
+
> registry belongs to a different project with no connection to this one. Running
|
|
97
|
+
> `npx vibe-shield` installs *their* package, not this repository's. Install from
|
|
98
|
+
> GitHub — the compiled CLI is committed, so it needs **no TypeScript** and starts
|
|
99
|
+
> in milliseconds. A scoped npm release (`@vibefoundry/vibe-shield`) may follow.
|
|
100
|
+
|
|
101
|
+
`npx github:avivgerz-hue/vibe-shield init` writes:
|
|
102
|
+
|
|
103
|
+
| File | What it does |
|
|
104
|
+
| --- | --- |
|
|
105
|
+
| `.claude/skills/vibe-shield/SKILL.md` | Agent Skill: forces Claude Code (and any SKILL.md-aware agent) to audit before finishing |
|
|
106
|
+
| `AGENTS.md` *(section)* | Same contract for Codex, Cursor, Windsurf, Amp, and anything that reads a repo instruction file |
|
|
107
|
+
| `.cursor/rules/vibe-shield.mdc` | `alwaysApply` Cursor rule |
|
|
108
|
+
| `.mcp.json` | MCP server entry (merged — your existing servers are preserved) |
|
|
109
|
+
| `.github/workflows/vibe-shield.yml` | Fails the pull request on new high/critical findings, uploads SARIF to code scanning |
|
|
110
|
+
| `.vibe-shieldignore` | The documented escape hatch (empty by default, with guidance) |
|
|
111
|
+
|
|
112
|
+
Nothing is overwritten without `--force`, and re-running is a no-op.
|
|
113
|
+
|
|
114
|
+
---
|
|
115
|
+
|
|
116
|
+
## 🧠 Why AI code needs its own scanner
|
|
117
|
+
|
|
118
|
+
Traditional SAST rules are built around human mistakes: a forgotten escape, a missing `prepare()`. Agent-written code fails differently — it is *confidently complete* and *structurally plausible*:
|
|
119
|
+
|
|
120
|
+
| What a human does | What an agent does |
|
|
121
|
+
| --- | --- |
|
|
122
|
+
| forgets to validate one input | writes a perfect endpoint with **no auth at all** |
|
|
123
|
+
| uses `eval` in one legacy file | puts a **tool `execute` that shells out** behind an LLM |
|
|
124
|
+
| commits a `.env` by accident | puts `SUPABASE_SERVICE_ROLE_KEY` behind `NEXT_PUBLIC_` because "it needed the env var" |
|
|
125
|
+
| writes `CREATE TABLE` | writes `CREATE TABLE` **and forgets the `ENABLE ROW LEVEL SECURITY`**, so the anon key reads everything |
|
|
126
|
+
| adds `dangerouslySetInnerHTML` to render a doc | renders **model output** with it, creating stored XSS with a step counter |
|
|
127
|
+
|
|
128
|
+
vibe-shield encodes those specific shapes: 60 signatures across five families, each one derived from a real agent-generated bug rather than a generic CWE checklist.
|
|
129
|
+
|
|
130
|
+
---
|
|
131
|
+
|
|
132
|
+
## 📊 Vulnerability Coverage Matrix
|
|
133
|
+
|
|
134
|
+
**48 code signatures + 12 agent-integrity signatures = 60.** Every rule has an id, a CWE, a vulnerable example, a fixed example, and ordered remediation steps — run `npx github:avivgerz-hue/vibe-shield rules` or `npx github:avivgerz-hue/vibe-shield explain <ID>`.
|
|
135
|
+
|
|
136
|
+
### Client-side secret & data exposure — `VSB-CLIENT-*`
|
|
137
|
+
|
|
138
|
+
| ID | Detects | CWE | Sev |
|
|
139
|
+
| --- | --- | --- | --- |
|
|
140
|
+
| `VSB-CLIENT-001` | `NEXT_PUBLIC_` / `VITE_` / `PUBLIC_` / `REACT_APP_` + a server-only secret name (`SERVICE_ROLE_KEY`, `OPENAI_API_KEY`, `STRIPE_SECRET_KEY`, `DATABASE_URL`…) | [CWE-200](https://cwe.mitre.org/data/definitions/200.html) | critical |
|
|
141
|
+
| `VSB-CLIENT-002` | server-only `process.env.*SECRET*` read inside a `"use client"` module | CWE-200 | high |
|
|
142
|
+
| `VSB-CLIENT-003` | `createClient(url, SERVICE_ROLE_KEY)` — RLS bypass from a client-reachable file | [CWE-250](https://cwe.mitre.org/data/definitions/250.html) | critical |
|
|
143
|
+
| `VSB-CLIENT-004` | `supabaseAdmin` / `adminClient` / `serviceRoleClient` referenced from browser code | CWE-250 | critical |
|
|
144
|
+
| `VSB-CLIENT-005` | `dangerouslyAllowBrowser: true`, provider SDK keyed in the browser | [CWE-749](https://cwe.mitre.org/data/definitions/749.html) | high |
|
|
145
|
+
| `VSB-CLIENT-006` | `.env*` assigns a live-looking credential to a public variable | [CWE-540](https://cwe.mitre.org/data/definitions/540.html) | critical |
|
|
146
|
+
| `VSB-CLIENT-007` | `next.config.js` `env` / `publicRuntimeConfig` / Vite `define` inlining a secret | [CWE-798](https://cwe.mitre.org/data/definitions/798.html) | critical |
|
|
147
|
+
|
|
148
|
+
### Unguarded mutation routes — `VSB-AUTH-*`
|
|
149
|
+
|
|
150
|
+
| ID | Detects | CWE | Sev |
|
|
151
|
+
| --- | --- | --- | --- |
|
|
152
|
+
| `VSB-AUTH-001` | App Router `POST/PUT/PATCH/DELETE` handler with no session check | [CWE-306](https://cwe.mitre.org/data/definitions/306.html) | critical |
|
|
153
|
+
| `VSB-AUTH-002` | Pages API mutation handler with no session check | CWE-306 | critical |
|
|
154
|
+
| `VSB-AUTH-003` | Express / Fastify / Koa / Hono mutation route with no auth middleware | [CWE-862](https://cwe.mitre.org/data/definitions/862.html) | high |
|
|
155
|
+
| `VSB-AUTH-004` | IDOR — ownership key from `body`/`query`/`params` (including the destructured local variable) | [CWE-639](https://cwe.mitre.org/data/definitions/639.html) | critical |
|
|
156
|
+
| `VSB-AUTH-005` | `"use server"` action mutating data with no auth | CWE-306 | high |
|
|
157
|
+
| `VSB-AUTH-006` | identity from `jwt.decode`, `x-user-id`, `atob(token.split('.')[1])`, `verify:false`, `algorithms:['none']` | [CWE-347](https://cwe.mitre.org/data/definitions/347.html) | critical |
|
|
158
|
+
| `VSB-AUTH-007` | admin/destructive surface that authenticates but never checks a role | [CWE-863](https://cwe.mitre.org/data/definitions/863.html) | high |
|
|
159
|
+
| `VSB-AUTH-008` | `const session = await auth()` captured and never enforced | [CWE-287](https://cwe.mitre.org/data/definitions/287.html) | medium |
|
|
160
|
+
| `VSB-AUTH-009` | cookie-session mutation with no CSRF token protection | [CWE-352](https://cwe.mitre.org/data/definitions/352.html) | medium |
|
|
161
|
+
|
|
162
|
+
### Database exposure — `VSB-DB-*`
|
|
163
|
+
|
|
164
|
+
| ID | Detects | CWE | Sev |
|
|
165
|
+
| --- | --- | --- | --- |
|
|
166
|
+
| `VSB-DB-001` | `CREATE TABLE` with no `ENABLE ROW LEVEL SECURITY` **anywhere in the project** (cross-file index) | [CWE-284](https://cwe.mitre.org/data/definitions/284.html) | critical\*/high |
|
|
167
|
+
| `VSB-DB-002` | permissive policy — `USING (true)`, `WITH CHECK (true)`, `USING (1=1)` | [CWE-732](https://cwe.mitre.org/data/definitions/732.html) | critical |
|
|
168
|
+
| `VSB-DB-003` | `anon`/`authenticated` policy whose `USING` never references `auth.uid()`/`auth.jwt()` | CWE-639 | high |
|
|
169
|
+
| `VSB-DB-004` | `GRANT … TO anon/public` (writes), incl. `GRANT ALL` | CWE-732 | critical/high |
|
|
170
|
+
| `VSB-DB-005` | `DISABLE ROW LEVEL SECURITY`, `DROP POLICY`, `BYPASSRLS` | CWE-284 | critical/high |
|
|
171
|
+
| `VSB-DB-006` | `$queryRawUnsafe`, `$executeRawUnsafe`, `knex.raw`, `sequelize.query`, `.raw()` with a dynamic string | [CWE-89](https://cwe.mitre.org/data/definitions/89.html) | critical |
|
|
172
|
+
| `VSB-DB-007` | hand-built SQL from request data — `${}`, `+`, Python f-strings, PHP interpolation | CWE-89 | critical |
|
|
173
|
+
| `VSB-DB-008` | Supabase service-role key outside a `server-only` boundary | CWE-250 | critical |
|
|
174
|
+
| `VSB-DB-009` | `createBucket(…, { public: true })` | CWE-732 | medium |
|
|
175
|
+
| `VSB-DB-010` | request data used as a table, column or `ORDER BY` identifier | CWE-89 | high |
|
|
176
|
+
|
|
177
|
+
\* critical when Supabase/PostgREST is detected — the anon key in every browser makes the table publicly addressable.
|
|
178
|
+
|
|
179
|
+
### AI / LLM & classic web — `VSB-AI-*`
|
|
180
|
+
|
|
181
|
+
| ID | Detects | CWE | Sev |
|
|
182
|
+
| --- | --- | --- | --- |
|
|
183
|
+
| `VSB-AI-001` | `exec`/`execSync`/`spawn({shell:true})`/`Bun.spawn`/`os.system`/`subprocess(shell=True)`/`Runtime.exec`/PHP backticks | [CWE-78](https://cwe.mitre.org/data/definitions/78.html) | critical |
|
|
184
|
+
| `VSB-AI-002` | LLM/MCP tool `execute` reaching a shell, `eval` or filesystem sink | [CWE-94](https://cwe.mitre.org/data/definitions/94.html) | critical |
|
|
185
|
+
| `VSB-AI-003` | model output into `eval`/`new Function`/`vm.runInNewContext` | CWE-94 | critical |
|
|
186
|
+
| `VSB-AI-004` | untrusted/RAG content interpolated into a `system` prompt, no delimiters (OWASP LLM01) | [CWE-74](https://cwe.mitre.org/data/definitions/74.html) | high |
|
|
187
|
+
| `VSB-AI-005` | Stripe webhook without `stripe.webhooks.constructEvent` | [CWE-345](https://cwe.mitre.org/data/definitions/345.html) | critical |
|
|
188
|
+
| `VSB-AI-006` | webhook signing secret read from the request | CWE-345 | high |
|
|
189
|
+
| `VSB-AI-007` | GitHub/Slack/Shopify/Svix webhook with no signature verification | CWE-345 | high |
|
|
190
|
+
| `VSB-AI-008` | `dangerouslySetInnerHTML`, `v-html`, `{@html}`, `innerHTML`, `document.write`, jQuery `.html()` | [CWE-79](https://cwe.mitre.org/data/definitions/79.html) | high |
|
|
191
|
+
| `VSB-AI-009` | client `eval`/`new Function`/string `setTimeout`/`javascript:` URLs | CWE-94 | high |
|
|
192
|
+
| `VSB-AI-010` | open redirect — `redirect(searchParams.get('next'))` | [CWE-601](https://cwe.mitre.org/data/definitions/601.html) | medium |
|
|
193
|
+
| `VSB-AI-011` | SSRF — `fetch(req.body.url)` | [CWE-918](https://cwe.mitre.org/data/definitions/918.html) | high |
|
|
194
|
+
| `VSB-AI-012` | path traversal — `path.join(dir, req.params.name)` | [CWE-22](https://cwe.mitre.org/data/definitions/22.html) | high |
|
|
195
|
+
| `VSB-AI-013` | multer/`formData()` upload with no size or type limit | [CWE-434](https://cwe.mitre.org/data/definitions/434.html) | medium |
|
|
196
|
+
| `VSB-AI-014` | public LLM endpoint with no rate limit or quota | [CWE-770](https://cwe.mitre.org/data/definitions/770.html) | medium |
|
|
197
|
+
| `VSB-AI-015` | `pickle.loads`, `yaml.load` without `SafeLoader`, PHP `unserialize` | [CWE-502](https://cwe.mitre.org/data/definitions/502.html) | high |
|
|
198
|
+
|
|
199
|
+
### Hardcoded credentials — `VSB-SECRET-*`
|
|
200
|
+
|
|
201
|
+
| ID | Detects | CWE | Sev |
|
|
202
|
+
| --- | --- | --- | --- |
|
|
203
|
+
| `VSB-SECRET-001` | vendor signature in a server file (Stripe, OpenAI, Anthropic, AWS, GitHub, Supabase, Slack, Twilio, SendGrid, npm, PyPI, HuggingFace, Docker, Vercel, Mailgun, Square, Shopify, Datadog) | CWE-798 | high |
|
|
204
|
+
| `VSB-SECRET-002` | generic high-entropy assignment to a credential-shaped name | CWE-798 | medium/high |
|
|
205
|
+
| `VSB-SECRET-003` | PEM private key material | [CWE-321](https://cwe.mitre.org/data/definitions/321.html) | critical |
|
|
206
|
+
| `VSB-SECRET-004` | hardcoded JWT whose payload carries `service_role` / `admin` | CWE-250 | critical |
|
|
207
|
+
| `VSB-SECRET-005` | the same vendor credential in a **browser-reachable** file | CWE-200 | critical |
|
|
208
|
+
| `VSB-SECRET-006` | `postgres:postgres@…`, `password = "admin"`, entropy < 2.5 | [CWE-1392](https://cwe.mitre.org/data/definitions/1392.html) | medium |
|
|
209
|
+
| `VSB-SECRET-007` | `.env` with live credentials — critical when git-tracked | CWE-540 | critical |
|
|
210
|
+
|
|
211
|
+
### Agent skill & MCP poisoning — `VSB-AGENT-*` (`audit-skills`)
|
|
212
|
+
|
|
213
|
+
| ID | Detects | CWE | Sev |
|
|
214
|
+
| --- | --- | --- | --- |
|
|
215
|
+
| `VSB-AGENT-001` | zero-width, bidi-override, Unicode-tag characters hiding instructions | CWE-74 | critical |
|
|
216
|
+
| `VSB-AGENT-002` | `curl`/`wget`/`nc` exfiltrating env vars or `.env` | CWE-200 | critical |
|
|
217
|
+
| `VSB-AGENT-003` | instructions to read `.env`, `~/.ssh`, `~/.aws/credentials`, `id_rsa`, keychain | [CWE-522](https://cwe.mitre.org/data/definitions/522.html) | critical |
|
|
218
|
+
| `VSB-AGENT-004` | registry-tag `npx -y`/`uvx`, remote MCP URL, `curl \| sh` install (git installs such as `github:owner/repo` are not flagged: they resolve to a commit, not a mutable tag) | [CWE-829](https://cwe.mitre.org/data/definitions/829.html) | high |
|
|
219
|
+
| `VSB-AGENT-005` | literal credentials in an `env` block, or `"*": "*"` env forwarding | [CWE-526](https://cwe.mitre.org/data/definitions/526.html) | critical/high |
|
|
220
|
+
| `VSB-AGENT-006` | "ignore previous instructions", "do not tell the user", "auto-approve", "skip confirmation" | CWE-74 | critical |
|
|
221
|
+
| `VSB-AGENT-007` | `Bash(*)`, `Write(*)`, `autoApprove: ["*"]`, `dangerouslySkipPermissions` | CWE-284 | high |
|
|
222
|
+
| `VSB-AGENT-008` | base64 payload / `eval(atob(…))` / `base64 -d \| sh` | CWE-94 | high |
|
|
223
|
+
| `VSB-AGENT-009` | "fetch <url> and follow every instruction" | CWE-829 | critical |
|
|
224
|
+
| `VSB-AGENT-010` | missing frontmatter, wildcard `allowed-tools` | CWE-250 | medium |
|
|
225
|
+
| `VSB-AGENT-011` | literal credential inside an instruction/MCP file | CWE-798 | critical |
|
|
226
|
+
| `VSB-AGENT-012` | `eval "$1"`, `sh -c "$VAR"` in a skill script | CWE-78 | medium |
|
|
227
|
+
|
|
228
|
+
---
|
|
229
|
+
|
|
230
|
+
## 🔌 Native MCP server
|
|
231
|
+
|
|
232
|
+
vibe-shield is itself a [Model Context Protocol](https://modelcontextprotocol.io) server, so
|
|
233
|
+
Claude Code, Claude Desktop, Cursor and Codex can call the scanner as a tool
|
|
234
|
+
instead of shelling out and parsing console output.
|
|
235
|
+
|
|
236
|
+
```bash
|
|
237
|
+
npx github:avivgerz-hue/vibe-shield mcp # JSON-RPC 2.0 over stdio
|
|
238
|
+
```
|
|
239
|
+
|
|
240
|
+
Three tools:
|
|
241
|
+
|
|
242
|
+
| Tool | What it does |
|
|
243
|
+
| --- | --- |
|
|
244
|
+
| `vibe_shield_scan` | Runs the 60 rules over a project or specific paths → grade, score, findings with file:line, CWE and remediation |
|
|
245
|
+
| `vibe_shield_audit_skills` | Audits `.claude/skills`, `.cursor` rules, `AGENTS.md` and MCP configs for prompt injection and exfiltration traps |
|
|
246
|
+
| `vibe_shield_explain_rule` | Full CWE detail, vulnerable/fixed examples and remediation for any rule id or slug |
|
|
247
|
+
|
|
248
|
+
`npx github:avivgerz-hue/vibe-shield init` writes the config for you, merging into `.mcp.json`
|
|
249
|
+
**without clobbering** servers you already have:
|
|
250
|
+
|
|
251
|
+
```jsonc
|
|
252
|
+
{
|
|
253
|
+
"mcpServers": {
|
|
254
|
+
"github": { "command": "npx", "args": ["-y", "@modelcontextprotocol/server-github"] },
|
|
255
|
+
"vibe-shield": { "command": "npx", "args": ["--yes", "github:avivgerz-hue/vibe-shield", "mcp"] }
|
|
256
|
+
}
|
|
257
|
+
}
|
|
258
|
+
```
|
|
259
|
+
|
|
260
|
+
Or add it by hand:
|
|
261
|
+
|
|
262
|
+
```bash
|
|
263
|
+
claude mcp add vibe-shield -- npx --yes github:avivgerz-hue/vibe-shield mcp
|
|
264
|
+
```
|
|
265
|
+
|
|
266
|
+
The server is zero-dependency, writes **only** JSON-RPC to stdout, honours
|
|
267
|
+
JSON-RPC notifications (no spurious replies), and returns tool errors as
|
|
268
|
+
`isError` results so a bad argument degrades one call instead of killing the
|
|
269
|
+
session. Requests are handled in arrival order.
|
|
270
|
+
|
|
271
|
+
An empty target is reported as `No files were scanned` rather than a green pass —
|
|
272
|
+
a scanner that says "clean" when it read nothing is worse than no scanner.
|
|
273
|
+
|
|
274
|
+
---
|
|
275
|
+
|
|
276
|
+
## 🤖 Use it as an Agent Skill
|
|
277
|
+
|
|
278
|
+
`npx github:avivgerz-hue/vibe-shield init` installs the skill. Once it is in a repository, every
|
|
279
|
+
agent session starts with a contract:
|
|
280
|
+
|
|
281
|
+
> You are not allowed to say "done" until the code you just wrote passes a security audit.
|
|
282
|
+
|
|
283
|
+
```bash
|
|
284
|
+
npx github:avivgerz-hue/vibe-shield init # Claude Code + Cursor + AGENTS.md + CI
|
|
285
|
+
npx github:avivgerz-hue/vibe-shield init --agent claude # just the skill
|
|
286
|
+
npx github:avivgerz-hue/vibe-shield init --no-workflow # skip the GitHub Action
|
|
287
|
+
```
|
|
288
|
+
|
|
289
|
+
**Why this works better than a prompt snippet:** it names the three commands, gives the agent a reading guide for a finding (`line:col`, rule id, message, CWE), states five hard rules (never suppress, never weaken, always rotate, report honestly, re-verify), and defines "done" as an exit code rather than an opinion.
|
|
290
|
+
|
|
291
|
+
The skill itself is [SKILL.md](./SKILL.md) — read it, it is short.
|
|
292
|
+
|
|
293
|
+
### Anthropic Agent Skills quickstart
|
|
294
|
+
|
|
295
|
+
```bash
|
|
296
|
+
# marketplace-style install
|
|
297
|
+
npx github:avivgerz-hue/vibe-shield init --agent claude
|
|
298
|
+
# → .claude/skills/vibe-shield/SKILL.md
|
|
299
|
+
|
|
300
|
+
# or globally, for every project
|
|
301
|
+
curl -sL https://raw.githubusercontent.com/avivgerz-hue/vibe-shield/main/SKILL.md \
|
|
302
|
+
-o ~/.claude/skills/vibe-shield/SKILL.md
|
|
303
|
+
```
|
|
304
|
+
|
|
305
|
+
---
|
|
306
|
+
|
|
307
|
+
## 🔁 CI/CD: fail the PR before the bug ships
|
|
308
|
+
|
|
309
|
+
### GitHub Actions (recommended)
|
|
310
|
+
|
|
311
|
+
`npx github:avivgerz-hue/vibe-shield init` installs a workflow that runs the scan, uploads SARIF to
|
|
312
|
+
code scanning, audits the agent surface, and fails the job:
|
|
313
|
+
|
|
314
|
+
```yaml
|
|
315
|
+
name: vibe-shield
|
|
316
|
+
on: [push, pull_request]
|
|
317
|
+
|
|
318
|
+
permissions:
|
|
319
|
+
contents: read
|
|
320
|
+
security-events: write
|
|
321
|
+
|
|
322
|
+
jobs:
|
|
323
|
+
scan:
|
|
324
|
+
runs-on: ubuntu-latest
|
|
325
|
+
steps:
|
|
326
|
+
- uses: actions/checkout@v4
|
|
327
|
+
- uses: actions/setup-node@v4
|
|
328
|
+
with:
|
|
329
|
+
node-version: 20
|
|
330
|
+
|
|
331
|
+
- name: Scan and upload SARIF
|
|
332
|
+
run: npx --yes github:avivgerz-hue/vibe-shield scan --format sarif --out vibe-shield.sarif --fail-on high
|
|
333
|
+
continue-on-error: true
|
|
334
|
+
|
|
335
|
+
- uses: github/codeql-action/upload-sarif@v3
|
|
336
|
+
if: always()
|
|
337
|
+
with:
|
|
338
|
+
sarif_file: vibe-shield.sarif
|
|
339
|
+
category: vibe-shield
|
|
340
|
+
|
|
341
|
+
- name: Enforce the gate
|
|
342
|
+
run: npx --yes github:avivgerz-hue/vibe-shield scan --fail-on high
|
|
343
|
+
```
|
|
344
|
+
|
|
345
|
+
Findings appear **inline on the pull request diff** (GitHub annotations) and in
|
|
346
|
+
the **Security → Code scanning** tab (SARIF), so a reviewer sees the vulnerability
|
|
347
|
+
on the line that introduced it.
|
|
348
|
+
|
|
349
|
+
### Any other CI
|
|
350
|
+
|
|
351
|
+
```bash
|
|
352
|
+
npx github:avivgerz-hue/vibe-shield scan --fail-on high # exit 1 on findings
|
|
353
|
+
npx github:avivgerz-hue/vibe-shield scan --format compact # one line per finding
|
|
354
|
+
npx github:avivgerz-hue/vibe-shield scan --format json > vibe-shield.json # full report
|
|
355
|
+
npx github:avivgerz-hue/vibe-shield scan --format summary # JSON, counts + grade only
|
|
356
|
+
npx github:avivgerz-hue/vibe-shield scan --format github # workflow annotations
|
|
357
|
+
```
|
|
358
|
+
|
|
359
|
+
Exit codes: **0** clean · **1** findings at/above `--fail-on` · **2** usage error
|
|
360
|
+
(a target path that does not exist, or a target that matches no scannable files).
|
|
361
|
+
An empty scan prints `NO FILES SCANNED — this is not a pass` and is never
|
|
362
|
+
reported as a grade — a gate pointed at an ignored path fails instead of
|
|
363
|
+
succeeding forever.
|
|
364
|
+
|
|
365
|
+
---
|
|
366
|
+
|
|
367
|
+
## 🎯 The scorecard
|
|
368
|
+
|
|
369
|
+
Grades are **deliberately unforgiving**, because "B+" for a leaked production key is how security tools lose credibility:
|
|
370
|
+
|
|
371
|
+
| Finding present | Maximum score | Best possible grade |
|
|
372
|
+
| --- | --- | --- |
|
|
373
|
+
| any **critical** | 29 | **F** |
|
|
374
|
+
| any **high** | 58 | C |
|
|
375
|
+
| any **medium** | 76 | B |
|
|
376
|
+
| any **low** | 88 | A- |
|
|
377
|
+
| nothing | 100 | **A+** |
|
|
378
|
+
|
|
379
|
+
Penalties per finding: critical −30, high −12, medium −5, low −2, info −0.5. One leaked live key fails the build on its own; a handful of informational notes never will.
|
|
380
|
+
|
|
381
|
+
---
|
|
382
|
+
|
|
383
|
+
## 🪄 It scans itself
|
|
384
|
+
|
|
385
|
+
This repository runs its own gate on every commit and in CI:
|
|
386
|
+
|
|
387
|
+
```console
|
|
388
|
+
$ npx github:avivgerz-hue/vibe-shield scan
|
|
389
|
+
🛡️ vibe-shield v1.0.0 · security scan
|
|
390
|
+
target /you/vibe-shield
|
|
391
|
+
|
|
392
|
+
🏆 GRADE A+ — 100/100
|
|
393
|
+
|
|
394
|
+
21 files · 172 KiB · 236 files/s · 89 ms · 48 rules
|
|
395
|
+
|
|
396
|
+
Nothing to fix. Keep the shield up. 🛡️
|
|
397
|
+
|
|
398
|
+
$ npx github:avivgerz-hue/vibe-shield audit-skills
|
|
399
|
+
🏆 GRADE A+ — 100/100
|
|
400
|
+
|
|
401
|
+
1 file · 6 KiB · 14 files/s · 69 ms · 12 rules
|
|
402
|
+
|
|
403
|
+
Nothing to fix. Keep the shield up. 🛡️
|
|
404
|
+
```
|
|
405
|
+
|
|
406
|
+
The detector catalogue is the one place that must contain the exact vulnerable
|
|
407
|
+
shapes and vendor key formats the rules match on — exactly like gitleaks and
|
|
408
|
+
semgrep. Those files are listed explicitly in [`.vibe-shieldignore`](./.vibe-shieldignore)
|
|
409
|
+
with a comment explaining why, instead of weakening a rule.
|
|
410
|
+
|
|
411
|
+
---
|
|
412
|
+
|
|
413
|
+
## 🔇 Suppressing a finding (legitimately)
|
|
414
|
+
|
|
415
|
+
Inline, with a rule id and a reason — the reason is required by review culture even though the parser does not enforce it:
|
|
416
|
+
|
|
417
|
+
```ts
|
|
418
|
+
// vibe-shield-disable-next-line VSB-AI-001 -- path comes from a closed enum
|
|
419
|
+
exec(`convert ${KNOWN_FORMATS[format]} out.png`);
|
|
420
|
+
```
|
|
421
|
+
|
|
422
|
+
```ts
|
|
423
|
+
// vibe-shield-disable-line VSB-AUTH-008
|
|
424
|
+
// vibe-shield-disable VSB-AI-014
|
|
425
|
+
// vibe-shield-enable
|
|
426
|
+
// vibe-shield-disable-file VSB-DB-004 -- generated edge schema
|
|
427
|
+
```
|
|
428
|
+
|
|
429
|
+
Suppressed findings are **counted and reported** in the scorecard, never silently dropped. Whole-path exclusions belong in `.vibe-shieldignore` (gitignore syntax) and should be reserved for generated code.
|
|
430
|
+
|
|
431
|
+
---
|
|
432
|
+
|
|
433
|
+
## ⚙️ How it works
|
|
434
|
+
|
|
435
|
+
```
|
|
436
|
+
discover ──▶ project context ──▶ rule gate ──▶ detect ──▶ suppress ──▶ score ──▶ report
|
|
437
|
+
│ │ │ │
|
|
438
|
+
│ │ │ └─ 60 rules, each a pure (FileContext) => RuleHit[]
|
|
439
|
+
│ │ └─ cheap content hints: most rules skip most files
|
|
440
|
+
│ └─ manifests, framework flags, cross-file RLS index, git index, middleware matchers
|
|
441
|
+
└─ one tree walk, gitignore/.vibe-shieldignore layering, binary + size + test/doc pruning
|
|
442
|
+
```
|
|
443
|
+
|
|
444
|
+
**Design constraints, in priority order:**
|
|
445
|
+
|
|
446
|
+
1. **Fast.** One `readdir` walk, one read per file, one comment-masking pass, and a cheap gate before any rule body runs. A 400-file project scans in ~200 ms; a typical Next.js app in under 100 ms. No AST, no parser, no dependency.
|
|
447
|
+
2. **Honest.** Every finding carries the exact snippet the developer wrote. Nothing is inferred from a filename alone, and every suppression is counted, not hidden.
|
|
448
|
+
3. **Composable.** `scanProject()` for the CLI, `runRulesOnFile()` for the auditor, `scoreFindings()` for anything that produces findings. The whole API is exported from `vibe-shield`.
|
|
449
|
+
|
|
450
|
+
**Zero runtime dependencies.** No network calls, no telemetry, no language server, no native binaries. `npx github:avivgerz-hue/vibe-shield scan` works offline and in an air-gapped CI runner.
|
|
451
|
+
|
|
452
|
+
---
|
|
453
|
+
|
|
454
|
+
## 📖 CLI reference
|
|
455
|
+
|
|
456
|
+
```
|
|
457
|
+
vibe-shield scan [paths...] Scan the project (default: `.`)
|
|
458
|
+
vibe-shield audit-skills [paths...] Audit agent skills, rules, AGENTS.md and MCP config
|
|
459
|
+
vibe-shield init Install the Agent Skill, AGENTS.md, .mcp.json and CI gate
|
|
460
|
+
vibe-shield mcp Serve the MCP tools over stdio
|
|
461
|
+
vibe-shield rules List every detection rule
|
|
462
|
+
vibe-shield explain <RULE_ID> Full documentation for one rule
|
|
463
|
+
vibe-shield version | help
|
|
464
|
+
```
|
|
465
|
+
|
|
466
|
+
| Flag | Default | Meaning |
|
|
467
|
+
| --- | --- | --- |
|
|
468
|
+
| `--format <fmt>` | `console` | `console` `compact` `summary` `json` `sarif` `github` |
|
|
469
|
+
| `--fail-on <sev>` | `critical` | Exit 1 at/above this severity |
|
|
470
|
+
| `--severity <sev>` | — | Only report at/above this severity |
|
|
471
|
+
| `--fix-prompt` | off | Write `SECURITY_PATCH_PROMPT.md` |
|
|
472
|
+
| `--out <file>` | — | Write the report to a file |
|
|
473
|
+
| `--rules <ids>` | all | Comma-separated ids/slugs, supports `VSB-DB-*` |
|
|
474
|
+
| `--ignore <glob>` | — | Extra ignore pattern (repeatable) |
|
|
475
|
+
| `--include-tests` | off | Include test/fixture files |
|
|
476
|
+
| `--include-docs` | off | Include Markdown and `docs/` |
|
|
477
|
+
| `--cache` | off | Reuse `.vibe-shield-cache.json` |
|
|
478
|
+
| `--max-findings <n>` | 50 | Console list cap (`0` = all) |
|
|
479
|
+
| `--quiet` | off | One-line verdict (`100/100 A+ — 0 critical, …`) |
|
|
480
|
+
| `--no-color` | auto | Disable ANSI (also honours `NO_COLOR`) |
|
|
481
|
+
| `--json` | off | Shorthand for `--format json` |
|
|
482
|
+
| `--no-gitignore` | off | Ignore `.gitignore` while discovering files |
|
|
483
|
+
| `--no-shield-ignore` | off | Ignore `.vibe-shieldignore` (e.g. to scan excluded fixtures) |
|
|
484
|
+
|
|
485
|
+
`init` takes `--target <dir>`, `--agent claude / cursor / generic / all`, `--no-workflow`,
|
|
486
|
+
`--force` and `--yes`.
|
|
487
|
+
|
|
488
|
+
---
|
|
489
|
+
|
|
490
|
+
## 🧩 Programmatic API
|
|
491
|
+
|
|
492
|
+
```ts
|
|
493
|
+
import { scanProject, formatSarif, buildPatchPrompt, auditSkills } from 'vibe-shield';
|
|
494
|
+
|
|
495
|
+
const result = await scanProject({ rootDir: process.cwd(), cache: true });
|
|
496
|
+
|
|
497
|
+
console.log(result.score.score, result.score.grade); // 0 'F'
|
|
498
|
+
console.log(result.findings[0].ruleId); // 'VSB-AUTH-001'
|
|
499
|
+
console.log(result.findings[0].cwe.id); // 'CWE-306'
|
|
500
|
+
|
|
501
|
+
process.stdout.write(formatSarif(result)); // SARIF 2.1.0
|
|
502
|
+
process.stdout.write(buildPatchPrompt(result)); // agent-ready fix list
|
|
503
|
+
|
|
504
|
+
const agentReport = await auditSkills(process.cwd()); // VSB-AGENT-* surface
|
|
505
|
+
```
|
|
506
|
+
|
|
507
|
+
---
|
|
508
|
+
|
|
509
|
+
## ❓ FAQ
|
|
510
|
+
|
|
511
|
+
**Does it replace a real SAST tool?**
|
|
512
|
+
No. It replaces the *absence* of one. vibe-shield is a fast, zero-dependency gate you can put in front of every agent-written commit today. Pair it with a deeper scanner (CodeQL, Semgrep) if you have one; the rules overlap far less than you would expect, because these are agent-specific shapes.
|
|
513
|
+
|
|
514
|
+
**Will it flag my secure code?**
|
|
515
|
+
The secure fixture (`tests/fixtures/secure/`) is a real Next.js + Supabase + Stripe app with auth, RLS, sanitisation, rate limiting and webhook verification — it scores 100/100 and the suite asserts **zero findings** on it. Each rule has guard clauses for the legitimate patterns: `import "server-only"` suppresses the service-role rules, Prisma tagged templates suppress the SQLi rule, `DOMPurify` suppresses the XSS rule, `auth.uid()` policies suppress the RLS rules.
|
|
516
|
+
|
|
517
|
+
**Does it send my code anywhere?**
|
|
518
|
+
No. There is no network code in the runtime at all — the package has zero dependencies and makes no outbound requests.
|
|
519
|
+
|
|
520
|
+
**Why not an AST?**
|
|
521
|
+
Because a 1.5 MB minified bundle, a Vue SFC, a PHP legacy file and a SQL migration all need to be scanned with the same tool, and a parser handles exactly one of those. The rules use structural brace matching plus a comment-masker that preserves offsets, which is enough for the shapes that matter and roughly two orders of magnitude faster.
|
|
522
|
+
|
|
523
|
+
**Why "vibe"?**
|
|
524
|
+
Because the code was written by vibe and reviewed by nobody. This is the reviewer.
|
|
525
|
+
|
|
526
|
+
---
|
|
527
|
+
|
|
528
|
+
## 🛠 Development
|
|
529
|
+
|
|
530
|
+
```bash
|
|
531
|
+
git clone https://github.com/avivgerz-hue/vibe-shield
|
|
532
|
+
cd vibe-shield
|
|
533
|
+
npm ci
|
|
534
|
+
npm run typecheck # tsc --noEmit (strict, noUnusedLocals)
|
|
535
|
+
npm test # 224 Vitest tests
|
|
536
|
+
npm run build # tsc + postbuild (shebang, chmod, CJS marker)
|
|
537
|
+
npm run verify # typecheck + test + build + self-scan + self-audit
|
|
538
|
+
```
|
|
539
|
+
|
|
540
|
+
See [CONTRIBUTING.md](./CONTRIBUTING.md) for the rule-authoring guide, and
|
|
541
|
+
[SECURITY.md](./SECURITY.md) for the threat model and disclosure policy.
|
|
542
|
+
|
|
543
|
+
---
|
|
544
|
+
|
|
545
|
+
## 📋 Requirements
|
|
546
|
+
|
|
547
|
+
- **Node.js ≥ 18.17** (uses `node:` builtins only — no polyfills, no native modules)
|
|
548
|
+
- Any OS: Linux, macOS, Windows
|
|
549
|
+
- No config file required; `package.json` is optional
|
|
550
|
+
|
|
551
|
+
---
|
|
552
|
+
|
|
553
|
+
## 📄 License
|
|
554
|
+
|
|
555
|
+
MIT © [VibeFoundry](./LICENSE) — use it, fork it, ship it.
|
|
556
|
+
|
|
557
|
+
---
|
|
558
|
+
|
|
559
|
+
## Part of the VibeFoundry Agent Suite
|
|
560
|
+
|
|
561
|
+
Three tools for the same problem: agents move fast, and something has to keep the output honest.
|
|
562
|
+
|
|
563
|
+
| | Tool | What it does |
|
|
564
|
+
| :---: | --- | --- |
|
|
565
|
+
| 🎨 | **[unslop](https://github.com/avivgerz-hue/unslop)** | Studio-grade UI design systems & Anti-Slop engine — stops agents from generating generic, interchangeable interfaces |
|
|
566
|
+
| 🛡️ | **[vibe-shield](https://github.com/avivgerz-hue/vibe-shield)** | Security scanner & MCP guard — stops agents from shipping leaked keys, unguarded routes and disabled RLS |
|
|
567
|
+
| 👻 | **[ghost-qa](https://github.com/avivgerz-hue/ghost-qa)** | Autonomous headless-browser QA & self-healing verification loop — stops agents from claiming a UI works when it does not |
|
|
568
|
+
|
|
569
|
+
```bash
|
|
570
|
+
npx github:avivgerz-hue/vibe-shield init # security gate
|
|
571
|
+
```
|