Cannot type TagInput onChange as string[]

Environment

• Package version(s): "@blueprintjs/core": 3.15.1
• Browser and OS versions: N/A

Minimal repro: https://codesandbox.io/s/2pz12p278r?fontsize=14

Steps to reproduce

1. Use <TagInput /> with strict: true and onChange: (newValues: string[]) => ...

Actual behavior

Type '(newValues: string[]) => void' is not assignable to type '(values: ReactNode[]) => boolean | void'.
  Types of parameters 'newValues' and 'values' are incompatible.
    Type 'ReactNode[]' is not assignable to type 'string[]'.
      Type 'ReactNode' is not assignable to type 'string'.
        Type 'undefined' is not assignable to type 'string'.ts(2322)

Expected behavior

A way of typing the onChange callback without getting type errors.

Possible solution

Maybe <TagInput<string[]> />? Or maybe this is fixable in the current typings.

[thelastnode]

Could a generic component with React.ReactNode[] default work? I think that would be a non-breaking change.

interface ITagInputProps<T> {
    values: T[];
    onChange: (values: T[]) => boolean | void;
}

class TagInput<T = React.ReactNode[]> extends React.Component<ITagInputProps<T>> {
}

[adidahiya]

There have been some recent changes to the TagInput API… closing this issue in favor of https://github.com/palantir/blueprint/issues/4390