collabmd 0.1.54 → 0.1.56

package/src/domain/excalidraw-room-codec.js

@@ -1,8 +1,8 @@
 import * as Y from 'yjs';
 
-const EXCALIDRAW_TYPE = 'excalidraw';
-const EXCALIDRAW_VERSION = 2;
-const EXCALIDRAW_SOURCE = 'collabmd';
+import { createEmptyScene, normalizeScene, tryParseSceneJson } from './excalidraw-scene.js';
+
+export { tryParseSceneJson as tryParseExcalidrawSceneJson } from './excalidraw-scene.js';
 
 export const EXCALIDRAW_APP_STATE_KEY = 'excalidraw-app-state';
 export const EXCALIDRAW_ELEMENTS_KEY = 'excalidraw-elements';
@@ -13,65 +13,10 @@ export const EXCALIDRAW_ROOM_TEXT_KEY = 'codemirror';
 export const EXCALIDRAW_SCHEMA_VERSION_KEY = 'schemaVersion';
 export const EXCALIDRAW_REPLACE_GENERATION_KEY = 'replaceGeneration';
 
-function createEmptyScene() {
-  return {
-    type: EXCALIDRAW_TYPE,
-    version: EXCALIDRAW_VERSION,
-    source: EXCALIDRAW_SOURCE,
-    elements: [],
-    appState: {
-      gridSize: null,
-      viewBackgroundColor: '#ffffff',
-    },
-    files: {},
-  };
-}
-
-function normalizeAppState(appState = {}) {
-  return {
-    gridSize: appState?.gridSize ?? null,
-    viewBackgroundColor: appState?.viewBackgroundColor ?? '#ffffff',
-  };
-}
-
-function normalizeScene(raw) {
-  if (!raw || typeof raw !== 'object') {
-    return createEmptyScene();
-  }
-
-  return {
-    type: EXCALIDRAW_TYPE,
-    version: EXCALIDRAW_VERSION,
-    source: EXCALIDRAW_SOURCE,
-    elements: Array.isArray(raw.elements) ? raw.elements : [],
-    appState: normalizeAppState(raw.appState),
-    files: raw.files && typeof raw.files === 'object' ? raw.files : {},
-  };
-}
-
-function parseSceneJson(rawJson, { fallbackToEmpty = true } = {}) {
-  if (!rawJson) {
-    return fallbackToEmpty ? createEmptyScene() : null;
-  }
-
-  try {
-    return normalizeScene(JSON.parse(rawJson));
-  } catch {
-    return fallbackToEmpty ? createEmptyScene() : null;
-  }
-}
-
 function cloneJsonValue(value) {
   return structuredClone(value);
 }
 
