Skip to content

PR作成時のPR Title文字数切り詰め処理を削除#115

Draft
kyu08 wants to merge 1 commit intomainfrom
fix/cjk-truncate-displaywidth
Draft

PR作成時のPR Title文字数切り詰め処理を削除#115
kyu08 wants to merge 1 commit intomainfrom
fix/cjk-truncate-displaywidth

Conversation

@kyu08
Copy link
Copy Markdown
Collaborator

@kyu08 kyu08 commented Apr 3, 2026

Summary

  • build_comment_entriesbody_preview で行っていた60文字バイト切り詰め処理を削除
  • CJK文字(UTF-8で3バイト)がバイト単位で途中切断され表示が壊れる問題があった。
    • が、そもそも切り詰め処理自体がない方が体験が良い気がしたので機能ごと削除した。
  • 対応するテストケースも削除

Test plan

  • make all (lint, format-check, test) パス確認済み

🤖 Generated with Claude Code

Truncation caused broken display with CJK characters (byte-based cut
instead of display-width-aware). Rather than fixing the truncation logic,
remove it entirely as full-length text is preferable.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 3, 2026 09:39
@kyu08 kyu08 changed the title コメントプレビューの文字列切り詰め処理を削除 PR作成時のPR Title文字数切り詰め処理を削除 Apr 3, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

build_comment_entries における body_preview の60バイト切り詰め処理を削除し、UTF-8のCJK文字がバイト境界で分断されて表示が壊れる問題を解消するPRです。

Changes:

  • lua/fude/comments/data.lua から body_preview の60バイト切り詰め処理を削除
  • それに対応していたテストケース(切り詰め検証)を削除

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
lua/fude/comments/data.lua body_preview のバイト単位切り詰めを削除し、マルチバイト文字の途中切断を回避
tests/fude/comments_spec.lua 旧仕様(60バイト切り詰め)前提のテストケースを削除

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