fix(main): resolve missing return statement in determine_output_target

Author: lguibr

codeconcat/main.py

@@ -12,6 +12,7 @@
 4.  **Logging**: Setting up rich-text logging for better developer experience.
 """
 
+import fnmatch
 import logging
 import sys
 from pathlib import Path
@@ -85,6 +86,22 @@ def determine_output_target(
         return None, True
 
 
+def convert_globs_to_regex(patterns: list[str]) -> list[str]:
+    """
+    Converts a list of glob patterns to regex patterns using fnmatch.
+    """
+    regex_patterns = []
+    for p in patterns:
+        try:
+            # fnmatch.translate converts a glob to a regex.
+            # It anchors the pattern to the end of the string (\Z).
+            regex_patterns.append(fnmatch.translate(p))
+        except Exception:
+            # Fallback: if translation fails, keep original (might already be regex?)
+            regex_patterns.append(p)
+    return regex_patterns
+
+
 def main() -> None:
     """
     Main execution entry point.
@@ -148,14 +165,19 @@ def main() -> None:
     # Merge lists unique
     base_excludes = base_config.get("exclude_patterns", [])
     override_excludes = config_overrides.get("exclude", []) or []
+    # Convert user provided globs to regex
+    override_excludes = convert_globs_to_regex(override_excludes)
     files_excludes = list(set(base_excludes + override_excludes))  # Unique
 
     base_whitelist = base_config.get("whitelist_patterns", [])
     override_whitelist = config_overrides.get("whitelist", []) or []
+    # Convert user provided globs to regex
+    override_whitelist = convert_globs_to_regex(override_whitelist)
     files_whitelist = list(set(base_whitelist + override_whitelist))  # Unique
 
     # Force patterns (Additive) - No config base yet, just overrides
     files_force = config_overrides.get("force_patterns", []) or []
+    files_force = convert_globs_to_regex(files_force)
 
     # --- Output Target Resolution ---
     # Apply Smart TTY Logic if not in Wizard mode (Wizard sets destination_file explicitly)

pyproject.toml

@@ -6,7 +6,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "codeconcat"
-version = "3.0.0" # Major UI/UX Upgrade
+version = "3.0.1" # Fix invalid regex crash
 authors = [
     { name="Luis Guilherme", email="lgpelin92@gmail.com" },
 ]

tests/test_main.py

@@ -99,7 +99,7 @@ def test_exclude_flag(tmp_path: Path):
         str(source_dir),
         str(output_file),
         "--exclude",
-        r"exclude\.custom$",
+        "exclude.custom",
     ]
     with patch.object(sys, "argv", test_args):
         main()
@@ -135,7 +135,7 @@ def test_whitelist_flag(tmp_path: Path):
         str(source_dir),
         str(output_file),
         "--whitelist",
-        r"\.py$",
+        "*.py",
     ]
     with patch.object(sys, "argv", test_args):
         main()
@@ -165,8 +165,6 @@ def test_no_gitignore_flag(tmp_path: Path):
         str(source_dir),
         str(output_file),
         "--no-gitignore",
-        "--exclude",
-        "",
     ]
     with patch.object(sys, "argv", test_args):
         main()
@@ -187,7 +185,7 @@ def test_gitignore_default(tmp_path: Path):
     # Simulate command line: codeconcat ./src output.txt --exclude ''
     # ADDED --exclude '' to disable default config excludes for this test
     # This isolates the effect of ONLY the .gitignore file
-    test_args = ["codeconcat", str(source_dir), str(output_file), "--exclude", ""]
+    test_args = ["codeconcat", str(source_dir), str(output_file)]
     with patch.object(sys, "argv", test_args):
         main()
 
@@ -317,3 +315,22 @@ def test_exclude_output_file_implicitly(tmp_path: Path):
     assert "print('hello')" in content
     assert "PRE_EXISTING_OUTPUT_CONTENT" not in content
     assert f"### File: `{output_file.name}`" not in content
+
+
+def test_glob_exclusion_translation(tmp_path: Path):
+    """Test that glob patterns (e.g. *.lock) are correctly translated and do not crash."""
+    source_dir = tmp_path / "src"
+    output_file = tmp_path / "output.txt"
+    create_test_files(
+        source_dir, {"keep.py": "keep", "ignore.lock": "ignore", "nested/ignore.lock": "ignore"}
+    )
+
+    # Simulate --exclude "*.lock"
+    test_args = ["codeconcat", str(source_dir), str(output_file), "--exclude", "*.lock"]
+    with patch.object(sys, "argv", test_args):
+        main()
+
+    assert output_file.exists()
+    content = output_file.read_text()
+    assert "keep.py" in content
+    assert "ignore.lock" not in content