From 5e7937f57edd886b1c13e080cc9607915ebe9b99 Mon Sep 17 00:00:00 2001
From: vvzvlad <git@vvzvlad.xyz>
Date: Tue, 15 Apr 2025 18:09:49 +0300
Subject: [PATCH] Refactor string trimming logic

---
 post_parser.py            | 36 ++++++++++++------------------------
 tests/test_post_parser.py |  6 +++---
 2 files changed, 15 insertions(+), 27 deletions(-)

diff --git a/post_parser.py b/post_parser.py
index 2057f5b..6cb45fb 100644
--- a/post_parser.py
+++ b/post_parser.py
@@ -176,42 +176,30 @@ class PostParser:
                 if first_line.isupper() and len(first_line) > 1:
                     first_line = first_line.lower().capitalize()
 
-                # --- Trim long strings (REVISED LOGIC - Find LAST space before limit) ---
+                # --- Trim long strings ---
                 cut_at = 37
                 max_extra_chars = 15
                 limit_index = cut_at + max_extra_chars # 52
 
                 if len(first_line) > cut_at:
-                    # Define the effective limit for searching and cutting
-                    cut_limit = min(len(first_line), limit_index)
+                    cut_limit = min(len(first_line), limit_index) # Define the effective limit for searching and cutting
+                    last_space_index = first_line.rfind(' ', 0, cut_limit) # Find the LAST space before the cut_limit
 
-                    # Find the LAST space before the cut_limit
-                    last_space_index = first_line.rfind(' ', 0, cut_limit)
-
-                    # Decide the cut index
-                    if last_space_index != -1 and last_space_index >= cut_at:
-                        # Found a suitable space within [cut_at, cut_limit)
-                        cut_index = last_space_index
+                    if last_space_index != -1 and last_space_index >= cut_at: # Decide the cut index
+                        cut_index = last_space_index # Found a suitable space within [cut_at, cut_limit)
                     else:
-                        # No suitable space found, cut at the hard limit
-                        cut_index = cut_limit
+                        cut_index = cut_limit # No suitable space found, cut at the hard limit
 
-                    # Slice the string
-                    title_segment = first_line[:cut_index]
+                    title_segment = first_line[:cut_index] # Slice the string
 
-                    # Clean trailing punctuation/spaces from the segment AFTER slicing
-                    title_segment = re.sub(r'[\s.,;:]+$', '', title_segment).strip()
+                    title_segment = re.sub(r'[\s.,;:]+$', '', title_segment).strip() # Clean trailing punctuation/spaces from the segment AFTER slicing
 
-                    # Only add ellipsis if the title was actually cut
-                    # (Check against original first_line length before cleaning segment)
-                    if len(first_line[:cut_index]) < len(first_line):
-                       processed_title = f"{title_segment}..."
+                    if len(first_line[:cut_index]) < len(first_line): # Only add ellipsis if the title was actually cut
+                        processed_title = f"{title_segment}..."
                     else: # Safeguard
-                       processed_title = title_segment
+                        processed_title = title_segment
                 else:
-                    # Length <= cut_at, use as is
-                    processed_title = first_line
-                # --- End Trim long strings (REVISED LOGIC) ---
+                    processed_title = first_line # Length <= cut_at, use as is
 
         # --- Decision Logic --- (Phase 2: Decide whether to use processed text or fallback)
 
diff --git a/tests/test_post_parser.py b/tests/test_post_parser.py
index 45f9b30..bfa8f1f 100644
--- a/tests/test_post_parser.py
+++ b/tests/test_post_parser.py
@@ -167,9 +167,9 @@ class TestPostParserGenerateTitle(unittest.TestCase):
 
 
     def test_generate_title_long_text_trimming_with_spaces(self):
-        cut_at = 37
-        max_extra = 15
-        limit = cut_at + max_extra # 52
+        #cut_at = 37
+        #max_extra = 15
+        #limit = cut_at + max_extra # 52
 
         # --- Test Cases Based on Correct Logic ---