Chain-spec Builder ignores SS58 prefix for invulnerables/session keys when generating chain-specs

[seadanda]

I added the coretime-kusama live preset here, but when it outputs the chain-spec it ignores the SS58 Prefix configured for the chain and prints the 5... i.e. substrate generic addresses for the invulnerables in the resulting json.

AFAICT this does not affect the chain spec in how it runs, since it can still get the account ID from this. Running a node with a chain-spec generated like this seems to sync fine.

It's the same account as shown when you check with subkey (subkey inspect -n kusama <account_id>) but just the wrong representation.

Can use string representation directly in the genesis in combination with to_ss58check_with_version but this means constructing a String representation directly in the json, instead of the real structs SessionKeys { .. } etc. which could lead to maintenance headaches.

It would be nice to get this working in general

Summary of discussion with @michalkucharczyk in chat:
Original chain-specs do not use the prefix and public key bytes are the same nevertheless
Could also post-process in the GenesisBuilder subsystem on the runtime side (as part of get_preset function)
Could be nice to include SS58 Prefix in runtime itself, then could also be used in presets.