Fix cmd init

[misrasaurabh1]

PR Type

Enhancement

---

Description

• Update usage instructions in init_codeflash command

• Simplify inquirer.Path prompts arguments

• Conditional telemetry disabling in pyproject.toml

• Remove redundant formatting lines

---

Changes walkthrough 📝

	Relevant files
Enhancement	cmd_init.py Refine CLI init and telemetry logic                                            codeflash/cli_cmds/cmd_init.py Updated usage_table entries for init command Removed normalize_to_absolute_path arguments Simplified custom tests prompt configuration Made telemetry disabling conditional +6/-13

---

Need help?

Type /help how to ... in the comments thread for any questions about PR-Agent usage.

Check out the documentation for more information.

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Usage Instructions The --all command usage now lacks the dynamic module_string interpolation, which might mislead users when a specific module is targeted. Ensure module-specific flags are explained consistently. usage_table.add_row("codeflash optimize <myscript.py>", "Trace and find the best optimizations for a script") usage_table.add_row("codeflash --all", "Optimize all functions in all files") Path Prompt Exists The new inquirer.Path for custom_tests_path no longer specifies exists=False, so it may require an existing directory and prevent creating new test directories. Consider re-adding or adjusting the exists parameter. "custom_tests_path", message="Enter the path to your tests directory", path_type=inquirer.Path.DIRECTORY ) Path Normalization Removing normalize_to_absolute_path could lead to inconsistent path resolutions. Verify that relative paths are handled correctly across platforms. inquirer.Path( "custom_path", message="Enter the path to your module directory", path_type=inquirer.Path.DIRECTORY, exists=True, )

[github-actions]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
Possible issue	Include module suffix in command Restore the dynamic module suffix so the --all command still reflects any scoped module selection. This ensures consistency with prior behavior when module_string is non-empty. codeflash/cli_cmds/cmd_init.py [109] -usage_table.add_row("codeflash --all", "Optimize all functions in all files") +usage_table.add_row( + f"codeflash --all{module_string if module_string else ''}", + "Optimize all functions in all files" +) Suggestion importance[1-10]: 9 __ Why: The dynamic --all command lost the module_string suffix, reintroducing it restores previous scoped behavior and prevents regression.	High
	Allow new test directories Re-enable exists=False and normalize_to_absolute_path=False so users can specify non-existent test directories and keep relative paths. Without these, inquirer will only accept existing directories and always normalize to absolute paths. codeflash/cli_cmds/cmd_init.py [339] -inquirer.Path("custom_tests_path", message="Enter the path to your tests directory", path_type=inquirer.Path.DIRECTORY) +inquirer.Path( + "custom_tests_path", + message="Enter the path to your tests directory", + path_type=inquirer.Path.DIRECTORY, + exists=False, + normalize_to_absolute_path=False, +) Suggestion importance[1-10]: 8 __ Why: Restoring exists=False and normalize_to_absolute_path=False preserves the ability to specify non-existent test directories and maintain relative paths, matching original flexibility.	Medium
General	Always include disable-telemetry Always set the disable-telemetry key to avoid missing configuration entries. Unconditionally assigning not enable_telemetry preserves the original semantics in both branches. codeflash/cli_cmds/cmd_init.py [931-932] -if not enable_telemetry: - codeflash_section["disable-telemetry"] = not enable_telemetry +codeflash_section["disable-telemetry"] = not enable_telemetry Suggestion importance[1-10]: 5 __ Why: Unconditionally setting disable-telemetry ensures the key is always present, but this is a minor configuration consistency improvement.	Low