Skip to content

Fixed chatimport-bug - #109

Merged
Husky110 merged 3 commits into
LettuceAI:mainfrom
Husky110:main
Oct 16, 2026
Merged

Husky110 merged 3 commits into
LettuceAI:mainfrom
Husky110:main

Conversation

@Husky110

Copy link
Copy Markdown
Collaborator

Chatimports had a bug in which a simultanious created_at-timestamp messed up the order in the frontend.

Imported chats could get their messages shuffled because message order relies
solely on created_at (ms): there is no dedicated sequence column and ids are
random v4 UUIDs, so the tiebreaker is arbitrary. Sources with colliding
timestamps (e.g. second-granularity SillyTavern send dates, or missing dates)
produced identical created_at, so the order resolved non-deterministically and
re-shuffled across save/reload cycles.

- Import now assigns strictly increasing created_at in source-array order
  (running maximum, +1 on collision/backstep) in both the Lettuce backup
  importer and the SillyTavern jsonl importer (single and group chats). Real,
  already-increasing timestamps are left untouched.
- The missing-timestamp fallback anchors one minute before now so a turn sent
  right after the import still sorts after the imported history.
- read_session's message load gains the ", id ASC" tiebreaker to match the
  other load paths, so order is deterministic across reloads.
@Husky110 Husky110 closed this Oct 16, 2026
@Husky110 Husky110 reopened this Oct 16, 2026
android-actions/setup-android@v3 defaults to packages 'tools platform-tools',
but the legacy 'tools' SDK package has been removed from the Android repository.
sdkmanager then fails with "Failed to find package 'tools'" (exit 1), breaking
every Android CI run regardless of the committed changes. Pin packages to an
empty set in all four Android workflows (android-build, android-release-build,
pr-check, espeak-android-bundle); the runner already ships the command-line
tools and Gradle resolves platform/build-tools itself.
@Husky110
Husky110 merged commit a5597ca into LettuceAI:main Oct 16, 2026
5 checks passed
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.

2 participants