Skip to content

feat(mapgen-studio): remove Resources control from World console, codify pipeline-input zone boundary rule#1643

Merged
mateicanavra merged 1 commit into
mainfrom
design/world-console-map-params
Jun 12, 2026
Merged

feat(mapgen-studio): remove Resources control from World console, codify pipeline-input zone boundary rule#1643
mateicanavra merged 1 commit into
mainfrom
design/world-console-map-params

Conversation

@mateicanavra

Copy link
Copy Markdown
Owner

Remove the Resources control from the World/Map console footer while preserving all underlying plumbing.

The World console now authors only the settings the map pipeline actually reads — Size, Players, and Seed. Resources had no pipeline reader (StudioResourcesMode is written by the worker but consumed nowhere), so authoring it from the footer promised an effect the map never delivered.

WorldSettings.resources state, persistence, fingerprints, run-request fields, and the worker's StudioResourcesMode write are all left byte-identical. Runs continue carrying resources: "balanced" silently; when the resources vertical (placement S3/S5/A2) starts consuming the reserved wire, the same zone-boundary rule re-admits the select automatically.

The History affordance's tooltip and accessible name are trimmed to match — they now describe the last run only in terms the console itself authors: seed, size, and players.

This change also codifies the zone-boundary rule in system.md: a control belongs to the World console if and only if the map pipeline reads it to generate the map. Players qualifies (playerCountPlayersLandmass1/2 → landmass balancing); leader, civ, difficulty, and speed have no pipeline reads and stay in Game setup; resources has no pipeline read today and gets no control.

This was referenced Jun 12, 2026
@github-actions

Copy link
Copy Markdown

Railway preview (MapGen Studio): not provisioned for this PR.

Policy (Graphite stacks): previews are created only for the top-of-stack PR by default.

  • To force a preview for this PR: add label railway-preview
  • To suppress a preview: add label no-railway-preview

Debug: {"isClosed":false,"isFork":false,"suppress":false,"force":false,"isTopOfStack":false,"hasToken":true,"draft":false}

mateicanavra commented Jun 12, 2026

Copy link
Copy Markdown
Owner Author

Merge activity

  • Jun 12, 8:50 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jun 12, 10:10 PM UTC: Graphite rebased this pull request as part of a merge.
  • Jun 12, 10:10 PM UTC: @mateicanavra merged this pull request with Graphite.

@mateicanavra
mateicanavra changed the base branch from design/tile-game-geometry to graphite-base/1643 June 12, 2026 22:07
@mateicanavra
mateicanavra changed the base branch from graphite-base/1643 to main June 12, 2026 22:08
…leaves the UI, plumbing stays

The footer authors exactly what the map pipeline reads (Size, Players,
Seed); resources has no pipeline reader yet, so its select and History
mention leave the UI while WorldSettings.resources and all run/persistence
plumbing stay byte-identical — the StudioResourcesMode wire is deliberately
reserved for the placement stack's resources vertical. Codifies the
World/Game zone boundary rule (pipeline-input iff World console) in
system.md and the Pass-5 frame.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@mateicanavra
mateicanavra force-pushed the design/world-console-map-params branch from dc5c87e to e3507b3 Compare June 12, 2026 22:09
@mateicanavra
mateicanavra merged commit abc4986 into main Jun 12, 2026
This was referenced Jun 12, 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.

1 participant