fixed cli command

Author: AvrAlexandra

src/commands/history-metrics/chart-generator.ts

@@ -6,10 +6,18 @@ export async function generateHtmlChart(outputFile: string, charts: { data: any[
   const html = generateMultiChartHtml(charts);
 
   fs.writeFileSync(chartFile, html);
+  console.log(`📊 Chart generated: ${chartFile}`);
 
   if (process.env.NODE_ENV !== 'test') {
-    const open = (await import('open')).default;
-    await open(chartFile);
+    try {
+      const openModule = await import('open');
+      const open = openModule.default || openModule;
+      await open(chartFile);
+      console.log(`🌐 Chart opened in browser`);
+    } catch (error: any) {
+      console.log(`⚠️ Could not automatically open chart in browser: ${error.message}`);
+      console.log(`📁 Please manually open: ${chartFile}`);
+    }
   }
 }
 

src/commands/history-metrics/metrics-command.ts

@@ -14,16 +14,16 @@ import {
   generateVulnerabilityFixBySeverityChartData,
   generateVulnerabilityFixTimelinessChartData
 } from './chart-generator'
+import os from "os";
 
 export const metricsCommand = new Command()
   .name('metrics')
   .description('Run metrics on dependency history and optionally generate charts')
-  .argument('<historyFolder>', 'Folder where the base input/output folders are located')
   .option('--inputDir <inputDir>', 'Directory to look for input .json files', '')
-  .option('--results, -r <resultsFolder>', 'Folder to save results in', 'results')
+  .option('--results <resultsFolder>', 'Folder to save results in', 'results')
   .option('--metric <metricType>', 'Metric type to calculate')
   .option('--chart', 'Generate chart visualization', false)
-  .option('--chartType <chartType>', 'Chart type (bar | line | stacked | stacked-area)', 'bar')
+  .option('--chartType <chartType>', 'Chart types to generate (bar, line, stacked, stacked-area)')
   .option('--inputFiles <inputFiles...>', 'List of input files to use (without .json)')
   .action(runMetrics);
 
@@ -75,9 +75,17 @@ function isValidMetricType(metric: string): metric is MetricType {
   return metric in metricsRegistry;
 }
 
-export async function runMetrics(historyFolder: string, options: MetricOptions): Promise<void> {
+export async function runMetrics(options: MetricOptions): Promise<void> {
+  const homeDir = os.homedir();
   const metricType = options.metric;
 
+  // Ensure chartType is always an array with a default value
+  if (!options.chartType || !Array.isArray(options.chartType)) {
+    options.chartType = options.chartType ? [options.chartType as any] : ['bar'];
+  }
+
+  console.log('Chart types:', options.chartType);
+
   if (!isValidMetricType(metricType)) {
     console.error(`❌ Unknown metric type: ${metricType}`);
     return;
@@ -90,8 +98,8 @@ export async function runMetrics(historyFolder: string, options: MetricOptions):
     return;
   }
 
-  const inputBase = path.join(historyFolder, options.inputDir || '');
-  const outputBase = path.join(historyFolder, options.results);
+  const inputBase = path.join(homeDir, options.inputDir || '');
+  const outputBase = path.join(homeDir, options.results);
   fs.mkdirSync(outputBase, { recursive: true });
 
   let libraryInfo: any = undefined;

src/commands/history/history.ts

@@ -31,11 +31,13 @@ export interface Plugin {
 export const historyCommand = new Command()
   .name('history')
   .argument('[folders...]', 'A list of git repositories to analyse')
-  .option('--results, -r', 'The results folder', 'results')
-  .option('--plugins, -p [plugins...]', 'A list of plugins')
+  .option('--results <folder>', 'The results folder', 'results')
+  .option('--plugins <plugins...>', 'A list of plugins')
   .action(analyseHistory);
 
-export async function analyseHistory(folders: string[], options: HistoryOptions, useCache = true): Promise<void> {
+export async function  analyseHistory(folders: string[], options: HistoryOptions, useCache = true): Promise<void> {
+  console.log('Input folders:', folders);
+  console.log('Results folder:', options.results);
   const selectedPlugins: Plugin[] = getPluginsFromNames(options.plugins);
   if (folders.length === 0) {
     log.info('No folders provided to analyze.');