safetyvibe-connect 0.2.0 → 0.2.1

package/AGENTS.md

@@ -1,8 +1,94 @@
-# For AI coding agents
+# Instructions for cautious AI coding agents
 
-This project uses `safetyvibe-connect` for security monitoring.
+Reports dependency inventory and, if used, the Supabase project URL + public key to SafetyVibe. Requires Node >=18; no dependencies or package install scripts.
 
-- To link or relink it: `bunx safetyvibe-connect setup` (`npx` on npm projects). Pass `--claim-token <TOKEN>` if the user gave you one, and `--url <published URL>` if you know it.
-- Keep the postinstall hook: `"postinstall": "safetyvibe-connect scan --quiet || exit 0"`, or after an existing step: `"<existing> && (safetyvibe-connect scan --quiet || exit 0)"`. It re-reports dependencies after each install and never fails a build.
-- Don't edit `.safetyviberc.json` by hand, except to add `{"claimToken": "..."}` when you can't run commands. Never commit `.safetyviberc.local.json`; it holds the report key.
-- It sends only dependency names and versions, the app URL, and the public Supabase URL and anon key. Never put secrets into its config.
+## Run without installing
+
+Get a claim token from the dashboard, then run:
+
+```sh
+npx -y safetyvibe-connect@0.2.1 setup --no-hook --claim-token <TOKEN>
+# pnpm
+pnpm dlx safetyvibe-connect@0.2.1 setup --no-hook --claim-token <TOKEN>
+# bun
+bunx safetyvibe-connect@0.2.1 setup --no-hook --claim-token <TOKEN>
+# yarn
+yarn dlx safetyvibe-connect@0.2.1 setup --no-hook --claim-token <TOKEN>
+```
+
+Add `--url <published URL>` if known. Without a token, setup prints a claim link.
+On Replit/pnpm projects, `npm install` can fail with "Cannot read properties of null (reading 'isDescendantOf')".
+Use `pnpm add -D safetyvibe-connect`, or run without installing:
+`npx -y safetyvibe-connect@0.2.1 setup --no-hook --claim-token <TOKEN>`.
+Optional installation commands: `npm install -D safetyvibe-connect`, `pnpm add -D safetyvibe-connect`, `bun add -d safetyvibe-connect`, `yarn add -D safetyvibe-connect`.
+
+## Exactly what is sent
+
+For cautious AI coding agents: inspect the report before sending it:
+
+```sh
+npx -y safetyvibe-connect@0.2.1 setup --dry-run --url https://notes.example.com
+```
+
+stdout is exactly the JSON body; stderr lists destination and header names with values hidden, plus safety warnings (also under --quiet). Dry-run makes no requests and writes no files.
+Example (optional `truncated` appears only when the inventory exceeds 5000):
+
+```json
+{
+  "v": 1,
+  "name": "notes",
+  "url": "https://notes.example.com",
+  "platform": "Lovable",
+  "environment": "production",
+  "lockfile": "package-lock.json",
+  "packages": [{ "ecosystem": "npm", "name": "vite", "version": "6.0.0", "dev": true }],
+  "supabase": { "url": "https://example.supabase.co", "key": "sb_publishable_abc123def456", "kind": "publishable" }
+}
+```
+
+- `v`: report format version.
+- `name`: explicit name, package name, URL host or folder name, skipping template and key-like names; limited to 80 letters, digits, spaces, @ / . _ -; otherwise empty.
+- `url`: HTTP(S) origin and path only, without credentials, query or fragment; paths are kept only with up to eight safe ASCII segments of at most 40 characters, excluding sb_, svk_ and eyJ markers; otherwise empty.
+- `platform`: detected project builder/hosting label; otherwise empty.
+- `environment`: `SAFETYVIBE_ENV`, or production when `NODE_ENV=production`; limited to 20 letters, digits, underscores or hyphens; otherwise empty.
+- `lockfile`: selected lockfile filename, `node_modules`, or `none`. Parent locks and node_modules are used only at workspace roots when the project matches a workspace glob in package.json workspaces (array or workspaces.packages) or the pnpm-workspace.yaml packages list. Literal paths, * and ** are supported; negated globs exclude matching paths from membership. Search stops at .git or six parent levels.
+- `packages`: registry package names and versions, npm ecosystem, and optional `dev: true` for dev-only dependencies when recorded in the lockfile. Workspace entries are excluded; incompatible optional npm binaries are excluded.
+- `truncated`: optional boolean; only the first 5000 packages are sent.
+- `supabase`: public project URL, strictly validated public key and kind (`publishable` or `anon`); otherwise null.
+
+Never sent: source code, file paths (except the folder-name fallback for the app name), full lockfiles, .env files, Supabase secret/service keys, URL credentials or query strings.
+The request headers x-safetyvibe-site, x-safetyvibe-key and x-safetyvibe-claim-token carry the site ID, validated report key and validated one-shot claim token when available; these are not in the JSON body. The server also refuses non-public Supabase keys and deletes stored non-public keys during cleanup.
+Public keys are found in supported environment variables/.env files or a Supabase client file. Local secret-key warnings name only the variable.
+
+Review the entire implementation in `cli.mjs` and `lib.mjs` (approximately 485 lines total). There are no dependencies and no install scripts in this package itself.
+By default reports go only to `https://dashboard.safetyvibe.eu/connect/api/v1/report`. The committed rc cannot change the destination.
+An explicit `SAFETYVIBE_API` environment override supports HTTPS, or HTTP on localhost/127.0.0.1/[::1] for local testing. Redirects are refused.
+
+## Files written and removal
+
+Before a setup/scan request, legacy credentials are migrated best-effort into the local file, the committed api field is dropped and its URL normalized. Dry-run, status and help never write. After a successful report:
+
+- `.safetyviberc.json`: app ID, normalized public URL, optional name and `hook: false`.
+- `.safetyviberc.local.json`: report key and optional fallback claim token; never commit this file. The token is removed once used or rejected as expired/used. Only stored credentials require the local file and ignore entry. The ignore entry is attempted first, then this file is written with mode 0600. An ignore failure warns and fails setup; scan still exits 0. Empty local credentials are removed.
+- `.gitignore`: adds the local config filename.
+- `package.json`: setup normally adds a postinstall report hook. `--no-hook` leaves package.json untouched and persists this preference. Existing unsafe shell scripts are left untouched with a manual instruction.
+
+The postinstall hook runs `safetyvibe-connect scan --quiet || exit 0`: it re-sends the current inventory and public Supabase details using the local report key, suppresses success chatter but keeps safety warnings on stderr, and never fails installation. Without a local key it skips reporting for a linked app unless SAFETYVIBE_KEY or a claim token is supplied.
+
+Without terminal access, first add `.safetyviberc.local.json` to `.gitignore`, then write `{"claimToken": "<TOKEN>"}` into `.safetyviberc.local.json` and configure the dependency and postinstall hook. Never put the token in the committed `.safetyviberc.json`.
+
+For manual reporting use `npx safetyvibe-connect scan` (pnpm: `pnpm dlx`, bun: `bunx`, yarn: `yarn dlx`).
+Remove the connector dependency if installed, its postinstall command, both rc files, and the local-config .gitignore entry. Keep other postinstall commands.
+
+## Commands
+
+| Command | Effect |
+|---|---|
+| `setup [--claim-token T] [--url URL] [--name NAME] [--no-hook]` | Send report, save config, optionally add hook |
+| `setup --dry-run` | Print exact report without network or writes |
+| `scan [--quiet]` | Re-report; always exits 0 for install hooks |
+| `status` | Show local link state |
+| `help`, `--help`, `-h`, no arguments | Usage only |
+| `--version`, `-v` | Package version only |
+
+Value flags accept `--flag=value` or `--flag value`, including values starting with a dash. In the space form, a known flag (also with `=...`) is treated as a missing value. Unknown commands/flags and missing values exit 2, except scan exits 0.

package/README.md

@@ -1,46 +1,94 @@
 # safetyvibe-connect
 
