adjust cache expiration policy

[LawyZheng]

Important

Adjust cache expiration policy by removing unnecessary refreshes and adding cache keys to log messages in agent_functions.py and local.py.

• Cache Expiration:
  • Removed cache expiration refresh in get() method of LocalCache in local.py.
  • Added cache expiration refresh after successful SVG and CSS shape conversions in _convert_svg_to_string() and _convert_css_shape_to_string() in agent_functions.py.
• Logging:
  • Added key to log messages in _convert_svg_to_string() and _convert_css_shape_to_string() in agent_functions.py for better traceability.

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

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 528c9e7 in 14 seconds

More details

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

1. skyvern/forge/sdk/cache/local.py:16

• Draft comment:
  Removing await self.set(key, value) is correct. It was unnecessarily resetting cache expiration on access, which is not typical cache behavior.
• Reason this comment was not posted:
  Comment did not seem useful.

Workflow ID: wflow_vYmLgoLzAAfrv2P3

---

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