Fix subpackage import incorrectly removed when sibling subpackage is used (#180)

`import a.b` binds `a` in the namespace, so any `a.x.y` reference
relies on that binding. `is_match_sub_packages` was comparing the
import's root package against the full name string instead of just
its first component, causing attribute usages like `urllib.parse.urlparse`
to not match `import urllib.request`.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: hakancelikdev

src/unimport/statement.py

@@ -23,7 +23,7 @@ def __len__(self) -> int:
         return len(self.name.split("."))
 
     def is_match_sub_packages(self, name_name: str) -> bool:
-        return self.name.split(".")[0] == name_name
+        return self.name.split(".")[0] == name_name.split(".")[0]
 
     @property
     def scope(self):

tests/cases/analyzer/unused/subpackage_import_attribute_usage.py

@@ -0,0 +1,16 @@
+from typing import Union
+
+from unimport.statement import Import, ImportFrom, Name
+
+__all__ = ["NAMES", "IMPORTS", "UNUSED_IMPORTS"]
+
+
+NAMES: list[Name] = [
+    Name(lineno=4, name="urllib.parse.urlparse", is_all=False),
+    Name(lineno=4, name="urllib.parse", is_all=False),
+    Name(lineno=4, name="url", is_all=False),
+]
+IMPORTS: list[Union[Import, ImportFrom]] = [
+    Import(lineno=1, column=1, name="urllib.request", package="urllib.request"),
+]
+UNUSED_IMPORTS: list[Union[Import, ImportFrom]] = []

tests/cases/analyzer/unused/subpackage_import_same_branch.py

@@ -0,0 +1,16 @@
+from typing import Union
+
+from unimport.statement import Import, ImportFrom, Name
+
+__all__ = ["NAMES", "IMPORTS", "UNUSED_IMPORTS"]
+
+
+NAMES: list[Name] = [
+    Name(lineno=3, name="print", is_all=False),
+    Name(lineno=3, name="os.path.join", is_all=False),
+    Name(lineno=3, name="os.path", is_all=False),
+]
+IMPORTS: list[Union[Import, ImportFrom]] = [
+    Import(lineno=1, column=1, name="os.path", package="os.path"),
+]
+UNUSED_IMPORTS: list[Union[Import, ImportFrom]] = []

tests/cases/analyzer/unused/subpackage_import_simple_usage.py

@@ -0,0 +1,15 @@
+from typing import Union
+
+from unimport.statement import Import, ImportFrom, Name
+
+__all__ = ["NAMES", "IMPORTS", "UNUSED_IMPORTS"]
+
+
+NAMES: list[Name] = [
+    Name(lineno=3, name="print", is_all=False),
+    Name(lineno=3, name="os.getcwd", is_all=False),
+]
+IMPORTS: list[Union[Import, ImportFrom]] = [
+    Import(lineno=1, column=1, name="os.path", package="os.path"),
+]
+UNUSED_IMPORTS: list[Union[Import, ImportFrom]] = []

tests/cases/analyzer/unused/subpackage_import_unused.py

@@ -0,0 +1,16 @@
+from typing import Union
+
+from unimport.statement import Import, ImportFrom, Name
+
+__all__ = ["NAMES", "IMPORTS", "UNUSED_IMPORTS"]
+
+
+NAMES: list[Name] = [
+    Name(lineno=3, name="print", is_all=False),
+]
+IMPORTS: list[Union[Import, ImportFrom]] = [
+    Import(lineno=1, column=1, name="urllib.request", package="urllib.request"),
+]
+UNUSED_IMPORTS: list[Union[Import, ImportFrom]] = [
+    Import(lineno=1, column=1, name="urllib.request", package="urllib.request"),
+]

tests/cases/refactor/unused/subpackage_import_attribute_usage.py

@@ -0,0 +1,4 @@
+import urllib.request
+
+def parse_url(url):
+    return urllib.parse.urlparse(url)

tests/cases/refactor/unused/subpackage_import_same_branch.py

@@ -0,0 +1,3 @@
+import os.path
+
+print(os.path.join("a", "b"))

tests/cases/refactor/unused/subpackage_import_simple_usage.py

@@ -0,0 +1,3 @@
+import os.path
+
+print(os.getcwd())

tests/cases/refactor/unused/subpackage_import_unused.py

@@ -0,0 +1,2 @@
+
+print("hello")

tests/cases/source/unused/subpackage_import_attribute_usage.py

@@ -0,0 +1,4 @@
+import urllib.request
+
+def parse_url(url):
+    return urllib.parse.urlparse(url)