fix: resolve CI failure and SonarQube issues

Author: nikolay-e

src/treemapper/cli.py

@@ -42,9 +42,7 @@ def _validate_budget(budget: int | None) -> None:
 
 
 def _validate_alpha(alpha: float) -> None:
-    if alpha <= 0:
-        _exit_error(f"--alpha must be between 0 and 1 (exclusive), got {alpha}")
-    if alpha >= 1:
+    if not (0 < alpha < 1):
         _exit_error(f"--alpha must be between 0 and 1 (exclusive), got {alpha}")
 
 

src/treemapper/diffctx/edges/semantic/javascript.py

@@ -13,7 +13,7 @@
 _TS_EXTS = {".ts", ".tsx", ".mts", ".cts"}
 
 _EXPORT_DECL_RE = re.compile(
-    r"export\s+(?:(?:const|let|var|function\*?|class|async\s+function|interface|type|enum|abstract\s+class)\s+)(\w+)",
+    r"export\s+(?:const|let|var|function\*?|class|async\s+function|interface|type|enum|abstract\s+class)\s+(\w+)",
     re.MULTILINE,
 )
 _EXPORT_DEFAULT_NAME_RE = re.compile(

src/treemapper/diffctx/parsers/text.py

@@ -4,7 +4,7 @@
 from pathlib import Path
 
 from ..types import Fragment, FragmentId, extract_identifiers
-from .base import GENERIC_MAX_LINES, MIN_FRAGMENT_WORDS, check_library_available, create_snippet, find_sentence_boundary
+from .base import GENERIC_MAX_LINES, check_library_available, create_snippet, find_sentence_boundary
 
 logger = logging.getLogger(__name__)
 
@@ -97,9 +97,6 @@ def _flush_paragraph(
     ) -> None:
         if not para_sentences:
             return
-        combined = " ".join(para_sentences)
-        if len(combined.split()) < MIN_FRAGMENT_WORDS:
-            return
         frag = self._create_paragraph_fragment(path, lines, para_start, end_line)
         if frag:
             fragments.append(frag)

src/treemapper/diffctx/utility.py

@@ -176,9 +176,8 @@
 
 _COMMENT_PREFIXES = ("#", "//", "*", "/*", "--", '"""', "'''", "<!--")
 
-_EXTERNAL_IMPORT_RE = re.compile(
-    r"""(?:import\s+\{([^}]+)\}\s+from\s+['"]([^'"]+)['"]|from\s+(\S+)\s+import\s+(.+))""",
-)
+_JS_IMPORT_RE = re.compile(r"""import\s+\{([^}]+)\}\s+from\s+['"]([^'"]+)['"]""")
+_PY_IMPORT_RE = re.compile(r"""from\s+(\S+)\s+import\s+(.+)""")
 
 
 def _parse_import_names(names_str: str) -> set[str]:
@@ -193,12 +192,13 @@ def _parse_import_names(names_str: str) -> set[str]:
 def _collect_external_symbols(diff_text: str) -> frozenset[str]:
     symbols: set[str] = set()
     for line in _extract_changed_lines(diff_text):
-        for m in _EXTERNAL_IMPORT_RE.finditer(line):
+        for m in _JS_IMPORT_RE.finditer(line):
             js_names, js_source = m.group(1), m.group(2)
-            py_module, py_names = m.group(3), m.group(4)
-            if js_names and js_source and not js_source.startswith("."):
+            if not js_source.startswith("."):
                 symbols.update(_parse_import_names(js_names))
-            if py_module and py_names and not py_module.startswith("."):
+        for m in _PY_IMPORT_RE.finditer(line):
+            py_module, py_names = m.group(1), m.group(2)
+            if not py_module.startswith("."):
                 symbols.update(_parse_import_names(py_names))
     return frozenset(symbols)