Skip to content

feat(server): apply published map edits - #373

Closed
suixincc wants to merge 1 commit into
Bitcoindefi:mainfrom
suixincc:codex/map-persistence
Closed

suixincc wants to merge 1 commit into
Bitcoindefi:mainfrom
suixincc:codex/map-persistence

Conversation

@suixincc

Copy link
Copy Markdown

Server map persistence for published world edits

Resolves #3.

What it does

  • Loads published tile overrides from the existing /maps/:mapNum/overrides API response after the server reads its static maps.
  • Applies graphics and blocked-state overrides without mutating unrelated layers.
  • Applies published object and NPC entity placements to the runtime map.
  • Reuses the same published NPC placement overrides when spawning NPCs and creating dynamic map instances.
  • Keeps malformed API entries out of the runtime map.

Testing

  • pnpm test
  • pnpm lint
  • pnpm build

The new focused tests cover layer-aware tile application, entity replacement, and NPC placement merging.

@gitar-bot

gitar-bot Bot commented Sep 16, 2026

Copy link
Copy Markdown

Important

You are using the Gitar free plan. Upgrade to unlock code review, CI analysis, auto-apply, custom automations, and more.

Gitar

@suixincc

Copy link
Copy Markdown
Author

Superseded by #374 with the full canonical map persistence scope.

@suixincc suixincc closed this Sep 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Etapa 0: capa de persistencia de ediciones de mapa

1 participant