latticexyz / mud

MUD is a framework for building autonomous worlds
https://mud.dev
MIT License
693 stars 174 forks source link

fix(cli,world): resolve table by just name #2850

Closed holic closed 1 month ago

holic commented 1 month ago

resolveTableId (or specifically the context passed into resolveWithContext) changed slightly in #2803 where we started using namespaced table names, which broke some previous usages of just table names without a namespace. This adds back in support for just table names, with an error thrown if we can't resolve the table due to collisions (with an actionable error message to quickly resolve).

We had set up a new resolveTableId with the new config but never used it/exported it, so this also fixes that and removes the long-deprecated one as well as the now-unused v1 modules config.

changeset-bot[bot] commented 1 month ago

🦋 Changeset detected

Latest commit: 77d9e82fd33286eccc78b51b1bb9b851d311276a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 23 packages | Name | Type | | ----------------------------- | ----- | | @latticexyz/cli | Patch | | @latticexyz/world | Patch | | mock-game-contracts | Patch | | @latticexyz/dev-tools | Patch | | @latticexyz/store-sync | Patch | | @latticexyz/world-modules | Patch | | @latticexyz/store-indexer | Patch | | @latticexyz/abi-ts | Patch | | @latticexyz/block-logs-stream | Patch | | @latticexyz/common | Patch | | @latticexyz/config | Patch | | create-mud | Patch | | @latticexyz/faucet | Patch | | @latticexyz/gas-report | Patch | | @latticexyz/protocol-parser | Patch | | @latticexyz/query | Patch | | @latticexyz/react | Patch | | @latticexyz/recs | Patch | | @latticexyz/schema-type | Patch | | solhint-config-mud | Patch | | solhint-plugin-mud | Patch | | @latticexyz/store | Patch | | @latticexyz/utils | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR