Skip to content

docs(roadmap): retire three cliff TODOs that measurement has falsified - #69

Merged
wormeyman merged 1 commit into
mainfrom
docs/retire-falsified-cliff-todos
Jul 30, 2026
Merged

docs(roadmap): retire three cliff TODOs that measurement has falsified#69
wormeyman merged 1 commit into
mainfrom
docs/retire-falsified-cliff-todos

Conversation

@wormeyman

Copy link
Copy Markdown
Owner

Bookkeeping half of the issue #16-#24 audit. No code changes.

docs/noise/client-preview-ROADMAP.md is what someone reads before picking up cliff work, and it still listed three things as open that are not - one of them a to-do for a rule the game does not have.

roadmap line was now
"Ore excluded from cliffs" open, low priority withdrawn - no such rule exists
"Still deferred: fixImpossibleCells" deferred ported (PR #32, dea73ac), changes zero predictions
"Still deferred: wouldCollide" deferred moot - no Nauvis cliff touches water

Each was verified against the repo before editing, not taken from a summary: cliffPlacement.ts:105 and test/cliffFixImpossibleCells.spec.ts exist; dea73ac merged 2026-07-28T17:36Z.

The Nauvis residual itself stays open. PR #57 reclassified it from a field error to a rule error, and the roadmap now says that rather than pointing at three retired causes.

Done alongside this, on GitHub

This is the failure mode documented-rationales-expire describes: a register whose entries are wrong is worse than no register.

client-preview-ROADMAP.md still carried three items as open work that are
not open, and one of them as a to-do for a rule the game does not have.
The roadmap is what someone reads before picking up cliff work, so these
read as the next thing to build.

- "Ore excluded from cliffs" -> WITHDRAWN. generateCliffs() (0x1016229b4)
  touches no tile, entity or resource data, so there is no exclusion to
  port (#24, now closed). The observation behind it also fails its own
  null: region [0,0]'s 945 ore tiles are 2 blobs, and a torus-shift null
  leaves two of three regions not significant (P = 0.51, 0.29).
- "Still deferred: fixImpossibleCells" -> ported in PR #32 (dea73ac), and
  it changes ZERO predictions, so it never explained the ~6% residual.
- "Still deferred: wouldCollide" -> moot; no Nauvis cliff touches water.

The residual stays open, reclassified by PR #57 as a rule error rather
than a field error.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BkKb3S2cGyRFFHSzuCQBgj
@wormeyman
wormeyman merged commit b6de98d into main Jul 30, 2026
2 checks passed
@wormeyman
wormeyman deleted the docs/retire-falsified-cliff-todos branch July 30, 2026 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant