Address code review feedback: add comments and simplify deploy logic

Co-authored-by: lippytm <65956507+lippytm@users.noreply.github.com>

Author: Copilot

.github/workflows/deploy.yml

@@ -91,11 +91,23 @@ jobs:
           # S3_BUCKET: ${{ vars.S3_BUCKET }}
           echo "Environment variables configured"
 
+      - name: Determine package manager
+        id: package-manager
+        run: |
+          if [ -f "yarn.lock" ]; then
+            echo "manager=yarn" >> $GITHUB_OUTPUT
+          elif [ -f "package-lock.json" ]; then
+            echo "manager=npm" >> $GITHUB_OUTPUT
+          else
+            echo "manager=" >> $GITHUB_OUTPUT
+          fi
+
       - name: Setup Node.js
+        if: steps.package-manager.outputs.manager != ''
         uses: actions/setup-node@v4
         with:
           node-version: '20'
-          cache: ${{ hashFiles('yarn.lock') != '' && 'yarn' || (hashFiles('package-lock.json') != '' && 'npm' || '') }}
+          cache: ${{ steps.package-manager.outputs.manager }}
 
       - name: Install dependencies
         run: |
@@ -141,17 +153,3 @@ jobs:
           # - Discord webhook: similar to Slack
           # - Email notification
           # - etc.
-
-  # Separate job for each environment when triggered by push
-  deploy-on-push:
-    if: github.event_name == 'push'
-    runs-on: ubuntu-latest
-    environment: dev
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v4
-
-      - name: Deploy to dev environment
-        run: |
-          echo "Auto-deploying to dev on push to ${{ github.ref_name }}"
-          echo "Add your deployment logic here"

.github/workflows/lint-test.yml

@@ -112,6 +112,8 @@ jobs:
           flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
 
       - name: Check formatting with black
+        # Note: continue-on-error allows the workflow to pass even with formatting issues
+        # Remove this flag to enforce strict formatting standards
         run: |
           black --check .
         continue-on-error: true