fix(paginate): break loop on empty items with non-null nextToken

[aldorizona10-glitch]

Summary

Fixes an infinite loop in paginate() (issue #35) where the CLI hangs indefinitely when a filtered list endpoint returns { items: [], nextToken: 'cursor' }.

Problem

paginate() in src/lib/pagination.ts checked nextToken !== null to decide whether to keep looping, but never checked whether items was actually empty. Without --max-items set, the loop had no way to stop for this edge case, causing testsprite test list --project <id> with a zero-result filter to hang forever.

Fix

Added an early-exit guard after processing each page:

// Guard against infinite loop when API returns empty items with non-null nextToken.
if (page.items.length === 0) break;

This breaks the loop when the API returns zero items, regardless of what nextToken says. The nextToken from the server is still returned to the caller for potential resume.

Changes

• src/lib/pagination.ts: Added empty-items guard in paginate() loop
• test/pagination.test.ts: New file with 5 unit tests:
  • Collects all items across pages until nextToken is null
  • Breaks out of loop on empty items with non-null nextToken (the bug repro)
  • Respects maxItems cap
  • Handles single page with null nextToken
  • Handles consecutive empty pages then data

Testing

• npm run typecheck — clean
• npm run lint — clean
• npm run format:check — all files pass Prettier
• npm test — all existing tests + 5 new tests pass

Closes #35

Signed-off-by: Aldo Rizona aldorizona10@gmail.com

[aldorizona10-glitch]

Hi maintainers 👋

First-time contributor here — CI runs are awaiting approval ().

Quick summary of this PR:

• Fixes Bug: Infinite loop in paginate() when API returns empty items with non-null nextToken #35: infinite loop in paginate() when API returns empty items with non-null nextToken
• 2-line fix: if (page.items.length === 0) break;
• 5 new unit tests covering the bug repro + edge cases
• Typecheck, lint, format, build, and all existing tests pass locally

Coverage impact: this PR adds tests to src/lib/pagination.ts (previously had 14 tests, now 19). The 2 lines added are fully covered. Global coverage should remain ≥80%.

Happy to make any changes if needed. Thanks for reviewing!