fix: resolve CI failures in cross-platform testing and Windows builds

- Fix Windows TypeScript compilation errors by adding proper type annotations to api-server.ts
- Add missing .js extensions to ESM imports in cli.ts and commands/index.ts
- Fix cross-platform test CLI flag from --format json to --json
- Add JSON output support to index command for proper CI testing
- Add json option to IndexCliOptions interface
- Implement generateIndexFilesJson function for structured JSON output

Resolves cross-platform CI test failures and Windows build issues.
All platforms now properly detect filesystem capabilities and execute tests.

Author: Mearman

.github/workflows/cross-platform-tests.yml

@@ -174,7 +174,7 @@ jobs:
           
           # Test basic file operations
           echo "Testing file listing..."
-          node ../../dist/cli.js index --format json . || echo "Index command failed"
+          node ../../dist/cli.js index --json . || echo "Index command failed"
           
           # Test file moving with different path styles
           if [ "${{ runner.os }}" = "Windows" ]; then
@@ -201,8 +201,8 @@ jobs:
             echo "✅ Case-sensitive files coexist"
             
             # Test CLI handles case-sensitive files correctly
-            node ../../dist/cli.js index --format json . | grep -i "lowercase.md"
-            node ../../dist/cli.js index --format json . | grep -i "UPPERCASE.md"
+            node ../../dist/cli.js index --json . | grep -i "lowercase.md"
+            node ../../dist/cli.js index --json . | grep -i "UPPERCASE.md"
           else
             echo "❌ Expected case-sensitive files not found"
             ls -la

src/api-server.ts

@@ -124,7 +124,7 @@ async function handleRequest(
 
     // Handle auto-generated API routes
     const autoRoute = autoGeneratedApiRoutes.find(
-      (route) => route.method === method && route.path === path
+      (route: { method: string; path: string; handler: (req: http.IncomingMessage, res: http.ServerResponse, markmv: ReturnType<typeof createMarkMv>) => Promise<void> }) => route.method === method && route.path === path
     );
 
     if (autoRoute) {
@@ -134,7 +134,7 @@ async function handleRequest(
     }
 
     // Route not found
-    const availableRoutes = ['GET /health', ...getApiRoutePaths().map((p) => `POST ${p}`)];
+    const availableRoutes = ['GET /health', ...getApiRoutePaths().map((p: string) => `POST ${p}`)];
     const errorResponse = createErrorResponse(
       404,
       'NotFound',

src/cli.ts

@@ -1,12 +1,12 @@
 #!/usr/bin/env node
 
 import { Command } from 'commander';
-import { convertCommand } from './commands/convert';
-import { indexCommand } from './commands/index';
-import { joinCommand } from './commands/join';
-import { mergeCommand } from './commands/merge';
-import { moveCommand } from './commands/move';
-import { splitCommand } from './commands/split';
+import { convertCommand } from './commands/convert.js';
+import { indexCommand } from './commands/index.js';
+import { joinCommand } from './commands/join.js';
+import { mergeCommand } from './commands/merge.js';
+import { moveCommand } from './commands/move.js';
+import { splitCommand } from './commands/split.js';
 
 const program = new Command();
 

src/commands/index.test.ts

@@ -1,7 +1,7 @@
 import { join } from 'node:path';
 import { beforeEach, describe, expect, it, vi } from 'vitest';
-import { FileUtils } from '../utils/file-utils';
-import { indexCommand } from './index';
+import { FileUtils } from '../utils/file-utils.js';
+import { indexCommand } from './index.js';
 
 // Mock FileUtils
 vi.mock('../utils/file-utils');

src/commands/index.ts

@@ -1,7 +1,7 @@
 import { existsSync, statSync } from 'node:fs';
 import { join, relative, resolve } from 'node:path';
 import { glob } from 'glob';
-import { FileUtils } from '../utils/file-utils';
+import { FileUtils } from '../utils/file-utils.js';
 
 // Test PR creation with trailing spaces on main branch
 
@@ -99,6 +99,7 @@ interface IndexCliOptions {
   template?: string;
   dryRun?: boolean;
   verbose?: boolean;
+  json?: boolean;
   maxDepth?: number;
   noTraverseUp?: boolean;
   boundary?: string;
@@ -145,7 +146,65 @@ export async function indexCommand(
     ...(cliOptions.boundary && { boundary: cliOptions.boundary }),
   };
 
-  return generateIndexFiles(options, directory || '.');
+  if (cliOptions.json) {
+    return generateIndexFilesJson(options, directory || '.');
+  } else {
+    return generateIndexFiles(options, directory || '.');
+  }
+}
+
+/** Generate index files for markdown documentation (JSON output) */
+async function generateIndexFilesJson(options: IndexOptions, directory: string): Promise<void> {
+  const targetDir = resolve(directory);
+
+  if (!existsSync(targetDir)) {
+    throw new Error(`Directory not found: ${targetDir}`);
+  }
+
+  if (!statSync(targetDir).isDirectory()) {
+    throw new Error(`Path is not a directory: ${targetDir}`);
+  }
+
+  try {
+    // Discover markdown files
+    const files = await discoverMarkdownFiles(targetDir, options);
+
+    // Organize files based on strategy
+    const organizedFiles = organizeFiles(files, options);
+
+    // Convert to JSON output
+    const jsonOutput = {
+      directory: targetDir,
+      options: {
+        type: options.type,
+        strategy: options.strategy,
+        location: options.location,
+      },
+      totalFiles: files.length,
+      organizedFiles: Object.fromEntries(
+        Array.from(organizedFiles.entries()).map(([key, groupFiles]) => [
+          key,
+          groupFiles.map(file => ({
+            path: file.path,
+            relativePath: file.relativePath,
+            title: file.metadata.title || file.relativePath,
+          }))
+        ])
+      ),
+      files: files.map(file => ({
+        path: file.path,
+        relativePath: file.relativePath,
+        title: file.metadata.title || file.relativePath,
+      }))
+    };
+
+    console.log(JSON.stringify(jsonOutput, null, 2));
+  } catch (error) {
+    if (error instanceof Error) {
+      throw new Error(`Failed to generate index: ${error.message}`);
+    }
+    throw error;
+  }
 }
 
 /** Generate index files for markdown documentation */

test-data/cross-platform/UPPERCASE.md

@@ -0,0 +1 @@
+uppercase content

test-data/cross-platform/index.md

@@ -0,0 +1,15 @@
+---
+generated: true
+generator: markmv-index
+type: links
+strategy: directory
+updated: 2025-06-15T23:00:53.921Z
+---
+
+# Documentation Index
+
+## Root
+
+- [lowercase](lowercase.md)
+- [UPPERCASE](UPPERCASE.md)
+

test-data/cross-platform/lowercase.md

@@ -0,0 +1 @@
+lowercase content