@manojkmfsi/monodog 1.0.19 → 1.0.21

package/.turbo/turbo-build.log

@@ -1,4 +1,4 @@
-
-> @manojkmfsi/monodog@1.0.19 build /home/manoj/Documents/mjdog/packages/monoapp
-> rm -rf dist && tsc
-
+
+> @manojkmfsi/monodog@1.0.21 build /home/runner/work/monodog/monodog/packages/monoapp
+> rm -rf dist && tsc
+

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @manojkmfsi/monoapp
 
+## 1.0.21
+
+### Patch Changes
+
+- updated security,workspce fallback, docs
+
+## 1.0.20
+
+### Patch Changes
+
+- Restructure backend app
+
 ## 1.0.17
 
 ### Patch Changes

package/dist/config-loader.js

@@ -39,7 +39,7 @@ const path = __importStar(require("path"));
 // Global variable to hold the loaded config
 let config = null;
 /**
- * Loads the monodog-conf.json file from the monorepo root.
+ * Loads the monodog-config.json file from the monorepo root.
  * This should be called only once during application startup.
  * @returns The application configuration object.
  */
@@ -50,8 +50,8 @@ function loadConfig() {
     // 1. Determine the path to the config file
     // We assume the backend package is running from the monorepo root (cwd is root)
     // or that we can navigate up to the root from the current file's location.
-    const rootPath = path.resolve(process.cwd()); // Adjust based on your workspace folder depth  from root if needed
-    const configPath = path.resolve(rootPath, 'monodog-conf.json');
+    const rootPath = path.resolve(process.cwd());
+    const configPath = path.resolve(rootPath, 'monodog-config.json');
     createConfigFileIfMissing(rootPath);
     if (!fs.existsSync(configPath)) {
         console.error(`ERROR1: Configuration file not found at ${configPath}`);
@@ -61,21 +61,20 @@ function loadConfig() {
         // 2. Read and parse the JSON file
         const fileContent = fs.readFileSync(configPath, 'utf-8');
         const parsedConfig = JSON.parse(fileContent);
-        // 3. Optional: Add validation logic here (e.g., check if ports are numbers)
         // Cache and return
         config = parsedConfig;
         process.stderr.write('[Config] Loaded configuration from: ...\n');
         return config;
     }
     catch (error) {
-        console.error('ERROR: Failed to read or parse monodog-conf.json.');
+        console.error('ERROR: Failed to read or parse monodog-config.json.');
         console.error(error);
         process.exit(1);
     }
 }
 function createConfigFileIfMissing(rootPath) {
     // --- CONFIGURATION ---
-    const configFileName = 'monodog-conf.json';
+    const configFileName = 'monodog-config.json';
     const configFilePath = path.resolve(rootPath, configFileName);
     // The default content for the configuration file
     const defaultContent = {

package/dist/get-db-url.js

@@ -2,7 +2,6 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const config_loader_1 = require("./config-loader");
 function generateUrl() {
-    // const appConfig = loadConfig();
     const DATABASE_URL = `${config_loader_1.appConfig.database.path}`;
     process.env.DATABASE_URL = DATABASE_URL;
     process.stdout.write(DATABASE_URL);

package/dist/index.js

@@ -9,94 +9,233 @@ const express_1 = __importDefault(require("express"));
 const cors_1 = __importDefault(require("cors"));
 const path_1 = __importDefault(require("path"));
 const body_parser_1 = require("body-parser");
+const helmet_1 = __importDefault(require("helmet"));
 const config_loader_1 = require("./config-loader");
 const packageRoutes_1 = __importDefault(require("./routes/packageRoutes"));
 const commitRoutes_1 = __importDefault(require("./routes/commitRoutes"));
 const healthRoutes_1 = __importDefault(require("./routes/healthRoutes"));
 const configRoutes_1 = __importDefault(require("./routes/configRoutes"));
-// The main function exported and called by the CLI
-function startServer(rootPath, port, host) {
-    const app = (0, express_1.default)();
-    app.locals.rootPath = rootPath;
-    // --- Middleware ---
-    // 1. Logging Middleware
-    app.use((_req, _res, next) => {
-        console.log(`[SERVER] ${_req.method} ${_req.url} (Root: ${rootPath})`);
-        next();
-    });
-    app.use((0, cors_1.default)());
-    app.use((0, body_parser_1.json)());
-    app.use('/api/packages', packageRoutes_1.default);
-    // Get commit details
-    app.use('/api/commits/', commitRoutes_1.default);
-    // ---------- HEALTH --------------------
-    app.use('/api/health/', healthRoutes_1.default);
-    // ------------------------- CONFIGURATION TAB ------------------------- //
-    // Get all configuration files from the file system
-    app.use('/api/config/', configRoutes_1.default);
-    // 404 handler
-    app.use('*', (_, res) => {
-        res.status(404).json({
-            error: 'Endpoint not found',
-            timestamp: Date.now(),
-        });
+// Security constants
+const PORT_MIN = 1024;
+const PORT_MAX = 65535;
+// Validate port number
+function validatePort(port) {
+    const portNum = typeof port === 'string' ? parseInt(port, 10) : port;
+    if (isNaN(portNum) || portNum < PORT_MIN || portNum > PORT_MAX) {
+        throw new Error(`Port must be between ${PORT_MIN} and ${PORT_MAX}`);
+    }
+    return portNum;
+}
+// Global error handler
+const errorHandler = (err, req, res, _next) => {
+    const status = err.status || err.statusCode || 500;
+    const message = process.env.NODE_ENV === 'production'
+        ? 'Internal server error'
+        : err.message;
+    console.error('[ERROR]', {
+        status,
+        method: req.method,
+        path: req.path,
+        message: err.message,
     });
-    const PORT = parseInt(port ? port.toString() : '4000');
-    app
-        .listen(PORT, host, async () => {
-        console.log(`🚀 Backend server running on http://${host}:${PORT}`);
-        console.log(`📊 API endpoints available:`);
-        console.log(`   - GET  /api/health`);
-        console.log(`   - GET  /api/packages/refresh`);
-        console.log(`   - GET  /api/packages`);
-        console.log(`   - GET  /api/packages/:name`);
-        console.log(`   - PUT  /api/packages/update-config`);
-        console.log(`   - GET  /api/commits/:packagePath`);
-        console.log(`   - GET  /api/health/packages`);
-        console.log(`   - PUT  /api/config/files/:id`);
-        console.log(`   - GET  /api/config/files`);
-    })
-        .on('error', err => {
-        // Handle common errors like EADDRINUSE (port already in use)
-        if (err.message.includes('EADDRINUSE')) {
-            console.error(`Error: Port ${port} is already in use. Please specify a different port via configuration file.`);
-            process.exit(1);
-        }
-        else {
-            console.error('Server failed to start:', err);
-            process.exit(1);
-        }
+    res.status(status).json({
+        error: 'Internal server error'
     });
+};
+// The main function exported and called by the CLI
+function startServer(rootPath) {
+    try {
+        const port = config_loader_1.appConfig.server.port;
+        const host = config_loader_1.appConfig.server.host;
+        const validatedPort = validatePort(port);
+        const app = (0, express_1.default)();
+        // Set request timeout (30 seconds)
+        app.use((req, res, next) => {
+            req.setTimeout(30000);
+            res.setTimeout(30000);
+            next();
+        });
+        app.locals.rootPath = rootPath;
+        // Security middleware with CSP allowing API calls
+        const apiHost = host === '0.0.0.0' ? 'localhost' : host;
+        const apiUrl = process.env.API_URL || `http://${apiHost}:${validatedPort}`;
+        const dashboardHost = config_loader_1.appConfig.dashboard.host === '0.0.0.0' ? 'localhost' : config_loader_1.appConfig.dashboard.host;
+        const dashboardUrl = `http://${dashboardHost}:${config_loader_1.appConfig.dashboard.port}`;
+        app.use((0, helmet_1.default)({
+            contentSecurityPolicy: {
+                directives: {
+                    defaultSrc: ["'self'"],
+                    connectSrc: ["'self'", apiUrl, 'http://localhost:*', 'http://127.0.0.1:*'],
+                    scriptSrc: ["'self'"],
+                    imgSrc: ["'self'", 'data:', 'https:'],
+                },
+            },
+        }));
+        app.use((0, cors_1.default)({
+            origin: process.env.CORS_ORIGIN || dashboardUrl,
+            credentials: true,
+            methods: ['GET', 'POST', 'PUT', 'DELETE', 'OPTIONS'],
+            allowedHeaders: ['Content-Type', 'Authorization'],
+        }));
+        app.use((0, body_parser_1.json)({ limit: '1mb' }));
+        // Request logging middleware (safe version)
+        app.use((_req, _res, next) => {
+            console.log(`[${new Date().toISOString()}] ${_req.method} ${_req.path}`);
+            next();
+        });
+        app.use('/api/packages', packageRoutes_1.default);
+        // Get commit details
+        app.use('/api/commits/', commitRoutes_1.default);
+        // Health check endpoint
+        app.use('/api/health/', healthRoutes_1.default);
+        // Configuration endpoint
+        app.use('/api/config/', configRoutes_1.default);
+        // 404 handler
+        app.use('*', (_, res) => {
+            res.status(404).json({
+                error: 'Endpoint not found',
+                timestamp: Date.now(),
+            });
+        });
+        // Global error handler (must be last)
+        app.use(errorHandler);
+        const server = app.listen(validatedPort, host, () => {
+            console.log(`🚀 Backend server running on http://${host}:${validatedPort}`);
+            console.log(`📊 API endpoints available:`);
+            console.log(`   - GET  /api/health`);
+            console.log(`   - GET  /api/packages/refresh`);
+            console.log(`   - GET  /api/packages`);
+            console.log(`   - GET  /api/packages/:name`);
+            console.log(`   - PUT  /api/packages/update-config`);
+            console.log(`   - GET  /api/commits/:packagePath`);
+            console.log(`   - GET  /api/health/packages`);
+            console.log(`   - PUT  /api/config/files/:id`);
+            console.log(`   - GET  /api/config/files`);
+        });
+        server.on('error', (err) => {
+            // Handle common errors like EADDRINUSE (port already in use)
+            if (err.code === 'EADDRINUSE') {
+                console.error(`Error: Port ${validatedPort} is already in use. Please specify a different port.`);
+                process.exit(1);
+            }
+            else if (err.code === 'EACCES') {
+                console.error(`Error: Permission denied to listen on port ${validatedPort}. Use a port above 1024.`);
+                process.exit(1);
+            }
+            else {
+                console.error('Server failed to start:', err.message);
+                process.exit(1);
+            }
+        });
+        // Graceful shutdown
+        process.on('SIGTERM', () => {
+            console.log('SIGTERM signal received: closing HTTP server');
+            server.close(() => {
+                console.log('HTTP server closed');
+                process.exit(0);
+            });
+        });
+    }
+    catch (error) {
+        console.error('Failed to start server:', error.message);
+        process.exit(1);
+    }
 }
-function serveDashboard(rootPath, port, host) {
-    const app = (0, express_1.default)();
-    app.get('/env-config.js', (req, res) => {
-        res.setHeader('Content-Type', 'application/javascript');
-        res.send(`window.ENV = { API_URL: "${`${config_loader_1.appConfig.server.host}:${config_loader_1.appConfig.server.port}` || 'localhost:8999'}" };`);
-    });
-    // This code makes sure that any request that does not matches a static file
-    // in the build folder, will just serve index.html. Client side routing is
-    // going to make sure that the correct content will be loaded.
-    app.use((req, res, next) => {
-        if (/(.ico|.js|.css|.jpg|.png|.map)$/i.test(req.path)) {
+function serveDashboard(rootPath) {
+    try {
+        const port = config_loader_1.appConfig.dashboard.port;
+        const host = config_loader_1.appConfig.dashboard.host;
+        const validatedPort = validatePort(port);
+        const app = (0, express_1.default)();
+        // Security middleware
+        const serverHost = config_loader_1.appConfig.server.host === '0.0.0.0' ? 'localhost' : config_loader_1.appConfig.server.host;
+        const apiUrl = process.env.API_URL || `http://${serverHost}:${config_loader_1.appConfig.server.port}`;
+        app.use((0, helmet_1.default)({
+            contentSecurityPolicy: {
+                directives: {
+                    defaultSrc: ["'self'"],
+                    connectSrc: ["'self'", apiUrl, 'http://localhost:*', 'http://127.0.0.1:*'],
+                    scriptSrc: ["'self'"],
+                    imgSrc: ["'self'", 'data:', 'https:'],
+                },
+            },
+        }));
+        // Strict CORS for dashboard
+        app.use((0, cors_1.default)({
+            origin: false, // Don't allow any origin for static assets
+        }));
+        // Set request timeout
+        app.use((req, res, next) => {
+            req.setTimeout(30000);
+            res.setTimeout(30000);
             next();
-        }
-        else {
-            res.header('Cache-Control', 'private, no-cache, no-store, must-revalidate');
-            res.header('Expires', '-1');
-            res.header('Pragma', 'no-cache');
-            res.sendFile('index.html', {
-                root: path_1.default.resolve(__dirname, '..', 'monodog-dashboard', 'dist'),
+        });
+        app.get('/env-config.js', (req, res) => {
+            res.setHeader('Content-Type', 'application/javascript');
+            res.setHeader('Cache-Control', 'private, no-cache, no-store, must-revalidate');
+            const serverHost = config_loader_1.appConfig.server.host === '0.0.0.0' ? 'localhost' : config_loader_1.appConfig.server.host;
+            const apiUrl = process.env.API_URL || `http://${serverHost}:${config_loader_1.appConfig.server.port}`;
+            res.send(`window.ENV = { API_URL: "${apiUrl}" };`);
+        });
+        // This code makes sure that any request that does not matches a static file
+        // in the build folder, will just serve index.html. Client side routing is
+        // going to make sure that the correct content will be loaded.
+        app.use((req, res, next) => {
+            if (/(.ico|.js|.css|.jpg|.png|.map|.woff|.woff2|.ttf)$/i.test(req.path)) {
+                next();
+            }
+            else {
+                res.header('Cache-Control', 'private, no-cache, no-store, must-revalidate');
+                res.header('Expires', '-1');
+                res.header('Pragma', 'no-cache');
+                res.sendFile('index.html', {
+                    root: path_1.default.resolve(__dirname, '..', 'monodog-dashboard', 'dist'),
+                }, (err) => {
+                    if (err) {
+                        console.error('Error serving index.html:', err.message);
+                        res.status(500).json({ error: 'Internal server error' });
+                    }
+                });
+            }
+        });
+        const staticPath = path_1.default.join(__dirname, '..', 'monodog-dashboard', 'dist');
+        console.log('Serving static files from:', staticPath);
+        app.use(express_1.default.static(staticPath, {
+            maxAge: '1d',
+            etag: false,
+            dotfiles: 'deny', // Don't serve dot files
+        }));
+        // Global error handler
+        app.use(errorHandler);
+        const server = app.listen(validatedPort, host, () => {
+            console.log(`✅ Dashboard listening on http://${host}:${validatedPort}`);
+            console.log('Press Ctrl+C to quit.');
+        });
+        server.on('error', (err) => {
+            if (err.code === 'EADDRINUSE') {
+                console.error(`Error: Port ${validatedPort} is already in use.`);
+                process.exit(1);
+            }
+            else if (err.code === 'EACCES') {
+                console.error(`Error: Permission denied to listen on port ${validatedPort}.`);
+                process.exit(1);
+            }
+            else {
+                console.error('Server failed to start:', err.message);
+                process.exit(1);
+            }
+        });
+        // Graceful shutdown
+        process.on('SIGTERM', () => {
+            console.log('SIGTERM signal received: closing dashboard server');
+            server.close(() => {
+                console.log('Dashboard server closed');
+                process.exit(0);
             });
-        }
-    });
-    const staticPath = path_1.default.join(__dirname, '..', 'monodog-dashboard', 'dist');
-    console.log('Serving static files from:', staticPath);
-    app.use(express_1.default.static(staticPath));
-    // Start the server
-    const PORT = parseInt(port ? port.toString() : '8999');
-    app.listen(PORT, host, () => {
-        console.log(`App listening on ${host}:${port}`);
-        console.log('Press Ctrl+C to quit.');
-    });
+        });
+    }
+    catch (error) {
+        console.error('Failed to start dashboard:', error.message);
+        process.exit(1);
+    }
 }

package/dist/serve.js

@@ -2,102 +2,18 @@
 "use strict";
 /**
  * CLI Entry Point for serving Monodog.
- * * This script is executed when a user runs the serve command
+ * This script is executed when a user runs the serve command
  * in their project. It handles command-line arguments to determine
  * whether to:
  * 1. Start the API server for the dashboard.
  * 2. Start serving the dashboard frontend.
  */
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    var desc = Object.getOwnPropertyDescriptor(m, k);
-    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
-      desc = { enumerable: true, get: function() { return m[k]; } };
-    }
-    Object.defineProperty(o, k2, desc);
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
-    Object.defineProperty(o, "default", { enumerable: true, value: v });
-}) : function(o, v) {
-    o["default"] = v;
-});
-var __importStar = (this && this.__importStar) || (function () {
-    var ownKeys = function(o) {
-        ownKeys = Object.getOwnPropertyNames || function (o) {
-            var ar = [];
-            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
-            return ar;
-        };
-        return ownKeys(o);
-    };
-    return function (mod) {
-        if (mod && mod.__esModule) return mod;
-        var result = {};
-        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
-        __setModuleDefault(result, mod);
-        return result;
-    };
-})();
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
 Object.defineProperty(exports, "__esModule", { value: true });
-const path = __importStar(require("path"));
-const index_1 = require("./index"); // Assume index.ts exports this function
-const config_loader_1 = require("./config-loader");
-const fs_1 = __importDefault(require("fs"));
-// --- Argument Parsing ---
-// 1. Get arguments excluding the node executable and script name
-const args = process.argv.slice(2);
-// Default settings
-const DEFAULT_PORT = 8999;
-const rootPath = findMonorepoRoot();
-const port = config_loader_1.appConfig.server.port ?? DEFAULT_PORT; //Default port
-const host = config_loader_1.appConfig.server.host ?? 'localhost'; //Default host
-// --- Execution Logic ---
+const index_1 = require("./index");
+const utilities_1 = require("./utils/utilities");
+const rootPath = (0, utilities_1.findMonorepoRoot)();
 console.log(`Starting Monodog API server...`);
 console.log(`Analyzing monorepo at root: ${rootPath}`);
-// Start the Express server and begin analysis
-(0, index_1.startServer)(rootPath, port, host);
-(0, index_1.serveDashboard)(path.join(rootPath), config_loader_1.appConfig.dashboard.port, config_loader_1.appConfig.dashboard.host);
-/**
- * Find the monorepo root by looking for package.json with workspaces or pnpm-workspace.yaml
- */
-function findMonorepoRoot() {
-    let currentDir = __dirname;
-    while (currentDir !== path.parse(currentDir).root) {
-        const packageJsonPath = path.join(currentDir, 'package.json');
-        const pnpmWorkspacePath = path.join(currentDir, 'pnpm-workspace.yaml');
-        // Check if this directory has package.json with workspaces or pnpm-workspace.yaml
-        if (fs_1.default.existsSync(packageJsonPath)) {
-            try {
-                const packageJson = JSON.parse(fs_1.default.readFileSync(packageJsonPath, 'utf8'));
-                // If it has workspaces or is the root monorepo package
-                if (packageJson.workspaces || fs_1.default.existsSync(pnpmWorkspacePath)) {
-                    console.log('✅ Found monorepo root:', currentDir);
-                    return currentDir;
-                }
-            }
-            catch (error) {
-                // Continue searching if package.json is invalid
-            }
-        }
-        // Check if we're at the git root
-        const gitPath = path.join(currentDir, '.git');
-        if (fs_1.default.existsSync(gitPath)) {
-            console.log('✅ Found git root (likely monorepo root):', currentDir);
-            return currentDir;
-        }
-        // Go up one directory
-        const parentDir = path.dirname(currentDir);
-        if (parentDir === currentDir)
-            break; // Prevent infinite loop
-        currentDir = parentDir;
-    }
-    // Fallback to process.cwd() if we can't find the root
-    console.log('⚠️ Could not find monorepo root, using process.cwd():', process.cwd());
-    return process.cwd();
-}
+// Start the Express server and dashboard
+(0, index_1.startServer)(rootPath);
+(0, index_1.serveDashboard)(rootPath);

package/dist/services/commitService.js

@@ -4,7 +4,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getCommitsByPathService = void 0;
-const gitService_1 = require("../gitService");
+const gitService_1 = require("./gitService");
 const path_1 = __importDefault(require("path"));
 const fs_1 = __importDefault(require("fs"));
 const getCommitsByPathService = async (packagePath) => {

package/dist/{gitService.js → services/gitService.js}

@@ -32,107 +32,6 @@ const VALID_COMMIT_TYPES = [
     'build', // Build system changes (e.g., pnpm/npm scripts)
     'revert', // Reverting changes
 ];
-/**
- * The delimiter used to separate individual commit records in the git log output.
- * We use a unique string to ensure reliable splitting.
- */
-// const COMMIT_DELIMITER = '|---COMMIT-END---|';
-/**
- * The custom format string passed to 'git log'.
- * It uses JSON structure and our unique delimiter to make parsing consistent.
- * * %H: Commit hash
- * %pn: packageName name
- * %ae: Author email
- * %ad: Author date (ISO 8601 format)
- * %s: Subject (commit message's first line - used for 'message' and 'type' fields)
- */
-// const GIT_LOG_FORMAT = `{"hash": "%H", "author": "%ae", "packageName": "%pn", "date": "%ad", "message": "%s", "type": "%s"}${COMMIT_DELIMITER}`;
-// export class GitService {
-//   private repoPath: string;
-//   constructor(repoPath: string = process.cwd()) {
-//     // Allows specifying a custom path to the monorepo root
-//     this.repoPath = repoPath;
-//   }
-//   /**
-//    * Retrieves commit history for the repository, optionally filtered by a package path.
-//    * @param pathFilter The path to a package directory (e.g., 'packages/server').
-//    * If provided, only commits affecting that path are returned.
-//    * @returns A Promise resolving to an array of GitCommit objects.
-//    */
-//   public async getAllCommits(pathFilter?: string): Promise<GitCommit[]> {
-//     const pathArgument = pathFilter ? ` -- ${pathFilter}` : '';
-//     const command = `git log --pretty=format:'${GIT_LOG_FORMAT}' --date=iso-strict${pathArgument}`;
-//     try {
-//       console.log(`Executing Git command: ${command}`);
-//       const { stdout } = await execPromise(command, {
-//         cwd: this.repoPath,
-//         maxBuffer: 1024 * 5000,
-//       }); // Increase buffer for large repos
-//       // console.log(stdout)
-//       console.log(stdout, '--', this.repoPath);
-//       const escapedDelimiter = COMMIT_DELIMITER.replace(
-//         /[.*+?^${}()|[\]\\]/g,
-//         '\\$&'
-//       );
-//       // 1. Remove the final trailing delimiter, as it creates an empty string at the end of the array
-//       const cleanedOutput = stdout
-//         .trim()
-//         .replace(new RegExp(`${escapedDelimiter}$`), '');
-//       console.log(cleanedOutput);
-//       if (!cleanedOutput) {
-//         return [];
-//       }
-//       // 2. Split the output by our custom delimiter to get an array of JSON strings
-//       const jsonStrings = cleanedOutput.split(COMMIT_DELIMITER);
-//       // 3. Parse each string into a GitCommit object
-//       const commits: GitCommit[] = jsonStrings
-//         .map(jsonString => {
-//           try {
-//             // JSON.parse is necessary because the output starts as a string
-//             const commit = JSON.parse(jsonString);
-//             console.log(commit.type, commit['hash']);
-//             try {
-//               // FIX: Updated Regex to handle optional scope (e.g., 'feat(scope):')
-//               // Matches:
-//               // 1. (^(\w+)) - The commit type (e.g., 'feat')
-//               // 2. (\([^)]+\))? - The optional scope (e.g., '(setup)')
-//               // 3. (:|!) - Ends with a colon or an exclamation point (for breaking changes)
-//               const typeMatch = commit.type.match(/^(\w+)(\([^)]+\))?([:!])/);
-//               let extractedType = 'other';
-//               if (typeMatch) {
-//                 const rawType = typeMatch[1];
-//                 // NEW: Validate the extracted type against the list of known types
-//                 if (VALID_COMMIT_TYPES.includes(rawType)) {
-//                   extractedType = rawType;
-//                 }
-//               }
-//               commit.type = extractedType;
-//             } catch (e) {
-//               console.error('Failed to match commit type:', commit, e);
-//               // Skip malformed entries
-//               return null;
-//             }
-//             // // Set type to 'other' if the conventional format is not found
-//             // commit.type = typeMatch ? typeMatch[1] : 'other';
-//             return commit;
-//           } catch (e) {
-//             // console.log(jsonString)
-//             console.error('Failed to parse commit JSON:', jsonString, e);
-//             // Skip malformed entries
-//             return null;
-//           }
-//         })
-//         .filter((commit): commit is GitCommit => commit !== null);
-//       return commits;
-//     } catch (error) {
-//       console.error('Error fetching Git history:', error);
-//       // In a real application, you would handle this gracefully (e.g., throwing a custom error)
-//       throw new Error(
-//         'Failed to retrieve Git commit history. Is Git installed and is the path correct?'
-//       );
-//     }
-//   }
-// }
 class GitService {
     constructor(repoPath = process.cwd()) {
         this.repoPath = repoPath;