markdown-table-fixer 0.1.2__tar.gz → 0.1.4__tar.gz

markdown_table_fixer-0.1.2/README.md → markdown_table_fixer-0.1.4/PKG-INFO

@@ -1,3 +1,48 @@
+Metadata-Version: 2.4
+Name: markdown-table-fixer
+Version: 0.1.4
+Summary: Markdown table formatter and linter with GitHub integration
+Project-URL: Homepage, https://github.com/lfit/markdown-table-fixer
+Project-URL: Repository, https://github.com/lfit/markdown-table-fixer.git
+Project-URL: Bug Tracker, https://github.com/lfit/markdown-table-fixer/issues
+Author-email: The Linux Foundation <releng@linuxfoundation.org>
+License: Apache-2.0
+License-File: LICENSE
+Keywords: formatter,github,linter,markdown,pre-commit,table
+Classifier: Development Status :: 4 - Beta
+Classifier: Environment :: Console
+Classifier: Intended Audience :: Developers
+Classifier: License :: OSI Approved :: Apache Software License
+Classifier: Operating System :: OS Independent
+Classifier: Programming Language :: Python :: 3
+Classifier: Programming Language :: Python :: 3.10
+Classifier: Programming Language :: Python :: 3.11
+Classifier: Programming Language :: Python :: 3.12
+Classifier: Programming Language :: Python :: 3.13
+Classifier: Programming Language :: Python :: 3.14
+Classifier: Topic :: Software Development :: Quality Assurance
+Classifier: Topic :: Text Processing :: Markup :: Markdown
+Requires-Python: >=3.10
+Requires-Dist: aiolimiter>=1.2.1
+Requires-Dist: dependamerge>=0.1.19
+Requires-Dist: httpx[http2]>=0.28.1
+Requires-Dist: pydantic>=2.12.5
+Requires-Dist: pyyaml>=6.0.3
+Requires-Dist: rich>=14.2.0
+Requires-Dist: tenacity>=9.1.2
+Requires-Dist: typer>=0.20.0
+Requires-Dist: wcwidth>=0.2.14
+Provides-Extra: dev
+Requires-Dist: mypy>=1.17.1; extra == 'dev'
+Requires-Dist: pre-commit>=4.5.0; extra == 'dev'
+Requires-Dist: pytest-asyncio>=1.1.0; extra == 'dev'
+Requires-Dist: pytest-cov>=7.0.0; extra == 'dev'
+Requires-Dist: pytest-mock>=3.15.1; extra == 'dev'
+Requires-Dist: pytest>=9.0.2; extra == 'dev'
+Requires-Dist: ruff>=0.14.8; extra == 'dev'
+Requires-Dist: types-pyyaml>=6.0.12.20250915; extra == 'dev'
+Description-Content-Type: text/markdown
+
 <!--
 SPDX-License-Identifier: Apache-2.0
 SPDX-FileCopyrightText: 2025 The Linux Foundation
