modestbench 0.8.0 → 0.9.1

package/src/cli/commands/run.ts

@@ -11,11 +11,15 @@ import type {
   BenchmarkRun,
   ModestBenchConfig,
   Reporter,
+  ReporterConfig,
 } from '../../types/index.js';
 import type { CliContext } from '../index.js';
 
 import { ErrorCodes, ExitCodes } from '../../constants.js';
-import { resolveOutputPath } from '../../core/output-path-resolver.js';
+import {
+  generateTimestampedFilename,
+  resolveOutputPath,
+} from '../../core/output-path-resolver.js';
 import {
   type BudgetExceededError,
   InvalidArgumentError,
@@ -42,7 +46,7 @@ import { hasErrorCode, isError } from '../../utils/type-guards.js';
  * arguments provide values. They represent sensible defaults for running
  * benchmarks.
  */
-export const RUN_COMMAND_DEFAULTS = {
+const RUN_COMMAND_DEFAULTS = {
   bail: false,
   quiet: false,
   reporters: ['human'],
@@ -61,6 +65,7 @@ interface RunOptions {
   excludeTags?: string[] | undefined;
   iterations?: number | undefined;
   json?: boolean | undefined;
+  jsonPretty?: boolean | undefined;
   noColor?: boolean | undefined;
   outputDir?: string | undefined;
   outputFile?: string | undefined;
@@ -126,6 +131,7 @@ export const handleRunCommand = async (
       options.outputDir,
       options.outputFile,
       options.progress,
+      options.jsonPretty,
     );
 
     // Step 3: Discovery phase
@@ -389,7 +395,7 @@ const setupReporters = async (
   context: CliContext,
   config: {
     outputDir?: string;
-    reporterConfig?: Record<string, unknown>;
+    reporterConfig?: ReporterConfig;
     reporters?: string[];
   },
   isVerbose: boolean,
@@ -398,6 +404,7 @@ const setupReporters = async (
   explicitOutputDir?: string,
   explicitOutputFile?: string,
   progressOption?: boolean,
+  explicitJsonPretty?: boolean,
 ): Promise<Reporter[]> => {
   try {
     const reporters: Reporter[] = [];
@@ -426,7 +433,7 @@ const setupReporters = async (
             const outputPath = resolveOutputPath(
               outputDir,
               explicitOutputFile,
-              'results.csv',
+              generateTimestampedFilename('csv'),
             );
             reporter = new CsvReporter({
               includeHeaders: true,
@@ -451,11 +458,16 @@ const setupReporters = async (
             const outputPath = resolveOutputPath(
               outputDir,
               explicitOutputFile,
-              'results.json',
+              generateTimestampedFilename('json'),
             );
+            // Precedence: CLI flag > config file > default (false)
+            const prettyPrint =
+              explicitJsonPretty ??
+              config.reporterConfig?.json?.prettyPrint ??
+              false;
             reporter = new JsonReporter({
               ...(outputPath ? { outputPath } : {}),
-              prettyPrint: true,
+              prettyPrint,
             });
             break;
           }

package/src/cli/commands/test.ts

@@ -55,7 +55,7 @@ const getDefaultTestConfig = (
   tags: [],
   thresholds: {},
   time: 1000,
-  timeout: 30000,
+  timeout: 30_000,
   verbose,
   warmup,
 });