Skip to content

feat: add more chat report point#4416

Merged
ensorrow merged 2 commits intov3.8from
feat/chat-reporter
Mar 4, 2025
Merged

feat: add more chat report point#4416
ensorrow merged 2 commits intov3.8from
feat/chat-reporter

Conversation

@ensorrow
Copy link
Copy Markdown
Contributor

Types

  • 🎉 New Features

Background or solution

添加 & 完善 Chat 消息发送、接受、tool 调用、采纳|拒绝埋点

Changelog

feat: add more chat report point

@opensumi opensumi Bot added the 🎨 feature feature required label Feb 28, 2025
@ensorrow ensorrow requested review from Aaaaash, Ricbet and life2015 and removed request for Ricbet and life2015 February 28, 2025 13:36
Comment thread packages/ai-native/src/browser/mcp/base-apply.service.ts
Comment thread packages/ai-native/src/browser/mcp/mcp-server.feature.registry.ts
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 4, 2025

Codecov Report

Attention: Patch coverage is 10.00000% with 27 lines in your changes missing coverage. Please review.

Project coverage is 53.29%. Comparing base (9390712) to head (0d17c5a).
Report is 1 commits behind head on v3.8.

Files with missing lines Patch % Lines
...ive/src/browser/mcp/mcp-server.feature.registry.ts 0.00% 12 Missing and 1 partial ⚠️
...s/ai-native/src/browser/chat/chat-agent.service.ts 0.00% 9 Missing and 1 partial ⚠️
...es/ai-native/src/browser/mcp/base-apply.service.ts 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             v3.8    #4416      +/-   ##
==========================================
+ Coverage   53.27%   53.29%   +0.01%     
==========================================
  Files        1659     1659              
  Lines      102415   102434      +19     
  Branches    22168    22169       +1     
==========================================
+ Hits        54563    54590      +27     
+ Misses      39809    39805       -4     
+ Partials     8043     8039       -4     
Flag Coverage Δ
jsdom 48.81% <10.00%> (+0.01%) ⬆️
node 12.17% <10.00%> (+<0.01%) ⬆️

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.

moreJs
moreJs previously approved these changes Mar 4, 2025
Copy link
Copy Markdown
Contributor

@moreJs moreJs left a comment

Choose a reason for hiding this comment

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

+1

@ensorrow ensorrow force-pushed the feat/chat-reporter branch from a3eee97 to 0d17c5a Compare March 4, 2025 13:26
Copy link
Copy Markdown
Member

@Aaaaash Aaaaash left a comment

Choose a reason for hiding this comment

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

LGTM

@ensorrow ensorrow merged commit f72c34d into v3.8 Mar 4, 2025
@ensorrow ensorrow deleted the feat/chat-reporter branch March 4, 2025 13:45
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.

3 participants