match up script

Author: mtopo27

docs/platforms/apple/guides/ios/size-analysis/insights.mdx

@@ -4,8 +4,6 @@ sidebar_order: 5250
 description: See how Size Analysis surfaces trends for your iOS builds.
 ---
 
-TODO: ADD SOME PICS
-
 Size Analysis Insights point out how opportunities to reduce your iOS app's size. They spot patterns like duplicate files, oversized media, or unneeded assets, and list exactly what to fix along with the estimated size savings.
 
 ## iOS Insights
@@ -40,7 +38,7 @@ Below are a list of available insights for iOS builds, followed by more details
   has them) before stripping.
 </Alert>
 
-In Xcode, ensure your release configuration sets the `Debug Information Format` build setting to `DWARF with dSYM`.
+In Xcode ensure your release configuration sets the `Debug Information Format` build setting to `DWARF with dSYM`.
 
 The `strip` command can then remove the extra information:
 
@@ -60,21 +58,61 @@ To automate stripping in your project after building, add a Run Script phase tha
 #!/bin/bash
 set -e
 
-if [ "Release" != "${CONFIGURATION}" ]; then
-  echo "Skipping symbol stripping for ${CONFIGURATION} build."
-  exit 0
+echo "Starting the symbol stripping process..."
+
+if [ "Release" = "${CONFIGURATION}" ]; then
+  echo "Configuration is Release."
+
+  # Path to the app directory
+  APP_DIR_PATH="${BUILT_PRODUCTS_DIR}/${EXECUTABLE_FOLDER_PATH}"
+  echo "App directory path: ${APP_DIR_PATH}"
+
+  # Strip main binary
+  echo "Stripping main binary: ${APP_DIR_PATH}/${EXECUTABLE_NAME}"
+  strip -rSTx "${APP_DIR_PATH}/${EXECUTABLE_NAME}"
+  if [ $? -eq 0 ]; then
+    echo "Successfully stripped main binary."
+  else
+    echo "Failed to strip main binary." >&2
+  fi
+
+  # Path to the Frameworks directory
+  APP_FRAMEWORKS_DIR="${APP_DIR_PATH}/Frameworks"
+  echo "Frameworks directory path: ${APP_FRAMEWORKS_DIR}"
+
+  # Strip symbols from frameworks, if Frameworks/ exists at all
+  # ... as long as the framework is NOT signed by Apple
+  if [ -d "${APP_FRAMEWORKS_DIR}" ]; then
+    echo "Frameworks directory exists. Proceeding to strip symbols from frameworks."
+    find "${APP_FRAMEWORKS_DIR}" -type f -perm +111 -maxdepth 2 -mindepth 2 -exec bash -c '
+    codesign -v -R="anchor apple" "{}" &> /dev/null ||
+    (
+        echo "Stripping {}" &&
+        if [ -w "{}" ]; then
+            strip -rSTx "{}"
+            if [ $? -eq 0 ]; then
+                echo "Successfully stripped {}"
+            else
+                echo "Failed to strip {}" >&2
+            fi
+        else
+            echo "Warning: No write permission for {}"
+        fi
+    )
+    ' \;
+    if [ $? -eq 0 ]; then
+        echo "Successfully stripped symbols from frameworks."
+    else
+        echo "Failed to strip symbols from some frameworks." >&2
+    fi
+  else
+    echo "Frameworks directory does not exist. Skipping framework stripping."
+  fi
+else
+  echo "Configuration is not Release. Skipping symbol stripping."
 fi
 
-APP_DIR_PATH="${BUILT_PRODUCTS_DIR}/${EXECUTABLE_FOLDER_PATH}"
-echo "Stripping main binary: ${APP_DIR_PATH}/${EXECUTABLE_NAME}"
-strip -rSTx "${APP_DIR_PATH}/${EXECUTABLE_NAME}"
-
-APP_FRAMEWORKS_DIR="${APP_DIR_PATH}/Frameworks"
-if [ -d "${APP_FRAMEWORKS_DIR}" ]; then
-  find "${APP_FRAMEWORKS_DIR}" -maxdepth 2 -mindepth 2 -type f -perm -111 -exec bash -c '
-    codesign -v -R="anchor apple" "$1" &> /dev/null || strip -rSTx "$1"
-  ' _ {} \;
-fi
+echo "Symbol stripping process completed."
 ```
 
 Because Xcode generates dSYMs from the unstripped binary, list the dSYM as an Input File so the script runs after Xcode finishes generating it:
@@ -192,7 +230,7 @@ optimize_icon() {
 
 ### Loose Images
 
-**What it is**: Finds scaled @1x/@2x/@3x images stored outside of asset catalogs. App thinning will not remove these images, even when they are not needed for the user's device.
+**What it is**: Finds scaled @1x/@2x/@3x images stored outside of asset catalogs. This prevents app thinning from removing variants not needed for the user's device.
 
 **How to fix**: Move each image set into an asset catalog in Xcode.
 
@@ -226,11 +264,11 @@ optimize_icon() {
 
 **How to fix**: Start with low-effort cleanup, then automate comment stripping if you still ship large payloads.
 
-#### Option 1: Keep the Format Lean
+#### Option 1: Keep the format lean
 
 - Encode localized strings as plain text (`"key" = "value";`), not binary plists. Set **Strings File Output Encoding** (`STRINGS_FILE_OUTPUT_ENCODING`) to **UTF-8** in Xcode.
 
-#### Option 2: Strip Comments Automatically
+#### Option 2: Strip comments automatically
 
 String files can have comments that ship with the bundle. They help during translation but take space in production. A typical comment may look like: