fix: add CLI subcommands and simplify get_architecture

Address review feedback:
- Add build_hermesc and prepare_dest_root CLI subcommands so CI
  can execute the script directly instead of sourcing it
- Simplify get_architecture from 5 branches to 2
- Update usage comment to document new subcommands
- Remove redundant chmod +x calls (script is already 755 in git)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: Saadnajmi

.github/workflows/build-apple-slices-hermes.yml

@@ -47,7 +47,6 @@ jobs:
 
           echo "[HERMES] Building Hermes for $SLICE"
 
-          chmod +x ./utils/build-apple-framework.sh
           BUILD_TYPE="${{ matrix.flavor }}" ./utils/build-apple-framework.sh "$SLICE"
 
           echo "Moving from build_$SLICE to $FINAL_PATH"

.github/workflows/build-hermes-macos.yml

@@ -53,17 +53,12 @@ jobs:
           mv build_xrsimulator_${{ matrix.flavor }} build_xrsimulator
       - name: Prepare destroot folder
         shell: bash
-        run: |
-          chmod +x ./utils/build-apple-framework.sh
-          source ./utils/build-apple-framework.sh
-          prepare_dest_root_for_ci
+        run: ./utils/build-apple-framework.sh prepare_dest_root
       - name: Create universal xcframework
         shell: bash
         run: |
           echo "[HERMES] Creating the universal framework"
-          chmod +x ./utils/build-apple-framework.sh
           ./utils/build-apple-framework.sh build_framework
-
           chmod +x ./destroot/bin/hermesc
       - name: Package the Hermes Apple frameworks
         shell: bash

.github/workflows/build-hermesc-apple.yml

@@ -12,9 +12,7 @@ jobs:
         uses: ./.github/actions/setup-xcode
       - name: Build HermesC Apple
         shell: bash
-        run: |
-          source ./utils/build-apple-framework.sh
-          build_host_hermesc_if_needed
+        run: ./utils/build-apple-framework.sh build_hermesc
       - name: Upload HermesC Artifact
         uses: actions/upload-artifact@v4.3.4
         with:

utils/build-apple-framework.sh

@@ -8,9 +8,11 @@
 # Mac Catalyst) and combines them into a single universal xcframework.
 #
 # Usage:
-#   ./build-apple-framework.sh              # build all platforms + universal xcframework
-#   ./build-apple-framework.sh <platform>   # build a single platform (e.g. macosx, iphoneos)
-#   ./build-apple-framework.sh build_framework  # combine already-built slices into xcframework
+#   ./build-apple-framework.sh                   # build all platforms + universal xcframework
+#   ./build-apple-framework.sh <platform>        # build a single platform (e.g. macosx, iphoneos)
+#   ./build-apple-framework.sh build_framework   # combine already-built slices into xcframework
+#   ./build-apple-framework.sh build_hermesc     # build host hermesc compiler
+#   ./build-apple-framework.sh prepare_dest_root # assemble destroot from pre-built slices
 
 if [ "$CI" ]; then
   set -x
@@ -75,15 +77,9 @@ function get_release_version {
 # Given a specific target, retrieve the right architecture for it
 # $1 the target you want to build. Allowed values: iphoneos, iphonesimulator, catalyst, macosx, xros, xrsimulator, appletvos, appletvsimulator
 function get_architecture {
-    if [[ $1 == "iphoneos" || $1 == "xros" ]]; then
+    if [[ $1 == "iphoneos" || $1 == "xros" || $1 == "appletvos" ]]; then
       echo "arm64"
-    elif [[ $1 == "iphonesimulator" || $1 == "xrsimulator" ]]; then
-      echo "x86_64;arm64"
-    elif [[ $1 == "appletvos" ]]; then
-      echo "arm64"
-    elif [[ $1 == "appletvsimulator" ]]; then
-      echo "x86_64;arm64"
-    elif [[ $1 == "catalyst" || $1 == "macosx" ]]; then
+    elif [[ $1 == "iphonesimulator" || $1 == "xrsimulator" || $1 == "appletvsimulator" || $1 == "catalyst" || $1 == "macosx" ]]; then
       echo "x86_64;arm64"
     else
       echo "Error: unknown architecture passed $1"
@@ -318,6 +314,10 @@ if [[ -z $1 ]]; then
   create_framework
 elif [[ $1 == "build_framework" ]]; then
   build_universal_framework
+elif [[ $1 == "build_hermesc" ]]; then
+  build_host_hermesc_if_needed
+elif [[ $1 == "prepare_dest_root" ]]; then
+  prepare_dest_root_for_ci
 else
   build_framework "$1"
 fi