Automattic / Automattic/harper

Determiner before noun false positive

Open
#4,244 1 comment 1 reaction 0 assignees View on GitHub
bug false-positive grammar harper-core linting
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.

Image

**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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.