Automattic / Automattic/harper

Flag using singular countable common nouns/noun phrases without any kind of determiner

Open
#2,854 0 comments 0 reactions 0 assignees View on GitHub
enhancement harper-core linting
Dominant language
Rust
Stars
15.4k
Forks
627
Avg merge
1d 15h
Merged PRs (30d)
102

Description

This is a very common error made by native speakers of Russian and probably other Slavic languages. But they're not the only ones who make it.

Harper should probably suggest making the noun plural but sometimes a better fix might be introducing a determiner such as "a" or "an" or "the" or "this", etc. But if the noun is being changed then it probably agrees with a verb that needs to change too.

- **Dynamic array is** so good we got dynamic array 2 → **Dynamic arrays are** so good ...
- you have to abuse multiple language features to do basic data patterns in reasonably useable **way** → ... in reasonably useable **ways** / ... in **a** reasonably useable **way**
- Yea, that's actually **cool idea**. → Yea, that's actually **a cool idea**.
- Alignment is quite important **topic** for pre-pointer based structures. It could work in naive **way** until it doesn't. → ... quite **an** important **topic** ... / work in **a** naive **way** ...

Contributor guide

Open the contributing guide

Research direction

The issue provides example sentences but names no implementation file or test entry point. Start by locating Harper's grammar-checking rule system and reproducing the examples; done should include suggestions for missing determiners or plural forms while accounting for related verb agreement.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
tooling
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.