From 3d0f3938d0a27a99f5710ee346617df0a22e4807 Mon Sep 17 00:00:00 2001
From: PoolOfDeath20 <53143214+GervinFung@users.noreply.github.com>
Date: Sun, 22 Sep 2024 16:55:00 +0800
Subject: [PATCH] Remove unnecessary packages (#67)

* packages(eslint): remove unnecessary packages

* refactor(*): remove `isFalsy`
---
 apps/web/eslint.config.mjs                    |  2 ++
 apps/web/knip.ts                              |  3 ++
 apps/web/package.json                         |  6 ----
 apps/web/pages/templates/index.tsx            |  5 ++--
 .../src/web/components/templates/query.tsx    |  4 +--
 pnpm-lock.yaml                                | 30 ++-----------------
 6 files changed, 12 insertions(+), 38 deletions(-)

diff --git a/apps/web/eslint.config.mjs b/apps/web/eslint.config.mjs
index 4b51946..039b34a 100644
--- a/apps/web/eslint.config.mjs
+++ b/apps/web/eslint.config.mjs
@@ -1,6 +1,8 @@
 import process from 'process';
 
+// eslint-disable-next-line import/no-extraneous-dependencies
 import { includeIgnoreFile } from '@eslint/compat';
+// eslint-disable-next-line import/no-extraneous-dependencies
 import eslint from '@eslint/js';
 import { node, next } from '@poolofdeath20/eslint-config';
 import tseslint from 'typescript-eslint';
diff --git a/apps/web/knip.ts b/apps/web/knip.ts
index b7f8d23..663e944 100644
--- a/apps/web/knip.ts
+++ b/apps/web/knip.ts
@@ -16,6 +16,9 @@ const config: KnipConfig = {
 		'prettier',
 		'supabase',
 		'vite-node',
+		'@eslint/compat',
+		'@eslint/js',
+		'typescript-eslint',
 	],
 };
 
diff --git a/apps/web/package.json b/apps/web/package.json
index b9cd28a..c03e732 100644
--- a/apps/web/package.json
+++ b/apps/web/package.json
@@ -44,12 +44,9 @@
 	},
 	"devDependencies": {
 		"@ducanh2912/next-pwa": "^10.2.8",
-		"@eslint/compat": "^1.1.1",
-		"@eslint/js": "^9.8.0",
 		"@poolofdeath20/eslint-config": "^0.4.0",
 		"@poolofdeath20/tsconfig": "^0.1.1",
 		"@types/cors": "^2.8.17",
-		"@types/eslint__js": "^8.42.3",
 		"@types/file-saver": "^2.0.7",
 		"@types/jest-image-snapshot": "^6.4.0",
 		"@types/react": "^18.3.3",
@@ -59,8 +56,6 @@
 		"@typescript-eslint/parser": "^7.18.0",
 		"ci-info": "^4.0.0",
 		"eslint": "^9.9.1",
-		"eslint-plugin-jsx-a11y": "^6.9.0",
-		"eslint-plugin-react": "^7.35.0",
 		"gen-env-type-def": "^0.0.4",
 		"jest-image-snapshot": "^6.4.0",
 		"knip": "^5.27.0",
@@ -70,7 +65,6 @@
 		"sql-formatter": "15.3.2",
 		"supabase": "^1.187.10",
 		"typescript": "^5.5.4",
-		"typescript-eslint": "^8.0.1",
 		"vite-node": "^2.0.4",
 		"vitest": "^2.0.4"
 	}
diff --git a/apps/web/pages/templates/index.tsx b/apps/web/pages/templates/index.tsx
index 41eb6c0..ff25e8e 100644
--- a/apps/web/pages/templates/index.tsx
+++ b/apps/web/pages/templates/index.tsx
@@ -16,7 +16,6 @@ import {
 	Optional,
 	formQueryParamStringFromRecord,
 	Defined,
-	isFalsy,
 } from '@poolofdeath20/util';
 import { saveAs } from 'file-saver';
 import iwanthue from 'iwanthue';
