Skip to content

fix(utils): reject whitespace in decodeTokenIds with helpful error#1865

Merged
ryanio merged 1 commit intomainfrom
fix/reject-whitespace-in-decode-token-ids
Jan 14, 2026
Merged

fix(utils): reject whitespace in decodeTokenIds with helpful error#1865
ryanio merged 1 commit intomainfrom
fix/reject-whitespace-in-decode-token-ids

Conversation

@ryanio
Copy link
Collaborator

@ryanio ryanio commented Jan 14, 2026

Summary

  • Rejects input containing whitespace with a clear error message
  • Error message shows the expected format: '1,2,3' or '1:5' or '1,3:5,8' (no spaces)

Changes

  • Add whitespace check that throws a descriptive error
  • Add unit test verifying whitespace rejection

Test plan

  • All existing tests pass (553 passing)
  • New test verifies whitespace is rejected with helpful error message

🤖 Generated with Claude Code

Add explicit check for whitespace that throws a clear error message
showing the expected format: '1,2,3' or '1:5' or '1,3:5,8' (no spaces).

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@ryanio ryanio merged commit 8f1c08e into main Jan 14, 2026
7 of 8 checks passed
@ryanio ryanio deleted the fix/reject-whitespace-in-decode-token-ids branch January 14, 2026 20:44
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