chore: resolve remaining knip issues (#1991)

## Summary

Resolve all remaining knip issues — removes unused exports/types, adds missing direct dependencies, deletes dead code, and updates knip config.

**Dependency fixes:**
- Root: swapped unused `eslint-config-next`/`eslint-plugin-react-hooks` for actually-imported `@eslint/js`, `typescript-eslint`, `tslib`
- App: added directly-used transitive deps (`@codemirror/*`, `react-resizable`, `postcss-simple-vars`, `rimraf`, `serve`, `@next/eslint-plugin-next`, `eslint-plugin-react`); removed unused `@storybook/react`

**Dead code removal:**
- Removed ~100 unused exports/types across api and app packages (removed `export` keyword where used locally, deleted entirely where not)
- Fixed duplicate `DBRowTableIconButton` default+named export; updated consumers to use named import

**knip.json updates:**
- Added `fixtures.ts` entry point and `opamp/**` ignore for api package
- Excluded `enumMembers` and `duplicates` issue types
- Enabled `ignoreExportsUsedInFile`

### How to test locally or on Vercel

1. `yarn install && yarn knip` — should produce zero output
2. `make ci-lint` — all packages pass
3. `make ci-unit` — all unit tests pass

Author: brandon-pereira

knip.json

@@ -12,9 +12,15 @@
       ]
     },
     "packages/api": {
-      "entry": ["src/index.ts", "src/tasks/index.ts", "scripts/*.ts"],
+      "entry": [
+        "src/index.ts",
+        "src/tasks/index.ts",
+        "scripts/*.ts",
+        "src/fixtures.ts"
+      ],
       "project": ["src/**/*.ts"],
-      "ignoreDependencies": ["pino-pretty", "aws4"]
+      "ignoreDependencies": ["pino-pretty", "aws4"],
+      "ignore": ["src/opamp/**", "src/utils/logParser.ts"]
     },
     "packages/common-utils": {
       "entry": ["src/**/*.ts", "!src/__tests__/**", "!src/**/*.test.*"],
@@ -26,5 +32,6 @@
     "concurrently",
     "dotenv",
     "babel-plugin-react-compiler"
-  ]
+  ],
+  "exclude": ["enumMembers", "duplicates"]
 }

package.json

