MEMORY.md

@@ -7,25 +7,34 @@ the step queue and workflow) before starting work.
 
 Panicking on bad/unexpected errors is allowed and preferred over threading
 unlikely error returns through game code — e.g. write-side Close/encode failures
-where continuing would mean corrupt state. The game already unwinds C's exit()
+where continuing would mean corrupt state. Return errors where a caller
-calls via a gameEnd panic recovered in Run. Return errors where a caller
 genuinely handles them (save-file prompts, restore validation). Reserve
 deliberate `_ =` discards for true best-effort paths (scorefile writes,
 signal-time autosave), always with a comment saying why.
 
+C's exit() calls are not unwound: one game run is one process, so myExit
+(game/rip.go) restores the terminal via Terminal.Fini and calls os.Exit(0), and
+Run() never returns. There is nothing to recover — do not write code that
+expects to regain control after game-over. The testing consequence is that any
+death (combat, starvation, level drain, freezing) exits the _test binary_, so
+tests drive command() directly rather than Run(), and crash-sweep drives pin the
+hero each turn with the fortify() helper in game/run_test.go.
+
 ## Linting
 
-The .golangci.yml is the house standard and may only be modified with sneak's
+The .golangci.yml is byte-identical to the canonical shared config and must not
-explicit permission. To disable a linter, ask, explaining what the linter does;
+be edited — not even to add an exception. To disable a linter, ask sneak,
-he approves specific exceptions, which are recorded in the config's
+explaining what the linter does; approved exceptions are recorded as in-code
-"Repo-specific exceptions" block with the approval date. Approved so far:
+//nolint directives (file-level where a whole file is affected) carrying the
-paralleltest (2026-07-06); testpackage, exhaustive, and mnd (2026-07-07). The
+approval date, which is what keeps the config canonical. Approved so far:
-complexity linters (cyclop, gocognit, nestif) are enabled and clean as of
+testpackage, exhaustive, and mnd (2026-07-07). paralleltest was approved on
-refactor step 7 (2026-07-07): the whole golangci-lint run is 0 issues, so keep
+2026-07-06 but the exception is no longer in force — it was fixed instead, with
-it that way — decompose new hot spots rather than reaching for a nolint.
+t.Parallel() in all 32 tests. The complexity linters (cyclop, gocognit, nestif)
-Line-level //nolint with a reason is used sparingly for C-faithfulness (e.g. the
+are enabled and clean as of refactor step 7 (2026-07-07): the whole
-authentic "missle" message spellings) and provably-safe gosec conversions; each
+golangci-lint run is 0 issues, so keep it that way — decompose new hot spots
-needs a justifying comment.
+rather than reaching for a nolint. Line-level //nolint with a reason is used
+sparingly for C-faithfulness (e.g. the authentic "missle" message spellings) and
+provably-safe gosec conversions; each needs a justifying comment.
 
 ## Faithfulness
 
@@ -37,5 +46,8 @@ func_name)" breadcrumbs.
 
 ## Debugging
 
-Write real, committed test files with t.Logf output and run plain `go test -v`;
+Write real, committed test files with t.Logf output and run them with the make
-no throwaway scratch scripts. Successful debug probes become regression tests.
+targets — `make test` (or `make check` for the full gate); never raw `go test`.
+The target carries `-timeout 30s -race -cover` and reruns verbosely on failure,
+so a raw invocation silently drops the race detector. No throwaway scratch
+scripts. Successful debug probes become regression tests.

README.md

@@ -72,13 +72,15 @@ term/        tcell-backed terminal, replacing curses
 cmd/rogue/   the executable
 ```
 
-The engine package is fully headless-testable: `go test ./game/` runs scripted
+The engine package is fully headless-testable: `make test` runs scripted command
-command sequences, dungeon-generation golden checks, and an RNG compatibility
+sequences, dungeon-generation golden checks, and an RNG compatibility test
-test against the original C generator.
+against the original C generator.
 
 For development, the `Makefile` wraps the toolchain: `make fmt` (gofmt +
-prettier), `make lint` (golangci-lint), `make test`, and `make check` (all
+prettier), `make lint` (golangci-lint), `make test` (the suite, under the race
-three).
+detector with coverage and a timeout), and `make check` (all three). Use the
+targets rather than invoking `go test` directly — they carry the flags the
+project relies on.
 
 ## License
 

TODO.md

@@ -34,6 +34,29 @@ wizard commands).
 
 # Completed Steps
 
+- 2026-08-09 Stale-docs correction (`docs-staleness`, closes #3): four claims in
+  `MEMORY.md`/`TODO.md`/`README.md` had gone false and were misdirecting agents
+  — the reviewer on PR #9 repeated one of them verbatim. Each was re-verified
+  against the tree before rewriting. (1) `MEMORY.md` described C's `exit()`
+  being unwound by a `gameEnd` panic recovered in `Run`; refactor step 8 deleted
+  that, `gameEnd` appears nowhere in the sources, and `myExit` (`game/rip.go`)
+  now calls `Terminal.Fini` then `os.Exit(0)` while `Run()` never returns — so
+  the section states the exit model and its testing consequence (a death exits
+  the test binary; hence `fortify()` in `game/run_test.go`). (2) `MEMORY.md`
+  said approved lint exceptions live in a "Repo-specific exceptions" block in
+  `.golangci.yml`; no such block exists and the config is byte-identical to
+  canonical (sha256 `021cc83f…46bcb`), the approvals having moved to in-code
+  `//nolint` directives carrying their dates — and `paralleltest` was listed as
+  an approved disable when it was in fact fixed (no `paralleltest` token in the
+  tree; 32 `t.Parallel()` calls against 32 tests). (3) `MEMORY.md` "Debugging"
+  and (4) `README.md` both told the reader to run `go test` directly, which
+  since PR #9 silently drops `-timeout 30s -race -cover`; both now point at
+  `make test`/`make check`. Also dropped the false "currently v2.12.2" host
+  linter claim from the 2026-08-07 entry (the host is v2.10.1 and nothing is
+  pinned; the pin question is tracked separately). Documentation only — no code,
+  `Makefile`, or config change; `Next Step` deliberately not rotated, since this
+  was out-of-band issue work.
+
 - 2026-08-09 Policy-shaped `make test` (`make-test-policy-pattern`): the `test:`
   target was a bare `go test $(GO_PKGS)` and now runs
   `-timeout 30s -race -cover` with the mandated conditional verbose rerun (on
@@ -58,7 +81,7 @@ wizard commands).
   24 long lines wrapped or their comments tightened, control bytes in
   `term/tcell.go` as character literals, and two `wsl_v5` defer cuddles. The
   repo has no golangci-lint version pin to bump (no Dockerfile or CI;
-  `make lint` runs the host `golangci-lint`, currently v2.12.2).
+  `make lint` runs whatever `golangci-lint` is on the host).
 
 - 2026-07-24 Seed compatibility — item tables (seed-compat): instrumented the C
   reference on modern-rogue with a DUMP mode (testdata/c_seedcompat.patch) that