remove unused deps and check for unused dependencies in CI (#1355)

dimaMachina · web-flow · commit e5172e226e69 · 2025-12-22T22:05:51.000+02:00
* keep file structure for build `dist` directory in agents-cli

* remove clean-package from core

* upd

* upd sdk config

* sdk is ok

* update core setup

* update zod in ai-sdk-provider

* manage-api

* run api

* add typescript as external

* improve changeset

* try workaround

* use turbo for `@inkeep/create-agents build` command

* add build in cookbook

* fix ci tests

* better explanation

* fix test:e2e in create agents

* rollback

* try

* upd dev command

* add `@inkeep/create-agents#test:e2e` in turbo

* add scripts update github workflow label

* cleanup dependencies

* more unused dependencies

* more unused dependencies

* more unused dependencies

* add knip in cli

* sdk works

* core

* ai sdk provider

* create agents

* cli

* manage-api and run api

* use tags

* remove deps from dashboard

* remove more deps

* update knip config

* Refactor action steps for backend and dashboard builds

* upd

* lint cookbook

* Refactor action steps for clarity and efficiency

* Update agents-cookbook/package.json

* format and pnpm lock
diff --git a/.changeset/dull-pumas-tap.md b/.changeset/dull-pumas-tap.md
@@ -0,0 +1,12 @@
+---
+"@inkeep/ai-sdk-provider": minor
+"@inkeep/create-agents": minor
+"@inkeep/agents-core": minor
+"@inkeep/agents-sdk": minor
+"@inkeep/agents-manage-api": minor
+"@inkeep/agents-run-api": minor
+"@inkeep/agents-cli": minor
+"@inkeep/agents-manage-ui": minor
+---
+
+remove unused dependencies, lint unused dependencies with Knip
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -92,7 +92,7 @@ jobs:
       - name: Biome Format
         run: pnpm format:check
 
-      - name: Knip Unused Exports/Types Check
+      - name: Check for unused dependencies, exports, and types
         run: pnpm knip
 
       # Create summary report
diff --git a/agents-cli/package.json b/agents-cli/package.json
@@ -13,6 +13,7 @@
     "./package.json": "./package.json"
   },
   "scripts": {
+    "knip": "knip --directory .. --workspace agents-cli --dependencies",
     "build": "tsdown",
     "cli": "node ./dist/index.js",
     "postinstall": "node scripts/ensure-keytar.mjs || true",
diff --git a/agents-cookbook/knip.config.ts b/agents-cookbook/knip.config.ts
@@ -0,0 +1,9 @@
+import type { KnipConfig } from 'knip';
+
+export default {
+  workspaces: {
+    'agents-cookbook': {
+      entry: ['template-projects/**/*'],
+    },
+  },
+} satisfies KnipConfig;
diff --git a/agents-cookbook/package.json b/agents-cookbook/package.json
@@ -6,6 +6,7 @@
   "license": "MIT",
   "type": "module",
   "scripts": {
+    "knip": "knip --directory .. --workspace agents-cookbook --config agents-cookbook/knip.config.ts --dependencies",
     "start:weather": "tsx weather-project/index.ts",
     "dev:weather": "tsx --watch weather-project/index.ts",
     "type-check": "tsc --noEmit",
@@ -19,7 +20,6 @@
   },
   "devDependencies": {
     "@types/node": "^22.18.12",
-    "json-schema-to-ts": "^3.1.1",
     "tsx": "^4.20.6",
     "typescript": "^5.9.3"
   }
diff --git a/agents-docs/src/components/mdx/big-video.tsx b/agents-docs/src/components/mdx/big-video.tsx
@@ -6,18 +6,15 @@ interface BigVideoProps {
   height?: string;
 }
 
-export function BigVideo({ 
-  src, 
-  height = 'auto' 
-}: BigVideoProps) {
+export function BigVideo({ src, height = 'auto' }: BigVideoProps) {
   return (
     <video
       src={src}
       controls
       style={{
         borderRadius: '10px',
         display: 'block',
-        maxWidth: "800px",
+        maxWidth: '800px',
         width: '100%',
         height: height,
         margin: '0 auto',
@@ -26,4 +23,4 @@ export function BigVideo({
       Your browser does not support the video tag.
     </video>
   );
-}
+}
diff --git a/agents-docs/src/mdx-components.tsx b/agents-docs/src/mdx-components.tsx
@@ -27,9 +27,9 @@ import defaultMdxComponents from 'fumadocs-ui/mdx';
 import type { MDXComponents } from 'mdx/types';
 import { ComparisonTable } from '@/components/comparisons-table';
 import { AutoTypeTable } from '@/components/mdx/auto-type-table';
+import { BigVideo } from '@/components/mdx/big-video';
 import { Card } from '@/components/mdx/card'; // Add this line
 import { openapi } from '@/lib/openapi';
-import { BigVideo } from '@/components/mdx/big-video';
 
 // Snippet component for MDX snippets
 // This is a placeholder that should be replaced by remark-mdx-snippets plugin
diff --git a/agents-manage-api/package.json b/agents-manage-api/package.json
@@ -17,6 +17,7 @@
   },
   "type": "module",
   "scripts": {
+    "knip": "knip --directory .. --workspace agents-manage-api --dependencies",
     "dev": "vite",
     "build": "tsdown",
     "start": "node dist/index.js",
@@ -30,20 +31,14 @@
   "dependencies": {
     "@electric-sql/pglite": "^0.3.13",
     "@hono/mcp": "^0.1.5",
-    "@composio/core": "^0.2.4",
-    "@hono/node-server": "^1.14.3",
     "@hono/swagger-ui": "^0.5.1",
     "@inkeep/agents-core": "workspace:^",
     "@inkeep/agents-manage-mcp": "workspace:^",
     "@modelcontextprotocol/sdk": "1.24.3",
-    "@nangohq/node": "^0.69.5",
-    "@nangohq/types": "^0.69.5",
     "axios": "^1.7.9",
-    "dotenv": "^17.2.1",
     "drizzle-orm": "^0.44.4",
     "hono": "^4.10.4",
     "hono-pino": "^0.10.1",
-    "jose": "^6.1.0",
     "nanoid": "^5.1.5",
     "openid-client": "^6.6.4",
     "pino": "^9.7.0"
@@ -59,8 +54,6 @@
     "@hono/vite-dev-server": "^0.20.1",
     "@types/node": "^20.11.24",
     "@vitest/coverage-v8": "^3.2.4",
-    "nodemon": "^3.1.0",
-    "pino-pretty": "^13.0.0",
     "tsx": "^4.7.1",
     "typescript": "^5.3.3",
     "vite": "^7.1.11",
diff --git a/agents-manage-ui/knip.config.ts b/agents-manage-ui/knip.config.ts
@@ -2,11 +2,13 @@ import type { KnipConfig } from 'knip';
 
 export default {
   tags: ['-lintignore'],
-  ignoreDependencies: ['@svgr/webpack'],
+  ignoreDependencies: [
+    '@svgr/webpack', // Set as turbopack loader in `next.config.ts`
+    'postcss', // Bundled in Next.js
+    'pino-pretty', // Set the transport target in `agents-manage-ui/src/lib/logger.ts`
+  ],
   ignoreIssues: {
-    'src/components/ui/*': ['exports'],
-    'src/components/agent/configuration/model-options.tsx': ['exports'],
-    'cypress/env.d.ts': ['files'],
-    'cypress/cypress.config.ts': ['files'],
+    'agents-manage-ui/src/components/ui/*': ['exports'],
+    'agents-manage-ui/src/components/agent/configuration/model-options.tsx': ['exports'],
   },
 } satisfies KnipConfig;
diff --git a/agents-manage-ui/package.json b/agents-manage-ui/package.json
@@ -32,7 +32,7 @@
     "directory": "agents-manage-ui"
   },
   "scripts": {
-    "knip": "knip --fix-type exports --fix-type types --include exports,types",
+    "knip": "knip --directory .. --workspace agents-manage-ui --config agents-manage-ui/knip.config.ts --dependencies --fix-type exports --fix-type types --include exports,types",
     "dev": "next --port 3000",
     "build:sync": "cp -r public .next/standalone/agents-manage-ui && mkdir -p .next/standalone/agents-manage-ui/.next/ && cp -r .next/static .next/standalone/agents-manage-ui/.next/static",
     "build": "next build && pnpm build:sync",
@@ -50,23 +50,13 @@
     "test:e2e:ci": "start-server-and-test start http-get://localhost:3000/default/projects test:e2e:run"
   },
   "dependencies": {
-    "@ai-sdk/anthropic": "3.0.0-beta.66",
-    "@ai-sdk/gateway": "2.0.0-beta.68",
-    "@ai-sdk/google": "3.0.0-beta.62",
-    "@ai-sdk/openai": "3.0.0-beta.74",
-    "@ai-sdk/openai-compatible": "2.0.0-beta.41",
-    "@ai-sdk/react": "3.0.0-beta.124",
     "@better-auth/sso": "^1.4.0",
     "@hookform/resolvers": "^5.2.1",
     "@inkeep/agents-core": "workspace:^",
-    "@inkeep/agents-manage-api": "workspace:^",
-    "@inkeep/agents-run-api": "workspace:^",
     "@inkeep/agents-ui": "^0.15.5",
-    "@lezer/highlight": "^1.2.1",
     "@nangohq/frontend": "^0.69.5",
     "@nangohq/node": "^0.69.5",
     "@nangohq/types": "^0.69.5",
-    "@openrouter/ai-sdk-provider": "^1.2.0",
     "@radix-ui/react-alert-dialog": "^1.1.15",
     "@radix-ui/react-avatar": "^1.1.11",
     "@radix-ui/react-checkbox": "^1.3.2",
@@ -128,7 +118,6 @@
     "@tailwindcss/postcss": "^4",
     "@testing-library/jest-dom": "^6.8.0",
     "@testing-library/react": "^16.3.0",
-    "@testing-library/user-event": "^14.6.1",
     "@types/dagre": "^0.7.53",
     "@types/json-schema": "^7.0.15",
     "@types/node": "^20",
diff --git a/agents-run-api/package.json b/agents-run-api/package.json
@@ -13,6 +13,7 @@
   "type": "module",
   "license": "SEE LICENSE IN LICENSE.md",
   "scripts": {
+    "knip": "knip --directory .. --workspace agents-run-api --dependencies",
     "dev": "vite",
     "dev:with-bypass": "PORT=3003 vite",
     "dev:without-bypass": "PORT=3004 INKEEP_AGENTS_RUN_API_BYPASS_SECRET= vite",
@@ -32,9 +33,7 @@
     "@ai-sdk/google": "3.0.0-beta.62",
     "@ai-sdk/openai": "3.0.0-beta.74",
     "@ai-sdk/openai-compatible": "2.0.0-beta.41",
-    "@ai-sdk/react": "3.0.0-beta.124",
     "@alcyone-labs/modelcontextprotocol-sdk": "^1.16.0",
-    "@hono/node-server": "^1.14.3",
     "@hono/otel": "^0.4.0",
     "@hono/swagger-ui": "^0.5.1",
     "@inkeep/agents-core": "workspace:^",
@@ -48,26 +47,15 @@
     "@opentelemetry/resources": "^2.1.0",
     "@opentelemetry/sdk-node": "^0.205.0",
     "@opentelemetry/sdk-trace-base": "^2.1.0",
-    "@opentelemetry/sdk-trace-node": "^2.1.0",
     "@opentelemetry/semantic-conventions": "^1.37.0",
     "@vercel/sandbox": "^0.0.24",
     "ai": "6.0.0-beta.124",
     "ajv": "^8.17.1",
-    "ajv-formats": "^3.0.1",
-    "destr": "^2.0.5",
-    "dotenv": "^17.2.1",
     "drizzle-orm": "^0.44.4",
-    "exit-hook": "^4.0.0",
     "fetch-to-node": "^2.1.0",
     "hono": "^4.10.4",
     "jmespath": "^0.16.0",
-    "jose": "^6.1.0",
-    "json-schema-to-zod": "^2.6.1",
-    "llm-info": "^1.0.69",
-    "nanoid": "^5.1.5",
-    "pino": "^9.11.0",
-    "traverse": "^0.6.11",
-    "ts-pattern": "^5.7.1"
+    "llm-info": "^1.0.69"
   },
   "peerDependencies": {
     "@hono/zod-openapi": "^1.1.5",
@@ -80,12 +68,9 @@
     "@hono/vite-dev-server": "^0.20.1",
     "@opentelemetry/exporter-trace-otlp-proto": "^0.203.0",
     "@opentelemetry/sdk-metrics": "^2.1.0",
-    "@types/ajv": "^1.0.4",
     "@types/jmespath": "^0.15.2",
     "@types/node": "^20.11.24",
-    "@types/traverse": "^0.6.37",
     "@vitest/coverage-v8": "^3.2.4",
-    "nodemon": "^3.1.0",
     "tsx": "^4.7.1",
     "typescript": "^5.3.3",
     "vite": "^7.1.11",
diff --git a/agents-run-api/tsdown.config.ts b/agents-run-api/tsdown.config.ts
@@ -1,11 +1,10 @@
 import { readFileSync } from 'node:fs';
 import { dirname, resolve } from 'node:path';
-import type { Plugin } from 'rolldown';
-import { defineConfig } from 'tsdown';
+import { defineConfig, type UserConfig } from 'tsdown';
 import rootConfig from '../tsdown.config.ts';
 
 // Plugin to handle ?raw imports (Vite-style)
-function rawPlugin(): Plugin {
+function rawPlugin(): UserConfig['plugins'] {
   return {
     name: 'raw-import',
     resolveId(source, importer) {
diff --git a/package.json b/package.json
@@ -64,13 +64,12 @@
   },
   "version": "0.1.0",
   "dependencies": {
-    "npm-run-all": "^4.1.5",
     "tsdown": "^0.18.0",
     "zod": "^4.1.12",
     "@hono/zod-openapi": "^1.1.5"
   },
   "devDependencies": {
-    "knip": "^5.75.1",
+    "knip": "^5.75.2",
     "@biomejs/biome": "2.2.4",
     "@changesets/cli": "2.29.7",
     "drizzle-kit": "^0.31.5",
diff --git a/packages/agents-core/package.json b/packages/agents-core/package.json
@@ -70,6 +70,7 @@
     "./package.json": "./package.json"
   },
   "scripts": {
+    "knip": "knip --directory ../.. --workspace packages/agents-core --dependencies",
     "build": "tsdown",
     "test": "vitest --run",
     "test:unit": "vitest --run src/__tests__ --exclude src/__tests__/integration/**",
@@ -100,15 +101,13 @@
     "@composio/core": "^0.2.4",
     "@electric-sql/pglite": "^0.3.13",
     "@better-auth/sso": "^1.4.0",
-    "@hono/node-server": "^1.14.3",
     "@modelcontextprotocol/sdk": "^1.17.2",
     "@nangohq/node": "^0.69.5",
     "@nangohq/types": "^0.69.5",
     "@openrouter/ai-sdk-provider": "^1.2.0",
     "@opentelemetry/api": "^1.9.0",
     "ai": "6.0.0-beta.124",
     "ajv": "^8.17.1",
-    "ajv-formats": "^3.0.1",
     "better-auth": "^1.4.0",
     "destr": "^2.0.3",
     "dotenv": "^17.2.1",
diff --git a/packages/agents-sdk/knip.config.ts b/packages/agents-sdk/knip.config.ts
@@ -0,0 +1,5 @@
+import type { KnipConfig } from 'knip';
+
+export default {
+  tags: ['-lintignore'],
+} satisfies KnipConfig;
diff --git a/packages/agents-sdk/package.json b/packages/agents-sdk/package.json
@@ -12,6 +12,7 @@
     "./package.json": "./package.json"
   },
   "scripts": {
+    "knip": "knip --directory ../.. --workspace packages/agents-sdk --config packages/agents-sdk/knip.config.ts --dependencies",
     "build": "tsdown",
     "test": "ENVIRONMENT=test vitest --run",
     "test:watch": "ENVIRONMENT=test vitest",
@@ -25,9 +26,7 @@
   "dependencies": {
     "@inkeep/agents-core": "workspace:^",
     "@yarnpkg/lockfile": "^1.1.0",
-    "esbuild": "^0.25.10",
     "js-yaml": "^4.1.0",
-    "nanoid": "^5.1.5",
     "typescript": "^5.3.3",
     "zod": "^4.1.11"
   },
@@ -36,7 +35,6 @@
     "@types/node": "^20.11.24",
     "@types/yarnpkg__lockfile": "^1.1.0",
     "@vitest/coverage-v8": "^3.2.4",
-    "tsx": "^4.7.1",
     "vitest": "^3.2.4"
   },
   "packageManager": "pnpm@10.11.0",
diff --git a/packages/agents-sdk/src/telemetry-provider.ts b/packages/agents-sdk/src/telemetry-provider.ts
@@ -507,6 +507,7 @@ function wrapOtelSpan(otelSpan: {
  */
 function isOpenTelemetryAvailable(): boolean {
   try {
+    /** @lintignore Skip report as unlisted dependencies by Knip */
     require.resolve('@opentelemetry/api');
     return true;
   } catch {
diff --git a/packages/ai-sdk-provider/package.json b/packages/ai-sdk-provider/package.json
@@ -14,6 +14,7 @@
     }
   },
   "scripts": {
+    "knip": "knip --directory ../.. --workspace packages/ai-sdk-provider --dependencies",
     "build": "tsdown",
     "test": "vitest --run",
     "test:watch": "vitest --watch",
@@ -37,8 +38,7 @@
     "zod": "^4.1.12",
     "@ai-sdk/provider": "3.0.0-beta.22",
     "@ai-sdk/provider-utils": "4.0.0-beta.40",
-    "@inkeep/agents-core": "workspace:*",
-    "ai": "6.0.0-beta.124"
+    "@inkeep/agents-core": "workspace:*"
   },
   "publishConfig": {
     "access": "public",
diff --git a/packages/create-agents/package.json b/packages/create-agents/package.json
@@ -13,6 +13,7 @@
     "SUPPLEMENTAL_TERMS.md"
   ],
   "scripts": {
+    "knip": "knip --directory ../.. --workspace packages/create-agents --dependencies",
     "dev": "tsx src/index.ts",
     "build": "tsc",
     "lint": "biome lint --error-on-warnings",
@@ -46,7 +47,6 @@
     "@inkeep/agents-core": "workspace:*",
     "commander": "^12.0.0",
     "degit": "^2.8.4",
-    "drizzle-kit": "^0.31.5",
     "fs-extra": "^11.0.0",
     "picocolors": "^1.0.0"
   },
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
