collabmd 0.1.31 → 0.1.32

package/src/client/presentation/file-history-view-controller.js

@@ -1,5 +1,4 @@
 import { escapeHtml } from '../domain/vault-utils.js';
-import { resolveApiUrl } from '../domain/runtime-paths.js';
 import { buttonClassNames } from './components/ui/button.js';
 
 const HISTORY_PAGE_SIZE = 30;
@@ -62,6 +61,7 @@ function createEmptyHistoryState() {
 export class FileHistoryViewController {
   constructor({
     diffRenderer = null,
+    gitApiClient = null,
     onOpenCommitDiff = () => {},
     onOpenFile = () => {},
     onOpenPreview = () => {},
@@ -69,6 +69,7 @@ export class FileHistoryViewController {
     toastController = null,
   } = {}) {
     this.diffRenderer = diffRenderer;
+    this.gitApiClient = gitApiClient;
     this.onOpenCommitDiff = onOpenCommitDiff;
     this.onOpenFile = onOpenFile;
     this.onOpenPreview = onOpenPreview;
@@ -89,6 +90,7 @@ export class FileHistoryViewController {
     this.nextButton = document.getElementById('diffNextBtn');
     this.layoutToggle = document.getElementById('diffLayoutToggle');
     this.modeButtons = Array.from(document.querySelectorAll('[data-diff-mode]'));
+    this.isActive = false;
     this.currentFilePath = null;
     this.localChanges = null;
     this.history = createEmptyHistoryState();
@@ -104,6 +106,9 @@ export class FileHistoryViewController {
 
   initialize() {
     this.openEditorButton?.addEventListener('click', () => {
+      if (!this.isActive) {
+        return;
+      }
       if (!this.currentFilePath) {
         return;
       }
@@ -113,6 +118,9 @@ export class FileHistoryViewController {
 
     this.modeButtons.forEach((button) => {
       button.addEventListener('click', () => {
+        if (!this.isActive) {
+          return;
+        }
         const nextMode = button.getAttribute('data-diff-mode');
         if (!nextMode || nextMode === this.diffMode) {
           return;
@@ -123,6 +131,9 @@ export class FileHistoryViewController {
     });
 
     this.content?.addEventListener('click', (event) => {
+      if (!this.isActive) {
+        return;
+      }
       const selectionButton = event.target instanceof Element
         ? event.target.closest('[data-file-history-select]')
         : null;
@@ -160,6 +171,7 @@ export class FileHistoryViewController {
   }
 
   hide() {
+    this.isActive = false;
     this.page?.classList.add('hidden');
     this.content?.classList.remove('diff-content--history');
     if (this.content) {
@@ -175,7 +187,6 @@ export class FileHistoryViewController {
     this.historyListElement = null;
     this.historyListScrollTop = 0;
     this.selectionRequestToken = 0;
-    this.syncToolbar();
   }
 
   getEntries() {
@@ -229,6 +240,7 @@ export class FileHistoryViewController {
   }
 
   async openFileHistory({ filePath }) {
+    this.isActive = true;
     this.currentFilePath = String(filePath ?? '').trim() || null;
     this.localChanges = null;
     this.history = {
@@ -289,16 +301,11 @@ export class FileHistoryViewController {
       return null;
     }
 
-    const query = new URLSearchParams();
-    query.set('metaOnly', 'true');
-    query.set('path', this.currentFilePath);
-    query.set('scope', 'all');
-    const response = await fetch(resolveApiUrl(`/git/diff?${query.toString()}`));
-    const data = await response.json();
-    if (!response.ok) {
-      throw new Error(data.error || 'Failed to load local file changes');
-    }
-
+    const data = await this.gitApiClient.readDiff({
+      metaOnly: true,
+      path: this.currentFilePath,
+      scope: 'all',
+    });
     const file = data.files?.[0];
     if (!file) {
       return null;
@@ -322,16 +329,11 @@ export class FileHistoryViewController {
       };
     }
 
-    const query = new URLSearchParams();
-    query.set('path', this.currentFilePath);
-    query.set('limit', String(HISTORY_PAGE_SIZE));
-    query.set('offset', String(offset));
-    const response = await fetch(resolveApiUrl(`/git/file-history?${query.toString()}`));
-    const data = await response.json();
-    if (!response.ok) {
-      throw new Error(data.error || 'Failed to load file history');
-    }
-
+    const data = await this.gitApiClient.readFileHistory({
+      limit: HISTORY_PAGE_SIZE,
+      offset,
+      path: this.currentFilePath,
+    });
     const commits = Array.isArray(data.commits) ? data.commits : [];
     return {
       commits,
@@ -350,30 +352,20 @@ export class FileHistoryViewController {
     }
 
     if (entry.type === 'local') {
-      const query = new URLSearchParams();
-      query.set('path', this.currentFilePath || '');
-      query.set('scope', 'all');
-      const response = await fetch(resolveApiUrl(`/git/diff?${query.toString()}`));
-      const data = await response.json();
-      if (!response.ok) {
-        throw new Error(data.error || 'Failed to load local diff');
-      }
-
+      const data = await this.gitApiClient.readDiff({
+        path: this.currentFilePath || '',
+        scope: 'all',
+      });
       return {
         detail: data.files?.[0] ?? null,
         summary: data.summary ?? entry.summary ?? null,
       };
     }
 
-    const query = new URLSearchParams();
-    query.set('hash', entry.hash || '');
-    query.set('path', entry.pathAtCommit || '');
-    const response = await fetch(resolveApiUrl(`/git/commit?${query.toString()}`));
-    const data = await response.json();
-    if (!response.ok) {
-      throw new Error(data.error || 'Failed to load commit diff');
-    }
-
+    const data = await this.gitApiClient.readCommit({
+      hash: entry.hash || '',
+      path: entry.pathAtCommit || '',
+    });
     return {
       commit: data.commit ?? null,
       detail: data.files?.[0] ?? null,
@@ -564,6 +556,9 @@ export class FileHistoryViewController {
   }
 
   renderLoading() {
+    if (!this.isActive) {
+      return;
+    }
     this.page?.classList.remove('hidden');
     if (this.content) {
       this.content.classList.add('diff-content--history');
@@ -574,6 +569,9 @@ export class FileHistoryViewController {
   }
 
   renderEmpty(message) {
+    if (!this.isActive) {
+      return;
+    }
     this.page?.classList.remove('hidden');
     if (this.content) {
       this.content.classList.add('diff-content--history');
@@ -744,6 +742,9 @@ export class FileHistoryViewController {
   }
 
   render() {
+    if (!this.isActive) {
+      return;
+    }
     this.page?.classList.remove('hidden');
     this.captureListScroll();
     this.syncToolbar();

package/src/client/presentation/git-diff-view-controller.js

@@ -1,5 +1,4 @@
 import { escapeHtml } from '../domain/vault-utils.js';
-import { resolveApiUrl } from '../domain/runtime-paths.js';
 import { buttonClassNames } from './components/ui/button.js';
 
 function getPathLeaf(pathValue) {
@@ -146,6 +145,7 @@ function renderSplitRow(leftLine, rightLine) {
 
 export class GitDiffViewController {
   constructor({
+    gitApiClient = null,
     onBackToHistory = null,
     onCommitStaged = null,
     onOpenFile = null,
@@ -153,6 +153,7 @@ export class GitDiffViewController {
     onUnstageFile = null,
     toastController = null,
   } = {}) {
+    this.gitApiClient = gitApiClient;
     this.onBackToHistory = onBackToHistory;
     this.onCommitStaged = onCommitStaged;
     this.onOpenFile = onOpenFile;
@@ -176,6 +177,7 @@ export class GitDiffViewController {
     this.layoutToggle = document.getElementById('diffLayoutToggle');
     this.modeButtons = Array.from(document.querySelectorAll('[data-diff-mode]'));
     this.layoutButtons = Array.from(document.querySelectorAll('[data-diff-layout]'));
+    this.isActive = false;
     this.mode = 'unified';
     this.layoutMode = 'focused';
     this.source = 'workspace';
@@ -196,9 +198,22 @@ export class GitDiffViewController {
   }
 
   initialize() {
-    this.prevButton?.addEventListener('click', () => this.navigateFile(-1));
-    this.nextButton?.addEventListener('click', () => this.navigateFile(1));
+    this.prevButton?.addEventListener('click', () => {
+      if (!this.isActive) {
+        return;
+      }
+      this.navigateFile(-1);
+    });
+    this.nextButton?.addEventListener('click', () => {
+      if (!this.isActive) {
+        return;
+      }
+      this.navigateFile(1);
+    });
     this.backToHistoryButton?.addEventListener('click', () => {
+      if (!this.isActive) {
+        return;
+      }
       if (!this.commitMeta?.hash) {
         return;
       }
@@ -208,6 +223,9 @@ export class GitDiffViewController {
       });
     });
     this.openEditorButton?.addEventListener('click', () => {
+      if (!this.isActive) {
+        return;
+      }
       if (this.source !== 'workspace') {
         return;
       }
@@ -220,6 +238,9 @@ export class GitDiffViewController {
       this.onOpenFile?.(currentFile.path);
     });
     this.primaryActionButton?.addEventListener('click', () => {
+      if (!this.isActive) {
+        return;
+      }
       const action = this.getPrimaryAction();
       if (!action) {
         return;
@@ -228,9 +249,15 @@ export class GitDiffViewController {
       void this.handleFileAction(action);
     });
     this.commitButton?.addEventListener('click', () => {
+      if (!this.isActive) {
+        return;
+      }
       void this.handleFileAction('commit');
     });
     this.content?.addEventListener('click', (event) => {
+      if (!this.isActive) {
+        return;
+      }
       const loadButton = event.target instanceof Element
         ? event.target.closest('[data-load-full-diff]')
         : null;
@@ -265,6 +292,9 @@ export class GitDiffViewController {
     });
     this.modeButtons.forEach((button) => {
       button.addEventListener('click', () => {
+        if (!this.isActive) {
+          return;
+        }
         const nextMode = button.getAttribute('data-diff-mode');
         if (!nextMode || nextMode === this.mode) {
           return;
@@ -275,6 +305,9 @@ export class GitDiffViewController {
     });
     this.layoutButtons.forEach((button) => {
       button.addEventListener('click', () => {
+        if (!this.isActive) {
+          return;
+        }
         const nextLayout = button.getAttribute('data-diff-layout');
         if (!nextLayout || nextLayout === this.layoutMode) {
           return;
@@ -283,11 +316,15 @@ export class GitDiffViewController {
       });
     });
     this.scrollContainer?.addEventListener('scroll', () => {
+      if (!this.isActive) {
+        return;
+      }
       this.handleScrollSelection();
     });
   }
 
   hide() {
+    this.isActive = false;
     this.page?.classList.add('hidden');
     if (this.content) {
       this.content.innerHTML = '';
@@ -308,10 +345,10 @@ export class GitDiffViewController {
     this.historyFilePath = null;
     this.pendingAction = null;
     this.repoStatus = null;
-    this.syncToolbar();
   }
 
   async openWorkspaceDiff({ filePath = null, scope = 'all' } = {}) {
+    this.isActive = true;
     this.source = 'workspace';
     this.layoutMode = 'focused';
     this.commitHash = null;
@@ -327,19 +364,11 @@ export class GitDiffViewController {
     this.renderLoading('Loading diff summary...');
 
     try {
-      const query = new URLSearchParams();
-      query.set('scope', scope);
-      if (filePath) {
-        query.set('path', filePath);
-      }
-      query.set('metaOnly', 'true');
-
-      const response = await fetch(resolveApiUrl(`/git/diff?${query.toString()}`));
-      const data = await response.json();
-      if (!response.ok) {
-        throw new Error(data.error || 'Failed to load git diff');
-      }
-
+      const data = await this.gitApiClient.readDiff({
+        metaOnly: true,
+        path: filePath,
+        scope,
+      });
       this.data = data;
       const initialIndex = filePath
         ? Math.max(0, data.files.findIndex((file) => file.path === filePath))
@@ -368,6 +397,7 @@ export class GitDiffViewController {
   }
 
   async openCommitDiff({ hash, path = null, historyFilePath = null } = {}) {
+    this.isActive = true;
     this.source = 'commit';
     this.layoutMode = 'stacked';
     this.commitHash = String(hash ?? '').trim() || null;
@@ -383,16 +413,10 @@ export class GitDiffViewController {
     this.renderLoading('Loading commit summary...');
 
     try {
-      const query = new URLSearchParams();
-      query.set('hash', this.commitHash || '');
-      query.set('metaOnly', 'true');
-
-      const response = await fetch(resolveApiUrl(`/git/commit?${query.toString()}`));
-      const data = await response.json();
-      if (!response.ok) {
-        throw new Error(data.error || 'Failed to load git commit');
-      }
-
+      const data = await this.gitApiClient.readCommit({
+        hash: this.commitHash || '',
+        metaOnly: true,
+      });
       this.data = data;
       this.commitMeta = data.commit ?? null;
       const initialIndex = path
@@ -476,6 +500,9 @@ export class GitDiffViewController {
   }
 
   renderLoading(message = 'Loading git diff...') {
+    if (!this.isActive) {
+      return;
+    }
     this.page?.classList.remove('hidden');
     if (this.content) {
       this.content.innerHTML = `<div class="diff-empty-state">${escapeHtml(message)}</div>`;
@@ -485,6 +512,9 @@ export class GitDiffViewController {
   }
 
   renderEmpty(message) {
+    if (!this.isActive) {
+      return;
+    }
     this.page?.classList.remove('hidden');
     if (this.content) {
       this.content.innerHTML = `<div class="diff-empty-state">${escapeHtml(message)}</div>`;
@@ -1053,37 +1083,21 @@ export class GitDiffViewController {
       try {
         let detail;
         if (this.source === 'commit') {
-          const query = new URLSearchParams();
-          query.set('hash', this.commitHash || '');
-          query.set('path', filePath);
-          if (forceFullPatch) {
-            query.set('allowLargePatch', 'true');
-          }
-
-          const response = await fetch(resolveApiUrl(`/git/commit?${query.toString()}`));
-          const data = await response.json();
-          if (!response.ok) {
-            throw new Error(data.error || 'Failed to load commit file diff');
-          }
-
+          const data = await this.gitApiClient.readCommit({
+            allowLargePatch: forceFullPatch,
+            hash: this.commitHash || '',
+            path: filePath,
+          });
           detail = {
             ...file,
             ...(data.files?.[0] ?? {}),
           };
         } else {
-          const query = new URLSearchParams();
-          query.set('scope', this.requestScope);
-          query.set('path', filePath);
-          if (forceFullPatch) {
-            query.set('allowLargePatch', 'true');
-          }
-
-          const response = await fetch(resolveApiUrl(`/git/diff?${query.toString()}`));
-          const data = await response.json();
-          if (!response.ok) {
-            throw new Error(data.error || 'Failed to load file diff');
-          }
-
+          const data = await this.gitApiClient.readDiff({
+            allowLargePatch: forceFullPatch,
+            path: filePath,
+            scope: this.requestScope,
+          });
           detail = {
             ...file,
             ...(data.files?.[0] ?? {}),
@@ -1187,6 +1201,9 @@ export class GitDiffViewController {
   }
 
   render() {
+    if (!this.isActive) {
+      return;
+    }
     this.page?.classList.remove('hidden');
 
     const files = this.data?.files ?? [];

package/src/client/presentation/git-panel-controller.js

@@ -1,5 +1,4 @@
 import { escapeHtml } from '../domain/vault-utils.js';
-import { resolveApiUrl } from '../domain/runtime-paths.js';
 import { buttonClassNames } from './components/ui/button.js';
 import { segmentedButtonClassNames, segmentedControlClassNames } from './components/ui/segmented-control.js';
 
@@ -94,6 +93,7 @@ function renderHistoryRowTitle(commit = {}) {
 export class GitPanelController {
   constructor({
     enabled = true,
+    gitApiClient = null,
     onCommitStaged = () => {},
     onOpenPullBackup = () => {},
     onPullBranch = () => {},
@@ -109,6 +109,7 @@ export class GitPanelController {
     toastController = null,
   } = {}) {
     this.enabled = enabled;
+    this.gitApiClient = gitApiClient;
     this.onCommitStaged = onCommitStaged;
     this.onOpenPullBackup = onOpenPullBackup;
     this.onPullBranch = onPullBranch;
@@ -404,21 +405,12 @@ export class GitPanelController {
     }
 
     try {
-      const response = await fetch(resolveApiUrl(`/git/status${force ? '?force=true' : ''}`));
-      const data = await response.json();
-      if (!response.ok) {
-        throw new Error(data.error || 'Failed to load git status');
-      }
-
+      const data = await this.gitApiClient.readStatus({ force });
       this.status = data;
       this.pullBackups = [];
       if (data.isGitRepo) {
         try {
-          const backupResponse = await fetch(resolveApiUrl('/git/pull-backups'));
-          const backupData = await backupResponse.json();
-          if (!backupResponse.ok) {
-            throw new Error(backupData.error || 'Failed to load pull backups');
-          }
+          const backupData = await this.gitApiClient.readPullBackups();
           this.pullBackups = Array.isArray(backupData.backups) ? backupData.backups : [];
         } catch (backupError) {
           console.error('[git-panel] Failed to load pull backups:', backupError);
@@ -501,15 +493,10 @@ export class GitPanelController {
     this.render();
 
     try {
-      const query = new URLSearchParams();
-      query.set('limit', String(HISTORY_PAGE_SIZE));
-      query.set('offset', '0');
-      const response = await fetch(resolveApiUrl(`/git/history?${query.toString()}`));
-      const data = await response.json();
-      if (!response.ok) {
-        throw new Error(data.error || 'Failed to load git history');
-      }
-
+      const data = await this.gitApiClient.readHistory({
+        limit: HISTORY_PAGE_SIZE,
+        offset: 0,
+      });
       this.history = {
         commits: Array.isArray(data.commits) ? data.commits : [],
         error: '',
@@ -553,15 +540,10 @@ export class GitPanelController {
     this.render();
 
     try {
-      const query = new URLSearchParams();
-      query.set('limit', String(HISTORY_PAGE_SIZE));
-      query.set('offset', String(this.history.offset));
-      const response = await fetch(resolveApiUrl(`/git/history?${query.toString()}`));
-      const data = await response.json();
-      if (!response.ok) {
-        throw new Error(data.error || 'Failed to load git history');
-      }
-
+      const data = await this.gitApiClient.readHistory({
+        limit: HISTORY_PAGE_SIZE,
+        offset: this.history.offset,
+      });
       const nextCommits = Array.isArray(data.commits) ? data.commits : [];
       this.history = {
         commits: [...this.history.commits, ...nextCommits],

package/src/client/styles/export-document.css

@@ -2,17 +2,24 @@ html[data-surface='export-document'] {
   height: auto;
   min-height: 100%;
   overflow-x: hidden;
-  overflow-y: auto;
+  overflow-y: hidden;
   overscroll-behavior: auto;
+  scrollbar-width: none;
 }
 
 body[data-surface='export-document'] {
   height: auto;
   overflow-x: hidden;
-  overflow-y: auto;
+  overflow-y: hidden;
   overscroll-behavior: auto;
   background: var(--color-bg);
   color: var(--color-text);
+  scrollbar-width: none;
+}
+
+html[data-surface='export-document']::-webkit-scrollbar,
+body[data-surface='export-document']::-webkit-scrollbar {
+  display: none;
 }
 
 .export-page-shell {
@@ -78,10 +85,22 @@ body[data-surface='export-document'] {
   overflow: visible;
 }
 
+.export-content .bases-table-shell,
+.export-content pre {
+  overflow: visible;
+}
+
 .export-content table {
   white-space: normal;
 }
 
+.export-content pre code {
+  width: auto;
+  white-space: pre-wrap;
+  word-break: break-word;
+  overflow-wrap: anywhere;
+}
+
 .export-diagram {
   display: flex;
   justify-content: center;
@@ -197,8 +216,18 @@ body[data-surface='export-document'] {
 }
 
 @media print {
+  html[data-surface='export-document'],
   body[data-surface='export-document'] {
+    height: auto;
+    overflow: hidden !important;
     background: var(--color-surface);
+    scrollbar-width: none;
+  }
+
+  html[data-surface='export-document']::-webkit-scrollbar,
+  body[data-surface='export-document']::-webkit-scrollbar,
+  .export-content *::-webkit-scrollbar {
+    display: none;
   }
 
   .export-page-shell {
@@ -218,6 +247,20 @@ body[data-surface='export-document'] {
     border: 0;
     border-radius: 0;
     box-shadow: none;
+    overflow: visible;
+  }
+
+  .export-content pre,
+  .export-content .table-wrapper,
+  .export-content .bases-table-shell {
+    overflow: visible !important;
+  }
+
+  .export-content pre code {
+    width: auto;
+    white-space: pre-wrap;
+    word-break: break-word;
+    overflow-wrap: anywhere;
   }
 
   .export-content h1,