chore: add knip and clean up unused packages

Also update lint.yml.

Author: mrkvon

.github/workflows/lint.yml

@@ -19,11 +19,14 @@ jobs:
       - name: Install NPM packages
         run: yarn install --frozen-lockfile
 
-      - name: Run linters
-        uses: wearerequired/lint-action@v2
-        with:
-          eslint: true
-          eslint_args: --max-warnings 0
-          eslint_extensions: js,ts,tsx
-          prettier: true
-          continue_on_error: false
+      - name: Check code style
+        run: yarn lint --max-warnings 0
+
+      - name: Check formatting
+        run: yarn prettier . --check
+
+      - name: Check clutter
+        run: yarn knip
+
+      - name: Check that TypeScript compiles
+        run: yarn tsc

knip.json

@@ -0,0 +1,7 @@
+{
+  "project": ["**/*.{js,ts}"],
+  "entry": ["src/index.ts", "src/generate-api-docs.ts"],
+  "ignore": ["apidocs/*"],
+  "ignoreDependencies": ["@types/co-body"],
+  "ignoreBinaries": ["openssl"]
+}

package.json

@@ -7,35 +7,32 @@
     "start": "yarn build && node dist/index.js",
     "build": "rm -rf dist && tsc --project tsconfig.production.json && yarn copy-hbs",
     "copy-hbs": "cp src/templates/*.hbs dist/templates && cp src/templates/*.css dist/templates",
-    "format": "prettier 'src/**/*.ts' '**/*.{md,yml,yaml,json}' --write",
+    "format": "prettier . --write",
     "lint": "eslint . --ext .ts",
     "test": "NODE_ENV=vitest vitest",
     "generate-key": "openssl ecparam -name prime256v1 -genkey -noout -out ecdsa-p256-private.pem",
-    "generate-api-docs": "tsc && node dist/generate-api-docs.js"
+    "generate-api-docs": "tsc && node dist/generate-api-docs.js",
+    "knip": "knip"
   },
   "license": "MIT",
   "devDependencies": {
     "@solid/community-server": "^7.0.2",
-    "@types/chai": "^4.3.5",
     "@types/maildev": "^0.0.7",
-    "@types/sinon": "^10.0.15",
     "@typescript-eslint/eslint-plugin": "^6.0.0",
     "@typescript-eslint/parser": "^6.19.0",
-    "chai": "^4.3.7",
     "cheerio": "^1.0.0-rc.12",
     "cross-fetch": "^4.0.0",
     "eslint": "^8.13.0",
     "eslint-config-prettier": "^8.5.0",
     "eslint-import-resolver-typescript": "^3.6.1",
     "eslint-plugin-import": "^2.29.1",
     "eslint-plugin-prettier": "^5.0.0",
+    "knip": "^5.43.6",
     "maildev": "^2.1.0",
     "msw": "^2.0.14",
     "prettier": "^3.0.0",
     "puppeteer": "^22.2.0",
-    "sinon": "^15.2.0",
     "swagger-autogen": "^2.23.6",
-    "ts-node": "^10.9.1",
     "vitest": "^3.0.4"
   },
   "dependencies": {

src/templates/layout.hbs

@@ -15,4 +15,4 @@
         <a href='mailto:{{supportEmail}}'>{{supportEmail}}</a></p>
     </footer>
   </body>
-</html>
+</html>

src/templates/message.hbs

@@ -15,4 +15,4 @@
     class='action-button'
   >Reply on
     {{appName}}</a>
-</p>
+</p>

src/templates/verification.hbs

@@ -20,4 +20,4 @@
   {{appName}}
   was initialized by Solid identity
   {{actor}}. If you've received this message unexpectedly and you believe it to
-  be an error, please ignore it, or contact us for support.</p>
+  be an error, please ignore it, or contact us for support.</p>

src/test/helpers/index.ts

@@ -1,7 +1,7 @@
 import * as cheerio from 'cheerio'
 import { createAccount } from 'css-authn/dist/7.x.js'
+import { randomUUID } from 'node:crypto'
 import * as puppeteer from 'puppeteer'
-import { v4 as uuidv4 } from 'uuid'
 import { expect, vi } from 'vitest'
 import * as config from '../../config/index.js'
 import * as mailerService from '../../services/mailerService.js'
@@ -14,9 +14,9 @@ export const createRandomAccount = ({
   solidServer: string
 }) => {
   return createAccount({
-    username: uuidv4(),
-    password: uuidv4(),
-    email: uuidv4() + '@example.com',
+    username: randomUUID(),
+    password: randomUUID(),
+    email: randomUUID() + '@example.com',
     provider: solidServer,
   })
 }