Skip to content

Adds map of the Caribbean 🏴‍☠️#4067

Merged
evanpelle merged 2 commits into
openfrontio:mainfrom
RickD004:caribbean
May 30, 2026
Merged

Adds map of the Caribbean 🏴‍☠️#4067
evanpelle merged 2 commits into
openfrontio:mainfrom
RickD004:caribbean

Conversation

@RickD004
Copy link
Copy Markdown
Contributor

@RickD004 RickD004 commented May 29, 2026

Description:

Fixes #4069

Adds map of the Caribbean sea and its islands. Archipelago map with lots of islands, lots of water and a lot of trade.

This map has multiple large landmasses of similar size to prevent steamrolls (the largest islands and landmasses are around 30%), and many, many small islands where players can survive and trade. Players will have to island hop in order to win. 34 nations of Caribbean countries and territories, with an extra 28 AdditionalNations for a total of 62 nations for crowded HvN.

Heavy Island maps are very popular in the broader community and we dont have one for v32, so i figured it would be nice to have a very requested and popular world location

570k land tiles, fairly small for a map, would be right placed before World (600k tiles). Also adds some flags of caribbean regions.

CaribbeanSeaTeaser1.mp4

Terrain source from OpenTopography, already credited. No modification to the tests are needed for new maps added in Game.ts

Please complete the following:

  • 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

Please put your Discord username so you can be contacted if a bug or regression is found:

tri.star1011

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 29, 2026

Review Change Stack

Walkthrough

Adds Caribbean as a new regional map: registers the map type, supplies map data and manifest, adds country and localization entries, and updates playlist frequency weights.

Changes

Caribbean Map Integration

Layer / File(s) Summary
Game map type registration
src/core/game/Game.ts
GameMapType enum gains Caribbean; mapCategories.regional includes GameMapType.Caribbean.
Map data files and generator registration
map-generator/assets/maps/caribbean/info.json, map-generator/main.go, resources/maps/caribbean/manifest.json
Adds info.json with nations/additionalNations, registers caribbean in the map-generator registry, and adds manifest with map dimensions and nations.
Country resources and localization
resources/countries.json, resources/lang/en.json
Adds Barbuda, Mosquito Kingdom, Nevis, Yucatan entries and map.caribbean English label.
Playlist frequency configuration
src/server/MapPlaylist.ts
Inserts Caribbean: 5 into FREQUENCY and rearranges Labyrinth/Onion entries.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • evanpelle
  • FloPinguin

Poem

🌊 New isles on the coded sea,
Flags and names in harmony,
Enum set, maps registered true,
Playlists learn a turquoise hue,
Caribbean sails into view 🏝️

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding a new Caribbean map. It is concise, clear, and directly reflects the primary objective of the pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The pull request description clearly relates to the changeset, providing detailed context about the Caribbean map addition including design philosophy, nation count, map size, and feature additions.

✏️ 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

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

coderabbitai[bot]
coderabbitai Bot previously approved these changes May 29, 2026
evanpelle
evanpelle previously approved these changes May 29, 2026
Copy link
Copy Markdown
Collaborator

@evanpelle evanpelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@github-project-automation github-project-automation Bot moved this from Triage to Final Review in OpenFront Release Management May 29, 2026
@RickD004 RickD004 dismissed stale reviews from evanpelle and coderabbitai[bot] via 6fa8977 May 29, 2026 23:26
@evanpelle evanpelle added approved Approved for a PR, if you assigned to the issue. bypass-pr-check Bypass the automated PR check and removed approved Approved for a PR, if you assigned to the issue. labels May 30, 2026
@evanpelle evanpelle added this to the v32 milestone May 30, 2026
@evanpelle evanpelle merged commit c049a81 into openfrontio:main May 30, 2026
12 of 14 checks passed
@github-project-automation github-project-automation Bot moved this from Final Review to Complete in OpenFront Release Management May 30, 2026
@RickD004 RickD004 deleted the caribbean branch May 30, 2026 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bypass-pr-check Bypass the automated PR check

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

Carribean Map

2 participants