ensure codeflash doesn't crash during init

[KRRT7]

PR Type

Bug fix

---

Description

• Fix tuple type annotation syntax

• Safeguard default tests directory creation

---

Diagram Walkthrough

flowchart LR
  A["CLI init flow"] -- "create tests dir" --> B["Use default or 'tests' fallback"]
  C["Helper typing"] -- "correct tuple typing" --> D["Valid annotations"]

Loading

File Walkthrough

	Relevant files
Formatting	cfapi.py Trim superfluous module docstring                                                codeflash/api/cfapi.py Remove redundant module docstring. +0/-2
Bug fix	cmd_init.py Typing fix and robust tests dir fallback                                  codeflash/cli_cmds/cmd_init.py Fix typing: use tuple[list[str], Optional[str]]. Add fallback to "tests" when default is None/empty. +2/-2

---

[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 Directory creation robustness Consider using mkdir(parents=True, exist_ok=True) to avoid errors when the directory already exists or parent dirs are missing, especially since a fallback "tests" may already be present. tests_root = Path(curdir) / (default_tests_subdir or "tests") tests_root.mkdir() click.echo(f"✅ Created directory {tests_root}{os.path.sep}{LF}") Type annotation check Confirm that Optional is imported in this module; the new return annotation tuple[list[str], Optional[str]] relies on it and could cause a NameError if not imported. def get_suggestions(section: str) -> tuple[list[str], Optional[str]]: valid_subdirs = get_valid_subdirs() if section == CommonSections.module_root: return [d for d in valid_subdirs if d != "tests"], None if section == CommonSections.tests_root:

[github-actions]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
Possible issue	Make directory creation idempotent If the directory already exists, mkdir() will raise FileExistsError and crash init. Use exist_ok=True to make the operation idempotent and avoid failures during repeated runs. codeflash/cli_cmds/cmd_init.py [394-395] tests_root = Path(curdir) / (default_tests_subdir or "tests") -tests_root.mkdir() +tests_root.mkdir(exist_ok=True) Suggestion importance[1-10]: 7 __ Why: Using exist_ok=True prevents a FileExistsError if the target directory already exists, improving robustness without changing behavior otherwise.	Medium
	Fix incompatible return annotation Using tuple[list[str], Optional[str]] directly is invalid in Python versions prior to 3.9/3.10 without from future import annotations or typing.Tuple. To avoid runtime/type-checking issues, switch to Tuple[...] from typing, which is broadly compatible. codeflash/cli_cmds/cmd_init.py [267] -def get_suggestions(section: str) -> tuple[list[str], Optional[str]]: +from typing import Tuple, Optional +def get_suggestions(section: str) -> Tuple[list[str], Optional[str]]: + Suggestion importance[1-10]: 2 __ Why: The suggestion is unnecessary because the project uses from __future__ import annotations (seen in codeflash/api/cfapi.py), which makes tuple[...] valid; also, the change would require updating imports not shown here.	Low