github-rest-api 0.52.0__tar.gz → 0.53.1__tar.gz

{github_rest_api-0.52.0 → github_rest_api-0.53.1}/.github/workflows/merge-pr.yaml

@@ -42,13 +42,6 @@ on:
 permissions:
   contents: write
   pull-requests: write
-concurrency:
-  # Every run rescans all open PRs, so overlapping runs are pure duplicate work
-  # and the newest run always sees the freshest state. Do not cancel in
-  # progress: a run cancelled midway through the merge API call is worse than a
-  # redundant one.
-  group: merge-pull-request
-  cancel-in-progress: false
 jobs:
   merge_pull_request:
     # A triggering workflow that failed, was cancelled or was skipped left the
@@ -58,6 +51,15 @@ jobs:
     if: |
       github.event_name != 'workflow_run'
       || github.event.workflow_run.conclusion == 'success'
+    concurrency:
+      # Every run rescans all open PRs, so overlapping runs are pure duplicate
+      # work and the newest run always sees the freshest state. Do not cancel
+      # in progress: a run cancelled midway through the merge API call is worse
+      # than a redundant one. This is set on the job rather than the workflow
+      # so that a run whose `if` skips the job never joins the group, where it
+      # would replace (cancel) another PR's pending merge run.
+      group: merge-pull-request
+      cancel-in-progress: false
     uses: legendu-net/.github/.github/workflows/merge-pr.yaml@main
     with:
       dry_run: ${{ inputs.dry_run || false }}

