fix: restart from the matcher processor when switching sorter/renderer

Author: lambdalisue

denops/fall/picker.ts

@@ -473,17 +473,27 @@ export class Picker<T extends Detail> implements AsyncDisposable {
         }
         this.#sortProcessor.sorterIndex = index;
         reserve((denops) => {
-          this.#sortProcessor.start(denops, {
-            items: this.#matchProcessor.items,
+          // NOTE:
+          // We need to restart from the matcher processor because
+          // sorters and renderers applies changes in-place thus
+          // the items would be polluted.
+          this.#matchProcessor.start(denops, {
+            items: this.#collectProcessor.items,
+            query: this.#inputComponent.cmdline,
           });
         });
         break;
       }
       case "switch-sorter-at":
         this.#sortProcessor.sorterIndex = event.index;
         reserve((denops) => {
-          this.#sortProcessor.start(denops, {
-            items: this.#matchProcessor.items,
+          // NOTE:
+          // We need to restart from the matcher processor because
+          // sorters and renderers applies changes in-place thus
+          // the items would be polluted.
+          this.#matchProcessor.start(denops, {
+            items: this.#collectProcessor.items,
+            query: this.#inputComponent.cmdline,
           });
         });
         break;
@@ -498,17 +508,27 @@ export class Picker<T extends Detail> implements AsyncDisposable {
         }
         this.#renderProcessor.rendererIndex = index;
         reserve((denops) => {
-          this.#renderProcessor.start(denops, {
-            items: this.#matchProcessor.items,
+          // NOTE:
+          // We need to restart from the matcher processor because
+          // sorters and renderers applies changes in-place thus
+          // the items would be polluted.
+          this.#matchProcessor.start(denops, {
+            items: this.#collectProcessor.items,
+            query: this.#inputComponent.cmdline,
           });
         });
         break;
       }
       case "switch-renderer-at":
         this.#renderProcessor.rendererIndex = event.index;
         reserve((denops) => {
-          this.#renderProcessor.start(denops, {
-            items: this.#matchProcessor.items,
+          // NOTE:
+          // We need to restart from the matcher processor because
+          // sorters and renderers applies changes in-place thus
+          // the items would be polluted.
+          this.#matchProcessor.start(denops, {
+            items: this.#collectProcessor.items,
+            query: this.#inputComponent.cmdline,
           });
         });
         break;