Merge branch 'master' into MovingBubblesFresh-clean

Author: sbryngelson

.github/workflows/test.yml

@@ -41,21 +41,15 @@ jobs:
       - name: Lint Toolchain
         run: ./mfc.sh lint
 
-      - name: Lint Source - No Raw Directives
-        run: |
-          ! grep -iR '!\$acc\|!\$omp' --exclude="parallel_macros.fpp" --exclude="acc_macros.fpp" --exclude="omp_macros.fpp" --exclude="shared_parallel_macros.fpp" --exclude="syscheck.fpp" ./src/*
-
-      - name: Lint Source - No Double Precision Intrinsics
-        run: |
-          ! grep -iR 'double_precision\|dsqrt\|dexp\|dlog\|dble\|dabs\|double\ precision\|real(8)\|real(4)\|dprod\|dmin\|dmax\|dfloat\|dreal\|dcos\|dsin\|dtan\|dsign\|dtanh\|dsinh\|dcosh\|d0' --exclude-dir=syscheck --exclude="*nvtx*" --exclude="*precision_select*" ./src/*
-
-      - name: Lint Source - No Junk Code
-        run: |
-          ! grep -iR -e '\.\.\.' -e '\-\-\-' -e '===' ./src/*
+      - name: Lint Source
+        run: python3 toolchain/mfc/lint_source.py
 
       - name: Lint Docs
         run: python3 toolchain/mfc/lint_docs.py
 
+      - name: Lint Parameter Docs
+        run: python3 toolchain/mfc/lint_param_docs.py
+
   file-changes:
     name: Detect File Changes
     runs-on: 'ubuntu-latest'

toolchain/mfc/lint_param_docs.py

@@ -1,7 +1,8 @@
 """Check parameter documentation completeness.
 
 Tier 1 (blocking): Parameters with DESCRIPTIONS entries must appear in case.md.
-Tier 2 (warnings): Fortran namelist / REGISTRY sync checks.
+Tier 2 (blocking): Fortran namelist params must be in REGISTRY.
+Tier 3 (blocking): Scalar REGISTRY params must have descriptions.
 """
 
 from __future__ import annotations
@@ -90,9 +91,11 @@ def _parse_namelist_params(fpp_path: Path) -> set[str]:
         stripped = line.strip()
         if stripped.startswith("!") or stripped.startswith("#:") or stripped.startswith("#"):
             continue
-        if "namelist /user_inputs/" in stripped.lower():
-            in_namelist = True
-            accum = stripped.split("/user_inputs/", 1)[1] if "/user_inputs/" in stripped else ""
+        lower = stripped.lower()
+        if "namelist /user_inputs/" in lower:
+            idx = lower.index("/user_inputs/") + len("/user_inputs/")
+            accum += " " + stripped[idx:]
+            in_namelist = accum.rstrip().endswith("&")
             continue
         if in_namelist:
             if stripped.startswith("&"):