Unnecessary context value updates in ShopifyProvider component

[ekaterina-nikitina-de]

The spread operator creates a new object every time and that makes the context provider propagate new value on every render even if nothing was really changed in the props.

Proposal:
Replace the spread operator here and further with direct props as they are primitives:

export function ShopifyProvider({
  children,
  countryIsoCode,
  languageIsoCode,
  storeDomain,
  storefrontToken,
  storefrontApiVersion
}: ShopifyProviderProps): JSX.Element {
...

hydrogen/packages/hydrogen-react/src/ShopifyProvider.tsx

Line 32 in 8b9c726

...shopifyConfig

MDN: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Spread_syntax#spread_in_object_literals

const obj1 = {a:"1"};
const clonedObj = { ...obj1 };
clonedObj === obj1
false

[blittle]

Is this affecting your app's performance? It seems like a micro-optimization. Happy to accept a PR though.

[ekaterina-nikitina-de]

This has an indirect effect: some components that use React.Suspense throw the React #421 errors due to unexpected context updates. I'll create a PR.