From 4631000f5a7666c88402ce11a9ebb8cdef7a7dad Mon Sep 17 00:00:00 2001 From: Julius Marminge Date: Sat, 5 Sep 2026 01:38:51 -0700 Subject: [PATCH] ci: reject unused Codex client exports with Knip (#10036) --- docs/operations/development.md | 3 ++- package.json | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/docs/operations/development.md b/docs/operations/development.md index de5c6c64e3ab9ae95f27363227af822df043b220..39686807d6dd221244c82aab231fc190c0dabdfb 100644 --- a/docs/operations/development.md +++ b/docs/operations/development.md @@ -72,7 +72,8 @@ Windows investigation while that suite is not a required gate. ### Unused code `vp run knip:check` checks unused files and dependencies across the repo, then -unused exports and types in `packages/tailscale`. CI enforces both checks. +unused exports and types in `packages/tailscale` and `packages/effect-codex-app-server`. +CI enforces both checks. Use `vp run knip --workspace apps/web` to audit one workspace, including exports, or `vp run knip:production --workspace apps/web` to find code kept alive only by tests. The full export audit still has findings and is not a repo-wide CI gate. Extend the diff --git a/package.json b/package.json index 2882a1f9ab7e9fa8d04ac8d0bdc0edfefefbdae4..4fc0dafeb0e9a943bea3abf2d085976743dfa4ab 100644 --- a/package.json +++ b/package.json @@ -26,7 +26,7 @@ "tc": "vp run -r --concurrency-limit 2 typecheck", "lint": "vp lint --report-unused-disable-directives", "knip": "knip", - "knip:check": "knip --include files,dependencies --no-config-hints && knip --workspace packages/tailscale --exports --no-config-hints", + "knip:check": "knip --include files,dependencies --no-config-hints && knip --workspace packages/tailscale --workspace packages/effect-codex-app-server --exports --no-config-hints", "knip:production": "knip --production", "lint:mobile": "node scripts/mobile-native-static-check.ts", "test": "vp run -r test", -- 2.54.0