fix: resolve deployment issues with nx command and force build flag

- Use 'pnpm nx' instead of 'nx' to ensure command availability
- Fix force build flag handling for empty string inputs
- Add better error handling and debugging output

Author: jonphipps

.github/workflows/deploy-dev.yml

@@ -72,9 +72,11 @@ jobs:
       - name: Detect affected projects
         id: affected
         run: |
-          if [ "${{ inputs.force_build_all }}" = "true" ]; then
+          # Check for force build flag (handle empty string as false)
+          FORCE_BUILD="${{ inputs.force_build_all }}"
+          if [ "$FORCE_BUILD" = "true" ]; then
             echo "Force building all projects..."
-            projects=$(nx show projects --type=app | tr '\n' ' ')
+            projects=$(pnpm nx show projects --type=app | tr '\n' ' ')
             echo "projects=$projects" >> $GITHUB_OUTPUT
             echo "has-affected=true" >> $GITHUB_OUTPUT
           else
@@ -84,24 +86,24 @@ jobs:
             
             # Show what would be affected (including libraries for debugging)
             echo "All affected projects:"
-            nx show projects --affected || echo "Failed to get affected projects"
+            pnpm nx show projects --affected || echo "Failed to get affected projects"
             
             # Get affected apps for deployment
-            affected=$(nx show projects --affected --type=app 2>/dev/null | tr '\n' ' ' || echo "")
+            affected=$(pnpm nx show projects --affected --type=app 2>/dev/null | tr '\n' ' ' || echo "")
             if [ -z "$affected" ]; then
               echo "No affected app projects found"
               echo "Trying with explicit base SHA..."
               if [ -n "$NX_BASE" ]; then
-                affected=$(nx show projects --affected --type=app --base=$NX_BASE 2>/dev/null | tr '\n' ' ' || echo "")
+                affected=$(pnpm nx show projects --affected --type=app --base=$NX_BASE 2>/dev/null | tr '\n' ' ' || echo "")
               fi
               
               # Special handling for theme changes - if theme is affected, all apps should rebuild
               if [ -z "$affected" ]; then
                 echo "Checking if theme package is affected..."
-                theme_affected=$(nx show projects --affected | grep "@ifla/theme" || echo "")
+                theme_affected=$(pnpm nx show projects --affected | grep "@ifla/theme" || echo "")
                 if [ -n "$theme_affected" ]; then
                   echo "Theme package is affected - rebuilding all sites"
-                  affected=$(nx show projects --type=app | tr '\n' ' ')
+                  affected=$(pnpm nx show projects --type=app | tr '\n' ' ')
                   echo "All projects due to theme change: $affected"
                   echo "projects=$affected" >> $GITHUB_OUTPUT
                   echo "has-affected=true" >> $GITHUB_OUTPUT
@@ -124,10 +126,13 @@ jobs:
       - name: Build affected projects
         if: steps.affected.outputs.has-affected == 'true'
         run: |
-          if [ "${{ inputs.force_build_all }}" = "true" ]; then
-            nx run-many --target=build --all --parallel=1 --configuration=development
+          FORCE_BUILD="${{ inputs.force_build_all }}"
+          if [ "$FORCE_BUILD" = "true" ]; then
+            echo "Force building all projects"
+            pnpm nx run-many --target=build --all --parallel=1 --configuration=development
           else
-            nx affected --target=build --parallel=1 --configuration=development
+            echo "Building affected projects"
+            pnpm nx affected --target=build --parallel=1 --configuration=development
           fi
         env:
           DOCS_ENV: development