refactor(web): remove unused getCachedAgentsLite and buildAgentsDataLite

brandonkachen · brandonkachen · commit 25c5a3b63a1b · 2026-01-21T17:23:34.000-08:00
These functions were defined but never used anywhere in the codebase.
Removing them as dead code cleanup to reduce maintenance burden.
diff --git a/web/src/server/__tests__/agents-transform.test.ts b/web/src/server/__tests__/agents-transform.test.ts
@@ -1,9 +1,7 @@
 import { describe, it, expect } from '@jest/globals'
 import {
   buildAgentsData,
-  buildAgentsDataLite,
   type AgentRow,
-  type AgentRowSlim,
 } from '../agents-transform'
 
 describe('buildAgentsData', () => {
@@ -261,158 +259,3 @@ describe('buildAgentsData', () => {
   })
 })
 
-describe('buildAgentsDataLite', () => {
-  it('dedupes by latest, merges metrics, and omits version_stats', () => {
-    // AgentRowSlim has pre-extracted fields (name, description, tags) instead of data blob
-    const agents: AgentRowSlim[] = [
-      {
-        id: 'base',
-        version: '1.0.0',
-        name: 'Base',
-        description: 'desc',
-        tags: ['x'],
-        created_at: '2025-01-01T00:00:00.000Z',
-        publisher: {
-          id: 'codebuff',
-          name: 'Codebuff',
-          verified: true,
-          avatar_url: null,
-        },
-      },
-      // older duplicate by name should be ignored due to first-seen is latest ordering
-      {
-        id: 'base-old',
-        version: '0.9.0',
-        name: 'Base',
-        description: 'old',
-        tags: null,
-        created_at: '2024-12-01T00:00:00.000Z',
-        publisher: {
-          id: 'codebuff',
-          name: 'Codebuff',
-          verified: true,
-          avatar_url: null,
-        },
-      },
-      {
-        id: 'reviewer',
-        version: '2.1.0',
-        name: 'Reviewer',
-        description: null,
-        tags: null,
-        created_at: '2025-01-03T00:00:00.000Z',
-        publisher: {
-          id: 'codebuff',
-          name: 'Codebuff',
-          verified: true,
-          avatar_url: null,
-        },
-      },
-    ]
-
-    const usageMetrics = [
-      {
-        publisher_id: 'codebuff',
-        agent_name: 'Base',
-        total_invocations: 50,
-        total_dollars: 100,
-        avg_cost_per_run: 2,
-        unique_users: 4,
-        last_used: new Date('2025-01-05T00:00:00.000Z'),
-      },
-      {
-        publisher_id: 'codebuff',
-        agent_name: 'reviewer',
-        total_invocations: 5,
-        total_dollars: 5,
-        avg_cost_per_run: 1,
-        unique_users: 1,
-        last_used: new Date('2025-01-04T00:00:00.000Z'),
-      },
-    ]
-
-    const weeklyMetrics = [
-      {
-        publisher_id: 'codebuff',
-        agent_name: 'Base',
-        weekly_runs: 10,
-        weekly_dollars: 20,
-      },
-      {
-        publisher_id: 'codebuff',
-        agent_name: 'reviewer',
-        weekly_runs: 2,
-        weekly_dollars: 1,
-      },
-    ]
-
-    const out = buildAgentsDataLite({
-      agents,
-      usageMetrics: usageMetrics as any,
-      weeklyMetrics: weeklyMetrics as any,
-    })
-
-    // should have deduped to two agents
-    expect(out.length).toBe(2)
-
-    const base = out.find((a) => a.id === 'base')!
-    expect(base.name).toBe('Base')
-    expect(base.weekly_spent).toBe(20)
-    expect(base.weekly_runs).toBe(10)
-    expect(base.total_spent).toBe(100)
-    expect(base.usage_count).toBe(50)
-    expect(base.avg_cost_per_invocation).toBe(2)
-    expect(base.unique_users).toBe(4)
-    expect(base.version_stats).toBeUndefined()
-    expect(Object.prototype.hasOwnProperty.call(base, 'version_stats')).toBe(
-      false,
-    )
-
-    // sorted by weekly_spent desc
-    expect(out[0].weekly_spent! >= out[1].weekly_spent!).toBe(true)
-  })
-
-  it('handles missing metrics gracefully and omits version_stats', () => {
-    // AgentRowSlim with null name (should fall back to id)
-    const agents: AgentRowSlim[] = [
-      {
-        id: 'solo',
-        version: '0.1.0',
-        name: null,
-        description: 'no name provided',
-        tags: null,
-        created_at: new Date('2025-02-01T00:00:00.000Z'),
-        publisher: {
-          id: 'codebuff',
-          name: 'Codebuff',
-          verified: true,
-          avatar_url: null,
-        },
-      },
-    ]
-
-    const out = buildAgentsDataLite({
-      agents,
-      usageMetrics: [],
-      weeklyMetrics: [],
-    })
-
-    expect(out).toHaveLength(1)
-    const a = out[0]
-    // falls back to id when name missing
-    expect(a.name).toBe('solo')
-    // defaults present
-    expect(a.weekly_spent).toBe(0)
-    expect(a.weekly_runs).toBe(0)
-    expect(a.total_spent).toBe(0)
-    expect(a.usage_count).toBe(0)
-    expect(a.avg_cost_per_invocation).toBe(0)
-    expect(a.unique_users).toBe(0)
-    expect(a.last_used).toBeUndefined()
-    expect(a.version_stats).toBeUndefined()
-    expect(Object.prototype.hasOwnProperty.call(a, 'version_stats')).toBe(false)
-    expect(a.tags).toEqual([])
-    // created_at normalized to string
-    expect(typeof a.created_at).toBe('string')
-  })
-})
diff --git a/web/src/server/agents-data.ts b/web/src/server/agents-data.ts
@@ -4,7 +4,6 @@ import { unstable_cache } from 'next/cache'
 import { sql, eq, and, gte } from 'drizzle-orm'
 import {
   buildAgentsData,
-  buildAgentsDataLite,
   buildAgentsDataForSitemap,
   buildAgentsBasicInfo,
   buildAgentsMetricsMap,
@@ -164,87 +163,6 @@ export const fetchAgentsWithMetrics = async (): Promise<AgentData[]> => {
   })
 }
 
