diff --git a/src/main.ts b/src/main.ts index 1a7c01b..fe4847a 100644 --- a/src/main.ts +++ b/src/main.ts @@ -56,7 +56,7 @@ export default class OmnisearchPlugin extends Plugin { this.app.vault.on('create', async file => { if (isFileIndexable(file.path)) { await cacheManager.addToLiveCache(file.path) - searchEngine.addFromPaths([file.path], false) + searchEngine.addFromPaths([file.path]) } }) ) @@ -80,7 +80,7 @@ export default class OmnisearchPlugin extends Plugin { cacheManager.removeFromLiveCache(oldPath) cacheManager.addToLiveCache(file.path) searchEngine.removeFromPaths([oldPath]) - await searchEngine.addFromPaths([file.path], false) + await searchEngine.addFromPaths([file.path]) } }) ) @@ -157,10 +157,7 @@ async function populateIndex(): Promise { indexingStep.set(IndexingStepType.IndexingFiles) searchEngine.removeFromPaths(diff.toRemove.map(o => o.path)) - await searchEngine.addFromPaths( - diff.toAdd.map(o => o.path), - true - ) + await searchEngine.addFromPaths(diff.toAdd.map(o => o.path)) if (diff.toRemove.length || diff.toAdd.length) { indexingStep.set(IndexingStepType.WritingCache) diff --git a/src/notes-index.ts b/src/notes-index.ts index 6b2f243..1a35f57 100644 --- a/src/notes-index.ts +++ b/src/notes-index.ts @@ -45,7 +45,7 @@ export async function refreshIndex(): Promise { const paths = [...notesToReindex].map(n => n.path) if (paths.length) { searchEngine.removeFromPaths(paths) - searchEngine.addFromPaths(paths, false) + searchEngine.addFromPaths(paths) notesToReindex.clear() // console.log(`Omnisearch - Reindexed ${paths.length} file(s)`) } diff --git a/src/search/omnisearch.ts b/src/search/omnisearch.ts index ccbc971..3fb59a6 100644 --- a/src/search/omnisearch.ts +++ b/src/search/omnisearch.ts @@ -101,10 +101,7 @@ export class Omnisearch { * Add notes/PDFs/images to the search index * @param paths */ - public async addFromPaths( - paths: string[], - mustWriteToCache: boolean - ): Promise { + public async addFromPaths(paths: string[]): Promise { let documents = await Promise.all( paths.map(async path => await cacheManager.getDocument(path)) ) @@ -127,10 +124,6 @@ export class Omnisearch { // Add docs to minisearch await this.minisearch.addAllAsync(docs) - // Save the index - if (mustWriteToCache) { - await this.writeToCache() - } } } @@ -305,7 +298,6 @@ export class Omnisearch { .filter(s => s.value.startsWith('#')) .map(s => s.value) - // TODO: this already called in search(), pass each document in its SearchResult instead? const documents = await Promise.all( results.map(async result => await cacheManager.getDocument(result.id)) )