Skip to content

Conversation

@drizco
Copy link
Contributor

@drizco drizco commented Jan 7, 2026

ordering by id should take less effort than ordering by created_at and is low risk since we have no intention of backfilling chat events.

following this pr I will create a migration that adds indexes for the chat_history query that includes the id

Links

Testing story

Deployment strategy

Follow-up work

Privacy

Security

Caching

PR Creation Checklist:

  • Tests provide adequate coverage
  • Privacy impacts have been documented
  • Security impacts have been documented
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Follow-up work items (including potential tech debt) are tracked and linked

@drizco drizco marked this pull request as ready for review January 7, 2026 21:18
@drizco drizco merged commit 0cd3555 into staging Jan 7, 2026
6 checks passed
@drizco drizco deleted the ryan/aichat/fix/chat-history-order-by-id branch January 7, 2026 22:01
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.

3 participants