bmad-plus 0.4.0 → 0.4.2

package/oveanet-pack/seo-audit-360/scripts/seo_screenshot.py

@@ -1,202 +0,0 @@
-#!/usr/bin/env python3
-"""
-SEO Screenshot — Viewport screenshot capture for visual SEO analysis.
-
-Features:
-- Mobile and desktop viewport presets
-- Above-the-fold element detection
-- Full-page capture option
-- PNG output with configurable quality
-
-Requires: playwright (pip install playwright && playwright install chromium)
-
-Author: Laurent Rochetta
-License: MIT
-"""
-
-import argparse
-import sys
-
-
-VIEWPORTS = {
-    "mobile": {"width": 375, "height": 812, "device_scale_factor": 3, "is_mobile": True},
-    "tablet": {"width": 768, "height": 1024, "device_scale_factor": 2, "is_mobile": True},
-    "desktop": {"width": 1440, "height": 900, "device_scale_factor": 1, "is_mobile": False},
-    "desktop-hd": {"width": 1920, "height": 1080, "device_scale_factor": 1, "is_mobile": False},
-}
-
-
-def capture_screenshot(
-    url: str,
-    output: str = "screenshot.png",
-    viewport: str = "desktop",
-    full_page: bool = False,
-    wait_ms: int = 2000,
-):
-    """
-    Capture a viewport screenshot of a URL using Playwright.
-
-    Args:
-        url: URL to capture
-        output: Output file path (.png)
-        viewport: Viewport preset (mobile, tablet, desktop, desktop-hd)
-        full_page: Capture full page scroll or just viewport
-        wait_ms: Wait time after page load (ms)
-    """
-    try:
-        from playwright.sync_api import sync_playwright
-    except ImportError:
-        print(
-            "Error: playwright required.\n"
-            "Install: pip install playwright && playwright install chromium",
-            file=sys.stderr,
-        )
-        sys.exit(1)
-
-    vp = VIEWPORTS.get(viewport, VIEWPORTS["desktop"])
-
-    with sync_playwright() as p:
-        browser = p.chromium.launch(headless=True)
-        context = browser.new_context(
-            viewport={"width": vp["width"], "height": vp["height"]},
-            device_scale_factor=vp["device_scale_factor"],
-            is_mobile=vp["is_mobile"],
-            user_agent=(
-                "Mozilla/5.0 (iPhone; CPU iPhone OS 17_0 like Mac OS X) "
-                "AppleWebKit/605.1.15 Mobile/15E148 Safari/604.1"
-                if vp["is_mobile"]
-                else "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 "
-                "(KHTML, like Gecko) Chrome/131.0.0.0 Safari/537.36 BMADSEOEngine/2.0"
-            ),
-        )
-
-        page = context.new_page()
-
-        try:
-            page.goto(url, wait_until="networkidle", timeout=30000)
-        except Exception:
-            # Fallback: wait for load event instead
-            page.goto(url, wait_until="load", timeout=30000)
-
-        # Wait for dynamic content
-        page.wait_for_timeout(wait_ms)
-
-        # Capture screenshot
-        page.screenshot(path=output, full_page=full_page)
-
-        # Gather above-the-fold metrics
-        metrics = page.evaluate("""() => {
-            const viewportHeight = window.innerHeight;
-            const viewportWidth = window.innerWidth;
-
-            // Find CTAs above the fold
-            const ctas = [];
-            const buttons = document.querySelectorAll('a, button, [role="button"]');
-            buttons.forEach(el => {
-                const rect = el.getBoundingClientRect();
-                if (rect.top < viewportHeight && rect.bottom > 0) {
-                    const text = el.textContent.trim().substring(0, 50);
-                    if (text && (
-                        /sign.?up|get.?start|try|buy|contact|demo|free|download|subscribe/i.test(text)
-                    )) {
-                        ctas.push({
-                            text: text,
-                            tag: el.tagName,
-                            top: Math.round(rect.top),
-                            visible: rect.width > 0 && rect.height > 0,
-                        });
-                    }
-                }
-            });
-
-            // Find hero/LCP candidate
-            const images = document.querySelectorAll('img');
-            let largestImage = null;
-            let largestArea = 0;
-            images.forEach(img => {
-                const rect = img.getBoundingClientRect();
-                const area = rect.width * rect.height;
-                if (area > largestArea && rect.top < viewportHeight) {
-                    largestArea = area;
-                    largestImage = {
-                        src: img.src.substring(0, 100),
-                        width: Math.round(rect.width),
-                        height: Math.round(rect.height),
-                        top: Math.round(rect.top),
-                    };
-                }
-            });
-
-            // Check for horizontal scroll
-            const hasHorizontalScroll = document.documentElement.scrollWidth > viewportWidth;
-
-            // Font size check
-            const body = document.body;
-            const bodyFontSize = body ? parseFloat(getComputedStyle(body).fontSize) : 16;
-
-            return {
-                viewportWidth,
-                viewportHeight,
-                ctas_above_fold: ctas.length,
-                cta_details: ctas.slice(0, 5),
-                largest_image_above_fold: largestImage,
-                has_horizontal_scroll: hasHorizontalScroll,
-                body_font_size_px: bodyFontSize,
-                dom_element_count: document.querySelectorAll('*').length,
-            };
-        }""")
-
-        browser.close()
-
-    return metrics
-
-
-# ── CLI ────────────────────────────────────────────────────────────
-
-def main():
-    parser = argparse.ArgumentParser(
-        description="SEO Screenshot — Viewport capture (BMAD+ SEO Engine)"
-    )
-    parser.add_argument("url", help="URL to capture")
-    parser.add_argument("--output", "-o", default="screenshot.png", help="Output file path")
-    parser.add_argument(
-        "--viewport", "-v",
-        choices=list(VIEWPORTS.keys()), default="desktop",
-        help="Viewport preset"
-    )
-    parser.add_argument("--full", action="store_true", help="Capture full page (not just viewport)")
-    parser.add_argument("--wait", "-w", type=int, default=2000, help="Wait after load (ms)")
-    parser.add_argument("--json", "-j", action="store_true", help="Output metrics as JSON")
-
-    args = parser.parse_args()
-
-    import json
-
-    metrics = capture_screenshot(
-        url=args.url,
-        output=args.output,
-        viewport=args.viewport,
-        full_page=args.full,
-        wait_ms=args.wait,
-    )
-
-    print(f"Screenshot saved: {args.output}", file=sys.stderr)
-
-    if args.json:
-        print(json.dumps(metrics, indent=2))
-    else:
-        print(f"\nAbove-the-Fold Analysis ({args.viewport}):")
-        print(f"  Viewport: {metrics['viewportWidth']}×{metrics['viewportHeight']}")
-        print(f"  CTAs above fold: {metrics['ctas_above_fold']}")
-        for cta in metrics.get("cta_details", []):
-            print(f"    - \"{cta['text']}\" ({cta['tag']}, top: {cta['top']}px)")
-        if metrics.get("largest_image_above_fold"):
-            img = metrics["largest_image_above_fold"]
-            print(f"  Largest image: {img['width']}×{img['height']} at y={img['top']}px")
-        print(f"  Horizontal scroll: {'⚠️ YES' if metrics['has_horizontal_scroll'] else '✅ No'}")
-        print(f"  Body font size: {metrics['body_font_size_px']}px {'✅' if metrics['body_font_size_px'] >= 16 else '⚠️ <16px'}")
-        print(f"  DOM elements: {metrics['dom_element_count']:,}")
-
-
-if __name__ == "__main__":
-    main()

package/oveanet-pack/seo-audit-360/tests/fixtures/sample_page.html

@@ -1,62 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-<head>
-    <meta charset="UTF-8">
-    <title>SEO Test Page — BMAD+ Fixture</title>
-    <meta name="description" content="A test page for validating the SEO parse module with known elements.">
-    <meta name="robots" content="index, follow">
-    <meta name="viewport" content="width=device-width, initial-scale=1.0">
-    <meta property="og:title" content="SEO Test Page">
-    <meta property="og:type" content="website">
-    <meta property="og:url" content="https://example.com/test">
-    <meta name="twitter:card" content="summary_large_image">
-    <meta name="twitter:title" content="SEO Test Page">
-    <link rel="canonical" href="https://example.com/test">
-    <link rel="alternate" hreflang="en" href="https://example.com/en/test">
-    <link rel="alternate" hreflang="fr" href="https://example.com/fr/test">
-    <link rel="alternate" hreflang="x-default" href="https://example.com/test">
-</head>
-<body>
-    <h1>Main Heading of the Page</h1>
-    <p>This is a test paragraph with enough words to verify word count functionality in the parser module. We need at least a few sentences to make the test meaningful and realistic.</p>
-
-    <h2>Second Level Heading One</h2>
-    <p>Content under the first H2. This paragraph adds more text to increase the word count.</p>
-
-    <h2>Second Level Heading Two</h2>
-    <p>Another section with different content about SEO analysis and testing.</p>
-
-    <h3>Third Level Heading</h3>
-    <p>Detailed information under the H3 heading for testing hierarchy detection.</p>
-
-    <img src="/images/hero.jpg" alt="Hero image for testing" width="800" height="400" loading="lazy">
-    <img src="/images/no-alt.jpg" width="200" height="200">
-    <img src="/images/empty-alt.jpg" alt="" width="100" height="100">
-
-    <a href="https://example.com/about">About Us</a>
-    <a href="https://example.com/services">Our Services</a>
-    <a href="https://external.com/partner" rel="nofollow" target="_blank">Partner Link</a>
-    <a href="/relative-link">Relative Link</a>
-
-    <script type="application/ld+json">
-    {
-        "@context": "https://schema.org",
-        "@type": "Organization",
-        "name": "Test Company",
-        "url": "https://example.com",
-        "logo": "https://example.com/logo.png"
-    }
-    </script>
-
-    <script type="application/ld+json">
-    {
-        "@context": "https://schema.org",
-        "@type": "BreadcrumbList",
-        "itemListElement": [
-            {"@type": "ListItem", "position": 1, "name": "Home", "item": "https://example.com"},
-            {"@type": "ListItem", "position": 2, "name": "Test", "item": "https://example.com/test"}
-        ]
-    }
-    </script>
-</body>
-</html>

package/oveanet-pack/seo-audit-360/tests/test_apis.py

@@ -1,75 +0,0 @@
-"""
-Tests for seo_apis.py — API response parsing and error handling.
-
-Author: Laurent Rochetta
-"""
-
-import sys
-import os
-sys.path.insert(0, os.path.join(os.path.dirname(__file__), "..", "scripts"))
-
-# Temporarily unset API key for error tests
-original_key = os.environ.get("GOOGLE_API_KEY", "")
-
-
-class TestAPIKeyMissing:
-    """Test behavior when GOOGLE_API_KEY is not set."""
-
-    def setup_method(self):
-        os.environ.pop("GOOGLE_API_KEY", None)
-        # Reimport to pick up empty key
-        import importlib
-        import seo_apis
-        importlib.reload(seo_apis)
-        self.seo_apis = seo_apis
-
-    def teardown_method(self):
-        if original_key:
-            os.environ["GOOGLE_API_KEY"] = original_key
-
-    def test_pagespeed_without_key(self):
-        # Force the module to use an empty key
-        self.seo_apis.API_KEY = ""
-        result = self.seo_apis.run_pagespeed("https://example.com")
-        assert result.get("error") is not None
-        assert "GOOGLE_API_KEY" in result["error"]
-
-    def test_crux_without_key(self):
-        self.seo_apis.API_KEY = ""
-        result = self.seo_apis.run_crux("https://example.com")
-        assert result.get("error") is not None
-
-    def test_rich_results_without_key(self):
-        self.seo_apis.API_KEY = ""
-        result = self.seo_apis.run_rich_results_test("https://example.com")
-        assert result.get("error") is not None
-
-
-class TestResultStructure:
-    """Test that API functions return expected structures."""
-
-    def setup_method(self):
-        import importlib
-        import seo_apis
-        importlib.reload(seo_apis)
-        self.seo_apis = seo_apis
-
-    def test_pagespeed_result_keys(self):
-        self.seo_apis.API_KEY = ""
-        result = self.seo_apis.run_pagespeed("https://example.com")
-        # Even on error, should have expected structure
-        assert "error" in result
-
-    def test_crux_result_keys(self):
-        self.seo_apis.API_KEY = ""
-        result = self.seo_apis.run_crux("https://example.com")
-        assert "error" in result
-
-    def test_run_all_structure(self):
-        self.seo_apis.API_KEY = ""
-        result = self.seo_apis.run_all("https://example.com")
-        assert "pagespeed" in result
-        assert "crux" in result
-        assert "mobile_friendly" in result
-        assert "url" in result
-        assert "timestamp" in result

package/oveanet-pack/seo-audit-360/tests/test_crawl.py

@@ -1,121 +0,0 @@
-"""
-Tests for seo_crawl.py — URL normalization, link extraction, depth limiting.
-
-Author: Laurent Rochetta
-"""
-
-import sys
-import os
-sys.path.insert(0, os.path.join(os.path.dirname(__file__), "..", "scripts"))
-
-from seo_crawl import SEOCrawler
-
-
-class TestURLNormalization:
-    """Test URL normalization for deduplication."""
-
-    def setup_method(self):
-        self.crawler = SEOCrawler("https://example.com", max_depth=2, max_pages=25)
-
-    def test_strips_trailing_slash(self):
-        assert self.crawler.normalize_url("https://example.com/page/") == "https://example.com/page"
-
-    def test_preserves_root(self):
-        assert self.crawler.normalize_url("https://example.com/") == "https://example.com/"
-
-    def test_normalizes_scheme(self):
-        result = self.crawler.normalize_url("https://example.com/page")
-        assert result.startswith("https://")
-
-    def test_deduplicates(self):
-        url1 = self.crawler.normalize_url("https://example.com/page/")
-        url2 = self.crawler.normalize_url("https://example.com/page")
-        assert url1 == url2
-
-
-class TestInternalDetection:
-    def setup_method(self):
-        self.crawler = SEOCrawler("https://example.com", max_depth=2, max_pages=25)
-
-    def test_internal_url(self):
-        assert self.crawler.is_internal("https://example.com/about") is True
-
-    def test_external_url(self):
-        assert self.crawler.is_internal("https://other.com/page") is False
-
-    def test_subdomain_is_external(self):
-        assert self.crawler.is_internal("https://blog.example.com/post") is False
-
-
-class TestLinkExtraction:
-    def setup_method(self):
-        self.crawler = SEOCrawler("https://example.com", max_depth=2, max_pages=25)
-
-    def test_extracts_internal_links(self):
-        html = '''
-        <a href="/about">About</a>
-        <a href="https://example.com/contact">Contact</a>
-        '''
-        links = self.crawler.extract_links(html, "https://example.com/")
-        assert len(links) == 2
-
-    def test_ignores_external_links(self):
-        html = '<a href="https://other.com/page">External</a>'
-        links = self.crawler.extract_links(html, "https://example.com/")
-        assert len(links) == 0
-
-    def test_ignores_anchors(self):
-        html = '<a href="#section">Anchor</a>'
-        links = self.crawler.extract_links(html, "https://example.com/")
-        assert len(links) == 0
-
-    def test_ignores_javascript(self):
-        html = '<a href="javascript:void(0)">JS Link</a>'
-        links = self.crawler.extract_links(html, "https://example.com/")
-        assert len(links) == 0
-
-    def test_ignores_mailto(self):
-        html = '<a href="mailto:test@example.com">Email</a>'
-        links = self.crawler.extract_links(html, "https://example.com/")
-        assert len(links) == 0
-
-
-class TestTitleExtraction:
-    def setup_method(self):
-        self.crawler = SEOCrawler("https://example.com")
-
-    def test_extracts_title(self):
-        html = "<html><head><title>Test Page</title></head><body></body></html>"
-        assert self.crawler.extract_title(html) == "Test Page"
-
-    def test_missing_title(self):
-        html = "<html><body></body></html>"
-        assert self.crawler.extract_title(html) is None
-
-
-class TestWordCount:
-    def setup_method(self):
-        self.crawler = SEOCrawler("https://example.com")
-
-    def test_counts_visible_words(self):
-        html = "<html><body><p>This is a test with seven words.</p></body></html>"
-        assert self.crawler.count_words(html) == 7
-
-    def test_excludes_script_content(self):
-        html = '<html><body><p>Visible</p><script>var hidden = true;</script></body></html>'
-        count = self.crawler.count_words(html)
-        assert count == 1  # Only "Visible"
-
-
-class TestCrawlerConfig:
-    def test_max_pages_respected(self):
-        crawler = SEOCrawler("https://example.com", max_pages=5)
-        assert crawler.max_pages == 5
-
-    def test_max_depth_respected(self):
-        crawler = SEOCrawler("https://example.com", max_depth=1)
-        assert crawler.max_depth == 1
-
-    def test_base_domain_extracted(self):
-        crawler = SEOCrawler("https://www.example.com/page")
-        assert crawler.base_domain == "www.example.com"

package/oveanet-pack/seo-audit-360/tests/test_fetch.py

@@ -1,70 +0,0 @@
-"""
-Tests for seo_fetch.py — SSRF protection, URL handling, error cases.
-
-Author: Laurent Rochetta
-"""
-
-import sys
-import os
-sys.path.insert(0, os.path.join(os.path.dirname(__file__), "..", "scripts"))
-
-from seo_fetch import is_safe_url, fetch_page
-
-
-class TestSSRFProtection:
-    """Test SSRF prevention blocks private/loopback/reserved IPs."""
-
-    def test_blocks_localhost(self):
-        assert is_safe_url("http://127.0.0.1/admin") is False
-
-    def test_blocks_private_10(self):
-        assert is_safe_url("http://10.0.0.1/secret") is False
-
-    def test_blocks_private_192(self):
-        assert is_safe_url("http://192.168.1.1/") is False
-
-    def test_blocks_private_172(self):
-        assert is_safe_url("http://172.16.0.1/") is False
-
-    def test_allows_public_ip(self):
-        assert is_safe_url("https://93.184.216.34/") is True
-
-    def test_allows_public_domain(self):
-        assert is_safe_url("https://example.com/") is True
-
-    def test_blocks_empty_hostname(self):
-        assert is_safe_url("http:///nohost") is False
-
-    def test_blocks_zero_ip(self):
-        assert is_safe_url("http://0.0.0.0/") is False
-
-
-class TestFetchPage:
-    """Test fetch_page function behavior."""
-
-    def test_normalizes_url_without_scheme(self):
-        result = fetch_page("example.com", timeout=5)
-        assert result["url"] == "example.com"
-        # Should have attempted https://example.com
-
-    def test_blocks_invalid_scheme(self):
-        result = fetch_page("ftp://example.com/file")
-        assert result["error"] is not None
-        assert "Invalid URL scheme" in result["error"]
-
-    def test_blocks_ssrf(self):
-        result = fetch_page("http://127.0.0.1/admin")
-        assert result["error"] is not None
-        assert "Blocked" in result["error"]
-
-    def test_result_structure(self):
-        """Verify the result dict has all expected keys."""
-        result = fetch_page("https://example.com", timeout=5)
-        expected_keys = {"url", "final_url", "status_code", "content", "headers",
-                         "redirect_chain", "content_length", "response_time_ms", "error"}
-        assert expected_keys == set(result.keys())
-
-    def test_timeout_returns_error(self):
-        # Use a non-routable IP to force timeout
-        result = fetch_page("http://192.0.2.1/", timeout=1)
-        assert result["error"] is not None