update comments, remove redundant test

Author: yehorkardash

src/__tests__/unit/checks/keywords-urls.test.ts

@@ -33,11 +33,6 @@ describe('keywords guardrail', () => {
     expect(result.info?.matchedKeywords).toEqual([]);
   });
 
-  it('should return the correct result', async () => {
-		const result = await keywordsCheck({}, 'Hello, world!', KeywordsConfig.parse({ keywords: ['hello', 'world'] }));
-		expect(result.tripwireTriggered).toEqual(true);
-	});
-
 	it('should not match partial words', async () => {
 		const result = await keywordsCheck({}, 'Hello, world!', KeywordsConfig.parse({ keywords: ['orld'] }));
 		expect(result.tripwireTriggered).toEqual(false);

src/checks/keywords.ts

@@ -59,10 +59,10 @@ export const keywordsCheck: CheckFn<KeywordsContext, string, KeywordsConfig> = (
   );
 
   // \p{L}|\p{N}|_ - any unicode letter, number, or underscore. Alternative to \b
-	// (?<!\p{L}) - not preceded by a letter
-	// (?!\p{L}) - not followed by a letter
-	const patternText = `(?<!\\p{L}|\\p{N}|_)(?:${escapedKeywords.join('|')})(?!\\p{L}|\\p{N}|_)`;
-	const pattern = new RegExp(patternText, 'giu'); // case-insensitive, global, unicode
+  // (?<!\\p{L}|\\p{N}|_) - not preceded by a letter
+  // (?!\\p{L}|\\p{N}|_) - not followed by a letter
+  const patternText = `(?<!\\p{L}|\\p{N}|_)(?:${escapedKeywords.join('|')})(?!\\p{L}|\\p{N}|_)`;
+  const pattern = new RegExp(patternText, 'giu'); // case-insensitive, global, unicode
 
   const matches: string[] = [];
   let match;