Add clarification re: format of start and end dates, and (probably?) fix non-determinism error caused by SHOW_CONFIRM

This commit is contained in:
Laine
2025-03-19 12:48:49 -04:00
parent 850404e0d5
commit 3debef5781
2 changed files with 9 additions and 8 deletions

View File

@@ -50,6 +50,7 @@ guess_location_tool = ToolDefinition(
), ),
], ],
) )
# ----- Travel use cases tools ----- # ----- Travel use cases tools -----
search_flights_tool = ToolDefinition( search_flights_tool = ToolDefinition(
name="SearchFlights", name="SearchFlights",
@@ -198,12 +199,12 @@ future_pto_calc_tool = ToolDefinition(
ToolArgument( ToolArgument(
name="start_date", name="start_date",
type="string", type="string",
description="Start date of proposed PTO", description="Start date of proposed PTO, sent in the form yyyy-mm-dd",
), ),
ToolArgument( ToolArgument(
name="end_date", name="end_date",
type="string", type="string",
description="End date of proposed PTO", description="End date of proposed PTO, sent in the form yyyy-mm-dd",
), ),
ToolArgument( ToolArgument(
name="email", name="email",
@@ -221,12 +222,12 @@ book_pto_tool = ToolDefinition(
ToolArgument( ToolArgument(
name="start_date", name="start_date",
type="string", type="string",
description="Start date of proposed PTO", description="Start date of proposed PTO, sent in the form yyyy-mm-dd",
), ),
ToolArgument( ToolArgument(
name="end_date", name="end_date",
type="string", type="string",
description="End date of proposed PTO", description="End date of proposed PTO, sent in the form yyyy-mm-dd",
), ),
ToolArgument( ToolArgument(
name="email", name="email",

View File

@@ -26,11 +26,11 @@ with workflow.unsafe.imports_passed_through():
# Constants # Constants
MAX_TURNS_BEFORE_CONTINUE = 250 MAX_TURNS_BEFORE_CONTINUE = 250
SHOW_CONFIRM = True
show_confirm_env = os.getenv("SHOW_CONFIRM") show_confirm_env = os.getenv("SHOW_CONFIRM")
if show_confirm_env is not None: if show_confirm_env is not None and show_confirm_env.lower() == "false":
if show_confirm_env == "False": SHOW_CONFIRM = False
SHOW_CONFIRM = False else:
SHOW_CONFIRM = True
#ToolData as part of the workflow is what's accessible to the UI - see LLMResponse.jsx for example #ToolData as part of the workflow is what's accessible to the UI - see LLMResponse.jsx for example
class ToolData(TypedDict, total=False): class ToolData(TypedDict, total=False):