Repository navigation
fix(chat): show only nickname in message headers - #392
Merged
Merged
Conversation
lipluscodex
commented
Oct 11, 2026
lipluscodex
left a comment
Member
Author
There was a problem hiding this comment.
Codex Luna:自己レビュー完了(head 4184294)。
#391 の受け入れ条件を満たします。新着・履歴が共有する roomLine は speaker だけを表示し、ニックネーム未設定時の名前へのフォールバックを保ちます。アカウントの二段表示、full_name の保存・通信、宛先解決には変更がありません。仕様3文書と既存 UI テストの期待値も一致しています。
検証: frontend build、sidecar typecheck、既存 sidecar tests 79 passed / 5 skipped。変更範囲の逸脱・保留事項なし。
実装は親モデルを継承した low effort の delegate に依頼しました。表示条件が確定した小さな変更で、既存テスト・ビルドと親の差分レビューで確認できる範囲でした。
CI 全成功と CLEAN を確認後、auto モードで squash merge します。稼働画面への反映は merge 後に行います。
— Codex Luna
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.
Closes #391
チャット欄の話し手を新着・履歴とも投稿時の呼称だけで表示します。
アカウントの二段表示と元の名前のメタデータを維持し、仕様と既存 UI テストの期待値を更新しました。
検証: frontend build、sidecar typecheck、既存 sidecar tests(79 passed / 5 skipped)成功。ローカルの全体テストは実行プロセスの PULLCEPT_* 環境変数を除いて確認しました。
— Codex Luna