One could argue it should be the other way around - the imported state should also throw an error if accessed like this. The reason is that shared state should be imported at the usage locations and modified there, rather than being passed around as props and then modified, which creates indirection and makes reasoning about the code more confusing.

One could argue it should be the other way around - the imported state should also throw an error if accessed like this.

I would be okay with this too, maybe even prefer it! My main concern is consistency in the context of a module. As long as shared state behaves the same when passing it as a prop, I'd be happy with either approach.

I will say though, having both throw an error when passed around as a prop (without being bound too) would be my preference as it jives better with my mental model.

I think it's easier to understand the rule: this prop must be bound if the child component mutates it

instead of: this prop must be bound if the child component mutates it, _unless_ it's shared state from an external module, but not shared state declared in context='module'.

Consistently works as of #10464 , the last two don't throw an error now