Cache returns incorrect data when using aliases on different fields with the same name about apollo-cache-hermes HOT 4 OPEN

Thanks for the detailed example here!

The trouble here is that hermes isn't currently aware of type unions (nor fragments with type conditions) :( So, when it sees the array there, it doesn't know to redirect fields for First typed results to FirstFields, nor Second typed to SecondFields.

It looks like the second fragment is winning (without any warning or error!), and it's interpreting the query as if it were:

{
  foo {
    __typename
    id
    bar: foo
  }
}

• The first two results get no value for foo, because hermes is expecting a value under bar, and ignores unknown fields in the payload
• The result for the last entry is "correct" hermes behavior for aliased fields, however, and probably unlikely to change (because hermes can return more fields than you ask for)

from apollo-cache-hermes.

Until we support union types, I'm going to see if I can at least get an error in there to detect cases where fragments/queries disagree about field aliases

from apollo-cache-hermes.

…and also warnings about type constraints not being supported.

from apollo-cache-hermes.

Just as a note, I added a (skipped, failing) test to the repo a while back that reflects this behavior: 01c7b34

We should re-enable it when we want to fix this.

from apollo-cache-hermes.