Automattic / Automattic/harper
False Positive: safe way -> safeway (CompoundNouns)
- Dominant language
- Rust
- Stars
- 15.4k
- Forks
- 627
- Avg merge
- 1d 15h
- Merged PRs (30d)
- 106
Description
**What got flagged?**
`safe way` flagged with: `Did you mean the closed compound noun "safeway"?` (`CompoundNouns`)
**Why is this incorrect?**
"safeway" is not a compound noun meaning 'a safe way', but rather a restaurant (which should be capitalized).
In addition, accepting the correction gives another issue, as Harper recognizes that 'safeway' should be capitalized, being the name of a restaurant. (`SpellCheck`)
**Example of correct usage:**
```md
They found a safe way to leave the city.
```
Contributor guide
Research direction
Start by locating the CompoundNouns and SpellCheck rules and reproduce the provided Markdown example. Done means “safe way” is not offered “safeway” as a correction, while valid compound-noun suggestions and unrelated capitalization checks continue to behave correctly.
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
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100