Repository navigation
fix(band): preserve Unicode characters when truncating session names - #1385
Open
dungartoriaaa wants to merge 1 commit into
Open
dungartoriaaa wants to merge 1 commit into
dungartoriaaa wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #1384
Tiêu đề có 59 ký tự ASCII rồi tới
😀hiện bị cắt thành một surrogate đơn. Khi ghi CSV UTF-8, ký tự đó trở thành�. Bản sửa giới hạn tên phiên theo 60 Unicode code point ở cả prompt đầu tiên, tên hội thoại đã đổi và phiên import, giữ nguyên ký tự đầy đủ.Chỉ sửa 3 dòng xử lý tên và thêm kiểm thử hồi quy cho ba đường vào. Các kiểm thử mới đều thất bại trên
96608e0và đạt sau sửa. Kiểm thử CSV còn xác nhận tên được giữ nguyên khi mã hóa UTF-8.Kiểm chứng
npm testđạt 144 kiểm thử gốc và 30 kiểm thử pi.node scripts/check-rule-copies.jsđạt.node scripts/check-versions.jsđạt.python benchmarks/agentic/run.py --selftestđạt.claude plugin test .có 29 đạt và 5 lỗi có sẵn trên Windows. Trên bản gốc có 27 đạt và cùng 5 lỗi, trùng các trường hợp được báo trong band tests fail on Windows: fs.read mocks compare an exact POSIX path #1375. Hai kiểm thử mới đều đạt sau sửa.English summary
Truncate session names by Unicode code point instead of UTF-16 code unit in all three naming paths. This prevents a supplementary character at the 60-character boundary from becoming a lone surrogate and then
�in UTF-8 CSV or UI output. Regression coverage includes first prompts, custom titles, imported sessions and a CSV/board round trip. The two new plugin tests pass. Five existing Windows plugin-test failures reproduce unchanged on the base revision, as described in #1375. All required contribution checks pass.