Rule idea: no-mutation-props

[ianschmitz]

A component is never meant to modify its own props (https://facebook.github.io/react/docs/components-and-props.html#props-are-read-only).

I suggest we had a no-mutation-props rule to check for mutations of a components own props. It looks like we could likely copy the logic used in no-direct-mutation-state.js.

[viankakrisna]

maybe just go ahead and send a pr?

[ljharb]

Seems reasonable. However, this rule would have to warn any time the entire props object is passed to a function (which is a bad practice anyways) because that makes it impossible (in most cases) to determine if it's being mutated.