Skip to content

Add Hawaii map#3187

Merged
FloPinguin merged 3 commits intoopenfrontio:mainfrom
gabigabogabu:feature/hawaii-map
Feb 14, 2026
Merged

Add Hawaii map#3187
FloPinguin merged 3 commits intoopenfrontio:mainfrom
gabigabogabu:feature/hawaii-map

Conversation

@gabigabogabu
Copy link
Contributor

@gabigabogabu gabigabogabu commented Feb 12, 2026

Description:

Add Hawaii as a new regional map. Features 9 nations across the Hawaiian island chain (Niihau, Kauai, Oahu, Molokai, Lanai, Kahoolawe, Maui, Kona, Hilo). 612K land tiles at 3920x2544 408K land tiles at 3200x2076, terrain generated from real relief data with accurate volcanic peaks and coastal lowlands. Playlist frequency: 4.

Hawaii thumbnail

Checklist

  • 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

Source: https://commons.wikimedia.org/wiki/File:USA_Hawaii_relief_location_map.svg
Discord: gabigabogabu

@CLAassistant
Copy link

CLAassistant commented Feb 12, 2026

CLA assistant check
All committers have signed the CLA.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 12, 2026

Walkthrough

Adds Hawaii as a new map: data and manifest files, registers it in the map generator and playlist, exposes it in game map types, updates English localization, and adds credits entries.

Changes

Cohort / File(s) Summary
Hawaii Map Metadata
map-generator/assets/maps/hawaii/info.json, resources/maps/hawaii/manifest.json
Adds new JSON files describing the Hawaii map and territories: map dimensions, land-tile counts, and nine nations/islands (Niihau, Kauai, Oahu, Molokai, Lanai, Kahoolawe, Maui, Kona, Hilo) with flag: "us".
Game Type Definition
src/core/game/Game.ts
Adds GameMapType.Hawaii = "Hawaii" and includes it in mapCategories.regional.
Backend Registration
map-generator/main.go
Registers "hawaii" in the maps registry to make the new map available to the generator.
Playlist Configuration
src/server/MapPlaylist.ts
Adds Hawaii frequency entry (Hawaii: 4) to the map playlist weighting.
Localization
resources/lang/en.json
Adds "hawaii": "Hawaii" to the map translations (adjusted surrounding punctuation for valid JSON).
Credits
CREDITS.md
Adds two attribution blocks for "Hawaii Relief Map" with license and link entries.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

Aloha, nine isles now in the repo,
Small JSON shores where new pixels go,
Enum, playlist, locale sing along,
Credits and manifest hum the song,
Waves of code — a quiet, bright hello. 🌺

🚥 Pre-merge checks | ✅ 3 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Merge Conflict Detection ⚠️ Warning ❌ Merge conflicts detected (27 files):

⚔️ CREDITS.md (content)
⚔️ map-generator/main.go (content)
⚔️ resources/lang/en.json (content)
⚔️ src/client/HostLobbyModal.ts (content)
⚔️ src/client/LeaderboardModal.ts (content)
⚔️ src/client/SinglePlayerModal.ts (content)
⚔️ src/client/components/map/MapDisplay.ts (content)
⚔️ src/client/components/map/MapPicker.ts (content)
⚔️ src/client/graphics/layers/EventsDisplay.ts (content)
⚔️ src/client/graphics/layers/HeadsUpMessage.ts (content)
⚔️ src/client/graphics/layers/ImmunityTimer.ts (content)
⚔️ src/core/GameRunner.ts (content)
⚔️ src/core/configuration/Config.ts (content)
⚔️ src/core/configuration/DefaultConfig.ts (content)
⚔️ src/core/execution/BotExecution.ts (content)
⚔️ src/core/execution/NationExecution.ts (content)
⚔️ src/core/execution/PlayerExecution.ts (content)
⚔️ src/core/game/Game.ts (content)
⚔️ src/core/game/GameImpl.ts (content)
⚔️ src/core/game/GameView.ts (content)
⚔️ src/core/game/PlayerImpl.ts (content)
⚔️ src/core/game/RailNetwork.ts (content)
⚔️ src/core/game/RailNetworkImpl.ts (content)
⚔️ src/core/game/TrainStation.ts (content)
⚔️ src/server/MapPlaylist.ts (content)
⚔️ tests/Attack.test.ts (content)
⚔️ tests/client/LeaderboardModal.test.ts (content)

