Refactor model management and API integration for improved consistency

- Removed the `useHuggingFaceModels` hook and updated related components to directly use the new API endpoint for fetching HuggingFace models.
- Refactored model-related components to utilize the unified model structure, enhancing data handling and consistency across the application.
- Updated the ModelDownloadList and ModelListItem components to streamline model display and download logic.
- Cleaned up unused imports and hooks, improving code clarity and maintainability.
- Adjusted the ModelListHeader to include a slider for filtering models by size, enhancing user experience.

Author: georgi

AGENTS.md

@@ -174,7 +174,6 @@ The project includes many specialized hooks organized by functionality:
 
 #### Model Management Hooks
 
-- **`useHuggingFaceModels`**: Fetches and caches HuggingFace models with TanStack Query
 - **`useLoraModels`**: Manages LoRA (Low-Rank Adaptation) model data
 - **`useModelBasePaths`**: Provides base cache directories for HuggingFace Hub and Ollama models
 - **`useModelsWithSize`**: Fetches model data with size information for storage planning

build.py

@@ -340,7 +340,7 @@ def pack(self) -> None:
                 "install",
                 "-p",
                 str(self.ENV_DIR),
-                "ffmpeg",
+                "ffmpeg>=6,<7",
                 "cairo",
                 "git",
                 "x264",
@@ -410,9 +410,7 @@ def __init__(
         self.python_version = python_version
         self.env_name = env_name or self.DEFAULT_ENV_NAME
         self.env_prefix = (
-            Path(env_prefix).expanduser().resolve()
-            if env_prefix is not None
-            else None
+            Path(env_prefix).expanduser().resolve() if env_prefix is not None else None
         )
         combined_channels = list(self.DEFAULT_CHANNELS)
         if channels:
@@ -502,7 +500,9 @@ def create(
     ) -> None:
         if self.env_exists():
             if not force:
-                logger.info("Conda environment already exists; use --force to recreate it.")
+                logger.info(
+                    "Conda environment already exists; use --force to recreate it."
+                )
                 return
             logger.info("Removing existing conda environment before recreation.")
             self.remove(skip_confirmation=True)
@@ -633,7 +633,9 @@ def handle_env_info(args: argparse.Namespace) -> None:
     manager.info()
 
 
-def handle_env_list(args: argparse.Namespace) -> None:  # noqa: ARG001 - matches signature
+def handle_env_list(
+    args: argparse.Namespace,
+) -> None:  # noqa: ARG001 - matches signature
     manager = CondaEnvironmentManager()
     manager.list()
 
@@ -682,7 +684,9 @@ def build_parser() -> argparse.ArgumentParser:
     )
     build_parser.set_defaults(func=handle_build_command)
 
-    env_parser = subparsers.add_parser("env", help="Manage the Nodetool conda environment")
+    env_parser = subparsers.add_parser(
+        "env", help="Manage the Nodetool conda environment"
+    )
     env_subparsers = env_parser.add_subparsers(dest="env_command", required=True)
 
     target_parent = argparse.ArgumentParser(add_help=False)

web/src/__tests__/components/chat/composer/NodeToolsSelector.test.tsx

@@ -61,9 +61,65 @@ jest.mock("@mui/icons-material", () => ({
   Close: () => <svg data-testid="CloseIcon" />
 }));
 
-// Use the real store; we'll set its state in tests
+// Test fixtures and shared mocks need to be defined before jest.mock() that use them
+const mockOnChange = jest.fn();
+const mockSetSearchTerm = jest.fn();
+const mockSetHoveredNode = jest.fn();
 
