Skip to content

Commit 360b8a5

Browse files
committed
fix(llm): limit supported providers
1 parent 98cbba9 commit 360b8a5

3 files changed

Lines changed: 2 additions & 42 deletions

File tree

README.md

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,9 +51,8 @@ LLM_CRITIQUE_TEMPERATURE=0
5151
REVIEW_LANGUAGE=ru
5252
```
5353

54-
Для других провайдеров используйте соответствующий ключ, например
55-
`OPENAI_API_KEY`, `OPENROUTER_API_KEY` или `ANTHROPIC_API_KEY`. Провайдеры и
56-
модели каждой стадии задаются через `LLM_GENERATE_PROVIDER`,
54+
На текущий момент поддерживаются только провайдеры `gemini` и `ollama`.
55+
Провайдеры и модели каждой стадии задаются через `LLM_GENERATE_PROVIDER`,
5756
`LLM_GENERATE_MODEL`, `LLM_CRITIQUE_PROVIDER` и `LLM_CRITIQUE_MODEL`.
5857
`LLM_PROVIDER` остаётся общим значением по умолчанию, если отдельный провайдер
5958
стадии не указан.

lib/aireview/config.rb

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,9 +50,6 @@ class Config
5050

5151
PROVIDER_KEY_MAPPING = {
5252
'gemini' => 'GEMINI_API_KEY',
53-
'openai' => 'OPENAI_API_KEY',
54-
'openrouter' => 'OPENROUTER_API_KEY',
55-
'anthropic' => 'ANTHROPIC_API_KEY',
5653
'ollama' => nil
5754
}.freeze
5855

spec/config_spec.rb

Lines changed: 0 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -233,25 +233,6 @@
233233
end
234234
end
235235

236-
it 'validates API keys for generate and critique providers separately' do
237-
Dir.mktmpdir do |dir|
238-
config = described_class.load(
239-
cwd: dir,
240-
env: {
241-
'LLM_GENERATE_PROVIDER' => 'gemini',
242-
'LLM_GENERATE_MODEL' => 'gemini-2.5-pro',
243-
'GEMINI_API_KEY' => 'gemini-secret',
244-
'LLM_CRITIQUE_PROVIDER' => 'anthropic',
245-
'LLM_CRITIQUE_MODEL' => 'claude-sonnet-4'
246-
},
247-
logger: Logger.new(nil)
248-
)
249-
250-
expect { config.require_llm_configuration! }
251-
.to raise_error(Aireview::ConfigError, /critique.*anthropic/)
252-
end
253-
end
254-
255236
it 'requires a key only for the remote stage in a mixed configuration' do
256237
Dir.mktmpdir do |dir|
257238
config = described_class.load(
@@ -417,22 +398,5 @@
417398
end
418399
end
419400

420-
it 'loads OpenRouter API key from environment' do
421-
Dir.mktmpdir do |dir|
422-
config = described_class.load(
423-
cwd: dir,
424-
env: {
425-
'LLM_PROVIDER' => 'openrouter',
426-
'LLM_GENERATE_MODEL' => 'meta-llama/llama-3.3-70b-instruct:free',
427-
'LLM_CRITIQUE_MODEL' => 'meta-llama/llama-3.3-70b-instruct:free',
428-
'OPENROUTER_API_KEY' => 'sk-or-secret'
429-
},
430-
logger: Logger.new(nil)
431-
)
432-
433-
expect(config.llm_provider).to eq('openrouter')
434-
expect(config.provider_api_key).to eq('sk-or-secret')
435-
end
436-
end
437401
end
438402
end

0 commit comments

Comments
 (0)