[CG-7930] feat: remove unused imports after moving symbol & new api for removing unused symbols

[tkfoss]

• core/symbol.py changes related to moving and removing symbols and imports
  • updated move_to_file, _move_to_file
• typescript/symbol.py - refactor out generic parts, add unused import cleanup, implement typescript specific edge cases
• Add Python remove_unused_imports (backend for remove_unused_exports)
• Add Python import statement analysis methods
  • module_name: Extract module name from import statements
  • is_from_import: Detect 'from' style imports
  • is_star_import: Identify wildcard imports (from x import *)
  • is_future_import: Check for __future__ imports
• Add Python test suite for file operations and import management
  • Test splitting test files into multiple files
  • Verify correct function movement with dependencies
  • Check import cleanup during file splitting
  • Validate directory structure creation
  • Test basic unused import removal
  • Handle multiline import statements
  • Preserve used imports with aliases
  • Maintain comments during import cleanup
  • Support relative imports
  • Handle star imports (*) preservation
  • Verify type hint imports
  • Test multiple removal passes
• Add comprehensive test suite for TypeScript file moves
  • Basic function moves
  • Import handling (named, type, dynamic)
  • Decorators and comments
  • JSX components
  • Module augmentation
  • Re-export chains
  • Generic types
  • Move strategies

[linear]

CG-7930 Remove now-unused imports after moving symbol out of file

After moving a symbol (function or class) out of a file, if there are imports that are now unused because that was the only thing using them, remove those as well

symbol.move_to_file(..., remove_unused_imports=True)

[CLAassistant]

All committers have signed the CLA.

[codecov]

Codecov Report

Attention: Patch coverage is 57.85047% with 451 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
...en/sdk/typescript/move_symbol_to_file/test_move.py	41.50%	396 Missing ⚠️
...it/codegen/sdk/typescript/file/test_file_remove.py	73.23%	19 Missing ⚠️
...cript/move_symbol_to_file/test_move_tsx_to_file.py	11.11%	16 Missing ⚠️
src/codegen/sdk/typescript/symbol.py	78.72%	10 Missing ⚠️
src/codegen/sdk/typescript/file.py	83.87%	5 Missing ⚠️
src/codegen/sdk/core/symbol.py	86.95%	3 Missing ⚠️
src/codegen/sdk/python/file.py	85.71%	1 Missing ⚠️
.../sdk/python/function/test_function_move_to_file.py	93.75%	1 Missing ⚠️

Additional details and impacted files

[bagel897]

Can we rely on self.module?

[tkfoss]

you mean use self.module instead of the .module_name? it seems to miss some imports then. or should module_name itself be changed?

[bagel897]

If it's missing imports, that may be a bug in the SDK. Generally we want to use our nodes rather than tree-sitters post-parse

[bagel897]

Can we use remove_if_unused and apply changes to that function? It's able to account for pending transactions which is critical if we do multiple moves

[bagel897]

If it's missing imports, that may be a bug in the SDK. Generally we want to use our nodes rather than tree-sitters post-parse

[bagel897]

Can we use import_type?

[tkfoss]

continued in #855