@@ -56,6 +101,54 @@ Auto-fix issues found:
 markdown-table-fixer lint --auto-fix
 ```
 
+#### Respecting Markdownlint Disable Comments
+
+The tool automatically respects markdownlint disable/enable comments in your
+markdown files. Violations within disabled sections are not reported:
+
+```markdown
+<!-- markdownlint-disable MD013 -->
+| Long table that exceeds line length... |
+<!-- markdownlint-enable MD013 -->
+
+<!-- markdownlint-disable MD013 MD060 -->
+| Table with both line length and formatting issues disabled |
+<!-- markdownlint-enable MD013 MD060 -->
+```
+
+This behavior is consistent across both `lint` and `github` commands,
+ensuring that locally suppressed violations are not reported or fixed when
+processing pull requests.
+
+#### Violation Reporting
+
+The lint command provides detailed violation reporting by markdownlint rule code:
+
+```bash
+$ markdown-table-fixer lint .
+🔍 Scanning: .
+
+Summary:
+  Files scanned: 10
+  Files with issues: 1
+  Total violations: 66
+
+Files with issues:
+  README.md [66 Errors: MD013 (61), MD060 (5)]
+```
+
+The tool reports violations categorized by markdownlint rule:
+
+- **MD013**: Line length exceeds limit (auto-detected from `.markdownlintrc`)
+- **MD060**: Table formatting issues (misaligned pipes, spacing, etc.)
+
+**Note:** Violations in sections with markdownlint disable comments are
+automatically filtered out and not reported.
+
+The line length limit is automatically detected from your markdownlint
+configuration file (`.markdownlint.json`, `.markdownlint.yaml`, etc.). If no
+configuration file exists, the default is 80 characters.
+
 ### GitHub Mode
 
 The `github` command intelligently handles both individual pull requests and
@@ -110,6 +203,23 @@ markdown-table-fixer github https://github.com/owner/repo/pull/123 \
 - Requires Git operations (clone, amend, push)
 - Use when you need to amend commits or use sync strategies
 
+#### Scope Control
+
+By default, both methods scan and fix **all markdown files** in the
+repository. To limit processing to files changed in the PR, use the
+`--pr-changes-only` flag:
+
+```bash
+markdown-table-fixer github https://github.com/owner/repo/pull/123 \
+  --pr-changes-only
+```
+
+Benefits of limiting scope to PR changes:
+
+- Faster processing (fewer files to check)
+- Useful when you want to fix tables in modified files without touching others
+- Avoids touching unrelated markdown files
+
 #### Fix PRs Across an Organization
 
 Scan and fix tables in PRs across an entire GitHub organization:

markdown_table_fixer-0.1.2/PKG-INFO → markdown_table_fixer-0.1.4/README.md

@@ -1,44 +1,3 @@
-Metadata-Version: 2.4
-Name: markdown-table-fixer
-Version: 0.1.2
-Summary: Markdown table formatter and linter with GitHub integration
-Project-URL: Homepage, https://github.com/lfit/markdown-table-fixer
-Project-URL: Repository, https://github.com/lfit/markdown-table-fixer.git
-Project-URL: Bug Tracker, https://github.com/lfit/markdown-table-fixer/issues
-Author-email: The Linux Foundation <releng@linuxfoundation.org>
-License: Apache-2.0
-License-File: LICENSE
-Keywords: formatter,github,linter,markdown,pre-commit,table
-Classifier: Development Status :: 4 - Beta
-Classifier: Environment :: Console
-Classifier: Intended Audience :: Developers
-Classifier: License :: OSI Approved :: Apache Software License
-Classifier: Operating System :: OS Independent
-Classifier: Programming Language :: Python :: 3
-Classifier: Programming Language :: Python :: 3.10
-Classifier: Programming Language :: Python :: 3.11
-Classifier: Programming Language :: Python :: 3.12
-Classifier: Programming Language :: Python :: 3.13
-Classifier: Topic :: Software Development :: Quality Assurance
-Classifier: Topic :: Text Processing :: Markup :: Markdown
-Requires-Python: <3.14,>=3.10
-Requires-Dist: aiolimiter>=1.2.0
-Requires-Dist: httpx[http2]>=0.28.0
-Requires-Dist: pydantic>=2.10.3
-Requires-Dist: rich>=13.9.4
-Requires-Dist: tenacity>=9.0.0
-Requires-Dist: typer>=0.15.0
-Requires-Dist: wcwidth>=0.2.13
-Provides-Extra: dev
-Requires-Dist: mypy>=1.13.0; extra == 'dev'
-Requires-Dist: pre-commit>=4.0.1; extra == 'dev'
-Requires-Dist: pytest-asyncio>=0.25.0; extra == 'dev'
-Requires-Dist: pytest-cov>=6.0.0; extra == 'dev'
-Requires-Dist: pytest-mock>=3.14.0; extra == 'dev'
-Requires-Dist: pytest>=8.3.4; extra == 'dev'
-Requires-Dist: ruff>=0.8.4; extra == 'dev'
-Description-Content-Type: text/markdown
-
 <!--
 SPDX-License-Identifier: Apache-2.0
 SPDX-FileCopyrightText: 2025 The Linux Foundation
@@ -97,6 +56,54 @@ Auto-fix issues found:
 markdown-table-fixer lint --auto-fix
 ```
 
