Fix workato init to detect existing projects after user selection

[hovu96]

Summary

Fixed the project detection logic in workato init to check for existing projects AFTER the user selects which project they want to initialize, rather than prematurely prompting based on any .workatoenv in the current directory.

Problem

The old behavior would check for any .workatoenv file in the current directory BEFORE the user selected which project they wanted to initialize. This caused confusion when users had old project files from previous initializations that were unrelated to what they wanted to initialize.

Solution

• Removed premature check that looked for .workatoenv before user selection
• Added proper check after project selection using _find_all_projects()
• Detection now matches by project_id across entire workspace
• Interactive mode prompts for reinitialization if project exists
• Non-interactive mode fails with clear error if project exists
• Corrupted configs are gracefully skipped during detection

Changes Made

Files Modified

• src/workato_platform/cli/utils/config/manager.py

  • Removed lines 346-380 (premature check in _setup_project())
  • Added proper detection after project selection in both interactive and non-interactive modes
  • Detection uses _find_all_projects() and matches by project_id

• tests/unit/config/test_manager.py

  • Added 6 new comprehensive tests
  • Removed 3 obsolete tests testing old behavior
  • Fixed 1 existing test to work with new behavior

Test Coverage

Added comprehensive test coverage (6 new tests):

• No premature prompts with old .workatoenv files
• Detection after user selection
• User declining reinitialization
• Non-interactive mode error handling
• Corrupted config handling
• Matching by project_id instead of name

Test Results: All 899 tests passing

Quality Checks

✅ All type checks pass (mypy)
✅ All linter checks pass (ruff)
✅ Code properly formatted
✅ 96% test coverage on modified code
✅ All 899 tests passing

🤖 Generated with Claude Code

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
__init__.py	62	2	96%	12–13
cli
__init__.py	52	3	94%	49, 51, 60
containers.py	27	0	100%
cli/commands
__init__.py	0	0	100%
api_clients.py	286	9	96%	24, 297, 442–443, 476, 486, 576, 607, 615
api_collections.py	253	3	98%	25, 178, 340
assets.py	46	2	95%	51–52
connections.py	520	5	99%	578, 580, 586, 624, 973
data_tables.py	163	5	96%	28, 248, 262, 316–317
guide.py	166	1	99%	106
init.py	85	0	100%
profiles.py	197	0	100%
properties.py	95	1	98%	18
pull.py	171	2	98%	190–191
workspace.py	38	2	94%	57, 67
cli/commands/connectors
__init__.py	0	0	100%
command.py	88	2	97%	103, 152
connector_manager.py	203	4	98%	176, 292, 300–301
cli/commands/projects
__init__.py	0	0	100%
command.py	269	10	96%	353–356, 367, 433–435, 485, 489
project_manager.py	166	7	95%	48, 66, 263–264, 276, 317, 325
cli/commands/push
__init__.py	0	0	100%
command.py	132	4	96%	101, 104, 222, 300
cli/commands/recipes
__init__.py	0	0	100%
command.py	423	9	97%	113, 129–130, 267–270, 397, 703
validator.py	706	20	97%	174, 883, 1136, 1223, 1246, 1279, 1281–1282, 1359–1361, 1457–1458, 1517–1518, 1707–1708, 1736–1738
cli/utils
__init__.py	3	0	100%
exception_handler.py	198	6	96%	137, 184, 211, 238, 295, 330
gitignore.py	14	0	100%
ignore_patterns.py	23	0	100%
spinner.py	43	0	100%
version_checker.py	135	6	95%	24, 26, 33–34, 72, 102
cli/utils/config
__init__.py	5	0	100%
manager.py	467	19	95%	125, 136–138, 141, 154, 204–205, 357, 438–439, 478, 692, 835–836, 850–851, 912, 971
models.py	33	0	100%
profiles.py	309	10	96%	93, 189–190, 193, 228–230, 255–257
workspace.py	68	0	100%
TOTAL	5446	132	97%

[Copilot]

Pull Request Overview

Fixed workato init to detect existing projects after user selection instead of prematurely checking for .workatoenv files before the user chooses which project to initialize. This prevents confusion when old project files exist in the current directory that are unrelated to the project being initialized.

Key changes:

• Removed premature project detection check that occurred before user selection
• Added proper detection logic after project selection using _find_all_projects() and matching by project_id
• Enhanced error handling for both interactive and non-interactive modes when projects already exist

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/workato_platform/cli/utils/config/manager.py	Removed premature detection logic (lines 346-380), added post-selection detection in both _setup_project() and _setup_non_interactive() methods
tests/unit/config/test_manager.py	Removed 3 obsolete tests for old behavior, added 6 comprehensive tests for new detection logic, updated 1 existing test to work with new flow

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.