Merge pull request #40 from nova-rey/codex/rewrite-workflows-to-warn-only-mode

Author: nova-rey

.github/workflows/ci-diagnose.yml

@@ -1,49 +1,46 @@
 name: CI Diagnose
-on:
-  workflow_dispatch: {}
-  push:
-    branches: [ ci-debug/** ]
-permissions:
-  contents: read
+on: { workflow_dispatch: {}, push: { branches: [ ci-debug/** ] }, pull_request: {} }
+permissions: { contents: read }
 
 jobs:
   diagnose:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
+
       - name: Dump GitHub context
         run: |
           echo "event: ${{ github.event_name }}"
           echo "ref:   ${{ github.ref }}"
           echo "sha:   ${{ github.sha }}"
+
       - uses: actions/setup-python@v5
         with: { python-version: '3.12' }
+
       - name: Python env
         run: |
           python --version
           which python
+
       - name: Tree + key files
         run: |
           ls -la
-          echo "--- pyproject.toml ---"; [ -f pyproject.toml ] && sed -n '1,120p' pyproject.toml || echo "missing"
-          echo "--- setup.cfg ---"; [ -f setup.cfg ] && sed -n '1,120p' setup.cfg || echo "missing"
+          echo "--- pyproject.toml ---"; [ -f pyproject.toml ] && sed -n '1,200p' pyproject.toml || echo "missing"
+          echo "--- setup.cfg ---"; [ -f setup.cfg ] && sed -n '1,200p' setup.cfg || echo "missing"
           echo "--- requirements*.txt ---"; ls -1 requirements*.txt 2>/dev/null || echo "none"
+
       - name: Test discovery
         run: |
-          echo "tests tree:"
           find tests -maxdepth 2 -type f -name "*.py" 2>/dev/null || echo "no tests dir"
+
       - name: Import sanity
         env: { PYTHONPATH: ${{ github.workspace }} }
         run: |
-          python - <<'PY'
-try:
-  import crapssim as m
-  print("import crapssim OK:", getattr(m,'__file__',None))
-except Exception as e:
-  print("import crapssim FAILED:", e)
-try:
-  import crapssim_api as m
-  print("import crapssim_api OK:", getattr(m,'__file__',None))
-except Exception as e:
-  print("import crapssim_api FAILED:", e)
+          python - <<'PY' || true
+mods = ["crapssim","crapssim_api"]
+for m in mods:
+    try:
+        __import__(m); print("OK import:", m)
+    except Exception as e:
+        print("FAIL import:", m, "->", e)
 PY

.github/workflows/coverage.yml

@@ -25,28 +25,20 @@ jobs:
           [ -d "crapssim_api" ] && COV_MODS="$COV_MODS --cov=crapssim_api"
           echo "mods=$COV_MODS" >> $GITHUB_OUTPUT
 
-      - name: Run pytest with coverage (quiet)
-        id: run_cov
+      - name: Run pytest with coverage (warn-only)
         env: { PYTHONPATH: ${{ github.workspace }}, PYTHONHASHSEED: '0' }
-        shell: bash
         run: |
           mkdir -p reports
-          set +e
           if ls tests/**/*.py tests/*.py >/dev/null 2>&1; then
-            pytest -q ${{ steps.cov.outputs.mods }} --cov-report=xml:coverage.xml -m "not stress" > reports/coverage_quiet.log 2>&1
-            echo "rc=$?" >> $GITHUB_OUTPUT
+            pytest -q ${{ steps.cov.outputs.mods }} --cov-report=xml:coverage.xml -m "not stress" \
+              > reports/coverage_quiet.log 2>&1 || true
+            if [ ! -f coverage.xml ]; then
+              echo '<coverage version="0"/>' > coverage.xml
+            fi
           else
             echo '<coverage version="0"/>' > coverage.xml
             echo "No tests found." > reports/coverage_quiet.log
-            echo "rc=0" >> $GITHUB_OUTPUT
           fi
-          exit 0
-
-      - name: If failed, rerun first failing test verbosely
-        if: steps.run_cov.outputs.rc != '0'
-        env: { PYTHONPATH: ${{ github.workspace }}, PYTHONHASHSEED: '0' }
-        run: |
-          pytest -vv -x --maxfail=1 --lf -ra --durations=25 | tee reports/coverage_verbose.log
 
       - uses: actions/upload-artifact@v4
         if: always()
@@ -55,4 +47,3 @@ jobs:
           path: |
             coverage.xml
             reports/coverage_quiet.log
-            reports/coverage_verbose.log

.github/workflows/examples-smoke.yml

@@ -1,6 +1,7 @@
 name: Examples Smoke
 on: { push: {}, pull_request: {}, workflow_dispatch: {} }
 permissions: { contents: read }
+
 jobs:
   examples:
     runs-on: ubuntu-latest
@@ -9,12 +10,20 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-python@v5
         with: { python-version: '3.11' }
-      - name: Run examples if present
+
+      - name: Run examples if present (warn-only)
         env: { PYTHONPATH: ${{ github.workspace }} }
-        shell: bash
         run: |
           if [ -f examples/run_examples.py ]; then
-            python examples/run_examples.py
+            mkdir -p reports
+            python examples/run_examples.py 2>&1 | tee reports/examples.log || true
           else
             echo "No examples/run_examples.py; skipping."
           fi
+
+      - uses: actions/upload-artifact@v4
+        if: always()
+        with:
+          name: examples-logs
+          path: reports/examples.log
+          if-no-files-found: ignore

.github/workflows/import-smoke.yml

@@ -11,27 +11,22 @@ jobs:
       - uses: actions/setup-python@v5
         with: { python-version: '3.11' }
 
-      - name: Import modules from source tree
-        id: imp
+      - name: Import modules from source (warn-only)
         env: { PYTHONPATH: ${{ github.workspace }} }
-        shell: bash
         run: |
-          set +e
           mkdir -p reports
-          python - <<'PY' | tee reports/import-smoke.log
-import importlib, sys
+          python - <<'PY' | tee reports/import-smoke.log || true
 mods = [("crapssim","core"), ("crapssim_api","api")]
-failed = False
+bad = []
 for mod, label in mods:
     try:
-        m = importlib.import_module(mod)
-        print(f"OK: import {mod} ->", getattr(m,"__file__",None))
+        m = __import__(mod); print("OK:", mod, "->", getattr(m,"__file__",None))
     except Exception as e:
-        failed = True
-        print(f"FAIL: import {mod}: {e}")
-if failed:
-    sys.exit(2)
+        print("WARN:", mod, "import failed ->", e)
+        bad.append(mod)
+print("Import result:", "OK" if len(bad) < len(mods) else "PARTIAL/MISSING")
 PY
+
       - uses: actions/upload-artifact@v4
         if: always()
         with: