ci: relocate Nim toolchain to $RUNNER_TEMP

Adds parent-nim-install-directory: ${{ runner.temp }} to setup-nim-action
in both ci.yml and docs.yml. Nim now installs to $RUNNER_TEMP/.nim_runtime/
instead of vendoring under $GITHUB_WORKSPACE/.nim_runtime/.

Effects:
- nph --check . works normally (no vendored Nim source tree to recurse into)
- ci.yml restored to nph --check . (was: nph --check src tests workaround)
- docs.yml unchanged path resolution (already used `command -v nim`)
- Removes nph-scope gotcha from CLAUDE.md / AGENTS.md (no longer applicable)

Author: elijahr

.github/workflows/ci.yml

@@ -19,21 +19,17 @@ jobs:
       - uses: jiro4989/setup-nim-action@v2
         with:
           nim-version: ${{ matrix.nim-version }}
+          parent-nim-install-directory: ${{ runner.temp }}
       - uses: actions/cache@v4
         with:
           path: ~/.nimble
           key: nimble-${{ runner.os }}-${{ hashFiles('**/*.nimble') }}
       - name: nimble install deps
         run: nimble install --depsOnly -y
       - name: nph format check
-        # Scope to project source dirs only. `nph --check .` would recurse into
-        # `.nim_runtime/` (the Nim source tree pulled in by setup-nim-action)
-        # and segfault on malformed test files (e.g. nimsuggest/tests/tconcept1.nim).
-        # The pre-commit hook already operates only on tracked .nim files, so this
-        # matches local behavior.
         run: |
           nimble install -y nph@0.7.0   # PIN: must match .pre-commit-config.yaml local nph hook
-          nph --check src tests
+          nph --check .
         continue-on-error: ${{ matrix.nim-version == 'devel' }}
       - name: nimble check
         run: nimble check

.github/workflows/docs.yml

@@ -13,7 +13,9 @@ jobs:
       - uses: actions/setup-python@v5
         with: { python-version: "3.12" }
       - uses: jiro4989/setup-nim-action@v2
-        with: { nim-version: stable }
+        with:
+          nim-version: stable
+          parent-nim-install-directory: ${{ runner.temp }}
       - name: Make Nim compiler module importable
         # mkdocstrings-nim's extractor compiles a helper that does
         # `import compiler/ast` etc. The compiler module ships at

AGENTS.md

@@ -29,13 +29,6 @@ Tests live in `tests/` and are discovered by `nimble test`.
 - **Formatting authority:** `nph` (pinned to v0.7.0 in pre-commit, ci.yml, and CLAUDE.md Setup).
 - **Pre-commit guard:** `nph` and `nimble check` on commit. No pre-push stage (Nim has no slow type-check analogue to mypy).
 
-## Gotchas
-
-- **CI tool scoping:** Always scope formatters/checkers to `src tests` (not `.`) —
-  `jiro4989/setup-nim-action@v2` vendors a copy of the Nim source tree at `.nim_runtime/`
-  containing intentionally malformed test fixtures (e.g. `nimsuggest/tests/tconcept1.nim`)
-  that crash nph 0.7.0 with SIGSEGV.
-
 ## The justfile is the canonical command surface
 
 `nimble` is an implementation detail. Add new dev workflows as `just` recipes,

CLAUDE.md

@@ -62,13 +62,6 @@ The pre-commit `nph` hook, the CI `nph --check` step, and the local `just lint`/
 all assume the SAME nph version (currently 0.7.0). Bump in all three places together,
 or skew will cause "format clean locally, fails in CI" footguns.
 
-### `nph --check` scope
-In CI (`jiro4989/setup-nim-action@v2`), the action vendors the full Nim source tree under
-`.nim_runtime/`. Running `nph --check .` recurses into it and crashes on
-`nimsuggest/tests/tconcept1.nim` (intentionally malformed; nph 0.7.0 SIGSEGVs, exit 139).
-The `ci.yml` here uses `nph --check src tests` for that reason. The pre-commit hook is
-unaffected (operates on git-tracked files only, and `.nim_runtime/` is gitignored).
-
 ### `mkdocstrings-nim` requires `default_handler: nim`
 Without `default_handler: nim` in mkdocs.yml, mkdocstrings 1.x routes documentation blocks
 to a non-existent `python` handler and the build fails with a confusing