auto-cr-cmd 2.0.21 → 2.0.23

package/dist/index.js

@@ -73,7 +73,7 @@ var auto_cr_rules_1 = require("auto-cr-rules");
 var loader_1 = require("./rules/loader");
 function run() {
     return __awaiter(this, arguments, void 0, function (filePaths, ruleDir) {
-        var t, validPaths, allFiles, _i, validPaths_1, targetPath, stat, directoryFiles, customRules, rules, _a, allFiles_1, file, error_1;
+        var t, validPaths, allFiles, _i, validPaths_1, targetPath, stat, directoryFiles, scannableFiles, customRules, rules, filesWithErrors, filesWithWarnings, filesWithOptimizing, _a, scannableFiles_1, file, summary, error_1;
         if (filePaths === void 0) { filePaths = []; }
         return __generator(this, function (_b) {
             switch (_b.label) {
@@ -84,12 +84,12 @@ function run() {
                     _b.trys.push([1, 6, , 7]);
                     if (filePaths.length === 0) {
                         consola_1.consola.info(t.noPathsProvided());
-                        return [2 /*return*/];
+                        return [2 /*return*/, { scannedFiles: 0, filesWithErrors: 0, filesWithWarnings: 0, filesWithOptimizing: 0 }];
                     }
                     validPaths = filePaths.filter(function (candidate) { return (0, file_1.checkPathExists)(candidate); });
                     if (validPaths.length === 0) {
                         consola_1.consola.error(t.allPathsMissing());
-                        return [2 /*return*/];
+                        return [2 /*return*/, { scannedFiles: 0, filesWithErrors: 0, filesWithWarnings: 0, filesWithOptimizing: 0 }];
                     }
                     allFiles = [];
                     for (_i = 0, validPaths_1 = validPaths; _i < validPaths_1.length; _i++) {
@@ -105,38 +105,53 @@ function run() {
                     }
                     if (allFiles.length === 0) {
                         consola_1.consola.info(t.noFilesFound());
-                        return [2 /*return*/];
+                        return [2 /*return*/, { scannedFiles: 0, filesWithErrors: 0, filesWithWarnings: 0, filesWithOptimizing: 0 }];
                     }
+                    scannableFiles = allFiles.filter(function (candidate) { return !candidate.endsWith('.d.ts'); });
                     customRules = (0, loader_1.loadCustomRules)(ruleDir);
                     rules = __spreadArray(__spreadArray([], auto_cr_rules_1.builtinRules, true), customRules, true);
                     if (rules.length === 0) {
                         consola_1.consola.warn(t.noRulesLoaded());
-                        return [2 /*return*/];
+                        return [2 /*return*/, {
+                                scannedFiles: 0,
+                                filesWithErrors: 0,
+                                filesWithWarnings: 0,
+                                filesWithOptimizing: 0,
+                            }];
                     }
-                    _a = 0, allFiles_1 = allFiles;
+                    filesWithErrors = 0;
+                    filesWithWarnings = 0;
+                    filesWithOptimizing = 0;
+                    _a = 0, scannableFiles_1 = scannableFiles;
                     _b.label = 2;
                 case 2:
-                    if (!(_a < allFiles_1.length)) return [3 /*break*/, 5];
-                    file = allFiles_1[_a];
-                    if (file.endsWith('.d.ts')) {
-                        return [3 /*break*/, 4];
-                    }
+                    if (!(_a < scannableFiles_1.length)) return [3 /*break*/, 5];
+                    file = scannableFiles_1[_a];
                     return [4 /*yield*/, analyzeFile(file, rules)];
                 case 3:
-                    _b.sent();
+                    summary = _b.sent();
+                    if (summary.severityCounts.error > 0) {
+                        filesWithErrors += 1;
+                    }
+                    if (summary.severityCounts.warning > 0) {
+                        filesWithWarnings += 1;
+                    }
+                    if (summary.severityCounts.optimizing > 0) {
+                        filesWithOptimizing += 1;
+                    }
                     _b.label = 4;
                 case 4:
                     _a++;
                     return [3 /*break*/, 2];
-                case 5:
-                    consola_1.consola.log(' ');
-                    consola_1.consola.success(t.scanComplete());
-                    return [3 /*break*/, 7];
+                case 5: return [2 /*return*/, {
+                        scannedFiles: scannableFiles.length,
+                        filesWithErrors: filesWithErrors,
+                        filesWithWarnings: filesWithWarnings,
+                        filesWithOptimizing: filesWithOptimizing,
+                    }];
                 case 6:
                     error_1 = _b.sent();
-                    consola_1.consola.error(t.scanError(), error_1 instanceof Error ? error_1.message : error_1);
-                    process.exit(1);
-                    return [3 /*break*/, 7];
+                    throw error_1 instanceof Error ? error_1 : new Error(String(error_1));
                 case 7: return [2 /*return*/];
             }
         });
@@ -144,7 +159,7 @@ function run() {
 }
 function analyzeFile(file, rules) {
     return __awaiter(this, void 0, void 0, function () {
-        var source, reporter, t, ast, parseOptions, language, baseContext, sharedHelpers, _loop_1, _i, rules_1, rule;
+        var source, reporter, t, ast, parseOptions, language, baseContext, sharedHelpers, _loop_1, _i, rules_1, rule, summary;
         return __generator(this, function (_a) {
             switch (_a.label) {
                 case 0:
@@ -157,7 +172,13 @@ function analyzeFile(file, rules) {
                     }
                     catch (error) {
                         consola_1.consola.error(t.parseFileFailed({ file: file }), error instanceof Error ? error.message : error);
-                        return [2 /*return*/];
+                        return [2 /*return*/, {
+                                severityCounts: {
+                                    error: 1,
+                                    warning: 0,
+                                    optimizing: 0,
+                                },
+                            }];
                     }
                     language = (0, i18n_1.getLanguage)();
                     baseContext = (0, auto_cr_rules_1.createRuleContext)({
@@ -169,28 +190,35 @@ function analyzeFile(file, rules) {
                     });
                     sharedHelpers = baseContext.helpers;
                     _loop_1 = function (rule) {
-                        var scopedReporter_1, helpers, context, error_2;
+                        var scopedReporter_1, reporterWithRecord_1, helpers, context, error_2;
                         return __generator(this, function (_b) {
                             switch (_b.label) {
                                 case 0:
                                     _b.trys.push([0, 2, , 3]);
                                     scopedReporter_1 = reporter.forRule(rule);
-                                    helpers = __assign(__assign({}, sharedHelpers), { reportViolation: function (input, span) {
-                                            var normalized = normalizeViolationInputForCli(input, span);
-                                            if (typeof scopedReporter_1.record === 'function') {
-                                                scopedReporter_1.record(normalized);
+                                    reporterWithRecord_1 = scopedReporter_1;
+                                    helpers = __assign(__assign({}, sharedHelpers), { reportViolation: (function (input, span) {
+                                            var normalized = normalizeViolationInput(input, span);
+                                            if (typeof reporterWithRecord_1.record === 'function') {
+                                                reporterWithRecord_1.record({
+                                                    description: normalized.message,
+                                                    code: normalized.code,
+                                                    suggestions: normalized.suggestions,
+                                                    span: normalized.span,
+                                                    line: normalized.line,
+                                                });
                                                 return;
                                             }
                                             if (normalized.span) {
-                                                scopedReporter_1.errorAtSpan(normalized.span, normalized.description);
+                                                scopedReporter_1.errorAtSpan(normalized.span, normalized.message);
                                                 return;
                                             }
                                             if (typeof normalized.line === 'number') {
-                                                scopedReporter_1.errorAtLine(normalized.line, normalized.description);
+                                                scopedReporter_1.errorAtLine(normalized.line, normalized.message);
                                                 return;
                                             }
-                                            scopedReporter_1.error(normalized.description);
-                                        } });
+                                            scopedReporter_1.error(normalized.message);
+                                        }) });
                                     context = __assign(__assign({}, baseContext), { reporter: scopedReporter_1, helpers: helpers });
                                     return [4 /*yield*/, rule.run(context)];
                                 case 1:
@@ -217,27 +245,64 @@ function analyzeFile(file, rules) {
                     _i++;
                     return [3 /*break*/, 1];
                 case 4:
-                    reporter.flush();
-                    return [2 /*return*/];
+                    summary = reporter.flush();
+                    return [2 /*return*/, {
+                            severityCounts: summary.severityCounts,
+                        }];
             }
         });
     });
 }
-function normalizeViolationInputForCli(input, spanArg) {
-    var _a, _b;
+function normalizeViolationInput(input, spanArg) {
+    var _a;
     if (typeof input === 'string') {
         return {
-            description: input,
+            message: input,
             span: spanArg,
         };
     }
-    var description = (_a = input.description) !== null && _a !== void 0 ? _a : input.message;
+    if (input && typeof input === 'object') {
+        var candidate = input;
+        var description = typeof candidate.description === 'string'
+            ? candidate.description
+            : typeof candidate.message === 'string'
+                ? candidate.message
+                : undefined;
+        var code = typeof candidate.code === 'string' ? candidate.code : undefined;
+        var suggestions = void 0;
+        if (Array.isArray(candidate.suggestions)) {
+            var normalizedSuggestions = [];
+            for (var _i = 0, _b = candidate.suggestions; _i < _b.length; _i++) {
+                var entry = _b[_i];
+                if (typeof entry === 'string') {
+                    normalizedSuggestions.push({ text: entry });
+                    continue;
+                }
+                if (entry && typeof entry === 'object') {
+                    var suggestion = entry;
+                    if (typeof suggestion.text === 'string') {
+                        normalizedSuggestions.push({
+                            text: suggestion.text,
+                            link: typeof suggestion.link === 'string' ? suggestion.link : undefined,
+                        });
+                    }
+                }
+            }
+            if (normalizedSuggestions.length > 0) {
+                suggestions = normalizedSuggestions;
+            }
+        }
+        return {
+            message: description !== null && description !== void 0 ? description : 'Rule violation detected.',
+            span: (_a = candidate.span) !== null && _a !== void 0 ? _a : spanArg,
+            line: typeof candidate.line === 'number' ? candidate.line : undefined,
+            code: code,
+            suggestions: suggestions,
+        };
+    }
     return {
-        description: description !== null && description !== void 0 ? description : 'Rule violation detected.',
-        code: input.code,
-        suggestions: input.suggestions,
-        span: (_b = input.span) !== null && _b !== void 0 ? _b : spanArg,
-        line: input.line,
+        message: 'Rule violation detected.',
+        span: spanArg,
     };
 }
 commander_1.program
@@ -248,7 +313,7 @@ commander_1.program
 var options = commander_1.program.opts();
 var filePaths = commander_1.program.args.map(function (target) { return path_1.default.resolve(process.cwd(), target); });
 (function () { return __awaiter(void 0, void 0, void 0, function () {
-    var error_3, t;
+    var summary, t, language, resultMessage, error_3, t;
     var _a;
     return __generator(this, function (_b) {
         switch (_b.label) {
@@ -257,13 +322,31 @@ var filePaths = commander_1.program.args.map(function (target) { return path_1.d
                 (0, i18n_1.setLanguage)((_a = options.language) !== null && _a !== void 0 ? _a : process.env.LANG);
                 return [4 /*yield*/, run(filePaths, options.ruleDir)];
             case 1:
-                _b.sent();
-                process.exit(0);
+                summary = _b.sent();
+                t = (0, i18n_1.getTranslator)();
+                if (summary.scannedFiles > 0) {
+                    consola_1.consola.log(' ');
+                    language = (0, i18n_1.getLanguage)();
+                    resultMessage = language.startsWith('zh')
+                        ? " ".concat(t.scanComplete(), "\uFF0C\u672C\u6B21\u5171\u626B\u63CF").concat(summary.scannedFiles, "\u4E2A\u6587\u4EF6\uFF0C\u5176\u4E2D").concat(summary.filesWithErrors, "\u4E2A\u6587\u4EF6\u5B58\u5728\u9519\u8BEF\uFF0C").concat(summary.filesWithWarnings, "\u4E2A\u6587\u4EF6\u5B58\u5728\u8B66\u544A\uFF0C").concat(summary.filesWithOptimizing, "\u4E2A\u6587\u4EF6\u5B58\u5728\u4F18\u5316\u5EFA\u8BAE\uFF01")
+                        : " ".concat(t.scanComplete(), ", scanned ").concat(summary.scannedFiles, " files: ").concat(summary.filesWithErrors, " with errors, ").concat(summary.filesWithWarnings, " with warnings, ").concat(summary.filesWithOptimizing, " with optimizing hints!");
+                    if (summary.filesWithErrors > 0) {
+                        consola_1.consola.success(resultMessage);
+                        process.exit(1);
+                    }
+                    else {
+                        consola_1.consola.success(resultMessage);
+                        process.exit(0);
+                    }
+                }
+                else {
+                    process.exit(0);
+                }
                 return [3 /*break*/, 3];
             case 2:
                 error_3 = _b.sent();
                 t = (0, i18n_1.getTranslator)();
-                consola_1.consola.error(t.unexpectedError(), error_3);
+                consola_1.consola.error(t.scanError(), error_3 instanceof Error ? error_3.message : error_3);
                 process.exit(1);
                 return [3 /*break*/, 3];
             case 3: return [2 /*return*/];

package/dist/report/index.js

@@ -19,15 +19,33 @@ function createReporter(filePath, source) {
     var t = (0, i18n_1.getTranslator)();
     var language = (0, i18n_1.getLanguage)();
     var records = [];
+    var totalViolations = 0;
+    var errorViolations = 0;
+    var severityCounts = {
+        error: 0,
+        warning: 0,
+        optimizing: 0,
+    };
     var pushRecord = function (record) {
         records.push(record);
+        totalViolations += 1;
+        if (record.severity === auto_cr_rules_1.RuleSeverity.Error) {
+            errorViolations += 1;
+            severityCounts.error += 1;
+        }
+        else if (record.severity === auto_cr_rules_1.RuleSeverity.Warning) {
+            severityCounts.warning += 1;
+        }
+        else if (record.severity === auto_cr_rules_1.RuleSeverity.Optimizing) {
+            severityCounts.optimizing += 1;
+        }
     };
     var error = function (message) {
         pushRecord({
             tag: UNTAGGED_TAG,
             ruleName: 'general',
             severity: auto_cr_rules_1.RuleSeverity.Error,
-            description: message,
+            message: message,
         });
     };
     var errorAtLine = function (line, message) {
@@ -35,8 +53,8 @@ function createReporter(filePath, source) {
             tag: UNTAGGED_TAG,
             ruleName: 'general',
             severity: auto_cr_rules_1.RuleSeverity.Error,
-            description: message,
             line: line,
+            message: message,
         });
     };
     var errorAtSpan = function (spanLike, message) {
@@ -57,17 +75,17 @@ function createReporter(filePath, source) {
                 tag: tag,
                 ruleName: rule.name,
                 severity: severity,
-                description: payload.description,
+                message: payload.message,
                 line: payload.line,
                 code: payload.code,
                 suggestions: payload.suggestions,
             });
         };
         var scopedError = function (message) {
-            store({ description: message });
+            store({ message: message });
         };
         var scopedErrorAtLine = function (line, message) {
-            store({ description: message, line: line });
+            store({ message: message, line: line });
         };
         var scopedErrorAtSpan = function (spanLike, message) {
             var span = extractSpan(spanLike);
@@ -78,25 +96,36 @@ function createReporter(filePath, source) {
             var line = offsetToLine(span.start, offsets);
             scopedErrorAtLine(line, message);
         };
-        var record = function (violation) {
-            var line = resolveLine(violation, offsets);
+        var record = function (payload) {
+            var line = resolveLine(payload, offsets);
             store({
-                description: violation.description,
+                message: payload.description,
                 line: line,
-                code: violation.code,
-                suggestions: violation.suggestions,
+                code: payload.code,
+                suggestions: payload.suggestions,
             });
         };
-        return {
+        var reporterWithRecord = {
             error: scopedError,
             errorAtLine: scopedErrorAtLine,
             errorAtSpan: scopedErrorAtSpan,
             record: record,
         };
+        return reporterWithRecord;
     };
     var flush = function () {
+        var summary = {
+            totalViolations: totalViolations,
+            errorViolations: errorViolations,
+            severityCounts: {
+                error: severityCounts.error,
+                warning: severityCounts.warning,
+                optimizing: severityCounts.optimizing,
+            },
+        };
         if (records.length === 0) {
-            return;
+            resetCounters();
+            return summary;
         }
         var locale = language === 'zh' ? 'zh-CN' : 'en-US';
         var formatter = new Intl.DateTimeFormat(locale, {
@@ -108,7 +137,7 @@ function createReporter(filePath, source) {
         var indent = '    ';
         var colon = language === 'zh' ? '：' : ':';
         var headerGap = language === 'zh' ? '' : ' ';
-        records.forEach(function (violation, index) {
+        records.forEach(function (violation) {
             var timestamp = formatter.format(new Date());
             var tagLabel = t.ruleTagLabel({ tag: violation.tag });
             var severityIcon = t.reporterSeverityIcon({ severity: violation.severity });
@@ -117,10 +146,10 @@ function createReporter(filePath, source) {
             logger(header);
             var location = typeof violation.line === 'number' ? "".concat(filePath, ":").concat(violation.line) : filePath;
             consola_1.default.log("".concat(indent).concat(t.reporterFileLabel(), ": ").concat(location));
+            consola_1.default.log("".concat(indent).concat(t.reporterDescriptionLabel(), ": ").concat(violation.message));
             if (violation.code) {
                 consola_1.default.log("".concat(indent).concat(t.reporterCodeLabel(), ": ").concat(violation.code));
             }
-            consola_1.default.log("".concat(indent).concat(t.reporterDescriptionLabel(), ": ").concat(violation.description));
             if (violation.suggestions && violation.suggestions.length > 0) {
                 var suggestionSeparator = language === 'zh' ? '； ' : ' | ';
                 var suggestionLine = violation.suggestions
@@ -130,6 +159,15 @@ function createReporter(filePath, source) {
             }
         });
         records.length = 0;
+        resetCounters();
+        return summary;
+    };
+    var resetCounters = function () {
+        totalViolations = 0;
+        errorViolations = 0;
+        severityCounts.error = 0;
+        severityCounts.warning = 0;
+        severityCounts.optimizing = 0;
     };
     return Object.assign({ error: error, errorAtLine: errorAtLine, errorAtSpan: errorAtSpan }, {
         forRule: buildRuleReporter,
@@ -140,12 +178,12 @@ function getLoggerForSeverity(severity) {
     var _a;
     return (_a = severityLoggers[severity]) !== null && _a !== void 0 ? _a : consola_1.default.error;
 }
-function resolveLine(violation, offsets) {
-    if (typeof violation.line === 'number') {
-        return violation.line;
+function resolveLine(record, offsets) {
+    if (typeof record.line === 'number') {
+        return record.line;
     }
-    if (violation.span) {
-        return offsetToLine(violation.span.start, offsets);
+    if (record.span) {
+        return offsetToLine(record.span.start, offsets);
     }
     return undefined;
 }

package/dist/types/report/index.d.ts

@@ -1,6 +1,16 @@
 import type { Rule, RuleReporter } from 'auto-cr-rules';
 export interface Reporter extends RuleReporter {
     forRule(rule: Pick<Rule, 'name' | 'tag' | 'severity'>): RuleReporter;
-    flush(): void;
+    flush(): ReporterSummary;
+}
+interface ReporterSummary {
+    totalViolations: number;
+    errorViolations: number;
+    severityCounts: {
+        error: number;
+        warning: number;
+        optimizing: number;
+    };
 }
 export declare function createReporter(filePath: string, source: string): Reporter;
+export {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "auto-cr-cmd",
-  "version": "2.0.21",
+  "version": "2.0.23",
   "description": "Fast automated code review CLI powered by SWC-based static analysis",
   "main": "./dist/index.js",
   "types": "./dist/types/index.d.ts",
@@ -48,15 +48,15 @@
   "dependencies": {
     "@swc/core": "^1.13.20",
     "@swc/wasm": "^1.13.20",
-    "auto-cr-rules": "^2.0.21",
+    "auto-cr-rules": "^2.0.23",
     "commander": "^14.0.0",
     "consola": "^3.4.2"
   },
   "devDependencies": {
-    "@eslint/js": "^9.35.0",
+    "@eslint/js": "^9.37.0",
     "@swc/types": "^0.1.25",
     "@types/node": "^24.3.1",
-    "eslint": "^9.35.0",
+    "eslint": "^9.37.0",
     "eslint-plugin-prettier": "^5.5.4",
     "globals": "^16.3.0",
     "jiti": "^2.5.1",