refactor: remove commented code and improve function documentation

app.py

@@ -38,7 +38,6 @@ class SVGAnimationGenerator:
 
     def parse_svg(self, svg_content: str) -> dict:
         try:
-            # Remove namespace definitions and simplify the <svg> tag
             svg_content = re.sub(r'xmlns[^=]*="[^"]*"', "", svg_content)
             svg_content = re.sub(r"<svg[^>]*>", "<svg>", svg_content)
             return {"svg_content": svg_content}
@@ -49,7 +48,6 @@ class SVGAnimationGenerator:
         try:
             svg_content = parsed_svg["svg_content"]
 
-            # Convert SVG to PNG for MLLM analysis
             image_media_type, image_data = svg_to_png_base64(svg_content)
 
             if not image_data:
@@ -90,7 +88,6 @@ class SVGAnimationGenerator:
             decomposed_svg_match = re.search(
                 r"<decomposed_svg>(.*?)</decomposed_svg>", response_text, re.DOTALL
             )
-            # 오타 수정: suggenstions → suggestions
             animation_suggestions_match = re.search(
                 r"<animation_suggestions>(.*?)</animation_suggestions>",
                 response_text,
@@ -146,7 +143,6 @@ class SVGAnimationGenerator:
             decomposed_svg_match = re.search(
                 r"<decomposed_svg>(.*?)</decomposed_svg>", response_text, re.DOTALL
             )
-            # 오타 수정: suggenstions → suggestions
             animation_suggestions_match = re.search(
                 r"<animation_suggestions>(.*?)</animation_suggestions>",
                 response_text,
@@ -157,7 +153,6 @@ class SVGAnimationGenerator:
                 decomposed_svg_text = decomposed_svg_match.group(1).strip()
                 animation_suggestions = animation_suggestions_match.group(1).strip()
 
-                # Create viewer HTML with XSS protection
                 viewer_html = create_svg_viewer_html(decomposed_svg_text)
 
                 return decomposed_svg_text, animation_suggestions, viewer_html
@@ -207,28 +202,20 @@ class SVGAnimationGenerator:
 
 
 def _sanitize_svg(svg: str) -> str:
-    """안전한 미리보기를 위해 script 태그/inline on* 핸들러 제거"""
-    # <script>...</script> 제거
+    """Script tag/inline on*Handler removal for safe preview"""
     svg = re.sub(r"<\s*script\b[^>]*>.*?<\s*/\s*script\s*>", "", svg, flags=re.I | re.S)
-    # onload, onclick 등 inline 핸들러 제거
     svg = re.sub(r"\son[a-zA-Z]+\s*=\s*(['\"]).*?\1", "", svg, flags=re.I | re.S)
     return svg
 
 
 def _fix_svg_markup(svg: str) -> str:
-    """뷰어에 맞게 SVG 속성 정리(크기/비율/문제 스타일 수정)"""
+    """SVG attribute to fit the viewer (size/ratio/problem style modification)"""
     s = svg
-
-    # preserveAspectRatio 보정
     s = s.replace('preserveAspectRatio="none"', 'preserveAspectRatio="xMidYMid meet"')
-
-    # position:absolute 등 문제 스타일 제거/대체
     s = s.replace(
         'style="display: block; overflow: hidden; position: absolute; left: 0px; top: 0px;"',
         'style="display:block; width:100%; height:100%;"',
     )
-
-    # width/height 절대값 → 상대값
     s = re.sub(r'width="[^"]+"', 'width="100%"', s, count=1)
     s = re.sub(r'height="[^"]+"', 'height="100%"', s, count=1)
 
@@ -430,13 +417,10 @@ def create_svg_viewer_html(svg_content: str) -> str:
 
 
 def _extract_path_from_gradio_file(svg_file) -> str | None:
-    # gr.File 값이 문자열 경로인 경우
     if isinstance(svg_file, (str, pathlib.Path)):
         return str(svg_file)
-    # dict 형태({ 'name': '/tmp/...' })로 들어오는 경우
     if isinstance(svg_file, dict) and "name" in svg_file:
         return svg_file["name"]
-    # NamedString 등 파일 객체에 name 속성이 있는 경우
     if hasattr(svg_file, "name"):
         return svg_file.name
     return None
@@ -492,14 +476,13 @@ def predict_decompose_group(svg_file, svg_text, object_name):
     decomposed_svg = decompose_result["svg_content"]
     animation_suggestions = decompose_result["animation_suggestions"]
 
-    # Create viewer HTML
     decomposed_svg_viewer = create_svg_viewer_html(decomposed_svg)
 
     return (
-        decomposed_svg,  # For svg_content_hidden
-        decomposed_svg,  # For groups_summary (분석 결과 표시)
-        animation_suggestions,  # For animation_suggestion
-        decomposed_svg_viewer,  # For decomposed_svg_viewer
+        decomposed_svg,
+        decomposed_svg,
+        animation_suggestions,
+        decomposed_svg_viewer,
     )
 
 
@@ -509,7 +492,6 @@ def update_preview_from_html(html_content: str) -> str:
         return create_error_html("⚠️ HTML content is empty")
 
     try:
-        # Create iframe HTML - only escape quotes for srcdoc attribute
         safe_html_content = html_content.replace('"', "&quot;")
         preview_html = f"""
         <div style='padding: 20px; width:100%; height:520px; background: #fff; border: 1px solid #eee; border-radius: 8px; display: block;'>
@@ -549,7 +531,6 @@ def create_animation_preview(animation_desc: str, svg_content: str) -> tuple:
             error_html = create_error_html("❌ Failed to generate animation HTML")
             return error_html, animation_response
 
-        # Save the HTML content to the output directory
         output_dir = "output"
         os.makedirs(output_dir, exist_ok=True)
         html_path = os.path.join(output_dir, "animation_preview.html")
@@ -557,7 +538,6 @@ def create_animation_preview(animation_desc: str, svg_content: str) -> tuple:
             f.write(html_content)
         print(f"Animation preview saved to: {html_path}")
 
-        # Create iframe HTML - only escape quotes for srcdoc attribute
         fixed_html_content = fix_html_styles_for_preview(html_content)
         safe_html_content = fixed_html_content.replace('"', "&quot;")
 
@@ -594,15 +574,13 @@ example_list = {
 
 
 def load_example(example_choice):
-    # Find the selected example in the categories
     for category, examples in example_list.items():
         for example in examples:
             if example[1] == example_choice:
-                return example[0], example[1]  # Return both file path and object name
+                return example[0], example[1]
     return None, None
 
 
-# Flatten choices for dropdown
 example_choices = [
     example[1] for category in example_list.values() for example in category
 ]
@@ -624,7 +602,6 @@ with demo:
                         lines=8.4,
                     )
 
-                # Add example dropdown
                 example_dropdown = gr.Dropdown(
                     choices=example_choices, label="Try an Example", value=None
                 )
@@ -639,7 +616,6 @@ with demo:
                     value="corgi",
                 )
 
-                # Add dropdown change event
                 example_dropdown.change(
                     fn=load_example,
                     inputs=[example_dropdown],
@@ -715,31 +691,30 @@ with demo:
                     label="Output HTML",
                     lines=10,
                     placeholder="Generated HTML will appear here. You can edit this HTML and see live preview.",
-                    interactive=True,  # Make sure it's editable
+                    interactive=True,
                 )
 
-    # Event handlers
     process_btn.click(
         fn=predict_decompose_group,
         inputs=[svg_file, svg_text, object_name],
         outputs=[
-            svg_content_hidden,  # Store decomposed SVG for later use
-            groups_summary,  # Show analysis results
-            animation_suggestion,  # Show animation suggestions
-            decomposed_svg_viewer,  # Show SVG preview
+            svg_content_hidden,
+            groups_summary,
+            animation_suggestion,
+            decomposed_svg_viewer,
         ],
     )
 
     groups_feedback_btn.click(
         fn=generator.feedback_decompose_group,
         inputs=[
-            svg_content_hidden,  # Pass the SVG content directly
-            groups_feedback,  # Pass the feedback text
+            svg_content_hidden,
+            groups_feedback,
         ],
         outputs=[
-            svg_content_hidden,  # Update hidden SVG content
-            animation_suggestion,  # Update animation suggestions
-            decomposed_svg_viewer,  # Update SVG preview
+            svg_content_hidden,
+            animation_suggestion,
+            decomposed_svg_viewer,
         ],
     )
 
@@ -750,12 +725,11 @@ with demo:
             svg_content_hidden,
         ],
         outputs=[
-            animation_preview,  # Animation preview HTML
-            output_html,  # Raw HTML output
+            animation_preview,
+            output_html,
         ],
     )
 
-    # Real-time preview update when HTML is manually edited
     output_html.change(
         fn=update_preview_from_html,
         inputs=[output_html],