Skip to content

added default value and optimized xpath #140

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
sauravpanda merged 2 commits into from
Nov 14, 2025
Merged

added default value and optimized xpath #140

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
584e919
added default value and optimized xpath
sauravpanda Nov 14, 2025
3dcf69f
fixed cubic issues
sauravpanda Nov 14, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
50 changes: 39 additions & 11 deletions workflows/cli.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1233,6 +1233,7 @@ async def _run_workflow():

var_type = input_def.type.lower() # type is a direct attribute
is_required = input_def.required
default_value = getattr(input_def, 'default', None)

type_info_str = f'type: {var_type}'
if is_required:
Expand All @@ -1245,21 +1246,32 @@ async def _run_workflow():
if hasattr(input_def, 'format') and input_def.format:
format_info_str = f', format: {typer.style(input_def.format, fg=typer.colors.GREEN)}'

full_prompt_text = f'{prompt_question} ({status_str}, {type_info_str}{format_info_str})'
# Add default value information if available
default_info_str = ''
if default_value is not None:
default_info_str = f', default: {typer.style(str(default_value), fg=typer.colors.BLUE)}'

full_prompt_text = f'{prompt_question} ({status_str}, {type_info_str}{format_info_str}{default_info_str})'

input_val = None
if var_type == 'bool':
input_val = typer.confirm(full_prompt_text)
input_val = typer.confirm(full_prompt_text, default=default_value if default_value is not None else None)
elif var_type == 'number':
input_val = typer.prompt(full_prompt_text, type=float)
input_val = typer.prompt(
full_prompt_text, type=float, default=default_value if default_value is not None else ...
)
elif var_type == 'string': # Default to string for other unknown types as well
input_val = typer.prompt(full_prompt_text, type=str)
input_val = typer.prompt(
full_prompt_text, type=str, default=default_value if default_value is not None else ...
)
else: # Should ideally not happen if schema is validated, but good to have a fallback
typer.secho(
f"Warning: Unknown type '{var_type}' for variable '{input_def.name}'. Treating as string.",
fg=typer.colors.YELLOW,
)
input_val = typer.prompt(full_prompt_text, type=str)
input_val = typer.prompt(
full_prompt_text, type=str, default=default_value if default_value is not None else ...
)

inputs[input_def.name] = input_val
typer.echo() # Add space after each prompt
Expand Down Expand Up @@ -1369,6 +1381,7 @@ async def _run_workflow_no_ai():

var_type = input_def.type.lower() # type is a direct attribute
is_required = input_def.required
default_value = getattr(input_def, 'default', None)

type_info_str = f'type: {var_type}'
if is_required:
Expand All @@ -1381,21 +1394,32 @@ async def _run_workflow_no_ai():
if hasattr(input_def, 'format') and input_def.format:
format_info_str = f', format: {typer.style(input_def.format, fg=typer.colors.GREEN)}'

full_prompt_text = f'{prompt_question} ({status_str}, {type_info_str}{format_info_str})'
# Add default value information if available
default_info_str = ''
if default_value is not None:
default_info_str = f', default: {typer.style(str(default_value), fg=typer.colors.BLUE)}'

full_prompt_text = f'{prompt_question} ({status_str}, {type_info_str}{format_info_str}{default_info_str})'

input_val = None
if var_type == 'bool':
input_val = typer.confirm(full_prompt_text)
input_val = typer.confirm(full_prompt_text, default=default_value if default_value is not None else None)
elif var_type == 'number':
input_val = typer.prompt(full_prompt_text, type=float)
input_val = typer.prompt(
full_prompt_text, type=float, default=default_value if default_value is not None else ...
)
elif var_type == 'string': # Default to string for other unknown types as well
input_val = typer.prompt(full_prompt_text, type=str)
input_val = typer.prompt(
full_prompt_text, type=str, default=default_value if default_value is not None else ...
)
else: # Should ideally not happen if schema is validated, but good to have a fallback
typer.secho(
f"Warning: Unknown type '{var_type}' for variable '{input_def.name}'. Treating as string.",
fg=typer.colors.YELLOW,
)
input_val = typer.prompt(full_prompt_text, type=str)
input_val = typer.prompt(
full_prompt_text, type=str, default=default_value if default_value is not None else ...
)

inputs[input_def.name] = input_val
typer.echo() # Add space after each prompt
Expand Down Expand Up @@ -2445,7 +2469,11 @@ def run_stored_workflow(
if inp.required
else typer.style('optional', fg=typer.colors.YELLOW)
)
typer.echo(f' • {inp.name} ({inp.type}, {required})')
default_value = getattr(inp, 'default', None)
default_str = (
f', default: {typer.style(str(default_value), fg=typer.colors.BLUE)}' if default_value is not None else ''
)
typer.echo(f' • {inp.name} ({inp.type}, {required}{default_str})')
typer.echo()
typer.echo('Options:')
typer.echo(f' 1. Run as tool: python cli.py run-stored-workflow {workflow_id} --prompt "Your task"')
Expand Down
2 changes: 1 addition & 1 deletion workflows/uv.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading
Loading