Tracker: v2 todos

[alvrs]

Tracker for various small v2 TODOs that are not specifically related to tablegen (those are tracked in #438)

World

• Add system call hooks (see feat: v2 - add store, world and schema-type, cli table code generation #422 (comment))
• Add support for ReverseMappingHook with composite keys (see feat(world): add naive ReverseMappingHook/Module #487)

Network

• Add key schema support (feat: add support for key schemas #480)

Tests

• Add tests for schema-type (see feat: v2 - add store, world and schema-type, cli table code generation #422 (comment)) test(schema-type): add more tests #1146
• Add tests for systems writing to the World (see feat: v2 - add store, world and schema-type, cli table code generation #422 (comment))
• Add tests for onBeforeSetField, onAfterSetField and deleteRecord to World.t.sol
• Add tests for decoding methods: feat: v2 event decoding #415

Decisions

• Turn SchemaTable into an codegen table instead of a special case #1134
• Add missing events to Store (for setting partial fields) and review existing events #1121

Misc

• Refactor store to use schema-type (see feat: v2 - add store, world and schema-type, cli table code generation #422 (comment))
• Revisit decision around mapping function selectors to resources vs directly to systems #1135
• Revisit bytes32 vs uint256 for table IDs (move back to bytes32 (instead of uint256) for tableIds #484)
• Revisit CLI error wrapping, use an approach like viem to preserve the underlying error
• Improve types through parsing config + decoding schemas/data
• Add support for fallback function in worldgen #1136 (see feat(cli): add registerFunctionSelectors to deploy cli #501 (comment))
• Add TS support for modules (so TS knows about their tables)
  • Add defineComponent support for module tables
• Revisit codegen approach for TS stuff added in feat(cli/recs/std-client): add ts definitions generator #536 (try to remove need for codegen and make TS inference work)
• Pass private key into forge script PostDeploy.sol so that you can vm.startBroadcast() without the private key arg (and thus centralizing where you pass in the private key)
  • I took a stab at this by adding --private-key flag to the forge script command, which worked, but needs silent: true to not log the private key. It also had sporadic nonce too low issues, which means this may need to tx.wait() before running post deploy, or keep track of the nonce internally so it can pass in the right one.
  • @dk1a says "for tables in PostDeploy you could probably just define _msgSender to be world, since scripts run in a contract, not a pure library"
• Revisit naming of core functions #1137
• Revisit whether getUniqueEntity should hash its nonce (feat(world): add UniqueEntityModule #552 (comment))

MODE

• Fix invalid fieldType for arrays (https://github.com/latticexyz/mud/pull/493/files#r1154345942)

[dk1a]

moved 4 tightcoder-specific todos to #1132

[alvrs]

Moved a couple gas optimization related items to #1133