Fix sorting on type filter + empty query (#880)

* Fix sorting on type filter + empty query

* nit

* nit

src/components/searchbox/NodeSearchBox.vue

@@ -106,15 +106,15 @@ const placeholder = computed(() => {
 const nodeDefStore = useNodeDefStore()
 const nodeFrequencyStore = useNodeFrequencyStore()
 const search = (query: string) => {
+  const queryIsEmpty = query === '' && props.filters.length === 0
   currentQuery.value = query
-  suggestions.value =
-    query === ''
-      ? nodeFrequencyStore.topNodeDefs
-      : [
-          ...nodeDefStore.nodeSearchService.searchNode(query, props.filters, {
-            limit: props.searchLimit
-          })
-        ]
+  suggestions.value = queryIsEmpty
+    ? nodeFrequencyStore.topNodeDefs
+    : [
+        ...nodeDefStore.nodeSearchService.searchNode(query, props.filters, {
+          limit: props.searchLimit
+        })
+      ]
 }
 
 const emit = defineEmits(['addFilter', 'removeFilter', 'addNode'])

src/services/nodeSearchService.ts

@@ -32,11 +32,10 @@ export class FuseSearch<T> {
   }
 
   public search(query: string, options?: FuseSearchOptions): T[] {
-    if (!query || query === '') {
-      return [...this.data]
-    }
+    const fuseResult = !query
+      ? this.data.map((x) => ({ item: x, score: 0 }))
+      : this.fuse.search(query, options)
 
-    const fuseResult = this.fuse.search(query, options)
     if (!this.advancedScoring) {
       return fuseResult.map((x) => x.item)
     }

tests-ui/tests/nodeSearchService.test.ts

@@ -51,7 +51,11 @@ const EXAMPLE_NODE_DEFS: ComfyNodeDefImpl[] = [
     category: 'latent/batch',
     output_node: false
   }
-].map((nodeDef) => plainToClass(ComfyNodeDefImpl, nodeDef))
+].map((nodeDef) => {
+  const def = plainToClass(ComfyNodeDefImpl, nodeDef)
+  def['postProcessSearchScores'] = (s) => s
+  return def
+})
 
 describe('nodeSearchService', () => {
   it('searches with input filter', () => {