chore: add knip to clean up bloat

.github/workflows/linter.yml

@@ -29,9 +29,15 @@ jobs:
           cache: "yarn"
           cache-dependency-path: "**/yarn.lock"
 
+      - name: Install root dependencies
+        run: yarn install --frozen-lockfile
+
       - name: Install SFU Server dependencies
         run: cd sfu-server && yarn install --frozen-lockfile
 
+      - name: Install E2E dependencies
+        run: cd e2e && yarn install --frozen-lockfile
+
       - uses: pre-commit/action@v3.0.0
 
       - name: Download Semgrep rules

.pre-commit-config.yaml

@@ -35,7 +35,7 @@ repos:
     hooks:
       - id: biome-check
         name: Biome Check (Frontend)
-        entry: bash -c 'cd frontend && npx biome check --write .'
+        entry: bash -lc 'cd frontend && node_modules/.bin/biome check --write --config-path biome.json "${@/#frontend\//}"' --
         language: system
         files: "frontend/.*\\.(js|ts|vue|json|jsx|tsx)$"
         # Ignore any files that might contain jinja / bundles
@@ -52,7 +52,7 @@ repos:
 
       - id: biome-check
         name: Biome Check (SFU Server)
-        entry: bash -c 'cd sfu-server && npx biome check --write .'
+        entry: bash -lc 'cd sfu-server && node_modules/.bin/biome check --write --config-path biome.json "${@/#sfu-server\//}"' --
         language: system
         files: "sfu-server/.*\\.(js|ts|json)$"
         exclude: |
@@ -61,6 +61,47 @@ repos:
               sfu-server/dist/.*
           )$
 
+  - repo: local
+    hooks:
+      - id: knip-e2e
+        name: Knip (E2E)
+        entry: yarn knip:e2e
+        language: system
+        pass_filenames: false
+        files: |
+          (?x)^(
+              e2e/.*|
+              knip\.json|
+              package\.json|
+              yarn\.lock
+          )$
+
+      - id: knip-frontend
+        name: Knip (Frontend)
+        entry: yarn knip:frontend
+        language: system
+        pass_filenames: false
+        files: |
+          (?x)^(
+              frontend/.*|
+              knip\.json|
+              package\.json|
+              yarn\.lock
+          )$
+
+      - id: knip-sfu-server
+        name: Knip (SFU Server)
+        entry: yarn knip:sfu-server
+        language: system
+        pass_filenames: false
+        files: |
+          (?x)^(
+              sfu-server/.*|
+              knip\.json|
+              package\.json|
+              yarn\.lock
+          )$
+
 ci:
   autoupdate_schedule: weekly
   skip: []

e2e/fixtures/test.ts

@@ -193,5 +193,4 @@ test.beforeEach(async ({ hostPage }) => {
 	await clearMeetingCreateRateLimit(hostPage.request);
 });
 
-export { expect, joinFromPreview, waitForMeetingReady };
-export type { Participant };
+export { expect, joinFromPreview };

frontend/biome.json

@@ -7,8 +7,7 @@
 	},
 	"files": {
 		"ignoreUnknown": false,
-		"include": ["**"],
-		"ignore": ["components.d.ts", "auto-imports.d.ts"]
+		"includes": ["**", "!components.d.ts", "!auto-imports.d.ts"]
 	},
 	"formatter": {
 		"enabled": true,
@@ -42,7 +41,7 @@
 	},
 	"overrides": [
 		{
-			"include": ["**/*.vue"],
+			"includes": ["**/*.vue"],
 			"linter": {
 				"rules": {
 					"style": {

frontend/components.d.ts

@@ -78,8 +78,6 @@ declare module 'vue' {
     RouterView: typeof import('vue-router')['RouterView']
     SettingsDialog: typeof import('./src/components/settings/SettingsDialog.vue')['default']
     SettingsLayoutBase: typeof import('./src/components/settings/SettingsLayoutBase.vue')['default']
-    SFUDashboard: typeof import('./src/components/SFUDashboard.vue')['default']
     Spinner: typeof import('./src/components/Spinner.vue')['default']
-    VideoPreview: typeof import('./src/components/VideoPreview.vue')['default']
   }
 }

frontend/package.json

@@ -17,7 +17,6 @@
 		"@mediapipe/selfie_segmentation": "0.1.1671057942",
 		"@timephy/rnnoise-wasm": "^1.0.0",
 		"emoji-mart": "^5.6.0",
-		"feather-icons": "^4.29.2",
 		"frappe-ui": "^0.1.271",
 		"mediasoup-client": "^3.18.8",
 		"reka-ui": "^2.6.0",

frontend/src/assets/Inter/inter.css

@@ -3,15 +3,17 @@
 	font-style: normal;
 	font-weight: 100;
 	font-display: swap;
-	src: url("Inter-Thin.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-Thin.woff2?v=3.12") format("woff2"),
 		url("Inter-Thin.woff?v=3.12") format("woff");
 }
 @font-face {
 	font-family: "Inter";
 	font-style: italic;
 	font-weight: 100;
 	font-display: swap;
-	src: url("Inter-ThinItalic.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-ThinItalic.woff2?v=3.12") format("woff2"),
 		url("Inter-ThinItalic.woff?v=3.12") format("woff");
 }
 
@@ -20,15 +22,17 @@
 	font-style: normal;
 	font-weight: 200;
 	font-display: swap;
-	src: url("Inter-ExtraLight.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-ExtraLight.woff2?v=3.12") format("woff2"),
 		url("Inter-ExtraLight.woff?v=3.12") format("woff");
 }
 @font-face {
 	font-family: "Inter";
 	font-style: italic;
 	font-weight: 200;
 	font-display: swap;
-	src: url("Inter-ExtraLightItalic.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-ExtraLightItalic.woff2?v=3.12") format("woff2"),
 		url("Inter-ExtraLightItalic.woff?v=3.12") format("woff");
 }
 
@@ -37,15 +41,17 @@
 	font-style: normal;
 	font-weight: 300;
 	font-display: swap;
-	src: url("Inter-Light.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-Light.woff2?v=3.12") format("woff2"),
 		url("Inter-Light.woff?v=3.12") format("woff");
 }
 @font-face {
 	font-family: "Inter";
 	font-style: italic;
 	font-weight: 300;
 	font-display: swap;
-	src: url("Inter-LightItalic.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-LightItalic.woff2?v=3.12") format("woff2"),
 		url("Inter-LightItalic.woff?v=3.12") format("woff");
 }
 
@@ -54,15 +60,17 @@
 	font-style: normal;
 	font-weight: 400;
 	font-display: swap;
-	src: url("Inter-Regular.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-Regular.woff2?v=3.12") format("woff2"),
 		url("Inter-Regular.woff?v=3.12") format("woff");
 }
 @font-face {
 	font-family: "Inter";
 	font-style: italic;
 	font-weight: 400;
 	font-display: swap;
-	src: url("Inter-Italic.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-Italic.woff2?v=3.12") format("woff2"),
 		url("Inter-Italic.woff?v=3.12") format("woff");
 }
 
