Skip to content

Commit 2391b76

Browse files
kWeglinskikWeglinski
authored
Merge pull request #42 from kWeglinski/fix/disable_dictionary
fix dictionary, fix sync between fields
2 parents cc7b8c0 + 9719c20 commit 2391b76

21 files changed

+3146
-285
lines changed

‎.eslintrc.cjs‎

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,15 @@ module.exports = {
99
ignorePatterns: ['dist', '.eslintrc.cjs'],
1010
parser: '@typescript-eslint/parser',
1111
plugins: ['react-refresh'],
12+
overrides: [
13+
{
14+
files: ['**/__tests__/**/*.ts', '**/__tests__/**/*.tsx'],
15+
rules: {
16+
'@typescript-eslint/no-require-imports': 'off',
17+
'@typescript-eslint/no-var-requires': 'off',
18+
},
19+
},
20+
],
1221
rules: {
1322
'react-refresh/only-export-components': [
1423
'warn',

‎babel.config.cjs‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
/* eslint-disable */
2+
module.exports = {
3+
presets: [
4+
['@babel/preset-env', { targets: { node: 'current' } }],
5+
],
6+
};

‎changelog.md‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,8 @@
1+
### 0.14.4
2+
3+
- Fix: when `DISABLE_DICTIONARY=true` languagetool was not returning any response.
4+
- Fix: recreated sync between input fields between tabs which was lost during zustand migration.
5+
16
### 0.14.3
27

38
- Fix: during the creation of improved fix tooltip I've introduced a bug that completely broke position where text is fixed. This is fixed now.

‎coverage/API.ts.html‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -403,7 +403,7 @@ <h1><a href="index.html">All files</a> API.ts</h1>
403403
<div class='footer quiet pad2 space-top1 center small'>
404404
Code coverage generated by
405405
<a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
406-
at 2025-08-02T20:21:56.395Z
406+
at 2026-02-07T20:31:51.235Z
407407
</div>
408408
<script src="prettify.js"></script>
409409
<script>

‎coverage/filterHelper.js.html‎

Lines changed: 214 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,214 @@
1+
2+
<!doctype html>
3+
<html lang="en">
4+
5+
<head>
6+
<title>Code coverage report for filterHelper.js</title>
7+
<meta charset="utf-8" />
8+
<link rel="stylesheet" href="prettify.css" />
9+
<link rel="stylesheet" href="base.css" />
10+
<link rel="shortcut icon" type="image/x-icon" href="favicon.png" />
11+
<meta name="viewport" content="width=device-width, initial-scale=1" />
12+
<style type='text/css'>
13+
.coverage-summary .sorter {
14+
background-image: url(sort-arrow-sprite.png);
15+
}
16+
</style>
17+
</head>
18+
19+
<body>
20+
<div class='wrapper'>
21+
<div class='pad1'>
22+
<h1><a href="index.html">All files</a> filterHelper.js</h1>
23+
<div class='clearfix'>
24+
25+
<div class='fl pad1y space-right2'>
26+
<span class="strong">92.3% </span>
27+
<span class="quiet">Statements</span>
28+
<span class='fraction'>12/13</span>
29+
</div>
30+
31+
32+
<div class='fl pad1y space-right2'>
33+
<span class="strong">88.23% </span>
34+
<span class="quiet">Branches</span>
35+
<span class='fraction'>15/17</span>
36+
</div>
37+
38+
39+
<div class='fl pad1y space-right2'>
40+
<span class="strong">100% </span>
41+
<span class="quiet">Functions</span>
42+
<span class='fraction'>3/3</span>
43+
</div>
44+
45+
46+
<div class='fl pad1y space-right2'>
47+
<span class="strong">92.3% </span>
48+
<span class="quiet">Lines</span>
49+
<span class='fraction'>12/13</span>
50+
</div>
51+
52+
53+
</div>
54+
<p class="quiet">
55+
Press <em>n</em> or <em>j</em> to go to the next uncovered block, <em>b</em>, <em>p</em> or <em>k</em> for the previous block.
56+
</p>
57+
<template id="filterTemplate">
58+
<div class="quiet">
59+
Filter:
60+
<input type="search" id="fileSearch">
61+
</div>
62+
</template>
63+
</div>
64+
<div class='status-line high'></div>
65+
<pre><table class="coverage">
66+
<tr><td class="line-count quiet"><a name='L1'></a><a href='#L1'>1</a>
67+
<a name='L2'></a><a href='#L2'>2</a>
68+
<a name='L3'></a><a href='#L3'>3</a>
69+
<a name='L4'></a><a href='#L4'>4</a>
70+
<a name='L5'></a><a href='#L5'>5</a>
71+
<a name='L6'></a><a href='#L6'>6</a>
72+
<a name='L7'></a><a href='#L7'>7</a>
73+
<a name='L8'></a><a href='#L8'>8</a>
74+
<a name='L9'></a><a href='#L9'>9</a>
75+
<a name='L10'></a><a href='#L10'>10</a>
76+
<a name='L11'></a><a href='#L11'>11</a>
77+
<a name='L12'></a><a href='#L12'>12</a>
78+
<a name='L13'></a><a href='#L13'>13</a>
79+
<a name='L14'></a><a href='#L14'>14</a>
80+
<a name='L15'></a><a href='#L15'>15</a>
81+
<a name='L16'></a><a href='#L16'>16</a>
82+
<a name='L17'></a><a href='#L17'>17</a>
83+
<a name='L18'></a><a href='#L18'>18</a>
84+
<a name='L19'></a><a href='#L19'>19</a>
85+
<a name='L20'></a><a href='#L20'>20</a>
86+
<a name='L21'></a><a href='#L21'>21</a>
87+
<a name='L22'></a><a href='#L22'>22</a>
88+
<a name='L23'></a><a href='#L23'>23</a>
89+
<a name='L24'></a><a href='#L24'>24</a>
90+
<a name='L25'></a><a href='#L25'>25</a>
91+
<a name='L26'></a><a href='#L26'>26</a>
92+
<a name='L27'></a><a href='#L27'>27</a>
93+
<a name='L28'></a><a href='#L28'>28</a>
94+
<a name='L29'></a><a href='#L29'>29</a>
95+
<a name='L30'></a><a href='#L30'>30</a>
96+
<a name='L31'></a><a href='#L31'>31</a>
97+
<a name='L32'></a><a href='#L32'>32</a>
98+
<a name='L33'></a><a href='#L33'>33</a>
99+
<a name='L34'></a><a href='#L34'>34</a>
100+
<a name='L35'></a><a href='#L35'>35</a>
101+
<a name='L36'></a><a href='#L36'>36</a>
102+
<a name='L37'></a><a href='#L37'>37</a>
103+
<a name='L38'></a><a href='#L38'>38</a>
104+
<a name='L39'></a><a href='#L39'>39</a>
105+
<a name='L40'></a><a href='#L40'>40</a>
106+
<a name='L41'></a><a href='#L41'>41</a>
107+
<a name='L42'></a><a href='#L42'>42</a>
108+
<a name='L43'></a><a href='#L43'>43</a>
109+
<a name='L44'></a><a href='#L44'>44</a></td><td class="line-coverage quiet"><span class="cline-any cline-neutral">&nbsp;</span>
110+
<span class="cline-any cline-neutral">&nbsp;</span>
111+
<span class="cline-any cline-neutral">&nbsp;</span>
112+
<span class="cline-any cline-neutral">&nbsp;</span>
113+
<span class="cline-any cline-neutral">&nbsp;</span>
114+
<span class="cline-any cline-neutral">&nbsp;</span>
115+
<span class="cline-any cline-neutral">&nbsp;</span>
116+
<span class="cline-any cline-neutral">&nbsp;</span>
117+
<span class="cline-any cline-neutral">&nbsp;</span>
118+
<span class="cline-any cline-neutral">&nbsp;</span>
119+
<span class="cline-any cline-neutral">&nbsp;</span>
120+
<span class="cline-any cline-neutral">&nbsp;</span>
121+
<span class="cline-any cline-neutral">&nbsp;</span>
122+
<span class="cline-any cline-neutral">&nbsp;</span>
123+
<span class="cline-any cline-yes">1x</span>
124+
<span class="cline-any cline-neutral">&nbsp;</span>
125+
<span class="cline-any cline-yes">4x</span>
126+
<span class="cline-any cline-neutral">&nbsp;</span>
127+
<span class="cline-any cline-neutral">&nbsp;</span>
128+
<span class="cline-any cline-yes">4x</span>
129+
<span class="cline-any cline-neutral">&nbsp;</span>
130+
<span class="cline-any cline-neutral">&nbsp;</span>
131+
<span class="cline-any cline-yes">4x</span>
132+
<span class="cline-any cline-neutral">&nbsp;</span>
133+
<span class="cline-any cline-neutral">&nbsp;</span>
134+
<span class="cline-any cline-yes">3x</span>
135+
<span class="cline-any cline-neutral">&nbsp;</span>
136+
<span class="cline-any cline-yes">2x</span>
137+
<span class="cline-any cline-no">&nbsp;</span>
138+
<span class="cline-any cline-neutral">&nbsp;</span>
139+
<span class="cline-any cline-yes">2x</span>
140+
<span class="cline-any cline-neutral">&nbsp;</span>
141+
<span class="cline-any cline-neutral">&nbsp;</span>
142+
<span class="cline-any cline-neutral">&nbsp;</span>
143+
<span class="cline-any cline-yes">2x</span>
144+
<span class="cline-any cline-neutral">&nbsp;</span>
145+
<span class="cline-any cline-neutral">&nbsp;</span>
146+
<span class="cline-any cline-yes">1x</span>
147+
<span class="cline-any cline-neutral">&nbsp;</span>
148+
<span class="cline-any cline-yes">3x</span>
149+
<span class="cline-any cline-neutral">&nbsp;</span>
150+
<span class="cline-any cline-yes">4x</span>
151+
<span class="cline-any cline-yes">4x</span>
152+
<span class="cline-any cline-neutral">&nbsp;</span></td><td class="text"><pre class="prettyprint lang-js">import { lookupWord } from "../../server/words.js";
153+
&nbsp;
154+
/**
155+
* Filters a LanguageTool API response.
156+
*
157+
* - Preserves all top‑level fields of the original result.
158+
* - For each match:
159+
* • If the rule category is "TYPOS" and dictionary filtering is enabled, remove the match when the word exists in the custom dictionary.
160+
* • Otherwise keep the match unchanged.
161+
*
162+
* @param {object} result The raw response from LanguageTool.
163+
* @param {boolean} enableDictionaryFilter Whether to enable dictionary-based filtering (default: true).
164+
* @returns {object} A new object with the same top‑level properties and a filtered `matches` array.
165+
*/
166+
export const filterResult = (result, enableDictionaryFilter = true) =&gt; {
167+
// Log for debugging purposes
168+
console.log("[FILTER] total matches received:", result.matches?.length ?? 0);
169+
&nbsp;
170+
// Clone the entire result to keep all other fields intact
171+
const filtered = { ...result };
172+
&nbsp;
173+
// Safely process the `matches` array if it exists
174+
filtered.matches = (result.matches || [])
175+
.map((match) =&gt; {
176+
// If dictionary filtering is enabled and this is a typo rule, check the custom dictionary
177+
if (enableDictionaryFilter &amp;&amp; match.rule?.category?.id === "TYPOS") {
178+
// Validate context exists before extracting text
179+
<span class="missing-if-branch" title="if path not taken" >I</span>if (!match.context || !match.context.text) {
180+
<span class="cstat-no" title="statement not covered" > return match;</span>
181+
}
182+
const exists = lookupWord(
183+
match.context.text.substr(match.context.offset, match.context.length)
184+
);
185+
// Remove the match when the word is present in the dictionary
186+
return exists ? false : match;
187+
}
188+
// Keep all other matches unchanged (or when dictionary filtering is disabled)
189+
return match;
190+
})
191+
.filter((elem) =&gt; elem); // Drop any `false` entries
192+
&nbsp;
193+
console.log("[FILTER] matches after dictionary filter:", filtered.matches?.length ?? <span class="branch-1 cbranch-no" title="branch not covered" >0)</span>;
194+
return filtered;
195+
};</pre></td></tr></table></pre>
196+
197+
<div class='push'></div><!-- for sticky footer -->
198+
</div><!-- /wrapper -->
199+
<div class='footer quiet pad2 space-top1 center small'>
200+
Code coverage generated by
201+
<a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
202+
at 2026-02-07T20:44:36.294Z
203+
</div>
204+
<script src="prettify.js"></script>
205+
<script>
206+
window.onload = function () {
207+
prettyPrint();
208+
};
209+
</script>
210+
<script src="sorter.js"></script>
211+
<script src="block-navigation.js"></script>
212+
</body>
213+
</html>
214+

‎coverage/index.html‎

Lines changed: 22 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -23,30 +23,30 @@ <h1>All files</h1>
2323
<div class='clearfix'>
2424

2525
<div class='fl pad1y space-right2'>
26-
<span class="strong">100% </span>
26+
<span class="strong">97.72% </span>
2727
<span class="quiet">Statements</span>
28-
<span class='fraction'>31/31</span>
28+
<span class='fraction'>43/44</span>
2929
</div>
3030

3131

3232
<div class='fl pad1y space-right2'>
33-
<span class="strong">100% </span>
33+
<span class="strong">88.88% </span>
3434
<span class="quiet">Branches</span>
35-
<span class='fraction'>1/1</span>
35+
<span class='fraction'>16/18</span>
3636
</div>
3737

3838

3939
<div class='fl pad1y space-right2'>
4040
<span class="strong">100% </span>
4141
<span class="quiet">Functions</span>
42-
<span class='fraction'>12/12</span>
42+
<span class='fraction'>15/15</span>
4343
</div>
4444

4545

4646
<div class='fl pad1y space-right2'>
47-
<span class="strong">100% </span>
47+
<span class="strong">97.29% </span>
4848
<span class="quiet">Lines</span>
49-
<span class='fraction'>24/24</span>
49+
<span class='fraction'>36/37</span>
5050
</div>
5151

5252

@@ -79,33 +79,33 @@ <h1>All files</h1>
7979
</tr>
8080
</thead>
8181
<tbody><tr>
82-
<td class="file high" data-value="API.ts"><a href="API.ts.html">API.ts</a></td>
83-
<td data-value="100" class="pic high">
84-
<div class="chart"><div class="cover-fill cover-full" style="width: 100%"></div><div class="cover-empty" style="width: 0%"></div></div>
82+
<td class="file high" data-value="languagecheck"><a href="languagecheck/index.html">languagecheck</a></td>
83+
<td data-value="92.3" class="pic high">
84+
<div class="chart"><div class="cover-fill" style="width: 92%"></div><div class="cover-empty" style="width: 8%"></div></div>
8585
</td>
86+
<td data-value="92.3" class="pct high">92.3%</td>
87+
<td data-value="13" class="abs high">12/13</td>
88+
<td data-value="88.23" class="pct high">88.23%</td>
89+
<td data-value="17" class="abs high">15/17</td>
8690
<td data-value="100" class="pct high">100%</td>
87-
<td data-value="15" class="abs high">15/15</td>
88-
<td data-value="100" class="pct high">100%</td>
89-
<td data-value="0" class="abs high">0/0</td>
90-
<td data-value="100" class="pct high">100%</td>
91-
<td data-value="8" class="abs high">8/8</td>
92-
<td data-value="100" class="pct high">100%</td>
93-
<td data-value="13" class="abs high">13/13</td>
91+
<td data-value="3" class="abs high">3/3</td>
92+
<td data-value="92.3" class="pct high">92.3%</td>
93+
<td data-value="13" class="abs high">12/13</td>
9494
</tr>
9595

9696
<tr>
97-
<td class="file high" data-value="utils.ts"><a href="utils.ts.html">utils.ts</a></td>
97+
<td class="file high" data-value="translate"><a href="translate/index.html">translate</a></td>
9898
<td data-value="100" class="pic high">
9999
<div class="chart"><div class="cover-fill cover-full" style="width: 100%"></div><div class="cover-empty" style="width: 0%"></div></div>
100100
</td>
101101
<td data-value="100" class="pct high">100%</td>
102-
<td data-value="16" class="abs high">16/16</td>
102+
<td data-value="31" class="abs high">31/31</td>
103103
<td data-value="100" class="pct high">100%</td>
104104
<td data-value="1" class="abs high">1/1</td>
105105
<td data-value="100" class="pct high">100%</td>
106-
<td data-value="4" class="abs high">4/4</td>
106+
<td data-value="12" class="abs high">12/12</td>
107107
<td data-value="100" class="pct high">100%</td>
108-
<td data-value="11" class="abs high">11/11</td>
108+
<td data-value="24" class="abs high">24/24</td>
109109
</tr>
110110

111111
</tbody>
@@ -116,7 +116,7 @@ <h1>All files</h1>
116116
<div class='footer quiet pad2 space-top1 center small'>
117117
Code coverage generated by
118118
<a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
119-
at 2025-08-02T20:21:56.395Z
119+
at 2026-02-07T20:52:40.279Z
120120
</div>
121121
<script src="prettify.js"></script>
122122
<script>

0 commit comments

Comments
 (0)