chore: refacto content

Author: sebastienfontaine

manifest.json

@@ -41,7 +41,7 @@
   "content_scripts": [{
     "matches": ["<all_urls>"],
     "js": [
-      "src/content.js"
+      "src/content.ts"
     ]
   }],
   "permissions": [

src/background/__tests__/TabRulesService.test.ts

@@ -43,7 +43,7 @@ vi.mock('../../common/storage', () => ({
 
 // Mock helpers module
 vi.mock('../../common/helpers', () => ({
-	_processUrlFragment: vi.fn((fragment, url) => url),
+	_processUrlFragment: vi.fn((_fragment, url) => url),
 }));
 
 import { _getRuleFromUrl, _shouldSkipUrl, _getDefaultRule, _setStorage } from '../../common/storage';
@@ -160,7 +160,16 @@ describe('TabRulesService', () => {
 				name: 'New Title (example.com)',
 				detection: 'CONTAINS',
 				url_fragment: 'https://example.com',
-				tab: { title: 'New Title' },
+				tab: {
+					title: 'New Title',
+					icon: null,
+					muted: false,
+					pinned: false,
+					protected: false,
+					unique: false,
+					title_matcher: null,
+					url_matcher: null,
+				},
 				is_enabled: true,
 			};
 

src/background/__tests__/WindowService.test.ts

@@ -28,7 +28,7 @@ describe('WindowService', () => {
 			const singleWindow = {
 				id: 1,
 				focused: true,
-				type: 'normal' as chrome.windows.WindowType,
+				type: 'normal' as 'normal',
 				tabs: [{ id: 1 }],
 			};
 
@@ -44,14 +44,14 @@ describe('WindowService', () => {
 			const focusedWindow = {
 				id: 1,
 				focused: true,
-				type: 'normal' as chrome.windows.WindowType,
+				type: 'normal' as 'normal',
 				tabs: [{ id: 1 }, { id: 2 }],
 			};
 
 			const otherWindow = {
 				id: 2,
 				focused: false,
-				type: 'normal' as chrome.windows.WindowType,
+				type: 'normal' as 'normal',
 				tabs: [{ id: 3 }, { id: 4 }],
 			};
 
@@ -72,14 +72,14 @@ describe('WindowService', () => {
 			const normalWindow = {
 				id: 1,
 				focused: true,
-				type: 'normal' as chrome.windows.WindowType,
+				type: 'normal' as 'normal',
 				tabs: [{ id: 1 }],
 			};
 
 			const popupWindow = {
 				id: 2,
 				focused: false,
-				type: 'popup' as chrome.windows.WindowType,
+				type: 'popup' as 'popup',
 				tabs: [{ id: 2 }],
 			};
 
@@ -96,14 +96,14 @@ describe('WindowService', () => {
 			const window1 = {
 				id: 1,
 				focused: false,
-				type: 'normal' as chrome.windows.WindowType,
+				type: 'normal' as 'normal',
 				tabs: [{ id: 1 }],
 			};
 
 			const window2 = {
 				id: 2,
 				focused: false,
-				type: 'normal' as chrome.windows.WindowType,
+				type: 'normal' as 'normal',
 				tabs: [{ id: 2 }],
 			};
 
@@ -124,14 +124,14 @@ describe('WindowService', () => {
 			const window1 = {
 				id: 1,
 				focused: true,
-				type: 'normal' as chrome.windows.WindowType,
+				type: 'normal' as 'normal',
 				tabs: [{ id: 1 }],
 			};
 
 			const window2 = {
 				id: 2,
 				focused: false,
-				type: 'normal' as chrome.windows.WindowType,
+				type: 'normal' as 'normal',
 				tabs: [],
 			};
 
@@ -148,14 +148,14 @@ describe('WindowService', () => {
 			const window1 = {
 				id: 1,
 				focused: true,
-				type: 'normal' as chrome.windows.WindowType,
+				type: 'normal' as 'normal',
 				tabs: [{ id: 1 }],
 			};
 
 			const window2 = {
 				id: 2,
 				focused: false,
-				type: 'normal' as chrome.windows.WindowType,
+				type: 'normal' as 'normal',
 				tabs: [{ id: undefined }, { id: 2 }],
 			};
 
@@ -176,14 +176,14 @@ describe('WindowService', () => {
 			const window1 = {
 				id: 1,
 				focused: true,
-				type: 'normal' as chrome.windows.WindowType,
+				type: 'normal' as 'normal',
 				tabs: [{ id: 1 }],
 			};
 
 			const window2 = {
 				id: 2,
 				focused: false,
-				type: 'normal' as chrome.windows.WindowType,
+				type: 'normal' as 'normal',
 				tabs: [{ id: 2 }],
 			};
 

src/components/options/center/sections/TabRules/EmptyRules.test.js

@@ -1,11 +1,21 @@
 import { beforeEach, describe, expect, it } from 'vitest';
-import { processTitle } from '../../../../../content.js';
+import { TitleService } from '../../../../../content/TitleService';
+import { RegexService } from '../../../../../content/RegexService';
 
 describe('Sample Rules', () => {
+	let titleService;
+
 	beforeEach(() => {
 		document.body.innerHTML = '';
+		const regexService = new RegexService();
+		titleService = new TitleService(regexService);
 	});
 
+	// Helper function to match the old processTitle API
+	const processTitle = (url, title, rule) => {
+		return titleService.processTitle(url, title, rule);
+	};
+
 	describe('Gmail Rule', () => {
 		it('should preserve title for Gmail', () => {
 			const rule = {