These conflicts must be resolved before merging into main.
Resolve conflicts locally and push changes to this branch.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Add Hawaii map' clearly and concisely describes the main change: adding a new Hawaii map to the game.
Description check ✅ Passed The description is well-related to the changeset, detailing the new Hawaii map with 9 island nations, terrain specifications, and playlist frequency.
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.


No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
resources/maps/hawaii/manifest.json (1)

44-48: Kahoolawe is a very small island — is it a fun nation to play?

Kahoolawe is tiny compared to the other islands. If it ends up with very few land tiles, it might feel unbalanced or not enjoyable as a playable nation. Since you playtested this, you probably already know — just double-checking it feels right in-game.


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 Feb 12, 2026
@FloPinguin
Copy link
Contributor

I'm not sure at all but this map might be too big?
Someone on the main discord said that the only map which lags for them is japan (2500x2500, 488k land tiles).

Copy link
Contributor

@Duwibi Duwibi left a comment

Choose a reason for hiding this comment

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

Simular to Flo, I also think the map would be too big to be able to be queued for public lobbies. With that size, it would probably lag on PC, let alone on lower-end phones and tablets. How did your own testing go?
The other thing I wanna ask is what base map did you use? (It needs to have copyright rules which allow for commercial use).
Overall, if the base map isn't under strict copyright and it runns somewhat smoothly, we could add it to the game to be played in singleplayer and private lobbies. But it's definitely too big for public lobbies imo 🤔

@github-project-automation github-project-automation bot moved this from Triage to Development in OpenFront Release Management Feb 12, 2026
@gabigabogabu
Copy link
Contributor Author

gabigabogabu commented Feb 12, 2026

I played a few solo matches on this map with no issues, though it was on a M4 MBA. I increased the size because it felt a bit cramped when it was Faroe-Island-map-sized-ish, but happy to scale it down.

Source is this licensed under Creative Commons

Copy link
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.

Since the map is license by-sa, please add attribution in the CREDITS.md

@gabigabogabu
Copy link
Contributor Author

Since the map is license by-sa, please add attribution in the CREDITS.md

Done

coderabbitai[bot]
coderabbitai bot previously approved these changes Feb 13, 2026
@gabigabogabu
Copy link
Contributor Author

Scaled down the map from 3920x2544 (612K land tiles) to 3200x2076 (408K land tiles)

Copy link
Contributor

@Duwibi Duwibi left a comment

Choose a reason for hiding this comment

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

I think it's ok for now. If it causes performence issues during the playtest, we can always remove it from public lobbies :)

@FloPinguin FloPinguin added this to the v30 milestone Feb 14, 2026
@FloPinguin FloPinguin added the Maps A new map, or adjustments to an existing map itself, its json, etc, label Feb 14, 2026
Copy link
Contributor

@FloPinguin FloPinguin 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 Development to Final Review in OpenFront Release Management Feb 14, 2026
@FloPinguin FloPinguin added this pull request to the merge queue Feb 14, 2026
Merged via the queue into openfrontio:main with commit 6e557c5 Feb 14, 2026
9 of 11 checks passed
@github-project-automation github-project-automation bot moved this from Final Review to Complete in OpenFront Release Management Feb 14, 2026
@gabigabogabu
Copy link
Contributor Author

Thanks guys!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maps A new map, or adjustments to an existing map itself, its json, etc,

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

5 participants