Remove test code and restore production daily CI workflow

- Remove temporary test-failure job
- Remove push trigger (keep only schedule and manual)
- Restore simple failure detection logic
- Workflow is now ready for production use

Author: dhalperi

.github/workflows/daily-ci.yml

@@ -4,28 +4,16 @@ on:
   schedule:
     - cron: "0 15 * * *" # 8AM Pacific; daily
   workflow_dispatch: # Allow manual triggering
-  push:
-    branches: [ add-daily-ci-workflow ] # Temporary for testing
 
 jobs:
-  # Temporary test job to simulate failure (remove after testing)
-  test-failure:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Simulate failure for testing
-        run: |
-          echo "Testing Slack notification on failure"
-          exit 1
-
-  # Run the existing CI workflow (will be skipped due to test-failure)
+  # Run the existing CI workflow
   ci:
     uses: ./.github/workflows/ci.yml
-    needs: test-failure
 
   # Send Slack notification on failure
   notify-slack-on-failure:
-    needs: [test-failure, ci]
-    if: always() && (needs.test-failure.result == 'failure' || needs.ci.result == 'failure' || needs.ci.result == 'skipped')
+    needs: ci
+    if: failure()
     runs-on: ubuntu-latest
     steps:
       - name: Send Slack notification on failure