Rule proposal: `proper-object-iterable-methods`

Fail

// Should use `Object.values`
for (const key of Object.keys(foo)) {
	bar(foo[key]);
}

// Should use `Object.entries`
for (const key of Object.keys(foo)) {
	bar(foo[key], key);
}

// Should use `Object.values`
const foo = Object.keys(bar).map(key => baz(bar[key]));

// Should use `Object.entries`
const foo = Object.keys(bar).map(key => baz(bar[key], key));

// Should use `Object.keys`
for (const [key] of Object.entries(foo)) {
	bar(key);
}

Pass

for (const key of Object.keys(foo)) {
	foo[key] = 'new value';
	delete foo[key];
	foo[key]++;
}

[fisker]

I think we should also enforce Object.keys over Object.entries in this case

for (const [key] of Object.entries(foo)) {
	bar(key);
}

So better come up a better name cover all them.

[fisker]

proper-object-iterable-methods?