Only functions names can have verbs (get, set, create, remove)

[fregante]

OK

function getMozzarella() {}
const createPizza = async () => {}


const mozzarella = getMozzarella();
const pizza = await createPizza(mozzarella);

NOT OK

const getMozzarella = await api();
const createPizza = new Pizza();

There are many verbs that should not be part of non-function variables, but I think the most common ones are enough:

get
set
unset
delete
add
remove
destroy

Maybe not:

clear // this is also an adjective
init // this could also be a noun for "initialization of X", a Promise that resolves with X

[sindresorhus]

Have you actually experienced this being a problem in the wild?

[sindresorhus]

What if it's a factory?

const getRandom = random({{from: 0, to: 10});

[fregante]

getRandom is a function, so that's fine

[fregante]

Have you actually experienced this being a problem in the wild?

Yep, twice today, you just saw it too

Line 22: refined-github/refined-github@12711ca#diff-f12ecb2375f1a7fe299ae1d0f3a1e55dR22
Line 23: refined-github/refined-github@12711ca#diff-f12ecb2375f1a7fe299ae1d0f3a1e55dR23

[sindresorhus]

getRandom is a function, so that's fine

But how do we statically detect that? It will only work if you actually call getRandom in the current scope or a child scope.

[fregante]

But how do we statically detect that?

TypeScript?

[sindresorhus]

Related: #347

[sindresorhus]

@fregante Can you try proposing this over at https://github.com/typescript-eslint/typescript-eslint first? If they, decline it, we can consider adding it here.

[fregante]

I opened a more generic issue there

[fregante]

It doesn't look like it will be accepted

[fisker]

It's a good idea, but like impossible to implement without type info.

[fregante]

Issue rightfully rejected there:

• Rule proposal: Ensure special verbs match the type (e.g. hasName is boolean or boolean function) typescript-eslint/typescript-eslint#3032

I think this could be implemented in a flat config with naming-convention or by just customizing it in XO. Should I open an issue on XO?