@dizzlkheinz/ynab-mcpb 0.18.3 → 0.19.0

package/vitest-reporters/split-json-reporter.ts

@@ -1,211 +1,254 @@
-import type { Reporter, File, Vitest } from 'vitest';
-import { writeFile, mkdir } from 'fs/promises';
-import { join } from 'path';
+import { mkdir, writeFile } from "node:fs/promises";
+import { join } from "node:path";
+import type { File, Reporter, Vitest } from "vitest";
+
+type TaskError = {
+	message?: string;
+	stack?: string;
+};
+
+type TaskResult = {
+	state?: string;
+	errors?: TaskError[];
+	duration?: number;
+};
+
+type TaskNode = {
+	type?: string;
+	name?: string;
+	suite?: { name?: string };
+	result?: TaskResult;
+	tasks?: TaskNode[];
+};
 
 /**
  * Custom Vitest reporter that splits test results into multiple JSON files
  * by project (unit, integration, e2e) and generates summary files
  */
 export default class SplitJsonReporter implements Reporter {
-  ctx!: Vitest;
-  private results = new Map<string, File[]>();
-
-  onInit(ctx: Vitest) {
-    this.ctx = ctx;
-  }
-
-  async onFinished(files?: File[]) {
-    if (!files) return;
-
-    const outputDir = join(process.cwd(), 'test-results');
-    await mkdir(outputDir, { recursive: true });
-
-    // Group files by project
-    const projectResults = new Map<string, File[]>();
-    const failedTests: any[] = [];
-    const summary = {
-      numTotalTestSuites: 0,
-      numPassedTestSuites: 0,
-      numFailedTestSuites: 0,
-      numTotalTests: 0,
-      numPassedTests: 0,
-      numFailedTests: 0,
-      startTime: Date.now(),
-      success: true,
-      projects: {} as Record<string, any>,
-    };
-
-    for (const file of files) {
-      const projectName = (file.projectName as string) || 'default';
-
-      if (!projectResults.has(projectName)) {
-        projectResults.set(projectName, []);
-      }
-      projectResults.get(projectName)!.push(file);
-
-      // Update summary
-      summary.numTotalTestSuites++;
-      const allTasks = this.getAllTasks(file);
-      const failedTasks = allTasks.filter((t) => t.result?.state === 'fail');
-
-      if (failedTasks.length > 0) {
-        summary.numFailedTestSuites++;
-        summary.success = false;
-
-        // Collect failed test details
-        failedTasks.forEach((task) => {
-          failedTests.push({
-            project: projectName,
-            file: file.filepath,
-            suite: task.suite?.name,
-            test: task.name,
-            error: task.result?.errors?.[0]?.message,
-            duration: task.result?.duration,
-          });
-        });
-      } else {
-        summary.numPassedTestSuites++;
-      }
-
-      summary.numTotalTests += allTasks.length;
-      summary.numPassedTests += allTasks.filter((t) => t.result?.state === 'pass').length;
-      summary.numFailedTests += failedTasks.length;
-    }
-
-    // Write per-project JSON files
-    for (const [projectName, projectFiles] of projectResults.entries()) {
-      const projectData = {
-        project: projectName,
-        numTestSuites: projectFiles.length,
-        numPassedTestSuites: projectFiles.filter((f) => {
-          const tasks = this.getAllTasks(f);
-          return tasks.every((t) => t.result?.state === 'pass');
-        }).length,
-        numFailedTestSuites: projectFiles.filter((f) => {
-          const tasks = this.getAllTasks(f);
-          return tasks.some((t) => t.result?.state === 'fail');
-        }).length,
-        testSuites: projectFiles.map((file) => ({
-          name: file.filepath,
-          status: this.getFileStatus(file),
-          duration: file.result?.duration || 0,
-          tests: this.getAllTasks(file).map((task) => ({
-            name: task.name,
-            status: task.result?.state || 'unknown',
-            duration: task.result?.duration || 0,
-            errors: task.result?.errors?.map((e) => ({
-              message: e.message,
-              stack: e.stack,
-            })),
-          })),
-        })),
-      };
-
-      // Calculate project summary
-      summary.projects[projectName] = {
-        total: projectFiles.length,
-        passed: projectData.numPassedTestSuites,
-        failed: projectData.numFailedTestSuites,
-      };
-
-      await writeFile(
-        join(outputDir, `${projectName}-tests.json`),
-        JSON.stringify(projectData, null, 2),
-        'utf-8',
-      );
-    }
-
-    // Write summary file
-    await writeFile(join(outputDir, 'summary.json'), JSON.stringify(summary, null, 2), 'utf-8');
-
-    // Write failed tests file (only if there are failures)
-    if (failedTests.length > 0) {
-      await writeFile(
-        join(outputDir, 'failed-tests.json'),
-        JSON.stringify({ numFailedTests: failedTests.length, failures: failedTests }, null, 2),
-        'utf-8',
-      );
-    }
-
-    // Write passed tests summary
-    const passedSummary = {
-      numPassedTests: summary.numPassedTests,
-      numPassedTestSuites: summary.numPassedTestSuites,
-      projects: Object.entries(summary.projects).map(([name, stats]) => ({
-        name,
-        passed: (stats as any).passed,
-      })),
-    };
-
-    await writeFile(
-      join(outputDir, 'passed-tests-summary.json'),
-      JSON.stringify(passedSummary, null, 2),
-      'utf-8',
-    );
-
-    // Write pointer file at test-results.json location
-    const pointerContent = {
-      message:
-        'Test results have been split into multiple files for better performance and readability.',
-      location: './test-results/',
-      files: {
-        'summary.json': 'High-level overview with totals and project breakdowns',
-        'unit-tests.json': 'All unit test results',
-        'integration-tests.json': 'All integration test results',
-        'e2e-tests.json': 'All end-to-end test results',
-        'failed-tests.json': 'Detailed failure information (only present if tests failed)',
-        'passed-tests-summary.json': 'Summary of passed tests',
-        'index.html': 'Interactive HTML report (open in browser)',
-      },
-      quickStats: {
-        total: summary.numTotalTests,
-        passed: summary.numPassedTests,
-        failed: summary.numFailedTests,
-        success: summary.success,
-      },
-      note: 'For detailed results, see the files in the test-results/ directory.',
-    };
-
-    await writeFile(
-      join(process.cwd(), 'test-results.json'),
-      JSON.stringify(pointerContent, null, 2),
-      'utf-8',
-    );
-
-    console.log('\n📊 Test results written to test-results/');
-    console.log(`   - summary.json (overview)`);
-    console.log(`   - *-tests.json (${projectResults.size} project file(s))`);
-    if (failedTests.length > 0) {
-      console.log(`   - failed-tests.json (${failedTests.length} failure(s))`);
-    }
-    console.log(`   - passed-tests-summary.json`);
-    console.log(`   - index.html (interactive report)`);
-    console.log(`\n💡 test-results.json contains a pointer to the detailed results`);
-  }
-
-  private getAllTasks(file: File): any[] {
-    const tasks: any[] = [];
-
-    const collectTasks = (suite: any) => {
-      if (suite.tasks) {
-        for (const task of suite.tasks) {
-          if (task.type === 'test') {
-            tasks.push(task);
-          } else if (task.type === 'suite') {
-            collectTasks(task);
-          }
-        }
-      }
-    };
-
-    collectTasks(file);
-    return tasks;
-  }
-
-  private getFileStatus(file: File): string {
-    const tasks = this.getAllTasks(file);
-    if (tasks.some((t) => t.result?.state === 'fail')) return 'failed';
-    if (tasks.every((t) => t.result?.state === 'pass')) return 'passed';
-    return 'unknown';
-  }
+	ctx!: Vitest;
+	private results = new Map<string, File[]>();
+
+	onInit(ctx: Vitest) {
+		this.ctx = ctx;
+	}
+
+	async onFinished(files?: File[]) {
+		if (!files) return;
+
+		const outputDir = join(process.cwd(), "test-results");
+		await mkdir(outputDir, { recursive: true });
+
+		// Group files by project
+		const projectResults = new Map<string, File[]>();
+		const failedTests: Array<{
+			project: string;
+			file: string;
+			suite?: string;
+			test?: string;
+			error?: string;
+			duration?: number;
+		}> = [];
+		const summary = {
+			numTotalTestSuites: 0,
+			numPassedTestSuites: 0,
+			numFailedTestSuites: 0,
+			numTotalTests: 0,
+			numPassedTests: 0,
+			numFailedTests: 0,
+			startTime: Date.now(),
+			success: true,
+			projects: {} as Record<
+				string,
+				{ total: number; passed: number; failed: number }
+			>,
+		};
+
+		for (const file of files) {
+			const projectName = (file.projectName as string) || "default";
+
+			if (!projectResults.has(projectName)) {
+				projectResults.set(projectName, []);
+			}
+			projectResults.get(projectName)?.push(file);
+
+			// Update summary
+			summary.numTotalTestSuites++;
+			const allTasks = this.getAllTasks(file);
+			const failedTasks = allTasks.filter((t) => t.result?.state === "fail");
+
+			if (failedTasks.length > 0) {
+				summary.numFailedTestSuites++;
+				summary.success = false;
+
+				// Collect failed test details
+				for (const task of failedTasks) {
+					failedTests.push({
+						project: projectName,
+						file: file.filepath,
+						suite: task.suite?.name,
+						test: task.name,
+						error: task.result?.errors?.[0]?.message,
+						duration: task.result?.duration,
+					});
+				}
+			} else {
+				summary.numPassedTestSuites++;
+			}
+
+			summary.numTotalTests += allTasks.length;
+			summary.numPassedTests += allTasks.filter(
+				(t) => t.result?.state === "pass",
+			).length;
+			summary.numFailedTests += failedTasks.length;
+		}
+
+		// Write per-project JSON files
+		for (const [projectName, projectFiles] of projectResults.entries()) {
+			const projectData = {
+				project: projectName,
+				numTestSuites: projectFiles.length,
+				numPassedTestSuites: projectFiles.filter((f) => {
+					const tasks = this.getAllTasks(f);
+					return tasks.every((t) => t.result?.state === "pass");
+				}).length,
+				numFailedTestSuites: projectFiles.filter((f) => {
+					const tasks = this.getAllTasks(f);
+					return tasks.some((t) => t.result?.state === "fail");
+				}).length,
+				testSuites: projectFiles.map((file) => ({
+					name: file.filepath,
+					status: this.getFileStatus(file),
+					duration: file.result?.duration || 0,
+					tests: this.getAllTasks(file).map((task) => ({
+						name: task.name,
+						status: task.result?.state || "unknown",
+						duration: task.result?.duration || 0,
+						errors: task.result?.errors?.map((e) => ({
+							message: e.message,
+							stack: e.stack,
+						})),
+					})),
+				})),
+			};
+
+			// Calculate project summary
+			summary.projects[projectName] = {
+				total: projectFiles.length,
+				passed: projectData.numPassedTestSuites,
+				failed: projectData.numFailedTestSuites,
+			};
+
+			await writeFile(
+				join(outputDir, `${projectName}-tests.json`),
+				JSON.stringify(projectData, null, 2),
+				"utf-8",
+			);
+		}
+
+		// Write summary file
+		await writeFile(
+			join(outputDir, "summary.json"),
+			JSON.stringify(summary, null, 2),
+			"utf-8",
+		);
+
+		// Write failed tests file (only if there are failures)
+		if (failedTests.length > 0) {
+			await writeFile(
+				join(outputDir, "failed-tests.json"),
+				JSON.stringify(
+					{ numFailedTests: failedTests.length, failures: failedTests },
+					null,
+					2,
+				),
+				"utf-8",
+			);
+		}
+
+		// Write passed tests summary
+		const passedSummary = {
+			numPassedTests: summary.numPassedTests,
+			numPassedTestSuites: summary.numPassedTestSuites,
+			projects: Object.entries(summary.projects).map(([name, stats]) => ({
+				name,
+				passed: stats.passed,
+			})),
+		};
+
+		await writeFile(
+			join(outputDir, "passed-tests-summary.json"),
+			JSON.stringify(passedSummary, null, 2),
+			"utf-8",
+		);
+
+		// Write pointer file at test-results.json location
+		const pointerContent = {
+			message:
+				"Test results have been split into multiple files for better performance and readability.",
+			location: "./test-results/",
+			files: {
+				"summary.json":
+					"High-level overview with totals and project breakdowns",
+				"unit-tests.json": "All unit test results",
+				"integration-tests.json": "All integration test results",
+				"e2e-tests.json": "All end-to-end test results",
+				"failed-tests.json":
+					"Detailed failure information (only present if tests failed)",
+				"passed-tests-summary.json": "Summary of passed tests",
+				"index.html": "Interactive HTML report (open in browser)",
+			},
+			quickStats: {
+				total: summary.numTotalTests,
+				passed: summary.numPassedTests,
+				failed: summary.numFailedTests,
+				success: summary.success,
+			},
+			note: "For detailed results, see the files in the test-results/ directory.",
+		};
+
+		await writeFile(
+			join(process.cwd(), "test-results.json"),
+			JSON.stringify(pointerContent, null, 2),
+			"utf-8",
+		);
+
+		console.log("\n📊 Test results written to test-results/");
+		console.log("   - summary.json (overview)");
+		console.log(`   - *-tests.json (${projectResults.size} project file(s))`);
+		if (failedTests.length > 0) {
+			console.log(`   - failed-tests.json (${failedTests.length} failure(s))`);
+		}
+		console.log("   - passed-tests-summary.json");
+		console.log("   - index.html (interactive report)");
+		console.log(
+			"\n💡 test-results.json contains a pointer to the detailed results",
+		);
+	}
+
+	private getAllTasks(file: File): TaskNode[] {
+		const tasks: TaskNode[] = [];
+
+		const collectTasks = (suite: TaskNode) => {
+			if (suite.tasks) {
+				for (const task of suite.tasks) {
+					if (task.type === "test") {
+						tasks.push(task);
+					} else if (task.type === "suite") {
+						collectTasks(task);
+					}
+				}
+			}
+		};
+
+		collectTasks(file);
+		return tasks;
+	}
+
+	private getFileStatus(file: File): string {
+		const tasks = this.getAllTasks(file);
+		if (tasks.some((t) => t.result?.state === "fail")) return "failed";
+		if (tasks.every((t) => t.result?.state === "pass")) return "passed";
+		return "unknown";
+	}
 }

