Merge pull request #38 from DGouron/feat/cli-init-wizard

feat(cli): add reviewflow init wizard and validate command

Author: DGouron

README.md

@@ -125,40 +125,65 @@ Review behavior is defined by [Claude Code skills](https://docs.anthropic.com/en
 
 ## Quick Start
 
-### Install globally
+### 1. Install
 
 ```bash
 npm install -g reviewflow
-# or
-yarn global add reviewflow
 ```
 
-### Or run directly with npx
+### 2. Initialize
 
 ```bash
-npx reviewflow --help
-npx reviewflow start
+reviewflow init
 ```
 
-### Configure & run
+The interactive wizard will:
+- Configure server port and usernames
+- Generate webhook secrets
+- Scan your filesystem for git repositories
+- Set up MCP server integration with Claude Code
 
-```bash
-# Configure
-cp .env.example .env
-cp config.example.json config.json
-# Edit config.json with your repositories
+For non-interactive setup: `reviewflow init --yes`
+
+### 3. Start
 
-# Start the server
+```bash
 reviewflow start
 # Dashboard at http://localhost:3847
 ```
 
 Then [configure a webhook](https://dgouron.github.io/review-flow/guide/quick-start) on your GitLab/GitHub project pointing to your server.
 
+### Validate your setup
+
+```bash
+reviewflow validate
+```
+
 For detailed setup, see the **[Quick Start Guide](https://dgouron.github.io/review-flow/guide/quick-start)**.
 
 ---
 
+## CLI Reference
+
+| Command | Description |
+|---------|-------------|
+| `reviewflow init` | Interactive setup wizard |
+| `reviewflow start` | Start the review server |
+| `reviewflow stop` | Stop the running daemon |
+| `reviewflow status` | Show server status |
+| `reviewflow logs` | Show daemon logs |
+| `reviewflow validate` | Validate configuration |
+
+| Init Flag | Description |
+|-----------|-------------|
+| `-y, --yes` | Accept all defaults (non-interactive) |
+| `--skip-mcp` | Skip MCP server configuration |
+| `--show-secrets` | Display full webhook secrets |
+| `--scan-path <path>` | Custom scan path (repeatable) |
+
+---
+
 ## Documentation
 
 | Topic | Link |

package.json

@@ -56,6 +56,7 @@
   "dependencies": {
     "@fastify/static": "^8.0.0",
     "@fastify/websocket": "^11.0.0",
+    "@inquirer/prompts": "^8.2.0",
     "@modelcontextprotocol/sdk": "^1.26.0",
     "dotenv": "^16.4.0",
     "fastify": "^5.0.0",

src/cli/formatters/initSummary.ts

@@ -0,0 +1,46 @@
+export interface InitSummaryInput {
+  configPath: string;
+  envPath: string;
+  port: number;
+  repositoryCount: number;
+  mcpStatus: 'configured' | 'already-configured' | 'claude-not-found' | 'skipped' | 'failed';
+  gitlabUsername: string;
+  githubUsername: string;
+}
+
+function mcpLine(status: InitSummaryInput['mcpStatus']): string {
+  switch (status) {
+    case 'configured':
+      return '  MCP server:    configured';
+    case 'already-configured':
+      return '  MCP server:    already configured';
+    case 'claude-not-found':
+      return '  MCP server:    skipped (Claude CLI not found)';
+    case 'skipped':
+      return '  MCP server:    skipped by user';
+    case 'failed':
+      return '  MCP server:    configuration failed';
+  }
+}
+
+export function formatInitSummary(input: InitSummaryInput): string {
+  const lines: string[] = [
+    '',
+    'ReviewFlow initialized successfully!',
+    '',
+    'Configuration:',
+    `  Config file:   ${input.configPath}`,
+    `  Env file:      ${input.envPath}`,
+    `  Port:          ${input.port}`,
+    `  Repositories:  ${input.repositoryCount}`,
+    mcpLine(input.mcpStatus),
+    '',
+    'Next steps:',
+    '  1. Configure webhook secrets on your GitLab/GitHub projects',
+    '  2. Start the server: reviewflow start',
+    '  3. Check status:     reviewflow validate',
+    '',
+  ];
+
+  return lines.join('\n');
+}

src/cli/parseCliArgs.ts

@@ -22,6 +22,19 @@ interface LogsArgs {
   lines: number;
 }
 
+interface InitArgs {
+  command: 'init';
+  yes: boolean;
+  skipMcp: boolean;
+  showSecrets: boolean;
+  scanPaths: string[];
+}
+
+interface ValidateArgs {
+  command: 'validate';
+  fix: boolean;
+}
+
 interface VersionArgs {
   command: 'version';
 }
@@ -30,9 +43,9 @@ interface HelpArgs {
   command: 'help';
 }
 
-export type CliArgs = StartArgs | StopArgs | StatusArgs | LogsArgs | VersionArgs | HelpArgs;
+export type CliArgs = StartArgs | StopArgs | StatusArgs | LogsArgs | InitArgs | ValidateArgs | VersionArgs | HelpArgs;
 
-const KNOWN_COMMANDS = ['start', 'stop', 'status', 'logs'] as const;
+const KNOWN_COMMANDS = ['start', 'stop', 'status', 'logs', 'init', 'validate'] as const;
 type KnownCommand = (typeof KNOWN_COMMANDS)[number];
 
 function hasFlag(args: string[], long: string, short?: string): boolean {
@@ -90,6 +103,33 @@ function parseLogsArgs(args: string[]): LogsArgs {
   };
 }
 
+function getAllFlagValues(args: string[], long: string): string[] {
+  const values: string[] = [];
+  for (let index = 0; index < args.length; index++) {
+    if (args[index] === long && args[index + 1] !== undefined) {
+      values.push(args[index + 1]);
+    }
+  }
+  return values;
+}
+
+function parseInitArgs(args: string[]): InitArgs {
+  return {
+    command: 'init',
+    yes: hasFlag(args, '--yes', '-y'),
+    skipMcp: hasFlag(args, '--skip-mcp'),
+    showSecrets: hasFlag(args, '--show-secrets'),
+    scanPaths: getAllFlagValues(args, '--scan-path'),
+  };
+}
+
+function parseValidateArgs(args: string[]): ValidateArgs {
+  return {
+    command: 'validate',
+    fix: hasFlag(args, '--fix'),
+  };
+}
+
 export function parseCliArgs(args: string[]): CliArgs {
   if (hasFlag(args, '--version', '-v')) {
     return { command: 'version' };
@@ -110,5 +150,9 @@ export function parseCliArgs(args: string[]): CliArgs {
       return parseStatusArgs(args);
     case 'logs':
       return parseLogsArgs(args);
+    case 'init':
+      return parseInitArgs(args);
+    case 'validate':
+      return parseValidateArgs(args);
   }
 }

src/frameworks/config/configLoader.ts

@@ -2,8 +2,16 @@ import { readFileSync, existsSync } from 'node:fs';
 import { execSync } from 'node:child_process';
 import { join } from 'node:path';
 import { config as loadEnv } from 'dotenv';
+import { getConfigDir } from '../../shared/services/configDir.js';
 
-loadEnv();
+const configDir = getConfigDir();
+const xdgEnvPath = join(configDir, '.env');
+
+if (existsSync(xdgEnvPath)) {
+  loadEnv({ path: xdgEnvPath });
+} else {
+  loadEnv();
+}
 
 // Types for simplified config input
 interface RepositoryInput {
@@ -214,13 +222,22 @@ function loadSecrets(): EnvSecrets {
 let cachedConfig: Config | null = null;
 let cachedSecrets: EnvSecrets | null = null;
 
+function resolveConfigPath(): string {
+  if (process.env.CONFIG_PATH) return process.env.CONFIG_PATH;
+  const cwdPath = join(process.cwd(), 'config.json');
+  if (existsSync(cwdPath)) return cwdPath;
+  const xdgPath = join(configDir, 'config.json');
+  if (existsSync(xdgPath)) return xdgPath;
+  return cwdPath;
+}
+
 export function loadConfig(): Config {
   if (cachedConfig) return cachedConfig;
 
-  const configPath = process.env.CONFIG_PATH || join(process.cwd(), 'config.json');
+  const configPath = resolveConfigPath();
 
   if (!existsSync(configPath)) {
-    throw new Error(`Fichier de configuration non trouvé : ${configPath}`);
+    throw new Error(`Configuration file not found: ${configPath}\nRun 'reviewflow init' to create one.`);
   }
 
   const rawContent = readFileSync(configPath, 'utf-8');