-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Open
Milestone
Description
Required prerequisites
- I have searched the Issue Tracker and Discussions that this hasn't already been reported. (+1 or comment there if it has.)
- Consider asking first in a Discussion.
Motivation
- Current string replacement approach (command.replace('python', ...)) incorrectly modifies commands like python-config and fails on complex shell commands like
VAR=1 python script.py.
updated in enhance: TerminalToolkit with performance improvements and bug fixes #3626 - TerminalToolkit implements its own Docker backend via use_docker_backend, but camel/runtimes/ already provides runtime abstractions: DockerRuntime, DaytonaRuntime, RemoteHttpRuntime, etc.linked issue: [Feature] Integrate TerminalToolkit with BaseRuntime module #3609
- current output collection logic
_collect_output_until_idle()incorrectly appends"[No new output]"status messages to actual output.
updated in enhance: TerminalToolkit with performance improvements and bug fixes #3626
Solution
No response
Alternatives
No response
Additional context
No response
coderabbitai
Metadata
Metadata
Assignees
Labels
No labels
Type
Projects
Status
No status