Merge pull request #2 from onyxdevs/cursor/implement-mac-desktop-application-1815

Implement mac desktop application

Author: onyxdevs

.prettierrc

@@ -1,11 +1,22 @@
 {
-    "semi": true,
-    "trailingComma": "es5",
-    "singleQuote": true,
-    "printWidth": 100,
+    "printWidth": 120,
     "tabWidth": 4,
     "useTabs": false,
-    "bracketSpacing": true,
-    "arrowParens": "always",
-    "endOfLine": "lf"
+    "semi": true,
+    "singleQuote": true,
+    "trailingComma": "all",
+    "overrides": [
+        {
+            "files": "*.yaml",
+            "options": {
+                "tabWidth": 4
+            }
+        },
+        {
+            "files": "*.yml",
+            "options": {
+                "tabWidth": 4
+            }
+        }
+    ]
 }

.yarn/install-state.gz

@@ -180,7 +180,53 @@ module.exports = [
         },
     },
     {
+        // Browser/Renderer JavaScript files (Electron renderer process)
+        files: ['packages/mac/src/renderer/**/*.js'],
+        languageOptions: {
+            ecmaVersion: 2020,
+            sourceType: 'script', // Renderer files use script mode
+            globals: {
+                // Browser globals
+                window: 'readonly',
+                document: 'readonly',
+                console: 'readonly',
+                setTimeout: 'readonly',
+                clearTimeout: 'readonly',
+                setInterval: 'readonly',
+                clearInterval: 'readonly',
+                alert: 'readonly',
+                confirm: 'readonly',
+                prompt: 'readonly',
+                // DOM globals
+                Element: 'readonly',
+                HTMLElement: 'readonly',
+                Event: 'readonly',
+                MouseEvent: 'readonly',
+                KeyboardEvent: 'readonly',
+                // Additional browser APIs
+                fetch: 'readonly',
+                localStorage: 'readonly',
+                sessionStorage: 'readonly',
+            },
+        },
+        plugins: {
+            prettier,
+        },
+        rules: {
+            'prettier/prettier': [
+                'error',
+                {
+                    tabWidth: 4,
+                },
+            ],
+            // Relax some rules for renderer files
+            'no-unused-vars': ['warn', { argsIgnorePattern: '^_' }],
+        },
+    },
+    {
+        // Node.js JavaScript files
         files: ['**/*.js', '**/*.mjs', '**/*.cjs'],
+        ignores: ['packages/mac/src/renderer/**/*.js'], // Exclude renderer files
         languageOptions: {
             ecmaVersion: 2020,
             sourceType: 'module',

eslint.config.js

@@ -45,11 +45,7 @@ module.exports = {
     ],
 
     // Global settings
-    collectCoverageFrom: [
-        'packages/*/src/**/*.ts',
-        '!packages/*/src/**/*.d.ts',
-        '!packages/*/src/**/index.ts',
-    ],
+    collectCoverageFrom: ['packages/*/src/**/*.ts', '!packages/*/src/**/*.d.ts', '!packages/*/src/**/index.ts'],
 
     coverageDirectory: 'coverage',
     coverageReporters: ['text', 'lcov', 'html'],

jest.config.js

@@ -37,12 +37,10 @@ describe('ConvertCommandCreator', () => {
 
         // Setup mocks
         const { SubtitleProcessor, ConfigManager } = require('@subzilla/core');
-        const mockProcessFile = jest
-            .fn<() => Promise<{ outputPath: string; backupPath: string }>>()
-            .mockResolvedValue({
-                outputPath: '/mock/output.srt',
-                backupPath: '/mock/backup.srt',
-            });
+        const mockProcessFile = jest.fn<() => Promise<{ outputPath: string; backupPath: string }>>().mockResolvedValue({
+            outputPath: '/mock/output.srt',
+            backupPath: '/mock/backup.srt',
+        });
 
         (SubtitleProcessor as any).mockImplementation(() => ({
             processFile: mockProcessFile,
@@ -152,7 +150,7 @@ describe('ConvertCommandCreator', () => {
                         html: true,
                         colors: true,
                     }),
-                })
+                }),
             );
         });
 
