shu/making scrolling debug logs

[wintonzheng]

• make scrolling related logs DEBUG logs
• make Registering LLM config debug log

---

🚀	This description was created by Ellipsis for commit fbad122

Summary:

Change log level from info to debug for specific actions in skyvern/forge/sdk/api/llm/config_registry.py and skyvern/webeye/utils/page.py.

Key points:

• Change log level from info to debug for registering LLM config in skyvern/forge/sdk/api/llm/config_registry.py.
• Change log level from info to debug for scrolling actions in skyvern/webeye/utils/page.py.
• Affects register_config method in LLMConfigRegistry class.
• Affects take_split_screenshots method in SkyvernFrame class.

---

Generated with ❤️ by ellipsis.dev

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to fbad122 in 35.222575 seconds

More details

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

1. skyvern/forge/sdk/api/llm/config_registry.py:35

• Draft comment:
  The change from INFO to DEBUG log level is appropriate for the context of registering LLM configurations, as it reduces log verbosity while still providing necessary debug information.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The PR changes the log level from INFO to DEBUG for the log statement in the register_config method of the LLMConfigRegistry class. This change is consistent with the PR description, which aims to make certain logs less verbose by changing them to DEBUG level. The change is straightforward and does not introduce any new issues or bugs. The use of llm_key as a parameter in the log message provides useful context for debugging without being overly verbose.

2. skyvern/webeye/utils/page.py:94

• Draft comment:
  Changing the log level from INFO to DEBUG for scrolling-related logs is appropriate as it reduces unnecessary verbosity in production logs while retaining useful debug information.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The PR changes the log level from INFO to DEBUG for the log statements related to scrolling actions in the take_split_screenshots method of the SkyvernFrame class. This change aligns with the PR description's intent to make scrolling-related logs less verbose by changing them to DEBUG level. The log statements provide context about the scrolling actions, such as the URL being scrolled, the number of screenshots taken, and the scroll positions before and after scrolling. These details are useful for debugging but do not need to be at the INFO level during normal operations.

Workflow ID: wflow_IajKT8wrZ04Osi9U

---

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