fix: resolve lint errors, remove unused file, and clean up code

- Fix no-base-to-string error in OperatorFilter with explicit type checks
- Disable react-hooks/rules-of-hooks for e2e tests (Playwright fixture use)
- Remove unused chipLabels.ts flagged by knip
- Auto-fix Array<T> to T[] syntax and simplify tailwind class
- Remove eslint-disable comments made redundant by config override
- Add popover level/spacing to DataTableFloatingBar
- Remove unused color prop from InterviewsTable export button

Author: jthrilly

app/dashboard/_components/InterviewsTable/InterviewsTable.tsx

@@ -179,7 +179,6 @@ export const InterviewsTable = ({
                 );
                 setShowExportModal(true);
               }}
-              color="primary"
               icon={<FileUp className="size-4" />}
             >
               Export Selected

components/DataTable/DataTableFloatingBar.tsx

@@ -28,6 +28,8 @@ export function DataTableFloatingBar<TData>({
       {selectedCount > 0 && (
         <MotionSurface
           key="floating-bar"
+          level="popover"
+          spacing="sm"
           initial={{ y: 100, opacity: 0 }}
           animate={{ y: 0, opacity: 1 }}
           exit={{ y: 100, opacity: 0 }}

components/DataTable/filters/OperatorFilter.tsx

@@ -108,7 +108,13 @@ export default function OperatorFilter({
             size="sm"
             options={entityOptions}
             value={selectedEntity}
-            onChange={(val) => setSelectedEntity(String(val ?? ''))}
+            onChange={(val) => {
+              if (typeof val === 'string' || typeof val === 'number') {
+                setSelectedEntity(String(val));
+              } else {
+                setSelectedEntity('');
+              }
+            }}
           />
         )}
         <div className="flex items-center gap-2">

components/DataTable/filters/chipLabels.ts

@@ -243,6 +243,14 @@ export default tseslint.config(
     },
   },
 
+  // E2E tests are not React — Playwright fixture `use()` triggers false positives
+  {
+    files: ['tests/e2e/**/*.ts'],
+    rules: {
+      'react-hooks/rules-of-hooks': 'off',
+    },
+  },
+
   // Declaration files require `interface` for ambient type merging
   {
     files: ['**/*.d.ts'],

eslint.config.mjs

@@ -206,7 +206,7 @@ export const interactiveStateVariants = cva({
       readOnly: cx('focus-within:border-input-contrast/70'),
       invalid: '',
       normal:
-        'has-[input:focus-visible,textarea:focus-visible]:focus-styles outline-[var(--focus-color,currentColor)]',
+        'has-[input:focus-visible,textarea:focus-visible]:focus-styles outline-(--focus-color,currentColor)',
     },
   },
   defaultVariants: {

styles/shared/controlVariants.ts

@@ -94,13 +94,11 @@ export const test = baseTest.extend<
   interview: async ({ page, captureInterview }, use) => {
     const interview = new InterviewFixture(page);
     interview.setCaptureFn(captureInterview);
-    // eslint-disable-next-line react-hooks/rules-of-hooks
     await use(interview);
   },
 
   stage: async ({ page }, use) => {
     const stage = new StageFixture(page);
-    // eslint-disable-next-line react-hooks/rules-of-hooks
     await use(stage);
   },
 });

tests/e2e/fixtures/interview-test.ts

@@ -219,8 +219,8 @@ export class ProtocolFixture {
    * Useful for debugging sync issues.
    */
   async getNetworkState(interviewId: string): Promise<{
-    nodes: Array<{ _uid: string; type: string; attributes: Record<string, unknown> }>;
-    edges: Array<{ _uid: string; type: string; from: string; to: string }>;
+    nodes: { _uid: string; type: string; attributes: Record<string, unknown> }[];
+    edges: { _uid: string; type: string; from: string; to: string }[];
     ego: { _uid: string; attributes: Record<string, unknown> };
     currentStep: number;
   }> {
@@ -234,8 +234,8 @@ export class ProtocolFixture {
     }
 
     const network = interview.network as {
-      nodes: Array<{ _uid: string; type: string; attributes: Record<string, unknown> }>;
-      edges: Array<{ _uid: string; type: string; from: string; to: string }>;
+      nodes: { _uid: string; type: string; attributes: Record<string, unknown> }[];
+      edges: { _uid: string; type: string; from: string; to: string }[];
       ego: { _uid: string; attributes: Record<string, unknown> };
     };