Versus

every change decides for itself

• #1user detailgetXthrow

  const u = await getUserById(id)

• #2order listgetXthrow

  const list = await getOrders()

• #3profile cardfetch_xthrow

  const p = await fetch_profile(id)

• #4empty statefetch_xnull

  if (!p) return null

• #5invoice rowfetch_xnull

  function fmtDate(d) { … }

• #6audit logfetch_xconsole

  console.error('bad date')

• #7team pageloadXnull

  const t = await loadTeam()

• #8date columnloadXnull

  function formatDate2(d) { … }

• #9billingloadXswallow

  try { … } catch {}

• #10settingsfetchXswallow

  const s = await fetchSettings()

“We don't write it that way here” — said four times, in one week.

decided once, applied every time

• · reads are named getX
• · failures throw AppError, never return null
• · dates go through @/lib/format

• #1user detailgetXthrow

  const u = await getUser(id)

• #2order listgetXthrow

  const list = await getOrders()

• #3profile cardgetXthrow

  const p = await getProfile(id)

• #4empty stategetXthrow

  if (!p) throw new AppError('no profile')

• #5invoice rowgetXthrow

  import { formatDate } from '@/lib/format'

• #6audit loggetXthrow

  throw new AppError('unparseable date')

• #7team pagegetXthrow

  const t = await getTeam()

• #8date columngetXthrow

  import { formatDate } from '@/lib/format'

• #9billinggetXthrow

  throw new AppError('charge failed', { cause })

• #10settingsgetXthrow

  const s = await getSettings()

Review spent its whole budget on whether the logic is right.