default-props-match-prop-types not working with flow

[danrot]

We are using flow in our project, also for typing the properties. And it seems as if the default-props-match-prop-types rule doesn't recognize flow types. Is that possible? Is that something that could be easily implemented?

[ljharb]

hmm - in flow, do you just use default arguments?

This rule isn't actually checking types; it's just checking that defaultProps has a matching entry in propTypes when the propType isn't required. Would you want it to enforce default arguments on any non-required prop?

[danrot]

I know that it is not about checking types, I actually want what you've described, but instead of checking if there is a matching entry in propTypes it should use the flow types instead. I'll paste an example:

type Props = {
    size: number,
    value: Point,
    active: boolean,
    onClick?: (value: Point) => void,
    arrowDirection?: 'left' | 'top-left' | 'top' | 'top-right' | 'right' | 'bottom-right' | 'bottom' | 'bottom-left',
};

export default class ImageFocusPointCell extends React.PureComponent<Props> {
    static defaultProps = {
        active: false,
        showArrow: true,
    };

    // other code not important for this rule
}

The showArrow key in the defaultProps doesn't exist in the flow type, and it would be nice if this rule (or another rule, if they are not easy to combine) would give me a hint. This often happens for us during refactorings / reviews, and can easily be overseen.

[ljharb]

Gotcha. That seems like a reasonable enhancement for this rule.

[PoliakovMaksym]

I faced the same issue today but only with functional (stateless) components.

Stateless component:

Stateful component:

[ljharb]

@PoliakovMaksym the stateful component error seems correct to me; you're not using anything there that requires it be stateful.

[PoliakovMaksym]

@ljharb the issue is not with the prefer-stateless-function. This error is totally fine. I showed it to point out that it is not the default-props-match-prop-types error. The thing is that default-props-match-prop-types is present in stateless component and absence in stateful one with the exact same code (except stateless vs stateful differences)

[ljharb]

gotcha, thanks for clarifying.

[mike1808]

what's the status of this issue?

[pbambusek]

Is there any progress on this? We use this quite extensively in our projects.

[vojtakopal]

Any updates? This would really help us. Thanks.