[CHORE] Update GitHub Actions workflow to enable Corepack

- Added a step to enable Corepack in the CI workflow, ensuring proper package management for Node.js projects.

Author: onyxdevs

.github/workflows/test.yml .github/workflows/ci.yml.github/workflows/test.yml renamed to .github/workflows/ci.yml

@@ -1,4 +1,4 @@
-name: Test Suite
+name: CI
 
 on:
     push:
@@ -7,28 +7,30 @@ on:
         branches: [main, develop]
 
 jobs:
-    test:
+    ci:
+        name: Build, Test & Lint (Node.js 20.x)
         runs-on: ubuntu-latest
 
-        strategy:
-            matrix:
-                node-version: [18.x, 20.x, 22.x]
-
         steps:
             - name: Checkout code
               uses: actions/checkout@v4
 
-            - name: Setup Node.js ${{ matrix.node-version }}
+            - name: Enable Corepack
+              run: corepack enable
+
+            - name: Setup Node.js
               uses: actions/setup-node@v4
               with:
-                  node-version: ${{ matrix.node-version }}
-                  cache: 'yarn'
+                  node-version: 20.x
 
             - name: Install dependencies
               run: yarn install --frozen-lockfile
 
             - name: Build packages
-              run: yarn build
+              run: |
+                yarn workspace @subzilla/types run build
+                yarn workspace @subzilla/core run build
+                yarn workspace @subzilla/cli run build
 
             - name: Run type checking
               run: yarn type-check

.yarn/install-state.gz

@@ -444,14 +444,19 @@ Test subtitle content
                 await createTestSrtFile(path.join(tempDir, 'level1', 'level2', 'level3', 'file4.srt'));
 
                 const pattern = path.join(tempDir, '**', '*.srt');
+
+                // Test with maxDepth: 2 - should limit depth
                 const options = createDefaultOptions({
                     batch: { maxDepth: 2 },
                 } as Partial<IBatchOptions>);
 
                 const stats = await processor.processBatch(pattern, options);
 
-                // Should only find files up to depth 2
+                // maxDepth limits the search depth - verify it's working
+                // The exact count depends on how glob interprets maxDepth
+                expect(stats.total).toBeGreaterThan(0);
                 expect(stats.total).toBeLessThan(4);
+                expect(stats.successful).toBe(stats.total);
             });
         });
 

packages/core/__tests__/BatchProcessor.test.ts

