Skip to content

feat: clientId no longer required in base language model#4411

Merged
life2015 merged 1 commit intov3.8from
feat/client-no-longer-required
Feb 26, 2025
Merged

feat: clientId no longer required in base language model#4411
life2015 merged 1 commit intov3.8from
feat/client-no-longer-required

Conversation

@life2015
Copy link
Copy Markdown
Member

Types

clientId 非必传,如果没有传入 clientId,则不使用工具

  • 🎉 New Features
  • 🐛 Bug Fixes
  • 📚 Documentation Changes
  • 💄 Code Style Changes
  • 💄 Style Changes
  • 🪚 Refactors
  • 🚀 Performance Improvements
  • 🏗️ Build System
  • ⏱ Tests
  • 🧹 Chores
  • Other Changes

Background or solution

Changelog

  • feat: clientId no longer required in base language model

@life2015 life2015 requested a review from Ricbet February 26, 2025 03:48
@opensumi opensumi Bot added the 🎨 feature feature required label Feb 26, 2025
@life2015 life2015 merged commit f7916df into v3.8 Feb 26, 2025
@life2015 life2015 deleted the feat/client-no-longer-required branch February 26, 2025 03:50
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 26, 2025

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 53.44%. Comparing base (b61cba5) to head (01ed73e).
Report is 1 commits behind head on v3.8.

Files with missing lines Patch % Lines
packages/ai-native/src/node/base-language-model.ts 0.00% 2 Missing and 2 partials ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             v3.8    #4411   +/-   ##
=======================================
  Coverage   53.44%   53.44%           
=======================================
  Files        1657     1657           
  Lines      102103   102103           
  Branches    22076    22084    +8     
=======================================
  Hits        54572    54572           
  Misses      39550    39550           
  Partials     7981     7981           
Flag Coverage Δ
jsdom 48.96% <0.00%> (ø)
node 12.20% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎨 feature feature required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants