Fix false 'directory not empty' warning during init

[hovu96]

Summary

Fixes false "directory not empty" warning that appears when running workato init in a newly created directory.

Problem

When initializing a project with workato init, the CLI shows an incorrect warning:

⚠️ Directory '<path>' is not empty. Proceed with initialization? [y/N]

This warning appears even though the directory was just created by the CLI itself. The issue occurs because:

1. ConfigManager.initialize() creates the project directory and .workatoenv file
2. The emptiness check runs after initialization and detects .workatoenv
3. The check doesn't distinguish between CLI-created files and user files

Solution

Modified the directory emptiness check in init.py to filter out CLI-managed files before determining if a directory contains user files.

CLI-managed files (now ignored):

• .workatoenv - Configuration file
• .gitignore - Git ignore patterns
• .workato-ignore - Workato ignore patterns

Changes:

• Added cli_managed_files set to define files that should be ignored
• Changed from any(project_dir.iterdir()) to checking only non-CLI-managed files
• Ensures warnings only appear for actual user files

Benefits

✅ No false warnings during clean initialization
✅ Legitimate warnings still work for user files
✅ Supports idempotent re-initialization
✅ Enables recovery from partial initialization
✅ No breaking changes

Test Plan

Added 4 comprehensive unit tests:

1. test_init_cli_managed_files_ignored_interactive - Verifies CLI files ignored in interactive mode
2. test_init_cli_managed_files_ignored_non_interactive - Verifies CLI files ignored in non-interactive mode
3. test_init_user_files_with_cli_files_triggers_warning - Ensures user files still trigger warnings
4. test_init_only_workatoenv_file_ignored - Tests most common case (only .workatoenv present)

Coverage: 100% of changed code
Results: All 21 init tests passing

Files Changed

• src/workato_platform/cli/commands/init.py - 11 lines modified
• tests/unit/commands/test_init.py - 246 lines added (4 new tests)

Manual Testing

Manually tested the following scenarios:

• ✅ workato init with new directory - No false warning
• ✅ workato init with existing empty directory - No warning
• ✅ workato init with existing non-empty directory - Appropriate warning shown
• ✅ workato init re-run in same directory - No false warning for CLI-managed files

🤖 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	89	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	5450	132	97%

[Copilot]

Pull Request Overview

This PR fixes a false "directory not empty" warning that incorrectly appears when running workato init in a directory containing only CLI-managed configuration files. The issue occurred because the emptiness check didn't distinguish between files created by the CLI itself (like .workatoenv) and actual user files.

Key Changes:

• Modified the directory emptiness check to filter out CLI-managed files (.workatoenv, .gitignore, .workato-ignore)
• Added logic to only warn users when actual user files are present in the directory
• Added comprehensive test coverage for various initialization scenarios

Reviewed Changes

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

File	Description
src/workato_platform/cli/commands/init.py	Updated emptiness check to exclude CLI-managed files from consideration
tests/unit/commands/test_init.py	Added 4 new test cases covering CLI file handling and fixed lambda signatures in existing tests

---

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

[oalami]

LGTM. Applied some Copilot suggested fixes, you'll need to review and merge. @hovu96