Skip to content

Fix levels 51–54 wrongly placed in Expert epoch#14

Merged
martinl5 merged 2 commits into
mainfrom
claude/nice-heisenberg-gjh8bp
Jun 10, 2026
Merged

Fix levels 51–54 wrongly placed in Expert epoch#14
martinl5 merged 2 commits into
mainfrom
claude/nice-heisenberg-gjh8bp

Conversation

@martinl5

Copy link
Copy Markdown
Owner

Summary

Fixes #9 — levels 51–54 (Maritime Trade Finance intro) were marked epoch: 'Expert' despite having difficulty: 2–3, sandwiching much easier levels between Expert difficulty-4 banking levels (41–50) and Expert difficulty-4 maritime levels (55–63).

  • Renumber old 51–54 → new 41–44, change epoch to 'Advanced' — these introductory maritime levels (simple SELECT, JOIN, basic CTE + RANK) now live at the end of the Advanced section where their difficulty belongs
  • Renumber old 41–50 → new 45–54, keeping epoch: 'Expert' — the complex banking CTEs/window-function levels shift up by 4
  • Levels 55–63 unchanged
  • Update LevelUpModal hint boundaries to match new epoch edges (40, 44, 54)

Test plan

  • npm test — all 278 tests pass (catalog integrity: contiguous epochs in order, sequential IDs; all 63 solution queries execute correctly)
  • npm run typecheck — no TypeScript errors
  • In the Level Navigator: levels 41–44 appear under Advanced (green), levels 45–54 appear under Expert (amber)
  • LevelProgressMap shows a single clean Advanced→Expert transition at level 45

https://claude.ai/code/session_01TkxyEqFBuqRCvRAzRLPEbs


Generated by Claude Code

Levels 51–54 (Maritime Trade Finance intro, difficulty 2–3) were
marked epoch:'Expert' but sandwiched between Expert difficulty-4
banking levels (41–50) and Expert difficulty-4 maritime levels
(55–63), creating a jarring difficulty regression.

Fix: renumber so the maritime intro levels come before the Expert
banking levels in the sequence:
  - Old 51–54 → New 41–44, epoch changed to 'Advanced'
  - Old 41–50 → New 45–54, epoch unchanged ('Expert')
  - 55–63 unchanged

Both catalog-integrity test constraints (contiguous epochs in order,
sequential IDs) continue to pass. LevelUpModal hint boundaries
updated to match the new epoch boundaries (40, 44, 54).

Fixes #9

https://claude.ai/code/session_01TkxyEqFBuqRCvRAzRLPEbs
@vercel

vercel Bot commented Jun 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sqlwak Ready Ready Preview, Comment Jun 10, 2026 11:57am

@martinl5 martinl5 marked this pull request as ready for review June 10, 2026 11:57
@martinl5 martinl5 merged commit 9d2461b into main Jun 10, 2026
3 checks passed
@martinl5 martinl5 deleted the claude/nice-heisenberg-gjh8bp branch June 10, 2026 11:58
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.

Some levels are wrongly placed

2 participants