@aigne/doc-smith 0.7.2 → 0.8.1

package/utils/conflict-detector.mjs

@@ -1,40 +1,5 @@
 import { CONFLICT_RESOLUTION_RULES, CONFLICT_RULES, RESOLUTION_STRATEGIES } from "./constants.mjs";
 
-/**
- * Detect internal conflicts within the same question (multi-select conflicts)
- * @param {string} questionType - Question type (documentPurpose, targetAudienceTypes)
- * @param {Array} selectedValues - User selected values
- * @returns {Array} List of conflicts
- */
-export function detectInternalConflicts(questionType, selectedValues) {
-  const rules = CONFLICT_RULES.internalConflicts[questionType] || [];
-
-  // Extract values from the selected items (handle both string arrays and object arrays)
-  const selectedValueStrings = selectedValues.map((item) =>
-    typeof item === "object" && item.value ? item.value : item,
-  );
-
-  const conflicts = [];
-
-  rules.forEach((rule) => {
-    // Check if all conflict items are selected
-    const hasConflict = rule.conflictItems.every((item) => selectedValueStrings.includes(item));
-
-    if (hasConflict) {
-      conflicts.push({
-        type: "internal",
-        questionType,
-        severity: rule.severity,
-        reason: rule.reason,
-        suggestion: rule.suggestion,
-        items: rule.conflictItems,
-      });
-    }
-  });
-
-  return conflicts;
-}
-
 /**
  * Get filtered options based on cross-question conflict rules
  * @param {string} targetQuestion - Target question type to filter
@@ -106,30 +71,6 @@ export function getFilteredOptions(targetQuestion, currentSelections, allOptions
   };
 }
 
-/**
- * Validate user selection for internal conflicts and return validation message
- * @param {string} questionType - Question type (documentPurpose, targetAudienceTypes)
- * @param {Array} selectedValues - User selected values
- * @returns {string|boolean} Error message if conflicts exist, true if valid
- */
-export function validateSelection(questionType, selectedValues) {
-  const conflicts = detectInternalConflicts(questionType, selectedValues);
-
-  if (conflicts.length === 0) {
-    return true;
-  }
-
-  // Return error message for severe conflicts
-  const severeConflicts = conflicts.filter((c) => c.severity === "severe");
-  if (severeConflicts.length > 0) {
-    const conflict = severeConflicts[0];
-    return `Conflict detected: ${conflict.reason}. ${conflict.suggestion}`;
-  }
-
-  // For moderate conflicts, allow but warn
-  return true;
-}
-
 /**
  * Detect conflicts in user configuration selections that can be resolved through structure planning
  * @param {Object} config - User configuration

package/utils/constants.mjs

@@ -513,30 +513,10 @@ export const RESOLUTION_STRATEGIES = {
 export const D2_CONFIG = `vars: {
   d2-config: {
     layout-engine: elk
-    theme-id: 0
+    theme-id: 300
     theme-overrides: {
-      N1: "#2AA7A1"
-      N2: "#73808C"
-
-      N4: "#FFFFFF"
-      N5: "#FAFBFC"
-
-      N7: "#ffffff"
-
-      B1: "#8EDDD9"
-      B2: "#C9DCE6"
-      B3: "#EEF9F9"
-      B4: "#F7F8FA"
-      B5: "#FCFDFD"
-      B6: "#E3E9F0"
-
-
-      AA2: "#9EB7C5"
-      AA4: "#E3EBF2"
-      AA5: "#F6FAFC"
-
-      AB4: "#B8F1F6"
-      AB5: "#E3F8FA"
+      AA4: "#90EFDF"
+      AA5: "#D2FFF7"
     }
   }
 }`;

package/utils/file-utils.mjs

@@ -159,6 +159,11 @@ export async function loadGitignore(dir) {
  * @returns {Promise<string[]>} Array of file paths
  */
 export async function getFilesWithGlob(dir, includePatterns, excludePatterns, gitignorePatterns) {
+  if (!includePatterns || includePatterns.length === 0) {
+    console.warn("No include patterns provided");
+    return [];
+  }
+
   // Prepare all ignore patterns
   const allIgnorePatterns = [];
 

package/utils/kroki-utils.mjs

@@ -52,6 +52,10 @@ export async function getD2Svg({ content, strict = false }) {
 
 // Helper: save d2 svg assets alongside document
 export async function saveD2Assets({ markdown, docsDir }) {
+  if (!markdown) {
+    return markdown;
+  }
+
   const codeBlockRegex = /```d2\n([\s\S]*?)```/g;
 
   const { replaced } = await runIterator({

package/utils/markdown-checker.mjs

@@ -88,7 +88,7 @@ function checkDeadLinks(markdown, source, allowedLinks, errorMessages) {
     if (!path) continue;
 
     // Check if this link is in the allowed links set
-    if (!allowedLinks.has(trimLink)) {
+    if (!allowedLinks.has(path)) {
       errorMessages.push(
         `Found a dead link in ${source}: [${match[1]}](${trimLink}), ensure the link exists in the structure plan path`,
       );
@@ -243,7 +243,7 @@ function checkLocalImages(markdown, source, errorMessages, markdownFilePath, bas
  */
 function checkContentStructure(markdown, source, errorMessages) {
   const lines = markdown.split("\n");
-  const allCodeBlockRegex = /^\s*```(?:\w+)?$/;
+  const allCodeBlockRegex = /^\s*```(?:[a-zA-Z0-9_,\-+.#/:=]+)?$/;
 
   // State variables for different checks
   let inCodeBlock = false;
@@ -300,7 +300,7 @@ function checkContentStructure(markdown, source, errorMessages) {
   }
 
   // Check if content ends with proper punctuation (indicating completeness)
-  const validEndingPunctuation = [".", "。", ")", "|", "*", ">"];
+  const validEndingPunctuation = [".", "。", ")", "|", "*", ">", "`"];
   const trimmedText = markdown.trim();
   const hasValidEnding = validEndingPunctuation.some((punct) => trimmedText.endsWith(punct));
 

package/utils/mermaid-validator.mjs

@@ -5,19 +5,6 @@
 
 import { getMermaidWorkerPool, shutdownMermaidWorkerPool } from "./mermaid-worker-pool.mjs";
 
-/**
- * Worker-based mermaid validation - DEPRECATED but kept for compatibility
- * This function now delegates to the worker pool implementation
- * @param {string} content - Mermaid diagram content
- * @returns {boolean} - True if syntax is valid
- * @throws {Error} - If syntax is invalid
- * @deprecated Use validateMermaidSyntax instead which uses worker pool
- */
-export async function validateMermaidWithOfficialParser(content) {
-  // Delegate to the new worker-based implementation
-  return await validateMermaidSyntax(content);
-}
-
 /**
  * Basic mermaid syntax validation fallback
  * Used when worker validation fails due to environment issues

package/utils/utils.mjs

@@ -674,13 +674,15 @@ export function getAvailablePaths(userInput = "") {
       addExactPathMatch(searchTerm, results);
     }
 
-    // Remove duplicates based on value (path)
+    // Remove duplicates based on absolute path (real deduplication)
     const uniqueResults = [];
-    const seenPaths = new Set();
+    const seenAbsolutePaths = new Set();
 
     for (const item of results) {
-      if (!seenPaths.has(item.value)) {
-        seenPaths.add(item.value);
+      // Normalize to absolute path for proper deduplication
+      const absolutePath = normalizePath(item.value);
+      if (!seenAbsolutePaths.has(absolutePath)) {
+        seenAbsolutePaths.add(absolutePath);
         uniqueResults.push(item);
       }
     }
@@ -809,7 +811,11 @@ export async function getGitHubRepoInfo(repoUrl) {
     const apiUrl = `https://api.github.com/repos/${owner}/${repo}`;
 
     const response = await fetch(apiUrl);
-    if (!response.ok) return null;
+
+    if (!response.ok) {
+      console.warn("Failed to fetch GitHub repository info:", repoUrl, response.statusText);
+      return null;
+    }
 
     const data = await response.json();
     return {