Skip to content

maintain token ordering in all scenarios#937

Merged
ShanaryS merged 1 commit intomainfrom
tokenize-bulk-fix
Mar 12, 2026
Merged

maintain token ordering in all scenarios#937
ShanaryS merged 1 commit intomainfrom
tokenize-bulk-fix

Conversation

@ShanaryS
Copy link
Copy Markdown
Collaborator

If some of the text was already cached then the ordering of the tokens would be off. This doesn't matter currently as we don't use tokenizeBulk for anything order sensitive, as it's mainly used to fill the tokenizeCache or get all tokens for clipboard imports.

@ShanaryS ShanaryS self-assigned this Mar 12, 2026
@ShanaryS ShanaryS added the bug Something isn't working label Mar 12, 2026
@ShanaryS ShanaryS merged commit 2954c8f into main Mar 12, 2026
2 checks passed
@ShanaryS ShanaryS deleted the tokenize-bulk-fix branch March 12, 2026 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant