fix: exportName metadata for JSXMemberExpressions that use named imports

[JohnDaly]

Changes

Fix for issue described in: #4389

Components declared using JSXMemberExpression nodes were not hydrating properly, due to how the component-export attribute was being determined in the astroJSX Babel plugin.

Example:

const SampleComponent = () => { ... }

export const Namespace = { SampleComponent };

The component-export attribute for this component (<Namespace.SampleComponent client:load />) was 'SampleComponent', but will now be 'Namespace.SampleComponent', which can be resolved properly via the hydration logic here:

astro/packages/astro/src/runtime/server/astro-island.ts

Lines 72 to 75 in dfa3519

	this.Component = componentModule;
	for (const part of componentExport.split('.')) {
	this.Component = this.Component[part]
	}

Testing

Tested locally. I also added to the existing e2e/namespaced-component.test.js.

Docs

No documentation updates needed. This is a minor bug fix.

[changeset-bot]

🦋 Changeset detected

Latest commit: ae9f82f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 13 packages

Name	Type
astro	Patch
@e2e/astro-component	Patch
@e2e/error-react-spectrum	Patch
@e2e/error-sass	Patch
@e2e/errors	Patch
@e2e/lit-component	Patch
@e2e/preact-component	Patch
@e2e/react-component	Patch
@e2e/solid-component	Patch
@e2e/solid-recurse	Patch
@e2e/svelte-component	Patch
@e2e/e2e-tailwindcss	Patch
@e2e/ts-resolution	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[JohnDaly]

Components that use an ImportNamespaceSpecifier will have their import name set to '*', which is handled here:

astro/packages/astro/src/jsx/babel.ts

Lines 71 to 73 in dfa3519

	if (meta.name === '*') {
	meta.name = getTagName(node).split('.').slice(1).join('.')!;
	}

In other cases, we'll just use the tag name, which will be resolved in the same way that the namespace is. That happens here:

astro/packages/astro/src/runtime/server/astro-island.ts

Lines 72 to 75 in dfa3519

	this.Component = componentModule;
	for (const part of componentExport.split('.')) {
	this.Component = this.Component[part]
	}

[matthewp]

Note that @natemoo-re is going to work on this issue.

[natemoo-re]

Thanks for the fix, looks great!