fix: remove 15 orphaned compatibility bridge files#23
Conversation
All 15 deleted files are underscore-prefixed compatibility shims in desloppify/languages/_framework/treesitter/ that re-exported symbols from their canonical grouped-namespace locations via a _compat_bridge helper. The package __init__.py docstring explicitly flags these as \"compatibility shims only\" and instructs new code to import from the grouped namespaces. Static analysis confirms zero direct importers for any of the 15 files. Deleted: _compat_bridge.py (infrastructure for the shims) _complexity_function_metrics.py → analysis.complexity_function_metrics _complexity_nesting.py → analysis.complexity_nesting _extractors.py → analysis.extractors _import_cache.py → imports.resolver_cache _import_graph.py → imports.graph _import_resolvers_backend.py → imports.resolvers_backend _import_resolvers_functional.py → imports.resolvers_functional _import_resolvers_scripts.py → imports.resolvers_scripts _normalize.py → imports.normalize _smells.py → analysis.smells _specs_compiled.py → specs.compiled _specs_functional.py → specs.functional _specs_scripting.py → specs.scripting _unused_imports.py → analysis.unused_imports
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 24 minutes and 31 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughDeleted a centralized compatibility bridge utility ( Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
Two import blocks were unsorted. Fix with ruff --fix to satisfy I001.
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
Summary
desloppify/languages/_framework/treesitter/analysis.extractors,imports.graph,specs.compiled)__init__.pydocstring explicitly describes these as "compatibility shims only" and directs new code to the grouped namespacesdesloppify.languages._framework.treesitterstill imports cleanly after deletionDeleted files and their canonical replacements
_compat_bridge.py_complexity_function_metrics.pyanalysis.complexity_function_metrics_complexity_nesting.pyanalysis.complexity_nesting_extractors.pyanalysis.extractors_import_cache.pyimports.resolver_cache_import_graph.pyimports.graph_import_resolvers_backend.pyimports.resolvers_backend_import_resolvers_functional.pyimports.resolvers_functional_import_resolvers_scripts.pyimports.resolvers_scripts_normalize.pyimports.normalize_smells.pyanalysis.smells_specs_compiled.pyspecs.compiled_specs_functional.pyspecs.functional_specs_scripting.pyspecs.scripting_unused_imports.pyanalysis.unused_importsTest plan
python3.11 -m desloppify scan --skip-slow --profile cipost-mergeSummary by CodeRabbit