Automattic / Automattic/harper
Determiner before noun false positive
- Dominant language
- Rust
- Stars
- 15.4k
- Forks
- 627
- Avg merge
- 1d 15h
- Merged PRs (30d)
- 106
Description
**What got flagged?**
If I can convince Josnel his company is fading into irrelevance, that he should let me destroy it all for a sizable donation to his next app, he’ll see reason.
Between see and reason, Harper suggests adding an a, the, or an. But the phrasing is already correct.
**Why is this incorrect?**
The phrasing I'm using is a common expression and grammatically correct.
**Example of correct usage:**
See above on what got flagged. I've noticed the word see triggering this a lot lately.
Contributor guide
Research direction
Start by reproducing the reported sentence in Harper and trace the determiner-before-noun diagnostic that flags “see reason.” Locate the rule and its existing regression coverage, then ensure this common expression is not flagged while similar determiner omissions remain detected.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- devtools
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100