From 878f57c1b72277b963c45077f8a98af5021dd070 Mon Sep 17 00:00:00 2001
From: Christine Betts <chrstn@uw.edu>
Date: Thu, 12 Feb 2026 15:54:15 -0500
Subject: [PATCH] address comments

---
 .../cli/src/config/extensionRegistryClient.ts |  7 ++++++
 .../views/ExtensionRegistryView.test.tsx      | 24 +++++++++----------
 .../views/ExtensionRegistryView.tsx           |  6 +++--
 3 files changed, 23 insertions(+), 14 deletions(-)

diff --git a/packages/cli/src/config/extensionRegistryClient.ts b/packages/cli/src/config/extensionRegistryClient.ts
index aeda50dc48..80baf6b562 100644
--- a/packages/cli/src/config/extensionRegistryClient.ts
+++ b/packages/cli/src/config/extensionRegistryClient.ts
@@ -40,6 +40,13 @@ export class ExtensionRegistryClient {
     ExtensionRegistryClient.fetchPromise = null;
   }
 
+  /**
+   * Returns all extensions from the registry.
+   */
+  async getAllExtensions(): Promise<RegistryExtension[]> {
+    return this.fetchAllExtensions();
+  }
+
   async getExtensions(
     page: number = 1,
     limit: number = 10,
diff --git a/packages/cli/src/ui/components/views/ExtensionRegistryView.test.tsx b/packages/cli/src/ui/components/views/ExtensionRegistryView.test.tsx
index 6932bd3f7b..b774155ce9 100644
--- a/packages/cli/src/ui/components/views/ExtensionRegistryView.test.tsx
+++ b/packages/cli/src/ui/components/views/ExtensionRegistryView.test.tsx
@@ -14,7 +14,7 @@ import {
 } from '../../../config/extensionRegistryClient.js';
 import { type ExtensionManager } from '../../../config/extension-manager.js';
 
-vi.mock('../../config/extensionRegistryClient.js');
+vi.mock('../../../config/extensionRegistryClient.js');
 
 const mockExtensions = [
   {
@@ -37,6 +37,12 @@ describe('ExtensionRegistryView', () => {
   });
 
   it('should render loading state initially', async () => {
+    // Return a promise that doesn't resolve immediately to keep the loading state active
+    vi.spyOn(
+      ExtensionRegistryClient.prototype,
+      'getAllExtensions',
+    ).mockReturnValue(new Promise(() => {}));
+
     const mockExtensionManager = {
       getExtensions: vi.fn().mockReturnValue([]),
     };
@@ -52,11 +58,8 @@ describe('ExtensionRegistryView', () => {
   it('should render extensions after fetching', async () => {
     vi.spyOn(
       ExtensionRegistryClient.prototype,
-      'getExtensions',
-    ).mockResolvedValue({
-      extensions: mockExtensions as unknown as RegistryExtension[],
-      total: 2,
-    });
+      'getAllExtensions',
+    ).mockResolvedValue(mockExtensions as unknown as RegistryExtension[]);
 
     const mockExtensionManager = {
       getExtensions: vi.fn().mockReturnValue([]),
@@ -83,7 +86,7 @@ describe('ExtensionRegistryView', () => {
   it('should render error message on fetch failure', async () => {
     vi.spyOn(
       ExtensionRegistryClient.prototype,
-      'getExtensions',
+      'getAllExtensions',
     ).mockRejectedValue(new Error('Fetch failed'));
 
     const mockExtensionManager = {
@@ -110,11 +113,8 @@ describe('ExtensionRegistryView', () => {
   it('should call onSelect when an item is selected', async () => {
     vi.spyOn(
       ExtensionRegistryClient.prototype,
-      'getExtensions',
-    ).mockResolvedValue({
-      extensions: mockExtensions as unknown as RegistryExtension[],
-      total: 2,
-    });
+      'getAllExtensions',
+    ).mockResolvedValue(mockExtensions as unknown as RegistryExtension[]);
     const onSelect = vi.fn();
 
     const mockExtensionManager = {
diff --git a/packages/cli/src/ui/components/views/ExtensionRegistryView.tsx b/packages/cli/src/ui/components/views/ExtensionRegistryView.tsx
index 9e65d7ef6d..4b2b454f23 100644
--- a/packages/cli/src/ui/components/views/ExtensionRegistryView.tsx
+++ b/packages/cli/src/ui/components/views/ExtensionRegistryView.tsx
@@ -54,9 +54,11 @@ export function ExtensionRegistryView({
     let active = true;
     const fetchExtensions = async () => {
       try {
-        const result = await client.getExtensions(1, 1000); // Fetch a large enough batch
+        // Fetch all extensions to enable comprehensive local fuzzy search.
+        // Display virtualization/pagination is handled by SearchableList.
+        const extensions = await client.getAllExtensions();
         if (active) {
-          setExtensions(result.extensions);
+          setExtensions(extensions);
           setLoading(false);
         }
       } catch (err) {