Skip to content

Commit f5def41

Browse files
la14-1spawn-qa-botlouisgv
authored
refactor: remove dead exported types from picker.ts and spawn-config.ts (#2553)
PickOption, PickConfig, and PickResult interfaces in picker.ts were exported but never imported by any external module. SpawnConfig type in spawn-config.ts was similarly exported but not used outside the module. Made all four private to reduce the public API surface. Bump CLI patch version to 0.17.2. -- qa/code-quality Co-authored-by: spawn-qa-bot <qa@openrouter.ai> Co-authored-by: L <6723574+louisgv@users.noreply.github.com>
1 parent ecc876f commit f5def41

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

packages/cli/src/picker.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,21 +21,21 @@ import { spawnSync } from "node:child_process";
2121
import * as fs from "node:fs";
2222
import { tryCatch, unwrapOr } from "./shared/result.js";
2323

24-
export interface PickOption {
24+
interface PickOption {
2525
value: string;
2626
label: string;
2727
hint?: string;
2828
subtitle?: string;
2929
}
3030

31-
export interface PickConfig {
31+
interface PickConfig {
3232
message: string;
3333
options: PickOption[];
3434
defaultValue?: string;
3535
deleteKey?: boolean;
3636
}
3737

38-
export interface PickResult {
38+
interface PickResult {
3939
action: "select" | "delete" | "cancel";
4040
value: string | null;
4141
index: number;

packages/cli/src/shared/spawn-config.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ const SpawnConfigSchema = v.object({
1818
setup: v.optional(SpawnConfigSetupSchema),
1919
});
2020

21-
export type SpawnConfig = v.InferOutput<typeof SpawnConfigSchema>;
21+
type SpawnConfig = v.InferOutput<typeof SpawnConfigSchema>;
2222

2323
/** Maximum config file size (1 MB) */
2424
const MAX_CONFIG_SIZE = 1024 * 1024;

0 commit comments

Comments
 (0)