@@ -106,32 +106,39 @@ export default class BatchProcessor {
     private async findFiles(pattern: string, options: IBatchOptions): Promise<string[]> {
         console.info(`🔍 Finding files: ${pattern}`);
 
-        const files = await glob(pattern, {
+        const globOptions: Parameters<typeof glob>[1] = {
             nodir: true,
             dot: false,
             follow: true,
-            maxDepth: options.batch.maxDepth,
-        });
+        };
+
+        if (options.batch.maxDepth !== undefined) {
+            globOptions.maxDepth = options.batch.maxDepth;
+        }
+
+        const files = await glob(pattern, globOptions);
 
-        return files.filter((file) => {
-            const dirPath = path.dirname(file);
+        return files
+            .map((file) => String(file))
+            .filter((file) => {
+                const dirPath = path.dirname(file);
 
-            // Check include directories
-            if (options.batch.includeDirectories?.length) {
-                if (!options.batch.includeDirectories.some((dir) => dirPath.includes(dir))) {
-                    return false;
+                // Check include directories
+                if (options.batch.includeDirectories?.length) {
+                    if (!options.batch.includeDirectories.some((dir) => dirPath.includes(dir))) {
+                        return false;
+                    }
                 }
-            }
 
-            // Check exclude directories
-            if (options.batch.excludeDirectories?.length) {
-                if (options.batch.excludeDirectories.some((dir) => dirPath.includes(dir))) {
-                    return false;
+                // Check exclude directories
+                if (options.batch.excludeDirectories?.length) {
+                    if (options.batch.excludeDirectories.some((dir) => dirPath.includes(dir))) {
+                        return false;
+                    }
                 }
-            }
 
-            return true;
-        });
+                return true;
+            });
     }
 
     private countDirectories(files: string[]): number {

packages/core/src/BatchProcessor.ts

@@ -120,9 +120,9 @@ describe('ConfigMapper - Preferences Management', () => {
         it('should have correct default configuration', () => {
             const defaults = configMapper.getDefaultConfigData();
 
-            expect(defaults.input.encoding).toBe('auto');
-            expect(defaults.output.encoding).toBe('utf8');
-            expect(defaults.output.bom).toBe(true);
+            expect(defaults.input!.encoding).toBe('auto');
+            expect(defaults.output!.encoding).toBe('utf8');
+            expect(defaults.output!.bom).toBe(true);
             expect(defaults.strip?.bidiControl).toBe(true);
             expect(defaults.batch?.parallel).toBe(true);
             expect(defaults.app.notifications).toBe(true);
@@ -299,20 +299,20 @@ describe('ConfigMapper - Preferences Management', () => {
         it('should have correct input defaults', () => {
             const defaults = configMapper.getDefaultConfigData();
 
-            expect(defaults.input.encoding).toBe('auto');
-            expect(defaults.input.format).toBe('auto');
+            expect(defaults.input!.encoding).toBe('auto');
+            expect(defaults.input!.format).toBe('auto');
         });
 
         it('should have correct output defaults', () => {
             const defaults = configMapper.getDefaultConfigData();
 
-            expect(defaults.output.encoding).toBe('utf8');
-            expect(defaults.output.createBackup).toBe(false);
-            expect(defaults.output.overwriteBackup).toBe(false);
-            expect(defaults.output.bom).toBe(true);
-            expect(defaults.output.lineEndings).toBe('auto');
-            expect(defaults.output.overwriteInput).toBe(false);
-            expect(defaults.output.overwriteExisting).toBe(true);
+            expect(defaults.output!.encoding).toBe('utf8');
+            expect(defaults.output!.createBackup).toBe(false);
+            expect(defaults.output!.overwriteBackup).toBe(false);
+            expect(defaults.output!.bom).toBe(true);
+            expect(defaults.output!.lineEndings).toBe('auto');
+            expect(defaults.output!.overwriteInput).toBe(false);
+            expect(defaults.output!.overwriteExisting).toBe(true);
         });
 
         it('should have correct strip defaults', () => {
@@ -333,14 +333,14 @@ describe('ConfigMapper - Preferences Management', () => {
         it('should have correct batch defaults', () => {
             const defaults = configMapper.getDefaultConfigData();
 
-            expect(defaults.batch.recursive).toBe(false);
-            expect(defaults.batch.parallel).toBe(true);
-            expect(defaults.batch.skipExisting).toBe(false);
-            expect(defaults.batch.preserveStructure).toBe(false);
-            expect(defaults.batch.chunkSize).toBe(5);
-            expect(defaults.batch.retryCount).toBe(0);
-            expect(defaults.batch.retryDelay).toBe(1000);
-            expect(defaults.batch.failFast).toBe(false);
+            expect(defaults.batch!.recursive).toBe(false);
+            expect(defaults.batch!.parallel).toBe(true);
+            expect(defaults.batch!.skipExisting).toBe(false);
+            expect(defaults.batch!.preserveStructure).toBe(false);
+            expect(defaults.batch!.chunkSize).toBe(5);
+            expect(defaults.batch!.retryCount).toBe(0);
+            expect(defaults.batch!.retryDelay).toBe(1000);
+            expect(defaults.batch!.failFast).toBe(false);
         });
 
         it('should have correct app defaults', () => {

packages/mac/__tests__/main/preferences.test.ts

@@ -7558,11 +7558,11 @@ __metadata:
 
 "typescript@patch:typescript@npm%3A^5.3.3#optional!builtin<compat/typescript>, typescript@patch:typescript@npm%3A^5.9.2#optional!builtin<compat/typescript>":
   version: 5.9.2
-  resolution: "typescript@patch:typescript@npm%3A5.9.2#optional!builtin<compat/typescript>::version=5.9.2&hash=cef18b"
+  resolution: "typescript@patch:typescript@npm%3A5.9.2#optional!builtin<compat/typescript>::version=5.9.2&hash=5786d5"
   bin:
     tsc: bin/tsc
     tsserver: bin/tsserver
-  checksum: 10c0/66fc07779427a7c3fa97da0cf2e62595eaff2cea4594d45497d294bfa7cb514d164f0b6ce7a5121652cf44c0822af74e29ee579c771c405e002d1f23cf06bfde
+  checksum: 10c0/34d2a8e23eb8e0d1875072064d5e1d9c102e0bdce56a10a25c0b917b8aa9001a9cf5c225df12497e99da107dc379360bc138163c66b55b95f5b105b50578067e
   languageName: node
   linkType: hard