@@ -420,7 +419,7 @@ const Templates = () => {
 						<ButtonGroup gap={4}>
 							<Button
 								colorScheme="messenger"
-								isDisabled={isFalsy(selected.length)}
+								isDisabled={!selected.length}
 								onClick={() => {
 									copyNotification.unset();
 									clipboard.copy(combineTemplates(selected));
@@ -431,7 +430,7 @@ const Templates = () => {
 							<Divider orientation="vertical" />
 							<Button
 								colorScheme="messenger"
-								isDisabled={isFalsy(selected.length)}
+								isDisabled={!selected.length}
 								onClick={() => {
 									const zip = selected.reduce(
 										(zip, template) => {
diff --git a/apps/web/src/web/components/templates/query.tsx b/apps/web/src/web/components/templates/query.tsx
index 3e7fb6d..5c91cb3 100644
--- a/apps/web/src/web/components/templates/query.tsx
+++ b/apps/web/src/web/components/templates/query.tsx
@@ -3,7 +3,7 @@ import type { DeepReadonly } from '@poolofdeath20/util';
 
 import { Box, Text } from '@chakra-ui/react';
 import styled from '@emotion/styled';
-import { equalTo, isFalsy } from '@poolofdeath20/util';
+import { equalTo } from '@poolofdeath20/util';
 import Fuse from 'fuse.js';
 import React from 'react';
 import Select from 'react-select';
@@ -69,7 +69,7 @@ const QuerySection = (
 							.at(0)?.score ?? 1) <= 0.5
 					);
 				}}
-				isDisabled={isFalsy(props.templates.all.length)}
+				isDisabled={!props.templates.all.length}
 				isMulti
 				loadingMessage={() => {
 					return (
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
index 4c94db0..f9e6ac0 100644
--- a/pnpm-lock.yaml
+++ b/pnpm-lock.yaml
@@ -111,24 +111,15 @@ importers:
       '@ducanh2912/next-pwa':
         specifier: ^10.2.8
         version: 10.2.8(next@14.2.5(@babel/core@7.25.2)(react-dom@18.3.1(react@18.3.1))(react@18.3.1))(webpack@5.93.0)
-      '@eslint/compat':
-        specifier: ^1.1.1
-        version: 1.1.1
-      '@eslint/js':
-        specifier: ^9.8.0
-        version: 9.8.0
       '@poolofdeath20/eslint-config':
         specifier: ^0.4.0
-        version: 0.4.0(@eslint/compat@1.1.1)(@eslint/js@9.8.0)(@next/eslint-plugin-next@14.2.13)(@types/eslint__js@8.42.3)(@typescript-eslint/eslint-plugin@7.18.0(@typescript-eslint/parser@7.18.0(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(@typescript-eslint/parser@7.18.0(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(eslint-plugin-import@2.30.0(@typescript-eslint/parser@7.18.0(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(eslint@9.9.1(jiti@1.21.6)))(eslint-plugin-jsx-a11y@6.9.0(eslint@9.9.1(jiti@1.21.6)))(eslint-plugin-react-hooks@4.6.2(eslint@9.9.1(jiti@1.21.6)))(eslint-plugin-react@7.35.0(eslint@9.9.1(jiti@1.21.6)))(typescript-eslint@8.0.1(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))
+        version: 0.4.0(@eslint/compat@1.1.1)(@eslint/js@9.9.1)(@next/eslint-plugin-next@14.2.13)(@types/eslint__js@8.42.3)(@typescript-eslint/eslint-plugin@7.18.0(@typescript-eslint/parser@7.18.0(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(@typescript-eslint/parser@7.18.0(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(eslint-plugin-import@2.30.0(@typescript-eslint/parser@7.18.0(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(eslint@9.9.1(jiti@1.21.6)))(eslint-plugin-jsx-a11y@6.9.0(eslint@9.9.1(jiti@1.21.6)))(eslint-plugin-react-hooks@4.6.2(eslint@9.9.1(jiti@1.21.6)))(eslint-plugin-react@7.35.0(eslint@9.9.1(jiti@1.21.6)))(typescript-eslint@8.0.1(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))
       '@poolofdeath20/tsconfig':
         specifier: ^0.1.1
         version: 0.1.1
       '@types/cors':
         specifier: ^2.8.17
         version: 2.8.17
-      '@types/eslint__js':
-        specifier: ^8.42.3
-        version: 8.42.3
       '@types/file-saver':
         specifier: ^2.0.7
         version: 2.0.7
@@ -156,12 +147,6 @@ importers:
       eslint:
         specifier: ^9.9.1
         version: 9.9.1(jiti@1.21.6)
-      eslint-plugin-jsx-a11y:
-        specifier: ^6.9.0
-        version: 6.9.0(eslint@9.9.1(jiti@1.21.6))
-      eslint-plugin-react:
-        specifier: ^7.35.0
-        version: 7.35.0(eslint@9.9.1(jiti@1.21.6))
       gen-env-type-def:
         specifier: ^0.0.4
         version: 0.0.4
@@ -189,9 +174,6 @@ importers:
       typescript:
         specifier: ^5.5.4
         version: 5.5.4
-      typescript-eslint:
-        specifier: ^8.0.1
-        version: 8.0.1(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4)
       vite-node:
         specifier: ^2.0.4
         version: 2.0.4(@types/node@22.0.0)(terser@5.31.3)
@@ -1479,10 +1461,6 @@ packages:
     resolution: {integrity: sha512-4Bfj15dVJdoy3RfZmmo86RK1Fwzn6SstsvK9JS+BaVKqC6QQQQyXekNaC+g+LKNgkQ+2VhGAzm6hO40AhMR3zQ==}
     engines: {node: ^18.18.0 || ^20.9.0 || >=21.1.0}
 
-  '@eslint/js@9.8.0':
-    resolution: {integrity: sha512-MfluB7EUfxXtv3i/++oh89uzAr4PDI4nn201hsp+qaXqsjAWzinlZEHEfPgAX4doIlKvPG/i0A9dpKxOLII8yA==}
-    engines: {node: ^18.18.0 || ^20.9.0 || >=21.1.0}
-
   '@eslint/js@9.9.1':
     resolution: {integrity: sha512-xIDQRsfg5hNBqHz04H1R3scSVwmI+KUbqjsQKHKQ1DAUSaUjYPReZZmS/5PNiKu1fUvzDd6H7DEDKACSEhu+TQ==}
     engines: {node: ^18.18.0 || ^20.9.0 || >=21.1.0}
@@ -6595,8 +6573,6 @@ snapshots:
     transitivePeerDependencies:
       - supports-color
 
-  '@eslint/js@9.8.0': {}
-
   '@eslint/js@9.9.1': {}
 
   '@eslint/object-schema@2.1.4': {}
@@ -6718,10 +6694,10 @@ snapshots:
   '@pkgjs/parseargs@0.11.0':
     optional: true
 
-  '@poolofdeath20/eslint-config@0.4.0(@eslint/compat@1.1.1)(@eslint/js@9.8.0)(@next/eslint-plugin-next@14.2.13)(@types/eslint__js@8.42.3)(@typescript-eslint/eslint-plugin@7.18.0(@typescript-eslint/parser@7.18.0(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(@typescript-eslint/parser@7.18.0(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(eslint-plugin-import@2.30.0(@typescript-eslint/parser@7.18.0(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(eslint@9.9.1(jiti@1.21.6)))(eslint-plugin-jsx-a11y@6.9.0(eslint@9.9.1(jiti@1.21.6)))(eslint-plugin-react-hooks@4.6.2(eslint@9.9.1(jiti@1.21.6)))(eslint-plugin-react@7.35.0(eslint@9.9.1(jiti@1.21.6)))(typescript-eslint@8.0.1(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))':
+  '@poolofdeath20/eslint-config@0.4.0(@eslint/compat@1.1.1)(@eslint/js@9.9.1)(@next/eslint-plugin-next@14.2.13)(@types/eslint__js@8.42.3)(@typescript-eslint/eslint-plugin@7.18.0(@typescript-eslint/parser@7.18.0(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(@typescript-eslint/parser@7.18.0(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(eslint-plugin-import@2.30.0(@typescript-eslint/parser@7.18.0(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(eslint@9.9.1(jiti@1.21.6)))(eslint-plugin-jsx-a11y@6.9.0(eslint@9.9.1(jiti@1.21.6)))(eslint-plugin-react-hooks@4.6.2(eslint@9.9.1(jiti@1.21.6)))(eslint-plugin-react@7.35.0(eslint@9.9.1(jiti@1.21.6)))(typescript-eslint@8.0.1(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))':
     dependencies:
       '@eslint/compat': 1.1.1
-      '@eslint/js': 9.8.0
+      '@eslint/js': 9.9.1
       '@next/eslint-plugin-next': 14.2.13
       '@types/eslint__js': 8.42.3
       '@typescript-eslint/eslint-plugin': 7.18.0(@typescript-eslint/parser@7.18.0(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4))(eslint@9.9.1(jiti@1.21.6))(typescript@5.5.4)