remove dead code

[KRRT7]

No description provided.

[KRRT7]

tbh it wasn't very good and I did most of this with warp.

[claude]

Claude finished @KRRT7's task in 9m 18s —— View job

---

PR Review Summary

• Gather context
• Triage PR scope
• Lint and typecheck (7 ruff issues fixed, pushed)
• Resolve stale threads (none found)
• Code review
• Duplicate detection
• Test coverage
• Post summary

---

Prek Checks

7 ruff issues auto-fixed and pushed (style: auto-fix ruff import sorting and remove redundant noqa comments):

• I001 (unsorted-imports) in 5 files: github_workflow.py, init_auth.py, init_java.py, init_javascript.py, github_utils.py — each had two separate from codeflash.cli_cmds.console import lines (one for apologize_and_exit, one for the existing symbols). These should be merged into a single import line.
• RUF100 (unused-noqa) in registry.py — the # noqa: F401 added to the python/javascript support imports is redundant because the as _ pattern already suppresses the F401 warning.

No mypy errors were introduced by this PR (all 46 errors across 7 files were pre-existing).

---

Code Review

Bug fixed: broken test import (pushed as fix: remove test for deleted create_pyproject_toml function)

tests/test_setup/test_config.py imported create_pyproject_toml and had a TestCreatePyprojectToml class testing it — but the function was deleted from codeflash/setup/config_writer.py in this PR. This caused an ImportError at collection time, breaking the entire test module (20 tests affected). The import and test class have been removed.

Other observations (no action required):

• codeflash/api/aiservice.py: docstring on code_repair was silently removed along with the dead code above it. Not a bug but worth noting.
• codeflash/languages/registry.py L53–59: mypy reports Name "_" already defined (by an import) for the three import support as _ patterns — pre-existing, not introduced here.
• The apologize_and_exit move to console.py is clean and correct.

---

Duplicate Detection

No duplicates detected. The only added code (apologize_and_exit in console.py) has a single definition with no equivalent elsewhere.

---

Test Coverage

config_writer.py: 62% — acceptable. console.py (18%) and registry.py (16%) are pre-existing low-coverage files, not impacted by this PR's changes.

---

Bot PR

Merged open codeflash-ai[bot] PR #1853 (⚡️ Speed up re_extract_from_cache by 18,223% — all CI passed, mergeable).

[KRRT7]

@claude fix the mypy issue

[claude]

Claude finished @KRRT7's task in 2m 37s —— View job

---

Fixed mypy issue in registry.py

• Gather context
• Read registry.py and understand the issue
• Fix the Name "_" already defined mypy error
• Run prek to verify — all checks pass
• Commit and push

Root cause: _ensure_languages_registered() used from X import support as _ three times, causing mypy to report Name "_" already defined (by an import) on lines 56 and 59.

Fix: Replaced the three separate import as _ statements with a loop using importlib.import_module():

for _lang_module in (
    "codeflash.languages.python.support",
    "codeflash.languages.javascript.support",
    "codeflash.languages.java.support",
):
    with contextlib.suppress(ImportError):
        importlib.import_module(_lang_module)

mypy now reports no issues found for registry.py. Pushed as fix: resolve mypy "Name _ already defined" error in registry.py.

[KRRT7]

@claude fix mypy issue

[claude]

Claude finished @KRRT7's task in 4m 5s —— View job

---

Fixed mypy CI failure

• Gather context
• Check current mypy/prek status
• Remove deleted file from mypy_allowlist.txt
• Commit and push

Root cause: mypy_allowlist.txt still referenced codeflash/cli_cmds/cli_common.py, which was deleted in this PR. CI was failing immediately with:

mypy: can't read file 'codeflash/cli_cmds/cli_common.py': No such file or directory

Fix: Removed the deleted file from mypy_allowlist.txt (line 36). Pushed as fix: remove deleted cli_common.py from mypy_allowlist.txt.
| Branch