Skip to content

refactor: remove dead discover_functions implementations#1613

Closed
KRRT7 wants to merge 3 commits into
mainfrom
where-test-discovered-
Closed

refactor: remove dead discover_functions implementations#1613
KRRT7 wants to merge 3 commits into
mainfrom
where-test-discovered-

Conversation

@KRRT7
Copy link
Copy Markdown
Contributor

@KRRT7 KRRT7 commented Feb 20, 2026

Summary

  • Replace PythonSupport.discover_functions body with raise NotImplementedError — this method was dead code since the routing in find_all_functions_in_file always sends Python files to the AST-based _find_all_functions_in_python_file path
  • Replace JavaScriptSupport.discover_functions body with raise NotImplementedError — only reachable via _find_all_functions_via_language_support, which is gated off for Python; JS/TS optimization is not yet active through the CLI

Test plan

  • Verify existing Python optimization paths still work (--file, --all, -m pytest)
  • Run existing test suite to confirm no regressions

@KRRT7 KRRT7 closed this Feb 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant