From ebb70f5865bbf7b4f69d181380e64bee9fd0fb64 Mon Sep 17 00:00:00 2001 From: rob Date: Wed, 14 Jan 2026 05:01:01 -0400 Subject: [PATCH] Fix Browse All to use list_tools instead of search_tools search_tools requires a query, but list_tools allows browsing without a search term. Use list_tools when no query or tags are set. Co-Authored-By: Claude Opus 4.5 --- src/cmdforge/gui/pages/registry_page.py | 29 +++++++++++++++++-------- 1 file changed, 20 insertions(+), 9 deletions(-) diff --git a/src/cmdforge/gui/pages/registry_page.py b/src/cmdforge/gui/pages/registry_page.py index 2ed6dad..23fe4f4 100644 --- a/src/cmdforge/gui/pages/registry_page.py +++ b/src/cmdforge/gui/pages/registry_page.py @@ -32,15 +32,26 @@ class SearchWorker(QThread): def run(self): try: client = RegistryClient() - result = client.search_tools( - self.query, - category=self.category if self.category and self.category != "All" else None, - tags=self.tags, - page=self.page, - per_page=self.per_page, - sort=self.sort, - include_facets=True - ) + category = self.category if self.category and self.category != "All" else None + + # Use list_tools for browsing (no query/tags), search_tools for searching + if not self.query and not self.tags: + result = client.list_tools( + category=category, + page=self.page, + per_page=self.per_page, + sort=self.sort + ) + else: + result = client.search_tools( + self.query, + category=category, + tags=self.tags, + page=self.page, + per_page=self.per_page, + sort=self.sort, + include_facets=True + ) self.finished.emit(result) except Exception as e: self.error.emit(str(e))