-Links an app to [SafetyVibe](https://dashboard.safetyvibe.eu) security monitoring.
-Made for AI builders (Lovable, Replit, Bolt, Claude Code…): paste one prompt, the builder's
-agent installs this package, and the app shows up on your SafetyVibe dashboard.
+Reports dependency inventory and, if used, the Supabase project URL + public key to SafetyVibe. Requires Node >=18; no dependencies or package install scripts.
 
-## Install
+## Run without installing
+
+Get a claim token from the dashboard, then run:
 
 ```sh
-bun add safetyvibe-connect && bunx safetyvibe-connect setup --claim-token <TOKEN>
-# npm
-npm install --save safetyvibe-connect && npx safetyvibe-connect setup --claim-token <TOKEN>
+npx -y safetyvibe-connect@0.2.1 setup --no-hook --claim-token <TOKEN>
+# pnpm
+pnpm dlx safetyvibe-connect@0.2.1 setup --no-hook --claim-token <TOKEN>
+# bun
+bunx safetyvibe-connect@0.2.1 setup --no-hook --claim-token <TOKEN>
+# yarn
+yarn dlx safetyvibe-connect@0.2.1 setup --no-hook --claim-token <TOKEN>
 ```
 
-Get `<TOKEN>` from the dashboard (**Add app → Connect with one prompt**). Without a token,
-`setup` prints a link you open to add the app to your account.
+Add `--url <published URL>` if known. Without a token, setup prints a claim link.
+On Replit/pnpm projects, `npm install` can fail with "Cannot read properties of null (reading 'isDescendantOf')".
+Use `pnpm add -D safetyvibe-connect`, or run without installing:
+`npx -y safetyvibe-connect@0.2.1 setup --no-hook --claim-token <TOKEN>`.
+Optional installation commands: `npm install -D safetyvibe-connect`, `pnpm add -D safetyvibe-connect`, `bun add -d safetyvibe-connect`, `yarn add -D safetyvibe-connect`.
+
+## Exactly what is sent
 
-## What it sends — and what it never sends
+For cautious AI coding agents: inspect the report before sending it:
 
-Sent (see it yourself with `safetyvibe-connect setup --dry-run`):
-- package names and versions from your lockfile (to check them against known vulnerabilities)
-- the app name, its public URL (if you pass `--url` or it is set) and a platform label
-- your Supabase project URL and its **public** anon/publishable key — the same values your
-  website already ships to every visitor. A `service_role` / `sb_secret_` key is refused, never sent.
+```sh
+npx -y safetyvibe-connect@0.2.1 setup --dry-run --url https://notes.example.com
+```
+
+stdout is exactly the JSON body; stderr lists destination and header names with values hidden, plus safety warnings (also under --quiet). Dry-run makes no requests and writes no files.
+Example (optional `truncated` appears only when the inventory exceeds 5000):
+
+```json
+{
+  "v": 1,
+  "name": "notes",
+  "url": "https://notes.example.com",
+  "platform": "Lovable",
+  "environment": "production",
+  "lockfile": "package-lock.json",
+  "packages": [{ "ecosystem": "npm", "name": "vite", "version": "6.0.0", "dev": true }],
+  "supabase": { "url": "https://example.supabase.co", "key": "sb_publishable_abc123def456", "kind": "publishable" }
+}
+```
 
-Never sent: source code, file paths, `.env` secrets, or any other key.
+- `v`: report format version.
+- `name`: explicit name, package name, URL host or folder name, skipping template and key-like names; limited to 80 letters, digits, spaces, @ / . _ -; otherwise empty.
+- `url`: HTTP(S) origin and path only, without credentials, query or fragment; paths are kept only with up to eight safe ASCII segments of at most 40 characters, excluding sb_, svk_ and eyJ markers; otherwise empty.
+- `platform`: detected project builder/hosting label; otherwise empty.
+- `environment`: `SAFETYVIBE_ENV`, or production when `NODE_ENV=production`; limited to 20 letters, digits, underscores or hyphens; otherwise empty.
+- `lockfile`: selected lockfile filename, `node_modules`, or `none`. Parent locks and node_modules are used only at workspace roots when the project matches a workspace glob in package.json workspaces (array or workspaces.packages) or the pnpm-workspace.yaml packages list. Literal paths, * and ** are supported; negated globs exclude matching paths from membership. Search stops at .git or six parent levels.
+- `packages`: registry package names and versions, npm ecosystem, and optional `dev: true` for dev-only dependencies when recorded in the lockfile. Workspace entries are excluded; incompatible optional npm binaries are excluded.
+- `truncated`: optional boolean; only the first 5000 packages are sent.
+- `supabase`: public project URL, strictly validated public key and kind (`publishable` or `anon`); otherwise null.
 
-## What it changes in your project
+Never sent: source code, file paths (except the folder-name fallback for the app name), full lockfiles, .env files, Supabase secret/service keys, URL credentials or query strings.
+The request headers x-safetyvibe-site, x-safetyvibe-key and x-safetyvibe-claim-token carry the site ID, validated report key and validated one-shot claim token when available; these are not in the JSON body. The server also refuses non-public Supabase keys and deletes stored non-public keys during cleanup.
+Public keys are found in supported environment variables/.env files or a Supabase client file. Local secret-key warnings name only the variable.
 
-- `.safetyviberc.json` (safe to commit) — this app's SafetyVibe id and public URL. No secrets.
-- `.safetyviberc.local.json` (added to `.gitignore`) — this app's report key. It can update this one
-  app's package list, link and Supabase details on SafetyVibe; it cannot read anything. Don't commit it.
-- `package.json` → `scripts.postinstall` gets `safetyvibe-connect scan --quiet || exit 0`, so the package
-  list is re-reported after each install. It never fails an install or build, even after you remove the
-  connector.
-- The one-prompt claim token links one app within 24 hours and is then useless; the CLI removes it.
+Review the entire implementation in `cli.mjs` and `lib.mjs` (approximately 485 lines total). There are no dependencies and no install scripts in this package itself.
+By default reports go only to `https://dashboard.safetyvibe.eu/connect/api/v1/report`. The committed rc cannot change the destination.
+An explicit `SAFETYVIBE_API` environment override supports HTTPS, or HTTP on localhost/127.0.0.1/[::1] for local testing. Redirects are refused.
 
-Remove it any time: delete the postinstall entry, `.safetyviberc.json`, `.safetyviberc.local.json`, and the dependency.
+## Files written and removal
+
+Before a setup/scan request, legacy credentials are migrated best-effort into the local file, the committed api field is dropped and its URL normalized. Dry-run, status and help never write. After a successful report:
+
+- `.safetyviberc.json`: app ID, normalized public URL, optional name and `hook: false`.
+- `.safetyviberc.local.json`: report key and optional fallback claim token; never commit this file. The token is removed once used or rejected as expired/used. Only stored credentials require the local file and ignore entry. The ignore entry is attempted first, then this file is written with mode 0600. An ignore failure warns and fails setup; scan still exits 0. Empty local credentials are removed.
+- `.gitignore`: adds the local config filename.
+- `package.json`: setup normally adds a postinstall report hook. `--no-hook` leaves package.json untouched and persists this preference. Existing unsafe shell scripts are left untouched with a manual instruction.
+
+The postinstall hook runs `safetyvibe-connect scan --quiet || exit 0`: it re-sends the current inventory and public Supabase details using the local report key, suppresses success chatter but keeps safety warnings on stderr, and never fails installation. Without a local key it skips reporting for a linked app unless SAFETYVIBE_KEY or a claim token is supplied.
+
+Without terminal access, first add `.safetyviberc.local.json` to `.gitignore`, then write `{"claimToken": "<TOKEN>"}` into `.safetyviberc.local.json` and configure the dependency and postinstall hook. Never put the token in the committed `.safetyviberc.json`.
+
+For manual reporting use `npx safetyvibe-connect scan` (pnpm: `pnpm dlx`, bun: `bunx`, yarn: `yarn dlx`).
+Remove the connector dependency if installed, its postinstall command, both rc files, and the local-config .gitignore entry. Keep other postinstall commands.
 
 ## Commands
 
-| command | does |
+| Command | Effect |
 |---|---|
-| `setup [--claim-token T] [--url URL] [--dry-run]` | report + write `.safetyviberc.json` + add the postinstall hook |
-| `scan [--quiet]` | report again (used by the hook; always exits 0) |
-| `status` | show what is linked |
+| `setup [--claim-token T] [--url URL] [--name NAME] [--no-hook]` | Send report, save config, optionally add hook |
+| `setup --dry-run` | Print exact report without network or writes |
+| `scan [--quiet]` | Re-report; always exits 0 for install hooks |
+| `status` | Show local link state |
+| `help`, `--help`, `-h`, no arguments | Usage only |
+| `--version`, `-v` | Package version only |
+
+Value flags accept `--flag=value` or `--flag value`, including values starting with a dash. In the space form, a known flag (also with `=...`) is treated as a missing value. Unknown commands/flags and missing values exit 2, except scan exits 0.

package/cli.mjs

@@ -1,123 +1,183 @@
 #!/usr/bin/env node
-// safetyvibe-connect — link this app to SafetyVibe security monitoring.
-//   setup [--claim-token T] [--url https://my-app.lovable.app] [--dry-run]
-//   scan  [--quiet]          re-report (runs from the postinstall hook; never fails a build)
-//   status                   show what is linked
-// Sends ONLY: package names+versions, app name, public URL, platform label, and the
-// Supabase project URL + its public anon/publishable key. Run with --dry-run to see it.
-import { readFileSync, writeFileSync, existsSync } from 'node:fs';
+import { readFileSync, writeFileSync, chmodSync, unlinkSync } from 'node:fs';
 import { join } from 'node:path';
-import { buildReport } from './lib.mjs';
+import { buildReport, findSupabase, normPublicUrl, urlNeedsCleaning, withHook, HOOK, packageManager, readPackages } from './lib.mjs';
 
-const RC = '.safetyviberc.json';              // committed: app id + URL (no secrets)
-const RC_LOCAL = '.safetyviberc.local.json';  // git-ignored: the app's report key
-const HOOK = 'safetyvibe-connect scan --quiet || exit 0';   // works in sh and cmd.exe; never fails an install
+const RC = '.safetyviberc.json';
+const LOCAL = '.safetyviberc.local.json';
 const DEFAULT_API = 'https://dashboard.safetyvibe.eu/connect/api/v1';
-
+const REPORT_KEY = /^svk_[A-Za-z0-9_-]{32}$/;
+const CLAIM_TOKEN = /^[A-Za-z0-9_-]{16,64}$/;
+const UUID = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
+const USAGE = 'Usage: safetyvibe-connect setup|scan|status|help [--claim-token TOKEN] [--url URL] [--name NAME] [--dry-run] [--quiet] [--no-hook] [--help/-h] [--version/-v]';
 const argv = process.argv.slice(2);
-const cmd = argv[0] && !argv[0].startsWith('--') ? argv[0] : 'setup';
-const flag = (name) => { const i = argv.indexOf('--' + name); return i !== -1 ? (argv[i + 1] && !argv[i + 1].startsWith('--') ? argv[i + 1] : true) : undefined; };
-const quiet = !!flag('quiet');
+const flags = {};
+let cmd = ''; let error = '';
+// Complete argument validation before reading even package.json.
+for (let i = 0; i < argv.length; i++) {
+  const arg = argv[i];
+  if (!arg.startsWith('-')) {
+    if (cmd || !['setup', 'scan', 'status', 'help'].includes(arg)) { error = 'Unknown command or argument.'; break; }
+    cmd = arg; continue;
+  }
+  const [raw, ...rest] = arg.split('=');
+  const name = ({ '-h': 'help', '-v': 'version' })[raw] || raw.replace(/^--/, '');
+  if (['claim-token', 'url', 'name'].includes(name)) {
+    const value = rest.length ? rest.join('=') : argv[++i];
+    if (!value || (!rest.length && /^(?:--(?:dry-run|quiet|no-hook|help|version|claim-token|url|name)|-h|-v)(?:=|$)/.test(value))) { error = 'Missing value for --' + name + '.'; break; }
+    flags[name] = value;
+  } else if (['dry-run', 'quiet', 'no-hook', 'help', 'version'].includes(name) && !rest.length) flags[name] = true;
+  else { error = 'Unknown flag.'; break; }
+}
+if (error) {
+  const scan = cmd === 'scan' || argv[0] === 'scan';
+  if (!(scan && argv.includes('--quiet'))) console.error(USAGE + '\n' + error);
+  process.exit(scan ? 0 : 2);
+}
+if (!argv.length || cmd === 'help' || flags.help) { console.log(USAGE); process.exit(0); }
+if (!cmd && !flags.version) { console.error(USAGE + '\nA command is required.'); process.exit(2); }
+const VERSION = JSON.parse(readFileSync(new URL('./package.json', import.meta.url), 'utf8')).version;
+if (flags.version) { console.log(VERSION); process.exit(0); }
 const root = process.env.INIT_CWD || process.cwd();
-const say = (...a) => { if (!quiet) console.log(...a); };
-
-const readJsonFile = (f) => { try { return JSON.parse(readFileSync(join(root, f), 'utf8')) || {}; } catch { return {}; } };
-function readRc() { return { ...readJsonFile(RC), ...readJsonFile(RC_LOCAL) }; }
+const say = (...args) => { if (!flags.quiet && !flags['dry-run']) console.log(...args); };
+const alert = (message) => console.error(message);
+const safeText = value => String(value).replace(/[\x00-\x1f\x7f-\x9f]/g, '').slice(0, 300);
+const readJson = (file) => { try { return JSON.parse(readFileSync(join(root, file), 'utf8')) || {}; } catch { return {}; } };
+function readRc() {
+  const pub = readJson(RC); const local = readJson(LOCAL);
+  return { ...pub, ...Object.fromEntries(['key', 'claimToken'].filter(k => Object.hasOwn(local, k)).map(k => [k, local[k]])) };
+}
 function writeRc(rc) {
-  const { key, ...pub } = rc;
+  const { key, claimToken, api, ...pub } = rc;
+  pub.url = normPublicUrl(pub.url || '');
+  let ignored = true;
+  if (key || claimToken) try {
+    let ignore = ''; try { ignore = readFileSync(join(root, '.gitignore'), 'utf8'); } catch (e) { if (e.code !== 'ENOENT') throw e; }
+    if (!ignore.split(/\r?\n/).some(l => l.trim() === LOCAL)) writeFileSync(join(root, '.gitignore'), ignore + (ignore && !ignore.endsWith('\n') ? '\n' : '') + LOCAL + '\n');
+  } catch { ignored = false; }
   try {
+    if (key || claimToken) {
+      writeFileSync(join(root, LOCAL), JSON.stringify({ ...(key ? { key } : {}), ...(claimToken ? { claimToken } : {}) }, null, 2) + '\n', { mode: 0o600 });
+      chmodSync(join(root, LOCAL), 0o600);
+    } else {
+      try { unlinkSync(join(root, LOCAL)); } catch (e) { if (e.code !== 'ENOENT') throw e; }
+    }
     writeFileSync(join(root, RC), JSON.stringify(pub, null, 2) + '\n');
-    if (key) { writeFileSync(join(root, RC_LOCAL), JSON.stringify({ key }, null, 2) + '\n'); ignoreLocal(); }
-    return true;
-  } catch { return false; }   // read-only build sandboxes: nothing to persist
-}
-function ignoreLocal() {
-  const f = join(root, '.gitignore');
-  let t = ''; try { t = readFileSync(f, 'utf8'); } catch {}
-  if (t.split(/\r?\n/).some((l) => l.trim() === RC_LOCAL)) return;
-  try { writeFileSync(f, t + (t && !t.endsWith('\n') ? '\n' : '') + RC_LOCAL + '\n'); } catch {}
+  } catch { return false; }
+  if (!ignored) alert('saved the report key but could not add it to .gitignore - add .safetyviberc.local.json to .gitignore');
+  return ignored ? 'saved' : 'unignored';
 }
 
 function addHook() {
-  const f = join(root, 'package.json');
-  let pj; try { pj = JSON.parse(readFileSync(f, 'utf8')); } catch { return 'no package.json'; }
-  pj.scripts = pj.scripts || {};
-  const cur = String(pj.scripts.postinstall || '');
-  // Only the connector's own failure is ignored: the user's existing postinstall keeps its exit code.
-  const ours = (s) => s.replace(/\s*&&\s*\(?\s*safetyvibe-connect scan[^&()]*\)?\s*$/, '').replace(/^\(?\s*safetyvibe-connect scan[^&()]*\)?\s*$/, '');
-  const base = ours(cur).trim();
-  const want = base ? base + ' && (' + HOOK + ')' : HOOK;
-  if (cur === want) return 'already there';
-  pj.scripts.postinstall = want;
-  try { writeFileSync(f, JSON.stringify(pj, null, 2) + '\n'); return 'added'; } catch { return 'could not write package.json'; }
+  let pj; let original;
+  try { original = readFileSync(join(root, 'package.json'), 'utf8'); pj = JSON.parse(original); } catch { return 'skipped (no readable package.json)'; }
+  if (!pj || typeof pj !== 'object' || Array.isArray(pj)) return 'skipped (invalid package.json)';
+  if (Object.hasOwn(pj, 'scripts') && (!pj.scripts || typeof pj.scripts !== 'object' || Array.isArray(pj.scripts))) return 'skipped (invalid scripts)';
+  const result = pj.scripts && Object.hasOwn(pj.scripts, 'postinstall') && typeof pj.scripts.postinstall !== 'string'
+    ? { status: 'unsafe' } : withHook(pj.scripts?.postinstall || '');
+  if (result.status === 'unsafe') { alert('Could not safely chain postinstall. Add this line by hand: "postinstall": "' + HOOK + '"'); return 'skipped'; }
+  if (result.status === 'already there') return result.status;
+  pj.scripts = { ...pj.scripts, postinstall: result.script };
+  try { writeFileSync(join(root, 'package.json'), JSON.stringify(pj, null, /\n(\t| {4}| {2})\S/.exec(original)?.[1] || 2) + (/\r?\n$/.exec(original)?.[0] || '')); return 'added'; }
+  catch { return 'could not write package.json'; }
 }
