vodorod/Dorod-Sky
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Fix typos (#2807)

Browse Source
This commit is contained in:
Asher Foa
2025-06-27 21:26:21 -04:00
committed by GitHub
parent 8cad3aae18
commit a6bf217559
8 changed files with 29 additions and 29 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
skyvern/forge/prompts/skyvern/check-evaluation-goal.j2
View File

@@ -7,7 +7,7 @@ Reply in JSON format with the following keys:
"thought": str, // Think step by step. Describe your thought in thi field.
"is_booking": bool, // True if the goal is to book something, including room, flight and so on.
"is_including_date": bool, // True if the goal includes date information.
"tweaked_user_goal": str, // If is_booking is True and is_including_date is True, repick a date within the next two months to replace the orignal date in the goal. Otherwise, return the orignal user goal.
"tweaked_user_goal": str, // If is_booking is True and is_including_date is True, repick a date within the next two months to replace the original date in the goal. Otherwise, return the original user goal.
}
User goal

2
skyvern/forge/prompts/skyvern/confirm-multi-selection-finish.j2
View File

@@ -8,7 +8,7 @@ Reply in JSON format with the following keys:
{
"page_info": str, // Think step by step. Describe the page information you parsed from the HTML elements and the screenshot. Your decision should be based on the current page information.
"think": str, // Think step by step. Describe how you think the user has finished the mini goal in the current opened dropdown selection.
"is_multiple_selection": bool, // True if it's a multi-level selection, otheriwse False.
"is_multiple_selection": bool, // True if it's a multi-level selection, otherwise False.
"is_mini_goal_finished": bool, // True if the user has finished the mini goal in the current opened dropdown selection, False otherwise.
}

2
skyvern/forge/prompts/skyvern/extract-information.j2
View File

@@ -9,7 +9,7 @@ Do not ever include anything other than the JSON object in your output, and do n
If you are unable to extract the requested information for a specific field in the json schema, please output a null value for that field.
If you are trying to extract the href links which are using the jinja style like "{% raw %}{{}}{% endraw %}", please keep the orignal string.
If you are trying to extract the href links which are using the jinja style like "{% raw %}{{}}{% endraw %}", please keep the original string.
User Data Extraction Goal: {{ data_extraction_goal }}

2
skyvern/forge/sdk/api/llm/config_registry.py
View File

@@ -855,7 +855,7 @@ if settings.ENABLE_NOVITA:
# Get the credentials json file. See documentation: https://support.google.com/a/answer/7378726?hl=en
# my_vertex_credentials = json.dumps(json.load(open("my_credentials_file.json")))
# Set the value of my_vertex_credentials as the environment variable VERTEX_CREDENTIALS
# NOTE: If you want to specify a location, make sure the model is availale in the target location.
# NOTE: If you want to specify a location, make sure the model is available in the target location.
# See documentation: https://cloud.google.com/vertex-ai/generative-ai/docs/learn/locations#united-states
if settings.ENABLE_VERTEX_AI and settings.VERTEX_CREDENTIALS:
if not settings.VERTEX_PROJECT_ID:

34
skyvern/webeye/actions/handler.py
View File

