Skip to content

Conversation

@tbrand
Copy link
Contributor

@tbrand tbrand commented Sep 1, 2025

Description of Changes

  • Reasoning の切り替えを toggle で制御できるように (token budget は modal のまま)
  • UI 上の reasoning を extended thinking に rename
  • InputChatContent の textarea の領域を変更
    • 以前は flex-col で textarea と buttons を縦に分割していたが、これによりスクロールバーの位置がおかしかった。これを flex-row に変更した。)
    • この影響で InputChatContent は最低でも 2 行表示 (textarea 1 行と buttons で計 2 行になった)
    • その影響で、InputChatContent をページ下部に配置しているページの padding bottom を 36 から 48 に変更
  • InputChatContent の buttons に tooltip を表示するように
    • 小さい tooltip を表示するオプションを Tooltip.tsx に新設
  • Deepseek-R1 の reasoning 機能の有効化
  • Chat.tsx の overrideModelParameters や AdditionalModelRequestFields の reasoningConfig は undefined を受け付けないように変更
    • reasoning が無効なモデルに overrideModelParameters を送っても問題ないことが確認できたため、コード可読性の観点から undefined を削除
    • むしろ、reasoning が enabled で budgetTokens が undefined の際にエラーになるため、それも型定義上発生しないように修整された

Checklist

  • Modified relevant documentation
  • Verified operation in local environment
  • Executed npm run cdk:test and if there are snapshot differences, execute npm run cdk:test:update-snapshot to update snapshots

Related Issues

#1210

saved_system_prompts: Saved System Prompts
search_by_title: Search by title
settings: Settings
settings: Apply
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

設定画面の「適用」ボタンが Settings なのが違和感合ったため Apply に変更しました。

overrideModelParameters={overrideModelParameters}
setOverrideModelParameters={setOverrideModelParameters}
/>
</div>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

項目が少ないので ExpandableField を一旦削除しました。

@maekawataiki maekawataiki merged commit 097ed01 into main Sep 2, 2025
6 checks passed
@tbrand tbrand deleted the reasoning-toggle branch September 4, 2025 03:05
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.

3 participants