chore: fix all knip unused export warnings in CLI package

- Remove dead code: formatEventRow, formatTraceRow, getSeverityColor,
  getSpanEventBody (replaced by formatDynamicRow)
- Remove unused re-export of ROW_DATA_ALIASES from eventQuery.ts
- Un-export internal-only symbols: INTERNAL_ROW_FIELDS,
  processRowToWhereClause, buildAliasWith, _origDebug, _origWarn
- Un-export internal-only interfaces: ApiClientOptions, MeResponse,
  ConnectionResponse, DashboardTileConfig, DashboardTile,
  DashboardFilter, DashboardResponse

Author: wrn14897

packages/cli/src/api/client.ts

@@ -30,7 +30,7 @@ import { loadSession, saveSession, clearSession } from '@/utils/config';
 // API Client (session management + REST calls)
 // ------------------------------------------------------------------
 
-export interface ApiClientOptions {
+interface ApiClientOptions {
   apiUrl: string;
 }
 
@@ -266,7 +266,7 @@ export class ProxyClickhouseClient extends BaseClickhouseClient {
 // Response types (matching the internal API shapes)
 // ------------------------------------------------------------------
 
-export interface MeResponse {
+interface MeResponse {
   accessKey: string;
   createdAt: string;
   email: string;
@@ -321,7 +321,7 @@ export interface SourceResponse {
   sessionSourceId?: string;
 }
 
-export interface ConnectionResponse {
+interface ConnectionResponse {
   id: string;
   _id: string;
   name: string;
@@ -341,7 +341,7 @@ export interface SavedSearchResponse {
   orderBy?: string;
 }
 
-export interface DashboardTileConfig {
+interface DashboardTileConfig {
   name?: string;
   source?: string;
   type?: string;
@@ -350,7 +350,7 @@ export interface DashboardTileConfig {
   [key: string]: unknown;
 }
 
-export interface DashboardTile {
+interface DashboardTile {
   id: string;
   x: number;
   y: number;
@@ -360,14 +360,14 @@ export interface DashboardTile {
   containerId?: string;
 }
 
-export interface DashboardFilter {
+interface DashboardFilter {
   key: string;
   displayName?: string;
   keyExpression?: string;
   sourceId?: string;
 }
 
-export interface DashboardResponse {
+interface DashboardResponse {
   id: string;
   _id: string;
   name: string;

packages/cli/src/api/eventQuery.ts

@@ -104,9 +104,6 @@ export async function buildEventSearchQuery(
 
 // ---- Full row fetch (SELECT *) -------------------------------------
 
-// Re-export shared modules for consumers
-export { ROW_DATA_ALIASES } from '@/shared/rowDataPanel';
-
 // ---- Trace waterfall query (all spans for a traceId) ----------------
 
 export interface TraceSpansQueryOptions {

packages/cli/src/components/EventViewer/utils.ts

@@ -51,17 +51,6 @@ export function flatten(s: string): string {
     .trim();
 }
 
-export function getSeverityColor(
-  sev: string,
-): 'red' | 'yellow' | 'blue' | 'gray' | undefined {
-  const s = sev.toLowerCase();
-  if (s === 'error' || s === 'fatal' || s === 'critical') return 'red';
-  if (s === 'warn' || s === 'warning') return 'yellow';
-  if (s === 'info') return 'blue';
-  if (s === 'debug' || s === 'trace') return 'gray';
-  return undefined;
-}
-
 /**
  * Format a row generically — just stringify each value in column order.
  * Used when the user has a custom select clause.
@@ -80,84 +69,6 @@ export function formatDynamicRow(
   };
 }
 
-function formatTraceRow(
-  row: EventRow,
-  source: SourceResponse,
-  timestamp: string,
-): FormattedRow {
-  const spanName = source.spanNameExpression
-    ? String(row[source.spanNameExpression] ?? '')
-    : '';
-  const service = source.serviceNameExpression
-    ? String(row[source.serviceNameExpression] ?? '')
-    : '';
-  const durationRaw = source.durationExpression
-    ? String(row[source.durationExpression] ?? '')
-    : '';
-  const statusCode = source.statusCodeExpression
-    ? String(row[source.statusCodeExpression] ?? '')
-    : '';
-  const traceId = source.traceIdExpression
-    ? String(row[source.traceIdExpression] ?? '')
-    : '';
-
-  let durationStr = '';
-  if (durationRaw) {
-    const dur = Number(durationRaw);
-    const precision = source.durationPrecision ?? 3;
-    if (precision === 9) {
-      durationStr = `${(dur / 1_000_000).toFixed(1)}ms`;
-    } else if (precision === 6) {
-      durationStr = `${(dur / 1_000).toFixed(1)}ms`;
-    } else {
-      durationStr = `${dur.toFixed(1)}ms`;
-    }
-  }
-
-  const statusLabel =
-    statusCode === '2' ? 'ERROR' : statusCode === '1' ? 'WARN' : 'OK';
-  const color =
-    statusCode === '2'
-      ? ('red' as const)
-      : statusCode === '1'
-        ? ('yellow' as const)
-        : undefined;
-
-  return {
-    cells: [
-      timestamp,
-      service,
-      spanName,
-      durationStr,
-      statusLabel,
-      traceId.slice(0, 16),
-    ],
-    severityColor: color,
-  };
-}
-
-export function formatEventRow(
-  row: EventRow,
-  source: SourceResponse,
-): FormattedRow {
-  const tsExpr = source.timestampValueExpression ?? 'TimestampTime';
-  const timestamp = String(row[tsExpr] ?? row['Timestamp'] ?? '');
-
-  if (source.kind === 'trace') {
-    return formatTraceRow(row, source, timestamp);
-  }
-
-  const bodyExpr = source.bodyExpression ?? 'Body';
-  const sevExpr = source.severityTextExpression ?? 'SeverityText';
-  const rawBody = String(row[bodyExpr] ?? JSON.stringify(row));
-  const severity = String(row[sevExpr] ?? '');
-
-  return {
-    cells: [timestamp, severity, flatten(rawBody)],
-    severityColor: getSeverityColor(severity),
-  };
-}
-
 export function formatShortDate(d: Date): string {
   return d
     .toISOString()

packages/cli/src/shared/source.ts

@@ -19,10 +19,6 @@ export function getFirstTimestampValueExpression(valueExpression: string) {
   return splitAndTrimWithBracket(valueExpression)[0];
 }
 
-export function getSpanEventBody(eventModel: SourceResponse) {
-  return eventModel.spanNameExpression;
-}
-
 export function getDisplayedTimestampValueExpression(
   eventModel: SourceResponse,
 ) {

packages/cli/src/shared/useRowWhere.ts

@@ -27,7 +27,7 @@ const MAX_STRING_LENGTH = 512;
 export type WithClause = NonNullable<BuilderChartConfig['with']>[number];
 
 // Internal row field names used by the table component for row tracking
-export const INTERNAL_ROW_FIELDS = {
+const INTERNAL_ROW_FIELDS = {
   ID: '__hyperdx_id',
   ALIAS_WITH: '__hyperdx_alias_with',
 } as const;
@@ -43,7 +43,7 @@ type ColumnWithMeta = ColumnMetaType & {
   jsType: JSDataType | null;
 };
 
-export function processRowToWhereClause(
+function processRowToWhereClause(
   row: Record<string, unknown>,
   columnMap: Map<string, ColumnWithMeta>,
 ): string {
@@ -159,7 +159,7 @@ export function buildColumnMap(
 /**
  * Build aliasWith array from aliasMap.
  */
-export function buildAliasWith(
+function buildAliasWith(
   aliasMap: Record<string, string | undefined> | undefined,
 ): WithClause[] {
   return aliasMapToWithClauses(aliasMap) ?? [];

packages/cli/src/utils/silenceLogs.ts

@@ -6,8 +6,8 @@
  * (e.g. auth, sources) that write directly to stderr.
  */
 
-export const _origDebug = console.debug;
-export const _origWarn = console.warn;
+const _origDebug = console.debug;
+const _origWarn = console.warn;
 export const _origError = console.error;
 
 console.debug = () => {};