Skip to content

Hermes v1を維持しつつiOS起動クラッシュを修正#5475

Merged
TinyKitten merged 3 commits intodevfrom
claude/fix-ios-hermes-crash-oPHtp
Feb 27, 2026
Merged

Hermes v1を維持しつつiOS起動クラッシュを修正#5475
TinyKitten merged 3 commits intodevfrom
claude/fix-ios-hermes-crash-oPHtp

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Feb 27, 2026

  • app.config.tsのexpo-assetプラグイン重複エントリを削除
  • hermes-compilerのバージョン固定を削除してExpo/RNの自動解決に委任

https://claude.ai/code/session_01LFCzVgpnrCDTqn7qcbymgH

Summary by CodeRabbit

リリースノート

Chores(保守)

  • iOS プラットフォームでHermes V1 JavaScriptエンジンのサポートを有効化しました
  • ビルド設定から重複するプラグイン定義を削除し、設定ファイルをクリーンアップしました

- app.config.tsのexpo-assetプラグイン重複エントリを削除
- hermes-compilerのバージョン固定を削除してExpo/RNの自動解決に委任

https://claude.ai/code/session_01LFCzVgpnrCDTqn7qcbymgH
- PodfileにRCT_HERMES_V1_ENABLED環境変数の設定行を追加(Expoテンプレートとの差分)
- Podfile.properties.jsonにexpo.useHermesV1プロパティを追加
- hermes-compilerオーバーライドを復元(Hermes V1に必須)
- @react-native/js-polyfillsと@react-native/metro-configをRN 0.83に対応するバージョンに更新
- app.config.tsのexpo-assetプラグイン重複を削除

https://claude.ai/code/session_01LFCzVgpnrCDTqn7qcbymgH
@github-actions github-actions bot added the ios label Feb 27, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 41a87d9 and 268e200.

📒 Files selected for processing (3)
  • app.config.ts
  • ios/Podfile
  • ios/Podfile.properties.json
💤 Files with no reviewable changes (1)
  • app.config.ts

📝 Walkthrough

Walkthrough

このプルリクエストは、Expo設定ファイルの重複削除とiOSビルド構成でのHermes V1有効化を実施しています。app.config.tsから重複した'expo-asset'プラグインエントリを削除し、Podfileプロパティと環境変数によってHermes V1サポートを有効にしています。

Changes

コホート / ファイル(s) 概要
設定クリーンアップ
app.config.ts
プラグイン配列から重複した'expo-asset'エントリを削除しました。機能上の変更なし。
Hermes V1有効化
ios/Podfile, ios/Podfile.properties.json
新しい設定エントリ「expo.useHermesV1」を追加し、Podfileで対応する環境変数RCT_HERMES_V1_ENABLEDを設定しました。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • hermes v1導入 #5471: このPRと同様にHermes V1を有効化するプロジェクト設定変更を実施しており、expo-build-propertiesプラグインオプションの追加とPodfileおよびPodfile.propertiesの設定が関連しています。

Poem

🐰 重複を取り払い、
Hermes V1の光を灯す✨
設定ファイルたちが、
心を一つにして、
iOSビルドへ駆け出す!🚀

🚥 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のタイトルは「Hermes v1を維持しつつiOS起動クラッシュを修正」で、変更内容の主要な目的(iOS起動クラッシュの修正とHermes v1の維持)を明確に表しており、実際の変更内容(app.config.tsの重複削除、Podfileの設定追加、Podfile.properties.jsonの設定追加)と関連性があります。
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 claude/fix-ios-hermes-crash-oPHtp

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

@TinyKitten TinyKitten merged commit de58dfc into dev Feb 27, 2026
7 checks passed
@TinyKitten TinyKitten deleted the claude/fix-ios-hermes-crash-oPHtp branch February 27, 2026 10:45
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.

2 participants