Skip to content

Retire the unreachable seismic-moment and earthquake-profile modules - #1045

Merged
2 commits merged into
mainfrom
codex/editor-seismic-cluster-20260819-1900
Aug 20, 2026
Merged

Retire the unreachable seismic-moment and earthquake-profile modules#1045
2 commits merged into
mainfrom
codex/editor-seismic-cluster-20260819-1900

Conversation

@zkWizard

Copy link
Copy Markdown
Owner

What this removes

Ten src/lib modules and their ten unit tests. Every one of them has been
unreachable from the app bundle since the week it was written (Jul 13 – 27,
2026) — confirmed by node scripts/walk-wired.mjs, not by a stale list.

They form one closed subtree: seismicMoment.ts is imported by five of the
seismicMoment* modules, and nothing outside the group imports any of them
except their own .test.ts files.

Deleted modules (git history preserves them — resurrect deliberately)

module src test last touched
seismicDepthProfile.ts 116 115 2026-07-13
seismicDispersion.ts 193 142 2026-07-14
seismicIntereventTime.ts 255 194 2026-07-27
seismicMagnitudeProfile.ts 137 124 2026-07-15
seismicMomentByDepth.ts 203 164 2026-07-14
seismicMomentConcentration.ts 180 120 2026-07-14
seismicMomentDepthCentroid.ts 207 150 2026-07-15
seismicMomentRate.ts 160 156 2026-07-15
magnitudeFrequency.ts 162 160 2026-07-14
earthquakeTimeline.ts 104 125 2026-07-16

Kept deliberately

  • seismicMomentScaleBasis.ts — 8 days old, inside the 14-day protection
    window.
  • seismicMoment.ts — still imported by seismicMomentScaleBasis.ts, so it
    is not unreachable once that module stays.

Everything else in the geology library (earthquakes.ts, earthquakeContext.ts,
magnitudeScale.ts, seismicFixedDepth.ts, seismicNetworkGeometry.ts,
volcanoElevationProfile.ts, …) is untouched.

Deletion criteria, all satisfied

The two source edits

phenologyLevelProfile.ts and volcanoElevationProfile.ts each carried a
doc-comment cross-reference to seismicDepthProfile.ts. Those clauses are
dropped so no comment points at a deleted file; the quantile method they were
describing (R-7 / NumPy-default) is still named in place, and one redundant
"it parallels …" sentence goes with them. No scientific claim, citation,
threshold, or math is altered anywhere in this PR.

Line deltas

  • Source only: −1,719 (+4 / −1,723)
  • With tests: −3,169 (+4 / −3,173)

src/lib: 276 → 266 modules; the walk now reports 154 wired, 112
unreachable
.

Gates

npx tsc --noEmit clean · npm test 289 files / 4,502 tests green ·
npx prettier --write on both touched files (unchanged) · npm run build every
chunk ok.

Visible: none — dead code removal. The two edits are inside /** */ blocks;
no rendered string, panel, or export changes.

🤖 Generated with Claude Code

zkWizard and others added 2 commits August 19, 2026 13:53
Ten src/lib modules and their ten unit tests, unreachable from the app
bundle since they were written (Jul 13 - 27). They form one closed
subtree: seismicMoment is imported only by five of them, and nothing
outside the group imports any of them except their own tests.

seismicMoment.ts and seismicMomentScaleBasis.ts are kept: the latter is
8 days old and still imports the former.

Two wired modules carried doc-comment cross-references to
seismicDepthProfile.ts; those clauses are dropped so no comment points
at a deleted file. The quantile method they described (R-7 /
NumPy-default) is still named in place.

No behaviour change: none of these modules reach the bundle.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@zkWizard zkWizard closed this pull request by merging all changes into main in 4de49d3 Aug 20, 2026
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