+#### Respecting Markdownlint Disable Comments
+
+The tool automatically respects markdownlint disable/enable comments in your
+markdown files. Violations within disabled sections are not reported:
+
+```markdown
+<!-- markdownlint-disable MD013 -->
+| Long table that exceeds line length... |
+<!-- markdownlint-enable MD013 -->
+
+<!-- markdownlint-disable MD013 MD060 -->
+| Table with both line length and formatting issues disabled |
+<!-- markdownlint-enable MD013 MD060 -->
+```
+
+This behavior is consistent across both `lint` and `github` commands,
+ensuring that locally suppressed violations are not reported or fixed when
+processing pull requests.
+
+#### Violation Reporting
+
+The lint command provides detailed violation reporting by markdownlint rule code:
+
+```bash
+$ markdown-table-fixer lint .
+🔍 Scanning: .
+
+Summary:
+  Files scanned: 10
+  Files with issues: 1
+  Total violations: 66
+
+Files with issues:
+  README.md [66 Errors: MD013 (61), MD060 (5)]
+```
+
+The tool reports violations categorized by markdownlint rule:
+
+- **MD013**: Line length exceeds limit (auto-detected from `.markdownlintrc`)
+- **MD060**: Table formatting issues (misaligned pipes, spacing, etc.)
+
+**Note:** Violations in sections with markdownlint disable comments are
+automatically filtered out and not reported.
+
+The line length limit is automatically detected from your markdownlint
+configuration file (`.markdownlint.json`, `.markdownlint.yaml`, etc.). If no
+configuration file exists, the default is 80 characters.
+
 ### GitHub Mode
 
 The `github` command intelligently handles both individual pull requests and
@@ -151,6 +158,23 @@ markdown-table-fixer github https://github.com/owner/repo/pull/123 \
 - Requires Git operations (clone, amend, push)
 - Use when you need to amend commits or use sync strategies
 
+#### Scope Control
+
+By default, both methods scan and fix **all markdown files** in the
+repository. To limit processing to files changed in the PR, use the
+`--pr-changes-only` flag:
+
+```bash
+markdown-table-fixer github https://github.com/owner/repo/pull/123 \
+  --pr-changes-only
+```
+
+Benefits of limiting scope to PR changes:
+
+- Faster processing (fewer files to check)
+- Useful when you want to fix tables in modified files without touching others
+- Avoids touching unrelated markdown files
+
 #### Fix PRs Across an Organization
 
 Scan and fix tables in PRs across an entire GitHub organization:

{markdown_table_fixer-0.1.2 → markdown_table_fixer-0.1.4}/pyproject.toml

@@ -10,7 +10,7 @@ name = "markdown-table-fixer"
 dynamic = ["version"]
 description = "Markdown table formatter and linter with GitHub integration"
 readme = "README.md"
