More translations

Author: Hermesiss

.github/workflows/test.yml

@@ -14,8 +14,6 @@ jobs:
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4
-        with:
-          fetch-depth: 0 # Shallow clones should be disabled for better relevancy of analysis
 
       - name: Setup Node.js
         uses: actions/setup-node@v4
@@ -26,21 +24,8 @@ jobs:
       - name: Install dependencies
         run: npm ci
 
-      - name: Run tests with coverage
-        run: npm run test:coverage
-
-      - name: SonarQube Scan
-        uses: SonarSource/sonarqube-scan-action@master
-        env:
-          SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
-          SONAR_HOST_URL: ${{ secrets.SONAR_HOST_URL }}
-
-      - name: Upload coverage reports
-        uses: actions/upload-artifact@v4
-        with:
-          name: coverage-report
-          path: coverage/
-          retention-days: 30
+      - name: Run tests
+        run: npm test
 
   notify:
     name: Notify on Failure

.husky/pre-commit

@@ -0,0 +1 @@
+npm run type-check && npm test

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "devnullifier",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "DevNullifier: Clean application data, dev caches (node_modules, .cache, Library, Binary, Intermediate, etc) with Electron, Vue 3, and Vuetify 3.",
   "main": "dist-main/main/main.js",
   "scripts": {
@@ -24,7 +24,8 @@
     "test:coverage": "npm run test:coverage:renderer && npm run test:coverage:main",
     "test:coverage:renderer": "vitest run --coverage",
     "test:coverage:main": "vitest run -c vitest.main.config.ts --coverage",
-    "release": "electron-builder --publish always"
+    "release": "electron-builder --publish always",
+    "prepare": "husky"
   },
   "keywords": [
     "electron",
@@ -61,6 +62,7 @@
     "cross-env": "^7.0.3",
     "electron": "^37.2.0",
     "electron-builder": "^24.0.0",
+    "husky": "^9.1.7",
     "jsdom": "^26.1.0",
     "typescript": "^5.8.3",
     "vite": "^5.0.0",

package.json

@@ -9,13 +9,14 @@ describe("getDirectorySize", () => {
 
   beforeEach(async () => {
     // Create a temporary directory
-    tempDir = path.join(os.tmpdir(), `test-${Date.now()}`);
+    tempDir = path.join(os.tmpdir(), `test-dirsize-${Date.now()}-${Math.floor(Math.random() * 1000)}`);
     if (existsSync(tempDir)) {
       await fs.rm(tempDir, { recursive: true, force: true });
     }
-    await fs.mkdir(tempDir);
+    await fs.mkdir(tempDir, { recursive: true });
   });
 
+
   afterEach(async () => {
     // Clean up - remove temp directory and all contents
     try {

src/main/__tests__/dirSize.test.ts

@@ -26,20 +26,28 @@ vi.mock("electron", () => ({
 
 describe("fileUtils", () => {
   let tempDir: string;
-  const mockUserDataPath = path.join(os.tmpdir(), `test-userdata-${Date.now()}`);
+  let mockUserDataPath: string;
 
   beforeEach(async () => {
     vi.clearAllMocks();
-    // Create temporary directories
-    tempDir = path.join(os.tmpdir(), `test-${Date.now()}`);
+    // Create temporary directories with unique names
+    const suffix = `${Date.now()}-${Math.floor(Math.random() * 1000)}`;
+    tempDir = path.join(os.tmpdir(), `test-fileutils-${suffix}`);
+    mockUserDataPath = path.join(os.tmpdir(), `test-userdata-${suffix}`);
+    
     if (existsSync(tempDir)) {
       await fs.rm(tempDir, { recursive: true, force: true });
     }
+    if (existsSync(mockUserDataPath)) {
+      await fs.rm(mockUserDataPath, { recursive: true, force: true });
+    }
+    
     await fs.mkdir(tempDir, { recursive: true });
     await fs.mkdir(mockUserDataPath, { recursive: true });
     setUserDataPath(mockUserDataPath);
   });
 
+
   afterEach(async () => {
     // Clean up - remove temp directories and all contents
     try {

src/main/__tests__/fileUtils.test.ts

@@ -139,16 +139,22 @@ export async function getDirectorySize(dirPath: string): Promise<number> {
     const entries = await fs.readdir(dirPath, { withFileTypes: true });
     for (const entry of entries) {
       const fullPath = path.join(dirPath, entry.name);
-      if (entry.isDirectory()) {
-        size += await getDirectorySize(fullPath);
-      } else {
-        const stats = await fs.stat(fullPath);
-        size += stats.size;
+      try {
+        if (entry.isDirectory()) {
+          size += await getDirectorySize(fullPath);
+        } else {
+          const stats = await fs.stat(fullPath);
+          size += stats.size;
+        }
+      } catch (err) {
+        // Ignore errors for individual files/directories (e.g., permission denied, busy)
+        console.warn(`Error accessing ${fullPath}:`, err);
       }
     }
   } catch (err) {
-    // Ignore errors (e.g., permission denied) and return size of what we can access.
-    console.log("Error getting directory size:", err);
+    // Ignore errors for readdir (e.g., directory deleted while reading)
+    console.warn(`Error reading directory ${dirPath}:`, err);
   }
   return size;
 }
+

src/main/appDataCleaner.ts

@@ -77,7 +77,7 @@ const initialCategories: DeveloperCategory[] = [
         detectionFiles: ['main.py', 'requirements.txt', 'pyproject.toml', '.venv/', '*.py'],
         cachePatterns: ['__pycache__/', '.pytest_cache/', '.mypy_cache/', '.tox/', '.hypothesis/', '.coverage', 'build/', 'dist/'],
         warning: true,
-        warningText: 'venv/ might be the main virtual environment – be careful if no lockfile exists'
+        warningText: ''
     },
     {
         id: 'nodejs',
@@ -86,7 +86,7 @@ const initialCategories: DeveloperCategory[] = [
         detectionFiles: ['package.json', 'yarn.lock', 'vite.config.js', 'next.config.js', 'tsconfig.json'],
         cachePatterns: ['node_modules/', 'dist/', 'build/', '.next/', '.nuxt/', '.angular/', '.vite/', '.turbo/', '.expo/', '.parcel-cache/', '.eslintcache', '.cache/'],
         warning: true,
-        warningText: 'node_modules/ may exist in subfolders if using monorepos'
+        warningText: ''
     },
     {
         id: 'unity',
@@ -95,7 +95,7 @@ const initialCategories: DeveloperCategory[] = [
         detectionFiles: ['Assets/', 'ProjectSettings/', 'Packages/', '*.unity'],
         cachePatterns: ['Library/', 'Temp/', 'Obj/', 'Build/', 'Builds/', '.vs/', 'Logs/', 'MemoryCaptures/', 'UserSettings/'],
         warning: true,
-        warningText: 'Library/ is only needed at the Unity project root — safe elsewhere'
+        warningText: ''
     },
     {
         id: 'unreal',
@@ -104,7 +104,7 @@ const initialCategories: DeveloperCategory[] = [
         detectionFiles: ['*.uproject', 'Source/', 'Content/'],
         cachePatterns: ['Binaries/', 'Build/', 'Intermediate/', 'Saved/', 'DerivedDataCache/', 'Plugins/**/Intermediate/', 'Plugins/**/Binaries/'],
         warning: true,
-        warningText: 'Saved/Config/ might contain useful settings — review before deleting'
+        warningText: ''
     },
     {
         id: 'rust',
@@ -121,7 +121,7 @@ const initialCategories: DeveloperCategory[] = [
         detectionFiles: ['build.gradle', 'pom.xml', 'AndroidManifest.xml', 'src/main/java/'],
         cachePatterns: ['build/', 'out/', '.gradle/', '.idea/', 'target/', '.settings/'],
         warning: true,
-        warningText: '.idea/ may contain custom settings — remove only cache/temp files inside'
+        warningText: ''
     },
     {
         id: 'dotnet',
@@ -138,7 +138,7 @@ const initialCategories: DeveloperCategory[] = [
         detectionFiles: ['CMakeLists.txt', 'Makefile', '.vscode/launch.json', '*.cpp', '*.h'],
         cachePatterns: ['build/', 'CMakeFiles/', 'cmake-build-debug/', 'cmake-build-release/', 'Debug/', 'Release/'],
         warning: true,
-        warningText: 'build/ might be used manually — check first'
+        warningText: ''
     },
     {
         id: 'xcode',
@@ -147,7 +147,7 @@ const initialCategories: DeveloperCategory[] = [
         detectionFiles: ['*.xcodeproj', 'Info.plist', 'Podfile', '*.swift'],
         cachePatterns: ['DerivedData/', 'build/', '*.xcuserdata/', '*.xcuserdatad/'],
         warning: true,
-        warningText: 'Global DerivedData/ lives in ~/Library/Developer/Xcode/DerivedData'
+        warningText: ''
     },
     {
         id: 'php',
@@ -156,7 +156,7 @@ const initialCategories: DeveloperCategory[] = [
         detectionFiles: ['artisan', 'composer.json', 'routes/web.php', 'app/'],
         cachePatterns: ['vendor/', 'bootstrap/cache/', 'storage/framework/cache/', '.phpunit.result.cache'],
         warning: true,
-        warningText: 'Only delete vendor/ if composer.lock is present'
+        warningText: ''
     },
     {
         id: 'symfony',
@@ -173,7 +173,7 @@ const initialCategories: DeveloperCategory[] = [
         detectionFiles: ['.ipynb', 'notebooks/', 'train.py', 'requirements.txt', 'wandb/'],
         cachePatterns: ['checkpoints/', 'runs/', 'logs/', '.ipynb_checkpoints/', '.cache/', 'wandb/', 'mlruns/'],
         warning: true,
-        warningText: "Don't delete checkpoints unless you're sure they're not needed"
+        warningText: ''
     },
     {
         id: 'docker',
@@ -206,7 +206,7 @@ const initialCategories: DeveloperCategory[] = [
         detectionFiles: ['.idea/', '.vscode/', '.history/', '*.sublime-*', '.DS_Store'],
         cachePatterns: ['.idea/workspace.xml', '.idea/caches/', '.vscode/ipch/', '.DS_Store', 'Thumbs.db'],
         warning: true,
-        warningText: "Don't remove .idea/modules.xml or .vscode/settings.json if project-specific"
+        warningText: ''
     }
 ]
 

src/renderer/components/DeveloperCleaner.vue

@@ -36,9 +36,9 @@
                     </div>
                     <div :class="secondaryTextClass">
                         {{ cacheGroup.matches.length }}
-                        {{ cacheGroup.matches.length === 1 ? 'match' : 'matches' }} •
-                        {{ formatSize(cacheGroup.selectedSize) }} of {{
-                            formatSize(cacheGroup.totalSize) }} selected
+                        {{ cacheGroup.matches.length === 1 ? t('common.match') : t('common.matches') }} •
+                        {{ formatSize(cacheGroup.selectedSize) }} {{ t('common.of') }} {{
+                            formatSize(cacheGroup.totalSize) }} {{ t('common.selected') }}
                     </div>
                 </div>
 
@@ -55,16 +55,16 @@
                     <div class="d-flex justify-space-between align-center mb-2">
                         <v-btn size="x-small" variant="text" @click="selectAllCachesInGroup(cacheGroup)"
                             :disabled="allGroupCachesSelected(cacheGroup)">
-                            Select All
+                            {{ t('common.select_all') }}
                         </v-btn>
                         <v-btn size="x-small" variant="text" @click="deselectAllCachesInGroup(cacheGroup)"
                             :disabled="noGroupCachesSelected(cacheGroup)">
-                            Deselect All
+                            {{ t('common.deselect_all') }}
                         </v-btn>
                         <v-chip size="x-small" color="primary">
                             {{cacheGroup.matches.filter(m => m.selected).length}}/{{
                                 cacheGroup.matches.length }}
-                            selected
+                            {{ t('common.selected') }}
                         </v-chip>
                     </div>
 
@@ -89,26 +89,30 @@
         <v-divider v-if="cacheGroups.length > 1" class="mt-2 mb-1" />
         <div class="d-flex justify-space-between align-center mt-1">
             <v-btn size="x-small" variant="text" @click="selectAllCaches" :disabled="allCachesSelected">
-                Select All
+                {{ t('common.select_all') }}
             </v-btn>
             <v-btn size="x-small" variant="text" @click="deselectAllCaches" :disabled="noCachesSelected">
-                Deselect All
+                {{ t('common.deselect_all') }}
             </v-btn>
             <v-chip size="x-small" color="primary">
-                {{ selectedCacheCount }} of {{ totalCacheCount }}
-                selected
+                {{ selectedCacheCount }} {{ t('common.of') }} {{ totalCacheCount }}
+                {{ t('common.selected') }}
             </v-chip>
         </div>
     </div>
 </template>
 
 <script setup lang="ts">
 import { computed } from 'vue'
+import { useI18n } from 'vue-i18n'
 import { useTheme } from 'vuetify'
 import type { CacheGroup } from '@/types'
 import { getTypeColor } from '@/utils/categoryColors'
 import { formatSize } from '@/utils/formatters'
 
+// i18n
+const { t } = useI18n()
+
 // Theme
 const theme = useTheme()
 

src/renderer/components/DeveloperCleaner/CacheGroup.vue

@@ -6,7 +6,8 @@
                     <v-icon :class="{ 'rotate-90': showCategories }" class="mr-2 transition-transform">
                         mdi-chevron-right
                     </v-icon>
-                    {{ t('common.dev_categories') }} ({{ enabledCount }}/{{ categories.length }} {{ t('common.enabled') }})
+                    {{ t('common.dev_categories') }} ({{ enabledCount }}/{{ categories.length }} {{ t('common.enabled')
+                    }})
                     <v-spacer />
                     <v-chip size="small" v-if="!hasEnabledCategories"
                         :color="hasEnabledCategories ? 'success' : 'warning'">
@@ -39,7 +40,9 @@
                                         <v-checkbox v-model="category.enabled" color="primary" hide-details
                                             density="compact" @change="saveCategoryStates" />
                                         <div class="flex-grow-1 ml-1 d-flex align-center justify-space-between">
-                                            <span class="text-caption font-weight-medium">{{ category.name }}</span>
+                                            <span class="text-caption font-weight-medium">
+                                                {{ t('common.category_names.' + category.id) }}
+                                            </span>
                                             <v-btn icon variant="text" size="x-small"
                                                 @click.stop="showCategoryInfo(category)">
                                                 <v-icon size="small" :color="category.warning ? 'warning' : 'info'">