high_cognitive_complexity

Reduce nesting depth with early returns and guard clauses low effort

// Before: deeply nested
function validate(user) {
  if (user) {
    if (user.email) {
      if (isValid(user.email)) {
        return true;
      }
    }
  }
  return false;
}

// After: guard clauses
function validate(user) {
  if (!user) return false;
  if (!user.email) return false;
  return isValid(user.email);
}

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.