jhogue/inclusive-language

Issue with Regex or order of operations

Open

#9 opened on Aug 18, 2025

View on GitHub
 (0 comments) (0 reactions) (0 assignees)CSS (1 fork)github user discovery
bugenhancementhelp wanted

Repository metrics

Stars
 (3 stars)
PR merge metrics
 (PR metrics pending)

Description

Line 109 and 100 of the JS file state:

// Right now we have a problem where words in the explanation are also searched and flagged as a problematic word/phrase. Test with the word “black” // Because of this, a phase like “black list” will never match because “black” is matched first

I'd like to figure out a way around this. I would bet that the Regex (line 52) might be able to avoid matching on words inside the title attribute. Or, the order in which words are matched needs to change. Or or the results need to be stored in an array before they are output, which might avoid being checked again in a loop.

Contributor guide

Research direction
Examine the regex on line 52 and the order of word matching. Check if words in the title attribute are being matched by accident. Consider storing matches in an array before outputting to avoid double checking.
Tech stack
javascript
Domain
tooling
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
RegexJavaScript
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75