remove some unused functions

Author: charleslien

common/knowledge.md

@@ -10,12 +10,6 @@ This package contains code shared between the `web` (Next.js frontend/backend) a
 - **Constants (`src/constants`)**: Shared constant values
 - **API Keys (`src/api-keys`)**: Encryption/decryption utilities for sensitive data
 
-## API Key Encryption (`src/api-keys/crypto.ts`)
-
-- Provides functions for encrypting/decrypting API keys (`encryptAndStoreApiKey`, `retrieveAndDecryptApiKey`, `clearApiKey`)
-- **Security**: Functions require the 32-byte `API_KEY_ENCRYPTION_SECRET` from environment variables
-- The secret must **never** be stored in the `common` package - calling environments retrieve it from their respective `env` files
-
 ## Database Migrations
 
 - Schema defined in `common/src/db/schema.ts`

common/src/analytics.ts

@@ -1,8 +1,6 @@
 import { env } from '@codebuff/internal'
 import { PostHog } from 'posthog-node'
 
-import { logger } from './util/logger'
-
 import type { AnalyticsEvent } from './constants/analytics-events'
 import type { Logger } from '@codebuff/types/logger'
 
@@ -70,23 +68,3 @@ export function trackEvent({
     logger.error({ error }, 'Failed to track event')
   }
 }
-
-export function logError(
-  error: Error,
-  userId?: string,
-  properties?: Record<string, any>,
-) {
-  if (!client) {
-    logger.warn(
-      { error: error.message, userId },
-      'Analytics client not initialized, skipping error logging',
-    )
-    return
-  }
-
-  try {
-    client.captureException(error, userId ?? 'unknown', properties)
-  } catch (error) {
-    logger.error({ error }, 'Failed to log error')
-  }
-}