fix: render neutral/rebelled hexes on map#10
Merged
Conversation
WorldLayout only included hexes with ownerId > 0 in the hexOwners map, causing unowned hexes to render as black voids. Now all hexes are included — owned hexes show colored overlays, neutral ones show a dim gray outline. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add namedAgents[owner][nameHash] mapping to AgentRegistry so the same address cannot create two agents with the same name. create_agent MCP tool is now idempotent — returns the existing agent if already created. - AgentRegistry: add uniqueness check, getAgentByName view, cleanup on remove - MCP chain/tools: findAgentByName + idempotent create_agent - agent-runner: ensureAgent leverages idempotent create, logs reuse - Fix stale tool descriptions (slots 12→6, cooldown 60s→5s, combat rewards) - 2 new tests: duplicate revert + same-name-different-owner ok Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Router: 0x4968505A0F9C17e573adE046F39B0E9b91E79d11 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- AgentRegistry: add ownerAgentIds mapping, getAgentsByOwner view, cleanup on removeAgent - MCP: add getAgentsByOwner chain method + get_my_agents tool - Redeploy to Gravity Testnet with fresh state Router: 0x96EBC8b846795d19130e1Dd944B61Ab90696bA1a Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add get_my_agents tool to both docs - Remove phantom claim_hex / get_claimable_hexes (not implemented) - Fix expansion description: combat-only, no hex claiming - Add idempotent agent creation note to skill.md - Update happiness boost details in skill.md Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- llm.ts: remove claim_hex/get_claimable_hexes from selfTools (not implemented) - state-storage.md: fix AgentRegistry schema (remove gold field, add namedAgents/ownerAgentIds), update operations, add GameEngine to Router diagram, replace gold references with ore Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
WorldLayout filtered out locations with ownerId=0, which hid agents standing on rebelled hexes. Now all hex locations are included in the layout regardless of ownership. Neutral hexes show "Neutral" as owner. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
WorldLayout.tsonly included owned hexes in the hexOwners map, skipping unowned ones entirelyHexMapScene.tsonly rendered hexes present in hexOwnersFix
Test plan
🤖 Generated with Claude Code