Remove dual Hermes version outputs from CI workflows (#56739)

Summary:
Pull Request resolved: #56739

## Summary
  - Simplify `publish-release.yml`: remove `HERMES_V1_VERSION` output, read only `HERMES_VERSION_NAME`
  - Simplify `create-draft-release.yml`: remove `hermesV1Version` input
  - Simplify `createDraftRelease.js`: remove `hermesV1Version` parameter and "Hermes V1 dSYMS" section from release notes
  - Simplify `prebuild-ios-core.yml`: read `HERMES_VERSION_NAME` instead of `HERMES_V1_VERSION_NAME`
  - Delete unused `prepare-hermes-v1-app` action, `hermes-v1.patch`, and `selectLatestHermesV1Version.js`

## Changelog:
[Internal]

  ## Test plan
  - [x] JS tests: `yarn jest --no-watchman createDraftRelease-test.js` — 9/9 tests pass

Reviewed By: cortinico

Differential Revision: D104381269

Author: cipolleschi

.github/actions/prepare-hermes-v1-app/action.yml

@@ -121,8 +121,12 @@ describe('Create Draft Release', () => {
   });
 
   describe('#_computeBody', () => {
-    it('computes body for release when no hermes versions are passed', async () => {
+    it('falls back to HERMES_VERSION_NAME from version.properties when no hermes version is passed', async () => {
       const version = '0.77.1';
+      const hermesVersion = '250829098.0.13';
+      jest.spyOn(fs, 'readFileSync').mockImplementationOnce(() => {
+        return `HERMES_VERSION_NAME=${hermesVersion}\n`;
+      });
       const changelog = `## v${version}
 ### Breaking Changes
 - [PR #9012](https://github.com/facebook/react-native/pull/9012) - Some other change
@@ -140,13 +144,9 @@ describe('Create Draft Release', () => {
 
 ---
 
-Hermes dSYMS:
-- [Debug](https://repo1.maven.org/maven2/com/facebook/hermes/hermes-ios/${version}/hermes-ios-${version}-hermes-framework-dSYM-debug.tar.gz)
-- [Release](https://repo1.maven.org/maven2/com/facebook/hermes/hermes-ios/${version}/hermes-ios-${version}-hermes-framework-dSYM-release.tar.gz)
-
 Hermes V1 dSYMS:
-- [Debug](https://repo1.maven.org/maven2/com/facebook/hermes/hermes-ios/${version}/hermes-ios-${version}-hermes-framework-dSYM-debug.tar.gz)
-- [Release](https://repo1.maven.org/maven2/com/facebook/hermes/hermes-ios/${version}/hermes-ios-${version}-hermes-framework-dSYM-release.tar.gz)
+- [Debug](https://repo1.maven.org/maven2/com/facebook/hermes/hermes-ios/${hermesVersion}/hermes-ios-${hermesVersion}-hermes-framework-dSYM-debug.tar.gz)
+- [Release](https://repo1.maven.org/maven2/com/facebook/hermes/hermes-ios/${hermesVersion}/hermes-ios-${hermesVersion}-hermes-framework-dSYM-release.tar.gz)
 
 ReactNativeDependencies dSYMs:
 - [Debug](https://repo1.maven.org/maven2/com/facebook/react/react-native-artifacts/${version}/react-native-artifacts-${version}-reactnative-dependencies-dSYM-debug.tar.gz)
@@ -172,7 +172,6 @@ View the whole changelog in the [CHANGELOG.md file](https://github.com/facebook/
     it('computes body for release when hermes versions are passed', async () => {
       const version = '0.77.1';
       const hermesVersion = '0.15.0';
-      const hermesV1Version = '250829098.0.2';
       const changelog = `## v${version}
 ### Breaking Changes
 - [PR #9012](https://github.com/facebook/react-native/pull/9012) - Some other change
@@ -184,25 +183,16 @@ View the whole changelog in the [CHANGELOG.md file](https://github.com/facebook/
 #### iOS
 - [PR #3436](https://github.com/facebook/react-native/pull/3436) - Some other change
 - [PR #3437](https://github.com/facebook/react-native/pull/3437) - Some other change`;
-      const body = _computeBody(
-        changelog,
-        version,
-        hermesVersion,
-        hermesV1Version,
-      );
+      const body = _computeBody(changelog, version, hermesVersion);
 
       expect(body).toEqual(`${changelog}
 
 ---
 
-Hermes dSYMS:
+Hermes V1 dSYMS:
 - [Debug](https://repo1.maven.org/maven2/com/facebook/hermes/hermes-ios/${hermesVersion}/hermes-ios-${hermesVersion}-hermes-framework-dSYM-debug.tar.gz)
 - [Release](https://repo1.maven.org/maven2/com/facebook/hermes/hermes-ios/${hermesVersion}/hermes-ios-${hermesVersion}-hermes-framework-dSYM-release.tar.gz)
 
-Hermes V1 dSYMS:
-- [Debug](https://repo1.maven.org/maven2/com/facebook/hermes/hermes-ios/${hermesV1Version}/hermes-ios-${hermesV1Version}-hermes-framework-dSYM-debug.tar.gz)
-- [Release](https://repo1.maven.org/maven2/com/facebook/hermes/hermes-ios/${hermesV1Version}/hermes-ios-${hermesV1Version}-hermes-framework-dSYM-release.tar.gz)
-
 ReactNativeDependencies dSYMs:
 - [Debug](https://repo1.maven.org/maven2/com/facebook/react/react-native-artifacts/${version}/react-native-artifacts-${version}-reactnative-dependencies-dSYM-debug.tar.gz)
 - [Release](https://repo1.maven.org/maven2/com/facebook/react/react-native-artifacts/${version}/react-native-artifacts-${version}-reactnative-dependencies-dSYM-release.tar.gz)

.github/workflow-scripts/__tests__/createDraftRelease-test.js

@@ -40,21 +40,33 @@ function _extractChangelog(version) {
   return changelog.slice(changelogStarts, changelogEnds).join('\n').trim();
 }
 
-function _computeBody(changelog, version, hermesVersion, hermesV1Version) {
-  hermesVersion = hermesVersion ?? version;
-  hermesV1Version = hermesV1Version ?? version;
+function _readHermesVersionFromProperties() {
+  const propertiesPath =
+    'packages/react-native/sdks/hermes-engine/version.properties';
+  const content = String(fs.readFileSync(propertiesPath, 'utf8'));
+  const match = content.match(/^HERMES_VERSION_NAME=(.+)$/m);
+  if (!match) {
+    throw new Error(`HERMES_VERSION_NAME not found in ${propertiesPath}`);
+  }
+  return match[1].trim();
+}
+
+function _computeBody(changelog, version, hermesVersion) {
+  // The workflow input is optional and arrives as an empty string when unset,
+  // so treat both missing and empty as "fall back to version.properties".
+  // Falling back to ${version} would produce 404 dSYMS URLs since Hermes V1
+  // versions (e.g. 250829098.0.13) don't track RN versions.
+  if (hermesVersion == null || hermesVersion === '') {
+    hermesVersion = _readHermesVersionFromProperties();
+  }
   return `${changelog}
 
 ---
 
-Hermes dSYMS:
+Hermes V1 dSYMS:
 - [Debug](https://repo1.maven.org/maven2/com/facebook/hermes/hermes-ios/${hermesVersion}/hermes-ios-${hermesVersion}-hermes-framework-dSYM-debug.tar.gz)
 - [Release](https://repo1.maven.org/maven2/com/facebook/hermes/hermes-ios/${hermesVersion}/hermes-ios-${hermesVersion}-hermes-framework-dSYM-release.tar.gz)
 
-Hermes V1 dSYMS:
-- [Debug](https://repo1.maven.org/maven2/com/facebook/hermes/hermes-ios/${hermesV1Version}/hermes-ios-${hermesV1Version}-hermes-framework-dSYM-debug.tar.gz)
-- [Release](https://repo1.maven.org/maven2/com/facebook/hermes/hermes-ios/${hermesV1Version}/hermes-ios-${hermesV1Version}-hermes-framework-dSYM-release.tar.gz)
-
 ReactNativeDependencies dSYMs:
 - [Debug](https://repo1.maven.org/maven2/com/facebook/react/react-native-artifacts/${version}/react-native-artifacts-${version}-reactnative-dependencies-dSYM-debug.tar.gz)
 - [Release](https://repo1.maven.org/maven2/com/facebook/react/react-native-artifacts/${version}/react-native-artifacts-${version}-reactnative-dependencies-dSYM-release.tar.gz)
@@ -123,21 +135,15 @@ function moveToChangelogBranch(version) {
   run(`git checkout -b changelog/v${version}`);
 }
 
-async function createDraftRelease(
-  version,
-  latest,
-  token,
-  hermesVersion,
-  hermesV1Version,
-) {
+async function createDraftRelease(version, latest, token, hermesVersion) {
   if (version.startsWith('v')) {
     version = version.substring(1);
   }
 
   _verifyTagExists(version);
   moveToChangelogBranch(version);
   const changelog = _extractChangelog(version);
-  const body = _computeBody(changelog, version, hermesVersion, hermesV1Version);
+  const body = _computeBody(changelog, version, hermesVersion);
   const release = await _createDraftReleaseOnGitHub(
     version,
     body,
@@ -155,4 +161,5 @@ module.exports = {
   _extractChangelog,
   _computeBody,
   _createDraftReleaseOnGitHub,
+  _readHermesVersionFromProperties,
 };

.github/workflow-scripts/createDraftRelease.js

@@ -6,11 +6,7 @@ on:
       hermesVersion:
         required: false
         type: string
-        description: The version of Hermes to use for this release (eg. 0.15.0). If not specified, it will use React Native Version
-      hermesV1Version:
-        required: false
-        type: string
-        description: The version of Hermes V1 to use for this release (eg. 250829098.0.2). If not specified, it will use React Native Version
+        description: The version of Hermes to use for this release (eg. 250829098.0.2). If not specified, it will read HERMES_VERSION_NAME from version.properties
 
 jobs:
   create-draft-release:
@@ -37,7 +33,7 @@ jobs:
             const {createDraftRelease} = require('./.github/workflow-scripts/createDraftRelease.js');
             const version = '${{ github.ref_name }}';
             const {isLatest} = require('./.github/workflow-scripts/publishTemplate.js');
-            return (await createDraftRelease(version, isLatest(), '${{secrets.REACT_NATIVE_BOT_GITHUB_TOKEN}}', '${{ inputs.hermesVersion }}', '${{ inputs.hermesV1Version }}')).id;
+            return (await createDraftRelease(version, isLatest(), '${{secrets.REACT_NATIVE_BOT_GITHUB_TOKEN}}', '${{ inputs.hermesVersion }}')).id;
           result-encoding: string
       - name: Upload release assets for DotSlash
         uses: actions/github-script@v8

.github/workflow-scripts/hermes-v1.patch

@@ -47,6 +47,9 @@ jobs:
       - name: Set Hermes version
         shell: bash
         run: |
+          # Nightly RN builds resolve Hermes from npm's latest-v1 dist-tag (TODO:
+          # rename to 'latest' once V1 is the only Hermes on npm). Stable builds
+          # use the version pinned in version.properties.
           if [ "${{ inputs.use-hermes-nightly }}" == "true" ]; then
             HERMES_VERSION="latest-v1"
           else

.github/workflow-scripts/selectLatestHermesV1Version.js

@@ -19,26 +19,19 @@ jobs:
           echo "Setting release type to release"
           echo "RELEASE_TYPE=release" >> $GITHUB_OUTPUT
 
-  set_hermes_versions:
+  set_hermes_version:
     runs-on: ubuntu-latest
     if: github.repository == 'facebook/react-native'
     outputs:
-      HERMES_VERSION: ${{ steps.set_hermes_versions.outputs.HERMES_VERSION }}
-      HERMES_V1_VERSION: ${{ steps.set_hermes_versions.outputs.HERMES_V1_VERSION }}
+      HERMES_VERSION: ${{ steps.set_hermes_version.outputs.HERMES_VERSION }}
     steps:
       - name: Checkout
         uses: actions/checkout@v6
-      - id: set_hermes_versions
+      - id: set_hermes_version
         run: |
-          echo "Setting hermes versions to latest"
           hermes_version=$(grep -oE 'HERMES_VERSION_NAME=([0-9]+\.[0-9]+\.[0-9]+)' packages/react-native/sdks/hermes-engine/version.properties | cut -d'=' -f2)
-
           echo "HERMES_VERSION=$hermes_version" >> $GITHUB_OUTPUT
-          echo "HERMES_V1_VERSION=$hermes_version" >> $GITHUB_OUTPUT
-      - name: Print hermes versions
-        run: |
-          echo "HERMES_VERSION=${{ steps.set_hermes_versions.outputs.HERMES_VERSION }}"
-          echo "HERMES_V1_VERSION=${{ steps.set_hermes_versions.outputs.HERMES_V1_VERSION }}"
+          echo "HERMES_VERSION=$hermes_version"
 
   prebuild_apple_dependencies:
     if: github.repository == 'facebook/react-native'
@@ -142,9 +135,8 @@ jobs:
     secrets: inherit
 
   create_draft_release:
-    needs: [generate_changelog, set_hermes_versions]
+    needs: [generate_changelog, set_hermes_version]
     uses: ./.github/workflows/create-draft-release.yml
     secrets: inherit
     with:
-      hermesVersion: ${{ needs.set_hermes_versions.outputs.HERMES_VERSION }}
-      hermesV1Version: ${{ needs.set_hermes_versions.outputs.HERMES_V1_VERSION }}
+      hermesVersion: ${{ needs.set_hermes_version.outputs.HERMES_VERSION }}