Address review comments

Author: yufengwangca

docs/guides/fabric_synchronization_guide.md

@@ -6,8 +6,24 @@
 
 ## Fabric Sync Example Applications
 
-Fabric-Admin and Fabric-Bridge applications are provided to demonstrate Fabric
-Synchronization feature. You can find them in the examples.
+Fabric-Admin and Fabric-Bridge example applications are provided to demonstrate
+Fabric Synchronization feature. You can find them in the examples.
+
+![matter_fabric_synchronization](images/matter_fabric_synchronization.png)
+
+Fabric-Admin example app implements the Fabric Administrator role and
+communicate with the Fabric-Bridge-App on the other side, facilitating the
+Fabric Synchronization process.
+
+Fabric-Bridge-App example app implements the Aggregator device type with Fabric
+Synchronization condition met and demonstrates the end-to-end Fabric
+Synchronization feature using dynamic endpoints.
+
+Fabric Synchronization can be triggered from either side. The initiator of the
+Fabric Synchronization process, who shares their devices, takes on the
+Commissioner role. The recipient of the Fabric Synchronization request, who
+receives the shared devices, assumes the Commissionee role. This flexibility
+enables a seamless and efficient synchronization process.
 
 ### Building the Example Application
 

docs/guides/images/matter_fabric_synchronization.png

@@ -0,0 +1,41 @@
+#!/bin/bash
+
+# Default paths
+FABRIC_ADMIN_PATH="./fabric-admin"
+FABRIC_ADMIN_LOG="/tmp/fabric_admin.log"
+
+# Detect platform and set default paths
+if [[ $(uname -m) == "x86_64" ]]; then
+    FABRIC_ADMIN_PATH="out/debug/standalone/fabric-admin"
+fi
+
+# Check if --verbose option is specified
+VERBOSE=false
+for arg in "$@"; do
+    if [ "$arg" == "--verbose" ]; then
+        VERBOSE=true
+        break
+    fi
+done
+
+# Kill fabric-admin if it is running
+echo "Checking for running fabric-admin process..."
+fabric_admin_pid=$(pgrep -f "$FABRIC_ADMIN_PATH")
+if [ ! -z "$fabric_admin_pid" ]; then
+    echo "Found fabric-admin with PID $fabric_admin_pid, attempting to kill..."
+    kill -9 "$fabric_admin_pid"
+    echo "Killed fabric-admin with PID $fabric_admin_pid"
+fi
+
+# Remove /tmp/chip_* files and directories
+echo "Removing /tmp/chip_* files and directories..."
+sudo rm -rf /tmp/chip_*
+echo "Removed /tmp/chip_* files and directories"
+
+# Start fabric-admin with or without log file path based on --verbose option
+echo "Starting fabric-admin..."
+if [ "$VERBOSE" = true ]; then
+    "$FABRIC_ADMIN_PATH"
+else
+    "$FABRIC_ADMIN_PATH" --log-file-path "$FABRIC_ADMIN_LOG"
+fi

examples/fabric-admin/scripts/run_fabric_sink.sh

@@ -0,0 +1,43 @@
+#!/bin/bash
+
+# Default paths
+FABRIC_ADMIN_PATH="./fabric-admin"
+FABRIC_BRIDGE_APP_PATH="./fabric-bridge-app"
+FABRIC_ADMIN_LOG="/tmp/fabric_admin.log"
+FABRIC_BRIDGE_APP_LOG="/tmp/fabric_bridge_app.log"
+BLE_ADAPTER=""
+
+# Detect platform and set default paths
+PLATFORM=$(uname -m)
+if [[ "$PLATFORM" == "x86_64" ]]; then
+    FABRIC_ADMIN_PATH="out/debug/standalone/fabric-admin"
+    FABRIC_BRIDGE_APP_PATH="out/debug/standalone/fabric-bridge-app"
+elif [[ "$PLATFORM" == "armv7l" ]]; then
+    # Assuming armv7l is RP4
+    BLE_ADAPTER="--ble-adapter 0"
+fi
+
+# Check if --verbose option is specified
+VERBOSE=false
+for arg in "$@"; do
+    if [ "$arg" == "--verbose" ]; then
+        VERBOSE=true
+        break
+    fi
+done
+
+# Stop any running instances and clean up
+./stop_fabric_source.sh
+
+# Start fabric-bridge-app if available and redirect its output to /dev/null
+if [ -f "$FABRIC_BRIDGE_APP_PATH" ]; then
+    "$FABRIC_BRIDGE_APP_PATH" >"$FABRIC_BRIDGE_APP_LOG" 2>&1 &
+    echo "Started fabric-bridge-app"
+fi
+
+# Start fabric-admin with or without log file path based on --verbose option
+if [ "$VERBOSE" = true ]; then
+    "$FABRIC_ADMIN_PATH" $BLE_ADAPTER
+else
+    "$FABRIC_ADMIN_PATH" $BLE_ADAPTER --log-file-path "$FABRIC_ADMIN_LOG"
+fi