@@ -1058,7 +1058,7 @@ async def handle_input_text_action(
skyvern_element = blocking_element
except Exception:
LOG.info(
"Failed to find the blocking element, continue with the orignal element",
"Failed to find the blocking element, continue with the original element",
exc_info=True,
task_id=task.task_id,
step_id=step.step_id,
@@ -1290,9 +1290,9 @@ async def handle_select_option_action(
if not await skyvern_element.is_selectable():
# 1. find from children
# TODO: 2. find from siblings and their chidren
# TODO: 2. find from siblings and their children
LOG.info(
"Element is not selectable, try to find the selectable element in the chidren",
"Element is not selectable, try to find the selectable element in the children",
tag_name=tag_name,
action=action,
)
@@ -1302,7 +1302,7 @@ async def handle_select_option_action(
selectable_child = await skyvern_element.find_selectable_child(dom=dom)
except Exception as e:
LOG.error(
"Failed to find selectable element in chidren",
"Failed to find selectable element in children",
exc_info=True,
tag_name=tag_name,
action=action,
@@ -1347,7 +1347,7 @@ async def handle_select_option_action(
blocking_element, exist = await skyvern_element.find_blocking_element(dom=dom)
except Exception:
LOG.warning(
"Failed to find the blocking element, continue to select on the orignal <select>",
"Failed to find the blocking element, continue to select on the original <select>",
task_id=task.task_id,
step_id=step.step_id,
exc_info=True,
@@ -1949,10 +1949,10 @@ async def chain_click(
action_results.append(ActionFailure(FailToClick(action.element_id, anchor="for", msg=str(e))))
try:
# sometimes the element is the direct chidren of the label, instead of using for="xx" attribute
# sometimes the element is the direct children of the label, instead of using for="xx" attribute
# since it's a click action, the target element we're searching should only be INPUT
LOG.info(
"Chain click: it's a label element. going to check for input of the direct chidren",
"Chain click: it's a label element. going to check for input of the direct children",
task_id=task.task_id,
action=action,
element=str(skyvern_element),
@@ -1986,7 +1986,7 @@ async def chain_click(
action_results.append(ActionFailure(FailToClick(action.element_id, anchor="attr_id", msg=str(e))))
try:
# sometimes the element is the direct chidren of the label, instead of using for="xx" attribute
# sometimes the element is the direct children of the label, instead of using for="xx" attribute
# so we check the direct parent if it's a label element
LOG.info(
"Chain click: it's a non-label element. going to find the bound label element by direct parent",
@@ -2257,10 +2257,10 @@ async def input_or_auto_complete_input(
element_id=skyvern_element.get_id(),
)
# 1. press the orignal text to see if there's a match
# 1. press the original text to see if there's a match
# 2. call LLM to find 5 potential values based on the orginal text
# 3. try each potential values from #2
# 4. call LLM to tweak the orignal text according to the information from #3, then start #1 again
# 4. call LLM to tweak the original text according to the information from #3, then start #1 again
# FIXME: try the whole loop for once now, to speed up skyvern
MAX_AUTO_COMPLETE_ATTEMP = 1
@@ -2439,7 +2439,7 @@ async def sequentially_select_from_dropdown(
)
return None
# TODO: only suport the third-level dropdown selection now
# TODO: only support the third-level dropdown selection now
MAX_SELECT_DEPTH = 3
values: list[str | None] = []
select_history: list[CustomSingleSelectResult] = []
@@ -2716,7 +2716,7 @@ async def select_from_dropdown(
) -> CustomSingleSelectResult:
"""
force_select: is used to choose an element to click even there's no dropdown menu;
targe_value: only valid when force_select is "False". When target_value is not empty, the matched option must be relevent to target value;
targe_value: only valid when force_select is "False". When target_value is not empty, the matched option must be relevant to target value;
None will be only returned when:
1. force_select is false and no dropdown menu popped
2. force_select is false and match value is not relevant to the target value
@@ -2854,7 +2854,7 @@ async def select_from_dropdown(
# sometimes we have multiple elements pointed to the same value,
# but only one option is clickable on the page
LOG.debug(
"Searching option with the same value in incremetal elements",
"Searching option with the same value in incremental elements",
value=value,
elements=incremental_scraped.element_tree,
)
@@ -3090,7 +3090,7 @@ async def try_to_find_potential_scrollable_element(
"""
check any <ul> or <role="listbox"> element in the chidlren.
if yes, return the found element,
eles, return the orginal one
else, return the orginal one
"""
found_element_id = await skyvern_element.find_children_element_id_by_callback(
cb=is_ul_or_listbox_element_factory(incremental_scraped=incremental_scraped, task=task, step=step),
@@ -3107,7 +3107,7 @@ async def try_to_find_potential_scrollable_element(
skyvern_element = await SkyvernElement.create_from_incremental(incremental_scraped, found_element_id)
except Exception:
LOG.debug(
"Failed to get head element by found element id, use the orignal element id",
"Failed to get head element by found element id, use the original element id",
element_id=found_element_id,
step_id=step.step_id,
task_id=task.task_id,
@@ -3158,7 +3158,7 @@ async def scroll_down_to_load_all_options(
# wait until animation ends, otherwise the scroll operation could be overwritten
await asyncio.sleep(2)
# scoll a little back and scoll down to trigger the loading
# scroll a little back and scroll down to trigger the loading
await page.mouse.wheel(0, -1e-5)
await page.mouse.wheel(0, 1e-5)
# wait for while to load new options
@@ -3181,7 +3181,7 @@ async def scroll_down_to_load_all_options(
else:
LOG.warning("Timeout to load all options, maybe some options will be missed")
# scoll back to the start point and wait for a while to make all options invisible on the page
# scroll back to the start point and wait for a while to make all options invisible on the page
if dropdown_menu_element_handle is None:
LOG.info("element handle is None, using mouse to scroll back", element_id=scrollable_element.get_id())
await page.mouse.wheel(0, -scroll_pace)

10
skyvern/webeye/scraper/domUtils.js
View File

@@ -599,7 +599,7 @@ function isDOMNodeRepresentDiv(element) {
}
const style = getElementComputedStyle(element);
const children = getChildElements(element);
// flex ususally means there are multiple elements in the div as a line or a column
// flex usually means there are multiple elements in the div as a line or a column
// if the children elements are not just one, we should keep it in the HTML tree to represent a tree structure
if (style?.display === "flex" && children.length > 1) {
return true;
@@ -1479,7 +1479,7 @@ async function buildElementTree(
return;
}
// skip proccessing option element as they are already added to the select.options
// skip processing option element as they are already added to the select.options
if (tagName === "option") {
return;
}
@@ -1499,7 +1499,7 @@ async function buildElementTree(
}
let shadowDOMchildren = [];
// sometimes the shadowRoot is not visible, but the elemnets in the shadowRoot are visible
// sometimes the shadowRoot is not visible, but the elements in the shadowRoot are visible
if (element.shadowRoot) {
shadowDOMchildren = getChildElements(element.shadowRoot);
}
@@ -1530,7 +1530,7 @@ async function buildElementTree(
(isParentSVG = element.closest("svg")) &&
isParentSVG.getAttribute("unique_id")
) {
// if elemnet is the children of the <svg> with an unique_id
// if element is the children of the <svg> with an unique_id
elementObj = await buildElementObject(frame, element, interactable);
} else if (tagName === "div" && isDOMNodeRepresentDiv(element)) {
elementObj = await buildElementObject(frame, element, interactable);
@@ -2586,7 +2586,7 @@ async function getIncrementElements(wait_until_finished = true) {
}
}
// cleanup the chidren tree, remove the duplicated element
// cleanup the children tree, remove the duplicated element
// search starting from the shallowest node:
// 1. if deeper, the node could only be the children of the shallower one or no related one.
// 2. if depth is same, the node could only be duplicated one or no related one.

4
skyvern/webeye/utils/dom.py
View File

@@ -844,8 +844,8 @@ class DomUtil:
raise MissingElement(selector=css, element_id=element_id)
else:
# WARNING: current xpath is based on the tag name.
# It can only represent the element possition in the DOM tree with tag name, it's not 100% reliable.
# As long as the current possition has the same element with the tag name, the locator can be found.
# It can only represent the element position in the DOM tree with tag name, it's not 100% reliable.
# As long as the current position has the same element with the tag name, the locator can be found.
# (maybe) we should validate the element hash to make sure the element is the same?
LOG.warning("Fallback to locator element by xpath.", xpath=xpath, element_id=element_id)
locator = frame_content.locator(f"xpath={xpath}")