Skip to content

[#390] github-issue-390-to-no-provide#393

Merged
nrslib merged 1 commit intomainfrom
takt/390/github-issue-390-to-no-provide
Feb 26, 2026
Merged

[#390] github-issue-390-to-no-provide#393
nrslib merged 1 commit intomainfrom
takt/390/github-issue-390-to-no-provide

Conversation

@nrslib
Copy link
Owner

@nrslib nrslib commented Feb 25, 2026

Summary

背景

#386 の修正で defaultValue: 'claude' を削除し、movement-level provider が正しく優先されるようになった。
しかし、provider の解決ロジックが2箇所に分かれている構造自体は残っている。

現状

箇所 用途 解決ロジック
pieceExecution.ts:558 ログ表示・analytics resolveMovementProviderModel()
runner.ts:32-37 実際のエージェント実行 resolveProviderModelCandidates()

両者は同じ優先順位を意図しているが、独立した実装のため将来的にズレるリスクがある。

提案

AgentRunner.resolveProviderAndModel() を static public にし、pieceExecution.ts の表示側からも同じメソッドを呼ぶようにする。
これにより「表示された provider = 実行に使われる provider」が構造的に保証される。

Execution Report

Piece default completed successfully.

Closes #390

@nrslib nrslib marked this pull request as ready for review February 26, 2026 14:26
@nrslib nrslib merged commit 551299d into main Feb 26, 2026
3 checks passed
@nrslib nrslib deleted the takt/390/github-issue-390-to-no-provide branch February 26, 2026 14:45
@nrslib nrslib mentioned this pull request Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

表示用と実行用の provider 解決ロジックを一元化する

1 participant