Skip to content

Conversation

@sardidefcon
Copy link
Contributor

Description:

This PR adds a new map called "The Box." The map is a completely flat, square landmass (2048x2048px) with no water. It's a very simple map that focuses on player combat and only allows land-based trade.
The map has nine default nations: one in each corner and along each side, plus one nation in the center.

  • 9 nations by default
  • Without water, only land trade
  • Without geographical features
  • A classic OpenFront experience
thebox

Discord username to be contacted if a bug or regression is found:

sardidefcon

A new flat map with a size of 2048x2048px without bodies of water
Updated to include the "thebox" map
Added the necessary files for generating the "thebox" map
@CLAassistant
Copy link

CLAassistant commented Jan 28, 2026

CLA assistant check
All committers have signed the CLA.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 28, 2026

Walkthrough

Three files were added to introduce a new map named "The Box" to the map generation system. The map includes static asset definitions and a complete manifest with nine nations positioned at specified coordinates. One line was added to register the map in the main registry.

Changes

Cohort / File(s) Summary
Map Generator Asset
map-generator/assets/maps/thebox/info.json
New static JSON file defining "The Box" map with a nations array containing nine entries, each with coordinates, flag field, and nation name
Map Generator Registry
map-generator/main.go
Added single map entry {Name: "thebox"} to the public maps registry
Resource Map Manifest
resources/maps/thebox/manifest.json
New manifest file specifying map dimensions (2048x2048 base, with 512x512 and 1024x1024 scaled variants) and nine named nations with assigned coordinates and flags

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

Poem

🗺️ A box is born upon the map, nine nations take their place,
With coordinates and colors, they claim their destined space,
Registry updated, manifest complete,
The Box now joins the collection—welcome to the feat! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: adding a new map called 'The Box' to the project.
Description check ✅ Passed The description is well-detailed and directly related to the changeset, explaining the map's characteristics, design philosophy, and technical specifications.
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

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

@github-project-automation github-project-automation bot moved this from Triage to Complete in OpenFront Release Management Jan 28, 2026
@Duwibi
Copy link
Contributor

Duwibi commented Jan 28, 2026

I haven't tested it, but this PR currently only has the main files for the map, excluding the ones that actually load it in game. It still needs to be added to en.json, MapPlaylist.ts and Game.ts. 👍

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

Labels

None yet

Projects

Status: Complete

Development

Successfully merging this pull request may close these issues.

3 participants