package/vitest.config.ts

@@ -1,99 +1,103 @@
-import { defineConfig } from 'vitest/config';
+import { defineConfig } from "vitest/config";
 
-const integrationFiles = ['src/**/*.integration.test.ts'];
-const isVerbose = !!process.env['VERBOSE_TESTS'];
+const integrationFiles = ["src/**/*.integration.test.ts"];
+const isVerbose = !!process.env.VERBOSE_TESTS;
 
 export default defineConfig({
-  test: {
-    environment: 'node',
-    globals: true,
-    setupFiles: ['src/__tests__/setup.ts'],
-    exclude: ['src/__tests__/testRunner.ts'],
-    // Use projects to target unit/integration/e2e groups (Vitest v3)
-    projects: [
-      {
-        test: {
-          name: 'unit',
-          include: ['src/**/*.{test,spec}.ts'],
-          exclude: [
-            'src/**/*.integration.test.ts',
-            'src/**/*.e2e.test.ts',
-            // YNABMCPServer.test.ts requires real YNAB API token and makes API calls,
-            // so it runs as part of integration tests, not unit tests
-            'src/server/__tests__/YNABMCPServer.test.ts',
-          ],
-        },
-      },
-      {
-        test: {
-          name: 'integration:core',
-          include: integrationFiles,
-          env: {
-            INTEGRATION_TEST_TIER: 'core',
-          },
-          testTimeout: 30000,
-          hookTimeout: 10000,
-        },
-      },
-      {
-        test: {
-          name: 'integration:domain',
-          include: integrationFiles,
-          env: {
-            INTEGRATION_TEST_TIER: 'domain',
-          },
-          testTimeout: 60000,
-          hookTimeout: 15000,
-        },
-      },
-      {
-        test: {
-          name: 'integration:full',
-          include: integrationFiles,
-          env: {
-            INTEGRATION_TEST_TIER: 'full',
-          },
-          testTimeout: 120000,
-          hookTimeout: 30000,
-          fileParallelism: false,
-          maxWorkers: 1,
-        },
-      },
-      {
-        test: {
-          name: 'e2e',
-          include: ['src/**/*.e2e.test.ts'],
-        },
-      },
-    ],
-    coverage: {
-      provider: 'v8',
-      include: ['src/**/*.ts'],
-      exclude: [
-        'src/**/*.{test,spec}.ts',
-        'src/**/*.d.ts',
-        'src/index.ts', // Entry point excluded from coverage
-        'src/__tests__/**/*.ts', // Test utilities excluded from coverage
-      ],
-      reporter: ['text', 'json', 'html'],
-      reportsDirectory: './coverage',
-      thresholds: {
-        global: {
-          branches: 80,
-          functions: 80,
-          lines: 80,
-          statements: 80,
-        },
-      },
-    },
-    reporters: [isVerbose ? 'verbose' : 'dot', 'html', './vitest-reporters/split-json-reporter.ts'],
-    outputFile: {
-      html: './test-results/index.html',
-    },
-  },
-  resolve: {
-    alias: {
-      '@': '/src',
-    },
-  },
+	test: {
+		environment: "node",
+		globals: true,
+		setupFiles: ["src/__tests__/setup.ts"],
+		exclude: ["src/__tests__/testRunner.ts"],
+		// Use projects to target unit/integration/e2e groups (Vitest v3)
+		projects: [
+			{
+				test: {
+					name: "unit",
+					include: ["src/**/*.{test,spec}.ts"],
+					exclude: [
+						"src/**/*.integration.test.ts",
+						"src/**/*.e2e.test.ts",
+						// YNABMCPServer.test.ts requires real YNAB API token and makes API calls,
+						// so it runs as part of integration tests, not unit tests
+						"src/server/__tests__/YNABMCPServer.test.ts",
+					],
+				},
+			},
+			{
+				test: {
+					name: "integration:core",
+					include: integrationFiles,
+					env: {
+						INTEGRATION_TEST_TIER: "core",
+					},
+					testTimeout: 30000,
+					hookTimeout: 10000,
+				},
+			},
+			{
+				test: {
+					name: "integration:domain",
+					include: integrationFiles,
+					env: {
+						INTEGRATION_TEST_TIER: "domain",
+					},
+					testTimeout: 60000,
+					hookTimeout: 15000,
+				},
+			},
+			{
+				test: {
+					name: "integration:full",
+					include: integrationFiles,
+					env: {
+						INTEGRATION_TEST_TIER: "full",
+					},
+					testTimeout: 120000,
+					hookTimeout: 30000,
+					fileParallelism: false,
+					maxWorkers: 1,
+				},
+			},
+			{
+				test: {
+					name: "e2e",
+					include: ["src/**/*.e2e.test.ts"],
+				},
+			},
+		],
+		coverage: {
+			provider: "v8",
+			include: ["src/**/*.ts"],
+			exclude: [
+				"src/**/*.{test,spec}.ts",
+				"src/**/*.d.ts",
+				"src/index.ts", // Entry point excluded from coverage
+				"src/__tests__/**/*.ts", // Test utilities excluded from coverage
+			],
+			reporter: ["text", "json", "html"],
+			reportsDirectory: "./coverage",
+			thresholds: {
+				global: {
+					branches: 80,
+					functions: 80,
+					lines: 80,
+					statements: 80,
+				},
+			},
+		},
+		reporters: [
+			isVerbose ? "verbose" : "dot",
+			"html",
+			"./vitest-reporters/split-json-reporter.ts",
+		],
+		outputFile: {
+			html: "./test-results/index.html",
+		},
+	},
+	resolve: {
+		alias: {
+			"@": "/src",
+		},
+	},
 });