refactor(registry): replace fs with fileHandler

- Improved file handling using FileHandler
- Enhanced error handling for file operations
- Simplified updater loading process
- Added readDir method to FileHandler
- Updated tests for UpdaterRegistry

Author: taj54

dist/index.js

@@ -6,7 +6,7 @@
  * Author: Taj <[email protected]>
  * Homepage: https://github.com/taj54/universal-version-bump#readme
  * License: MIT
- * Generated on Wed, 27 Aug 2025 08:13:44 GMT
+ * Generated on Wed, 27 Aug 2025 09:27:33 GMT
  */
 require('./sourcemap-register.js');/******/ (() => { // webpackBootstrap
 /******/ 	var __webpack_modules__ = ({
@@ -32903,20 +32903,20 @@ var __importStar = (this && this.__importStar) || (function () {
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.UpdaterRegistry = void 0;
 const path = __importStar(__nccwpck_require__(6928));
-const fs = __importStar(__nccwpck_require__(9896));
+const fileHandler_1 = __nccwpck_require__(1013);
 /**
  * Registry for managing updaters.
  */
 class UpdaterRegistry {
     constructor() {
         this.updaters = new Map();
+        this.fileHandler = new fileHandler_1.FileHandler();
     }
     /**
      * Dynamically loads and registers all updaters from the updaters directory.
      */
-    async loadUpdaters() {
-        const updatersPath = path.join(__dirname, '../updaters');
-        const files = fs.readdirSync(updatersPath);
+    async loadUpdaters(updatersPath = path.join(__dirname, '../updaters')) {
+        const files = this.fileHandler.readDir(updatersPath);
         for (const file of files) {
             if (file.endsWith('Updater.ts')) {
                 const modulePath = path.join(updatersPath, file);
@@ -33374,6 +33374,18 @@ class FileHandler {
     writeFile(filePath, content) {
         fs_1.default.writeFileSync(filePath, content);
     }
+    /**
+     * Reads the contents of a directory.
+     * @param dirPath The path to the directory.
+     * @returns An array of file names in the directory.
+     * @throws Error if the directory does not exist.
+     */
+    readDir(dirPath) {
+        if (!this.fileExists(dirPath)) {
+            throw new errors_1.FileNotFoundError(`Directory not found: ${dirPath}`);
+        }
+        return fs_1.default.readdirSync(dirPath);
+    }
 }
 exports.FileHandler = FileHandler;
 

dist/index.js.map

@@ -1,20 +1,23 @@
 import { UpdaterInterface as IUpdater } from '../interface';
 import * as path from 'path';
-import * as fs from 'fs';
+import { FileHandler } from '../utils/fileHandler';
 
 /**
  * Registry for managing updaters.
  */
 export class UpdaterRegistry {
   private updaters: Map<string, IUpdater> = new Map();
+  private fileHandler: FileHandler;
+
+  constructor() {
+    this.fileHandler = new FileHandler();
+  }
 
   /**
    * Dynamically loads and registers all updaters from the updaters directory.
    */
-  async loadUpdaters(): Promise<void> {
-    const updatersPath = path.join(__dirname, '../updaters');
-    const files = fs.readdirSync(updatersPath);
-
+  async loadUpdaters(updatersPath: string = path.join(__dirname, '../updaters')): Promise<void> {
+    const files = this.fileHandler.readDir(updatersPath);
     for (const file of files) {
       if (file.endsWith('Updater.ts')) {
         const modulePath = path.join(updatersPath, file);

src/registry/updaterRegistry.ts

@@ -36,4 +36,17 @@ export class FileHandler {
   writeFile(filePath: string, content: string): void {
     fs.writeFileSync(filePath, content);
   }
+
+  /**
+   * Reads the contents of a directory.
+   * @param dirPath The path to the directory.
+   * @returns An array of file names in the directory.
+   * @throws Error if the directory does not exist.
+   */
+  readDir(dirPath: string): string[] {
+    if (!this.fileExists(dirPath)) {
+      throw new FileNotFoundError(`Directory not found: ${dirPath}`);
+    }
+    return fs.readdirSync(dirPath);
+  }
 }

src/utils/fileHandler.ts

@@ -0,0 +1,22 @@
+import { describe, it, expect } from 'vitest';
+import { UpdaterRegistry } from '../../src/registry';
+
+describe('UpdaterRegistry', () => {
+  it('should load all updaters from the updaters directory', async () => {
+    const registry = new UpdaterRegistry();
+    await registry.loadUpdaters();
+    const allUpdaters = registry.getAllUpdaters();
+
+    // There are 6 updaters in the src/updaters directory
+    expect(allUpdaters).toHaveLength(6);
+
+    // Check if all platforms are loaded
+    const platforms = allUpdaters.map((updater) => updater.platform);
+    expect(platforms).toContain('node');
+    expect(platforms).toContain('python');
+    expect(platforms).toContain('rust');
+    expect(platforms).toContain('go');
+    expect(platforms).toContain('docker');
+    expect(platforms).toContain('php');
+  });
+});