feat:format import

Author: lizy-coding

packages/flutterguard_cli/bin/flutterguard.dart

@@ -1,27 +1,13 @@
 import 'dart:io';
 
 import 'package:args/args.dart';
-import 'package:path/path.dart' as p;
 
-import 'package:flutterguard_cli/src/config_loader.dart';
-import 'package:flutterguard_cli/src/file_collector.dart';
 import 'package:flutterguard_cli/src/report_generator.dart';
-import 'package:flutterguard_cli/src/rules/circular_dependency.dart';
-import 'package:flutterguard_cli/src/rules/large_units.dart';
-import 'package:flutterguard_cli/src/rules/layer_violation.dart';
-import 'package:flutterguard_cli/src/rules/lifecycle_resource.dart';
-import 'package:flutterguard_cli/src/rules/missing_const_constructor.dart';
-import 'package:flutterguard_cli/src/rules/module_violation.dart';
-import 'package:flutterguard_cli/src/static_issue.dart';
+import 'package:flutterguard_cli/src/scanner.dart';
 
 const _version = '0.1.0';
 
 void main(List<String> args) {
-  if (args.contains('--version') || args.contains('-V')) {
-    stdout.writeln('flutterguard $_version');
-    exit(0);
-  }
-
   final scanParser = ArgParser()
     ..addOption('path',
         abbr: 'p', defaultsTo: '.', help: 'Project path to scan')
@@ -46,23 +32,39 @@ void main(List<String> args) {
         defaultsTo: 'none',
         allowed: ['none', 'high', 'medium', 'low'],
         help: 'Fail threshold for CI gate')
-    ..addOption('min-score', help: 'Minimum score threshold (0-100)');
+    ..addOption('min-score', help: 'Minimum score threshold (0-100)')
+    ..addFlag('help', abbr: 'h', help: 'Show scan usage', negatable: false);
 
   final parser = ArgParser()
     ..addCommand('scan', scanParser)
-    ..addFlag('help', abbr: 'h', help: 'Show usage', negatable: false);
+    ..addFlag('help', abbr: 'h', help: 'Show usage', negatable: false)
+    ..addFlag('version', abbr: 'V', help: 'Show version', negatable: false);
 
   try {
     final results = parser.parse(args);
 
+    if (results['version'] == true) {
+      stdout.writeln('flutterguard $_version');
+      exit(0);
+    }
+
     if (results['help'] == true || args.isEmpty || args.first == 'help') {
       _printUsage(parser);
       exit(0);
     }
 
     final command = results.command;
-    if (command?.name == 'scan') {
-      _handleScan(command!);
+    if (command == null) {
+      _printUsage(parser);
+      exit(0);
+    }
+
+    if (command.name == 'scan') {
+      if (command['help'] == true) {
+        _printScanUsage(scanParser);
+        exit(0);
+      }
+      _handleScan(command);
     } else {
       _printUsage(parser);
       exit(0);
@@ -76,106 +78,63 @@ void main(List<String> args) {
 }
 
 void _handleScan(ArgResults args) {
-  final projectPath = p.normalize(p.absolute(args['path'] as String));
-  final configPath = args['config'] as String;
-  final outputDir = args['output'] as String;
   final format = args['format'] as String;
   final verbose = args['verbose'] as bool;
   final failOn = args['fail-on'] as String;
   final minScoreStr = args['min-score'] as String?;
+  final minScore = _parseMinScore(minScoreStr);
 
-  if (!Directory(projectPath).existsSync()) {
-    stderr.writeln('Error: Project path "$projectPath" does not exist.');
+  late final ScanResult result;
+  try {
+    result = FlutterGuardScanner.scan(
+      projectPath: args['path'] as String,
+      configPath: args['config'] as String,
+      outputDir: args['output'] as String,
+      writeJson: format == 'json',
+    );
+  } on ScanException catch (e) {
+    stderr.writeln('Error: ${e.message}');
+    exit(2);
+  } on FormatException catch (e) {
+    stderr.writeln('Error: ${e.message}');
     exit(2);
   }
 
-  final config = ScanConfig.fromFile(p.join(projectPath, configPath));
-
-  final files = FileCollector.collect(projectPath, config);
-
-  if (files.isEmpty) {
+  if (result.files.isEmpty) {
     stderr.writeln('No Dart files found. Check your include/exclude patterns.');
     exit(0);
   }
 
-  final allIssues = <StaticIssue>[];
-
-  allIssues.addAll(LargeUnitsRule(
-    largeFileConfig: config.rules.largeFile,
-    largeClassConfig: config.rules.largeClass,
-    largeBuildMethodConfig: config.rules.largeBuildMethod,
-  ).analyze(files));
-  allIssues.addAll(
-      LifecycleResourceRule(config.rules.lifecycleResource).analyze(files));
-  if (config.architecture.layerViolationEnabled) {
-    allIssues.addAll(LayerViolationRule(
-      config.architecture.layers,
-      projectPath: projectPath,
-    ).analyze(files));
-  }
-  if (config.architecture.moduleViolationEnabled) {
-    allIssues.addAll(ModuleViolationRule(
-      config.architecture.modules,
-      projectPath: projectPath,
-    ).analyze(files));
-  }
-  allIssues.addAll(MissingConstConstructorRule(
-    config.rules.missingConstConstructor,
-  ).analyze(files));
-  allIssues.addAll(CircularDependencyRule(
-    enabled: config.architecture.detectCycles,
-    projectPath: projectPath,
-  ).analyze(files));
-
-  allIssues.sort((a, b) {
-    final levelOrder = {
-      RiskLevel.high: 0,
-      RiskLevel.medium: 1,
-      RiskLevel.low: 2,
-    };
-    final cmp = levelOrder[a.level]!.compareTo(levelOrder[b.level]!);
-    if (cmp != 0) return cmp;
-    return a.file.compareTo(b.file);
-  });
-
-  final reportDir =
-      p.isAbsolute(outputDir) ? outputDir : p.join(projectPath, outputDir);
-  Directory(reportDir).createSync(recursive: true);
-
-  if (format == 'json') {
-    final json = ReportGenerator.generateJson(
-      projectPath: projectPath,
-      issues: allIssues,
-    );
-    File(p.join(reportDir, 'report.json')).writeAsStringSync(json);
-  }
-
   final stdoutOutput = ReportGenerator.generateStdout(
-    projectPath: projectPath,
-    issues: allIssues,
+    projectPath: result.projectPath,
+    issues: result.issues,
+    scannedFileCount: result.files.length,
     verbose: verbose,
   );
   stdout.writeln(stdoutOutput);
 
   if (failOn != 'none') {
-    if (ReportGenerator.shouldFail(allIssues, failOn)) {
+    if (ReportGenerator.shouldFail(result.issues, failOn)) {
       stderr
           .writeln('CI gate failed: Issues found at or above "$failOn" level.');
       exit(1);
     }
   }
 
-  if (minScoreStr != null) {
-    final minScore = int.tryParse(minScoreStr);
-    if (minScore != null) {
-      final score = ReportGenerator.calculateScore(allIssues);
-      if (score < minScore) {
-        stderr.writeln(
-            'CI gate failed: Score $score is below minimum $minScore.');
-        exit(1);
-      }
-    }
+  if (minScore != null && result.score < minScore) {
+    stderr.writeln(
+        'CI gate failed: Score ${result.score} is below minimum $minScore.');
+    exit(1);
+  }
+}
+
+int? _parseMinScore(String? value) {
+  if (value == null) return null;
+  final score = int.tryParse(value);
+  if (score == null || score < 0 || score > 100) {
+    throw const FormatException('Expected --min-score to be an integer 0-100.');
   }
+  return score;
 }
 
 void _printUsage(ArgParser parser) {
@@ -206,3 +165,11 @@ void _printUsage(ArgParser parser) {
   stdout.writeln('  flutterguard scan -p ./my_flutter_app');
   stdout.writeln('  flutterguard scan -p . --format json --fail-on high');
 }
+
+void _printScanUsage(ArgParser scanParser) {
+  stdout.writeln('FlutterGuard — scan command');
+  stdout.writeln();
+  stdout.writeln('Usage: flutterguard scan [options]');
+  stdout.writeln();
+  stdout.writeln(scanParser.usage);
+}

packages/flutterguard_cli/lib/flutterguard_cli.dart

@@ -11,4 +11,6 @@ export 'src/rules/layer_violation.dart';
 export 'src/rules/lifecycle_resource.dart';
 export 'src/rules/missing_const_constructor.dart';
 export 'src/rules/module_violation.dart';
+export 'src/scanner.dart';
+export 'src/source_utils.dart';
 export 'src/static_issue.dart';