Failed to convert css shape to string shape by LLM log should be warning

[wintonzheng]

Important

Change logging level from exception to warning in _convert_css_shape_to_string() for CSS shape conversion failures.

• Logging:
  • Change logging level from exception to warning in _convert_css_shape_to_string() in agent_functions.py when CSS shape conversion fails.

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

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to d5cd169 in 7 seconds

More details

• Looked at 18 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. skyvern/forge/agent_functions.py:254

• Draft comment:
  Using exc_info=True with LOG.warning is not typical and might not be necessary. Consider removing exc_info=True unless the traceback is needed for a warning log.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The change from LOG.exception to LOG.warning is appropriate given the context. However, the use of exc_info=True with LOG.warning is not typical and might not be necessary.

Workflow ID: wflow_KaHjKjEPWPMqVpiD

---

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