Skip to content

Conversation

@cmworkato
Copy link
Collaborator

Problem

Commands with ID arguments were failing with validation error:
Input should be a valid integer [type=int_type, input_value='67040776', input_type=str]

Solution

Added type=int to all numeric ID arguments in click options:

Recipe Commands

  • workato recipes start --id
  • workato recipes stop --id
  • workato recipes start --folder-id
  • workato recipes stop --folder-id

Connection Commands

  • workato connections update --connection-id
  • workato connections pick-list --id
  • workato connections list --parent-id

Testing

✅ Verified commands now accept string input and convert to integers properly
✅ No more type validation errors
✅ All other ID options already had correct typing

Files Changed

  • src/workato_platform_cli/cli/commands/recipes/command.py
  • src/workato_platform_cli/cli/commands/connections.py

- Add type=int to recipe start/stop --id options
- Add type=int to recipe start/stop --folder-id options
- Add type=int to connection --connection-id and --id options
- Add type=int to connection --parent-id option

Resolves validation error: 'Input should be a valid integer [type=int_type, input_value='67040776', input_type=str]'
@github-actions
Copy link

github-actions bot commented Nov 3, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py68494%10–11, 69–70
cli
   __init__.py52394%49, 51, 60
   containers.py270100% 
cli/commands
   __init__.py00100% 
   api_clients.py291996%27, 302, 448–449, 483, 493, 584, 615, 623
   api_collections.py257398%28, 183, 347
   assets.py47295%55–56
   connections.py526599%591, 593, 599, 637, 988
   data_tables.py165596%31, 253, 267, 321–322
   guide.py166199%106
   init.py900100% 
   profiles.py2030100% 
   properties.py97198%21
   pull.py172298%193–194
   workspace.py39294%61, 71
cli/commands/connectors
   __init__.py00100% 
   command.py90297%110, 159
   connector_manager.py203498%176, 292, 300–301
cli/commands/projects
   __init__.py00100% 
   command.py2721096%359–362, 373, 439–441, 491, 495
   project_manager.py166795%48, 66, 263–264, 276, 317, 325
cli/commands/push
   __init__.py00100% 
   command.py133496%109, 112, 230, 308
cli/commands/recipes
   __init__.py00100% 
   command.py427997%117, 133–134, 272–275, 403, 709
   validator.py7062097%174, 883, 1136, 1223, 1246, 1279, 1281–1282, 1359–1361, 1457–1458, 1517–1518, 1707–1708, 1736–1738
cli/utils
   __init__.py30100% 
   exception_handler.py3042591%134–135, 140–141, 143–144, 170–171, 177, 220, 267, 294, 321, 378, 413, 472, 474–475, 480–481, 483–487
   gitignore.py140100% 
   ignore_patterns.py230100% 
   spinner.py430100% 
   version_checker.py135695%24, 26, 33–34, 72, 102
cli/utils/config
   __init__.py50100% 
   manager.py4671995%125, 136–138, 141, 154, 204–205, 357, 438–439, 478, 692, 835–836, 850–851, 912, 971
   models.py330100% 
   profiles.py3091096%93, 189–190, 193, 228–230, 255–257
   workspace.py680100% 
TOTAL560115397% 

@oalami oalami self-requested a review November 4, 2025 01:16
Copy link
Collaborator

@oalami oalami left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@oalami oalami merged commit 54f215b into main Nov 4, 2025
5 checks passed
@oalami oalami deleted the bugfix/id-must-be-int branch November 4, 2025 01:17
@cmworkato cmworkato restored the bugfix/id-must-be-int branch November 4, 2025 01:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants