Add team-based channel analysis routes

backend/app/services/slack/channels.py

@@ -502,7 +502,7 @@ async def select_channels_for_analysis(
             if install_bot:
                 api_client = SlackApiClient(workspace.access_token)
 
-            # Unselect all channels if we are setting for_analysis=True 
+            # Unselect all channels if we are setting for_analysis=True
             # This is for backward compatibility with old behavior
             if for_analysis:
                 # First, unselect all channels

backend/tests/services/slack/test_channels.py

@@ -275,7 +275,9 @@ async def test_select_channels_for_analysis_without_bot_install(
     assert "bot_installation" not in result
 
     # Verify the db operations
-    assert mock_db_session.execute.call_count == 5  # Including all_channels query for debugging
+    assert (
+        mock_db_session.execute.call_count == 5
+    )  # Including all_channels query for debugging
     assert mock_db_session.commit.called
 
 
@@ -371,5 +373,7 @@ async def test_select_channels_for_analysis_with_bot_install(
         mock_client.join_channel.assert_called_once_with(channel_without_bot.slack_id)
 
         # Verify the db operations
-        assert mock_db_session.execute.call_count == 5  # Including all_channels query for debugging
+        assert (
+            mock_db_session.execute.call_count == 5
+        )  # Including all_channels query for debugging
         assert mock_db_session.commit.called

frontend/src/App.tsx

@@ -41,6 +41,9 @@ import {
   IntegrationDetailPage,
   IntegrationConnectPage,
   TeamChannelSelectorPage,
+  TeamChannelAnalysisPage,
+  TeamAnalysisResultPage,
+  TeamChannelAnalysisHistoryPage,
 } from './pages/integration'
 
 // Profile Pages
@@ -266,6 +269,36 @@ function App() {
                   </ProtectedRoute>
                 }
               />
+              <Route
+                path="/dashboard/integrations/:integrationId/channels/:channelId/analyze"
+                element={
+                  <ProtectedRoute>
+                    <AppLayout>
+                      <TeamChannelAnalysisPage />
+                    </AppLayout>
+                  </ProtectedRoute>
+                }
+              />
+              <Route
+                path="/dashboard/integrations/:integrationId/channels/:channelId/analysis/:analysisId"
+                element={
+                  <ProtectedRoute>
+                    <AppLayout>
+                      <TeamAnalysisResultPage />
+                    </AppLayout>
+                  </ProtectedRoute>
+                }
+              />
+              <Route
+                path="/dashboard/integrations/:integrationId/channels/:channelId/history"
+                element={
+                  <ProtectedRoute>
+                    <AppLayout>
+                      <TeamChannelAnalysisHistoryPage />
+                    </AppLayout>
+                  </ProtectedRoute>
+                }
+              />
 
               {/* Profile routes */}
               <Route

frontend/src/__tests__/components/integration/TeamChannelSelector.test.tsx

@@ -5,6 +5,12 @@ import TeamChannelSelector from '../../../components/integration/TeamChannelSele
 import IntegrationContext from '../../../context/IntegrationContext'
 import { ResourceType } from '../../../lib/integrationService'
 
+// Mock React Router
+vi.mock('react-router-dom', () => ({
+  useNavigate: () => vi.fn(),
+  Link: ({ children }: { children: React.ReactNode }) => children,
+}))
+
 // Mock the useIntegration hook functionality through context
 const mockIntegrationContext = {
   // State

frontend/src/__tests__/context/IntegrationContext.test.tsx

@@ -37,6 +37,7 @@ vi.mock('../../lib/integrationService', () => ({
     getSelectedChannels: vi.fn(),
     selectChannelsForAnalysis: vi.fn(),
     analyzeChannel: vi.fn(),
+    analyzeResource: vi.fn(),
   },
   IntegrationType: {
     SLACK: 'slack',
@@ -221,7 +222,7 @@ describe('IntegrationContext', () => {
       status: 'success',
       message: 'Channels selected for analysis',
     })
-    vi.mocked(integrationService.analyzeChannel).mockResolvedValue(
+    vi.mocked(integrationService.analyzeResource).mockResolvedValue(
       mockAnalysisResult
     )
 
@@ -706,7 +707,7 @@ describe('IntegrationContext', () => {
       })
 
       // Check that the service was called correctly
-      expect(integrationService.analyzeChannel).toHaveBeenCalledWith(
+      expect(integrationService.analyzeResource).toHaveBeenCalledWith(
         'test-int-1',
         'res-1',
         options

frontend/src/components/integration/TeamChannelSelector.tsx

@@ -20,8 +20,16 @@ import {
   Button,
   Flex,
   useToast,
+  Tooltip,
 } from '@chakra-ui/react'
-import { FiSearch, FiSettings, FiCheck, FiBarChart2 } from 'react-icons/fi'
+import {
+  FiSearch,
+  FiSettings,
+  FiCheck,
+  FiBarChart2,
+  FiClock,
+} from 'react-icons/fi'
+import { useNavigate } from 'react-router-dom'
 import { ResourceType } from '../../lib/integrationService'
 import useIntegration from '../../context/useIntegration'
 
@@ -36,6 +44,7 @@ const TeamChannelSelector: React.FC<TeamChannelSelectorProps> = ({
   integrationId,
 }) => {
   const toast = useToast()
+  const navigate = useNavigate()
   const {
     currentResources,
     loadingResources,
@@ -348,20 +357,43 @@ const TeamChannelSelector: React.FC<TeamChannelSelectorProps> = ({
                 </Td>
                 <Td>
                   <HStack spacing={1}>
-                    <IconButton
-                      aria-label="Analyze channel"
-                      icon={<FiBarChart2 />}
-                      size="sm"
-                      variant="ghost"
-                      title="View analysis"
-                    />
-                    <IconButton
-                      aria-label="Channel settings"
-                      icon={<FiSettings />}
-                      size="sm"
-                      variant="ghost"
-                      title="Settings"
-                    />
+                    <Tooltip label="Analyze channel">
+                      <IconButton
+                        aria-label="Analyze channel"
+                        icon={<FiBarChart2 />}
+                        size="sm"
+                        variant="ghost"
+                        colorScheme="blue"
+                        onClick={() =>
+                          navigate(
+                            `/dashboard/integrations/${integrationId}/channels/${channel.id}/analyze`
+                          )
+                        }
+                      />
+                    </Tooltip>
+                    <Tooltip label="Analysis history">
+                      <IconButton
+                        aria-label="Analysis history"
+                        icon={<FiClock />}
+                        size="sm"
+                        variant="ghost"
+                        colorScheme="teal"
+                        onClick={() =>
+                          navigate(
+                            `/dashboard/integrations/${integrationId}/channels/${channel.id}/history`
+                          )
+                        }
+                      />
+                    </Tooltip>
+                    <Tooltip label="Channel settings">
+                      <IconButton
+                        aria-label="Channel settings"
+                        icon={<FiSettings />}
+                        size="sm"
+                        variant="ghost"
+                        title="Settings"
+                      />
+                    </Tooltip>
                   </HStack>
                 </Td>
               </Tr>

frontend/src/context/IntegrationContext.tsx

@@ -1095,7 +1095,7 @@ export const IntegrationProvider: React.FC<{ children: React.ReactNode }> = ({
       }))
 
       try {
-        const result = await integrationService.analyzeChannel(
+        const result = await integrationService.analyzeResource(
           integrationId,
           channelId,
           options
@@ -1115,7 +1115,12 @@ export const IntegrationProvider: React.FC<{ children: React.ReactNode }> = ({
           loadingChannelSelection: false,
         }))
 
-        return result
+        // Cast the result to the expected return type
+        return {
+          status: 'success',
+          analysis_id:
+            (result as { analysis_id?: string }).analysis_id || 'unknown',
+        }
       } catch (error) {
         setState((prev) => ({
           ...prev,

frontend/src/lib/apiClient.ts

@@ -32,7 +32,7 @@ export class ApiClient {
   protected async get<T>(
     path: string,
     params?: Record<string, string>
-  ): Promise<T> {
+  ): Promise<T | ApiError> {
     const url = new URL(`${this.baseUrl}${path}`)
 
     if (params) {
@@ -43,43 +43,91 @@ export class ApiClient {
       })
     }
 
-    const response = await fetch(url.toString(), {
-      method: 'GET',
-      credentials: 'include',
-      headers: {
-        Accept: 'application/json',
-        'Content-Type': 'application/json',
-      },
-    })
+    try {
+      const response = await fetch(url.toString(), {
+        method: 'GET',
+        credentials: 'include',
+        headers: {
+          Accept: 'application/json',
+          'Content-Type': 'application/json',
+        },
+      })
 
-    if (!response.ok) {
-      throw new Error(`API Error: ${response.status} ${response.statusText}`)
-    }
+      // If the response is not OK, return an ApiError instead of throwing
+      if (!response.ok) {
+        let errorDetail: string
+        try {
+          // Try to parse error details from response
+          const errorJson = await response.json()
+          errorDetail =
+            errorJson.detail || errorJson.message || response.statusText
+        } catch {
+          // If can't parse JSON, use status text
+          errorDetail = response.statusText
+        }
 
-    return response.json()
+        return {
+          status: response.status,
+          message: `API Error: ${response.status} ${response.statusText}`,
+          detail: errorDetail,
+        } as ApiError
+      }
+
+      return response.json()
+    } catch (error) {
+      // Network errors or other fetch exceptions
+      return {
+        status: 'NETWORK_ERROR',
+        message: `Network Error: ${error instanceof Error ? error.message : 'Unknown error'}`,
+      } as ApiError
+    }
   }
 
   // Standard POST request
   protected async post<T>(
     path: string,
     data?: Record<string, unknown>
-  ): Promise<T> {
+  ): Promise<T | ApiError> {
     const url = `${this.baseUrl}${path}`
 
-    const response = await fetch(url, {
-      method: 'POST',
-      credentials: 'include',
-      headers: {
-        Accept: 'application/json',
-        'Content-Type': 'application/json',
-      },
-      body: data ? JSON.stringify(data) : undefined,
-    })
+    try {
+      const response = await fetch(url, {
+        method: 'POST',
+        credentials: 'include',
+        headers: {
+          Accept: 'application/json',
+          'Content-Type': 'application/json',
+        },
+        body: data ? JSON.stringify(data) : undefined,
+      })
 
-    if (!response.ok) {
-      throw new Error(`API Error: ${response.status} ${response.statusText}`)
-    }
+      // If the response is not OK, return an ApiError instead of throwing
+      if (!response.ok) {
+        let errorDetail: string
+        try {
+          // Try to parse error details from response
+          const errorJson = await response.json()
+          errorDetail =
+            errorJson.detail || errorJson.message || response.statusText
+        } catch {
+          // If can't parse JSON, use status text
+          errorDetail = response.statusText
+        }
 
-    return response.json()
+        return {
+          status: response.status,
+          message: `API Error: ${response.status} ${response.statusText}`,
+          detail: errorDetail,
+        } as ApiError
+      }
+
+      return response.json()
+    } catch (error) {
+      // Network errors or other fetch exceptions
+      return {
+        status: 'NETWORK_ERROR',
+        message: `Network Error: ${error instanceof Error ? error.message : 'Unknown error'}`,
+      } as ApiError
+    }
   }
 }