collabmd 0.1.44 → 0.1.46

package/src/server/domain/docx-exporter.js

@@ -1,4 +1,5 @@
 const DOCX_EXPORTER_CREATOR = 'CollabMD';
+let converterPromise = null;
 
 function normalizeDocxBuffer(result) {
   if (Buffer.isBuffer(result)) {
@@ -16,47 +17,37 @@ function normalizeDocxBuffer(result) {
   throw new Error('DOCX exporter returned an unsupported payload');
 }
 
-export class DocxExporter {
-  constructor({
-    loadConverter = () => import('@turbodocx/html-to-docx'),
-  } = {}) {
-    this.loadConverter = loadConverter;
-    this.converterPromise = null;
-  }
-
-  async ensureConverter() {
-    if (this.converterPromise) {
-      return this.converterPromise;
-    }
-
-    this.converterPromise = this.loadConverter().then((module) => {
+async function ensureConverter() {
+  if (!converterPromise) {
+    converterPromise = import('@turbodocx/html-to-docx').then((module) => {
       const converter = module?.default ?? module?.HTMLToDOCX ?? module;
       if (typeof converter !== 'function') {
         throw new Error('DOCX converter failed to load');
       }
       return converter;
     });
-    return this.converterPromise;
   }
 
-  async render({
-    html,
-    title = '',
-  } = {}) {
-    const converter = await this.ensureConverter();
-    const result = await converter(String(html ?? ''), null, {
-      creator: DOCX_EXPORTER_CREATOR,
-      font: 'Arial',
-      footer: false,
-      pageNumber: false,
-      table: {
-        row: {
-          cantSplit: true,
-        },
+  return converterPromise;
+}
+
+export async function renderDocx({
+  html,
+  title = '',
+} = {}) {
+  const converter = await ensureConverter();
+  const result = await converter(String(html ?? ''), null, {
+    creator: DOCX_EXPORTER_CREATOR,
+    font: 'Arial',
+    footer: false,
+    pageNumber: false,
+    table: {
+      row: {
+        cantSplit: true,
       },
-      title: String(title ?? ''),
-    });
+    },
+    title: String(title ?? ''),
+  });
 
-    return normalizeDocxBuffer(result);
-  }
+  return normalizeDocxBuffer(result);
 }

package/src/server/domain/ripgrep-search-service.js

@@ -22,7 +22,9 @@ const TEXT_SEARCH_GLOBS = Object.freeze([
   '*.mermaid',
   '*.puml',
   '*.plantuml',
+  '*.dsl',
   '*.drawio',
+  '*.excalidraw',
 ]);
 
 function normalizePositiveInt(value, fallback, { min = 1, max = Number.MAX_SAFE_INTEGER } = {}) {
@@ -106,7 +108,7 @@ function createRipgrepArgs(query, {
     '--max-filesize',
     String(maxFileSize || DEFAULT_MAX_FILE_SIZE),
     '--max-count',
-    String(maxSnippetsPerFile),
+    String(Math.max(Number(maxSnippetsPerFile) || DEFAULT_MAX_SNIPPETS_PER_FILE, 1) + 1),
     '--glob',
     '!.git/**',
     '--glob',
@@ -171,6 +173,7 @@ export function parseRipgrepJson(stdout, {
         kind: getVaultFileKind(filePath) ?? 'text',
         matchCount: 0,
         snippets: [],
+        truncated: false,
       };
       filesByPath.set(filePath, fileGroup);
       files.push(fileGroup);
@@ -187,6 +190,7 @@ export function parseRipgrepJson(stdout, {
       matchCount += 1;
 
       if (fileGroup.snippets.length >= maxSnippetsPerFile) {
+        fileGroup.truncated = true;
         truncated = true;
         continue;
       }
@@ -276,6 +280,7 @@ export class RipgrepSearchService {
   async search({
     limit = DEFAULT_MAX_FILES,
     query = '',
+    signal = null,
   } = {}) {
     const normalizedQuery = String(query ?? '').trim();
     const maxFiles = normalizePositiveInt(limit, this.maxFiles, {
@@ -308,6 +313,7 @@ export class RipgrepSearchService {
         cwd: this.vaultDir,
         encoding: 'utf8',
         maxBuffer: this.maxBufferBytes,
+        ...(signal ? { signal } : {}),
         timeout: this.timeoutMs,
       });
       const parsed = parseRipgrepJson(result.stdout, {

package/src/server/domain/workspace-state.js

@@ -42,6 +42,20 @@ export function createWorkspaceMetadata(pathValue, type, info = {}) {
   };
 }
 
+function enrichWorkspaceEntries(entries, metadata) {
+  const enrichedEntries = new Map(entries);
+  metadata.forEach((entryMetadata, pathValue) => {
+    const entry = enrichedEntries.get(pathValue);
+    const mtimeMs = Number(entryMetadata?.mtimeMs);
+    if (!entry || !Number.isFinite(mtimeMs) || entry.mtimeMs === mtimeMs) {
+      return;
+    }
+
+    enrichedEntries.set(pathValue, { ...entry, mtimeMs });
+  });
+  return enrichedEntries;
+}
+
 export function collectWorkspaceStateStats(entries = new Map(), metadata = new Map()) {
   const filePaths = [];
   const markdownPaths = [];
@@ -75,8 +89,11 @@ export function collectWorkspaceStateStats(entries = new Map(), metadata = new M
 export function createWorkspaceStateSnapshot(entries = new Map(), metadata = new Map(), {
   scannedAt = Date.now(),
 } = {}) {
-  const normalizedEntries = entries instanceof Map ? entries : new Map(entries ?? []);
   const normalizedMetadata = metadata instanceof Map ? metadata : new Map(metadata ?? []);
+  const normalizedEntries = enrichWorkspaceEntries(
+    entries instanceof Map ? entries : new Map(entries ?? []),
+    normalizedMetadata,
+  );
   const stats = collectWorkspaceStateStats(normalizedEntries, normalizedMetadata);
 
   return {
@@ -385,21 +402,24 @@ export function createWorkspaceTree(entries = new Map()) {
       return;
     }
 
-    if (entry?.type === 'directory' || entry?.nodeType === 'directory') {
-      nodesByPath.set(normalizedPath, {
+    const mtimeMs = Number(entry?.mtimeMs);
+    const node = entry?.type === 'directory' || entry?.nodeType === 'directory'
+      ? {
         children: [],
         name: entry?.name ?? basename(normalizedPath),
         path: normalizedPath,
         type: 'directory',
-      });
-      return;
-    }
+      }
+      : {
+        name: entry?.name ?? basename(normalizedPath),
+        path: normalizedPath,
+        type: entry?.type ?? getVaultTreeNodeType(normalizedPath),
+      };
 
-    nodesByPath.set(normalizedPath, {
-      name: entry?.name ?? basename(normalizedPath),
-      path: normalizedPath,
-      type: entry?.type ?? getVaultTreeNodeType(normalizedPath),
-    });
+    if (node.type !== 'directory' && Number.isFinite(mtimeMs)) {
+      node.mtimeMs = mtimeMs;
+    }
+    nodesByPath.set(normalizedPath, node);
   });
 
   nodesByPath.forEach((node, pathValue) => {
@@ -423,6 +443,7 @@ export function workspaceEntriesEqual(left = {}, right = {}) {
     && left.nodeType === right.nodeType
     && left.parentPath === right.parentPath
     && left.path === right.path
+    && left.mtimeMs === right.mtimeMs
     && left.type === right.type
   );
 }

package/src/server/infrastructure/git/diff-service.js

@@ -1,5 +1,6 @@
 import { countPatchLines, parseNumstatOutput, parseUnifiedDiff } from './parsers.js';
-import { createDiffResponse, createEmptyStats } from './responses.js';
+import { createEmptyStats } from './responses.js';
+import { getVaultFileKind } from '../../../domain/file-kind.js';
 
 function createSectionMap(sections = []) {
   return new Map(sections.map((section) => [section.key, section]));
@@ -33,6 +34,7 @@ function mergeScopedFile(existingFile, nextFile, scope) {
     return {
       ...nextFile,
       ...nextFlags,
+      fileKind: nextFile.fileKind || getVaultFileKind(nextFile.path),
       scope,
       stats: createEmptyStats(),
     };
@@ -44,6 +46,7 @@ function mergeScopedFile(existingFile, nextFile, scope) {
     return {
       ...existingFile,
       ...nextFile,
+      fileKind: existingFile.fileKind || getVaultFileKind(nextFile.path),
       hasStagedChanges: existingFile.hasStagedChanges || nextFlags.hasStagedChanges,
       hasUntrackedChanges: existingFile.hasUntrackedChanges || nextFlags.hasUntrackedChanges,
       hasWorkingTreeChanges: existingFile.hasWorkingTreeChanges || nextFlags.hasWorkingTreeChanges,
@@ -54,6 +57,7 @@ function mergeScopedFile(existingFile, nextFile, scope) {
 
   return {
     ...existingFile,
+    fileKind: existingFile.fileKind || getVaultFileKind(nextFile.path),
     hasStagedChanges: existingFile.hasStagedChanges || nextFlags.hasStagedChanges,
     hasUntrackedChanges: existingFile.hasUntrackedChanges || nextFlags.hasUntrackedChanges,
     hasWorkingTreeChanges: existingFile.hasWorkingTreeChanges || nextFlags.hasWorkingTreeChanges,
@@ -162,12 +166,18 @@ export class GitDiffService {
   async getDiff({ allowLargePatch = false, metaOnly = false, path = null, scope = 'working-tree' } = {}) {
     const isGitRepo = await this.commandRunner.isGitRepo();
     if (!isGitRepo) {
-      return createDiffResponse({
+      return {
         files: [],
         isGitRepo: false,
         metaOnly,
+        path: null,
         scope,
-      });
+        summary: {
+          additions: 0,
+          deletions: 0,
+          filesChanged: 0,
+        },
+      };
     }
 
     const resolvedScope = scope === 'staged' || scope === 'all'
@@ -214,13 +224,14 @@ export class GitDiffService {
         scope,
       });
 
-      return createDiffResponse({
+      return {
         files: scopedFiles,
+        isGitRepo: true,
         metaOnly: true,
         path,
         scope,
         summary: scopeSummary,
-      });
+      };
     }
 
     const isSinglePathRequest = Boolean(path);
@@ -234,7 +245,7 @@ export class GitDiffService {
       };
 
       if (!allowLargePatch && fileLineCount > this.maxInitialPatchLines) {
-        return createDiffResponse({
+        return {
           files: [{
             ...currentFile,
             canLoadFullPatch: true,
@@ -246,11 +257,12 @@ export class GitDiffService {
             },
             tooLarge: true,
           }],
+          isGitRepo: true,
           metaOnly: false,
           path,
           scope,
           summary,
-        });
+        };
       }
 
       const syntheticFiles = await this.untrackedFileService.buildSyntheticDiffs([currentFile]);
@@ -263,7 +275,7 @@ export class GitDiffService {
         },
       };
 
-      return createDiffResponse({
+      return {
         files: [{
           ...currentFile,
           ...detail,
@@ -271,11 +283,12 @@ export class GitDiffService {
           patchLineCount: countPatchLines(detail),
           tooLarge: false,
         }],
+        isGitRepo: true,
         metaOnly: false,
         path,
         scope,
         summary,
-      });
+      };
     }
 
     let singleFileSummary = null;
@@ -293,7 +306,7 @@ export class GitDiffService {
         && singleFileSummary.filesChanged > 0
         && (singleFileSummary.additions + singleFileSummary.deletions) > this.maxInitialPatchLines
       ) {
-        return createDiffResponse({
+        return {
           files: [{
             ...currentFile,
             canLoadFullPatch: true,
@@ -305,11 +318,12 @@ export class GitDiffService {
             },
             tooLarge: true,
           }],
+          isGitRepo: true,
           metaOnly: false,
           path,
           scope,
           summary: singleFileSummary,
-        });
+        };
       }
     }
 
@@ -362,6 +376,7 @@ export class GitDiffService {
       return {
         ...file,
         ...detail,
+        fileKind: file.fileKind || detail.fileKind || getVaultFileKind(file.path),
         canLoadFullPatch: false,
         patchLineCount,
         tooLarge: false,
@@ -378,12 +393,13 @@ export class GitDiffService {
       filesChanged: 0,
     });
 
-    return createDiffResponse({
+    return {
       files: mergedFiles,
+      isGitRepo: true,
       metaOnly: false,
       path,
       scope,
       summary: isSinglePathRequest ? singleFileSummary ?? summary : summary,
-    });
+    };
   }
 }

package/src/server/infrastructure/git/git-service.js

@@ -1,6 +1,10 @@
-import { createGitRequestError } from './errors.js';
+import { execFile as execFileCallback } from 'node:child_process';
+import { access } from 'node:fs/promises';
+import { join } from 'node:path';
+import { promisify } from 'node:util';
+
+import { createRequestError as createGitRequestError } from '../http/http-errors.js';
 import { normalizeRelativeGitPath } from './path-utils.js';
-import { GitCommandRunner } from './command-runner.js';
 import { GitDiffService } from './diff-service.js';
 import { GitHistoryService } from './history-service.js';
 import { parseNameStatusOutput } from './parsers.js';
@@ -9,6 +13,8 @@ import { GitStatusService } from './status-service.js';
 import { GitUntrackedFileService } from './untracked-files.js';
 import { PullBackupStore } from '../persistence/pull-backup-store.js';
 
+const execFile = promisify(execFileCallback);
+
 function buildAuthorEnv(author = null) {
   const name = String(author?.name ?? '').trim();
   const email = String(author?.email ?? '').trim();
@@ -35,12 +41,53 @@ export class GitService {
     statusCacheTtlMs = 2_000,
     vaultDir,
   }) {
-    this.commandRunner = new GitCommandRunner({
-      commandEnv,
-      enabled,
-      execFileImpl,
-      vaultDir,
-    });
+    const gitExecFile = execFileImpl ?? execFile;
+    this.commandRunner = {
+      async isGitRepo() {
+        if (!enabled) {
+          return false;
+        }
+
+        try {
+          await access(join(vaultDir, '.git'));
+          return true;
+        } catch {
+          return false;
+        }
+      },
+      async execGit(args, { env = null } = {}) {
+        const result = await gitExecFile('git', ['-c', 'core.quotepath=false', ...args], {
+          cwd: vaultDir,
+          encoding: 'utf8',
+          env: {
+            ...process.env,
+            ...(commandEnv ?? {}),
+            ...(env ?? {}),
+          },
+          maxBuffer: 5 * 1024 * 1024,
+          timeout: 10_000,
+        });
+
+        return String(result.stdout ?? '');
+      },
+      async execGitBuffer(args, { env = null } = {}) {
+        const result = await gitExecFile('git', ['-c', 'core.quotepath=false', ...args], {
+          cwd: vaultDir,
+          encoding: null,
+          env: {
+            ...process.env,
+            ...(commandEnv ?? {}),
+            ...(env ?? {}),
+          },
+          maxBuffer: 5 * 1024 * 1024,
+          timeout: 10_000,
+        });
+
+        return Buffer.isBuffer(result.stdout)
+          ? result.stdout
+          : Buffer.from(result.stdout ?? '');
+      },
+    };
     this.untrackedFileService = new GitUntrackedFileService({ vaultDir });
     this.statusService = new GitStatusService({
       commandRunner: this.commandRunner,
@@ -177,7 +224,7 @@ export class GitService {
       throw createGitRequestError(
         409,
         'Cannot pull because local and remote commits have diverged; fast-forward only pull is not possible.',
-        'pull_diverged_ff_only',
+        { requestCode: 'pull_diverged_ff_only' },
       );
     }
 
@@ -263,6 +310,10 @@ export class GitService {
     return this.historyService.getFileSnapshot({ hash, path });
   }
 
+  async getFileAttachment({ hash, path } = {}) {
+    return this.historyService.getFileAttachment({ hash, path });
+  }
+
   async pathExistsAtRef(ref, path) {
     try {
       await this.commandRunner.execGit(['cat-file', '-e', `${ref}:${path}`]);
@@ -431,7 +482,7 @@ export class GitService {
       return createGitRequestError(
         409,
         'Pull applied remote updates, but reapplying local changes caused conflicts. Review the conflicted files and the pull backup summary.',
-        'pull_conflicted_after_autostash',
+        { requestCode: 'pull_conflicted_after_autostash' },
       );
     }