chore: repoint instruction-file references at .claude/CLAUDE.md

The review workflow read the now-removed root CLAUDE.md and cited its Code Review Priorities section; both reads were guarded with '|| true', so the review step would have run with empty standards rather than failing. lint_source.py pointed contributors at the removed .claude/rules/common-pitfalls.md; its docstring already carries the full explanation, so the pointer now names the compiler-bugs reproducer only.

Written with assistance from Claude Code.

Author: sbryngelson

.github/workflows/claude-code-review.yml

@@ -206,7 +206,7 @@ jobs:
 
             Hard scope rules:
             - Do NOT inspect checked-out repository code except:
-              - ./CLAUDE.md
+              - ./.claude/CLAUDE.md
               - ./.claude/rules/*.md (max 10 files)
               - ${{ steps.review_input.outputs.review_diff_path }}
               - ${{ steps.review_input.outputs.changed_files_path }}
@@ -225,7 +225,7 @@ jobs:
 
             Allowed workflow:
             1) ls -1 .claude/rules 2>/dev/null || true
-            2) cat CLAUDE.md 2>/dev/null || true
+            2) cat .claude/CLAUDE.md 2>/dev/null || true
             3) find .claude/rules -maxdepth 1 -name "*.md" -print | head -n 10 | xargs -I{} cat "{}" 2>/dev/null || true
             4) cat "${{ steps.review_input.outputs.changed_files_path }}"
             5) cat "${{ steps.review_input.outputs.review_diff_path }}"
@@ -238,7 +238,7 @@ jobs:
             - Do NOT restate the full PR summary.
             - If there are no high-confidence findings, leave .claude-review/output.md empty and STOP.
 
-            Review standard (in priority order per CLAUDE.md "Code Review Priorities"):
+            Review standard (in priority order):
             1. Correctness (logic bugs, numerical issues, array bounds)
             2. Precision discipline (stp vs wp mixing)
             3. Memory management (@:ALLOCATE/@:DEALLOCATE pairing, GPU pointer setup)

toolchain/mfc/lint_source.py

@@ -630,7 +630,7 @@ def check_device_routine_element_args(repo_root: Path) -> list[str]:
     the element as garbage and never writes it back. Either alone is fine, every `routine` level
     is affected, and the loop counts when it sits in anything the routine calls. Copy the element
     to a scalar before the call and receive results into a scalar. See
-    .claude/rules/common-pitfalls.md and sbryngelson/compiler-bugs cce/acc-routine-element-by-reference.
+    sbryngelson/compiler-bugs cce/acc-routine-element-by-reference.
     """
     src_dir = repo_root / SRC_DIR
     files = {src: src.read_text(encoding="utf-8").splitlines() for src in _fortran_fpp_files(src_dir)}
@@ -683,7 +683,7 @@ def check_device_routine_element_args(repo_root: Path) -> list[str]:
                 for arg in _split_top_level(stmt[m.end() : j - 1]):
                     e = _ELEMENT_ARG.match(arg)
                     if e and ":" not in arg and not _VALUE_CALL_NAMES.match(e.group(1)):
-                        errors.append(f"  {rel}:{line_no} `{arg}` into `{name}` (a device routine with a seq loop): pass a scalar, see common-pitfalls.md")
+                        errors.append(f"  {rel}:{line_no} `{arg}` into `{name}` (a device routine with a seq loop): pass a scalar, see sbryngelson/compiler-bugs cce/acc-routine-element-by-reference")
     return errors