Aguilera Engineering

/security-review

/security-review reads the diff on the current branch and flags security risks: injection, broken auth, secret exposure, unsafe deserialization, dependency vulnerabilities surfaced in the diff, and similar.

You’re about to open a PR that touches authentication. You want a security-focused pass before reviewers do.

~/code/api  >  /security-review

  Reviewing 3 changed files for security issues…

  HIGH   auth/middleware.go:42
         JWT secret falls back to a hardcoded string when ENV is unset.
         An attacker with the binary can forge valid tokens.

  MED    handlers/upload.go:89
         File extension checked before MIME type. Bypass by renaming
         a .php to .png.

  LOW    go.mod
         golang.org/x/crypto v0.17.0 → v0.18.0 patches CVE-2023-48795
         (Terrapin, SSH prefix truncation). Not in the diff hot path,
         but worth updating.

  3 issues found. Run with --fix to apply safe mechanical fixes.

I reach for it before opening any PR that touches auth, file handling, or external input — it catches the class of mistakes that are obvious in review but easy to miss while writing.

#claude-code

Reply to this post by email ↪