Skip to content

Conversation

wanglam
Copy link
Collaborator

@wanglam wanglam commented Sep 2, 2025

Description

  1. Pass empty executor memory when PER Agent execution, speed up the first step display
  2. Using parent interaction id to fetch PER Agent execution result
  3. Only polling task when paragraph result no messageId and executorMemoryId

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • New functionality has user manual doc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@wanglam wanglam force-pushed the refactor-with-interaction-id branch from 06279dc to 7887d8e Compare September 2, 2025 09:26
Copy link

codecov bot commented Sep 2, 2025

Codecov Report

❌ Patch coverage is 76.31579% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.07%. Comparing base (fd3b5b7) to head (7632158).

Files with missing lines Patch % Lines
...s/deep_research/services/per_agent_task_service.ts 0.00% 4 Missing ⚠️
...eep_research/services/per_agent_message_service.ts 92.30% 2 Missing ⚠️
server/services/ml_service.ts 33.33% 2 Missing ⚠️
public/utils/ml_commons_apis.ts 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #105      +/-   ##
==========================================
+ Coverage   64.78%   65.07%   +0.28%     
==========================================
  Files          57       58       +1     
  Lines        1806     1841      +35     
  Branches      444      447       +3     
==========================================
+ Hits         1170     1198      +28     
- Misses        634      641       +7     
  Partials        2        2              

☔ 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.

@wanglam wanglam changed the title Refactor with parent interaction Refactor with parent interaction id Sep 2, 2025
@wanglam wanglam marked this pull request as ready for review September 2, 2025 10:09
@wanglam wanglam force-pushed the refactor-with-interaction-id branch 3 times, most recently from 5787842 to 6bc2785 Compare September 10, 2025 03:39
@wanglam wanglam force-pushed the refactor-with-interaction-id branch from 9f9e17d to 7632158 Compare September 11, 2025 04:28
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.

1 participant