Skip to content

fix: edit instruction#4456

Merged
Aaaaash merged 6 commits intov3.8from
fix/edit-instruction
Mar 11, 2025
Merged

fix: edit instruction#4456
Aaaaash merged 6 commits intov3.8from
fix/edit-instruction

Conversation

@ensorrow
Copy link
Copy Markdown
Contributor

@ensorrow ensorrow commented Mar 8, 2025

Types

  • 🐛 Bug Fixes

Background or solution

fix: add user's intention in apply prompt & fix getDiagnosticInfos error

Changelog

fix: add user's intention in apply prompt & fix getDiagnosticInfos error

@opensumi opensumi Bot added the 🐞 bug Something isn't working label Mar 8, 2025
chore: remove useless comment
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 8, 2025

Codecov Report

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

Project coverage is 53.26%. Comparing base (66e0c9f) to head (e49f7f7).
Report is 1 commits behind head on v3.8.

Files with missing lines Patch % Lines
...es/ai-native/src/browser/mcp/base-apply.service.ts 0.00% 2 Missing ⚠️
...-native/src/browser/mcp/tools/handlers/EditFile.ts 0.00% 2 Missing ⚠️
...ckages/ai-native/src/browser/chat/apply.service.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             v3.8    #4456      +/-   ##
==========================================
- Coverage   53.26%   53.26%   -0.01%     
==========================================
  Files        1661     1661              
  Lines      102562   102563       +1     
  Branches    22185    22186       +1     
==========================================
- Hits        54631    54630       -1     
  Misses      39881    39881              
- Partials     8050     8052       +2     
Flag Coverage Δ
jsdom 48.75% <0.00%> (+<0.01%) ⬆️
node 12.20% <0.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.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ensorrow ensorrow requested review from Aaaaash, AhkunTa and erha19 March 8, 2025 08:31
@Ricbet Ricbet changed the title Fix/edit instruction fix: edit instruction Mar 10, 2025
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

@Aaaaash Aaaaash merged commit 48cce5c into v3.8 Mar 11, 2025
@Aaaaash Aaaaash deleted the fix/edit-instruction branch March 11, 2025 02:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants