ci: fix smoke workflow JSON parse + enforce pytest (#715)

Fixes:
1. JSON parse: file-by-file loop (no xargs arg overflow)
2. YAML parse: file-by-file with per-file error reporting
3. Python compile: file-by-file with per-file error reporting
4. Shell syntax: file-by-file with per-file error reporting
5. Pytest: runs tests/ directory, FAILS on test failure (removed || true)

Every parse step now reports which file failed, not just "exit 1".

Closes #715.

.gitea/workflows/smoke.yml

@@ -1,5 +1,5 @@
 name: Smoke Test
-'on':
+"on":
   pull_request:
   push:
     branches: [main]
@@ -11,22 +11,40 @@ jobs:
       - uses: actions/setup-python@v5
         with:
           python-version: '3.11'
-      - name: Install parse dependencies
+      - name: Install dependencies
         run: |
-          python3 -m pip install --quiet pyyaml
+          pip install --quiet pyyaml pytest
       - name: Parse check
         run: |
-          find . \( -name '*.yml' -o -name '*.yaml' \) | grep -v .gitea | xargs -r python3 -c "import sys,yaml; [yaml.safe_load(open(f)) for f in sys.argv[1:]]"
-          find . -name '*.json' | while read f; do python3 -m json.tool "$f" > /dev/null || exit 1; done
-          find . -name '*.py' | xargs -r python3 -m py_compile
-          find . -name '*.sh' | xargs -r bash -n
+          # YAML parse
+          find . \( -name '*.yml' -o -name '*.yaml' \) | grep -v .gitea | while read f; do
+            python3 -c "import yaml; yaml.safe_load(open('$f'))" || { echo "FAIL: $f"; exit 1; }
+          done
+          # JSON parse (file-by-file to avoid xargs arg overflow)
+          find . -name '*.json' | grep -v node_modules | while read f; do
+            python3 -m json.tool "$f" > /dev/null || { echo "FAIL: $f"; exit 1; }
+          done
+          # Python compile
+          find . -name '*.py' | while read f; do
+            python3 -m py_compile "$f" || { echo "FAIL: $f"; exit 1; }
+          done
+          # Shell syntax
+          find . -name '*.sh' | while read f; do
+            bash -n "$f" || { echo "FAIL: $f"; exit 1; }
+          done
           echo "PASS: All files parse"
       - name: Secret scan
         run: |
-          if grep -rE 'sk-or-|sk-ant-|ghp_|AKIA' . --include='*.yml' --include='*.py' --include='*.sh' 2>/dev/null | grep -v '.gitea' | grep -v 'detect_secrets' | grep -v 'test_trajectory_sanitize'; then exit 1; fi
+          if grep -rE 'sk-or-|sk-ant-|ghp_|AKIA' . --include='*.yml' --include='*.py' --include='*.sh' 2>/dev/null | grep -v '.gitea' | grep -v 'detect_secrets' | grep -v 'test_trajectory_sanitize'; then
+            echo "FAIL: Secrets detected"
+            exit 1
+          fi
           echo "PASS: No secrets"
       - name: Pytest
         run: |
-          pip install pytest pyyaml 2>/dev/null || true
-          python3 -m pytest tests/ -q --tb=short 2>&1 || true
-          echo "PASS: pytest complete"
+          if [ -d tests/ ]; then
+            python3 -m pytest tests/ -q --tb=short
+            echo "PASS: Tests passed"
+          else
+            echo "SKIP: No tests/ directory"
+          fi