@@ -169,7 +167,7 @@ describe('ConvertCommandCreator', () => {
                 expect.objectContaining({
                     retryCount: 3,
                     retryDelay: 2000,
-                })
+                }),
             );
         });
 
@@ -190,7 +188,7 @@ describe('ConvertCommandCreator', () => {
                     overwriteInput: true,
                     overwriteExisting: true,
                     overwriteBackup: false,
-                })
+                }),
             );
         });
 
@@ -213,7 +211,7 @@ describe('ConvertCommandCreator', () => {
                 expect.objectContaining({
                     backupOriginal: true,
                     bom: false,
-                })
+                }),
             );
         });
 
@@ -242,7 +240,7 @@ describe('ConvertCommandCreator', () => {
                 '🧬 Output options:',
                 expect.objectContaining({
                     lineEndings: 'lf',
-                })
+                }),
             );
         });
     });

packages/cli/__tests__/commands/convert-command.test.ts

@@ -24,13 +24,11 @@ export class BatchCommandCreator extends BaseCommandCreator<IBatchCommandOptions
                         common: {
                             strip: createStripOptions(options, config),
                             backupOriginal: options.backup ?? config.output?.createBackup,
-                            overwriteBackup:
-                                options.overwriteBackup ?? config.output?.overwriteBackup,
+                            overwriteBackup: options.overwriteBackup ?? config.output?.overwriteBackup,
                             bom: options.bom ?? config.output?.bom,
                             lineEndings: options.lineEndings ?? config.output?.lineEndings,
                             overwriteInput: options.overwriteInput ?? config.output?.overwriteInput,
-                            overwriteExisting:
-                                options.overwriteExisting ?? config.output?.overwriteExisting,
+                            overwriteExisting: options.overwriteExisting ?? config.output?.overwriteExisting,
                             retryCount: options.retryCount
                                 ? parseInt(options.retryCount, 10)
                                 : config.batch?.retryCount,
@@ -43,19 +41,11 @@ export class BatchCommandCreator extends BaseCommandCreator<IBatchCommandOptions
                             outputDir: options.outputDir ?? config.output?.directory,
                             recursive: options.recursive ?? config.batch?.recursive ?? false,
                             parallel: options.parallel ?? config.batch?.parallel ?? false,
-                            skipExisting:
-                                options.skipExisting ?? config.batch?.skipExisting ?? false,
-                            maxDepth: options.maxDepth
-                                ? parseInt(options.maxDepth, 10)
-                                : config.batch?.maxDepth,
-                            includeDirectories:
-                                options.includeDirs ?? config.batch?.includeDirectories,
-                            excludeDirectories:
-                                options.excludeDirs ?? config.batch?.excludeDirectories,
-                            preserveStructure:
-                                options.preserveStructure ??
-                                config.batch?.preserveStructure ??
-                                false,
+                            skipExisting: options.skipExisting ?? config.batch?.skipExisting ?? false,
+                            maxDepth: options.maxDepth ? parseInt(options.maxDepth, 10) : config.batch?.maxDepth,
+                            includeDirectories: options.includeDirs ?? config.batch?.includeDirectories,
+                            excludeDirectories: options.excludeDirs ?? config.batch?.excludeDirectories,
+                            preserveStructure: options.preserveStructure ?? config.batch?.preserveStructure ?? false,
                             chunkSize: options.chunkSize ?? config.batch?.chunkSize,
                         },
                     };

packages/cli/src/commands/batch-command.ts

@@ -27,14 +27,9 @@ export class ConvertCommandCreator extends BaseCommandCreator<IConvertCommandOpt
                         bom: options.bom ?? config.output?.bom,
                         lineEndings: options.lineEndings ?? config.output?.lineEndings,
                         overwriteInput: options.overwriteInput ?? config.output?.overwriteInput,
