Skip to content

Conversation

@lcomplete
Copy link
Owner

@lcomplete lcomplete commented Jan 20, 2026

Summary

  • Replace complex multi-field fallback sorting (collectedAt -> savedAt -> archivedAt) with simpler createdAt-based sorting for unsorted pages
  • Ensure collectedAt is always set when pages are added to library, preventing null values in batch operations
  • Simplify PageListSort enum by removing multi-field sort logic

Test plan

  • Verify unsorted pages list displays correctly sorted by creation time
  • Test batch move operations to ensure collectedAt is properly set
  • Confirm regular collection sorting (COLLECTED_AT) still works as expected

🤖 Generated with Claude Code

lcomplete and others added 2 commits January 21, 2026 03:49
Replace complex multi-field fallback (collectedAt -> savedAt -> archivedAt)
with simpler createdAt sorting for unsorted pages. Also ensure collectedAt
is always set when pages are added to library to prevent null values.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@lcomplete lcomplete merged commit 665d244 into main Jan 20, 2026
1 check passed
@augmentcode
Copy link

augmentcode bot commented Jan 20, 2026

🤖 Augment PR Summary

Summary: Refactors “unsorted” page sorting to use a single, always-present timestamp (createdAt) instead of a multi-field fallback.

Changes:

  • Updates client sort labeling so UNSORTED_SAVED_AT is shown as “Created”
  • Simplifies PageListSort by removing multi-field sort support and using createdAt for unsorted pages
  • Updates page item mapping/pagination record timestamp for unsorted lists to use createdAt
  • Ensures collectedAt is populated via batch operations and like-to-library flows by coalescing to createdAt (or publish time where applicable)

Technical Notes: Server-side sort building in PageListService is simplified to a single-field sort now that unsorted no longer requires multi-field ordering.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 1 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

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