@aigne/doc-smith 0.4.5 → 0.6.0

package/agents/manage-prefs.mjs

@@ -0,0 +1,203 @@
+import { readPreferences, removeRule, writePreferences } from "../utils/preferences-utils.mjs";
+
+/**
+ * List all user preferences with formatted display
+ * @returns {Object} Result with formatted message
+ */
+function listPreferences() {
+  const preferences = readPreferences();
+
+  if (preferences.rules.length === 0) {
+    return { message: "No preferences found." };
+  }
+
+  let message = "# User Preferences\n\n";
+
+  // Add format explanation
+  message += "**Format explanation:**\n";
+  message += "- 🟢 = Active preference, ⚪ = Inactive preference\n";
+  message += "- [scope] = Preference scope (global, structure, document, translation)\n";
+  message += "- ID = Unique preference identifier\n";
+  message += "- Paths = Specific file paths (if applicable)\n\n";
+
+  preferences.rules.forEach((rule) => {
+    const status = rule.active ? "🟢" : "⚪";
+    const pathsInfo = rule.paths ? ` | Paths: ${rule.paths.join(", ")}` : "";
+
+    // First line: status, scope, ID and paths info
+    message += `${status} [${rule.scope}] ${rule.id}${pathsInfo}\n`;
+
+    // Second line: rule content (truncated if too long)
+    const maxRuleLength = 120;
+    const ruleText =
+      rule.rule.length > maxRuleLength ? `${rule.rule.substring(0, maxRuleLength)}...` : rule.rule;
+    message += `   ${ruleText}\n `;
+
+    // Add blank line after each record
+    message += `\n`;
+  });
+
+  return { message };
+}
+
+/**
+ * Remove preferences by IDs or interactive selection
+ * @param {string[]} id - Array of preference IDs to remove
+ * @param {Object} options - Options with prompts interface
+ * @returns {Object} Result with success message
+ */
+async function removePreferences(id, options) {
+  const preferences = readPreferences();
+  let targetIds = id;
+
+  if (!targetIds || targetIds.length === 0) {
+    // Interactive selection
+    if (preferences.rules.length === 0) {
+      return { message: "No preferences found to remove." };
+    }
+
+    const choices = preferences.rules.map((rule) => ({
+      name: `${rule.active ? "🟢" : "⚪"} [${rule.scope}] ${rule.rule.substring(0, 60)}${rule.rule.length > 60 ? "..." : ""}`,
+      value: rule.id,
+      description: `ID: ${rule.id}`,
+    }));
+
+    targetIds = await options.prompts.checkbox({
+      message: "Select preferences to remove:",
+      choices,
+      validate: (answer) => {
+        if (answer.length === 0) {
+          return "Please select at least one preference to remove";
+        }
+        return true;
+      },
+    });
+
+    if (!targetIds || targetIds.length === 0) {
+      return { message: "No preferences selected for removal." };
+    }
+  }
+
+  // Process the target IDs
+  const results = [];
+  for (const ruleId of targetIds) {
+    const success = removeRule(ruleId);
+    results.push({ id: ruleId, success });
+  }
+
+  const successCount = results.filter((r) => r.success).length;
+  const failedCount = targetIds.length - successCount;
+  const message =
+    failedCount > 0
+      ? `Successfully removed ${successCount} preferences, ${failedCount} failed.`
+      : `Successfully removed ${successCount} preferences.`;
+
+  return { message };
+}
+
+/**
+ * Toggle preferences active status by IDs or interactive selection
+ * @param {string[]} id - Array of preference IDs to toggle
+ * @param {Object} options - Options with prompts interface
+ * @returns {Object} Result with success message
+ */
+async function togglePreferences(id, options) {
+  const preferences = readPreferences();
+  let targetIds = id;
+
+  if (!targetIds || targetIds.length === 0) {
+    // Interactive selection
+    if (preferences.rules.length === 0) {
+      return { message: "No preferences found to toggle." };
+    }
+
+    const choices = preferences.rules.map((rule) => ({
+      name: `${rule.active ? "🟢" : "⚪"} [${rule.scope}] ${rule.rule.substring(0, 60)}${rule.rule.length > 60 ? "..." : ""}`,
+      value: rule.id,
+      description: `ID: ${rule.id}`,
+    }));
+
+    targetIds = await options.prompts.checkbox({
+      message: "Select preferences to toggle active status:",
+      choices,
+      validate: (answer) => {
+        if (answer.length === 0) {
+          return "Please select at least one preference to toggle";
+        }
+        return true;
+      },
+    });
+
+    if (!targetIds || targetIds.length === 0) {
+      return { message: "No preferences selected for toggling." };
+    }
+  }
+
+  // Process the target IDs
+  const results = [];
+  const prefs = readPreferences();
+
+  for (const ruleId of targetIds) {
+    const rule = prefs.rules.find((r) => r.id === ruleId);
+    if (rule) {
+      rule.active = !rule.active;
+      results.push({ id: ruleId, success: true, newStatus: rule.active });
+    } else {
+      results.push({ id: ruleId, success: false, error: "Rule not found" });
+    }
+  }
+
+  writePreferences(prefs);
+
+  const successCount = results.filter((r) => r.success).length;
+  const failedCount = targetIds.length - successCount;
+  const message =
+    failedCount > 0
+      ? `Successfully toggled ${successCount} preferences, ${failedCount} failed.`
+      : `Successfully toggled ${successCount} preferences.`;
+
+  return { message };
+}
+
+export default async function prefs({ list, remove, toggle, id }, options) {
+  if (list) {
+    return listPreferences();
+  }
+
+  if (remove) {
+    return await removePreferences(id, options);
+  }
+
+  if (toggle) {
+    return await togglePreferences(id, options);
+  }
+
+  return { message: "Please specify an action: --list, --remove, or --toggle." };
+}
+
+prefs.input_schema = {
+  type: "object",
+  properties: {
+    list: {
+      type: "boolean",
+      description: "List all preferences",
+    },
+    remove: {
+      type: "boolean",
+      description: "Remove preferences",
+    },
+    toggle: {
+      type: "boolean",
+      description: "Toggle preferences active status",
+    },
+    id: {
+      type: "array",
+      items: {
+        type: "string",
+      },
+      description: "Preference IDs to manage",
+    },
+  },
+};
+
+prefs.description = "Manage user preferences learned from feedback";

