Discussion on: @cshooks/usetrie, A React Hook for Typeahead/Autocomplete

View post

Will check out the repo this week Sung. Just back from vacation, so my body is back from the beach but not my brain. 😉

OK, just glanced at the code quickly, so just a few comments about improving the TypeScript code. 😉

interface Children {
   [key: string]: Node;
}

Can be written using TypeScript's built-in generic Record type.

type Children = Record<string, Node>;

constructor(public character: string = '') {}

You don't need to specify the string type as it's inferred by the default value ''.

For the type for getText, you can create a type and reuse it instead of using (obj: any) => string; in multiple places. Also you could rewrite this type as

<T>(obj: T) => string;

And extend T so that it respects some criteria.

For public methods, you don't need to specify the public keyword. It's the default.
For type Words = Word[];, I wouldn't bother with this type. Just use Word[].