Skip to content

Conversation

@hovu96
Copy link
Contributor

@hovu96 hovu96 commented Nov 7, 2025

Summary

Fixes incorrect package name references throughout the codebase by replacing all occurrences of workato_platform with the correct package name workato_platform_cli.

Changes Made

  • docs/DEVELOPER_GUIDE.md: Updated project structure paths and command examples (4 fixes)
  • scripts/parse_connection_docs.py: Updated output file paths (2 fixes)
  • src/workato_platform_cli/cli/commands/recipes/validator.py: Updated cache directory path (1 fix)
  • .pre-commit-config.yaml: Updated exclude patterns for all pre-commit hooks (5 fixes)

Impact

This resolves inconsistencies that could lead to:

  • Confusion for new developers reading incorrect documentation
  • Pre-commit hooks not properly excluding generated client code
  • Scripts writing output to incorrect directory paths
  • Cache files stored in wrong location

Test Plan

  • All pre-commit hooks pass
  • Documentation now reflects actual package structure
  • Script paths point to correct directories
  • Cache path uses correct directory name

Related

🤖 Generated with Claude Code

Replace all occurrences of 'workato_platform' with 'workato_platform_cli' to match the actual package name.

Changes:
- docs/DEVELOPER_GUIDE.md: Update project structure paths and command example (4 fixes)
- scripts/parse_connection_docs.py: Update output file paths (2 fixes)
- src/workato_platform_cli/cli/commands/recipes/validator.py: Update cache directory path (1 fix)
- .pre-commit-config.yaml: Update exclude patterns for all hooks (5 fixes)

This fixes inconsistencies in documentation, pre-commit hook exclusions, script output paths, and cache file locations.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions
Copy link

github-actions bot commented Nov 7, 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.py1070100% 
   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__.py40100% 
   exception_handler.py3182791%134–135, 140–141, 143–144, 174–175, 181, 184–185, 228, 275, 302, 329, 386, 421, 480, 482–483, 488–489, 491–495
   gitignore.py140100% 
   ignore_patterns.py230100% 
   spinner.py430100% 
   token_input.py451762%70, 98, 100, 102–104, 107, 124, 127–128, 132–133, 136, 139–140, 142, 144
   version_checker.py135695%24, 26, 33–34, 72, 102
cli/utils/config
   __init__.py50100% 
   manager.py5443493%127, 138, 149, 157–159, 162, 165, 177, 227–228, 399, 417, 421, 424–427, 443, 464, 478, 491, 533, 574, 652–653, 690, 904, 1047–1048, 1062–1063, 1119, 1178
   models.py330100% 
   profiles.py3091096%93, 189–190, 193, 228–230, 255–257
   workspace.py680100% 
TOTAL575518796% 

@hovu96 hovu96 self-assigned this Nov 7, 2025
@hovu96 hovu96 requested a review from j-madrone November 7, 2025 08:29
@hovu96 hovu96 merged commit a8f880c into main Nov 8, 2025
5 checks passed
@hovu96 hovu96 deleted the bug/DEVP-508-incorrect-package-name-references-throughout-codebase branch November 8, 2025 06:28
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