Skip to content

nameIpa使用箇所をnameRomanIpaに書き換え#5601

Merged
TinyKitten merged 4 commits intodevfrom
feature/use-name-roman-ipa
Mar 15, 2026
Merged

nameIpa使用箇所をnameRomanIpaに書き換え#5601
TinyKitten merged 4 commits intodevfrom
feature/use-name-roman-ipa

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Mar 15, 2026

Summary

  • GraphQLクエリおよびコード内で nameIpa を参照していた箇所を nameRomanIpa に変更
  • TrainType のGraphQLフラグメント(TinyTrainTypeFields, TrainTypeNestedFields)に nameRomanIpa フィールドを追加
  • 英語TTS生成時に TrainTypenameRomanIpa を発音記号として ph() に渡すよう修正
  • phoneme.ts の引数名を nameRomanIpa にリネーム
  • GraphQL codegenを再生成

Test plan

  • 英語TTSで路線名・駅名・種別名の発音が正しく生成されることを確認
  • npm run typecheck が通ることを確認
  • npm test が通ることを確認

🤖 Generated with Claude Code

Summary by CodeRabbit

リリースノート

  • Refactor
    • 表示用の音声表記フィールドを統一して名称を変更しました(IPA系のフィールド置換)。
  • New Features
    • 音声合成で駅名・路線名のローマ字IPA表記が優先的に利用されるようになり、発音の一貫性が向上します。
  • Tests
    • テストデータとフィクスチャを新しい表記に合わせて更新しました。

TinyKitten and others added 2 commits March 15, 2026 12:37
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions github-actions bot added the react label Mar 15, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 15, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9d148afb-fea8-4784-9ead-bb49791f2dc3

📥 Commits

Reviewing files that changed from the base of the PR and between 34a1802 and 4f00164.

📒 Files selected for processing (12)
  • src/__fixtures__/line.ts
  • src/components/TrainTypeBoxJL.test.tsx
  • src/components/TrainTypeBoxJO.test.tsx
  • src/components/TrainTypeBoxJRKyushu.test.tsx
  • src/components/TrainTypeBoxSaikyo.test.tsx
  • src/hooks/useCurrentTrainType.test.tsx
  • src/hooks/useGetStationsWithTermination.test.tsx
  • src/hooks/useRefreshStation.test.tsx
  • src/hooks/useTransferLinesFromStation.test.tsx
  • src/utils/__fixtures__/search.ts
  • src/utils/test/factories.ts
  • src/utils/trainTypeString.test.ts
✅ Files skipped from review due to trivial changes (1)
  • src/hooks/useRefreshStation.test.tsx

📝 Walkthrough

Walkthrough

GraphQL型・フラグメントとそれを消費するTTS/音素ユーティリティの公開型名を一括で rename: nameIpanameRomanIpa。型定義、クエリ/フラグメント、TTSマッピング、phonemeユーティリティ、関連フィクスチャ/テストのフィールドを更新。

変更内容

Cohort / File(s) Summary
GraphQL型定義
src/@types/graphql.d.ts
公開型(Line, LineNested, Station, StationNested, TrainType, TrainTypeNested 等)で nameIpa を削除し nameRomanIpa を追加/置換。関連するフラグメント結果型も同様に更新。
GraphQLフラグメント / クエリ
src/lib/graphql/queries.ts
複数フラグメント(TINY_TRAIN_TYPE_FRAGMENT, LINE_NESTED_FRAGMENT, STATION_FRAGMENT, LINE_ROUTE_FRAGMENT 等)で nameIpanameRomanIpa に置換または追加し、返却データ形状を一致させる。
TTS テキスト生成フック
src/hooks/useTTSText.ts
英語/日本語テンプレート内の phoneme 呼び出しや境界表現で nameIpanameRomanIpa に置換。Next/Arriving/transfer 等の音素マッピングを更新。
音素ユーティリティ
src/utils/phoneme.ts
wrapPhoneme の引数名と JSDoc を nameIpanameRomanIpa に変更し、内部条件も対応。
ユーティリティデータ
src/utils/jr.ts, src/__fixtures__/line.ts, src/utils/__fixtures__/search.ts, src/utils/test/factories.ts
JRラインやフィクスチャ/テストファクトリのオブジェクトに nameRomanIpa: null を追加し、モック/fixture 形状を更新。
テスト更新
src/components/*/*.test.tsx, src/hooks/*/*.test.tsx, src/utils/trainTypeString.test.ts
テストモック/ファクトリで nameIpa / nameRomanIpa を追加または初期化(主に null)して型整合性を確保。

Sequence Diagram(s)

(該当なし — フィールド名の置換と型/テンプレート更新のため図は省略)

推定コード審査難度

🎯 3 (Moderate) | ⏱️ ~25 分

Possibly related PRs

🐰 名は移りて Ipa は去りぬ、RomanIpa と共に跳ねるよ
型の道をぴょんと駆け回り
音素の小径にも新しい印
テストはぴったり、フィクスチャもにっこり
さあ発話よ、整列した名前で♪

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PRタイトルは、GraphQLの nameIpa フィールドを nameRomanIpa に変更するという主要な変更内容を正確かつ簡潔に表現しており、概要と一致している。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/use-name-roman-ipa
📝 Coding Plan
  • Generate coding plan for human review comments

Comment @coderabbitai help to get the list of available commands and usage tips.

TinyKitten and others added 2 commits March 15, 2026 12:38
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@TinyKitten TinyKitten self-assigned this Mar 15, 2026
@TinyKitten TinyKitten merged commit cdd6523 into dev Mar 15, 2026
6 checks passed
@TinyKitten TinyKitten deleted the feature/use-name-roman-ipa branch March 15, 2026 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant