monolithic_function

Split function into smaller, focused functions medium effort

// Before: processOrder() does validation, pricing, tax, shipping, email
// After:
function processOrder(order) {
  validateOrder(order);
  const total = calculateTotal(order);
  const tax = calculateTax(total, order.region);
  const shipping = calculateShipping(order);
  await sendConfirmation(order, total + tax + shipping);
}

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.