Vault- & Strategy- related fixes

[0xNeshi]

• add missing Strategy.address field -> @SovereignAndrey do we even need to track this? It's the only (relevant) field being updated with event VaultConfigUpdated removed, see https://github.com/AngelProtocolFinance/ap-subgraph/pull/32#issuecomment-1720748853
• update Strategy creation logic
• fix vault.address assigment
• add missing .save() calls
• make registrar.ts mapping reexport everything from local-registrar.ts. Reasoning: Registrar and LocalRegistrar share logic, with the former adding new behavior. That means everything defined for local-registrar.ts applies to registrar.ts and it's safe to reexport its functions.
• add missing vault.totalShares calculations

[SovereignAndrey]

add missing Strategy.address field -> @SovereignAndrey do we even need to track this? It's the only (relevant) field being updated with event VaultConfigUpdated

I'm struggling to remember/figure out why we even allow a Vault strategy & registrar address to change at all. :thinking: Registrar/Local Registrar would be behind a proxy, so it's address shouldn't change. The strategy address linked for a vault shouldn't change, as we'd depreciate/close and strategy rather than move a vault to another strategy.

My vote is to not bother setting/tracking the address and that the strategy key should serve the lookup. Will ask about the above update logic concerns in our group chat to Steve. :+1: