codewave-openclaw-installer 2.1.10 → 2.1.12

package/skills/ppt-master/scripts/svg_quality_checker.py

@@ -89,6 +89,18 @@ class SVGQualityChecker:
             # 5. Check text wrapping methods
             self._check_text_elements(content, result)
 
+            # 6. Check text overflow (beyond viewBox boundaries)
+            self._check_text_overflow(content, result)
+
+            # 7. Check bottom red line presence
+            self._check_bottom_line(content, result)
+
+            # 8. Check file size (warn if too large)
+            file_size = svg_path.stat().st_size
+            if file_size > 5 * 1024 * 1024:  # 5MB
+                result['warnings'].append(f"Large SVG file: {file_size // 1024}KB (may slow down export)")
+            result['info']['file_size_kb'] = file_size // 1024
+
             # Determine pass/fail
             result['passed'] = len(result['errors']) == 0
 
@@ -206,7 +218,13 @@ class SVGQualityChecker:
         if re.search(r'<g[^>]*\sopacity\s*=', content_lower):
             result['errors'].append("Detected forbidden <g opacity> (set opacity on each child element individually)")
         if re.search(r'<image[^>]*\sopacity\s*=', content_lower):
-            result['errors'].append("Detected forbidden <image opacity> (use overlay mask approach)")
+            # Allow opacity on background images (first <image> child of <svg>)
+            # Only flag non-background images with opacity
+            image_opacity_matches = re.findall(r'<image[^>]*\sopacity\s*=\s*"([^"]*)"[^>]*/?\s*>', content, re.IGNORECASE)
+            bg_pattern = re.search(r'<svg[^>]*>\s*\n?\s*<image[^>]*opacity', content, re.IGNORECASE)
+            non_bg_opacity_count = len(image_opacity_matches) - (1 if bg_pattern else 0)
+            if non_bg_opacity_count > 0:
+                result['errors'].append(f"Detected {non_bg_opacity_count} non-background <image opacity> (use overlay mask approach)")
 
     def _check_fonts(self, content: str, result: Dict):
         """Check font usage"""
@@ -268,6 +286,71 @@ class SVGQualityChecker:
                 f"Detected {len(text_matches)} potentially overly long single-line text(s) (consider using tspan for wrapping)"
             )
 
+    def _check_text_overflow(self, content: str, result: Dict):
+        """Check if text elements extend beyond viewBox boundaries"""
+        # Get viewBox dimensions
+        viewbox_match = re.search(r'viewBox="0 0 (\d+) (\d+)"', content)
+        if not viewbox_match:
+            return
+        vb_width = int(viewbox_match.group(1))
+        vb_height = int(viewbox_match.group(2))
+
+        overflow_count = 0
+
+        # Check <text> x/y coordinates
+        for m in re.finditer(r'<text[^>]*\bx="([^"]*)"[^>]*\by="([^"]*)"', content):
+            try:
+                x = float(m.group(1))
+                y = float(m.group(2))
+                if x < -10 or x > vb_width + 10:
+                    overflow_count += 1
+                if y < -10 or y > vb_height + 50:  # allow small bottom overflow for descenders
+                    overflow_count += 1
+            except (ValueError, TypeError):
+                pass
+
+        # Check <tspan> y coordinates for vertical overflow
+        for m in re.finditer(r'<tspan[^>]*\by="([^"]*)"', content):
+            try:
+                y = float(m.group(1))
+                if y > vb_height + 20:
+                    overflow_count += 1
+            except (ValueError, TypeError):
+                pass
+
+        if overflow_count > 0:
+            result['errors'].append(
+                f"Detected {overflow_count} text element(s) potentially overflowing viewBox ({vb_width}x{vb_height})")
+
+    def _check_bottom_line(self, content: str, result: Dict):
+        """Check for bottom red line (required on every page)"""
+        # Look for a rect at y=716 with height=4 and red-ish fill
+        has_bottom_line = bool(re.search(
+            r'<rect[^>]*y="71[456]"[^>]*height="[3-6]"[^>]*fill="#[Cc]00000"',
+            content
+        )) or bool(re.search(
+            r'<rect[^>]*fill="#[Cc]00000"[^>]*y="71[456]"[^>]*height="[3-6]"',
+            content
+        ))
+        # Also check for line at bottom
+        if not has_bottom_line:
+            has_bottom_line = bool(re.search(
+                r'<line[^>]*y1="71[456]"[^>]*y2="71[456]"[^>]*stroke="#[Cc]00000"',
+                content
+            ))
+        if not has_bottom_line:
+            # More relaxed: any red rect near bottom
+            has_bottom_line = bool(re.search(
+                r'<rect[^>]*y="7[01]\d"[^>]*fill="#[Cc]00000"',
+                content
+            ))
+
+        if not has_bottom_line:
+            result['warnings'].append("Missing bottom red line (expected <rect y=\"716\" height=\"4\" fill=\"#C00000\"/>)")
+            result['info']['has_bottom_line'] = False
+        else:
+            result['info']['has_bottom_line'] = True
+
     def _categorize_issue(self, error_msg: str) -> str:
         """Categorize issue type"""
         if 'viewBox' in error_msg:
@@ -424,24 +507,79 @@ class SVGQualityChecker:
         print(f"\n[REPORT] Check report exported: {output_file}")
 
 
+def auto_fix_svg(svg_file: str) -> Dict:
+    """
+    Auto-fix common SVG issues.
+    Returns dict with fix counts.
+    """
+    path = Path(svg_file)
+    with open(path, 'r', encoding='utf-8') as f:
+        content = f.read()
+
+    original = content
+    fixes = []
+
+    # Fix 1: Replace rgba() with rgb() + opacity
+    rgba_matches = re.findall(r'rgba\(\s*(\d+)\s*,\s*(\d+)\s*,\s*(\d+)\s*,\s*([\d.]+)\s*\)', content)
+    for r, g, b, a in rgba_matches:
+        old = f'rgba({r},{g},{b},{a})'
+        # Also match with spaces
+        content = re.sub(
+            rf'rgba\(\s*{r}\s*,\s*{g}\s*,\s*{b}\s*,\s*{re.escape(a)}\s*\)',
+            f'rgb({r},{g},{b})',
+            content
+        )
+        fixes.append(f"rgba→rgb: rgba({r},{g},{b},{a})")
+
+    # Fix 2: Remove <style> blocks
+    if '<style' in content.lower():
+        content = re.sub(r'<style[^>]*>.*?</style>', '', content, flags=re.DOTALL | re.IGNORECASE)
+        fixes.append("Removed <style> block")
+
+    # Fix 3: Remove class attributes
+    if re.search(r'\bclass\s*=\s*"[^"]*"', content):
+        content = re.sub(r'\s*class\s*=\s*"[^"]*"', '', content)
+        fixes.append("Removed class attributes")
+
+    # Fix 4: Add missing bottom red line
+    has_bottom_line = bool(re.search(r'<rect[^>]*y="7[01]\d"[^>]*fill="#[Cc]00000"', content))
+    if not has_bottom_line:
+        # Insert before </svg>
+        bottom_line = '<rect x="0" y="716" width="1280" height="4" fill="#C00000"/>'
+        content = content.replace('</svg>', f'{bottom_line}\n</svg>')
+        fixes.append("Added bottom red line")
+
+    # Fix 5: Remove <g opacity="..."> by moving opacity to children
+    g_opacity_matches = re.findall(r'<g\s+opacity="([^"]+)">', content)
+    for op in g_opacity_matches:
+        # Simple removal of group opacity (conservative — just remove the opacity attr)
+        content = re.sub(r'<g\s+opacity="[^"]+"\s*>', '<g>', content, count=1)
+        fixes.append(f"Removed <g opacity=\"{op}\">")
+
+    if content != original:
+        with open(path, 'w', encoding='utf-8') as f:
+            f.write(content)
+
+    return {'file': path.name, 'fixes': fixes, 'changed': content != original}
+
+
 def main() -> None:
     """Run the CLI entry point."""
     if len(sys.argv) < 2:
         print("PPT Master - SVG Quality Check Tool\n")
         print("Usage:")
-        print("  python3 scripts/svg_quality_checker.py <svg_file>")
-        print("  python3 scripts/svg_quality_checker.py <directory>")
+        print("  python3 scripts/svg_quality_checker.py <svg_file_or_directory>")
+        print("  python3 scripts/svg_quality_checker.py <directory> --fix    # Auto-fix common issues")
+        print("  python3 scripts/svg_quality_checker.py <directory> --fix --check  # Fix then re-check")
         print("  python3 scripts/svg_quality_checker.py --all examples")
-        print("\nExamples:")
-        print("  python3 scripts/svg_quality_checker.py examples/project/svg_output/slide_01.svg")
-        print("  python3 scripts/svg_quality_checker.py examples/project/svg_output")
-        print("  python3 scripts/svg_quality_checker.py examples/project")
         sys.exit(0)
 
     checker = SVGQualityChecker()
+    do_fix = '--fix' in sys.argv
+    do_check_after_fix = '--check' in sys.argv
 
     # Parse arguments
-    target = sys.argv[1]
+    target = [a for a in sys.argv[1:] if not a.startswith('--')][0] if any(not a.startswith('--') for a in sys.argv[1:]) else sys.argv[1]
     expected_format = None
 
     if '--format' in sys.argv:
@@ -449,6 +587,29 @@ def main() -> None:
         if idx + 1 < len(sys.argv):
             expected_format = sys.argv[idx + 1]
 
+    # Auto-fix mode
+    if do_fix:
+        target_path = Path(target)
+        if target_path.is_file():
+            svg_files = [target_path]
+        else:
+            svg_output = target_path / 'svg_output' if (target_path / 'svg_output').exists() else target_path
+            svg_files = sorted(svg_output.glob('*.svg'))
+
+        print(f"\n[FIX] Auto-fixing {len(svg_files)} SVG file(s)...\n")
+        total_fixes = 0
+        for svg_file in svg_files:
+            result = auto_fix_svg(str(svg_file))
+            if result['changed']:
+                total_fixes += len(result['fixes'])
+                print(f"  [FIXED] {result['file']}: {', '.join(result['fixes'])}")
+        print(f"\n[FIX] Total fixes applied: {total_fixes}")
+
+        if not do_check_after_fix:
+            print("\n[TIP] Run with --fix --check to verify fixes")
+            sys.exit(0)
+        print("\n--- Re-checking after fixes ---\n")
+
     # Execute check
     if target == '--all':
         # Check all example projects

package/skills/ppt-master/scripts/svg_to_pptx/pptx_builder.py

@@ -351,10 +351,16 @@ def create_pptx_with_native_svg(
                     arcname = file_path.relative_to(extract_dir)
                     zf.write(file_path, arcname)
 
+        failed_count = len(svg_files) - success_count
         if verbose:
             print()
             print(f"[Done] Saved: {output_path}")
-            print(f"  Succeeded: {success_count}, Failed: {len(svg_files) - success_count}")
+            print(f"  Succeeded: {success_count}, Failed: {failed_count}")
+            if failed_count > 0:
+                failed_indices = [i + 1 for i in range(len(svg_files))
+                                  if i + 1 not in range(1, success_count + failed_count + 1)]
+                # List failed slides by checking which ones threw exceptions
+                print(f"  Note: {failed_count} slide(s) failed and were skipped.")
             if use_compat_mode and has_any_image:
                 print(f"  Mode: Office compatibility mode (supports all Office versions)")
                 if PNG_RENDERER == 'svglib' and renderer_hint:

package/skills/ppt-master/scripts/svg_to_pptx/pptx_cli.py

@@ -15,6 +15,20 @@ from .pptx_slide_xml import TRANSITIONS
 
 def main() -> None:
     """CLI entry point for the SVG to PPTX conversion tool."""
+    try:
+        _main_impl()
+    except KeyboardInterrupt:
+        print("\nInterrupted by user.")
+        sys.exit(130)
+    except SystemExit:
+        raise
+    except Exception as e:
+        print(f"Fatal error: {type(e).__name__}: {e}", file=sys.stderr)
+        sys.exit(1)
+
+
+def _main_impl() -> None:
+    """Internal implementation of main CLI logic."""
     transition_choices = (
         ['none'] + (list(TRANSITIONS.keys()) if TRANSITIONS
                     else ['fade', 'push', 'wipe', 'split', 'strips', 'cover', 'random'])