From a67438bdcc2778e285644ee174165cb7ab567836 Mon Sep 17 00:00:00 2001
From: yusyus <yusufkaraaslan.yk@pm.me>
Date: Tue, 3 Feb 2026 21:00:34 +0300
Subject: [PATCH] fix: Update test version checks to 2.9.0 and remove
 whitespace

- Update version checks in test_package_structure.py from 2.8.0 to 2.9.0
- Update version check in test_cli_paths.py from 2.8.0 to 2.9.0
- Remove trailing whitespace from blank lines in code_analyzer.py (lines 1436-1504)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
---
 src/skill_seekers/cli/code_analyzer.py | 20 ++++++++++----------
 tests/test_cli_paths.py                |  2 +-
 tests/test_package_structure.py        |  8 ++++----
 3 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/src/skill_seekers/cli/code_analyzer.py b/src/skill_seekers/cli/code_analyzer.py
index 1ab5e01..bcc4da4 100644
--- a/src/skill_seekers/cli/code_analyzer.py
+++ b/src/skill_seekers/cli/code_analyzer.py
@@ -1433,7 +1433,7 @@ class CodeAnalyzer:
     def _analyze_godot_scene(self, content: str, file_path: str) -> dict[str, Any]:
         """
         Analyze Godot .tscn scene file.
-        
+
         Extracts:
         - Node hierarchy
         - Script attachments
@@ -1443,7 +1443,7 @@ class CodeAnalyzer:
         nodes = []
         resources = []
         scripts = []
-        
+
         # Extract external resources
         for match in re.finditer(r'\[ext_resource.*?type="(.+?)".*?path="(.+?)".*?id="(.+?)"\]', content):
             res_type, path, res_id = match.groups()
@@ -1452,28 +1452,28 @@ class CodeAnalyzer:
                 "path": path,
                 "id": res_id
             })
-            
+
             # Track scripts separately
             if res_type == "Script":
                 scripts.append({
                     "path": path,
                     "id": res_id
                 })
-        
+
         # Extract nodes
         for match in re.finditer(r'\[node name="(.+?)".*?type="(.+?)".*?\]', content):
             node_name, node_type = match.groups()
-            
+
             # Check if node has a script attached
             script_match = re.search(rf'\[node name="{re.escape(node_name)}".*?script = ExtResource\("(.+?)"\)', content, re.DOTALL)
             attached_script = script_match.group(1) if script_match else None
-            
+
             nodes.append({
                 "name": node_name,
                 "type": node_type,
                 "script": attached_script
             })
-        
+
         return {
             "file": file_path,
             "nodes": nodes,
@@ -1485,11 +1485,11 @@ class CodeAnalyzer:
                 "resource_count": len(resources)
             }
         }
-    
+
     def _analyze_godot_resource(self, content: str, file_path: str) -> dict[str, Any]:
         """
         Analyze Godot .tres resource file.
-        
+
         Extracts:
         - Resource type and class
         - Script reference
@@ -1501,7 +1501,7 @@ class CodeAnalyzer:
         resource_type = None
         script_class = None
         script_path = None
-        
+
         # Extract resource header
         header_match = re.search(r'\[gd_resource type="(.+?)"(?:\s+script_class="(.+?)")?\s+', content)
         if header_match:
diff --git a/tests/test_cli_paths.py b/tests/test_cli_paths.py
index b353008..afd5a54 100644
--- a/tests/test_cli_paths.py
+++ b/tests/test_cli_paths.py
@@ -138,7 +138,7 @@ class TestUnifiedCLIEntryPoints(unittest.TestCase):
 
             # Should show version
             output = result.stdout + result.stderr
-            self.assertIn("2.8.0", output)
+            self.assertIn("2.9.0", output)
 
         except FileNotFoundError:
             # If skill-seekers is not installed, skip this test
diff --git a/tests/test_package_structure.py b/tests/test_package_structure.py
index 960e20e..f64d5b0 100644
--- a/tests/test_package_structure.py
+++ b/tests/test_package_structure.py
@@ -24,7 +24,7 @@ class TestCliPackage:
         import skill_seekers.cli
 
         assert hasattr(skill_seekers.cli, "__version__")
-        assert skill_seekers.cli.__version__ == "2.8.0"
+        assert skill_seekers.cli.__version__ == "2.9.0"
 
     def test_cli_has_all(self):
         """Test that skill_seekers.cli package has __all__ export list."""
@@ -88,7 +88,7 @@ class TestMcpPackage:
         import skill_seekers.mcp
 
         assert hasattr(skill_seekers.mcp, "__version__")
-        assert skill_seekers.mcp.__version__ == "2.8.0"
+        assert skill_seekers.mcp.__version__ == "2.9.0"
 
     def test_mcp_has_all(self):
         """Test that skill_seekers.mcp package has __all__ export list."""
@@ -108,7 +108,7 @@ class TestMcpPackage:
         import skill_seekers.mcp.tools
 
         assert hasattr(skill_seekers.mcp.tools, "__version__")
-        assert skill_seekers.mcp.tools.__version__ == "2.8.0"
+        assert skill_seekers.mcp.tools.__version__ == "2.9.0"
 
 
 class TestPackageStructure:
@@ -212,7 +212,7 @@ class TestRootPackage:
         import skill_seekers
 
         assert hasattr(skill_seekers, "__version__")
-        assert skill_seekers.__version__ == "2.8.0"
+        assert skill_seekers.__version__ == "2.9.0"
 
     def test_root_has_metadata(self):
         """Test that skill_seekers root package has metadata."""