Skip to content

Commit 4a20f80

Browse files
committed
chore(dead-exports): drop stale fixture allowlist entries
Linux CI stopped reporting those path-loaded fixture exports after intern shipped as a workspace package. Ignore comments keep Darwin ts-prune from treating the same exports as new dead code.
1 parent e89e44e commit 4a20f80

6 files changed

Lines changed: 6 additions & 12 deletions

File tree

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
export const notes: string[] = [];
22

3+
// ts-prune-ignore-next
34
export function addNote(note: string): void {
45
notes.push(note);
56
}
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,2 @@
1+
// ts-prune-ignore-next
12
export const data = { seed: 7 };

‎evals/completion/tasks/sum-fix/fixture/sum.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
// ts-prune-ignore-next
12
export function total(values: number[]): number {
23
let sum = 0;
34
for (const value of values) {

‎evals/completion/tasks/version-endpoint/fixture/service.ts‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ export interface Reply {
33
body: unknown;
44
}
55

6+
// ts-prune-ignore-next
67
export function handleRequest(method: string, path: string): Reply {
78
if (method === "GET" && path === "/health") {
89
return { status: 200, body: { ok: true } };

‎scripts/dead-export-allowlist.txt‎

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -29,18 +29,6 @@ vendor/
2929
src/auth/codex/constants.ts: CODEX_REFRESH_SKEW_MS
3030
src/auth/codex/constants.ts: CODEX_HEADLESS_REFRESH_INTERVAL_MS
3131

32-
# Plugin fixture entry point. Loaded by file path from the fixture's plugin
33-
# manifest by the plugin-registration tests, so it has no static importers.
34-
tests/fixtures/plugins/implement-feature/src/index.ts
35-
36-
# Eval completion-harness task fixtures (CL-7932 lane). Loaded by file path
37-
# from each task's script.json by the harness runner, so they have no static
38-
# importers.
39-
evals/completion/tasks/decline-refactor/fixture/notes.ts: addNote
40-
evals/completion/tasks/stall-read/fixture/data.ts: data
41-
evals/completion/tasks/sum-fix/fixture/sum.ts: total
42-
evals/completion/tasks/version-endpoint/fixture/service.ts: handleRequest
43-
4432
# ts-prune parser false positives: bare tokens on `as const satisfies ...`
4533
# lines inside live exports. None of these is an export declaration.
4634
src/provider/reasoning-effort.ts: satisfies

‎tests/fixtures/plugins/implement-feature/src/index.ts‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,12 @@ import { implementFeature } from "./workflows/implement-feature.js";
22
import type { CommandPlugin } from "../../../../../src/tui/commands/registry.js";
33
import type { WorkflowPlugin } from "../../../../../src/workflows/definition.js";
44

5+
// ts-prune-ignore-next
56
export const workflowPlugin: WorkflowPlugin = {
67
workflows: [implementFeature],
78
};
89

10+
// ts-prune-ignore-next
911
export const commandPlugin: CommandPlugin = {
1012
commands: [
1113
{

0 commit comments

Comments
 (0)