Skip to content

fix: ルーム名と背景の文字数をAPIとそろえる - #36

Merged
Eotel merged 1 commit into
mainfrom
codex/unicode-form-limits-19
Oct 25, 2026
Merged

Eotel merged 1 commit into
mainfrom
codex/unicode-form-limits-19

Conversation

@Eotel

@Eotel Eotel commented Oct 25, 2026

Copy link
Copy Markdown
Contributor

Summary

「𠮷」を含むルーム名・背景が、上限の半分で切られる不具合を修正します。ルーム名120文字、背景4,000文字を画面とAPIで同じように扱います。

Pydantic → OpenAPI → form-limits.ts
                   → Home / Operator の保存前検査
                     → 超過した下書きを保持し、フォーム内にエラー表示

#19 の境界調査で、辞書の同じ問題は #35 で修正済みです。このPRは残る3入力へ対応し、背景の検査を共通化します。辞書の改版・永続化、認識Adapter、AI取消、手修正の保護、公開可否は既存の担当層に残します。参照元と残す理由は ARCHITECTURE.md に記録しています。

Closes #19

Evidence

  • Before: 補助漢字の貼り付けテストで、ルーム名は expected 120 / received 60、背景は expected 4000 / received 2000。AIなしで修正すると入力欄が消える問題と、保存後の表示解除漏れも回帰テストで再現しました。
    After: bash scripts/check.sh 成功。Node 22.23.4、frontend 145件・backend 196件、型検査0エラー。生成契約も一致します。ローカルのfrontend件数には別作業の未コミットテスト1件を含み、このPRには含めていません。
  • 実ブラウザ+隔離した実APIで、120/4,000文字の登録、121/4,001文字の保持と送信拒否、キーボード送信の拒否を確認。AIなしでも修正中のフォーカスを保ち、保存成功後に通常表示へ戻ります。APIへの不正更新は422で保存済み設定を保持しました。
  • 取消、既存ルームへの再接続、公開前編集も確認。保存前の観客字幕は0件、保存後は手修正した1件だけです。390px幅でも横はみ出しなし。別セッション監査でP0・P1なし。音声・外部AIは起動していません。

検証記録

超過した背景を保持

修正中の入力欄とフォーカスを保持

Merge Danger

Door: two-way

既存データの変換やAPI形式の変更はありません。必要ならこのコミットをrevertできます。Pydanticの検証は保持します。

Blast Radius: forms

対象はルーム登録・編集とAI背景設定です。OS別認識や外部サービスへの実接続は今回の検証対象外です。

投稿セッション:合成音声でGemini・GPT動作確認

@Eotel Eotel self-assigned this Oct 25, 2026
@Eotel
Eotel merged commit e0d8344 into main Oct 25, 2026
2 checks passed
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.

整理方針: 字幕入力・AI 校正・生成 API 型の責務を整理する

1 participant