chore: integrate knip for dead-code detection (#158)

Co-authored-by: opencode <noreply@opencode.ai>

Author: antfubot

.github/workflows/ci.yml

@@ -12,6 +12,8 @@ permissions:
 jobs:
   unit-test:
     uses: sxzz/workflows/.github/workflows/unit-test.yml@main
+    with:
+      lint: pnpm run lint && pnpm run knip
 
   e2e:
     runs-on: ubuntu-latest

AGENTS.md

@@ -24,6 +24,7 @@ pnpm dev          # tsdown --watch
 pnpm test         # pnpm build && vitest (api snapshot guards against stale dist)
 pnpm typecheck    # turbo run typecheck (per-package tsc --noEmit)
 pnpm lint --fix   # ESLint via @antfu/eslint-config
+pnpm knip         # unused files/dependencies/exports across every workspace
 pnpm start        # tsx src/index.ts
 ```
 
@@ -33,6 +34,8 @@ The `pnpm test` script intentionally runs `build` first so `tsnapi` snapshots co
 
 `pnpm typecheck` runs `scripts/verify-typecheck-coverage.ts` first — it fails the command (and CI, since CI just runs `pnpm typecheck`) if any workspace package has a `tsconfig.json` but no `typecheck` script, so a new package can't silently join the same blind spot. A package that genuinely can't typecheck yet needs a documented exception in that script, not a missing script.
 
+`pnpm knip` finds unused files, dependencies, and exports across every workspace (config in `knip.jsonc`). It runs against source directly — no prior build needed. Most workspaces need no configuration; `knip.jsonc` only carries per-workspace overrides for cases knip's defaults can't infer on their own: a package's non-`index.ts` `exports` subpaths (knip's package.json→`dist`→`src` source mapping needs a workspace `tsconfig.json` `outDir`, which conflicts with this repo's cross-workspace `src/*.ts` imports, so multi-entry packages list their `exports`-mapped entry files explicitly instead — keep that list in sync with each `tsdown.config.ts`), config files knip's plugins don't discover in a nested location (a Next.js app rooted below the workspace root, `storybook-solidjs-vite` not matching the Storybook plugin trigger), and dependencies referenced dynamically outside its static import graph (icon collections consumed by UnoCSS at build time, plugin packages loaded via a runtime `import()` string). Prefer fixing the underlying gap or a scoped `ignoreDependencies`/`entry` override over a blanket `ignore`.
+
 ## Conventions
 
 - RPC functions must use `defineRpcFunction`; always namespace IDs `devframes:plugin:<slug>:<fn-name>` (matching the plugin's `@devframes/plugin-<slug>` package name).
@@ -145,7 +148,7 @@ Range allocation:
 ## Before PRs
 
 ```sh
-pnpm lint && pnpm test && pnpm typecheck && pnpm build
+pnpm lint && pnpm knip && pnpm test && pnpm typecheck && pnpm build
 ```
 
 Follow conventional commits (`feat:`, `fix:`, etc.).

design/uno.config.ts

@@ -49,5 +49,3 @@ export const designConfig = defineConfig({
     'z-drawer-content': 'z-[90]',
   },
 })
-
-export default designConfig

examples/files-inspector/tests/_utils.ts

@@ -19,7 +19,7 @@ import { resolve } from 'pathe'
 import devframe from '../src/devframe'
 
 const HERE = fileURLToPath(new URL('.', import.meta.url))
-export const CLIENT_DIST = resolve(HERE, '../dist/client')
+const CLIENT_DIST = resolve(HERE, '../dist/client')
 
 /**
  * Asserts the Preact client has been built. Tests boot a dev server

examples/json-render/package.json

@@ -23,8 +23,5 @@
     "@devframes/json-render-ui": "workspace:*",
     "cac": "catalog:deps",
     "devframe": "workspace:*"
-  },
-  "devDependencies": {
-    "get-port-please": "catalog:deps"
   }
 }

examples/next-devframe-hub/src/client/postcss.config.mjs

@@ -1,4 +1,3 @@
-/** @type {import('postcss-load-config').Config} */
 export default {
   plugins: {
     '@unocss/postcss': {},

examples/next-runtime-snapshot/package.json

@@ -25,7 +25,8 @@
     "devframe": "workspace:*",
     "next": "catalog:frontend",
     "react": "catalog:frontend",
-    "react-dom": "catalog:frontend"
+    "react-dom": "catalog:frontend",
+    "valibot": "catalog:deps"
   },
   "devDependencies": {
     "@types/react": "catalog:types",

examples/next-runtime-snapshot/src/client/app/components/connect.tsx

@@ -9,7 +9,7 @@ import { useMemo } from 'react'
 // bundle stays free of node-only server code.
 const NAMESPACE = 'example:next-runtime-snapshot'
 
-export type SnapshotCtx = DevframeScopedClientContext<typeof NAMESPACE>
+type SnapshotCtx = DevframeScopedClientContext<typeof NAMESPACE>
 
 interface ConnectionState {
   ctx: SnapshotCtx | null

examples/next-runtime-snapshot/src/client/postcss.config.mjs

@@ -1,4 +1,3 @@
-/** @type {import('postcss-load-config').Config} */
 export default {
   plugins: {
     '@unocss/postcss': {},

examples/next-runtime-snapshot/tests/_utils.ts

@@ -1,6 +1,5 @@
 import type { StartedServer } from 'devframe/node'
 import process from 'node:process'
-import { fileURLToPath } from 'node:url'
 import { DEVFRAME_CONNECTION_META_FILENAME } from 'devframe/constants'
 import {
   createH3DevframeHost,
@@ -13,9 +12,6 @@ import { H3 } from 'h3'
 import { resolve } from 'pathe'
 import devframe from '../src/devframe'
 
-const HERE = fileURLToPath(new URL('.', import.meta.url))
-export const CLIENT_DIST = resolve(HERE, '../dist/client')
-
 export interface SnapshotServer extends StartedServer {
   basePath: string
 }