| author |  Julius Marminge <julius0216@outlook.com> 2026-09-05 01:38:51-07:00 |
| committer |  GitHub <noreply@github.com> 2026-09-05 01:38:51-07:00 |
| log | 4631000f5a7666c88402ce11a9ebb8cdef7a7dad |
| tree | a8dddafa0a734b08e03b3753a34f3b756c924e02 |
| parent | 62e4ae400555e62a42d5cb401d744ba6dd0b401b |
| signature | Signed by PGP key B5690EEEBB952194 |
ci: reject unused Codex client exports with Knip (#10036)
2 files changed, 3 insertions(+), 2 deletions(-)
docs
operations
- package.json
docs/operations/development.md+2-1
| ... | @@ -72,7 +72,8 @@ Windows investigation while that suite is not a required gate. | ... | @@ -72,7 +72,8 @@ Windows investigation while that suite is not a required gate. |
| 72 | ### Unused code | 72 | ### Unused code |
| 73 | 73 | ||
| 74 | `vp run knip:check` checks unused files and dependencies across the repo, then | 74 | `vp run knip:check` checks unused files and dependencies across the repo, then |
| 75 | unused exports and types in `packages/tailscale`. CI enforces both checks. | 75 | unused exports and types in `packages/tailscale` and `packages/effect-codex-app-server`. |
| 76 | CI enforces both checks. | ||
| 76 | Use `vp run knip --workspace apps/web` to audit one workspace, including exports, | 77 | Use `vp run knip --workspace apps/web` to audit one workspace, including exports, |
| 77 | or `vp run knip:production --workspace apps/web` to find code kept alive only by tests. | 78 | or `vp run knip:production --workspace apps/web` to find code kept alive only by tests. |
| 78 | The full export audit still has findings and is not a repo-wide CI gate. Extend the | 79 | The full export audit still has findings and is not a repo-wide CI gate. Extend the |
package.json+1-1
| ... | @@ -26,7 +26,7 @@ | ... | @@ -26,7 +26,7 @@ |
| 26 | "tc": "vp run -r --concurrency-limit 2 typecheck", | 26 | "tc": "vp run -r --concurrency-limit 2 typecheck", |
| 27 | "lint": "vp lint --report-unused-disable-directives", | 27 | "lint": "vp lint --report-unused-disable-directives", |
| 28 | "knip": "knip", | 28 | "knip": "knip", |
| 29 | "knip:check": "knip --include files,dependencies --no-config-hints && knip --workspace packages/tailscale --exports --no-config-hints", | 29 | "knip:check": "knip --include files,dependencies --no-config-hints && knip --workspace packages/tailscale --workspace packages/effect-codex-app-server --exports --no-config-hints", |
| 30 | "knip:production": "knip --production", | 30 | "knip:production": "knip --production", |
| 31 | "lint:mobile": "node scripts/mobile-native-static-check.ts", | 31 | "lint:mobile": "node scripts/mobile-native-static-check.ts", |
| 32 | "test": "vp run -r test", | 32 | "test": "vp run -r test", |