Vuex and Pinia support for no-undef-properties

[marcjfj-vmlyr]

This PR adds support for Vuex and Pinia methods to the no-undef-properties rule, allowing properties defined by mapState, mapActions, mapGetters, mapMutations, and mapWritableState to be recognized as defined properties.

Test cases have been included for this enhancement.

[kazupon]

I think this rule should be supported by pinia or 3rd party plugin.
You can discuss in the issue
vuejs/pinia#2612

[marcjfj-vmlyr]

I think this rule should be supported by pinia or 3rd party plugin. You can discuss in the issue vuejs/pinia#2612

Sure but it looks like the maintainers are welcoming these kinds of enhancements, as a similar fix was added for vue/no-unused-properties. See discussion here #1675.

[nirinsanity]

This is much required. Thanks to @marcjfj-vmlyr for working on this. Please merge as soon as possible.

[FloEdelmann]

Really nice, thank you!

I have a few small suggestions.

[marcjfj-vmlyr]

@FloEdelmann Thanks for reviewing! I've made the requested changes.

[FloEdelmann]

Thanks! I have one more change request, then it's good to go from my side 🙂

[marcjfj-vmlyr]

Invalid test cases have been added.

[FloEdelmann]

Thanks! One more minor thing I just noticed.

[FloEdelmann]

Looks good to me now 🙂

[horosin]

this looks great, can't wait for it to ship!

[ota-meshi]

LGTM! Thank you!

[tmcdos]

There is a problem with this fix - it works only if mapGetters or mapState or mapMutations appears in the source code BEFORE the actual reference to the property or method.
For example, in the following code no linting error will be emitted

  created()
  {
    if (this.getCountries.length === 0) this.fetchCountries();
  },
methods:
{
        ...mapMutations(['setCountries']),
      fetchCountries()
      {
          // ... fetching from server
          this.setCountries(data);
      }
}

However, in the following code a linting error setCountries is not defined vue/no-undef-properties will be emitted

  created()
  {
    if (this.getCountries.length === 0)
    {
          // ... fetching from server
          this.setCountries(data); // <==== linting error - but it should not be
    }
  },
methods:
{
        ...mapMutations(['setCountries']),
}

[FloEdelmann]

Please open a new issue following the bug template.

• EDIT: Done: vue/no-undef-properties in v9.30 still emits error for Vuex mutation or getter #2600