[Question] How to properly define namespaced react components?

[bryceosterhaus]

I typically use namespacing in my react components to group similar components. I was wondering what the best approach to defining a type here would be. I generally do something like the example below, however using as ... isn't ideal for me. Any other ideas how to properly type a react component that also has namespaced components?

interface IProps extends React.HTMLAttributes<HTMLDivElement> {
	someProp?: any;
}

type TMyComponent = React.ForwardRefExoticComponent<IProps> & {
	ChildComponent: typeof ChildComponent;
};

import ChildComponent from './ChildComponent';

const MyComponent = React.forwardRef(
	({someProp, ...otherProps}: IProps, ref) => {
		return <div {...otherProps} />;
	}
) as TMyComponent;

MyComponent.ChildComponent = ChildComponent;

export default MyComponent;

[swyxio]

seems fine. i rarely use forwardRef :)

[bryceosterhaus]

Unfortunately we have to use lots of forwardRefs since its for a component library. Reason this came up is because this sort of type declaration doesnt work well with react-docgen. We need to also send a fix on their end, but just wanted to make sure I wasn't missing anything on the TS side.

Thanks for your help!

[swyxio]

cc @ferdaber this seems in your wheelhouse

[ferdaber]

It's a little bit annoying but what you can do is use Object.assign. It works for both default and non default exports, but kind of messes with the components' display names (you may have to assign them manually instead of relying on Function.prototype.name.

import ChildComponent from './ChildComponent';
interface IProps extends React.HTMLAttributes<HTMLDivElement> {
	someProp?: any;
}

const _MyComponent = React.forwardRef(
	({someProp, ...otherProps}: IProps, ref: React.Ref<HTMLDivElement>) => {
		return <div {...otherProps} ref={ref} />;
	}
)
_MyComponent.displayName = 'MyComponent'

type TMyComponent = React.ForwardRefExoticComponent<IProps> & {
	ChildComponent: typeof ChildComponent;
};
// this will have type TMyComponent, or more specifically
//  `typeof _MyComponent & { ChildComponent: typeof ChildComponent }`
const MyComponent = Object.assign(_MyComponent, { ChildComponent })
export default MyComponent;

[bryceosterhaus]

@ferdaber thanks for the feedback! I'll give this a try and see how it looks.

[ferdaber]

You can also create a helper function to do all of this:

import ChildComponent from './ChildComponent'
function createNamespacedComponent<T extends React.JSXElementConstructor<any>, U>(getComponent: () => T, namespaceMembers: U): T & U {
  const NamespaceComponent = getComponent();
  return Object.assign(NamespaceComponent, namespaceMembers);
}

const MyComponent = createNamespaceComponent(
  () => {
    const MyComponent = React.forwardRef(
	({someProp, ...otherProps}: IProps, ref: React.Ref<HTMLDivElement>) => {
		return <div {...otherProps} ref={ref} />;
	}
    )
    MyComponent.displayName = 'MyComponent'
    return MyComponent
  },
  { ChildComponent }
)

[bryceosterhaus]

Using object.assign turned out working well. It also looks pretty clean and the TS inference seems to work better than my previous implementation. Here is the general idea of what it looks like now

//...
const Input = React.forwardRef<HTMLInputElement, IProps>((props, ref) => (
  <input {...props} ref={ref} />
));

export default Object.assign(Input, {
  Group,
  GroupInsetItem,
  GroupItem,
  GroupText
});

[swyxio]

damn that looks pretty sweet! can you PR this in somewhere that fits pls @bryceosterhaus

[bryceosterhaus]

#166, let me know if you need me to add anything else.

[swyxio]

resolved! this went great i think!