Remove extra args from config (#27)

We don't have a use case for them yet.
This commit is contained in:
Jaana Dogan
2025-04-18 11:26:39 -07:00
committed by GitHub
parent 3afaa8033b
commit 328846c6e3

View File

@@ -11,13 +11,11 @@ export class Config {
private apiKey: string; private apiKey: string;
private model: string; private model: string;
private targetDir: string; private targetDir: string;
private extraArgs: (string | number)[]; // Captures positional arguments
constructor(apiKey: string, model: string, targetDir: string, extraArgs: (string | number)[]) { constructor(apiKey: string, model: string, targetDir: string) {
this.apiKey = apiKey; this.apiKey = apiKey;
this.model = model; this.model = model;
this.targetDir = targetDir; this.targetDir = targetDir;
this.extraArgs = extraArgs;
} }
getApiKey(): string { getApiKey(): string {
@@ -31,10 +29,6 @@ export class Config {
getTargetDir(): string { getTargetDir(): string {
return this.targetDir; return this.targetDir;
} }
getExtraArgs(): (string | number)[] {
return this.extraArgs;
}
} }
export function loadConfig(): Config { export function loadConfig(): Config {
@@ -44,7 +38,6 @@ export function loadConfig(): Config {
process.env.GEMINI_API_KEY || "", process.env.GEMINI_API_KEY || "",
argv.model || process.env.GEMINI_API_KEY || DEFAULT_GEMINI_MODEL, argv.model || process.env.GEMINI_API_KEY || DEFAULT_GEMINI_MODEL,
argv.target_dir || process.cwd(), argv.target_dir || process.cwd(),
argv._,
); );
} }
@@ -53,7 +46,6 @@ export const globalConfig = loadConfig(); // TODO(jbd): Remove global state.
interface CliArgs { interface CliArgs {
target_dir: string | undefined; target_dir: string | undefined;
model: string | undefined; model: string | undefined;
_: (string | number)[]; // Captures positional arguments
// Add other expected args here if needed // Add other expected args here if needed
// e.g., verbose?: boolean; // e.g., verbose?: boolean;
} }