Skip to content

Commit 96339dc

Browse files
committed
chore: clean up unused exports and devDependencies flagged by knip
- Remove export from resetIdentifierLimiter and identifierLimiterSize in identifier-limit.ts — no test imports these from the middleware - Fix superagent type reference in streaming test — swap import("superagent").Response to import("supertest").Response - Remove @astrojs/check from website devDependencies — not in any script - Tighten knip.json: drop redundant entry patterns and unused ignores
1 parent f99550b commit 96339dc

4 files changed

Lines changed: 6 additions & 10 deletions

File tree

knip.json

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,11 @@
11
{
22
"workspaces": {
33
"packages/api-server": {
4-
"entry": ["src/server.ts", "build.mjs"],
4+
"entry": ["src/server.ts"],
55
"project": ["src/**/*.ts"]
66
},
77
"packages/dashboard": {
8-
"entry": ["src/main.tsx"],
98
"project": ["src/**/*.{ts,tsx}"]
109
}
11-
},
12-
"ignore": ["docs/**", "lib/**"]
10+
}
1311
}

packages/api-server/src/middleware/identifier-limit.ts

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ import {
2020
* (we never silently accept a tainted value into logs)
2121
*
2222
* The limiter is a module-level singleton so every request hits the
23-
* same Map. Tests can reset it via `resetIdentifierLimiter()`.
23+
* same Map.
2424
*/
2525

2626
const SAFE_PATTERN = /^[A-Za-z0-9_.:-]{1,128}$/;
@@ -33,12 +33,11 @@ const limiter = new IdentifierLimiter(
3333
),
3434
);
3535

36-
export function resetIdentifierLimiter(): void {
36+
function resetIdentifierLimiter(): void {
3737
limiter.reset();
3838
}
3939

40-
/** Exposed for tests that want to inspect the singleton state. */
41-
export function identifierLimiterSize(): number {
40+
function identifierLimiterSize(): number {
4241
return limiter.size();
4342
}
4443

packages/api-server/tests/streaming-virtual-keys.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ async function startFakeUpstream(): Promise<void> {
5858
upstreamUrl = `http://127.0.0.1:${(upstreamServer.address() as AddressInfo).port}`;
5959
}
6060

61-
function collectStream(res: import("superagent").Response): string {
61+
function collectStream(res: import("supertest").Response): string {
6262
return res.body as unknown as string;
6363
}
6464

packages/website/package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
"astro": "astro"
1212
},
1313
"devDependencies": {
14-
"@astrojs/check": "^0.9.4",
1514
"@astrojs/starlight": "^0.30.6",
1615
"astro": "^5.1.5",
1716
"sharp": "^0.33.5",

0 commit comments

Comments
 (0)