Skip to content

Adds Gulf of Guinea map#4590

Merged
evanpelle merged 2 commits into
openfrontio:mainfrom
RickD004:gulfofguinea
Jul 13, 2026
Merged

Adds Gulf of Guinea map#4590
evanpelle merged 2 commits into
openfrontio:mainfrom
RickD004:gulfofguinea

Conversation

@RickD004

Copy link
Copy Markdown
Contributor

Description:

Adds Gulf of Guinea map to the game:
https://en.wikipedia.org/wiki/Gulf_of_Guinea

Final map of the "pirate maps" for v33 , alongside Irish Sea and Levant. The map is designed so that all the trade from the mainland, which covers 2 of the four corners of the "square" (inverse L) passes through 3 small islands, which although small in size they will super-boost pirates in that zone. The mainland is otherwise fairly lineal, playing like maps like Conakry.

Also some sub-saharan african representation

The map is decorated with Impassable terrain in its southern border, and has additional nations for Hvn and Solo games

Grabacion.de.pantalla.2026-07-12.203008.mp4

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

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

tri.star1011

@coderabbitai

coderabbitai Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

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: 7811a4a3-fa5e-4a0a-8b65-65d99b367974

📥 Commits

Reviewing files that changed from the base of the PR and between a6de362 and bc4b062.

📒 Files selected for processing (2)
  • map-generator/assets/maps/gulfofguinea/info.json
  • resources/maps/gulfofguinea/manifest.json
🚧 Files skipped from review as they are similar to previous changes (2)
  • map-generator/assets/maps/gulfofguinea/info.json
  • resources/maps/gulfofguinea/manifest.json

Walkthrough

Adds the Gulf Of Guinea map with metadata, nation definitions, country entries, localization, a thumbnail, resource manifests, and runtime map registration.

Changes

Gulf Of Guinea map

Layer / File(s) Summary
Map definition and assets
map-generator/assets/maps/gulfofguinea/info.json, resources/maps/gulfofguinea/manifest.json, resources/maps/gulfofguinea/thumbnail.webp
Defines map metadata, dimensions, scale configurations, nations, additional nations, and the thumbnail asset.
Country data and localization
resources/countries.json, resources/lang/en.json
Adds country or historical-region entries and the map.gulfofguinea English label.
Runtime map registration
src/core/game/Maps.gen.ts
Adds the GameMapType.GulfOfGuinea enum member and its exported map metadata entry.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Sequence Diagram(s)

sequenceDiagram
  participant MapManifest
  participant TranslationCatalog
  participant MapsRegistry
  MapManifest->>MapsRegistry: provide GulfOfGuinea metadata
  TranslationCatalog->>MapsRegistry: provide map.gulfofguinea label
  MapsRegistry->>MapsRegistry: register GulfOfGuinea
Loading

Possibly related PRs

Suggested labels: Maps

Suggested reviewers: evanpelle, flopinguin

Poem

New shores appear where blue waves run,
Gulf nations gather beneath the sun.
Flags and coordinates mark the way,
A fresh map joins the game today. 🌊

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: adding the Gulf of Guinea map.
Description check ✅ Passed The description is directly related to the map addition and matches the pull request objectives.
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.

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.

@RickD004 RickD004 added this to the v33 milestone Jul 13, 2026

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@map-generator/assets/maps/gulfofguinea/info.json`:
- Around line 316-318: Update the place name in the map location entry
identified by the name “Fulenzem” to “Foulenzem”; leave its coordinates and flag
unchanged.
🪄 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: 203bf7a0-12b0-4df1-8a5c-28dcb8bcbdf3

📥 Commits

Reviewing files that changed from the base of the PR and between e57939a and a6de362.

⛔ Files ignored due to path filters (11)
  • map-generator/assets/maps/gulfofguinea/image.png is excluded by !**/*.png
  • resources/flags/Ambazonia.svg is excluded by !**/*.svg
  • resources/flags/Biafra.svg is excluded by !**/*.svg
  • resources/flags/Kingdom of Benin.svg is excluded by !**/*.svg
  • resources/flags/Kingdom of Dahomey.svg is excluded by !**/*.svg
  • resources/flags/Kingdom of Kongo.svg is excluded by !**/*.svg
  • resources/flags/Mali Empire.svg is excluded by !**/*.svg
  • resources/flags/gq-an.svg is excluded by !**/*.svg
  • resources/maps/gulfofguinea/map.bin is excluded by !**/*.bin
  • resources/maps/gulfofguinea/map16x.bin is excluded by !**/*.bin
  • resources/maps/gulfofguinea/map4x.bin is excluded by !**/*.bin
📒 Files selected for processing (6)
  • map-generator/assets/maps/gulfofguinea/info.json
  • resources/countries.json
  • resources/lang/en.json
  • resources/maps/gulfofguinea/manifest.json
  • resources/maps/gulfofguinea/thumbnail.webp
  • src/core/game/Maps.gen.ts

Comment thread map-generator/assets/maps/gulfofguinea/info.json
@github-project-automation github-project-automation Bot moved this from Triage to Development in OpenFront Release Management Jul 13, 2026
@RickD004 RickD004 added the Maps A new map, or adjustments to an existing map itself, its json, etc, label Jul 13, 2026
@github-project-automation github-project-automation Bot moved this from Development to Final Review in OpenFront Release Management Jul 13, 2026
@evanpelle
evanpelle merged commit 558f2e2 into openfrontio:main Jul 13, 2026
11 checks passed
@github-project-automation github-project-automation Bot moved this from Final Review to Complete in OpenFront Release Management Jul 13, 2026
@RickD004
RickD004 deleted the gulfofguinea branch July 13, 2026 21:51
CanArslanDev referenced this pull request in CanArslanDev/SkyFront Jul 15, 2026
## Description:

Adds Gulf of Guinea map to the game:
https://en.wikipedia.org/wiki/Gulf_of_Guinea

Final map of the "pirate maps" for v33 , alongside Irish Sea and Levant.
The map is designed so that all the trade from the mainland, which
covers 2 of the four corners of the "square" (inverse L) passes through
3 small islands, which although small in size they will super-boost
pirates in that zone. The mainland is otherwise fairly lineal, playing
like maps like Conakry.

Also some sub-saharan african representation

The map is decorated with Impassable terrain in its southern border, and
has additional nations for Hvn and Solo games

https://github.com/user-attachments/assets/9c9fd8c0-fefb-46c4-8720-55e90401681e

## Please complete the following:

- [x] I have added screenshots for all UI updates
- [x] I process any text displayed to the user through translateText()
and I've added it to the en.json file
- [x] I have added relevant tests to the test directory

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

tri.star1011

(cherry picked from commit 558f2e20db0e0f51c0070f6db6db217fcdccdb87)
This was referenced Jul 18, 2026
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.

2 participants