autosnippet 3.0.1 → 3.0.2

package/lib/cli/KnowledgeSyncService.js

@@ -14,23 +14,25 @@
  *  - 内部: SetupService.stepDatabase() 委托调用（skipViolations = true）
  */
 
+import { randomUUID } from 'node:crypto';
 import fs from 'node:fs';
 import path from 'node:path';
-import { randomUUID } from 'node:crypto';
-import { RECIPES_DIR, CANDIDATES_DIR } from '../infrastructure/config/Defaults.js';
-import { parseKnowledgeMarkdown, computeKnowledgeHash } from '../service/knowledge/KnowledgeFileWriter.js';
-import { KnowledgeEntry } from '../domain/knowledge/KnowledgeEntry.js';
+import { CANDIDATES_DIR, RECIPES_DIR } from '../infrastructure/config/Defaults.js';
 import Logger from '../infrastructure/logging/Logger.js';
+import {
+  computeKnowledgeHash,
+  parseKnowledgeMarkdown,
+} from '../service/knowledge/KnowledgeFileWriter.js';
 
 export class KnowledgeSyncService {
   /**
    * @param {string} projectRoot
    */
   constructor(projectRoot) {
-    this.projectRoot   = projectRoot;
-    this.recipesDir    = path.join(projectRoot, RECIPES_DIR);
+    this.projectRoot = projectRoot;
+    this.recipesDir = path.join(projectRoot, RECIPES_DIR);
     this.candidatesDir = path.join(projectRoot, CANDIDATES_DIR);
-    this.logger        = Logger.getInstance();
+    this.logger = Logger.getInstance();
   }
 
   /**
@@ -52,8 +54,8 @@ export class KnowledgeSyncService {
       synced: 0,
       created: 0,
       updated: 0,
-      violations: [],  // 手动编辑的文件列表
-      orphaned: [],    // DB 有但 .md 不存在
+      violations: [], // 手动编辑的文件列表
+      orphaned: [], // DB 有但 .md 不存在
       skipped: 0,
     };
 
@@ -70,7 +72,7 @@ export class KnowledgeSyncService {
 
     // ── 2. 准备 upsert 语句 ──
     const upsertStmt = dryRun ? null : this._prepareUpsert(db);
-    const auditStmt  = (dryRun || skipViolations) ? null : this._prepareAuditInsert(db);
+    const auditStmt = dryRun || skipViolations ? null : this._prepareAuditInsert(db);
 
     // ── 3. 逐文件同步 ──
     const syncedIds = new Set();
@@ -78,7 +80,7 @@ export class KnowledgeSyncService {
     for (const { absPath, relPath } of mdFiles) {
       try {
         const content = fs.readFileSync(absPath, 'utf8');
-        const parsed  = parseKnowledgeMarkdown(content, relPath);
+        const parsed = parseKnowledgeMarkdown(content, relPath);
 
         if (!parsed.id) {
           this.logger.warn(`KnowledgeSyncService: skip file without id — ${relPath}`);
@@ -115,7 +117,9 @@ export class KnowledgeSyncService {
 
         report.synced++;
       } catch (err) {
-        this.logger.error(`KnowledgeSyncService: failed to sync ${relPath}`, { error: err.message });
+        this.logger.error(`KnowledgeSyncService: failed to sync ${relPath}`, {
+          error: err.message,
+        });
         report.skipped++;
       }
     }
@@ -144,13 +148,15 @@ export class KnowledgeSyncService {
    * @returns {{ absPath: string, relPath: string }[]}
    */
   _collectMdFiles(dir, prefix) {
-    if (!fs.existsSync(dir)) return [];
+    if (!fs.existsSync(dir)) {
+      return [];
+    }
 
     const results = [];
     const walk = (curDir, base) => {
       for (const entry of fs.readdirSync(curDir, { withFileTypes: true })) {
         const full = path.join(curDir, entry.name);
-        const rel  = base ? `${base}/${entry.name}` : entry.name;
+        const rel = base ? `${base}/${entry.name}` : entry.name;
 
         if (entry.isDirectory()) {
           walk(full, rel);
@@ -179,50 +185,50 @@ export class KnowledgeSyncService {
     const contentHash = computeKnowledgeHash(rawContent);
 
     return {
-      id:                parsed.id,
-      title:             parsed.title || '',
-      trigger:           parsed.trigger || '',
-      description:       parsed.description || '',
-      lifecycle:         parsed.lifecycle || 'pending',
-      lifecycleHistory:  JSON.stringify(parsed.lifecycleHistory || []),
-      autoApprovable:    parsed.autoApprovable ? 1 : 0,
-      language:          parsed.language || 'swift',
-      category:          parsed.category || 'general',
-      kind:              parsed.kind || 'pattern',
-      knowledgeType:     parsed.knowledgeType || 'code-pattern',
-      complexity:        parsed.complexity || 'intermediate',
-      scope:             parsed.scope || 'universal',
-      difficulty:        parsed.difficulty || null,
-      tags:              JSON.stringify(parsed.tags || []),
-      content:           JSON.stringify(parsed.content || {}),
-      relations:         JSON.stringify(parsed.relations || {}),
-      constraints:       JSON.stringify(parsed.constraints || {}),
-      reasoning:         JSON.stringify(parsed.reasoning || {}),
-      quality:           JSON.stringify(parsed.quality || {}),
-      stats:             JSON.stringify(parsed.stats || {}),
-      headers:           JSON.stringify(parsed.headers || []),
-      headerPaths:       JSON.stringify(parsed.headerPaths || []),
-      moduleName:        parsed.moduleName || '',
-      includeHeaders:    parsed.includeHeaders ? 1 : 0,
-      topicHint:         parsed.topicHint || null,
-      whenClause:        parsed.whenClause || null,
-      doClause:          parsed.doClause || null,
-      dontClause:        parsed.dontClause || null,
-      coreCode:          parsed.coreCode || null,
-      agentNotes:        parsed.agentNotes ? JSON.stringify(parsed.agentNotes) : null,
-      aiInsight:         parsed.aiInsight || null,
-      reviewedBy:        parsed.reviewedBy || null,
-      reviewedAt:        parsed.reviewedAt || null,
-      rejectionReason:   parsed.rejectionReason || null,
-      source:            parsed.source || 'file-sync',
-      sourceFile:        relPath,
+      id: parsed.id,
+      title: parsed.title || '',
+      trigger: parsed.trigger || '',
+      description: parsed.description || '',
+      lifecycle: parsed.lifecycle || 'pending',
+      lifecycleHistory: JSON.stringify(parsed.lifecycleHistory || []),
+      autoApprovable: parsed.autoApprovable ? 1 : 0,
+      language: parsed.language || 'unknown',
+      category: parsed.category || 'general',
+      kind: parsed.kind || 'pattern',
+      knowledgeType: parsed.knowledgeType || 'code-pattern',
+      complexity: parsed.complexity || 'intermediate',
+      scope: parsed.scope || 'universal',
+      difficulty: parsed.difficulty || null,
+      tags: JSON.stringify(parsed.tags || []),
+      content: JSON.stringify(parsed.content || {}),
+      relations: JSON.stringify(parsed.relations || {}),
+      constraints: JSON.stringify(parsed.constraints || {}),
+      reasoning: JSON.stringify(parsed.reasoning || {}),
+      quality: JSON.stringify(parsed.quality || {}),
+      stats: JSON.stringify(parsed.stats || {}),
+      headers: JSON.stringify(parsed.headers || []),
+      headerPaths: JSON.stringify(parsed.headerPaths || []),
+      moduleName: parsed.moduleName || '',
+      includeHeaders: parsed.includeHeaders ? 1 : 0,
+      topicHint: parsed.topicHint || null,
+      whenClause: parsed.whenClause || null,
+      doClause: parsed.doClause || null,
+      dontClause: parsed.dontClause || null,
+      coreCode: parsed.coreCode || null,
+      agentNotes: parsed.agentNotes ? JSON.stringify(parsed.agentNotes) : null,
+      aiInsight: parsed.aiInsight || null,
+      reviewedBy: parsed.reviewedBy || null,
+      reviewedAt: parsed.reviewedAt || null,
+      rejectionReason: parsed.rejectionReason || null,
+      source: parsed.source || 'file-sync',
+      sourceFile: relPath,
       sourceCandidateId: parsed.sourceCandidateId || null,
-      createdBy:         parsed.createdBy || 'file-sync',
-      createdAt:         parsed.createdAt || now,
-      updatedAt:         parsed.updatedAt || now,
-      publishedAt:       parsed.publishedAt || null,
-      publishedBy:       parsed.publishedBy || null,
-      contentHash:       contentHash,
+      createdBy: parsed.createdBy || 'file-sync',
+      createdAt: parsed.createdAt || now,
+      updatedAt: parsed.updatedAt || now,
+      publishedAt: parsed.publishedAt || null,
+      publishedBy: parsed.publishedBy || null,
+      contentHash: contentHash,
     };
   }
 
@@ -231,24 +237,55 @@ export class KnowledgeSyncService {
    */
   _prepareUpsert(db) {
     const cols = [
-      'id', 'title', 'trigger', 'description',
-      'lifecycle', 'lifecycleHistory', 'autoApprovable',
-      'language', 'category', 'kind', 'knowledgeType', 'complexity', 'scope', 'difficulty',
+      'id',
+      'title',
+      'trigger',
+      'description',
+      'lifecycle',
+      'lifecycleHistory',
+      'autoApprovable',
+      'language',
+      'category',
+      'kind',
+      'knowledgeType',
+      'complexity',
+      'scope',
+      'difficulty',
       'tags',
-      'content', 'relations', 'constraints', 'reasoning', 'quality', 'stats',
-      'headers', 'headerPaths', 'moduleName', 'includeHeaders',
-      'topicHint', 'whenClause', 'doClause', 'dontClause', 'coreCode',
-      'agentNotes', 'aiInsight',
-      'reviewedBy', 'reviewedAt', 'rejectionReason',
-      'source', 'sourceFile', 'sourceCandidateId',
-      'createdBy', 'createdAt', 'updatedAt',
-      'publishedAt', 'publishedBy',
+      'content',
+      'relations',
+      'constraints',
+      'reasoning',
+      'quality',
+      'stats',
+      'headers',
+      'headerPaths',
+      'moduleName',
+      'includeHeaders',
+      'topicHint',
+      'whenClause',
+      'doClause',
+      'dontClause',
+      'coreCode',
+      'agentNotes',
+      'aiInsight',
+      'reviewedBy',
+      'reviewedAt',
+      'rejectionReason',
+      'source',
+      'sourceFile',
+      'sourceCandidateId',
+      'createdBy',
+      'createdAt',
+      'updatedAt',
+      'publishedAt',
+      'publishedBy',
       'contentHash',
     ];
 
     // ON CONFLICT 更新除 id, createdBy, createdAt 以外的所有列
-    const updateCols = cols.filter(c => !['id', 'createdBy', 'createdAt'].includes(c));
-    const setClauses = updateCols.map(c => `${c} = excluded.${c}`).join(',\n      ');
+    const updateCols = cols.filter((c) => !['id', 'createdBy', 'createdAt'].includes(c));
+    const setClauses = updateCols.map((c) => `${c} = excluded.${c}`).join(',\n      ');
 
     const sql = `
       INSERT INTO knowledge_entries (${cols.join(', ')})
@@ -293,7 +330,7 @@ export class KnowledgeSyncService {
         JSON.stringify({ file: filePath, expectedHash, actualHash }),
         'violation_detected',
         null,
-        0,
+        0
       );
     } catch (err) {
       this.logger.warn('KnowledgeSyncService: failed to log violation', {
@@ -312,11 +349,13 @@ export class KnowledgeSyncService {
   _detectOrphans(db, syncedIds, dryRun) {
     const orphanIds = [];
     try {
-      const rows = db.prepare(
-        `SELECT id, sourceFile FROM knowledge_entries 
+      const rows = db
+        .prepare(
+          `SELECT id, sourceFile FROM knowledge_entries 
          WHERE lifecycle NOT IN ('deprecated') 
          AND sourceFile IS NOT NULL`
-      ).all();
+        )
+        .all();
 
       for (const row of rows) {
         if (!syncedIds.has(row.id)) {