Skip to content

fix: resolve failing tests and implement missing utilities - #459

Open
stooit wants to merge 1 commit into
mainfrom
quantcode/e2e-tier1-912-1794285700
Open

stooit wants to merge 1 commit into
mainfrom
quantcode/e2e-tier1-912-1794285700

Conversation

@stooit

@stooit stooit commented Nov 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • calculator: divide throws on division by zero
  • date-utils: formatRelative day rounding off-by-one fixed (symmetric past/future)
  • string-utils: implemented truncate (word boundary, ellipsis counts toward maxLength, never exceeds maxLength); wordCount handles repeated whitespace
  • task-manager: implemented remove, update (whitelists title/description/priority), sortBy (priority/status/createdAt)
  • validator: isEmail accepts long TLDs; isUrl accepts ports
  • Removed stale TODO/BUG comments

Verification

bun test: 60 pass, 0 fail (was 44/16). tsc --noEmit clean. Reviewed twice, no critical/high issues.

Assumptions

  • Sort orders: priority high→low, status pending→in_progress→completed, createdAt oldest first.
  • truncate with maxLength < 3 returns a hard slice without ellipsis.
  • No test files modified, no dependencies added.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant