fix(store,world): fix StoreRead.getDynamicFieldLength (#2680)

Co-authored-by: alvarius <[email protected]>

.changeset/gorgeous-seals-smile.md

@@ -0,0 +1,7 @@
+---
+"@latticexyz/store": patch
+---
+
+Patched `StoreRead.getDynamicFieldLength` to properly read `StoreCore.getDynamicFieldLength`.
+
+Previously `StoreRead.getDynamicFieldLength` incorrectly read from `StoreCore.getFieldLength`, which expected a `fieldIndex` instead of a `dynamicFieldIndex`, and thereby returned an invalid result if the table had both static and dynamic fields (in which case `fieldIndex` != `dynamicFieldIndex`). `StoreRead` is used for external reads from the `Store`/`World` contract, so this bug only materialized in external table reads (ie from `Systems` outside the root namespace) of the dynamic length of a field in a table with both static and dynamic fields.

docs/pages/store/reference/misc.mdx

@@ -2688,5 +2688,5 @@ Contains a constant representing the version of the Store protocol.
 _Identifier for the current Store protocol version._
 
 ```solidity
-bytes32 constant STORE_VERSION = "2.0.0";
+bytes32 constant STORE_VERSION = "2.0.1";
 ```

docs/pages/world/reference/misc.mdx

@@ -104,5 +104,5 @@ Contains a constant representing the version of the World protocol.
 _Identifier for the current World protocol version._
 
 ```solidity
-bytes32 constant WORLD_VERSION = "2.0.0";
+bytes32 constant WORLD_VERSION = "2.0.1";
 ```

e2e/packages/contracts/mud.config.ts

@@ -45,10 +45,18 @@ export default defineWorld({
     },
     StaticArray: {
       schema: {
+        num: "uint256",
         value: "uint256[3]",
       },
       key: [],
     },
+    DynamicArray: {
+      schema: {
+        num: "uint256",
+        value: "uint256[]",
+      },
+      key: [],
+    },
   },
   modules: [
     {