Automattic / Automattic/harper
Flag using singular countable common nouns/noun phrases without any kind of determiner
- 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
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