ci: check knip as part of automated checks (#1521)

## Description

Added a new Knip job to the CI workflow to detect unused files, exports, and dependencies. This job runs on Ubuntu with Node.js 20.x, installs dependencies with `npm ci`, and executes the `npm run knip` command.

## Type of Change

- [x] Improvement
- [x] Cleanup/Refactor

## Checklist

- [x] I have tested this does not break current pipelines / runs functionality
- [x] I have tested the changes on staging

## Test Instructions

1. Verify that the new Knip job runs successfully in the CI pipeline
2. Check that Knip correctly identifies unused code and dependencies

Author: maxy-shpfy

.github/workflows/ci.yml

@@ -60,3 +60,21 @@ jobs:
 
       - name: Run linting
         run: npm run lint
+
+  knip:
+    name: Knip
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: '20.x'
+          cache: 'npm'
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Run knip
+        run: npm run knip

tests/e2e/helpers.ts

@@ -26,7 +26,7 @@ export function locateFlowViewport(page: Page): Locator {
 /**
  * Locates the React Flow pane (for canvas interactions)
  */
-export function locateFlowPane(page: Page): Locator {
+function locateFlowPane(page: Page): Locator {
   return page.locator(".react-flow__pane");
 }
 
@@ -167,7 +167,7 @@ export async function dragComponentToCanvas(
 /**
  * Locates a task node by its name
  */
-export function locateNodeByName(page: Page, nodeName: string): Locator {
+function locateNodeByName(page: Page, nodeName: string): Locator {
   return page.locator(`[data-testid="rf__node-task_${nodeName}"]`);
 }
 
@@ -178,14 +178,14 @@ export function locateNodeByName(page: Page, nodeName: string): Locator {
 /**
  * Locates the context panel container
  */
-export function locateContextPanelContainer(page: Page): Locator {
+function locateContextPanelContainer(page: Page): Locator {
   return page.locator('[data-testid="context-panel-container"]');
 }
 
 /**
  * Locates a specific context panel by name
  */
-export function locateContextPanel(page: Page, panelName: string): Locator {
+function locateContextPanel(page: Page, panelName: string): Locator {
   const container = locateContextPanelContainer(page);
   return container.locator(`[data-context-panel="${panelName}"]`);
 }