housebroken-cli 0.2.5__tar.gz → 0.2.7__tar.gz

{housebroken_cli-0.2.5 → housebroken_cli-0.2.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: housebroken-cli
-Version: 0.2.5
+Version: 0.2.7
 Summary: Agent-written pull requests that do not make a mess in someone else's house.
 Project-URL: Repository, https://github.com/lenamonj/housebroken
 License-Expression: MIT

{housebroken_cli-0.2.5 → housebroken_cli-0.2.7}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "housebroken-cli"
-version = "0.2.5"
+version = "0.2.7"
 description = "Agent-written pull requests that do not make a mess in someone else's house."
 dynamic = ["readme"]
 license = "MIT"

{housebroken_cli-0.2.5 → housebroken_cli-0.2.7}/scripts/inbox.sh

@@ -120,10 +120,24 @@ if [ "$do_ci" -eq 1 ]; then
   echo "== census of open pull requests: failing, blocked, conflicting, changes requested, draft =="
   prs="$(gh search prs --author "$user" --state open --limit 100 --json repository,number,isDraft --jq '.[] | "\(.repository.nameWithOwner) \(.number) \(.isDraft)"' 2>/dev/null)"
   total=$(printf '%s\n' "$prs" | grep -c . || true)
+  # GitHub search returns a short page when it is throttled, and a census of half
+  # the pull requests reads exactly like a clean one. Compare against the count the
+  # API itself reports and say so loudly rather than silently flagging fewer.
+  claimed="$(gh api "search/issues?q=is:pr+author:$user+is:open&per_page=1" --jq '.total_count' 2>/dev/null)"
+  case "$claimed" in
+    ''|*[!0-9]*) echo "CENSUS UNVERIFIED: could not read the open pull request count from the API" ;;
+    *) [ "$total" -ge "$claimed" ] || echo "CENSUS INCOMPLETE: search returned $total of $claimed open pull requests, so this census covers only part of them; re-run when the search rate limit recovers" ;;
+  esac
   flagged=0
+  unreadable=0
+  unreadable_list=""
   while read -r repo num draft; do
     [ -n "$repo" ] || continue
-    j="$(gh pr view "$num" --repo "$repo" --json mergeable,reviewDecision,statusCheckRollup 2>/dev/null)" || continue
+    j="$(gh pr view "$num" --repo "$repo" --json mergeable,reviewDecision,statusCheckRollup 2>/dev/null)"
+    if [ -z "$j" ]; then
+      # A pull request we could not read is not a clean one: say so rather than skipping in silence.
+      unreadable=$((unreadable + 1)); unreadable_list="$unreadable_list $repo#$num"; continue
+    fi
     why="$(jq -r --arg draft "$draft" '
       [ ( [.statusCheckRollup[]? | (.conclusion // .state // "")] | if any(. == "FAILURE" or . == "ERROR") then "FAILING" elif any(. == "ACTION_REQUIRED") then "needs-run-approval" else empty end ),
         ( if .mergeable == "CONFLICTING" then "CONFLICTING" else empty end ),
@@ -135,6 +149,7 @@ if [ "$do_ci" -eq 1 ]; then
       echo "$repo#$num: $why${fails:+  [$fails]}"
     fi
   done <<<"$prs"
+  [ "$unreadable" -eq 0 ] || echo "CENSUS UNREADABLE: could not read $unreadable of $total pull requests, so their state is unknown, not clean:$unreadable_list"
   echo "$flagged of $total open pull requests flagged"
 fi