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.