Rule suggestion - warn when custom error message is missing in expect function

[MindaugasMateika]

What do you think about the rule which should warn about missing custom error messages? I'm not very familiar with rule creation and what rules are ok/not ok so just putting it as a suggestion now.

Docs: https://playwright.dev/docs/test-assertions#custom-expect-message

+ Good
await expect(page.locator('text=Name'), 'should be logged in').toBeVisible()

- Bad
await expect(page.locator('text=Name')).toBeVisible()

+ Good
await expect.soft(page.locator('text=Name'), 'should be logged in').toBeVisible()

- Bad
await expect.soft(page.locator('text=Name')).toBeVisible()

[mskelton]

I'm totally fine with this as a rule, so long as it is not enabled by default.

[mxschmitt]

+1 like @mskelton said.