-
-async function post(api, body, headers) {
-  const ctl = new AbortController(); const t = setTimeout(() => ctl.abort(), 10000);
-  try {
-    const r = await fetch(api.replace(/\/+$/, '') + '/report', {
-      method: 'POST', signal: ctl.signal,
-      headers: { 'content-type': 'application/json', 'user-agent': 'safetyvibe-connect/0.2.0', ...headers },
-      body: JSON.stringify(body),
-    });
-    let j = {}; try { j = await r.json(); } catch {}
-    return { ok: r.ok, status: r.status, body: j };
-  } finally { clearTimeout(t); }
+function apiEndpoint() {
+  const u = new URL(process.env.SAFETYVIBE_API || DEFAULT_API);
+  if (u.username || u.password || u.search || u.hash || !(u.protocol === 'https:' || (u.protocol === 'http:' && ['localhost', '127.0.0.1', '[::1]'].includes(u.hostname)))) throw new Error('Invalid SAFETYVIBE_API');
+  return u.href.replace(/\/+$/, '') + '/report';
 }
-
-async function report({ setup }) {
+async function report() {
+  const setup = cmd === 'setup';
+  let destination;
+  try { destination = apiEndpoint(); } catch { alert('Invalid SAFETYVIBE_API: use HTTPS or local HTTP.'); return 1; }
   const rc = readRc();
-  const claimToken = typeof flag('claim-token') === 'string' ? flag('claim-token') : (rc.claimToken || process.env.SAFETYVIBE_CLAIM_TOKEN || '');
-  const url = typeof flag('url') === 'string' ? flag('url') : (rc.url || '');
-  const api = process.env.SAFETYVIBE_API || rc.api || DEFAULT_API;
-  const body = buildReport(root, { url });
-
-  if (flag('dry-run')) { console.log(JSON.stringify(body, null, 2)); return 0; }
-
-  const key = rc.key || process.env.SAFETYVIBE_KEY || '';      // CI: provide the report key as a secret env var
-  // Linked app, but this machine has no key (fresh clone / CI): never create a stray new app.
-  if (rc.site && !key && !claimToken) {
-    say('SafetyVibe: this app is linked (' + rc.site + '), but its report key is not on this machine. ' +
-        'Set SAFETYVIBE_KEY, or run "safetyvibe-connect setup --claim-token <new prompt token>" to relink.');
-    return 0;
-  }
-  const headers = {};
+  const rawToken = flags['claim-token'] || rc.claimToken || process.env.SAFETYVIBE_CLAIM_TOKEN || '';
+  const token = typeof rawToken === 'string' && CLAIM_TOKEN.test(rawToken) ? rawToken : '';
+  if (rawToken && !token) alert('Invalid claim token in --claim-token, SAFETYVIBE_CLAIM_TOKEN or .safetyviberc.local.json/.safetyviberc.json.');
+  const givenUrl = flags.url || rc.url || process.env.SAFETYVIBE_URL || '';
+  const url = normPublicUrl(givenUrl);
+  if (urlNeedsCleaning(givenUrl)) alert('Removed credentials/query/fragment or unsafe path from URL; only the public URL is stored and sent');
+  const body = buildReport(root, { url, name: flags.name || rc.name });
+  const headers = { 'content-type': 'application/json', 'user-agent': 'safetyvibe-connect/' + VERSION };
+  const rawKey = rc.key || process.env.SAFETYVIBE_KEY || '';
+  const key = typeof rawKey === 'string' && REPORT_KEY.test(rawKey) ? rawKey : '';
+  if (rawKey && !key) alert('Invalid report key in SAFETYVIBE_KEY or .safetyviberc.local.json/.safetyviberc.json.');
   if (rc.site) headers['x-safetyvibe-site'] = rc.site;
   if (rc.site && key) headers['x-safetyvibe-key'] = key;
-  if (claimToken) headers['x-safetyvibe-claim-token'] = claimToken;    // with x-safetyvibe-site: relink that app
-
-  let res;
-  try { res = await post(api, body, headers); }
-  catch { say('SafetyVibe: could not reach ' + api + ' (will retry on the next install).'); return setup ? 1 : 0; }
-  if (!res.ok) { say('SafetyVibe: ' + (res.body.error || 'report rejected (HTTP ' + res.status + ')')); return setup ? 1 : 0; }
-
-  const b = res.body;
-  const next = { ...rc, site: b.site || rc.site, url: url || rc.url || undefined };
-  if (b.key) next.key = b.key;
-  delete next.claimToken;                      // one-shot: never keep a used claim token around
-  // Also rewrite when a key still sits in the committed file (older setups): move it to the local file.
-  const keyInCommitted = !!readJsonFile(RC).key;
-  if (keyInCommitted || JSON.stringify(next) !== JSON.stringify(rc)) writeRc(next);
-
-  const hook = setup ? addHook() : null;
-  const sb = body.supabase ? 'Supabase project ' + new URL(body.supabase.url).hostname : 'no Supabase project found';
-  say(`SafetyVibe: reported ${body.packages.length} packages (${body.lockfile}), ${sb}${body.url ? ', ' + body.url : ''}.`);
-  if (hook) say('SafetyVibe: postinstall hook ' + hook + ' (re-reports after every install).');
-  if (b.claimed) say('SafetyVibe: linked to your dashboard: ' + (b.appUrl || 'https://dashboard.safetyvibe.eu'));
-  else if (b.claimUrl) say('SafetyVibe: open this link to add the app to your dashboard: ' + b.claimUrl);
-  return 0;
-}
-
-async function main() {
-  if (cmd === 'status') {
-    const rc = readRc();
-    console.log(rc.site ? `Linked to SafetyVibe app ${rc.site}${rc.url ? ' (' + rc.url + ')' : ''}.` : (rc.claimToken ? 'Not reported yet (claim token present).' : 'Not linked. Run: safetyvibe-connect setup'));
+  if (token) headers['x-safetyvibe-claim-token'] = token;
+  for (const variable of findSupabase(root).warnings) alert(`Warning: ${variable} holds a secret key. Variables starting with VITE_/NEXT_PUBLIC_/EXPO_PUBLIC_/PUBLIC_ are shipped to every visitor; rotate that key in Supabase.`);
+  if (flags['dry-run']) {
+    console.log(JSON.stringify(body, null, 2));
+    console.error(destination + ' headers: ' + Object.keys(headers).join(', ') + ' (values hidden)');
     return 0;
   }
-  if (cmd === 'scan') { try { await report({ setup: false }); } catch {} return 0; }   // never fail an install/build
-  if (cmd === 'setup') return report({ setup: true });
-  console.log('Usage: safetyvibe-connect setup [--claim-token T] [--url URL] [--dry-run] | scan [--quiet] | status');
-  return 1;
+  let saveFailed = false;
+  const legacy = readJson(RC);
+  if (['key', 'claimToken', 'api'].some(k => Object.hasOwn(legacy, k)) || (legacy.url && legacy.url !== normPublicUrl(legacy.url))) {
+    const saved = writeRc(rc);
+    if (!saved) alert('Could not save .safetyviberc.json migration; remove credentials manually.');
+    saveFailed = saved !== 'saved';
+  }
+  const manager = packageManager(root);
+  const runner = { npm: 'npx', pnpm: 'pnpm dlx', yarn: 'yarn dlx', bun: 'bunx' }[manager];
+  if (rc.site && !key && !token) {
+    say('SafetyVibe: report key is not on this machine. Set SAFETYVIBE_KEY or get a new prompt and run ' + runner + ' safetyvibe-connect setup --claim-token <TOKEN>.');
+    return setup ? 1 : 0;
+  }
+  let response; let b;
+  const ctl = new AbortController(); const timer = setTimeout(() => ctl.abort(), 10000);
+  try {
+    response = await fetch(destination, { method: 'POST', headers, body: JSON.stringify(body), signal: ctl.signal, redirect: 'error' });
+    try { b = await response.json(); } catch {}
+  } catch { alert('SafetyVibe: could not reach ' + new URL(destination).host + '.'); return 1; }
+  finally { clearTimeout(timer); }
+  if (typeof b?.warning === 'string') alert('SafetyVibe: ' + safeText(b.warning));
+  if (!response.ok) {
+    if (['claim_expired', 'claim_used'].includes(b?.code)) {
+      if (rc.claimToken && token === rc.claimToken) {
+        delete rc.claimToken;
+        if (!writeRc(rc)) alert('Could not remove claimToken from .safetyviberc.local.json/.safetyviberc.json; remove it manually before retrying.');
+      }
+      alert('SafetyVibe: claim token expired or already used. Get a new prompt from your dashboard.');
+    }
+    alert('SafetyVibe: report rejected: ' + (typeof b?.error === 'string' ? safeText(b.error) : 'HTTP ' + response.status));
+    return 1;
+  }
+  if (!b || typeof b !== 'object' || Array.isArray(b) || typeof b.site !== 'string' || !UUID.test(b.site)) {
+    alert('SafetyVibe: unexpected response from ' + new URL(destination).host + ' (HTTP ' + response.status + ')'); return 1;
+  }
+  const next = { ...rc, site: b.site, url, ...(flags.name ? { name: body.name } : {}), ...(flags['no-hook'] ? { hook: false } : {}) };
+  if (typeof b.key === 'string' && REPORT_KEY.test(b.key)) next.key = b.key;
+  else if (b.key) alert('Invalid report key in server response.');
+  delete next.claimToken;
+  const saved = writeRc(next);
+  if (!saved) {
+    alert('SafetyVibe: report sent; app id ' + b.site + '; could not save .safetyviberc.json here, so future reports from this folder cannot reach the same app - set SAFETYVIBE_KEY or run setup in a writable checkout');
+    return 1;
+  }
+  const inventory = readPackages(root);
+  say(`SafetyVibe: reported ${body.packages.length} packages (${body.packages.filter(p => p.dev).length} dev-only).`);
+  if (body.truncated) alert(`Only the first 5000 of ${inventory.total} packages were reported.`);
+  if (setup && next.hook !== false) {
+    const hook = addHook();
+    if (hook === 'could not write package.json') { alert('SafetyVibe: report sent; app id ' + b.site + '; could not write package.json.'); return 1; }
+    say('SafetyVibe: postinstall hook ' + hook + '.');
+  } else if (setup) say('Re-report manually: ' + runner + ' safetyvibe-connect scan');
+  if (b.claimed) say('SafetyVibe: linked to your dashboard: https://dashboard.safetyvibe.eu');
+  else {
+    let claim = '';
+    try {
+      if (typeof b.claimUrl === 'string' && !/[\x00-\x1f\x7f-\x9f]/.test(b.claimUrl)) {
+        const u = new URL(b.claimUrl);
+        if (u.protocol === 'https:' && !u.username && !u.password && u.origin === new URL(destination).origin) claim = u.href;
+      }
+    } catch {}
+    say(claim ? 'SafetyVibe: open this link to add the app to your dashboard: ' + claim : 'SafetyVibe: open your SafetyVibe dashboard');
+  }
+  return saveFailed || saved === 'unignored' ? 1 : 0;
 }
-
-main().then((code) => process.exit(code), () => process.exit(cmd === 'scan' ? 0 : 1));
+async function main() {
+  if (cmd === 'status') { const rc = readRc(); say(rc.site ? 'Linked to SafetyVibe app ' + rc.site + (normPublicUrl(rc.url) ? ' (' + normPublicUrl(rc.url) + ')' : '') + '.' : 'Not linked.'); return 0; }
+  return report();
+}
+main().then(code => process.exit(cmd === 'scan' ? 0 : code), () => { alert('SafetyVibe: could not complete report.'); process.exit(cmd === 'scan' ? 0 : 1); });

package/lib.mjs

@@ -1,9 +1,9 @@
-// safetyvibe-connect — pure helpers (no network, no writes). Tested in connect.test.mjs.
-// What leaves the machine is exactly buildReport()'s return value: package names +
-// versions, app name, public URL, platform/environment labels, and the Supabase project
-// URL + its PUBLIC (anon/publishable) key. Never source code, file paths or secrets.
-import { readFileSync, existsSync, readdirSync } from 'node:fs';
-import { join, basename } from 'node:path';
+// safetyvibe-connect - pure helpers (no network, no writes). Tested in connect.test.mjs.
+// Egress = buildReport() body + headers x-safetyvibe-site / x-safetyvibe-key /
+// x-safetyvibe-claim-token, only to the default API or SAFETYVIBE_API
+// (https, or http on localhost); redirects refused. Never source code or secret Supabase keys.
+import { readFileSync, existsSync, readdirSync, statSync, lstatSync, realpathSync } from 'node:fs';
+import { join, basename, dirname, sep, relative } from 'node:path';
 
 const MAX_PACKAGES = 5000;
 const read = (f) => { try { return readFileSync(f, 'utf8'); } catch { return null; } };
@@ -12,17 +12,19 @@ const readJson = (f) => { const t = read(f); if (t == null) return null; try { r
 // ---- lockfiles -------------------------------------------------------------
 // Each parser returns [{ name, version }] for every installed package (deduped later).
 
-export function fromPackageLock(json) {
+export function fromPackageLock(json, { platform = process.platform, arch = process.arch } = {}) {
   const out = [];
   if (json && json.packages && typeof json.packages === 'object') {
     for (const [path, v] of Object.entries(json.packages)) {
-      if (!path || !v || typeof v.version !== 'string') continue;   // "" = the root project
+      if (!path || !v || v.link || /^(workspace:|link:|file:)/.test(v.resolved || '') || typeof v.version !== 'string') continue;   // "" = the root project
+      const matches = (list, value) => !Array.isArray(list) || (!list.includes('!' + value) && (!list.some(x => !x.startsWith('!')) || list.includes(value) || list.includes('any')));
+      if (v.optional && (!matches(v.os, platform) || !matches(v.cpu, arch))) continue;
       const i = path.lastIndexOf('node_modules/');
       if (i === -1) continue;
-      out.push({ name: v.name || path.slice(i + 'node_modules/'.length), version: v.version });
+      out.push({ name: v.name || path.slice(i + 'node_modules/'.length), version: v.version, ...(v.dev === true ? { dev: true } : {}) });
     }
   } else if (json && json.dependencies) {                          // lockfileVersion 1
-    const walk = (deps) => { for (const [n, v] of Object.entries(deps || {})) { if (v && v.version) out.push({ name: n, version: v.version }); walk(v && v.dependencies); } };
+    const walk = (deps) => { for (const [n, v] of Object.entries(deps || {})) { if (v && v.version && !v.link && !/^(workspace:|link:|file:)/.test(v.resolved || '')) out.push({ name: n, version: v.version, ...(v.dev === true ? { dev: true } : {}) }); walk(v && v.dependencies); } };
     walk(json.dependencies);
   }
   return out;
@@ -31,7 +33,7 @@ export function fromPackageLock(json) {
 // bun.lock (text, JSONC): "packages": { "<key>": ["name@version", registry, {...}, integrity], ... }
 export function fromBunLock(text) {
   const clean = String(text).replace(/^\s*\/\/.*$/gm, '').replace(/,(\s*[}\]])/g, '$1');
-  let json; try { json = JSON.parse(clean); } catch { return []; }
+  const json = JSON.parse(clean);
   const out = [];
   for (const v of Object.values((json && json.packages) || {})) {
     const spec = Array.isArray(v) ? v[0] : null;
@@ -49,7 +51,7 @@ export function fromYarnLock(text) {
   const out = []; let names = null;
   for (const line of String(text).split(/\r?\n/)) {
     if (/^\S.*:\s*$/.test(line) && !line.startsWith('#') && !line.startsWith('__metadata')) {
-      names = line.replace(/:\s*$/, '').split(',').map((s) => s.trim().replace(/^"|"$/g, ''))
+      names = /@(workspace:|link:|file:)/.test(line) ? null : line.replace(/:\s*$/, '').split(',').map((s) => s.trim().replace(/^"|"$/g, ''))
         .map((s) => s.slice(0, s.lastIndexOf('@') > 0 ? s.lastIndexOf('@') : s.length));
       continue;
     }
@@ -68,6 +70,7 @@ export function fromPnpmLock(text) {
     if (!inPackages) continue;
     const m = line.match(/^ {2}'?\/?((?:@[^/@\s]+\/)?[^@\s/]+)@(\d[^('\s:]*)/);
     if (m) out.push({ name: m[1], version: m[2] });
+    else if (/^ {4}dev: true\s*$/.test(line) && out.length) out[out.length - 1].dev = true;
   }
   return out;
 }
@@ -76,7 +79,7 @@ export function fromPnpmLock(text) {
 export function fromNodeModules(root) {
   const out = []; const nm = join(root, 'node_modules');
   let entries = []; try { entries = readdirSync(nm); } catch { return out; }
-  const add = (dir) => { const pj = readJson(join(dir, 'package.json')); if (pj && pj.name && pj.version) out.push({ name: pj.name, version: pj.version }); };
+  const add = (dir) => { try { if (lstatSync(dir).isSymbolicLink() && !realpathSync(dir).startsWith(realpathSync(nm) + sep)) return; } catch { return; } const pj = readJson(join(dir, 'package.json')); if (pj && pj.name && pj.version) out.push({ name: pj.name, version: pj.version }); };
   for (const e of entries) {
     if (e.startsWith('.')) continue;
     if (e.startsWith('@')) { let sub = []; try { sub = readdirSync(join(nm, e)); } catch {} for (const s of sub) add(join(nm, e, s)); }
@@ -94,8 +97,37 @@ export function readPackages(root) {
     ['pnpm-lock.yaml', fromPnpmLock],
     ['yarn.lock', fromYarnLock],
   ];
-  for (const [file, parse] of tries) {
-    const t = read(join(root, file)); if (t == null) continue;
+  let lockRoot = root;
+  let current = root;
+  for (let depth = 0; depth <= 6; depth++) {
+    const ws = readJson(join(current, 'package.json'))?.workspaces;
+    const yaml = read(join(current, 'pnpm-workspace.yaml')) || '';
+    const block = /^packages:\s*\r?\n((?:[ \t]+.*\r?\n?)*)/m.exec(yaml)?.[1] || '';
+    const globs = [...(Array.isArray(ws) ? ws : Array.isArray(ws?.packages) ? ws.packages : []),
+      ...[...block.matchAll(/^\s+-\s+['"]?([^'"\r\n]+?)['"]?\s*$/gm)].map(m => m[1])];
+    const segments = relative(current, root).split(sep);
+    const matchesGlob = g => {
+      const parts = g.replace(/^\.\//, '').replace(/\/$/, '').split('/');
+      const segmentMatches = (pattern, value) => new RegExp('^' + pattern.split('*').map(s => s.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')).join('[^/]*') + '$').test(value);
+      const match = (i, j) => i === parts.length ? j === segments.length
+        : parts[i] === '**' ? match(i + 1, j) || (j < segments.length && match(i, j + 1))
+        : j < segments.length && segmentMatches(parts[i], segments[j]) && match(i + 1, j + 1);
+      return match(0, 0);
+    };
+    const workspace = globs.some(g => typeof g === 'string' && !g.startsWith('!') && matchesGlob(g))
+      && !globs.some(g => typeof g === 'string' && g.startsWith('!') && matchesGlob(g.slice(1)));
+    if (current === root || workspace) {
+      lockRoot = current;
+      if (workspace || tries.some(([f]) => existsSync(join(current, f))) || existsSync(join(current, 'bun.lockb'))) break;
+    }
+    if (existsSync(join(current, '.git')) || dirname(current) === current || depth === 6) break;
+    current = dirname(current);
+  }
+  const manager = (readJson(join(root, 'package.json'))?.packageManager || readJson(join(lockRoot, 'package.json'))?.packageManager || '').split('@')[0];
+  const preferred = { npm: ['package-lock.json', 'npm-shrinkwrap.json'], pnpm: ['pnpm-lock.yaml'], yarn: ['yarn.lock'], bun: ['bun.lock'] }[manager] || [];
+  const candidates = tries.filter(([f]) => existsSync(join(lockRoot, f))).sort(([a], [b]) => Number(preferred.includes(b)) - Number(preferred.includes(a)) || statSync(join(lockRoot, b)).mtimeMs - statSync(join(lockRoot, a)).mtimeMs);
+  for (const [file, parse] of candidates) {
+    const t = read(join(lockRoot, file)); if (t == null) continue;
     let ok = true;
     try { list = parse(t); } catch { list = []; ok = false; }
     if (list.length) { source = file; break; }
@@ -103,109 +135,168 @@ export function readPackages(root) {
     // yarn/pnpm parsers can't tell empty from unsupported, so those stay "unknown".
     if (ok && /\.json$|^bun\.lock$/.test(file)) { source = file; break; }
   }
-  if (!list.length && source === 'none') { list = fromNodeModules(root); if (list.length) source = 'node_modules'; }
-  const seen = new Set(); const pkgs = [];
+  if (!list.length && source === 'none') { list = fromNodeModules(root); if (!list.length && lockRoot !== root) list = fromNodeModules(lockRoot); if (list.length) source = 'node_modules'; }
+  const seen = new Map(); const pkgs = [];
   for (const p of list) {
     const k = p.name + '@' + p.version;
     // Registry versions only. git/file/link/http specs can carry tokens or local paths: never sent.
-    if (seen.has(k) || !/^(@[\w.-]+\/)?[\w.-]+$/.test(p.name) || !/^\d+\.\d+\.\d+[\w.+-]{0,40}$/.test(String(p.version))) continue;
-    seen.add(k); pkgs.push({ ecosystem: 'npm', name: p.name, version: p.version });
-    if (pkgs.length >= MAX_PACKAGES) break;
+    if ([p.name, p.version].some(v => /sb_secret_|svk_/i.test(String(v)) || keyKind(v) !== 'unknown')) continue;
+    if (!/^(@[\w.-]+\/)?[\w.-]+$/.test(p.name) || !/^\d+\.\d+\.\d+[\w.+-]{0,40}$/.test(String(p.version))) continue;
+    if (seen.has(k)) { if (p.dev !== true) delete seen.get(k).dev; continue; }
+    const row = { ecosystem: 'npm', name: p.name, version: p.version, ...(p.dev === true ? { dev: true } : {}) };
+    seen.set(k, row); pkgs.push(row);
+
   }
-  return { packages: pkgs, source };
+  return { packages: pkgs.slice(0, MAX_PACKAGES), source, ...(pkgs.length > MAX_PACKAGES ? { total: pkgs.length, truncated: true } : {}) };
 }
 
 // ---- Supabase (project URL + PUBLIC key only) ------------------------------
-export function keyKind(key) {
-  const k = String(key || '').trim();
-  if (/^sb_publishable_/.test(k)) return 'publishable';
-  if (/^sb_secret_/.test(k)) return 'secret';
-  const parts = k.split('.');
-  if (parts.length === 3) {
-    try {
-      const payload = JSON.parse(Buffer.from(parts[1].replace(/-/g, '+').replace(/_/g, '/'), 'base64').toString('utf8'));
-      if (payload && payload.role === 'anon') return 'anon';
-      if (payload && payload.role) return 'secret';               // service_role or anything else privileged
-    } catch { /* not a JWT */ }
-  }
-  return 'unknown';
+function jwtPayload(value) {
+  if (typeof value !== 'string' || value.length > 2000 || !/^[A-Za-z0-9_-]+\.[A-Za-z0-9_-]+\.[A-Za-z0-9_-]+$/.test(value)) return null;
+  try { const p = JSON.parse(Buffer.from(value.split('.')[1], 'base64url').toString('utf8')); return p && typeof p === 'object' && !Array.isArray(p) ? p : null; } catch { return null; }
 }
-
-function parseEnv(text) {
+export function publicKeyKind(value) {
+  if (typeof value !== 'string' || /sb_secret_|[\s#'"`]/i.test(value)) return null;
+  if (/^sb_publishable_[A-Za-z0-9_-]{8,200}$/.test(value)) return 'publishable';
+  const payload = jwtPayload(value);
+  const claims = new Set(['iss', 'ref', 'role', 'iat', 'exp', 'nbf', 'aud', 'sub']);
+  return payload?.role === 'anon'
+    && (!Object.hasOwn(payload, 'iss') || ['supabase', 'supabase-demo'].includes(payload.iss))
+    && (!Object.hasOwn(payload, 'ref') || (typeof payload.ref === 'string' && /^[a-z0-9]{10,40}$/.test(payload.ref)))
+    && Object.entries(payload).every(([key, value]) => claims.has(key)
+      && (typeof value === 'number' || (typeof value === 'string' && value.length <= 100 && !/sb_secret_|svk_|eyJ/i.test(value)))) ? 'anon' : null;
+}
+export function keyKind(value) {
+  const kind = publicKeyKind(value);
+  if (kind) return kind;
+  const payload = jwtPayload(value);
+  return /sb_secret_/i.test(String(value || '')) || (payload && typeof payload.role === 'string' && payload.role !== 'anon') ? 'secret' : 'unknown';
+}
+export function parseEnv(text) {
   const env = {};
   for (const line of String(text || '').split(/\r?\n/)) {
-    const m = line.match(/^\s*(?:export\s+)?([A-Z0-9_]+)\s*=\s*(.*)\s*$/);
-    if (m) env[m[1]] = m[2].replace(/^['"]|['"]$/g, '');
+    const m = line.match(/^\s*(?:export\s+)?([\w.-]+)\s*=\s*(.*)$/);
+    if (!m) continue;
+    const value = m[2]; const quote = value[0];
+    if (["'", '"', '`'].includes(quote)) {
+      let end = 1;
+      for (; end < value.length; end++) {
+        if (quote === '"' && value[end] === '\\') { end++; continue; }
+        if (value[end] === quote) break;
+      }
+      if (end === value.length) continue;
+      env[m[1]] = value.slice(1, end);
+    } else env[m[1]] = value.split('#')[0].trim();
   }
   return env;
 }
 
 const URL_VARS = ['VITE_SUPABASE_URL', 'NEXT_PUBLIC_SUPABASE_URL', 'PUBLIC_SUPABASE_URL', 'EXPO_PUBLIC_SUPABASE_URL', 'SUPABASE_URL'];
 const KEY_VARS = ['VITE_SUPABASE_PUBLISHABLE_KEY', 'VITE_SUPABASE_ANON_KEY', 'NEXT_PUBLIC_SUPABASE_PUBLISHABLE_KEY', 'NEXT_PUBLIC_SUPABASE_ANON_KEY',
-  'PUBLIC_SUPABASE_ANON_KEY', 'EXPO_PUBLIC_SUPABASE_ANON_KEY', 'SUPABASE_PUBLISHABLE_KEY', 'SUPABASE_ANON_KEY'];
+  'PUBLIC_SUPABASE_PUBLISHABLE_KEY', 'EXPO_PUBLIC_SUPABASE_PUBLISHABLE_KEY', 'PUBLIC_SUPABASE_ANON_KEY', 'EXPO_PUBLIC_SUPABASE_ANON_KEY', 'SUPABASE_PUBLISHABLE_KEY', 'SUPABASE_ANON_KEY'];
 
 export function findSupabase(root, procEnv = process.env) {
   const env = {};
   for (const f of ['.env', '.env.local', '.env.production', '.env.production.local']) Object.assign(env, parseEnv(read(join(root, f))));
   for (const k of [...URL_VARS, ...KEY_VARS]) if (procEnv[k]) env[k] = procEnv[k];
+  const warnings = KEY_VARS.filter(k => keyKind(env[k]) === 'secret');
   let url = URL_VARS.map((k) => env[k]).find(Boolean) || '';
-  let key = KEY_VARS.map((k) => env[k]).find(Boolean) || '';
+  let key = KEY_VARS.map((k) => env[k]).find(value => publicKeyKind(value)) || '';
   // Lovable hard-codes them in src/integrations/supabase/client.ts
   if (!url || !key) {
     for (const f of ['src/integrations/supabase/client.ts', 'src/integrations/supabase/client.js', 'src/lib/supabase.ts', 'src/lib/supabase.js', 'lib/supabase.ts', 'utils/supabase.ts']) {
       const t = read(join(root, f)); if (!t) continue;
       url = url || ((t.match(/["'`](https:\/\/[a-z0-9-]+\.supabase\.co)["'`]/i) || [])[1] || '');
-      key = key || ((t.match(/(?:PUBLISHABLE|ANON)_KEY\s*=\s*["'`]([^"'`]+)["'`]/i) || [])[1] || '');
+      for (const match of t.matchAll(/([\w]*(?:PUBLISHABLE|ANON)_KEY)\s*=\s*["'`]([^"'`]+)["'`]/gi)) {
+        if (keyKind(match[2]) === 'secret') warnings.push(match[1]);
+        if (!key && publicKeyKind(match[2])) key = match[2];
+      }
       if (url && key) break;
     }
   }
   if (!/^https:\/\/[a-z0-9-]+\.supabase\.(co|in)\/?$/i.test(url)) url = '';
   const kind = key ? keyKind(key) : 'none';
   // Never send anything that isn't provably a public key.
-  if (kind !== 'anon' && kind !== 'publishable') key = '';
-  return { url: url.replace(/\/$/, ''), key, kind };
+  if (!publicKeyKind(key)) key = '';
+  return { url: url.replace(/\/$/, ''), key, kind, warnings };
 }
 
 // ---- app identity ------------------------------------------------------------
-export function detectPlatform(root, procEnv = process.env) {
+const HOSTS = { 'lovable.app': 'Lovable', 'replit.app': 'Replit', 'repl.co': 'Replit', 'bolt.host': 'Bolt', 'base44.app': 'Base44', 'emergent.host': 'Emergent', 'emergentagent.com': 'Emergent', 'softgen.ai': 'Softgen', 'softgen.dev': 'Softgen', 'lp.dev': 'Leap', 'vercel.app': 'Vercel', 'netlify.app': 'Netlify', 'run.app': 'Google Cloud Run' };
+export function detectPlatform(root, procEnv = process.env, url = '') {
   const pj = readJson(join(root, 'package.json')) || {};
-  const deps = { ...(pj.dependencies || {}), ...(pj.devDependencies || {}) };
+  const deps = { ...pj.dependencies, ...pj.devDependencies };
   if (deps['lovable-tagger'] || existsSync(join(root, '.lovable'))) return 'Lovable';
-  if (procEnv.REPL_ID || existsSync(join(root, '.replit'))) return 'Replit';
-  if (existsSync(join(root, '.bolt'))) return 'Bolt';
+  if (existsSync(join(root, '.replit'))) return 'Replit';
+  for (const [marker, label] of Object.entries({ '.bolt': 'Bolt', '.emergent': 'Emergent', '.softgen': 'Softgen', '.leap': 'Leap', '.rocket': 'Rocket', '.v0': 'v0' })) if (existsSync(join(root, marker))) return label;
   if (deps['@base44/sdk']) return 'Base44';
+  if (Object.hasOwn(readJson(join(root, 'metadata.json')) || {}, 'requestFramePermissions') || (pj.name === 'react-example' && deps['@google/genai']) || ['ts', 'js', 'mjs', 'mts'].some(ext => /process\.env\.GEMINI_API_KEY/.test(read(join(root, 'vite.config.' + ext)) || ''))) return 'Google AI Studio';
+  try { const host = new URL(url).hostname; for (const [suffix, label] of Object.entries(HOSTS)) if (host.endsWith('.' + suffix)) return label; } catch {}
+  if (procEnv.REPL_ID) return 'Replit';
   if (procEnv.VERCEL) return 'Vercel';
   if (procEnv.NETLIFY) return 'Netlify';
   return '';
 }
-
-export function appName(root) {
-  const pj = readJson(join(root, 'package.json')) || {};
-  const n = String(pj.name || '').trim();
-  // Builder templates ship a generic package name; send none so the dashboard can name it properly.
-  // Known builder template names only (a real name like "email-template-editor" is kept).
-  const generic = /^(vite[_-]react[_-]shadcn[_-]ts|tanstack[_-]start[_-]ts|vite[_-]react[_-]typescript[_-]starter|vite[_-]project|rest[_-]express|my[_-]app|app|react[_-]app|next[_-]?js[_-]app|next[_-]app|nextjs)$/i;
-  if (n && !generic.test(n)) return n.slice(0, 80);
-  const dir = basename(root);
-  return /^(app|project|workspace|src|home|repl|dev-server|code|\.)$/i.test(dir) ? '' : dir.slice(0, 80);
+const TEMPLATES = new Set('workspace frontend backend client web app project src home repl code dev-server react-example vite-project vite-react-shadcn-ts tanstack-start-ts vite-react-typescript-starter rest-express my-app react-app nextjs next-app nextjs-app my-v0-project'.split(' '));
+// Real JWTs start with eyJ (base64 of {"); plain dotted names like notes.example.com stay allowed.
+const secretLike = value => keyKind(value) !== 'unknown' || /sb_secret_|svk_|eyJ/i.test(value);
+export function appName(root, explicit = '', url = '') {
+  let host = ''; try { host = new URL(url).hostname; if (Object.keys(HOSTS).some(s => host.endsWith('.' + s))) host = host.split('.')[0]; } catch {}
+  for (const candidate of [explicit, readJson(join(root, 'package.json'))?.name, host, basename(root)]) {
+    const raw = String(candidate || '').trim();
+    if (secretLike(raw)) continue;
+    const n = raw.replace(/[^A-Za-z0-9 @/._-]/g, '').trim();
+    if (n && !TEMPLATES.has(n.toLowerCase().replace(/_/g, '-')) && !secretLike(n) && !secretLike(n.slice(0, 80))) return n.slice(0, 80);
+  }
+  return '';
+}
+export function packageManager(root, env = process.env) {
+  const field = readJson(join(root, 'package.json'))?.packageManager?.split('@')[0];
+  if (['npm', 'pnpm', 'yarn', 'bun'].includes(field)) return field;
+  const source = readPackages(root).source;
+  if (source.startsWith('pnpm')) return 'pnpm';
+  if (source.startsWith('yarn')) return 'yarn';
+  if (source.startsWith('bun') || existsSync(join(root, 'bun.lockb'))) return 'bun';
+  if (source.endsWith('.json')) return 'npm';
+  return /^(pnpm|yarn|bun)\//.exec(env.npm_config_user_agent || '')?.[1] || 'npm';
+}
+export const HOOK = 'safetyvibe-connect scan --quiet || exit 0';
+export function withHook(current = '') {
+  const cur = String(current).trim();
+  const base = cur.replace(/\s*&&\s*\(?safetyvibe-connect scan(?: --quiet)?(?: \|\| exit 0)?\)?$/, '').replace(/^\(?safetyvibe-connect scan(?: --quiet)?(?: \|\| exit 0)?\)?$/, '').trim();
+  if (/^[&|;]|[&|;\\^]$|[\r\n`<>#;$%!]|\$\(|[()]/.test(base) || /(?<![&|])[&|](?![&|])/.test(base) || /['"]/.test(base)) return { script: current, status: 'unsafe' };
+  const script = base ? base + ' && (' + HOOK + ')' : HOOK;
+  return { script, status: script === cur ? 'already there' : 'added' };
 }
 
-export function normPublicUrl(u) {
-  try { const x = new URL(String(u)); return x.protocol === 'https:' || x.protocol === 'http:' ? x.origin + (x.pathname === '/' ? '' : x.pathname) : ''; }
-  catch { return ''; }
+function publicUrl(given) {
+  try {
+    const u = new URL(String(given));
+    if (!['http:', 'https:'].includes(u.protocol)) return { url: '', cleaning: false };
+    const keep = /^(\/[A-Za-z0-9._~-]{1,64}){0,8}\/?$/.test(u.pathname)
+      && u.pathname.split('/').every(segment => segment.length <= 40 && !/sb_|svk_|eyJ/i.test(segment));
+    return { url: u.origin + (keep && u.pathname !== '/' ? u.pathname : ''),
+      cleaning: Boolean(u.username || u.password || u.search || u.hash || !keep) };
+  } catch { return { url: '', cleaning: false }; }
 }
+export function normPublicUrl(given) { return publicUrl(given).url; }
+export function urlNeedsCleaning(given) { return publicUrl(given).cleaning; }
 
 export function buildReport(root, opts = {}, procEnv = process.env) {
-  const { packages, source } = readPackages(root);
+  const { packages, source, truncated } = readPackages(root);
+  const url = normPublicUrl(opts.url || procEnv.SAFETYVIBE_URL || '');
   const sb = findSupabase(root, procEnv);
+  const rawEnvironment = String(opts.environment || procEnv.SAFETYVIBE_ENV || (procEnv.NODE_ENV === 'production' ? 'production' : ''));
+  const environment = rawEnvironment.replace(/[^A-Za-z0-9_-]/g, '');
   return {
     v: 1,
-    name: appName(root),
-    url: normPublicUrl(opts.url || procEnv.SAFETYVIBE_URL || ''),
-    platform: detectPlatform(root, procEnv),
-    environment: String(opts.environment || procEnv.SAFETYVIBE_ENV || (procEnv.NODE_ENV === 'production' ? 'production' : '')).slice(0, 20),
+    name: appName(root, opts.name, url),
+    url,
+    platform: detectPlatform(root, procEnv, url),
+    environment: [rawEnvironment, environment, environment.slice(0, 20)].some(secretLike) ? '' : environment.slice(0, 20),
     lockfile: source,
     packages,
+    ...(truncated ? { truncated: true } : {}),
     supabase: sb.url && sb.key ? { url: sb.url, key: sb.key, kind: sb.kind } : null,
   };
 }

package/package.json

@@ -1,31 +1,31 @@
-{
-  "name": "safetyvibe-connect",
-  "version": "0.2.0",
-  "description": "SafetyVibe security monitoring for AI-built apps (Lovable, Replit, Bolt, Claude Code\u2026). Reports the app's dependency list, public URL and, if used, its Supabase project URL + public anon key. Never source code or secrets.",
-  "type": "module",
-  "bin": {
-    "safetyvibe-connect": "cli.mjs"
-  },
-  "files": [
-    "cli.mjs",
-    "lib.mjs",
-    "README.md",
-    "AGENTS.md"
-  ],
-  "engines": {
-    "node": ">=18"
-  },
-  "license": "UNLICENSED",
-  "homepage": "https://safetyvibe.eu",
-  "keywords": [
-    "security",
-    "vulnerabilities",
-    "lovable",
-    "replit",
-    "bolt",
-    "supabase",
-    "vibe-coding",
-    "safetyvibe"
-  ],
-  "author": "SafetyVibe <safetyvibe@advertis.ee>"
-}
+{
+  "name": "safetyvibe-connect",
+  "version": "0.2.1",
+  "description": "SafetyVibe security monitoring for AI-built apps (Lovable, Replit, Bolt, Claude Code\u2026). Reports the app's dependency list, public URL and, if used, its Supabase project URL + public anon key. Never source code or secrets.",
+  "type": "module",
+  "bin": {
+    "safetyvibe-connect": "cli.mjs"
+  },
+  "files": [
+    "cli.mjs",
+    "lib.mjs",
+    "README.md",
+    "AGENTS.md"
+  ],
+  "engines": {
+    "node": ">=18"
+  },
+  "license": "UNLICENSED",
+  "homepage": "https://safetyvibe.eu",
+  "keywords": [
+    "security",
+    "vulnerabilities",
+    "lovable",
+    "replit",
+    "bolt",
+    "supabase",
+    "vibe-coding",
+    "safetyvibe"
+  ],
+  "author": "SafetyVibe <safetyvibe@advertis.ee>"
+}