-function getRevisionKey(element) {
-  const version = Number.isFinite(Number(element?.version)) ? Number(element.version) : 0;
-  const versionNonce = Number.isFinite(Number(element?.versionNonce)) ? Number(element.versionNonce) : 0;
-  const updated = Number.isFinite(Number(element?.updated)) ? Number(element.updated) : 0;
-  return `${version}:${versionNonce}:${updated}`;
-}
-
 function compareElementVersions(left, right) {
   const leftVersion = Number(left?.version) || 0;
   const rightVersion = Number(right?.version) || 0;
@@ -143,14 +88,18 @@ function selectWinningElementFromSlot(slot) {
   }
 
   let winningElement = null;
-  slot.forEach((value) => {
+  let winningKey = '';
+  slot.forEach((value, key) => {
     const candidate = readRevisionEntry(value);
     if (!candidate || !candidate.id) {
       return;
     }
 
-    if (!winningElement || compareElementVersions(candidate, winningElement) > 0) {
+    // Y.Map iteration order can differ across replicas; equal revisions need a stable tie-break.
+    const comparison = winningElement ? compareElementVersions(candidate, winningElement) : 1;
+    if (comparison > 0 || (comparison === 0 && key > winningKey)) {
       winningElement = candidate;
+      winningKey = key;
     }
   });
 
@@ -160,12 +109,16 @@ function selectWinningElementFromSlot(slot) {
 function writeElementSlot(slot, element, {
   maxRevisions = 2,
 } = {}) {
-  const revisionKey = getRevisionKey(element);
+  // Each writer keeps its own candidate so concurrent version/nonce winners survive.
+  // ponytail: metadata grows with writers, not edits; authoritative replace compacts
+  // legacy keys. Never replace a live slot while reconnecting peers can edit it.
+  const revisionKey = `writer:${slot.doc.clientID}`;
+  const previous = readRevisionEntry(slot.get(revisionKey));
+  if (previous && compareElementVersions(element, previous) < 0) {
+    return false;
+  }
   const serialized = cloneJsonValue(element);
-  const previous = slot.get(revisionKey);
-  const previousSerialized = previous && typeof previous === 'object'
-    ? JSON.stringify(previous)
-    : '';
+  const previousSerialized = previous ? JSON.stringify(previous) : '';
   const nextSerialized = JSON.stringify(serialized);
   if (previousSerialized === nextSerialized) {
     return false;
@@ -199,7 +152,8 @@ function pruneElementSlot(slot, maxRevisions = 2) {
     if (!right.element) {
       return 1;
     }
-    return compareElementVersions(left.element, right.element);
+    return compareElementVersions(left.element, right.element)
+      || (left.key < right.key ? -1 : left.key > right.key ? 1 : 0);
   });
 
   while (revisions.length > maxRevisions) {
@@ -214,6 +168,14 @@ function sortSceneElements(elements) {
   return [...elements].sort(compareElementIndex);
 }
 
+export function normalizeExcalidrawRoomScene(rawScene, { includeDeleted = true } = {}) {
+  const scene = normalizeScene(rawScene);
+  scene.elements = sortSceneElements(scene.elements.filter((element) => (
+    element?.id && (includeDeleted || !element.isDeleted)
+  )));
+  return scene;
+}
+
 export function isExcalidrawRoomDocStructured(ydoc) {
   if (!ydoc) {
     return false;
@@ -259,10 +221,6 @@ export function buildExcalidrawRoomScene(ydoc, {
       return;
     }
 
-    if (!includeDeleted && winningElement.isDeleted) {
-      return;
-    }
-
     elements.push(winningElement);
   });
 
@@ -275,14 +233,14 @@ export function buildExcalidrawRoomScene(ydoc, {
     files[key] = cloneJsonValue(value);
   });
 
-  const scene = normalizeScene({
+  const scene = normalizeExcalidrawRoomScene({
     appState: {
       gridSize: appStateMap.get('gridSize'),
       viewBackgroundColor: appStateMap.get('viewBackgroundColor'),
     },
-    elements: sortSceneElements(elements),
+    elements,
     files,
-  });
+  }, { includeDeleted });
 
   return scene;
 }
@@ -291,9 +249,6 @@ export function serializeExcalidrawRoomScene(ydoc, {
   includeDeleted = false,
 } = {}) {
   const scene = buildExcalidrawRoomScene(ydoc, { includeDeleted });
-  if (!includeDeleted) {
-    scene.elements = scene.elements.filter((element) => !element.isDeleted);
-  }
   return JSON.stringify(scene);
 }
 
@@ -328,7 +283,7 @@ export function replaceExcalidrawRoomScene(ydoc, rawScene, {
     filesMap.set(key, cloneJsonValue(value));
   });
 
-  const nextAppState = normalizeAppState(scene.appState);
+  const nextAppState = scene.appState;
   appStateMap.set('gridSize', nextAppState.gridSize);
   appStateMap.set('viewBackgroundColor', nextAppState.viewBackgroundColor);
   meta.set(EXCALIDRAW_REPLACE_GENERATION_KEY, readExcalidrawReplaceGeneration(ydoc) + 1);
@@ -372,7 +327,7 @@ export function applySceneDiffToExcalidrawRoom(ydoc, rawScene, {
     changed = true;
   });
 
-  const nextAppState = normalizeAppState(scene.appState);
+  const nextAppState = scene.appState;
   Object.entries(nextAppState).forEach(([key, value]) => {
     if (appStateMap.get(key) === value) {
       return;
@@ -398,7 +353,7 @@ export function readLegacyExcalidrawRoomScene(ydoc) {
   }
 
   const rawJson = ydoc.getText(EXCALIDRAW_ROOM_TEXT_KEY).toString();
-  return parseSceneJson(rawJson, { fallbackToEmpty: false });
+  return tryParseSceneJson(rawJson);
 }
 
 export function migrateLegacyExcalidrawRoomData(ydoc, rawScene, options = {}) {
@@ -406,7 +361,3 @@ export function migrateLegacyExcalidrawRoomData(ydoc, rawScene, options = {}) {
   replaceExcalidrawRoomScene(ydoc, scene, options);
   return scene;
 }
-
-export function tryParseExcalidrawSceneJson(rawJson) {
-  return parseSceneJson(rawJson, { fallbackToEmpty: false });
-}

package/src/domain/excalidraw-scene.js

@@ -0,0 +1,48 @@
+export function createEmptyScene() {
+  return {
+    type: 'excalidraw',
+    version: 2,
+    source: 'collabmd',
+    elements: [],
+    appState: {
+      gridSize: null,
+      viewBackgroundColor: '#ffffff',
+    },
+    files: {},
+  };
+}
+
+export function normalizeScene(raw) {
+  if (!raw || typeof raw !== 'object') {
+    return createEmptyScene();
+  }
+
+  return {
+    type: 'excalidraw',
+    version: 2,
+    source: 'collabmd',
+    elements: Array.isArray(raw.elements) ? raw.elements : [],
+    appState: {
+      gridSize: raw.appState?.gridSize ?? null,
+      viewBackgroundColor: raw.appState?.viewBackgroundColor ?? '#ffffff',
+    },
+    files: raw.files && typeof raw.files === 'object' ? raw.files : {},
+  };
+}
+
+export function parseSceneJson(rawJson) {
+  const parsed = tryParseSceneJson(rawJson);
+  return parsed || createEmptyScene();
+}
+
+export function tryParseSceneJson(rawJson) {
+  if (!rawJson) {
+    return null;
+  }
+
+  try {
+    return normalizeScene(JSON.parse(rawJson));
+  } catch {
+    return null;
+  }
+}

package/src/domain/file-kind.js

@@ -1,6 +1,7 @@
 const MARKDOWN_FILE_EXTENSIONS = Object.freeze(['.md', '.markdown', '.mdx']);
 const HTML_FILE_EXTENSIONS = Object.freeze(['.html', '.htm']);
 const BASE_FILE_EXTENSION = '.base';
+const CANVAS_FILE_EXTENSION = '.canvas';
 const EXCALIDRAW_FILE_EXTENSION = '.excalidraw';
 const DRAWIO_FILE_EXTENSION = '.drawio';
 const MERMAID_FILE_EXTENSIONS = Object.freeze(['.mmd', '.mermaid']);
@@ -10,6 +11,7 @@ const STRUCTURIZR_FILE_EXTENSIONS = Object.freeze([STRUCTURIZR_FILE_EXTENSION]);
 const PDF_FILE_EXTENSION = '.pdf';
 const IMAGE_ATTACHMENT_EXTENSIONS = Object.freeze(['.png', '.jpg', '.jpeg', '.webp', '.gif', '.svg']);
 const DIAGRAM_FILE_EXTENSIONS = Object.freeze([
+  CANVAS_FILE_EXTENSION,
   EXCALIDRAW_FILE_EXTENSION,
   DRAWIO_FILE_EXTENSION,
   ...MERMAID_FILE_EXTENSIONS,
@@ -24,7 +26,7 @@ const VAULT_FILE_EXTENSIONS = Object.freeze([
   PDF_FILE_EXTENSION,
   ...IMAGE_ATTACHMENT_EXTENSIONS,
 ]);
-const STRIP_VAULT_EXTENSION_PATTERN = /\.(?:md|markdown|mdx|html?|base|excalidraw|drawio|mmd|mermaid|puml|plantuml|dsl|pdf|png|jpe?g|webp|gif|svg)$/i;
+const STRIP_VAULT_EXTENSION_PATTERN = /\.(?:md|markdown|mdx|html?|base|canvas|excalidraw|drawio|mmd|mermaid|puml|plantuml|dsl|pdf|png|jpe?g|webp|gif|svg)$/i;
 
 function normalizeFilePath(filePath) {
   return String(filePath ?? '').trim().toLowerCase();
@@ -37,6 +39,7 @@ function hasFileExtension(filePath, extensions) {
 
 export {
   BASE_FILE_EXTENSION,
+  CANVAS_FILE_EXTENSION,
   DIAGRAM_FILE_EXTENSIONS,
   DRAWIO_FILE_EXTENSION,
   EXCALIDRAW_FILE_EXTENSION,
@@ -64,6 +67,10 @@ export function getVaultFileKind(filePath) {
     return 'base';
   }
 
+  if (hasFileExtension(filePath, [CANVAS_FILE_EXTENSION])) {
+    return 'canvas';
+  }
+
   if (hasFileExtension(filePath, [EXCALIDRAW_FILE_EXTENSION])) {
     return 'excalidraw';
   }
@@ -129,6 +136,10 @@ export function isExcalidrawFilePath(filePath) {
   return getVaultFileKind(filePath) === 'excalidraw';
 }
 
+export function isCanvasFilePath(filePath) {
+  return getVaultFileKind(filePath) === 'canvas';
+}
+
 export function isBaseFilePath(filePath) {
   return getVaultFileKind(filePath) === 'base';
 }
@@ -160,6 +171,7 @@ export function isImageAttachmentFilePath(filePath) {
 export function isDiagramFilePath(filePath) {
   const kind = getVaultFileKind(filePath);
   return kind === 'excalidraw'
+    || kind === 'canvas'
     || kind === 'drawio'
     || kind === 'mermaid'
     || kind === 'plantuml'

package/src/server/application/workspace-reconciliation.js

@@ -11,8 +11,6 @@ import {
   createWorkspaceTree,
   deriveNextWorkspaceStateForApiMutation as deriveNextWorkspaceStateFromApiMutation,
   diffWorkspaceEntries,
-  isIncrementalWorkspaceMutationAction,
-  readWorkspacePathState as readWorkspacePathStateFromAdapter,
 } from '../domain/workspace-state.js';
 
 function createEventId() {
@@ -87,18 +85,6 @@ export class WorkspaceReconciliation {
     return this.workspaceTree;
   }
 
-  isIncrementalApiAction(action) {
-    return isIncrementalWorkspaceMutationAction(action);
-  }
-
-  async readWorkspacePathState(pathValue, {
-    expectDirectory = null,
-  } = {}) {
-    return readWorkspacePathStateFromAdapter(this.workspaceStateAdapter, pathValue, {
-      expectDirectory,
-    });
-  }
-
   async deriveNextWorkspaceStateForApiMutation(action, workspaceChange = {}) {
     return deriveNextWorkspaceStateFromApiMutation(this.workspaceStateAdapter, {
       action,

package/src/server/domain/backlink-index.js

@@ -49,7 +49,6 @@ export class BacklinkIndex {
     /** @type {Set<string>} file path membership set */
     this._fileSet = new Set();
     this._wikiTargetIndex = createWikiTargetIndex(this._fileList);
-    this._built = false;
     this._requestedBuildVersion = 0;
     this._completedBuildVersion = 0;
     this._buildPromise = null;
@@ -141,7 +140,7 @@ export class BacklinkIndex {
     this.rawTargetKeysBySource.clear();
     this.rawTargetSources.clear();
 
-    const snapshot = workspaceState ?? await this._resolveWorkspaceState();
+    const snapshot = workspaceState ?? await this.vaultFileStore.scanWorkspaceState();
     this._fileList = Array.from(snapshot?.filePaths ?? snapshot?.markdownPaths ?? []);
     this._fileSet = new Set(this._fileList);
     this._sourceFileList = Array.from(
@@ -165,23 +164,9 @@ export class BacklinkIndex {
       }
     }
 
-    this._built = true;
     console.log(`[backlinks] Index built: ${this._sourceFileList.length} markdown sources, ${this._fileList.length} targets, ${this.reverse.size} targets with backlinks`);
   }
 
-  async _resolveWorkspaceState() {
-    if (typeof this.vaultFileStore.scanWorkspaceState === 'function') {
-      return this.vaultFileStore.scanWorkspaceState();
-    }
-
-    const tree = await this.vaultFileStore.tree();
-    const filePaths = flattenTree(tree);
-    return {
-      filePaths,
-      markdownPaths: filePaths.filter((filePath) => isMarkdownFilePath(filePath)),
-    };
-  }
-
   /**
    * Incrementally update the index when a file's content changes.
    * Call this after every persist / write.
@@ -469,13 +454,6 @@ export class BacklinkIndex {
     return results;
   }
 
-  /**
-   * Get the count of backlinks for a file (cheap — no I/O).
-   */
-  getBacklinkCount(filePath) {
-    return this.reverse.get(filePath)?.size ?? 0;
-  }
-
   // --- Private methods ---
 
   _indexFile(filePath, content, { targetPathAliases = new Map() } = {}) {
@@ -621,16 +599,3 @@ export class BacklinkIndex {
     }
   }
 }
-
-/** Flatten a vault tree into an array of file paths. */
-function flattenTree(nodes) {
-  const files = [];
-  for (const node of nodes) {
-    if (node.type && node.type !== 'directory') {
-      files.push(node.path);
-    } else if (node.children) {
-      files.push(...flattenTree(node.children));
-    }
-  }
-  return files;
-}

package/src/server/domain/bases/base-index-snapshot-store.js

@@ -26,17 +26,11 @@ function cloneSnapshotRowRecord(rowRecord) {
   };
 }
 
-function createWorkspaceFileSetSignature(fileEntries = []) {
-  return fileEntries.join('\n');
-}
-
-function createRowCacheKey(filePath, workspaceState, fileSetSignature) {
+function createRowMetadataSignature(filePath, workspaceState) {
   const metadata = workspaceState?.metadata?.get(filePath) ?? null;
   return [
-    filePath,
     Number.isFinite(metadata?.mtimeMs) ? metadata.mtimeMs : '',
     Number.isFinite(metadata?.size) ? metadata.size : '',
-    fileSetSignature,
   ].join('\0');
 }
 
@@ -139,6 +133,8 @@ export class BaseIndexSnapshotStore {
     this.indexSnapshot = null;
     this.lastWorkspaceState = null;
     this.rowRecordCache = new Map();
+    this.rowRecordCacheFileSetSignature = '';
+    this.rowRecordCacheGeneration = 0;
   }
 
   async getWorkspaceState() {
@@ -305,13 +301,16 @@ export class BaseIndexSnapshotStore {
   async buildIndexSnapshot(workspaceState = null) {
     const resolvedWorkspaceState = workspaceState ?? await this.getWorkspaceState();
     const fileEntries = listWorkspaceFilePaths(resolvedWorkspaceState);
-    const fileSetSignature = createWorkspaceFileSetSignature(fileEntries);
+    const cacheGeneration = this.prepareRowRecordCache(fileEntries);
     const wikiTargetIndex = createWikiTargetIndex(fileEntries);
     const markdownContents = new Map();
+    // Keep reused rows available if another build replaces the shared cache during I/O.
+    const cachedRowRecords = new Map(fileEntries.map((filePath) => [
+      filePath, this.getCachedRowRecord(filePath, resolvedWorkspaceState, cacheGeneration),
+    ]));
 
     await mapWithConcurrency(resolvedWorkspaceState.markdownPaths ?? [], INDEX_READ_CONCURRENCY, async (filePath) => {
-      const cacheKey = createRowCacheKey(filePath, resolvedWorkspaceState, fileSetSignature);
-      if (!this.rowRecordCache.has(cacheKey)) {
+      if (!cachedRowRecords.get(filePath)) {
         markdownContents.set(filePath, await this.vaultFileStore.readMarkdownFile(filePath));
       }
     });
@@ -322,13 +321,16 @@ export class BaseIndexSnapshotStore {
     const rawTargetSourcesByKey = new Map();
     const nextRowRecordCache = new Map();
     fileEntries.forEach((filePath) => {
-      const cacheKey = createRowCacheKey(filePath, resolvedWorkspaceState, fileSetSignature);
-      const cachedRowRecord = this.rowRecordCache.get(cacheKey);
+      const cachedRowRecord = cachedRowRecords.get(filePath);
       const markdownContent = markdownContents.get(filePath) ?? null;
       const rowRecord = cachedRowRecord
         ? cloneSnapshotRowRecord(cachedRowRecord)
         : this.createSnapshotRow(filePath, resolvedWorkspaceState, wikiTargetIndex, markdownContent);
-      nextRowRecordCache.set(cacheKey, cloneSnapshotRowRecord(rowRecord));
+      nextRowRecordCache.set(filePath, {
+        generation: cacheGeneration,
+        metadataSignature: createRowMetadataSignature(filePath, resolvedWorkspaceState),
+        rowRecord: cloneSnapshotRowRecord(rowRecord),
+      });
       rowsByPath.set(filePath, rowRecord.row);
       forwardLinksByPath.set(filePath, rowRecord.forwardLinks);
       rawTargetKeysBySourcePath.set(filePath, rowRecord.rawTargetKeys);
@@ -411,7 +413,7 @@ export class BaseIndexSnapshotStore {
         return pathValue && isWorkspaceFileEntry(entry);
       });
 
-    const fileSetSignature = createWorkspaceFileSetSignature(snapshot.filePaths);
+    const cacheGeneration = this.prepareRowRecordCache(snapshot.filePaths);
     await mapWithConcurrency(filePathsToRefresh, INDEX_READ_CONCURRENCY, async (filePath) => {
       const markdownContent = isMarkdownFilePath(filePath)
         ? await this.vaultFileStore.readMarkdownFile(filePath)
@@ -419,11 +421,11 @@ export class BaseIndexSnapshotStore {
       const rowRecord = this.createSnapshotRow(filePath, workspaceState, snapshot.wikiTargetIndex, markdownContent, {
         targetPathAliases,
       });
-      this.invalidateRowRecordCachePath(filePath);
-      this.rowRecordCache.set(
-        createRowCacheKey(filePath, workspaceState, fileSetSignature),
-        cloneSnapshotRowRecord(rowRecord),
-      );
+      this.rowRecordCache.set(filePath, {
+        generation: cacheGeneration,
+        metadataSignature: createRowMetadataSignature(filePath, workspaceState),
+        rowRecord: cloneSnapshotRowRecord(rowRecord),
+      });
       this.removeRawTargetSourceContributions(snapshot, filePath);
       snapshot.rowsByPath.set(filePath, rowRecord.row);
       snapshot.forwardLinksByPath.set(filePath, rowRecord.forwardLinks);
@@ -431,12 +433,27 @@ export class BaseIndexSnapshotStore {
     });
   }
 
+  prepareRowRecordCache(filePaths) {
+    const signature = JSON.stringify(filePaths);
+    if (signature !== this.rowRecordCacheFileSetSignature) {
+      // Link resolution depends on every target path, including previously missing files.
+      this.rowRecordCache.clear();
+      this.rowRecordCacheFileSetSignature = signature;
+      this.rowRecordCacheGeneration += 1;
+    }
+    return this.rowRecordCacheGeneration;
+  }
+
+  getCachedRowRecord(filePath, workspaceState, generation) {
+    const cached = this.rowRecordCache.get(filePath);
+    return cached?.generation === generation
+      && cached.metadataSignature === createRowMetadataSignature(filePath, workspaceState)
+      ? cached.rowRecord
+      : null;
+  }
+
   invalidateRowRecordCachePath(filePath) {
-    Array.from(this.rowRecordCache.keys()).forEach((cacheKey) => {
-      if (cacheKey === filePath || cacheKey.startsWith(`${filePath}\0`)) {
-        this.rowRecordCache.delete(cacheKey);
-      }
-    });
+    this.rowRecordCache.delete(filePath);
   }
 
   async ensureIndexSnapshot({ basePath = '', sourcePath = '' } = {}) {

package/src/server/domain/bases/base-query-service.js

@@ -56,32 +56,6 @@ function sortRows(rows, sortChain = []) {
   });
 }
 
-function compareRowsBySortChain(left, right, sortChain = []) {
-  for (const sortConfig of sortChain) {
-    const delta = compareValues(left.rawCells[sortConfig.property], right.rawCells[sortConfig.property]);
-    if (delta !== 0) {
-      return sortConfig.direction === 'desc' ? -delta : delta;
-    }
-  }
-
-  return String(left.file.path ?? '').localeCompare(String(right.file.path ?? ''));
-}
-
-function insertSortedBounded(rows, row, {
-  limit,
-  sortChain = [],
-}) {
-  let insertIndex = rows.findIndex((existingRow) => compareRowsBySortChain(row, existingRow, sortChain) < 0);
-  if (insertIndex === -1) {
-    insertIndex = rows.length;
-  }
-
-  rows.splice(insertIndex, 0, row);
-  if (rows.length > limit) {
-    rows.pop();
-  }
-}
-
 function pruneFilterNodeForProperty(filterNode, propertyId = '') {
   if (!propertyId || !filterNode) {
     return filterNode;
@@ -225,21 +199,12 @@ export class BaseQueryService {
     columns = [],
     definition,
     evaluatedPropertyIds,
-    limit = null,
-    maxRows = this.maxResultRows,
     snapshot,
-    sortChain = [],
     search = '',
     thisFile,
   }) {
+    // ponytail: retain matching rows for one native sort; use bounded selection if heap profiles require it.
     const rows = [];
-    const effectiveMaxRows = Number.isInteger(maxRows) && maxRows > 0 ? maxRows : Infinity;
-    const effectiveLimit = Number.isInteger(limit) && limit > 0 ? limit : Infinity;
-    const canBoundDuringCollection = Number.isFinite(Math.min(effectiveLimit, effectiveMaxRows))
-      && sortChain.length > 0
-      && !activeView.groupBy?.property
-      && (activeView.summaries?.length ?? 0) === 0;
-    const hardLimit = Math.min(effectiveLimit, effectiveMaxRows);
 
     const filePaths = snapshot.filePaths ?? [];
     for (let chunkStart = 0; chunkStart < filePaths.length; chunkStart += BASE_QUERY_ROW_CHUNK_SIZE) {
@@ -275,14 +240,6 @@ export class BaseQueryService {
           continue;
         }
 
-        if (canBoundDuringCollection) {
-          insertSortedBounded(rows, candidateRow, {
-            limit: hardLimit,
-            sortChain,
-          });
-          continue;
-        }
-
         rows.push(candidateRow);
       }
 
@@ -390,10 +347,7 @@ export class BaseQueryService {
     let rows = await this.collectCandidateRows({
       ...context,
       astCache,
-      limit: requestedLimit,
-      maxRows: this.maxResultRows,
       search,
-      sortChain,
     });
     rows = sortRows(rows, sortChain);