@@ -9,6 +9,7 @@
   "devDependencies": {
     "@changesets/cli": "^2.26.2",
     "@dotenvx/dotenvx": "^1.51.1",
+    "@eslint/js": "^9.39.1",
     "@types/ungap__structured-clone": "^1.2.0",
     "@ungap/structured-clone": "^1.3.0",
     "babel-plugin-react-compiler": "^1.0.0",
@@ -17,18 +18,18 @@
     "dotenv-cli": "^8.0.0",
     "dotenv-expand": "^12.0.1",
     "eslint": "^9.39.1",
-    "eslint-config-next": "16.0.7",
     "eslint-config-prettier": "^9.1.0",
     "eslint-plugin-n": "^16.4.0",
     "eslint-plugin-prettier": "^5.2.1",
-    "eslint-plugin-react-hooks": "^7.0.1",
     "eslint-plugin-security": "^3.0.1",
     "eslint-plugin-simple-import-sort": "^12.1.1",
     "husky": "^8.0.3",
     "knip": "^6.0.1",
     "lint-staged": "^13.1.2",
     "nx": "21.3.11",
-    "prettier": "3.3.3"
+    "prettier": "3.3.3",
+    "tslib": "^2.6.0",
+    "typescript-eslint": "^8.46.0"
   },
   "scripts": {
     "setup": "yarn install && husky install",

packages/api/src/controllers/ai.ts

@@ -190,7 +190,7 @@ function normalizeParsedDate(parsed?: chrono.ParsedComponents): Date | null {
   return parsedDate;
 }
 
-export function parseTimeRangeInput(
+function parseTimeRangeInput(
   str: string,
   isUTC: boolean = false,
 ): [Date | null, Date | null] {
@@ -216,70 +216,14 @@ export function parseTimeRangeInput(
   }
 }
 
-export const LIVE_TAIL_TIME_QUERY = 'Live Tail';
-
-export const RELATIVE_TIME_OPTIONS: ([string, string] | 'divider')[] = [
-  // ['Last 15 seconds', '15s'],
-  // ['Last 30 seconds', '30s'],
-  // 'divider',
-  ['Last 1 minute', '1m'],
-  ['Last 5 minutes', '5m'],
-  ['Last 15 minutes', '15m'],
-  ['Last 30 minutes', '30m'],
-  ['Last 45 minutes', '45m'],
-  'divider',
-  ['Last 1 hour', '1h'],
-  ['Last 3 hours', '3h'],
-  ['Last 6 hours', '6h'],
-  ['Last 12 hours', '12h'],
-  'divider',
-  ['Last 1 days', '1d'],
-  ['Last 2 days', '2d'],
-  ['Last 7 days', '7d'],
-  ['Last 14 days', '14d'],
-  ['Last 30 days', '30d'],
-];
-
-export const DURATION_OPTIONS = [
-  '30s',
-  '1m',
-  '5m',
-  '15m',
-  '30m',
-  '1h',
-  '3h',
-  '6h',
-  '12h',
-];
-
-export const DURATIONS: Record<string, any> = {
-  '30s': { seconds: 30 },
-  '1m': { minutes: 1 },
-  '5m': { minutes: 5 },
-  '15m': { minutes: 15 },
-  '30m': { minutes: 30 },
-  '1h': { hours: 1 },
-  '3h': { hours: 3 },
-  '6h': { hours: 6 },
-  '12h': { hours: 12 },
-};
-
-export const dateParser = (input?: string) => {
-  if (!input) {
-    return null;
-  }
-  const parsed = chrono.casual.parse(input)[0];
-  return normalizeParsedDate(parsed?.start);
-};
-
 // TODO: Dedup from DBSearchPageFilters
 function isFieldPrimary(tableMetadata: TableMetadata | undefined, key: string) {
   return tableMetadata?.primary_key?.includes(key);
 }
 
 // TODO: Dedup w/ app/src/utils.ts
 // Date formatting
-export const mergePath = (path: string[], jsonColumns: string[] = []) => {
+const mergePath = (path: string[], jsonColumns: string[] = []) => {
   const [key, ...rest] = path;
   if (rest.length === 0) {
     return key;

packages/api/src/controllers/user.ts

@@ -16,14 +16,6 @@ export function findUserByEmail(email: string) {
   return User.findOne({ email: email.toLowerCase() });
 }
 
-export async function findUserByEmailInTeam(
-  email: string,
-  team: string | ObjectId,
-) {
-  // Case-insensitive email search - lowercase the email since User model stores emails in lowercase
-  return User.findOne({ email: email.toLowerCase(), team });
-}
-
 export function findUsersByTeam(team: string | ObjectId) {
   return User.find({ team }).sort({ createdAt: 1 });
 }

packages/api/src/middleware/cors.ts

@@ -2,6 +2,4 @@ import cors from 'cors';
 
 import { FRONTEND_URL } from '@/config';
 
-export const noCors = cors();
-
 export default cors({ credentials: true, origin: FRONTEND_URL });

packages/api/src/models/presetDashboardFilter.ts

@@ -7,8 +7,7 @@ import mongoose, { Schema } from 'mongoose';
 
 import type { ObjectId } from '.';
 
-export interface IPresetDashboardFilter
-  extends Omit<PresetDashboardFilter, 'source'> {
+interface IPresetDashboardFilter extends Omit<PresetDashboardFilter, 'source'> {
   _id: ObjectId;
   team: ObjectId;
   source: ObjectId;

packages/api/src/models/teamInvite.ts

@@ -1,7 +1,7 @@
 import mongoose, { Schema } from 'mongoose';
 import ms from 'ms';
 
-export interface ITeamInvite {
+interface ITeamInvite {
   createdAt: Date;
   email: string;
   name?: string;

packages/api/src/tasks/checkAlerts/template.ts

@@ -119,7 +119,7 @@ export const formatValueToMatchThreshold = (
   }).format(value);
 };
 
-export const notifyChannel = async ({
+const notifyChannel = async ({
   channel,
   message,
 }: {

packages/api/src/utils/common.ts

@@ -10,7 +10,7 @@ export type Json =
   | Json[]
   | { [key: string]: Json };
 
-export const useTry = <T>(fn: () => T): [null | Error | unknown, null | T] => {
+const useTry = <T>(fn: () => T): [null | Error | unknown, null | T] => {
   let output: null | T = null;
   let error: null | Error | unknown = null;
   try {
@@ -42,9 +42,6 @@ export const truncateString = (str: string, length: number) => {
   return str;
 };
 
-export const sleep = (ms: number) =>
-  new Promise(resolve => setTimeout(resolve, ms));
-
 export const convertMsToGranularityString = (ms: number): Granularity => {
   const granularitySizeSeconds = Math.ceil(ms / 1000);
 

packages/api/src/utils/errors.ts

@@ -50,24 +50,6 @@ export class Api404Error extends BaseError {
   }
 }
 
-export class Api401Error extends BaseError {
-  constructor(name: string) {
-    super(name, StatusCode.UNAUTHORIZED, true, 'Unauthorized');
-  }
-}
-
-export class Api403Error extends BaseError {
-  constructor(name: string) {
-    super(name, StatusCode.FORBIDDEN, true, 'Forbidden');
-  }
-}
-
-export class Api409Error extends BaseError {
-  constructor(name: string) {
-    super(name, StatusCode.CONFLICT, true, 'Conflict');
-  }
-}
-
 export const isOperationalError = (error: Error) => {
   if (error instanceof BaseError) {
     return error.isOperational;