-
Notifications
You must be signed in to change notification settings - Fork 6
Open
Labels
enhancementNew feature or requestNew feature or request
Description
CodeRabbit
Consider improving import detection robustness.
The current regex-based approach for detecting imports has limitations:
Won't detect multi-line imports like from package import (a, b, c)
Could match imports within strings or comments
Doesn't handle __import__() or importlib usage
Consider using a proper Python AST parser in the future for more accurate import detection. For now, document these limitations in a comment:
analyzeImports(code: string): string[] {
+ // Note: This regex-based approach has limitations and won't catch all import patterns
+ // Future improvement: Use Python AST parsing for accurate import detection
const lines = code.split('\n');
const requiredPackages = new Set<string>();if bots catch the stuff i was pushing under the rug... DavidGoodenough
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request