Entry info handling was a bit different than the other ones, but it didn't have to - the difference was that we were converting to serverEntryInfo, but this conversion was done earlier (now in stateSyncSpecs.entries.convertClientToServerInfos). The second conversion (here) was redundant and can be safely omitted.

A condition where innerSpec isn't present corresponds to the else clause in the original code - we're keeping the same behavior by continuing the loop (it is important because it lets us introduce new state syncs without breaking older clients).

Flow isn't that smart

Cannot get spec.innerHashSpec.hashKey because property hashKey is missing in undefined