TL;DR

Cache the result of this.hasIndexedContent() to avoid repeated database queries and improve performance.

Guidance

• Identify the first time content is indexed in the related SQLite tables and cache the result of this.hasIndexedContent() in a variable at that point.
• Modify the hasIndexedContent() method to return the cached result if it exists, instead of querying the database directly.
• Consider implementing a mechanism to update the cached result when the indexed content changes.
• Evaluate the trade-off between the performance gain and the potential for stale data if the cached result is not updated frequently enough.

Example

private hasIndexedContentCached: boolean | null = null;

async search(
  query: string,
  opts?: {
    maxResults?: number;
    minScore?: number;
    sessionKey?: string;
  },
): Promise<MemorySearchResult[]> {
  const preflight = resolveMemorySearchPreflight({
    query,
    hasIndexedContent: this.hasIndexedContent(),
  });
  // ...

private hasIndexedContent(): boolean {
  if (this.hasIndexedContentCached !== null) {
    return this.hasIndexedContentCached;
  }
  const result = /* query db directly */;
  this.hasIndexedContentCached = result;
  return result;
}

Notes

The proposed solution assumes that the result of this.hasIndexedContent() does not change frequently. If the result can change, additional logic will be needed to update the cached result.

Recommendation

Apply workaround: Cache the result of this.hasIndexedContent() to improve performance, as the potential performance gain outweighs the risk of stale data.