Skip to content

Update arena configs from maDisplayTools#111

Closed
github-actions[bot] wants to merge 1 commit into
mainfrom
auto/sync-arena-configs
Closed

Update arena configs from maDisplayTools#111
github-actions[bot] wants to merge 1 commit into
mainfrom
auto/sync-arena-configs

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Automated Arena Config Sync

This PR was automatically generated by the sync-arena-configs workflow.

Source: maDisplayTools/configs/arenas/ + configs/arena_registry/index.yaml

Changes

The js/arena-configs.js file has been regenerated from the latest YAML configs in maDisplayTools.

Review Checklist

  • Verify the config changes look correct
  • Test the arena editor and 3D viewer with the updated configs

mbreiser pushed a commit that referenced this pull request Jun 16, 2026
…gen (LAB-111 follow-up)

js/arena-configs.js is generated by scripts/generate-arena-configs.js
(JSON.stringify, 2-space). PR #110 ran Prettier over it (4-space), so the
generator and Prettier disagreed on every sync run — each sync then produced a
~180-line all-formatting diff (e.g. the bot's PR #111).

- .prettierignore: add js/arena-configs.js (generated, same principle as js/vendor/).
- regenerate to canonical generator output (reverts #110's reformat of this one
  file). Config data unchanged; ARENA_REGISTRY intact; npm test green.

Future syncs now diff only on real config changes (plus the timestamp).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@mbreiser

Copy link
Copy Markdown
Contributor

Superseded by main commit d8bf678. js/arena-configs.js is generated, so it's now in .prettierignore and regenerated to canonical generator output on main — this all-formatting diff (the file fighting Prettier after #110) is no longer needed. Config data was unchanged and ARENA_REGISTRY was intact here; the registry-wipe fix it validated is in 98c57a7 (LAB-111, Done). Closing.

@mbreiser mbreiser closed this Jun 16, 2026
@mbreiser
mbreiser deleted the auto/sync-arena-configs branch June 16, 2026 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant