feat: add hmr support for compound components

[Newbie012]

closes #493

[ArnaudBarre]

Thanks for working on this and adding the test!
I don't think we need to logic twice, I think this can be done like this:

if (isLikelyComponentType(exportValue)) {
  register(...)
} else if (/* is obj & all keys are component */ ) {
      for (const subKey in exportValue) {
        register(
          exportValue[subKey],
          filename + ' export ' + key + '$' + subKey,
        )
      }
}

[Newbie012]

I'll check and keep you posted

[ArnaudBarre]

My bad the code in isLikelyComponent is needed for the "validateRefreshBoundaryAndEnqueueUpdate" function.

[Newbie012]

You're fast! I was just about to comment that I don't think the tests are passing 😅 I reverted back to the original changes.

[Newbie012]

Is there something that I can help with in order to merge this change?

[ArnaudBarre]

@Newbie012 I was not very satisfied with the recursive nature of the current code. It would have return true for deeply nested object with only React component, but the runtime parts did not do it recursively. I think one level is enough.
Separating in two functions also avoid registering the top level object of compound components as react component (I think it's a no op but it's cleaner).
I also changed the code to bail out as fast as possible when testing for "compound components"

[GuskiS]

Can this be extended to support this syntax?

export function Accordion() {
  return <div></div>;
}

Accordion.Root = () => <div>Accordion Root</div>
Accordion.Item = () => <div>Accordion Item</div>