-requires-python = ">=3.10,<3.14"
+requires-python = ">=3.10"
 license = {text = "Apache-2.0"}
 authors = [
     {name = "The Linux Foundation", email = "releng@linuxfoundation.org"},
@@ -27,28 +27,32 @@ classifiers = [
     "Programming Language :: Python :: 3.11",
     "Programming Language :: Python :: 3.12",
     "Programming Language :: Python :: 3.13",
+    "Programming Language :: Python :: 3.14",
     "Topic :: Software Development :: Quality Assurance",
     "Topic :: Text Processing :: Markup :: Markdown",
 ]
 dependencies = [
-    "typer>=0.15.0",
-    "rich>=13.9.4",
-    "httpx[http2]>=0.28.0",
-    "pydantic>=2.10.3",
-    "aiolimiter>=1.2.0",
-    "tenacity>=9.0.0",
-    "wcwidth>=0.2.13",
+    "typer>=0.20.0",
+    "rich>=14.2.0",
+    "httpx[http2]>=0.28.1",
+    "pydantic>=2.12.5",
+    "aiolimiter>=1.2.1",
+    "tenacity>=9.1.2",
+    "wcwidth>=0.2.14",
+    "pyyaml>=6.0.3",
+    "dependamerge>=0.1.19",
 ]
 
 [project.optional-dependencies]
 dev = [
-    "pytest>=8.3.4",
-    "pytest-cov>=6.0.0",
-    "pytest-asyncio>=0.25.0",
-    "pytest-mock>=3.14.0",
-    "mypy>=1.13.0",
-    "ruff>=0.8.4",
-    "pre-commit>=4.0.1",
+    "pytest>=9.0.2",
+    "pytest-cov>=7.0.0",
+    "pytest-asyncio>=1.1.0",
+    "pytest-mock>=3.15.1",
+    "mypy>=1.17.1",
+    "ruff>=0.14.8",
+    "pre-commit>=4.5.0",
+    "types-PyYAML>=6.0.12.20250915",
 ]
 
 [project.urls]
@@ -72,7 +76,7 @@ packages = ["src/markdown_table_fixer"]
 include = ["src", "README.md", "LICENSE*", "pyproject.toml", ".pre-commit-hooks.yaml"]
 
 [tool.ruff]
-target-version = "py310"
+target-version = "py312"
 line-length = 80
 exclude = [
     "src/markdown_table_fixer/_version.py",
@@ -116,7 +120,7 @@ skip-magic-trailing-comma = false
 line-ending = "auto"
 
 [tool.mypy]
-python_version = "3.10"
+python_version = "3.12"
 strict = true
 warn_return_any = true
 warn_unused_configs = true

{markdown_table_fixer-0.1.2 → markdown_table_fixer-0.1.4}/src/markdown_table_fixer/_version.py

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
 
-__version__ = version = '0.1.2'
-__version_tuple__ = version_tuple = (0, 1, 2)
+__version__ = version = '0.1.4'
+__version_tuple__ = version_tuple = (0, 1, 4)
 
 __commit_id__ = commit_id = None

markdown_table_fixer-0.1.4/src/markdown_table_fixer/blocked_pr_scanner.py

@@ -0,0 +1,216 @@
+# SPDX-License-Identifier: Apache-2.0
+# SPDX-FileCopyrightText: 2025 The Linux Foundation
+
+"""
+Blocked PR scanner using dependamerge's GitHubService.
+
+This module wraps dependamerge's PR scanning functionality to provide
+consistent blocked PR detection across both tools.
+"""
+
+import asyncio
+from collections.abc import AsyncIterator
+import logging
+from typing import Any
+
+from dependamerge.github_service import GitHubService
+from dependamerge.models import UnmergeablePR
+
+from .progress_tracker import ProgressTracker
+
+
+class BlockedPRScanner:
+    """Scanner for blocked/unmergeable pull requests using dependamerge logic."""
+
+    def __init__(
+        self,
+        token: str,
+        progress_tracker: ProgressTracker | None = None,
+        max_repo_tasks: int = 10,
+    ):
+        """Initialize the blocked PR scanner.
+
+        Args:
+            token: GitHub authentication token
+            progress_tracker: Optional progress tracker for UI updates
+            max_repo_tasks: Maximum number of repositories to process concurrently
+        """
+        self.token = token
+        self.progress_tracker = progress_tracker
+        self.max_repo_tasks = max_repo_tasks
+        self.logger = logging.getLogger(__name__)
+
+        # Initialize the dependamerge GitHubService
+        self.service = GitHubService(
+            token=token,
+            progress_tracker=progress_tracker,
+            max_repo_tasks=max_repo_tasks,
+        )
+
+    async def close(self) -> None:
+        """Close the underlying HTTP client."""
+        await self.service.close()
+
+    async def scan_organization_for_blocked_prs(
+        self,
+        organization: str,
+        include_drafts: bool = False,
+    ) -> AsyncIterator[tuple[str, str, dict[str, Any], UnmergeablePR]]:
+        """Scan an organization for blocked/unmergeable pull requests.
+
+        This method uses dependamerge's GitHubService to analyze PRs and only
+        yields those that are truly blocked (have merge conflicts, failing checks,
+        behind base, etc.). This ensures consistency with dependamerge's reporting.
+
+        Args:
+            organization: Organization name to scan
+            include_drafts: Whether to include draft PRs (default False)
+
+        Yields:
+            Tuples of (owner, repo_name, pr_data, unmergeable_pr) where:
+            - owner: Repository owner
+            - repo_name: Repository name
+            - pr_data: Raw PR data dict from GraphQL
+            - unmergeable_pr: UnmergeablePR model with blocking reasons
+        """
+        self.logger.debug(
+            f"Starting blocked PR scan of organization: {organization}"
+        )
+
+        # Start progress tracker if available
+        if self.progress_tracker:
+            self.progress_tracker.start()
+
+        try:
+            # Use dependamerge's scan_organization which already:
+            # 1. Counts repositories
+            # 2. Paginates through PRs
+            # 3. Analyzes blocking conditions
+            # 4. Tracks progress correctly
+            scan_result = await self.service.scan_organization(
+                organization, include_drafts=include_drafts
+            )
+        finally:
+            # Stop progress tracker before yielding results
+            if self.progress_tracker:
+                self.progress_tracker.stop()
+
+        # Display summary
+        if self.progress_tracker:
+            from rich.console import Console
+
+            console = Console()
+            console.print(
+                f"\n✅ Scan completed\n"
+                f"📊 Found {len(scan_result.unmergeable_prs)} blocked PRs "
+                f"out of {scan_result.total_prs} total PRs across {scan_result.scanned_repositories} repositories"
+            )
+        else:
+            self.logger.info(
+                f"Scan complete: found {len(scan_result.unmergeable_prs)} blocked PRs "
+                f"out of {scan_result.total_prs} total PRs across {scan_result.scanned_repositories} repositories"
+            )
+
+        # Now we need to fetch the full PR data for each unmergeable PR
+        # to provide it in the same format as the old scanner
+        for unmergeable_pr in scan_result.unmergeable_prs:
+            # Split repository name
+            parts = unmergeable_pr.repository.split("/", 1)
+            if len(parts) != 2:
+                self.logger.warning(
+                    f"Invalid repository format: {unmergeable_pr.repository}"
+                )
+                continue
+
+            owner, repo_name = parts
+
+            # Create a minimal pr_data dict that matches what the old scanner returned
+            # This is primarily for compatibility with existing code
+            pr_data = {
+                "number": unmergeable_pr.pr_number,
+                "title": unmergeable_pr.title,
+                "author": {"login": unmergeable_pr.author},
+                "url": unmergeable_pr.url,
+                "createdAt": unmergeable_pr.created_at,
+                "updatedAt": unmergeable_pr.updated_at,
+                # Include blocking information for backward compatibility
+                "_unmergeable_reasons": [
+                    {
+                        "type": reason.type,
+                        "description": reason.description,
+                        "details": reason.details,
+                    }
+                    for reason in unmergeable_pr.reasons
+                ],
+            }
+
+            yield owner, repo_name, pr_data, unmergeable_pr
+
+    async def scan_organization_all_prs(
+        self,
+        organization: str,
+        include_drafts: bool = False,
+    ) -> AsyncIterator[tuple[str, str, dict[str, Any], bool]]:
+        """Scan an organization for ALL pull requests, marking which are blocked.
+
+        This is useful when you want to process all PRs but need to know which
+        ones are blocked.
+
+        Args:
+            organization: Organization name to scan
+            include_drafts: Whether to include draft PRs
+
+        Yields:
+            Tuples of (owner, repo_name, pr_data, is_blocked) where:
+            - owner: Repository owner
+            - repo_name: Repository name
+            - pr_data: Raw PR data dict from GraphQL
+            - is_blocked: Boolean indicating if PR is blocked
+        """
+        # For this use case, we need to fetch ALL PRs, not just blocked ones
+        # This requires using the lower-level iteration methods
+        # For now, we'll note this is a TODO and focus on the blocked-only case
+        raise NotImplementedError(
+            "Scanning all PRs (not just blocked) is not yet implemented. "
+            "Use scan_organization_for_blocked_prs() for blocked-only scanning."
+        )
+
+    def is_pr_blocked(self, pr: dict[str, Any]) -> tuple[bool, list[str]]:
+        """Check if a PR is blocked from merging (synchronous wrapper).
+
+        This method provides backward compatibility with the old scanner interface.
+        It uses the same logic as dependamerge's _analyze_pr_node.
+
+        Args:
+            pr: PR data from GraphQL
+
+        Returns:
+            Tuple of (is_blocked, list of blocking reasons as strings)
+        """
+        # Use asyncio to call the async version
+        return asyncio.run(self._is_pr_blocked_async(pr))
+
+    async def _is_pr_blocked_async(
+        self, pr: dict[str, Any]
+    ) -> tuple[bool, list[str]]:
+        """Async version of is_pr_blocked.
+
+        Args:
+            pr: PR data from GraphQL
+
+        Returns:
+            Tuple of (is_blocked, list of blocking reasons as strings)
+        """
+        # Use dependamerge's _analyze_pr_node to check if PR is blocked
+        result = await self.service._analyze_pr_node(
+            repo_full_name="temp/repo",  # Not used for just checking blocked status
+            pr=pr,
+            include_drafts=False,
+        )
+
+        if result is None:
+            return False, []
+
+        # Extract reason descriptions
+        reasons = [reason.description for reason in result.reasons]
+        return True, reasons