Skip to content

Commit 362e266

Browse files
committed
refactor: remove unnecessary methods from Gemini CLI runner
Removes a couple of methods that were duplicated from the base class.
1 parent 7022a71 commit 362e266

File tree

1 file changed

+1
-18
lines changed

1 file changed

+1
-18
lines changed

runner/codegen/gemini-cli/gemini-cli-runner.ts

Lines changed: 1 addition & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,4 @@
1-
import {
2-
LlmConstrainedOutputGenerateResponse,
3-
LlmGenerateFilesRequestOptions,
4-
LlmGenerateTextResponse,
5-
LlmRunner,
6-
} from '../llm-runner.js';
1+
import {LlmGenerateFilesRequestOptions, LlmRunner} from '../llm-runner.js';
72
import {join} from 'path';
83
import {mkdirSync} from 'fs';
94
import {writeFile} from 'fs/promises';
@@ -12,7 +7,6 @@ import {
127
getGeminiInstructionsFile,
138
getGeminiSettingsFile,
149
} from './gemini-files.js';
15-
import {UserFacingError} from '../../utils/errors.js';
1610
import {BaseCliAgentRunner} from '../base-cli-agent-runner.js';
1711

1812
const SUPPORTED_MODELS = ['gemini-2.5-pro', 'gemini-2.5-flash', 'gemini-2.5-flash-lite'];
@@ -25,17 +19,6 @@ export class GeminiCliRunner extends BaseCliAgentRunner implements LlmRunner {
2519
protected ignoredFilePatterns = ['**/GEMINI.md', '**/.geminiignore'];
2620
protected binaryName = 'gemini';
2721

28-
generateText(): Promise<LlmGenerateTextResponse> {
29-
// Technically we can make this work, but we don't need it at the time of writing.
30-
throw new UserFacingError('Generating text with Gemini CLI is not supported.');
31-
}
32-
33-
generateConstrained(): Promise<LlmConstrainedOutputGenerateResponse<any>> {
34-
// We can't support this, because there's no straightforward
35-
// way to tell the Gemini CLI to follow a schema.
36-
throw new UserFacingError('Constrained output with Gemini CLI is not supported.');
37-
}
38-
3922
getSupportedModels(): string[] {
4023
return SUPPORTED_MODELS;
4124
}

0 commit comments

Comments
 (0)