eresearchqut / ddb-repository / 26199482554

98%

push

github

fix: remove dead code branches and correct getItems return type

- getItems return type narrowed from Promise<Array<T> | undefined> to
  Promise<Array<T>>: the implementation always returns an array; the
  undefined case was never reachable
- Remove redundant ?. and || [] inside the if (page.Items) guard in
  getItems: the guard already proves truthiness
- Remove ?? [] fallbacks on getItems calls in JsonPointerRepository:
  now provably unnecessary with the corrected return type

Retained defensive guards backed by SDK optional types (result.Items ?? [],
if (page.Items) in scan) and the setAtPointer tokens.length === 0 guard
which prevents silent writes to current['undefined'] on invalid input.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

200 of 211 branches covered (94.79%)

Branch coverage included in aggregate %.

3 of 3 new or added lines in 2 files covered. (100.0%)

328 of 328 relevant lines covered (100.0%)

135.53 hits per line