Anx is a single syllable - issue #46

[RamakrishnanArun]

Fix for #46

Added a fix for the word anxious which is 2 syllables as you can see from the google search and dictionaries.

https://www.google.com/search?q=anxious

and
https://www.merriam-webster.com/dictionary/anxious
https://www.dictionary.com/browse/anxious?s=t

[wooorm]

Nice, thanks for working on a fix!

1. Could you check a couple of other words with those letters?

$ sed -n '/anx/p' < /usr/share/dict/words
anxietude
anxiety
anxious
anxiously
anxiousness
banxring
Caranx
lanx
Manx
Manxman
Manxwoman
overanxiety
overanxious
overanxiously
phalanx
phalanxed
planxty
reanxiety
unanxiety
unanxious
unanxiously
unanxiousness

2. Could you add the new test cases (as as a couple of the above, and anxious itself) to the javascript tests instead of the fixtures? Like so perhaps:

   syllable/test/index.js

   Lines 235 to 263 in d7f2e84

   	t.deepEqual(
   	[
   	'reminiscense',
   	'commonsense',
   	'innocense',
   	'recompense',
   	'condense',
   	'defense',
   	'dispense',
   	'expanse',
   	'expense',
   	'immense',
   	'intense',
   	'license',
   	'mouthrinse',
   	'nonsense',
   	'offense',
   	'pretense',
   	'response',
   	'suspense',
   	'cleanse',
   	'dense',
   	'rinse',
   	'sense',
   	'tense'
   	].map(syllable),
   	[4, 3, 3, 3, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 1, 1, 1, 1, 1],
   	'GH-39 ([aeiouy]nse$)'
   	)

[RamakrishnanArun]

Yeah, I realised later that some of those words didn't match, like anxiety for example. Looking at that now.

[RamakrishnanArun]

This is proving to be a little more tricky than I thought. I will step through the code a little more.

[wooorm]

Hmm, yeah this can be tricky, because of so many weird regex interplay. Syllables are hard!

If there is no way around it, problematic.json could be an escape hatch, but as that’s not scalable, code is preferred

[RamakrishnanArun]

I want to avoid adding it to the problematic.json file. There are other similar words like society that might benefit from these patterns too.

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (d7f2e84) to head (80b8c47).
Report is 12 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff            @@
##              main       #47   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines           74        74           
=========================================
  Hits            74        74           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.