Use the Hermes artifacts in actions (#53837)

j-piasecki · facebook-github-bot · commit 3be0d37b820c · 2025-09-18T07:37:47.000-07:00
Summary: Pull Request resolved: #53837 Changelog: [Internal] Differential Revision: D82721725
diff --git a/.github/actions/test-ios-helloworld/action.yml b/.github/actions/test-ios-helloworld/action.yml
@@ -25,21 +25,16 @@ runs:
       uses: ./.github/actions/setup-node
     - name: Run yarn install
       uses: ./.github/actions/yarn-install
-    - name: Create Hermes folder
-      shell: bash
-      run: mkdir -p "$HERMES_WS_DIR"
-    - name: Download Hermes
-      uses: actions/download-artifact@v4
-      with:
-        name: hermes-darwin-bin-${{ inputs.flavor }}
-        path: /tmp/hermes/hermes-runtime-darwin/
-    - name: Print Downloaded hermes
-      shell: bash
-      run: ls -lR "$HERMES_WS_DIR"
     - name: Setup ruby
       uses: ruby/setup-ruby@v1
       with:
         ruby-version: ${{ inputs.ruby-version }}
+    - name: Set nightly Hermes versions
+      shell: bash
+      run: |
+        node ./scripts/releases/set-hermes-versions.js --nightly
+    - name: Run yarn install again, with the correct hermes version
+      uses: ./.github/actions/yarn-install
     - name: Download ReactNativeDependencies
       uses: actions/download-artifact@v4
       with:
@@ -66,20 +61,6 @@ runs:
           args+=(--frameworks dynamic)
         fi
 
-        # Tarball is restored with capital flavors suffix, but somehow the tarball name from JS at line 96 returns as lowercased.
-        # Let's ensure that the tarballs have the right names
-
-        if [[ -f "$HERMES_WS_DIR/hermes-runtime-darwin/hermes-ios-Debug.tar.gz" ]]; then
-          mv "$HERMES_WS_DIR/hermes-runtime-darwin/hermes-ios-Debug.tar.gz" "$HERMES_WS_DIR/hermes-runtime-darwin/hermes-ios-debug.tar.gz"
-        fi
-
-        if [[ -f "$HERMES_WS_DIR/hermes-runtime-darwin/hermes-ios-Release.tar.gz" ]]; then
-          mv "$HERMES_WS_DIR/hermes-runtime-darwin/hermes-ios-Release.tar.gz" "$HERMES_WS_DIR/hermes-runtime-darwin/hermes-ios-release.tar.gz"
-        fi
-
-        BUILD_TYPE="${{ inputs.flavor }}"
-        TARBALL_FILENAME=$(node ../../packages/react-native/scripts/hermes/get-tarball-name.js --buildType "$BUILD_TYPE")
-        export HERMES_ENGINE_TARBALL_PATH="$HERMES_WS_DIR/hermes-runtime-darwin/$TARBALL_FILENAME"
         export RCT_USE_LOCAL_RN_DEP="/tmp/third-party/ReactNativeDependencies${{ inputs.flavor }}.xcframework.tar.gz"
         export RCT_TESTONLY_RNCORE_TARBALL_PATH="/tmp/ReactCore/ReactCore${{ inputs.flavor }}.xcframework.tar.gz"
 
diff --git a/.github/actions/test-ios-rntester/action.yml b/.github/actions/test-ios-rntester/action.yml
@@ -10,9 +10,6 @@ inputs:
   run-unit-tests:
     description: whether unit tests should run or not.
     default: "false"
-  hermes-tarball-artifacts-dir:
-    description: The directory where the hermes tarball artifacts are stored
-    default: /tmp/hermes/hermes-runtime-darwin
   flavor:
     description: The flavor of the build. Must be one of "Debug", "Release".
     default: Debug
@@ -36,56 +33,19 @@ runs:
       uses: ./.github/actions/setup-node
     - name: Run yarn
       uses: ./.github/actions/yarn-install
-    - name: Download Hermes
-      uses: actions/download-artifact@v4
-      with:
-        name: hermes-darwin-bin-${{ inputs.flavor }}
-        path: ${{ inputs.hermes-tarball-artifacts-dir }}
     - name: Setup ruby
       uses: ruby/setup-ruby@v1
       with:
         ruby-version: ${{ inputs.ruby-version }}
+    - name: Set nightly Hermes versions
+      shell: bash
+      run: |
+        node ./scripts/releases/set-hermes-versions.js --nightly
+    - name: Run yarn install again, with the correct hermes version
+      uses: ./.github/actions/yarn-install
     - name: Prepare IOS Tests
       if: ${{ inputs.run-unit-tests == 'true' }}
       uses: ./.github/actions/prepare-ios-tests
-    - name: Set HERMES_ENGINE_TARBALL_PATH envvar if Hermes tarball is present
-      shell: bash
-      run: |
-        HERMES_TARBALL_ARTIFACTS_DIR=${{ inputs.hermes-tarball-artifacts-dir }}
-        if [ ! -d $HERMES_TARBALL_ARTIFACTS_DIR ]; then
-          echo "Hermes tarball artifacts dir not present ($HERMES_TARBALL_ARTIFACTS_DIR). Build Hermes from source."
-          exit 0
-        fi
-
-        TARBALL_FILENAME=$(node ./packages/react-native/scripts/hermes/get-tarball-name.js --buildType "${{ inputs.flavor }}")
-        TARBALL_PATH=$HERMES_TARBALL_ARTIFACTS_DIR/$TARBALL_FILENAME
-
-        echo "Looking for $TARBALL_FILENAME in $HERMES_TARBALL_ARTIFACTS_DIR"
-        echo "$TARBALL_PATH"
-
-        if [ ! -f $TARBALL_PATH ]; then
-          echo "Hermes tarball not present ($TARBALL_PATH). Build Hermes from source."
-          exit 0
-        fi
-
-        echo "Found Hermes tarball at $TARBALL_PATH"
-        echo "HERMES_ENGINE_TARBALL_PATH=$TARBALL_PATH" >> $GITHUB_ENV
-    - name: Print Hermes version
-      shell: bash
-      run: |
-        HERMES_TARBALL_ARTIFACTS_DIR=${{ inputs.hermes-tarball-artifacts-dir }}
-        TARBALL_FILENAME=$(node ./packages/react-native/scripts/hermes/get-tarball-name.js --buildType "${{ inputs.flavor }}")
-        TARBALL_PATH=$HERMES_TARBALL_ARTIFACTS_DIR/$TARBALL_FILENAME
-        if [[ -e $TARBALL_PATH ]]; then
-          tar -xf $TARBALL_PATH
-          echo 'print(HermesInternal?.getRuntimeProperties?.()["OSS Release Version"])' > test.js
-          chmod +x ./destroot/bin/hermes
-          ./destroot/bin/hermes test.js
-          rm test.js
-          rm -rf destroot
-        else
-          echo 'No Hermes tarball found.'
-        fi
     - name: Download ReactNativeDependencies
       uses: actions/download-artifact@v4
       with:
@@ -105,7 +65,6 @@ runs:
     - name: Install CocoaPods dependencies
       shell: bash
       run: |
-        export HERMES_ENGINE_TARBALL_PATH=$HERMES_ENGINE_TARBALL_PATH
         export RCT_USE_LOCAL_RN_DEP="/tmp/third-party/ReactNativeDependencies${{ inputs.flavor }}.xcframework.tar.gz"
         export RCT_TESTONLY_RNCORE_TARBALL_PATH="/tmp/ReactCore/ReactCore${{ inputs.flavor }}.xcframework.tar.gz"
 
@@ -116,6 +75,7 @@ runs:
         cd packages/rn-tester
 
         bundle install
+        bundle exec pod update hermes-engine --no-repo-update
         bundle exec pod install
     - name: Build RNTester
       shell: bash
diff --git a/.github/workflows/test-all.yml b/.github/workflows/test-all.yml
@@ -127,9 +127,6 @@ jobs:
     runs-on: macos-14
     needs:
       [build_apple_slices_hermes, prepare_hermes_workspace, build_hermes_macos, prebuild_apple_dependencies, prebuild_react_native_core]
-    env:
-      HERMES_WS_DIR: /tmp/hermes
-      HERMES_TARBALL_ARTIFACTS_DIR: /tmp/hermes/hermes-runtime-darwin
     steps:
       - name: Checkout
         uses: actions/checkout@v4
@@ -145,9 +142,6 @@ jobs:
     runs-on: macos-14-large
     needs:
       [build_apple_slices_hermes, prepare_hermes_workspace, build_hermes_macos, prebuild_apple_dependencies, prebuild_react_native_core]
-    env:
-      HERMES_WS_DIR: /tmp/hermes
-      HERMES_TARBALL_ARTIFACTS_DIR: /tmp/hermes/hermes-runtime-darwin
     continue-on-error: true
     strategy:
       fail-fast: false
@@ -169,9 +163,6 @@ jobs:
     runs-on: macos-14-large
     needs:
       [test_ios_rntester]
-    env:
-      HERMES_WS_DIR: /tmp/hermes
-      HERMES_TARBALL_ARTIFACTS_DIR: /tmp/hermes/hermes-runtime-darwin
     strategy:
       fail-fast: false
       matrix:
@@ -259,9 +250,6 @@ jobs:
           REACT_NATIVE_PKG=$(find /tmp/react-native-tmp -type f -name "*.tgz")
           echo "React Native tgs is $REACT_NATIVE_PKG"
 
-          HERMES_PATH=$(find /tmp/react-native-tmp -type f -name "*.tar.gz")
-          echo "Hermes path is $HERMES_PATH"
-
           # For stable branches, we want to use the stable branch of the template
           # In all the other cases, we want to use "main"
           BRANCH=${{ github.ref_name }}
@@ -278,7 +266,7 @@ jobs:
           export RCT_USE_LOCAL_RN_DEP=/tmp/third-party/ReactNativeDependencies${{ matrix.flavor }}.xcframework.tar.gz
           # Disable prebuilds for now, as they are causing issues with E2E tests for 0.82-stable branch
           # export RCT_TESTONLY_RNCORE_TARBALL_PATH="/tmp/ReactCore/ReactCore${{ matrix.flavor }}.xcframework.tar.gz"
-          HERMES_ENGINE_TARBALL_PATH=$HERMES_PATH RCT_NEW_ARCH_ENABLED=$NEW_ARCH_ENABLED bundle exec pod install
+          RCT_NEW_ARCH_ENABLED=$NEW_ARCH_ENABLED bundle exec pod install
 
           xcodebuild \
             -scheme "RNTestProject" \
diff --git a/packages/gradle-plugin/react-native-gradle-plugin/src/main/kotlin/com/facebook/react/utils/DependencyUtils.kt b/packages/gradle-plugin/react-native-gradle-plugin/src/main/kotlin/com/facebook/react/utils/DependencyUtils.kt
@@ -231,12 +231,7 @@ internal object DependencyUtils {
     // explicitly in the version string? Remember to also update the logic for iOS
     val hermesVersionString =
         (hermesVersionProperties[INTERNAL_HERMES_VERSION_NAME] as? String).orEmpty()
-    val hermesVersion =
-        if (hermesVersionString.startsWith("0.0.0") || "-commitly-" in hermesVersionString) {
-          "$hermesVersionString-SNAPSHOT"
-        } else {
-          hermesVersionString
-        }
+    val hermesVersion = hermesVersionString
 
     val hermesV1Version =
         (hermesVersionProperties[INTERNAL_HERMES_V1_VERSION_NAME] as? String).orEmpty()
diff --git a/packages/react-native/sdks/hermes-engine/hermes-utils.rb b/packages/react-native/sdks/hermes-engine/hermes-utils.rb
@@ -254,15 +254,15 @@ def nightly_tarball_url(version)
   artifact_name = "hermes-ios-debug.tar.gz"
   namespace = "com/facebook/hermes"
 
-  xml_url = "https://central.sonatype.com/repository/maven-snapshots/#{namespace}/#{artifact_coordinate}/#{version}-SNAPSHOT/maven-metadata.xml"
+  xml_url = "https://central.sonatype.com/repository/maven-snapshots/#{namespace}/#{artifact_coordinate}/#{version}/maven-metadata.xml"
 
   response = Net::HTTP.get_response(URI(xml_url))
   if response.is_a?(Net::HTTPSuccess)
     xml = REXML::Document.new(response.body)
     timestamp = xml.elements['metadata/versioning/snapshot/timestamp'].text
     build_number = xml.elements['metadata/versioning/snapshot/buildNumber'].text
     full_version = "#{version}-#{timestamp}-#{build_number}"
-    final_url = "https://central.sonatype.com/repository/maven-snapshots/#{namespace}/#{artifact_coordinate}/#{version}-SNAPSHOT/#{artifact_coordinate}-#{full_version}-#{artifact_name}"
+    final_url = "https://central.sonatype.com/repository/maven-snapshots/#{namespace}/#{artifact_coordinate}/#{version}/#{artifact_coordinate}-#{full_version}-#{artifact_name}"
 
     return final_url
   else
diff --git a/private/helloworld/android/app/build.gradle b/private/helloworld/android/app/build.gradle
@@ -115,5 +115,5 @@ android {
 dependencies {
     // The version of react-native is set by the React Native Gradle Plugin
     implementation("com.facebook.react:react-android")
-    implementation("com.facebook.react:hermes-android")
+    implementation("com.facebook.hermes:hermes-android")
 }
diff --git a/private/helloworld/android/gradle.properties b/private/helloworld/android/gradle.properties
@@ -11,3 +11,4 @@ reactNativeArchitectures=arm64-v8a
 newArchEnabled=true
 hermesEnabled=true
 edgeToEdgeEnabled=false
+react.internal.useHermesNightly=true

Original file line number	Diff line number	Diff line change
`@@ -115,5 +115,5 @@ android {`
`115`	`115`	`dependencies {`
`116`	`116`	`// The version of react-native is set by the React Native Gradle Plugin`
`117`	`117`	`implementation("com.facebook.react:react-android")`
`118`		`- implementation("com.facebook.react:hermes-android")`
	`118`	`+ implementation("com.facebook.hermes:hermes-android")`
`119`	`119`	`}`