package/agents/publish-docs.mjs

@@ -3,7 +3,7 @@ import { publishDocs as publishDocsFn } from "@aigne/publish-docs";
 import chalk from "chalk";
 import { getAccessToken } from "../utils/auth-utils.mjs";
 import { DISCUSS_KIT_STORE_URL } from "../utils/constants.mjs";
-import { loadConfigFromFile, saveValueToConfig, getGithubRepoUrl } from "../utils/utils.mjs";
+import { getGithubRepoUrl, loadConfigFromFile, saveValueToConfig } from "../utils/utils.mjs";
 
 const DEFAULT_APP_URL = "https://docsmith.aigne.io";
 
@@ -98,6 +98,8 @@ export default async function publishDocs(
       boardName: projectInfo.name,
       boardDesc: projectInfo.description,
       boardCover: projectInfo.icon,
+      mediaFolder: docsDir,
+      cacheFilePath: join(".aigne", "doc-smith", "upload-cache.yaml"),
       boardMeta,
     });
 

package/agents/retranslate.yaml

@@ -37,6 +37,9 @@ skills:
           isTranslate: true
     iterate_on: selectedDocs
     concurrency: 3
+  - url: ./check-feedback-refiner.mjs
+    default_input:
+      stage: translation_refine
   - url: ./action-success.mjs
     default_input:
       action: "Document translated"

package/agents/structure-planning.yaml

@@ -26,6 +26,9 @@ input_schema:
     feedback:
       type: string
       description: 结构规划的反馈
+    userPreferences:
+      type: string
+      description: 用户偏好规则，YAML格式的结构规划和全局范围偏好
     docsType:
       type: string
       description: 文档类型，支持：general、getting-started、reference、faq

package/aigne.yaml

@@ -37,6 +37,9 @@ agents:
   - ./docs-mcp/analyze-docs-relevance.yaml
   - ./docs-mcp/read-doc-content.mjs
   - ./docs-mcp/analyze-content-relevance.yaml
+  - ./agents/check-feedback-refiner.mjs
+  - ./agents/feedback-refiner.yaml
+  - ./agents/manage-prefs.mjs
 cli:
   agents:
     - ./agents/input-generator.mjs
@@ -44,6 +47,7 @@ cli:
     - ./agents/detail-regenerator.yaml
     - ./agents/team-publish-docs.yaml
     - ./agents/retranslate.yaml
