Add Map - Dyslexdria - April Fools

[PatrickPlaysBadly]

Re-submission, previous request was not branched properly.

Description:

Add custom map Dyslexdria. Based off of large_world_map_recolor.tif and rivers from the XL World map. Map is intended for long term use. Suggesting that map be used as an April Fools gag by replacing thumbnail and title with 'World' during first run.

• I have added screenshots for all UI updates
• I process any text displayed to the user through translateText() and I've added it to the en.json file
• I have added relevant tests to the test directory
• I confirm I have thoroughly tested these changes and take full responsibility for any bugs introduced

Following:

• Build log, screen shots, and test videos. https://discord.com/channels/1284581928254701718/1456890656147374080

Discord username:

PlaysBadly

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ecd7b07c-2ab6-4609-9096-3e8ef1893865

📥 Commits

Reviewing files that changed from the base of the PR and between 4882a5a and 513edc5.

📒 Files selected for processing (2)

• resources/lang/en.json
• src/core/game/Game.ts

✅ Files skipped from review due to trivial changes (1)

• resources/lang/en.json

🚧 Files skipped from review as they are similar to previous changes (1)

• src/core/game/Game.ts

---

Walkthrough

Adds a new fantasy map "Dyslexdria": static map assets and manifest, an English localization key, registration in the game map enum, and a playlist frequency entry.

Changes

Cohort / File(s)	Summary
Map Generator Asset map-generator/assets/maps/dyslexdria/info.json	New JSON metadata file for the Dyslexdria map containing many nation entries (coordinates, flag, name). Static data only.
Runtime Map Manifest resources/maps/dyslexdria/manifest.json	New map manifest declaring dimensions (height, width, num_land_tiles, map16x, map4x), name: "Dyslexdria", and a large nations array.
Localization resources/lang/en.json	Added "dyslexdria": "Dyslexdria" under the map localization keys.
Game Type Registration src/core/game/Game.ts	Added Dyslexdria = "Dyslexdria" to GameMapType and included it in mapCategories.fantasy.
Map Playlist Weighting src/server/MapPlaylist.ts	Added Dyslexdria: 12 to the map frequency lookup used when building non-special map pools.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

A quirky map named Dyslexdria blooms,
Tiny flags and odd-shaped rooms,
JSON lines like stitched-up threads,
New places to roam in pixelled beds,
Click to travel — let fun resume.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the main change: adding a new map called Dyslexdria with an April Fools theme.
Description check	✅ Passed	The description is directly related to the changeset, providing context about the map, its origin, intended use, and confirming testing and localization work.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 3

🧹 Nitpick comments (2)

resources/maps/dyslexdria/manifest.json (1)

1-548: Run Prettier on this manifest to clear CI style warnings.

Formatting is the only issue flagged here.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@resources/maps/dyslexdria/manifest.json` around lines 1 - 548, The
manifest.json is failing CI due to formatting; run the project's configured
Prettier (or the repository formatter) on
resources/maps/dyslexdria/manifest.json (the JSON object containing keys like
"map", "map16x", "map4x", "name", and the "nations" array) and commit the
formatted file so the CI style warnings are cleared.

map-generator/assets/maps/dyslexdria/info.json (1)

1-335: Run Prettier for this file as well to remove CI style warnings.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@map-generator/assets/maps/dyslexdria/info.json` around lines 1 - 335, The
file map-generator/assets/maps/dyslexdria/info.json is not formatted per project
Prettier rules (CI style warnings); run your project's Prettier configuration
(e.g., via npx prettier --write) on this file to reformat the JSON, ensuring
consistent spacing, trailing commas removal, and proper indentation for the
top-level "name" and "nations" entries so the CI no longer reports style errors.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@resources/lang/en.json`:
- Around line 345-346: The JSON object is invalid because the entries for
"aegean" and "dyslexdria" are missing a separating comma; update the resource
JSON so there's a comma after the "aegean" entry (i.e., add a comma between the
"aegean" and "dyslexdria" keys) to restore valid JSON syntax and allow
parsing/pretty-printing to succeed.

In `@resources/maps/dyslexdria/`.~lock.map.bin#:
- Line 1: The commit accidentally includes a LibreOffice lock file
resources/maps/dyslexdria/.~lock.map.bin#; remove this file from the PR (delete
it from the index/working tree) and add a rule to .gitignore to prevent future
commits (e.g., ignore .*~lock* or specifically .~lock.map.bin#), then amend the
commit or create a new one that removes the file; ensure you reference the
filename ".~lock.map.bin#" and the directory "resources/maps/dyslexdria" when
editing the index and .gitignore.

In `@src/core/game/Game.ts`:
- Around line 142-143: GameMapType.Dyslexdria is defined but missing from the
playlist frequency map used by MapPlaylist (so it defaults to 0 and never
appears); update the frequency map in MapPlaylist (the object that maps
GameMapType keys to numeric frequencies) to include GameMapType.Dyslexdria with
an appropriate non-zero frequency value (match similar entries like other
fantasy maps) and also add the same entry where the secondary frequency table is
defined (the other occurrence noted), ensuring both places reference
GameMapType.Dyslexdria so it can be rotated into the public playlist.

---

Nitpick comments:
In `@map-generator/assets/maps/dyslexdria/info.json`:
- Around line 1-335: The file map-generator/assets/maps/dyslexdria/info.json is
not formatted per project Prettier rules (CI style warnings); run your project's
Prettier configuration (e.g., via npx prettier --write) on this file to reformat
the JSON, ensuring consistent spacing, trailing commas removal, and proper
indentation for the top-level "name" and "nations" entries so the CI no longer
reports style errors.

In `@resources/maps/dyslexdria/manifest.json`:
- Around line 1-548: The manifest.json is failing CI due to formatting; run the
project's configured Prettier (or the repository formatter) on
resources/maps/dyslexdria/manifest.json (the JSON object containing keys like
"map", "map16x", "map4x", "name", and the "nations" array) and commit the
formatted file so the CI style warnings are cleared.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e0501198-6632-4b51-a8e8-153be590f3d2

📥 Commits

Reviewing files that changed from the base of the PR and between 3013133 and 3902e0b.

⛔ Files ignored due to path filters (4)

• map-generator/assets/maps/dyslexdria/image.png is excluded by !**/*.png
• resources/maps/dyslexdria/map.bin is excluded by !**/*.bin
• resources/maps/dyslexdria/map16x.bin is excluded by !**/*.bin
• resources/maps/dyslexdria/map4x.bin is excluded by !**/*.bin

📒 Files selected for processing (7)

• map-generator/assets/maps/dyslexdria/info.json
• resources/lang/en.json
• resources/maps/dyslexdria/.~lock.map.bin#
• resources/maps/dyslexdria/TEMP april fools day thumbnail.webp
• resources/maps/dyslexdria/manifest.json
• resources/maps/dyslexdria/thumbnail.webp
• src/core/game/Game.ts

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ VariableVince
✅ patrickbadly-hub
❌ Duwibi
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[PatrickPlaysBadly]

"world" thumbnail added to Dyslexdria. Only intended for april 1st holiday. Original thumbnail included.