fix: coderabbit suggestions

Author: rameziophobia

.github/workflows/add-comment-on-pr-creation.yml

@@ -8,7 +8,8 @@ jobs:
   comment-on-pr:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/github-script@v6
+      - name: Add GitHub Comment for review app instructions
+        uses: actions/github-script@v6
         with:
           script: |
             github.rest.issues.createComment({

.github/workflows/deploy-to-control-plane-review.yml

@@ -33,7 +33,7 @@ jobs:
       - name: Get PR HEAD Ref
         if: ${{ github.event_name == 'issue_comment' }}
         id: getRef
-        run: echo "PR_REF=$(gh pr view $PR_NUMBER --repo ${{ github.repository }} --json headRefName | jq -r '.headRefName')" >> $GITHUB_OUTPUT
+        run: echo "PR_REF=$(gh pr view \"$PR_NUMBER\" --repo \"${{ github.repository }}\" --json headRefName | jq -r '.headRefName')" >> $GITHUB_OUTPUT
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
@@ -45,7 +45,7 @@ jobs:
 
       - name: Add GitHub Comment
         if: ${{ github.event_name == 'issue_comment' }}
-        uses: actions/github-script@v4
+        uses: actions/github-script@v6
         with:
           script: |
             github.rest.issues.createComment({
@@ -61,26 +61,24 @@ jobs:
           echo "GITHUB_REPOSITORY: \"$GITHUB_REPOSITORY\""
           if [ -z "$PR_NUMBER" ]; then
             echo "PR_NUMBER is not in the trigger event. Fetching PR number from open PRs."
-            REF=${{ github.ref }}
+            REF="${{ github.ref }}"
             REF=${REF#refs/heads/}  # Remove 'refs/heads/' prefix
             echo "REF: \"$REF\""
-            API_RESPONSE=$(curl --location --request GET "https://api.github.com/repos/$GITHUB_REPOSITORY/pulls?state=open" \
+            API_RESPONSE=$(curl --location --request GET "https://api.github.com/repos/${GITHUB_REPOSITORY}/pulls?state=open" \
             --header 'Authorization: Bearer ${{ secrets.GITHUB_TOKEN }}')
-            PR_NUMBER=$(echo $API_RESPONSE | jq '.[] | select(.head.ref=="'$REF'") | .number')
+            PR_NUMBER=$(echo "$API_RESPONSE" | jq '.[] | select(.head.ref=="'$REF'") | .number')
           fi
           echo "PR_NUMBER: $PR_NUMBER"
           if [ -z "$PR_NUMBER" ]; then
             echo "PR_NUMBER is not set. Aborting."
             exit 1
           fi
-          echo "PR_NUMBER=$PR_NUMBER" >> $GITHUB_ENV
-
+          echo "PR_NUMBER=\"$PR_NUMBER\"" >> $GITHUB_ENV
       - name: Get App Name
         run: |
           echo "PR_NUMBER: ${{ env.PR_NUMBER }}"
-          echo "APP_NAME=qa-react-webpack-rails-tutorial-pr-${{ env.PR_NUMBER }}" >> $GITHUB_ENV
+          echo "APP_NAME=qa-react-webpack-rails-tutorial-pr-${{ env.PR_NUMBER }}" >> "$GITHUB_ENV"
           echo "App Name: ${{ env.APP_NAME }}"
-
       - uses: ./.github/actions/deploy-to-control-plane
         with:
           app_name: ${{ env.APP_NAME }}