@@ -71,15 +79,17 @@
 	font-style: normal;
 	font-weight: 500;
 	font-display: swap;
-	src: url("Inter-Medium.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-Medium.woff2?v=3.12") format("woff2"),
 		url("Inter-Medium.woff?v=3.12") format("woff");
 }
 @font-face {
 	font-family: "Inter";
 	font-style: italic;
 	font-weight: 500;
 	font-display: swap;
-	src: url("Inter-MediumItalic.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-MediumItalic.woff2?v=3.12") format("woff2"),
 		url("Inter-MediumItalic.woff?v=3.12") format("woff");
 }
 
@@ -88,15 +98,17 @@
 	font-style: normal;
 	font-weight: 600;
 	font-display: swap;
-	src: url("Inter-SemiBold.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-SemiBold.woff2?v=3.12") format("woff2"),
 		url("Inter-SemiBold.woff?v=3.12") format("woff");
 }
 @font-face {
 	font-family: "Inter";
 	font-style: italic;
 	font-weight: 600;
 	font-display: swap;
-	src: url("Inter-SemiBoldItalic.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-SemiBoldItalic.woff2?v=3.12") format("woff2"),
 		url("Inter-SemiBoldItalic.woff?v=3.12") format("woff");
 }
 
@@ -105,15 +117,17 @@
 	font-style: normal;
 	font-weight: 700;
 	font-display: swap;
-	src: url("Inter-Bold.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-Bold.woff2?v=3.12") format("woff2"),
 		url("Inter-Bold.woff?v=3.12") format("woff");
 }
 @font-face {
 	font-family: "Inter";
 	font-style: italic;
 	font-weight: 700;
 	font-display: swap;
-	src: url("Inter-BoldItalic.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-BoldItalic.woff2?v=3.12") format("woff2"),
 		url("Inter-BoldItalic.woff?v=3.12") format("woff");
 }
 
@@ -122,15 +136,17 @@
 	font-style: normal;
 	font-weight: 800;
 	font-display: swap;
-	src: url("Inter-ExtraBold.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-ExtraBold.woff2?v=3.12") format("woff2"),
 		url("Inter-ExtraBold.woff?v=3.12") format("woff");
 }
 @font-face {
 	font-family: "Inter";
 	font-style: italic;
 	font-weight: 800;
 	font-display: swap;
-	src: url("Inter-ExtraBoldItalic.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-ExtraBoldItalic.woff2?v=3.12") format("woff2"),
 		url("Inter-ExtraBoldItalic.woff?v=3.12") format("woff");
 }
 
@@ -139,14 +155,16 @@
 	font-style: normal;
 	font-weight: 900;
 	font-display: swap;
-	src: url("Inter-Black.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-Black.woff2?v=3.12") format("woff2"),
 		url("Inter-Black.woff?v=3.12") format("woff");
 }
 @font-face {
 	font-family: "Inter";
 	font-style: italic;
 	font-weight: 900;
 	font-display: swap;
-	src: url("Inter-BlackItalic.woff2?v=3.12") format("woff2"),
+	src:
+		url("Inter-BlackItalic.woff2?v=3.12") format("woff2"),
 		url("Inter-BlackItalic.woff?v=3.12") format("woff");
 }

frontend/src/components/ChatPanel.vue

@@ -68,7 +68,7 @@
 
 <script setup>
 import data from "@emoji-mart/data";
-import { SearchIndex, init } from "emoji-mart";
+import { init, SearchIndex } from "emoji-mart";
 import { Button, FormControl } from "frappe-ui";
 import { computed, nextTick, onMounted, ref, toRefs, watch } from "vue";
 import EmojiPicker from "./EmojiPicker.vue";

frontend/src/components/MeetingPreview.vue

@@ -133,7 +133,7 @@
 </template>
 
 <script setup>
-import { Button, FormControl, createResource, toast } from "frappe-ui";
+import { Button, createResource, FormControl, toast } from "frappe-ui";
 import { computed, inject, nextTick, onMounted, ref, watch } from "vue";
 import ParticipantAvatarGroup from "../components/ParticipantAvatarGroup.vue";
 import PreviewToolbar from "../components/PreviewToolbar.vue";