-                        overwriteExisting:
-                            options.overwriteExisting ?? config.output?.overwriteExisting,
-                        retryCount: options.retryCount
-                            ? parseInt(options.retryCount, 10)
-                            : config.batch?.retryCount,
-                        retryDelay: options.retryDelay
-                            ? parseInt(options.retryDelay, 10)
-                            : config.batch?.retryDelay,
+                        overwriteExisting: options.overwriteExisting ?? config.output?.overwriteExisting,
+                        retryCount: options.retryCount ? parseInt(options.retryCount, 10) : config.batch?.retryCount,
+                        retryDelay: options.retryDelay ? parseInt(options.retryDelay, 10) : config.batch?.retryDelay,
                     };
 
                     console.log('🧬 Output options:', outputOptions);
@@ -45,9 +40,7 @@ export class ConvertCommandCreator extends BaseCommandCreator<IConvertCommandOpt
 
                     console.log('✨ Conversion successful!');
                     console.log(`Input file: ${inputFile}`);
-                    console.log(
-                        `Output file: ${options.output || this.getDefaultOutputPath(inputFile)}`
-                    );
+                    console.log(`Output file: ${options.output || this.getDefaultOutputPath(inputFile)}`);
 
                     if (options.backup || config.output?.createBackup) {
                         console.log(`Backup file: ${inputFile}.bak`);

packages/cli/src/commands/convert-command.ts

@@ -29,18 +29,15 @@ export class InfoCommandCreator extends BaseCommandCreator<IInfoCommandOptions>
                     const fileBuffer = await fs.readFile(inputFile);
 
                     // Detect encoding
-                    const detectedEncoding =
-                        await EncodingDetectionService.detectEncoding(inputFile);
+                    const detectedEncoding = await EncodingDetectionService.detectEncoding(inputFile);
 
                     // Check for BOM
                     const hasBOM = fileBuffer.slice(0, 3).equals(Buffer.from([0xef, 0xbb, 0xbf]));
 
                     // Count lines and entries
                     const content = fileBuffer.toString(detectedEncoding as NodeJS.BufferEncoding);
                     const lines = content.split(/\r?\n/);
-                    const entries = content
-                        .split(/\r?\n\r?\n/)
-                        .filter((entry) => entry.trim()).length;
+                    const entries = content.split(/\r?\n\r?\n/).filter((entry) => entry.trim()).length;
 
                     // Detect line endings
                     const hasCarriageReturn = content.includes('\r\n');
@@ -63,10 +60,7 @@ export class InfoCommandCreator extends BaseCommandCreator<IInfoCommandOptions>
                     console.log(`   • Total Lines: ${chalk.yellow(lines.length)}`);
                     console.log(`   • Subtitle Entries: ${chalk.yellow(entries)}`);
                 } catch (error) {
-                    console.error(
-                        chalk.red('❌ Error analyzing subtitle file:'),
-                        (error as Error).message
-                    );
+                    console.error(chalk.red('❌ Error analyzing subtitle file:'), (error as Error).message);
                     process.exit(1);
                 }
             },

packages/cli/src/commands/info-command.ts

@@ -1,9 +1,6 @@
 import { IStripOptions, IStripCommandOptions, IConfig } from '@subzilla/types';
 
-export function createStripOptions(
-    options: IStripCommandOptions,
-    config: IConfig
-): IStripOptions | undefined {
+export function createStripOptions(options: IStripCommandOptions, config: IConfig): IStripOptions | undefined {
     const stripOptions: IStripOptions = options.stripAll
         ? {
               html: true,

packages/cli/src/utils/strip-options.ts

@@ -52,8 +52,8 @@ describe('EncodingConversionService', () => {
             const content = 'Smart quotes: "Hello"';
             // Create buffer with Windows-1252 specific characters
             const buffer = Buffer.from([
-                0x53, 0x6d, 0x61, 0x72, 0x74, 0x20, 0x71, 0x75, 0x6f, 0x74, 0x65, 0x73, 0x3a, 0x20,
-                0x93, 0x48, 0x65, 0x6c, 0x6c, 0x6f, 0x94,
+                0x53, 0x6d, 0x61, 0x72, 0x74, 0x20, 0x71, 0x75, 0x6f, 0x74, 0x65, 0x73, 0x3a, 0x20, 0x93, 0x48, 0x65,
+                0x6c, 0x6c, 0x6f, 0x94,
             ]);
 
             const result = EncodingConversionService.convertToUtf8(buffer, 'windows-1252');