Automattic / Automattic/harper

False Positive: safe way -> safeway (CompoundNouns)

Open
#3,085 1 comment 1 reaction 0 assignees View on GitHub
bug harper-core linting
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

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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.