fix: Use ruamel.yaml instead of pyyaml

[creatorrr]

Signed-off-by: Diwank Singh Tomer [email protected]

---

Important

Replace pyyaml with ruamel.yaml for YAML processing and perform minor code cleanups.

• YAML Processing:
  • Replace pyyaml with ruamel.yaml in utils.py, template.py, and routers/tasks/router.py.
  • Add yaml.py in common/utils to define dump and load functions using ruamel.yaml.
• Code Cleanup:
  • Remove unused imports in excecute_api_call.py and execute_system.py.
  • Fix None comparison in execute_system.py.
  • Remove unused exception variable in update_execution.py.
• Dependencies:
  • Remove pyyaml and add ruamel-yaml in pyproject.toml.

^{This description was created by for 0854ead. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 0854ead in 26 seconds

More details

• Looked at 253 lines of code in 12 files
• Skipped 3 files when reviewing.
• Skipped posting 2 drafted comments based on config settings.

1. agents-api/agents_api/activities/utils.py:52

• Draft comment:
  Ensure that yaml.load is used correctly with the new yaml module from ruamel.yaml. The CSafeLoader is no longer applicable.
• Reason this comment was not posted:
  Marked as duplicate.

2. agents-api/agents_api/activities/utils.py:75

• Draft comment:
  Ensure that yaml.load and yaml.dump are used correctly with the new yaml module from ruamel.yaml. The CSafeLoader and CSafeDumper are no longer applicable.
• Reason this comment was not posted:
  Comment did not seem useful.

Workflow ID: wflow_ftZ0jnGTvAwowlnT

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.