chore: add knip and remove the dead code it reports (#247)

Adds knip (pnpm knip, knip.ts covering src, scripts, perf and e2e) and runs it in the CI frontend job. Removes what it reported: the unused codemirror and @types/dompurify dependencies, exports and test hooks used nowhere (AppLogo, Tooltip, barrel re-exports, duplicate default exports of lazily loaded settings pages), the unused iterateAll helper, and the zh entry for a deleted notification. rustfmt (used by lint-staged) is ignored as a toolchain binary.

Co-authored-by: jaw52 <[email protected]>

src/features/debugger/breakpoints/languages.ts

@@ -21,11 +21,6 @@ function load() {
     .finally(() => (loading = false));
 }
 
-/** For tests: languages that have a debugger (`null` = unknown, allow everything). */
-export function _setDebuggableLanguages(ids: Iterable<string> | null) {
-  debuggable = ids ? new Set(ids) : null;
-}
-
 /** Whether a new breakpoint may be placed in a file of `language`. */
 export function canPlaceBreakpoint(language: string): boolean {
   if (workflowSettings().debugger.breakpointsAnyFile) return true;

src/features/debugger/breakpoints/store.ts

@@ -229,10 +229,3 @@ export function setMuted(muted: boolean) {
   persist();
   for (const path of Object.keys(useBreakpoints.getState().byFile)) scheduleSync(path);
 }
-
-export function allBreakpoints(): Array<{ path: string; bp: BreakpointEntry }> {
-  const { byFile } = useBreakpoints.getState();
-  return Object.keys(byFile)
-    .sort()
-    .flatMap((path) => byFile[path]!.map((bp) => ({ path, bp })));
-}

src/features/debugger/state.ts

@@ -62,12 +62,6 @@ export interface ConsoleLine {
   spans: Span[];
 }
 
-export interface WatchResult {
-  expression: string;
-  result: Variable | null;
-  error: string | null;
-}
-
 interface DebugState {
   sessions: Record<number, DebugSession>;
   /** Session shown in the Debug tool window / title bar. */