+    - ./agents/manage-prefs.mjs
 mcp_server:
   agents:
     - ./docs-mcp/get-docs-structure.mjs

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aigne/doc-smith",
-  "version": "0.4.5",
+  "version": "0.6.0",
   "description": "",
   "publishConfig": {
     "access": "public"
@@ -12,13 +12,13 @@
   "author": "Arcblock <blocklet@arcblock.io> https://github.com/blocklet",
   "license": "MIT",
   "dependencies": {
-    "@aigne/aigne-hub": "^0.6.9",
-    "@aigne/anthropic": "^0.11.9",
-    "@aigne/cli": "^1.41.0",
-    "@aigne/core": "^1.55.0",
-    "@aigne/gemini": "^0.9.9",
-    "@aigne/openai": "^0.12.3",
-    "@aigne/publish-docs": "^0.5.7",
+    "@aigne/aigne-hub": "^0.6.10",
+    "@aigne/anthropic": "^0.11.10",
+    "@aigne/cli": "^1.41.1",
+    "@aigne/core": "^1.55.1",
+    "@aigne/gemini": "^0.9.10",
+    "@aigne/openai": "^0.12.4",
+    "@aigne/publish-docs": "^0.6.0",
     "chalk": "^5.5.0",
     "dompurify": "^3.2.6",
     "glob": "^11.0.3",
@@ -39,7 +39,8 @@
     "@biomejs/biome": "^2.1.4"
   },
   "scripts": {
-    "test": "echo \"Error: no test specified\" && exit 1",
+    "test": "bun test",
+    "test:watch": "bun test --watch",
     "lint": "biome check && pnpm -r run lint",
     "update:deps": "npx -y taze major -r -w -f -n '/@abtnode|@aigne|@arcblock|@blocklet|@did-connect|@did-pay|@did-space|@nft-store|@nft-studio|@ocap/' && pnpm install && pnpm run deduplicate",
     "deduplicate": "pnpm dedupe",

package/prompts/content-detail-generator.md

@@ -16,6 +16,11 @@
 {{ detailDataSources }}
 
 {{ additionalInformation }}
+
+<media_list>
+{{ assetsContent }}
+</media_list>
+
 </datasources>
 
 <terms>
@@ -56,6 +61,14 @@ parentId: {{parentId}}
 ** 使用 {{ locale }} 语言输出内容 **
 </user_rules>
 
+<user_preferences>
+{{userPreferences}}
+
+用户偏好使用规则：
+- 用户偏好来自用户之前操作中提供的反馈，生成结构规划中需要考虑用户的偏好，避免出现用户反馈的问题又重复出现
+- 用户偏好的权重低于本次用户提交的反馈
+</user_preferences>
+
 <rules>
 
 目标受众：{{targetAudience}}
@@ -85,7 +98,6 @@ parentId: {{parentId}}
 - 媒体资源以 markdown 格式提供，示例：![资源描述](https://xxxx)
 - 在生成结果中以 markdown 格式展示图片
 - 根据资源描述，在上下文相关的位置，合理的展示图片，让结果展示效果更丰富
-- 只使用完整的远程图片URL（如 https://example.com/image.jpg），禁止使用相对路径（如 ./images/photo.png 或 ../assets/logo.png），确保发布后图片能正常访问
 
 </media_rules>
 

package/prompts/document/detail-generator.md

@@ -27,6 +27,7 @@
 - **代码块原子性**：将每个代码块（例如 ```mermaid ... ```）视为一个**不可分割的原子单元**。必须一次性完整生成，从开始标记（```mermaid）到结束标记（```）之间的所有内容都不能省略或截断。
 - **确保 Markdown 语法**：Markdown 格式正确，特别是表格的分隔线（例如 `|---|---|---|`），需要与表格数据列数一致。
 - README 文件只做参考，你需要从代码中获取最新、最完整的信息
+- 忽略详情顶部的标签信息，这是程序处理的，不需要在生成时输出
 </document_rules>
 
 <TONE_STYLE>

package/prompts/feedback-refiner.md

@@ -0,0 +1,84 @@
+<role>
+你是"反馈→规则"转换器。将一次性的自然语言反馈提炼为**一条单句**、**可执行**、**可复用**的指令，
+并判断是否需要**持久化保存**，以及作用域（global/structure/document/translation）与是否仅限于"输入 paths 范围"。
+</role>
+
+<input>
+- feedback: {{feedback}}
+- stage: {{stage}}     # 可取：structure_planning | document_refine | translation_refine
+- paths: {{paths}}     # 本次命令输入的路径数组（可为空）。仅用于判断是否"限定到这些路径"。不要把它们写进输出。
+- existingPreferences: {{existingPreferences}}     # 当前已保存的用户偏好规则
+</input>
+
+<scope_rules>
+作用域判定启发式规则：
+
+**按 stage 分类**：
+- 若 stage=structure_planning：默认 `scope="structure"`，除非反馈显然是全局写作/语气/排除类政策（则用 `global`）。
+- 若 stage=document_refine：默认 `scope="document"`；若反馈是通用写作政策、排除策略且不依赖具体页面，则可提升为 `global`。
+- 若 stage=translation_refine：默认 `scope="translation"`；若反馈是翻译阶段的一般政策可保持此 scope。
+
+**路径限制判定**：
+- 若用户反馈显著只影响本批 `paths` 指向的范围（例如"examples 目录中的页面精简说明"），将 `limitToInputPaths=true`；否则为 `false`。
+- **永远不要**在输出中返回具体的 paths 列表。
+</scope_rules>
+
+<save_rules>
+是否保存判定规则：
+
+**一次性操作（不保存）**：
+- 只修正当下版本/错别字/个别句式/局部事实错误，且无稳定可复用价值 → `save=false`
+
+**可复用政策（保存）**：
+- 写作风格、结构约定、包含/排除项、翻译约定等可广泛适用且未来应持续执行 → `save=true`
+
+**重复性检查（不保存）**：
+- 若 `existingPreferences` 中已有**相似或覆盖**本次反馈意图的规则，则 `save=false`
+- 检查逻辑：对比反馈意图、规则含义、适用范围。若新反馈已被现有规则充分覆盖，无需重复保存
+- 若新反馈是对现有规则的**细化、补充或矛盾修正**，则仍可 `save=true`
+
+**判定原则**：
+- 优先避免重复保存；若难以判定是否重复，优先 `save=false`，以避免规则冗余
+</save_rules>
+
+<rule_format>
+规则写法要求：
+
+- 面向模型的**单句**指令；允许使用"必须/不得/总是"等明确措辞。
+- 不引入具体路径、不绑定具体文件名。
+- 例："写作面向初学者；术语首次出现必须给出简明解释。"
+</rule_format>
+
+<examples>
+示例1：
+- 输入：stage=document_refine，paths=["overview.md"]，feedback="示例页废话太多，代码要最小可运行。"
+- 输出：
+{"rule":"示例页面以最小可运行代码为主，移除与主题无关的说明段。","scope":"document","save":true,"limitToInputPaths":true}
+
+示例2：
+- 输入：stage=structure_planning，paths=[]，feedback="概览与教程结尾加"下一步"并给出2–3个链接。"
+- 输出：
+{"rule":"在概览与教程文档结尾添加"下一步"小节并提供 2–3 个本仓库内链接。","scope":"structure","save":true,"limitToInputPaths":false}
+
+示例3：
+- 输入：stage=translation_refine，paths=[]，feedback="变量名和代码不要翻译。"
+- 输出：
+{"rule":"翻译时保持代码与标识符原样，不得翻译。","scope":"translation","save":true,"limitToInputPaths":false}
+
+示例4：
+- 输入：stage=document_refine，paths=["overview.md"]，feedback="这段话事实有误，改成 2021 年发布。"
+- 输出：
+{"rule":"更正事实到正确年份。","scope":"document","save":false,"limitToInputPaths":true}
+
+示例5（去重案例）：
+- 输入：stage=document_refine，paths=[]，feedback="代码示例太复杂了，简化一下。"，existingPreferences="rules:\n  - rule: 示例页面以最小可运行代码为主，移除与主题无关的说明段。\n    scope: document\n    active: true"
+- 输出：
+{"rule":"简化代码示例的复杂度。","scope":"document","save":false,"limitToInputPaths":false}
+# 理由：现有规则已覆盖简化代码示例的意图
+
+示例6（非重复案例）：
+- 输入：stage=document_refine，paths=[]，feedback="代码注释要用英文写。"，existingPreferences="rules:\n  - rule: 示例页面以最小可运行代码为主，移除与主题无关的说明段。\n    scope: document\n    active: true"
+- 输出：
+{"rule":"代码注释必须使用英文编写。","scope":"document","save":true,"limitToInputPaths":false}
+# 理由：现有规则未涉及注释语言，属于新的规则维度
+</examples>

package/prompts/structure-planning.md

@@ -56,6 +56,14 @@
 {{ rules }}
 </user_rules>
 
+<user_preferences>
+{{userPreferences}}
+
+用户偏好使用规则：
+- 用户偏好来自用户之前操作中提供的反馈，生成结构规划中需要考虑用户的偏好，避免出现用户反馈的问题又重复出现
+- 用户偏好的权重低于本次用户提交的反馈
+</user_preferences>
+
 <rules>
 DataSources 使用规则：
 1. 结构规划时要要尽可能的把 DataSources 中的信息合理的进行规划展示，不能遗漏

package/prompts/translator.md

@@ -69,5 +69,13 @@
 {{ detailFeedback }}
 </review_feedback>
 
+<user_preferences>
+{{userPreferences}}
+
+用户偏好使用规则：
+- 用户偏好来自用户之前操作中提供的反馈，生成结构规划中需要考虑用户的偏好，避免出现用户反馈的问题又重复出现
+- 用户偏好的权重低于本次用户提交的反馈
+</user_preferences>
+
 指令：
 请将 <content> 中的内容（不包含最外层的 <content> 标签） **准确** 地翻译成 **{{ language }}**，并严格遵循翻译要求。

package/tests/{test-all-validation-cases.mjs → all-validation-cases.test.mjs}

@@ -1,5 +1,4 @@
-#!/usr/bin/env node
-
+import { afterAll, describe, expect, test } from "bun:test";
 import checkDetailResult from "../agents/check-detail-result.mjs";
 import { checkMarkdown } from "../utils/markdown-checker.mjs";
 import { shutdownValidation } from "../utils/mermaid-validator.mjs";
@@ -459,7 +458,7 @@ This content ends properly.
     category: "🧩 MERMAID VALIDATION",
     name: "Mermaid with subgraph reference issues (rendering failure)",
     expectPass: false,
-    expectedErrors: ["subgraph reference"],
+    expectedErrors: ["Mermaid syntax error"],
     content: `# Test Document
 
 \`\`\`mermaid
@@ -618,146 +617,70 @@ This comprehensive document demonstrates all validation rules in their correct u
   },
 ];
 
-async function runValidationTests() {
-  console.log("🧪 Comprehensive Markdown Validation Test Suite\n");
-  console.log("=".repeat(80));
-
-  let totalTests = 0;
-  let passedTests = 0;
-  let failedTests = 0;
-
-  let currentCategory = "";
-
-  for (const testCase of testCases) {
-    // Print category header if it changed
-    if (testCase.category !== currentCategory) {
-      currentCategory = testCase.category;
-      console.log(`\n${currentCategory}`);
-      console.log("-".repeat(80));
-    }
-
-    console.log(`\n📝 Testing: ${testCase.name}`);
-    totalTests++;
-
+describe("Markdown Validation Test Suite", () => {
+  afterAll(async () => {
+    // Shutdown worker pool to ensure clean exit
     try {
-      // Test with checkMarkdown directly
-      const errors = await checkMarkdown(testCase.content, "test", {
-        allowedLinks,
-      });
-
-      // Test with checkDetailResult wrapper
-      const wrapperResult = await checkDetailResult({
-        structurePlan: mockStructurePlan,
-        reviewContent: testCase.content,
-      });
-
-      const hasErrors = errors.length > 0;
-      const expectPass = testCase.expectPass;
-
-      // Verify test expectation
-      if (expectPass && !hasErrors) {
-        console.log("✅ PASS - Content correctly passed validation");
-        passedTests++;
-      } else if (!expectPass && hasErrors) {
-        console.log("✅ PASS - Content correctly failed validation");
-
-        // Check if expected error types are present
-        if (testCase.expectedErrors) {
-          const foundExpectedErrors = testCase.expectedErrors.every((expectedError) =>
-            errors.some((error) => error.toLowerCase().includes(expectedError.toLowerCase())),
-          );
+      await shutdownValidation();
+    } catch {
+      // Ignore shutdown errors in tests
+    }
+  });
 
-          if (foundExpectedErrors) {
-            console.log("✅ Expected error types found");
+  // Group tests by category
+  const testsByCategory = testCases.reduce((acc, testCase) => {
+    if (!acc[testCase.category]) {
+      acc[testCase.category] = [];
+    }
+    acc[testCase.category].push(testCase);
+    return acc;
+  }, {});
+
+  Object.entries(testsByCategory).forEach(([category, categoryTests]) => {
+    describe(category, () => {
+      categoryTests.forEach((testCase) => {
+        test(testCase.name, async () => {
+          // Test with checkMarkdown directly
+          const errors = await checkMarkdown(testCase.content, "test", {
+            allowedLinks,
+          });
+
+          // Test with checkDetailResult wrapper
+          const wrapperResult = await checkDetailResult({
+            structurePlan: mockStructurePlan,
+            reviewContent: testCase.content,
+          });
+
+          const hasErrors = errors.length > 0;
+          const expectPass = testCase.expectPass;
+
+          // Verify test expectation
+          if (expectPass) {
+            expect(hasErrors).toBe(false);
           } else {
-            console.log("⚠️  Expected error types not all found");
-            console.log(`   Expected: ${testCase.expectedErrors.join(", ")}`);
+            expect(hasErrors).toBe(true);
+
+            // Check if expected error types are present
+            if (testCase.expectedErrors) {
+              const foundExpectedErrors = testCase.expectedErrors.every((expectedError) =>
+                errors.some((error) => error.toLowerCase().includes(expectedError.toLowerCase())),
+              );
+              expect(foundExpectedErrors).toBe(true);
+            }
           }
-        }
-
-        passedTests++;
-      } else {
-        console.log(
-          `❌ FAIL - Expected ${expectPass ? "PASS" : "FAIL"} but got ${
-            hasErrors ? "FAIL" : "PASS"
-          }`,
-        );
-        failedTests++;
-      }
-
-      // Show error details for failing cases
-      if (hasErrors) {
-        console.log(`   Found ${errors.length} issue(s):`);
-        errors.slice(0, 3).forEach((error) => {
-          console.log(`   • ${error}`);
-        });
-        if (errors.length > 3) {
-          console.log(`   ... and ${errors.length - 3} more issues`);
-        }
-      }
-
-      // Verify consistency between direct call and wrapper
-      const wrapperErrors = wrapperResult.detailFeedback
-        ? wrapperResult.detailFeedback.split("\n").filter((line) => line.trim())
-        : [];
-
-      if (errors.length === wrapperErrors.length) {
-        console.log("✅ Direct call and wrapper consistent");
-      } else {
-        console.log(
-          `⚠️  Inconsistent results: direct=${errors.length}, wrapper=${wrapperErrors.length}`,
-        );
-      }
-    } catch (error) {
-      console.log(`❌ ERROR: ${error.message}`);
-      failedTests++;
-    }
-  }
 
-  // Final summary
-  console.log(`\n${"=".repeat(80)}`);
-  console.log("📊 TEST SUMMARY");
-  console.log("=".repeat(80));
-  console.log(`Total Tests: ${totalTests}`);
-  console.log(`Passed: ${passedTests} ✅`);
-  console.log(`Failed: ${failedTests} ❌`);
-  console.log(`Success Rate: ${((passedTests / totalTests) * 100).toFixed(1)}%`);
-
-  console.log("\n🔍 VALIDATION COVERAGE:");
-  console.log("✅ Link validation (dead links, allowed links)");
-  console.log("✅ Code block validation (indentation, completeness)");
-  console.log("✅ Content structure (line breaks, punctuation)");
-  console.log("✅ Table validation (column count consistency)");
-  console.log("✅ Mermaid validation (syntax, rendering issues)");
-  console.log("✅ Standard markdown linting (formatting rules)");
-  console.log("✅ Complex mixed scenarios");
-  console.log("✅ Edge cases and error conditions");
-
-  if (failedTests === 0) {
-    console.log("\n🎉 ALL TESTS PASSED! Validation system is working correctly.");
-  } else {
-    console.log(`\n⚠️  ${failedTests} test(s) failed. Please review the validation logic.`);
-  }
+          // Verify consistency between direct call and wrapper
+          const wrapperErrors = wrapperResult.detailFeedback
+            ? wrapperResult.detailFeedback.split("\n").filter((line) => line.trim())
+            : [];
 
-  // Shutdown worker pool to ensure clean exit
-  try {
-    await shutdownValidation();
-  } catch (error) {
-    console.error("Error shutting down validation:", error);
-  }
-}
+          // Note: We don't enforce exact equality as wrapper may format differently
+          expect(wrapperErrors.length > 0).toBe(hasErrors);
+        });
+      });
+    });
+  });
+});
 
 // Export test cases for external use
 export { testCases, mockStructurePlan, allowedLinks };
-
-// Run tests if this file is executed directly
-if (import.meta.url === `file://${process.argv[1]}`) {
-  runValidationTests()
-    .then(() => {
-      process.exit(0);
-    })
-    .catch((error) => {
-      console.error("Test suite error:", error);
-      process.exit(1);
-    });
-}