inconsistent_test_patterns

Standardize test file naming and organization low effort

// Pick ONE convention:
// Option A: co-located — src/auth/auth.test.ts
// Option B: mirrored   — test/auth/auth.test.ts
// Option C: __tests__  — src/auth/__tests__/auth.test.ts

// Then configure in vitest.config.ts:
export default { test: { include: ['**/*.test.ts'] } }

Suppress a confirmed non-issue with a committed .pullguardignore entry (pullguard ignore locally, or comment /pullguard ignore <fingerprint> <reason> on the PR — the fingerprint is printed in the PR comment’s Triage section). Entries support expiresAt for time-boxed snoozes.

Security findings at major or critical severity — and any critical finding — always surface: .pullguardignore cannot hide them. The reviewed paths that keep them visible are acknowledged (reviewed, stays in reports) and, for a confirmed false positive, a reasoned false_positive entry — visible and audited, excluded only from the merge block.