Skip to content

adapt worker#1523

Open
fengju0213 wants to merge 1 commit intoplanfrom
camel_migrate_worker
Open

adapt worker#1523
fengju0213 wants to merge 1 commit intoplanfrom
camel_migrate_worker

Conversation

@fengju0213
Copy link
Copy Markdown
Collaborator

@fengju0213 fengju0213 commented Mar 27, 2026

Summary

Replace eigent's full _process_task override with delegation to CAMEL's base SingleAgentWorker, which now handles unified streaming, execution context, and attempt tracking natively.

Changes

backend/app/utils/single_agent_worker.py (~340 → ~95 lines)

  • Remove complete _process_task reimplementation (inline streaming consumption, usage extraction, attempt recording all deleted)
  • Override _get_worker_agent to inject eigent-specific process_task_id
  • _process_task now only sets _current_task_id and delegates to super()

backend/tests/app/utils/test_single_agent_worker.py (~686 → ~140 lines)

  • Test eigent-specific behavior only: initialization, inheritance, process_task_id injection, delegation to base, cleanup on failure
  • Streaming/context/attempt tests covered by CAMEL side

Closes #

Description

Testing Evidence (REQUIRED)

  • I have included human-verified testing evidence in this PR.
  • This PR includes frontend/UI changes, and I attached screenshot(s) or screen recording(s).
  • No frontend/UI changes in this PR.

What is the purpose of this pull request?

  • Bug fix
  • New Feature
  • Documentation update
  • Other

Contribution Guidelines Acknowledgement

@fengju0213 fengju0213 changed the title update adapt worker Mar 27, 2026
@fengju0213
Copy link
Copy Markdown
Collaborator Author

pending camel-ai/camel#3968

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