immutable-data: Map Support

[xethorn]

Suggestion

Thank you for this amazing linter, this has helped us keep our codebase maintainable and our data flows easy to understand! I have a request: currently immutable-data supports array and disallows modifications. When it comes to other similar datatypes like Maps, the linter doesn't seem to have rules that prevent modifications.

Example with immutable-data set to error:

export const value = new Map();
value.set('foo', 'bar'); // no errors.

If I missed it, can you please point me in the right direction? Thank you!

[RebeccaStevens]

We don't currently have map and set support in this rule.
This is definitely something we can add though, however it will require type data, so we won't be able to support in vanilla js.

In the meantime, assuming you are using TypeScript, I would recommend a ReadonlyMap:

export const value: ReadonlyMap<string, string> = new Map();
value.set('foo', 'bar'); // error.

[xethorn]

Thank you. I think that's fine to not support this for vanilla js; and yes, set support would be amazing too.

[github-actions]

🎉 This issue has been resolved in version 9.0.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[xethorn]

@RebeccaStevens : I have no words, this is awesome 🫶 Thank you so much!