[PR #890] [MERGED] fix knip issues #921

New issue

Closed

opened 2026-05-06 13:10:51 +02:00 by BreizhHardware · 0 comments

BreizhHardware commented

2026-05-06 13:10:51 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cloudflare/vinext/pull/890
Author: @james-elicx
Created: 4/24/2026
Status: ✅ Merged
Merged: 4/24/2026
Merged by: @james-elicx

Base: main ← Head: claude/fervent-lovelace-7bdfcd

📝 Commits (4)

f69fd6b fix knip issues
2f32632 run knip in ci
d82d582 .
1d430d4 feedback

📊 Changes

47 files changed (+164 additions, -154 deletions)

View changed files

📝 .github/workflows/ci.yml (+1 -0)
📝 knip.ts (+27 -6)
📝 packages/vinext/src/build/layout-classification.ts (+2 -8)
📝 packages/vinext/src/build/nitro-route-rules.ts (+1 -1)
📝 packages/vinext/src/build/precompress.ts (+1 -1)
📝 packages/vinext/src/build/prerender.ts (+5 -8)
📝 packages/vinext/src/build/run-prerender.ts (+2 -2)
📝 packages/vinext/src/build/standalone.ts (+2 -2)
📝 packages/vinext/src/cloudflare/tpr.ts (+1 -1)
📝 packages/vinext/src/config/dotenv.ts (+3 -3)
📝 packages/vinext/src/deploy.ts (+2 -2)
📝 packages/vinext/src/init.ts (+1 -1)
📝 packages/vinext/src/plugins/async-hooks-stub.ts (+1 -1)
📝 packages/vinext/src/plugins/fonts.ts (+2 -2)
📝 packages/vinext/src/plugins/server-externals-manifest.ts (+1 -1)
📝 packages/vinext/src/routing/file-matcher.ts (+2 -2)
📝 packages/vinext/src/routing/utils.ts (+2 -2)
📝 packages/vinext/src/server/app-browser-state.ts (+5 -5)
📝 packages/vinext/src/server/app-browser-stream.ts (+3 -3)
📝 packages/vinext/src/server/app-elements.ts (+3 -3)

...and 27 more files

📄 Description

No description provided

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/cloudflare/vinext/pull/890 **Author:** [@james-elicx](https://github.com/james-elicx) **Created:** 4/24/2026 **Status:** ✅ Merged **Merged:** 4/24/2026 **Merged by:** [@james-elicx](https://github.com/james-elicx) **Base:** `main` ← **Head:** `claude/fervent-lovelace-7bdfcd` --- ### 📝 Commits (4) - [`f69fd6b`](https://github.com/cloudflare/vinext/commit/f69fd6b83d9093523958e1fae673e0821de5046a) fix knip issues - [`2f32632`](https://github.com/cloudflare/vinext/commit/2f32632c8350b25a66d9263df44f68d72efb2868) run knip in ci - [`d82d582`](https://github.com/cloudflare/vinext/commit/d82d582f9daced1dbb71c0e11434f0427262ddbe) . - [`1d430d4`](https://github.com/cloudflare/vinext/commit/1d430d46b15e0f749309b004b0c9208f3521114e) feedback ### 📊 Changes **47 files changed** (+164 additions, -154 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/ci.yml` (+1 -0) 📝 `knip.ts` (+27 -6) 📝 `packages/vinext/src/build/layout-classification.ts` (+2 -8) 📝 `packages/vinext/src/build/nitro-route-rules.ts` (+1 -1) 📝 `packages/vinext/src/build/precompress.ts` (+1 -1) 📝 `packages/vinext/src/build/prerender.ts` (+5 -8) 📝 `packages/vinext/src/build/run-prerender.ts` (+2 -2) 📝 `packages/vinext/src/build/standalone.ts` (+2 -2) 📝 `packages/vinext/src/cloudflare/tpr.ts` (+1 -1) 📝 `packages/vinext/src/config/dotenv.ts` (+3 -3) 📝 `packages/vinext/src/deploy.ts` (+2 -2) 📝 `packages/vinext/src/init.ts` (+1 -1) 📝 `packages/vinext/src/plugins/async-hooks-stub.ts` (+1 -1) 📝 `packages/vinext/src/plugins/fonts.ts` (+2 -2) 📝 `packages/vinext/src/plugins/server-externals-manifest.ts` (+1 -1) 📝 `packages/vinext/src/routing/file-matcher.ts` (+2 -2) 📝 `packages/vinext/src/routing/utils.ts` (+2 -2) 📝 `packages/vinext/src/server/app-browser-state.ts` (+5 -5) 📝 `packages/vinext/src/server/app-browser-stream.ts` (+3 -3) 📝 `packages/vinext/src/server/app-elements.ts` (+3 -3) _...and 27 more files_ </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>