-export const fetchAgentsWithMetricsLite = async (): Promise<AgentData[]> => {
-  const oneWeekAgo = new Date(Date.now() - 7 * 24 * 60 * 60 * 1000)
-
-  // Only extract the specific fields we need from the data JSON blob
-  // This avoids fetching the entire agent config (prompts, tools, etc.)
-  const agentsPromise = db
-    .select({
-      id: schema.agentConfig.id,
-      version: schema.agentConfig.version,
-      // Extract only needed fields from data JSON instead of entire blob
-      name: sql<string>`${schema.agentConfig.data}->>'name'`,
-      description: sql<string>`${schema.agentConfig.data}->>'description'`,
-      tags: sql<string[] | null>`${schema.agentConfig.data}->'tags'`,
-      created_at: schema.agentConfig.created_at,
-      publisher: {
-        id: schema.publisher.id,
-        name: schema.publisher.name,
-        verified: schema.publisher.verified,
-        avatar_url: schema.publisher.avatar_url,
-      },
-    })
-    .from(schema.agentConfig)
-    .innerJoin(
-      schema.publisher,
-      eq(schema.agentConfig.publisher_id, schema.publisher.id),
-    )
-    .orderBy(sql`${schema.agentConfig.created_at} DESC`)
-
-  const usageMetricsPromise = db
-    .select({
-      publisher_id: schema.agentRun.publisher_id,
-      agent_name: schema.agentRun.agent_name,
-      total_invocations: sql<number>`COUNT(*)`,
-      total_dollars: sql<number>`COALESCE(SUM(${schema.agentRun.total_credits}) / 100.0, 0)`,
-      avg_cost_per_run: sql<number>`COALESCE(AVG(${schema.agentRun.total_credits}) / 100.0, 0)`,
-      unique_users: sql<number>`COUNT(DISTINCT ${schema.agentRun.user_id})`,
-      last_used: sql<Date>`MAX(${schema.agentRun.created_at})`,
-    })
-    .from(schema.agentRun)
-    .where(
-      and(
-        eq(schema.agentRun.status, 'completed'),
-        sql`${schema.agentRun.agent_id} != 'test-agent'`,
-        sql`${schema.agentRun.publisher_id} IS NOT NULL`,
-        sql`${schema.agentRun.agent_name} IS NOT NULL`,
-      ),
-    )
-    .groupBy(schema.agentRun.publisher_id, schema.agentRun.agent_name)
-
-  const weeklyMetricsPromise = db
-    .select({
-      publisher_id: schema.agentRun.publisher_id,
-      agent_name: schema.agentRun.agent_name,
-      weekly_runs: sql<number>`COUNT(*)`,
-      weekly_dollars: sql<number>`COALESCE(SUM(${schema.agentRun.total_credits}) / 100.0, 0)`,
-    })
-    .from(schema.agentRun)
-    .where(
-      and(
-        eq(schema.agentRun.status, 'completed'),
-        gte(schema.agentRun.created_at, oneWeekAgo),
-        sql`${schema.agentRun.agent_id} != 'test-agent'`,
-        sql`${schema.agentRun.publisher_id} IS NOT NULL`,
-        sql`${schema.agentRun.agent_name} IS NOT NULL`,
-      ),
-    )
-    .groupBy(schema.agentRun.publisher_id, schema.agentRun.agent_name)
-
-  const [agents, usageMetrics, weeklyMetrics] = await Promise.all([
-    agentsPromise,
-    usageMetricsPromise,
-    weeklyMetricsPromise,
-  ])
-
-  return buildAgentsDataLite({
-    agents,
-    usageMetrics,
-    weeklyMetrics,
-  })
-}
-
 export const getCachedAgents = unstable_cache(
   fetchAgentsWithMetrics,
   ['agents-data'],
@@ -254,15 +172,6 @@ export const getCachedAgents = unstable_cache(
   },
 )
 
-export const getCachedAgentsLite = unstable_cache(
-  fetchAgentsWithMetricsLite,
-  ['agents-data-lite'],
-  {
-    revalidate: 600, // 10 minutes
-    tags: ['agents', 'store'],
-  },
-)
-
 // Minimal data for sitemap - only URL components and dates, no agent data blob
 export interface SitemapAgentData {
   id: string
diff --git a/web/src/server/agents-transform.ts b/web/src/server/agents-transform.ts
