fix: mcp app not found error

iamjr15 · iamjr15 · commit 3e8d52f1f54f · 2025-08-31T01:47:55.000+05:30
diff --git a/backend/pipedream/api.py b/backend/pipedream/api.py
@@ -354,34 +354,10 @@ async def get_available_pipedream_tools(
 
 @router.get("/apps", response_model=Dict[str, Any])
 async def get_pipedream_apps(
-    page: int = Query(1, ge=1),
     search: Optional[str] = Query(None),
     category: Optional[str] = Query(None)
 ):
-    logger.info(f"Fetching Pipedream apps registry, page: {page}")
-    
-    # Curated list of featured apps (shown first) - verified through discovery script
-    # Ordered by popularity (featured_weight) and category diversity
-    FEATURED_APPS = [
-        # Top productivity & collaboration (1M+ weight)
-        "notion", "google_sheets", "google_drive", "google_calendar", 
-        "supabase", "slack", "microsoft_teams",
-        
-        # Development & databases (100K+ weight)  
-        "github", "aws", "stripe", "salesforce_rest_api", "hubspot",
-        "woocommerce", "mongodb", "mysql", "postgresql",
-        
-        # Communication & marketing (10K+ weight)
-        "gmail", "telegram_bot_api", "sendgrid", "klaviyo", "zendesk",
-        "zoom", "twilio", "discord", "mailchimp",
-        
-        # Forms, productivity & file storage
-        "airtable_oauth", "typeform", "google_forms", "dropbox", 
-        "trello", "asana", "jira", "todoist", "clickup",
-        
-        # E-commerce & design
-        "shopify_developer_app", "figma", "linkedin", "google_analytics"
-    ]
+    logger.info(f"Fetching Pipedream apps registry")
     
     try:
         from pipedream.client import get_pipedream_client
@@ -400,10 +376,6 @@ async def get_pipedream_apps(
         params = {}
         if search:
             params["q"] = search  # Pipedream API uses 'q' for search
-        if page > 1:
-            # Pipedream API uses cursor-based pagination, not page numbers
-            # For now, we'll just return the first page
-            logger.warning(f"Page {page} requested, but Pipedream API uses cursor-based pagination. Returning first page.")
         
         session = await client._get_session()
         response = await session.get(url, headers=headers, params=params)
@@ -412,38 +384,11 @@ async def get_pipedream_apps(
         data = response.json()
         apps = data.get("data", [])
         
-        # Apply curation logic (only if no search query to preserve search results)
-        if not search:
-            # Separate featured and non-featured apps
-            featured_apps = []
-            other_apps = []
-            featured_slugs = set(FEATURED_APPS)
-            
-            for app in apps:
-                app_slug = app.get("name_slug", "").lower()
-                if app_slug in featured_slugs:
-                    featured_apps.append(app)
-                else:
-                    other_apps.append(app)
-            
-            # Sort featured apps by the order in FEATURED_APPS list
-            featured_apps.sort(key=lambda app: FEATURED_APPS.index(app.get("name_slug", "").lower()) 
-                             if app.get("name_slug", "").lower() in FEATURED_APPS else len(FEATURED_APPS))
-            
-            # Combine: featured first, then others
-            curated_apps = featured_apps + other_apps
-            
-            logger.info(f"Applied curation: {len(featured_apps)} featured apps, {len(other_apps)} other apps")
-        else:
-            curated_apps = apps
-            logger.info(f"Search query provided, skipping curation")
-        
-        logger.info(f"Successfully fetched {len(curated_apps)} apps from Pipedream registry")
+        logger.info(f"Successfully fetched {len(apps)} apps from Pipedream registry")
         return {
             "success": True,
-            "apps": curated_apps,
-            "page_info": data.get("page_info", {}),
-            "total_count": data.get("page_info", {}).get("total_count", 0)
+            "apps": apps,
+            "total_count": len(apps)
         }
         
     except Exception as e:
diff --git a/frontend/src/components/integrations/pipedream/pipedream-registry.tsx b/frontend/src/components/integrations/pipedream/pipedream-registry.tsx
@@ -2,7 +2,7 @@ import React, { useState, useMemo } from 'react';
 import { Button } from '@/components/ui/button';
 import { Input } from '@/components/ui/input';
 import { Card, CardContent } from '@/components/ui/card';
-import { Badge } from '@/components/ui/badge';
+
 import { Search, Loader2, ExternalLink, Zap, User, CheckCircle2, Plus } from 'lucide-react';
 import { Dialog, DialogContent, DialogHeader, DialogTitle, DialogDescription } from '@/components/ui/dialog';
 import { usePipedreamApps } from '@/hooks/react-query/pipedream/use-pipedream';
@@ -28,15 +28,15 @@ export const PipedreamRegistry: React.FC<PipedreamRegistryProps> = ({
 }) => {
   const [search, setSearch] = useState('');
   const [selectedCategory] = useState<string>(''); // Category filtering removed - always show all apps
-  const [page, setPage] = useState(1);
+
   // Removed viewMode - using consistent grid layout
   const [showToolSelector, setShowToolSelector] = useState(false);
   const [selectedProfile, setSelectedProfile] = useState<PipedreamProfile | null>(null);
   const [showProfileManager, setShowProfileManager] = useState(false);
   const [selectedAppForProfile, setSelectedAppForProfile] = useState<{ app_slug: string; app_name: string } | null>(null);
 
   const queryClient = useQueryClient();
-  const { data: appsData, isLoading, error, refetch } = usePipedreamApps(page, search, selectedCategory);
+  const { data: appsData, isLoading, error, refetch } = usePipedreamApps(search, selectedCategory);
   const { data: profiles } = usePipedreamProfiles();
   
   // Removed allAppsData query - no longer needed without category filtering
@@ -45,7 +45,6 @@ export const PipedreamRegistry: React.FC<PipedreamRegistryProps> = ({
 
   const handleSearch = (value: string) => {
     setSearch(value);
-    setPage(1);
   };
 
   const handleSearchSubmit = (e: React.FormEvent) => {
@@ -189,14 +188,7 @@ export const PipedreamRegistry: React.FC<PipedreamRegistryProps> = ({
               {app.description}
             </p>
 
-            {/* Featured Badge */}
-            {app.featured_weight > 100000 && (
-              <div className="mb-4">
-                <Badge variant="default" className="text-xs bg-primary/10 text-primary border-primary/20">
-                  Featured
-                </Badge>
-              </div>
-            )}
+
 
             {/* Connection Status */}
             <div className="mt-auto">
@@ -287,62 +279,13 @@ export const PipedreamRegistry: React.FC<PipedreamRegistryProps> = ({
 
           {!isLoading && appsData?.apps && appsData.apps.length > 0 && (
             <>
-              {/* Featured Apps Section - only show on first page with no search */}
-              {page === 1 && !search && (
-                <>
-                  <div className="mb-8">
-                    <div className="flex items-center gap-2 mb-4">
-                      <h3 className="text-lg font-semibold text-foreground">Featured Apps</h3>
-                      <Badge variant="outline" className="text-xs">Popular</Badge>
-                    </div>
-                    <div className="grid grid-cols-1 sm:grid-cols-2 lg:grid-cols-3 xl:grid-cols-4 2xl:grid-cols-5 gap-4 max-w-full">
-                      {appsData.apps.filter(app => app.featured_weight > 100000).slice(0, 8).map((app: PipedreamApp) => (
-                        <AppCard key={`featured-${app.id}`} app={app} />
-                      ))}
-                    </div>
-                  </div>
-                  
-                  {/* All Apps Section */}
-                  <div className="mb-6">
-                    <h3 className="text-lg font-semibold text-foreground mb-4">All Apps</h3>
-                    <div className="grid grid-cols-1 sm:grid-cols-2 lg:grid-cols-3 xl:grid-cols-4 2xl:grid-cols-5 gap-4 max-w-full">
-                      {appsData.apps.map((app: PipedreamApp) => (
-                        <AppCard key={app.id} app={app} />
-                      ))}
-                    </div>
-                  </div>
-                </>
-              )}
-              
-              {/* Regular view for search results or subsequent pages */}
-              {(page > 1 || search) && (
-                <div className="grid grid-cols-1 sm:grid-cols-2 lg:grid-cols-3 xl:grid-cols-4 2xl:grid-cols-5 gap-4 max-w-full">
-                  {appsData.apps.map((app: PipedreamApp) => (
-                    <AppCard key={app.id} app={app} />
-                  ))}
-                </div>
-              )}
+              <div className="grid grid-cols-1 sm:grid-cols-2 lg:grid-cols-3 xl:grid-cols-4 2xl:grid-cols-5 gap-4 max-w-full">
+                {appsData.apps.map((app: PipedreamApp) => (
+                  <AppCard key={app.id} app={app} />
+                ))}
+              </div>
+
 
-              {appsData.page_info && appsData.page_info.end_cursor && (
-                <div className="flex justify-center pt-8">
-                  <Button
-                    onClick={() => setPage(page + 1)}
-                    disabled={isLoading}
-                    variant="default"
-                    size="lg"
-                    className="px-8 py-2 bg-primary hover:bg-primary/90 text-primary-foreground"
-                  >
-                    {isLoading ? (
-                      <>
-                        <Loader2 className="h-4 w-4 animate-spin mr-2" />
-                        Loading more apps...
-                      </>
-                    ) : (
-                      'Load More Apps'
-                    )}
-                  </Button>
-                </div>
-              )}
             </>
           )}
 
@@ -357,7 +300,6 @@ export const PipedreamRegistry: React.FC<PipedreamRegistryProps> = ({
                 <Button
                   onClick={() => {
                     setSearch('');
-                    setPage(1);
                   }}
                   variant="default"
                   className="px-6"
diff --git a/frontend/src/hooks/react-query/pipedream/keys.ts b/frontend/src/hooks/react-query/pipedream/keys.ts
@@ -6,8 +6,8 @@ export const pipedreamKeys = {
   config: () => [...pipedreamKeys.all, 'config'] as const,
   workflows: () => [...pipedreamKeys.all, 'workflows'] as const,
   workflowRuns: (workflowId: string) => [...pipedreamKeys.all, 'workflow-runs', workflowId] as const,
-  apps: (page: number, search?: string, category?: string) => [...pipedreamKeys.all, 'apps', page, search || '', category || ''] as const,
-  appsSearch: (query: string, page: number, category?: string) => [...pipedreamKeys.all, 'apps', 'search', query, page, category || ''] as const,
+  apps: (search?: string, category?: string) => [...pipedreamKeys.all, 'apps', search || '', category || ''] as const,
+  appsSearch: (query: string, category?: string) => [...pipedreamKeys.all, 'apps', 'search', query, category || ''] as const,
   availableTools: () => [...pipedreamKeys.all, 'available-tools'] as const,
   mcpDiscovery: (options?: { app_slug?: string; oauth_app_id?: string; custom?: boolean }) => 
     [...pipedreamKeys.all, 'mcp-discovery', options?.app_slug, options?.oauth_app_id, options?.custom] as const,
diff --git a/frontend/src/hooks/react-query/pipedream/use-pipedream.ts b/frontend/src/hooks/react-query/pipedream/use-pipedream.ts
@@ -62,13 +62,13 @@ export const usePipedreamHealthCheck = () => {
   });
 };
 
-export const usePipedreamApps = (page = 1, search?: string, category?: string) => {
+export const usePipedreamApps = (search?: string, category?: string) => {
   const pipedreamApi = usePipedreamApi();
   const { disableWindowFocus, disableMount, disableReconnect, disableInterval } = useRefetchControl();
   
   return useQuery({
-    queryKey: pipedreamKeys.apps(page, search, category),
-    queryFn: () => pipedreamApi.getApps(page, search, category),
+    queryKey: pipedreamKeys.apps(search, category),
+    queryFn: () => pipedreamApi.getApps(search, category),
     staleTime: 10 * 60 * 1000, // 10 minutes
     refetchOnWindowFocus: !disableWindowFocus,
     refetchOnMount: !disableMount,
diff --git a/frontend/src/hooks/react-query/pipedream/utils.ts b/frontend/src/hooks/react-query/pipedream/utils.ts
@@ -106,7 +106,6 @@ export interface PipedreamApp {
   name_slug: string;
   description: string;
   categories: string[];
-  featured_weight: number;
   auth_type: string;
   img_src?: string; // Official Pipedream app icon URL
   custom_fields_json: string;
@@ -120,12 +119,6 @@ export interface PipedreamApp {
 export interface PipedreamAppResponse {
   success: boolean;
   apps: PipedreamApp[];
-  page_info: {
-    total_count: number;
-    count: number;
-    start_cursor?: string;
-    end_cursor?: string;
-  };
   total_count: number;
 }
 
@@ -321,11 +314,10 @@ export const usePipedreamApi = () => {
 
     // App Discovery
     async getApps(
-      page = 1,
       search?: string,
       category?: string
     ): Promise<PipedreamAppResponse> {
-      const queryParams = new URLSearchParams({ page: page.toString() });
+      const queryParams = new URLSearchParams();
       if (search) queryParams.append('search', search);
       if (category) queryParams.append('category', category);
 
@@ -502,7 +494,6 @@ export const pipedreamApi = {
 
   // App Discovery
   async getApps(
-    page = 1,
     search?: string,
     category?: string
   ): Promise<PipedreamAppResponse> {
