Skip to content

fix: 辞書の入力制限を共有し、補助漢字を正しい文字数で保存 - #35

Merged
Eotel merged 1 commit into
mainfrom
codex/dictionary-contract-19
Oct 25, 2026
Merged

Eotel merged 1 commit into
mainfrom
codex/dictionary-contract-19

Conversation

@Eotel

@Eotel Eotel commented Oct 25, 2026

Copy link
Copy Markdown
Contributor

Summary

APIが受け付ける補助漢字80文字を、画面では40文字までしか入力できない問題を修正します。Refs #19

- 画面のHTML maxLength=80(UTF-16)
+ Unicodeの文字数で80文字まで保存、超過した下書きは保持
- 文字数・件数の上限をフロントとバックで個別に定義
+ models.pyのDICTIONARY_LIMITSから画面用の値を生成

生成した規則を入力検査、単語追加の可否、件数・上限の説明に使います。API型とUnicode case foldingの生成を維持し、生成結果の一致検査も辞書の上限を対象にします。API・上限値・保存形式は変更しません。

入力中の検査とAPIの検証は、それぞれ画面の下書きと外部からの要求を守るために残します。認識・AI要求の辞書取得、手修正と公開、取消の既存の担当も維持し、責務をARCHITECTURE.mdへ記録しました。

Evidence

  • Before: Rootの貼り付けテストで、補助漢字80文字が40文字となる失敗を確認。
    After: 80文字を保存でき、81文字はAPIを呼ばず、下書きとエラーを保持します。
  • 実ブラウザと隔離した実FastAPIで、表記・読み各80文字(UTF-16では各160)の保存を確認。81文字は画面とAPIの両方で拒否し、APIの保存改版と内容を保持しました。検証記録
  • bash scripts/check.sh成功。ローカルのフロントエンド143件、バックエンド195件、生成契約の一致、型検査、ビルドが成功しました。
  • 別セッションで画像・差分を監査しP0・P1なし。音声入力・再生・実AI提供元への送信は開始していません。隔離した試験サーバーは終了しました。

80文字の保存

81文字の下書きとエラー

Merge Danger

Door: two-way

Blast Radius: dictionary

生成ファイルの参照先をアプリ内部で更新しています。frontendはprivate packageで、HTTPのAPI形式は維持します。ルーム名とトークの背景にも同じHTML制限が残るため、次の計画に分けて続けます。#19はその確認まで開いておきます。

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

@Eotel Eotel self-assigned this Oct 25, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-25T12:32:23.941199Z 7bfa1e5 PR opened
🔒 Security Review ✅ Completed 2026-10-25T12:33:41.352807Z 7bfa1e5 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@Eotel
Eotel merged commit 26fbe02 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.

1 participant