autosnippet 3.0.0 → 3.0.2

package/lib/service/guard/ExclusionManager.js

@@ -4,14 +4,14 @@
  * 持久化到 AutoSnippet/guard-exclusions.json（Git 友好，跟随知识库提交）
  */
 
-import { readFileSync, writeFileSync, mkdirSync, existsSync, unlinkSync } from 'node:fs';
-import { join, dirname, relative } from 'node:path';
+import { existsSync, mkdirSync, readFileSync, unlinkSync, writeFileSync } from 'node:fs';
+import { dirname, join } from 'node:path';
 import Logger from '../../infrastructure/logging/Logger.js';
 import pathGuard from '../../shared/PathGuard.js';
 
 export class ExclusionManager {
   #exclusionsPath;
-  #data;   // { pathExclusions: [], ruleExclusions: {}, globalRuleExclusions: [] }
+  #data; // { pathExclusions: [], ruleExclusions: {}, globalRuleExclusions: [] }
 
   constructor(projectRoot, options = {}) {
     const kbDir = options.knowledgeBaseDir || 'AutoSnippet';
@@ -30,9 +30,13 @@ export class ExclusionManager {
    * @param {{ reason?: string }} meta
    */
   addPathExclusion(pattern, meta = {}) {
-    if (!pattern) return;
-    const exists = this.#data.pathExclusions.find(e => e.pattern === pattern);
-    if (exists) return;
+    if (!pattern) {
+      return;
+    }
+    const exists = this.#data.pathExclusions.find((e) => e.pattern === pattern);
+    if (exists) {
+      return;
+    }
     this.#data.pathExclusions.push({
       pattern,
       reason: meta.reason || '',
@@ -45,14 +49,14 @@ export class ExclusionManager {
    * 检查文件路径是否被排除
    */
   isPathExcluded(filePath) {
-    return this.#data.pathExclusions.some(e => this.#matchGlob(filePath, e.pattern));
+    return this.#data.pathExclusions.some((e) => this.#matchGlob(filePath, e.pattern));
   }
 
   /**
    * 移除路径排除
    */
   removePathExclusion(pattern) {
-    this.#data.pathExclusions = this.#data.pathExclusions.filter(e => e.pattern !== pattern);
+    this.#data.pathExclusions = this.#data.pathExclusions.filter((e) => e.pattern !== pattern);
     this.#save();
   }
 
@@ -68,7 +72,9 @@ export class ExclusionManager {
       this.#data.ruleExclusions[ruleId] = [];
     }
     const list = this.#data.ruleExclusions[ruleId];
-    if (list.find(e => e.filePath === filePath)) return;
+    if (list.find((e) => e.filePath === filePath)) {
+      return;
+    }
     list.push({ filePath, reason: meta.reason || '', addedAt: new Date().toISOString() });
     this.#save();
   }
@@ -77,10 +83,14 @@ export class ExclusionManager {
    * 检查规则在特定文件是否被排除
    */
   isRuleExcluded(ruleId, filePath) {
-    if (this.isRuleGloballyDisabled(ruleId)) return true;
+    if (this.isRuleGloballyDisabled(ruleId)) {
+      return true;
+    }
     const list = this.#data.ruleExclusions[ruleId];
-    if (!list) return false;
-    return list.some(e => e.filePath === filePath || this.#matchGlob(filePath, e.filePath));
+    if (!list) {
+      return false;
+    }
+    return list.some((e) => e.filePath === filePath || this.#matchGlob(filePath, e.filePath));
   }
 
   /**
@@ -88,8 +98,10 @@ export class ExclusionManager {
    */
   removeRuleExclusion(ruleId, filePath) {
     const list = this.#data.ruleExclusions[ruleId];
-    if (!list) return;
-    this.#data.ruleExclusions[ruleId] = list.filter(e => e.filePath !== filePath);
+    if (!list) {
+      return;
+    }
+    this.#data.ruleExclusions[ruleId] = list.filter((e) => e.filePath !== filePath);
     if (this.#data.ruleExclusions[ruleId].length === 0) {
       delete this.#data.ruleExclusions[ruleId];
     }
@@ -102,7 +114,9 @@ export class ExclusionManager {
    * 全局禁用某条规则
    */
   addGlobalRuleExclusion(ruleId, meta = {}) {
-    if (this.#data.globalRuleExclusions.find(e => e.ruleId === ruleId)) return;
+    if (this.#data.globalRuleExclusions.find((e) => e.ruleId === ruleId)) {
+      return;
+    }
     this.#data.globalRuleExclusions.push({
       ruleId,
       reason: meta.reason || '',
@@ -115,14 +129,16 @@ export class ExclusionManager {
    * 检查规则是否被全局禁用
    */
   isRuleGloballyDisabled(ruleId) {
-    return this.#data.globalRuleExclusions.some(e => e.ruleId === ruleId);
+    return this.#data.globalRuleExclusions.some((e) => e.ruleId === ruleId);
   }
 
   /**
    * 移除全局规则排除
    */
   removeGlobalRuleExclusion(ruleId) {
-    this.#data.globalRuleExclusions = this.#data.globalRuleExclusions.filter(e => e.ruleId !== ruleId);
+    this.#data.globalRuleExclusions = this.#data.globalRuleExclusions.filter(
+      (e) => e.ruleId !== ruleId
+    );
     this.#save();
   }
 
@@ -134,10 +150,16 @@ export class ExclusionManager {
    * @returns {object[]} 过滤后的违反列表
    */
   applyExclusions(violations) {
-    return violations.filter(v => {
-      if (v.filePath && this.isPathExcluded(v.filePath)) return false;
-      if (v.ruleId && v.filePath && this.isRuleExcluded(v.ruleId, v.filePath)) return false;
-      if (v.ruleId && this.isRuleGloballyDisabled(v.ruleId)) return false;
+    return violations.filter((v) => {
+      if (v.filePath && this.isPathExcluded(v.filePath)) {
+        return false;
+      }
+      if (v.ruleId && v.filePath && this.isRuleExcluded(v.ruleId, v.filePath)) {
+        return false;
+      }
+      if (v.ruleId && this.isRuleGloballyDisabled(v.ruleId)) {
+        return false;
+      }
       return true;
     });
   }
@@ -147,15 +169,21 @@ export class ExclusionManager {
    */
   importExclusions(config) {
     if (config.pathExclusions) {
-      for (const e of config.pathExclusions) this.addPathExclusion(e.pattern, e);
+      for (const e of config.pathExclusions) {
+        this.addPathExclusion(e.pattern, e);
+      }
     }
     if (config.ruleExclusions) {
       for (const [ruleId, list] of Object.entries(config.ruleExclusions)) {
-        for (const e of list) this.addRuleExclusion(ruleId, e.filePath, e);
+        for (const e of list) {
+          this.addRuleExclusion(ruleId, e.filePath, e);
+        }
       }
     }
     if (config.globalRuleExclusions) {
-      for (const e of config.globalRuleExclusions) this.addGlobalRuleExclusion(e.ruleId, e);
+      for (const e of config.globalRuleExclusions) {
+        this.addGlobalRuleExclusion(e.ruleId, e);
+      }
     }
   }
 
@@ -183,17 +211,23 @@ export class ExclusionManager {
       if (existsSync(this.#exclusionsPath)) {
         return JSON.parse(readFileSync(this.#exclusionsPath, 'utf-8'));
       }
-    } catch { /* silent */ }
+    } catch {
+      /* silent */
+    }
     return { pathExclusions: [], ruleExclusions: {}, globalRuleExclusions: [] };
   }
 
   #save() {
     try {
       const dir = dirname(this.#exclusionsPath);
-      if (!existsSync(dir)) mkdirSync(dir, { recursive: true });
+      if (!existsSync(dir)) {
+        mkdirSync(dir, { recursive: true });
+      }
       writeFileSync(this.#exclusionsPath, JSON.stringify(this.#data, null, 2));
     } catch (err) {
-      Logger.getInstance().warn('ExclusionManager: failed to persist exclusions', { error: err.message });
+      Logger.getInstance().warn('ExclusionManager: failed to persist exclusions', {
+        error: err.message,
+      });
     }
   }
 
@@ -206,10 +240,14 @@ export class ExclusionManager {
       if (existsSync(oldPath) && !existsSync(this.#exclusionsPath)) {
         const content = readFileSync(oldPath, 'utf-8');
         const dir = dirname(this.#exclusionsPath);
-        if (!existsSync(dir)) mkdirSync(dir, { recursive: true });
+        if (!existsSync(dir)) {
+          mkdirSync(dir, { recursive: true });
+        }
         writeFileSync(this.#exclusionsPath, content);
         unlinkSync(oldPath);
-        Logger.getInstance().info('ExclusionManager: migrated guard-exclusions.json to knowledge base dir');
+        Logger.getInstance().info(
+          'ExclusionManager: migrated guard-exclusions.json to knowledge base dir'
+        );
       }
     } catch {
       // 迁移失败不阻断启动