-// Do not mock the store itself; we will stub the selector hook instead
+const mockNodeMetadata: Record<string, any> = {
+  "tool.node1": {
+    node_type: "tool.node1",
+    title: "Tool Node 1",
+    namespace: "tool",
+    properties: [],
+    outputs: [],
+    expose_as_tool: true
+  },
+  "tool.node2": {
+    node_type: "tool.node2",
+    title: "Tool Node 2",
+    namespace: "tool",
+    properties: [],
+    outputs: [],
+    expose_as_tool: true
+  },
+  "regular.node": {
+    node_type: "regular.node",
+    title: "Regular Node",
+    namespace: "regular",
+    properties: [],
+    outputs: [],
+    expose_as_tool: false
+  }
+};
+
+const mockSearchResults = [
+  mockNodeMetadata["tool.node1"],
+  mockNodeMetadata["tool.node2"]
+];
+
+// Mock the NodeMenuStore hook directly so we can control searchTerm and results
+jest.mock("../../../../stores/NodeMenuStore", () => {
+  let currentStore: any = {
+    searchTerm: "",
+    setSearchTerm: (_term: string) => {},
+    searchResults: [],
+    isLoading: false,
+    selectedPath: [],
+    hoveredNode: null,
+    setHoveredNode: (_node: any) => {}
+  };
+  const useNodeToolsMenuStore = (selector?: any) =>
+    selector ? selector(currentStore) : currentStore;
+  const __setNodeToolsMenuStore = (newStore: any) => {
+    currentStore = newStore;
+  };
+  return {
+    __esModule: true,
+    useNodeToolsMenuStore,
+    __setNodeToolsMenuStore
+  };
+});
 
 // Mock the SearchInput component
 jest.mock("../../../../components/search/SearchInput", () => ({
@@ -150,6 +206,7 @@ jest.mock("lodash", () => ({
 }));
 
 import useMetadataStore from "../../../../stores/MetadataStore";
+import { __setNodeToolsMenuStore } from "../../../../stores/NodeMenuStore";
 import { useStoreWithEqualityFn } from "zustand/traditional";
 
 // Mock the useStoreWithEqualityFn hook to return from our mock store
@@ -163,48 +220,11 @@ jest.mock("react", () => ({
   useRef: jest.fn(() => ({ current: null }))
 }));
 
-// Test fixtures
-const mockNodeMetadata: Record<string, any> = {
-  "tool.node1": {
-    node_type: "tool.node1",
-    title: "Tool Node 1",
-    namespace: "tool",
-    properties: [],
-    outputs: [],
-    expose_as_tool: true
-  },
-  "tool.node2": {
-    node_type: "tool.node2",
-    title: "Tool Node 2",
-    namespace: "tool",
-    properties: [],
-    outputs: [],
-    expose_as_tool: true
-  },
-  "regular.node": {
-    node_type: "regular.node",
-    title: "Regular Node",
-    namespace: "regular",
-    properties: [],
-    outputs: [],
-    expose_as_tool: false
-  }
-};
-
-const mockSearchResults = [
-  mockNodeMetadata["tool.node1"],
-  mockNodeMetadata["tool.node2"]
-];
-
 const renderComponent = (props: any) => {
   return render(<NodeToolsSelector {...props} />);
 };
 
 describe("NodeToolsSelector", () => {
-  const mockOnChange = jest.fn();
-  const mockSetSearchTerm = jest.fn();
-  const mockSetHoveredNode = jest.fn();
-
   const baseProps = {
     value: [],
     onChange: mockOnChange
@@ -214,30 +234,21 @@ describe("NodeToolsSelector", () => {
     metadata: mockNodeMetadata
   };
 
-  const mockNodeMenuStore = {
-    searchTerm: "",
-    setSearchTerm: mockSetSearchTerm,
-    searchResults: mockSearchResults,
-    isLoading: false,
-    selectedPath: [],
-    hoveredNode: null,
-    setHoveredNode: mockSetHoveredNode,
-    // Add missing properties that might be used
-    availableNodes: mockSearchResults,
-    filteredNodes: mockSearchResults
-  };
-
   beforeEach(() => {
     jest.clearAllMocks();
 
     // Initialize real store state
     useMetadataStore.setState({ metadata: mockNodeMetadata });
 
-    (useStoreWithEqualityFn as jest.Mock).mockImplementation(
-      (_store, selector) => {
-        return selector ? selector(mockNodeMenuStore) : mockNodeMenuStore;
-      }
-    );
+    __setNodeToolsMenuStore({
+      searchTerm: "",
+      setSearchTerm: (term: string) => mockSetSearchTerm(term),
+      searchResults: mockSearchResults,
+      isLoading: false,
+      selectedPath: [],
+      hoveredNode: null,
+      setHoveredNode: (node: any) => mockSetHoveredNode(node)
+    });
   });
 
   describe("Basic Rendering", () => {