{github_rest_api-0.52.0 → github_rest_api-0.53.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: github-rest-api
-Version: 0.52.0
+Version: 0.53.1
 Summary: Simple wrapper of GitHub REST APIs.
 Author-email: Ben Du <longendu@yahoo.com>
 Classifier: Programming Language :: Python :: 3 :: Only

{github_rest_api-0.52.0 → github_rest_api-0.53.1}/github_rest_api/scripts/lint_repo.py

@@ -22,6 +22,7 @@ import argparse
 import os
 import re
 import shutil
+import stat
 import subprocess as sp
 import sys
 from collections.abc import Callable, Sequence
@@ -29,6 +30,7 @@ from dataclasses import dataclass, field
 from pathlib import Path
 from typing import Literal, cast
 
+from dulwich.index import IndexEntry
 from dulwich.repo import Repo
 
 #: The outcome of running one check.
@@ -44,6 +46,12 @@ LANGUAGES = ["python", "rust", "golang", "bash", "fish", "lua", "markdown"]
 def list_tracked_files(root: Path) -> list[str]:
     """List all git-tracked files in a repository.
 
+    Symbolic links are excluded: a link's content is linted through its
+    target (when tracked), and linting the link too would run every check
+    twice on the same content -- or, worse, with conflicting options (e.g. a
+    `CLAUDE.md` -> `AGENTS.md` link would get a different mdformat flavor than
+    its target).
+
     :param root: The root directory of the git repository (not an arbitrary
         subdirectory of one -- the returned paths, and every command a check
         later runs, are relative to this exact directory).
@@ -54,7 +62,10 @@ def list_tracked_files(root: Path) -> list[str]:
         # `surrogateescape` so a non-UTF-8 filename (which git tracks fine)
         # doesn't crash the whole run; it just won't print legibly.
         return sorted(
-            name.decode(errors="surrogateescape") for name in repo.open_index()
+            name.decode(errors="surrogateescape")
+            for name, entry in repo.open_index().items()
+            # A conflicted entry (mid-merge) has no single mode; keep it.
+            if not (isinstance(entry, IndexEntry) and stat.S_ISLNK(entry.mode))
         )
     finally:
         repo.close()
@@ -363,7 +374,7 @@ def build_markdown_checks(files: Sequence[str], fix: bool = False) -> list[Check
     md_files = _files_matching(files, "*.md")
     if not md_files:
         return []
-    skill_files = [f for f in md_files if Path(f).name == "SKILL.md"]
+    skill_files = [f for f in md_files if Path(f).name in ("SKILL.md", "AGENTS.md")]
     other_files = [f for f in md_files if f not in skill_files]
     checks = []
     if other_files:
@@ -380,7 +391,7 @@ def build_markdown_checks(files: Sequence[str], fix: bool = False) -> list[Check
     if skill_files:
         checks.append(
             Check(
-                "markdown: mdformat (SKILL.md)",
+                "markdown: mdformat (SKILL.md/AGENTS.md)",
                 command=_cmd(
                     fix,
                     ["mdformat", "--number", *skill_files],

{github_rest_api-0.52.0 → github_rest_api-0.53.1}/pyproject.toml

@@ -4,7 +4,7 @@ requires = [ "hatchling" ]
 
 [project]
 name = "github-rest-api"
-version = "0.52.0"
+version = "0.53.1"
 description = "Simple wrapper of GitHub REST APIs."
 readme = "README.md"
 authors = [ { name = "Ben Du", email = "longendu@yahoo.com" } ]

{github_rest_api-0.52.0 → github_rest_api-0.53.1}/tests/test_lint_repo.py

@@ -58,6 +58,16 @@ def test_list_tracked_files_ignores_untracked_files(tmp_path):
     assert list_tracked_files(tmp_path) == ["tracked.py"]
 
 
+def test_list_tracked_files_excludes_symlinks(tmp_path):
+    _init_repo(tmp_path, {"AGENTS.md": "# Agents\n"})
+    (tmp_path / "CLAUDE.md").symlink_to("AGENTS.md")
+    porcelain.add(str(tmp_path), paths=["CLAUDE.md"])
+    # The link is tracked by git, but only its target should be linted.
+    with Repo(str(tmp_path)) as repo:
+        assert b"CLAUDE.md" in repo.open_index()
+    assert list_tracked_files(tmp_path) == ["AGENTS.md"]
+
+
 def test_list_tracked_files_survives_a_non_utf8_filename(tmp_path):
     # Git tracks arbitrary byte sequences as filenames; decoding with
     # `surrogateescape` must not raise on one that isn't valid UTF-8. Written
@@ -174,18 +184,21 @@ def test_build_rust_scripts_checks_includes_all_files():
 def test_build_markdown_checks_splits_skill_md(tmp_path):
     (tmp_path / "docs").mkdir()
     (tmp_path / "docs" / "SKILL.md").write_text("")
+    (tmp_path / "AGENTS.md").write_text("")
     (tmp_path / "README.md").write_text("")
-    checks = build_markdown_checks(["README.md", "docs/SKILL.md"])
+    checks = build_markdown_checks(["README.md", "AGENTS.md", "docs/SKILL.md"])
     by_name = {c.name: c for c in checks}
     assert by_name["markdown: mdformat"].command == ["mdformat", "--check", "README.md"]
-    assert by_name["markdown: mdformat (SKILL.md)"].command == [
+    assert by_name["markdown: mdformat (SKILL.md/AGENTS.md)"].command == [
         "mdformat",
         "--check",
         "--number",
+        "AGENTS.md",
         "docs/SKILL.md",
     ]
     assert by_name["markdown: codespell"].command == [
         "codespell",
+        "AGENTS.md",
         "README.md",
         "docs/SKILL.md",
     ]
@@ -531,24 +544,35 @@ def test_build_lua_checks_fix_mode():
 def test_build_markdown_checks_fix_mode(tmp_path):
     (tmp_path / "docs").mkdir()
     (tmp_path / "docs" / "SKILL.md").write_text("")
+    (tmp_path / "AGENTS.md").write_text("")
     (tmp_path / "README.md").write_text("")
-    checks = build_markdown_checks(["README.md", "docs/SKILL.md"], fix=True)
+    checks = build_markdown_checks(
+        ["README.md", "AGENTS.md", "docs/SKILL.md"], fix=True
+    )
     by_name = {c.name: c.command for c in checks}
     assert by_name["markdown: mdformat"] == ["mdformat", "README.md"]
-    assert by_name["markdown: mdformat (SKILL.md)"] == [
+    assert by_name["markdown: mdformat (SKILL.md/AGENTS.md)"] == [
         "mdformat",
         "--number",
+        "AGENTS.md",
         "docs/SKILL.md",
     ]
     assert by_name["markdown: codespell"] == [
         "codespell",
         "-w",
+        "AGENTS.md",
         "README.md",
         "docs/SKILL.md",
     ]
     # lychee has no autofix and stays non-fatal: unchanged.
     lychee = next(c for c in checks if c.name == "markdown: lychee")
-    assert lychee.command == ["lychee", "--no-progress", "README.md", "docs/SKILL.md"]
+    assert lychee.command == [
+        "lychee",
+        "--no-progress",
+        "AGENTS.md",
+        "README.md",
+        "docs/SKILL.md",
+    ]
     assert lychee.fatal is False
 
 

{github_rest_api-0.52.0 → github_rest_api-0.53.1}/uv.lock

@@ -542,7 +542,7 @@ wheels = [
 
 [[package]]
 name = "github-rest-api"
-version = "0.52.0"
+version = "0.53.1"
 source = { editable = "." }
 dependencies = [
     { name = "dulwich" },