Field Notes
How we scope a secure code review without boiling the ocean
Teams often ask for “a full review of the monorepo.” That request usually collapses into a shallow skim. We start instead with the journeys that can move money, issue tokens, or elevate privilege.
Name the journeys
Ask stakeholders to list three user journeys that would embarrass the company if broken. Map those journeys to handlers, jobs, and shared libraries. Everything else waits for a later phase.
Freeze the tag
Findings that point at a moving branch create arguments instead of tickets. Tag the candidate, record the commit hash in the engagement letter, and review that snapshot.
Agree severity language early
Whether you use CVSS or an internal High/Medium/Low scale, decide before the first finding is written. Mid-review renegotiation of severity is where trust erodes.
Leave an explicit out-of-scope list
Marketing sites, read-only blogs, and deprecated services should appear on paper as excluded. Clients sleep better when the report does not pretend those areas were examined.