housebroken-cli 0.6.7__tar.gz → 0.6.8__tar.gz

{housebroken_cli-0.6.7 → housebroken_cli-0.6.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: housebroken-cli
-Version: 0.6.7
+Version: 0.6.8
 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.6.7 → housebroken_cli-0.6.8}/pyproject.toml

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

{housebroken_cli-0.6.7 → housebroken_cli-0.6.8}/scripts/branch-check.sh

@@ -184,6 +184,6 @@ fi
 # The stamp file-pr.sh reads: this head passed, against this base, at this time.
 head=$(git rev-parse HEAD)
 mkdir -p "$home/branch-checks"
-printf 'head: %s\nbase: %s\nchecked: %s\n' "$head" "$(git rev-parse "$base")" "$(date -u +%Y-%m-%dT%H:%M:%SZ)" > "$home/branch-checks/$head"
+printf 'head: %s\nbase: %s\nbase-ref: %s\nchecked: %s\n' "$head" "$(git rev-parse "$base")" "$base" "$(date -u +%Y-%m-%dT%H:%M:%SZ)" > "$home/branch-checks/$head"
 echo
 echo "branch-check: nothing mechanical blocks this branch."

{housebroken_cli-0.6.7 → housebroken_cli-0.6.8}/scripts/file-pr.sh

@@ -47,9 +47,11 @@ gates:
               be under 24 hours old and not read RESULT: BAN; when it reads
               RESULT: DISCLOSE the body must name the AI use
   prose       no tool footer, session trailer, co-author trailer, em dash or
-              en dash, in any case, in the --body-file or the --title
+              en dash, in any case, in the --body-file or the --title; a
+              body of at most 120 words
   branch      $HOUSEBROKEN_HOME/branch-checks/<head> must exist, written by
-              branch-check.sh when this head passed
+              branch-check.sh when this head passed, against the branch this
+              files into (--base, or the repository's default branch)
   review      run from the clone being filed; the report at
               $HOUSEBROKEN_HOME/reviews/review-<owner>-<repo>-<head12>.md must
               pass review.sh check for this head and the --body-file, and an
@@ -128,6 +130,7 @@ check_text() {
 # as its own argument or after an equals sign.
 bodyfile=""
 title=""
+filebase=""
 pass=()
 while [ "$#" -gt 0 ]; do
   case "$1" in
@@ -135,8 +138,10 @@ while [ "$#" -gt 0 ]; do
     --body-file=*) bodyfile="${1#--body-file=}"; pass+=("$1") ;;
     -t|--title) title="${2:-}"; pass+=("$1" "${2:-}"); shift ;;
     --title=*) title="${1#--title=}"; pass+=("$1") ;;
-    -B|--base|-l|--label|-a|--assignee|-r|--reviewer|-m|--milestone|-p|--project) pass+=("$1" "${2:-}"); shift ;;
-    --base=*|--label=*|--assignee=*|--reviewer=*|--milestone=*|--project=*) pass+=("$1") ;;
+    -B|--base) filebase="${2:-}"; pass+=("$1" "${2:-}"); shift ;;
+    --base=*) filebase="${1#--base=}"; pass+=("$1") ;;
+    -l|--label|-a|--assignee|-r|--reviewer|-m|--milestone|-p|--project) pass+=("$1" "${2:-}"); shift ;;
+    --label=*|--assignee=*|--reviewer=*|--milestone=*|--project=*) pass+=("$1") ;;
     -d|--draft|--no-maintainer-edit) pass+=("$1") ;;
     *)
       echo "REFUSED: file-pr.sh does not pass $1 to gh pr create: the body comes from one --body-file, the head is the branch you are on, the repository is the first argument" >&2
@@ -152,6 +157,11 @@ if [ "$bodyfile" = "-" ] || [ ! -f "$bodyfile" ]; then
 fi
 check_text "body file $bodyfile" "$(cat "$bodyfile")"
 check_text "--title" "$title"
+words=$(wc -w < "$bodyfile" | tr -d ' ')
+if [ "$words" -gt 120 ]; then
+  echo "REFUSED: the body is $words words - say what was wrong, what the change does and how it was verified in 120 or fewer" >&2
+  exit 2
+fi
 if [ "$policy_result" = "DISCLOSE" ] && ! grep -qiE '\bAI\b|LLM|Claude|Copilot|ChatGPT|language model|assist' "$bodyfile"; then
   echo "REFUSED: $repo requires AI use to be disclosed ($policy) and the body does not name it" >&2
   exit 2
@@ -162,6 +172,22 @@ if [ ! -f "$home/branch-checks/$head" ]; then
   echo "REFUSED: branch-check.sh has not passed head ${head:0:12} - run it from this clone first" >&2
   exit 2
 fi
+# Every diff gate read the diff against the stamp's base; it must be the branch this files into.
+checked=$(sed -n 's/^base-ref: //p' "$home/branch-checks/$head" | tr -d '\r')
+if [ -z "$checked" ]; then
+  echo "REFUSED: the branch-check stamp for head ${head:0:12} names no base - rerun branch-check.sh" >&2
+  exit 2
+fi
+target="$filebase"
+if [ -z "$target" ]; then
+  target=$(gh repo view "$repo" --json defaultBranchRef --jq .defaultBranchRef.name 2>/dev/null | tr -d '\r')
+  [ -n "$target" ] || { echo "REFUSED: cannot read the default branch of $repo - pass --base" >&2; exit 2; }
+fi
+case "$checked" in
+  "$target"|*/"$target") ;;
+  *) echo "REFUSED: branch-check.sh measured this head against $checked, but it is filed into $target - rerun branch-check.sh --base <remote>/$target" >&2
+     exit 2 ;;
+esac
 review="$home/reviews/review-${repo%%/*}-${repo##*/}-${head:0:12}.md"
 if [ ! -f "$review" ]; then
   echo "REFUSED: no adversarial review at $review - run review.sh brief and have a different model attack this head" >&2

{housebroken_cli-0.6.7 → housebroken_cli-0.6.8}/scripts/prior-art.sh

@@ -9,7 +9,9 @@
 # 2026-09-04) argued against a ruling already made in CLOSED issue #98, which
 # nobody read, so this script prints the last non-author comment on every closed
 # item. A term containing a slash or a dot is treated as a file path and the
-# commit history of that path is scanned for the PRs that touched it.
+# commit history of that path is scanned for the PRs that touched it, and
+# every open PR that changes it is listed, which no text search finds
+# (Shopify/ruby-lsp #4103 deleted the file a fix was written against).
 # Written 2026-09-07.
 #
 # Usage:
@@ -34,7 +36,8 @@ usage: prior-art.sh owner/repo <term>... [--out]
 
   owner/repo  the upstream repository to search
   <term>      a symbol, phrase or file path; a term with a slash or a dot is
-              also treated as a path and its commit history is scanned
+              also treated as a path: its commit history is scanned and
+              every open PR that changes it is listed
   --out       save the printout under $HOUSEBROKEN_HOME/prior-art/
 
 environment:
@@ -70,7 +73,8 @@ calls=0
 items=0
 tmp=$(mktemp) || { echo "prior-art.sh: cannot create temp file" >&2; exit 1; }
 row=$(mktemp) || { echo "prior-art.sh: cannot create temp file" >&2; exit 1; }
-trap 'rm -f "$tmp" "$row"' EXIT
+openfiles=$(mktemp) || { echo "prior-art.sh: cannot create temp file" >&2; exit 1; }
+trap 'rm -f "$tmp" "$row" "$openfiles"' EXIT
 
 # last comment on a closed item written by somebody other than its author
 last_word() {
@@ -132,6 +136,30 @@ for term in "${terms[@]}"; do
         printf '| PR | #%s | merged | %s | %s | %s |  |\n' "$num" "$date" "$author" "$subject" >>"$tmp"
         items=$((items+1))
       done <"$row"
+
+      # an open PR that changes the path is prior art even when no title or body names it
+      if [ ! -s "$openfiles" ]; then
+        # shellcheck disable=SC2016  # $p is a jq variable
+        gh pr list -R "$repo" --state open --limit 200 --json number,createdAt,author,title,changedFiles,files \
+          --jq '.[] | . as $p | (if .changedFiles > (.files | length) then ["MORE"] else [.files[].path] end)[] |
+                [($p.number|tostring), $p.createdAt[0:10], $p.author.login,
+                 ($p.title|gsub("[|]";"/")|gsub("[\n\r\t]";" ")), .] | @tsv' >"$openfiles" ||
+          { echo "prior-art.sh: open PR listing failed for $repo" >&2; exit 1; }
+        calls=$((calls+1))
+        # GitHub returns at most 100 files per PR here; a larger PR is read in full
+        awk -F'\t' '$5 == "MORE"' "$openfiles" | while IFS=$'\t' read -r num date author title _; do
+          gh api --paginate "repos/$repo/pulls/$num/files?per_page=100" --jq '.[].filename' |
+            while IFS= read -r f; do printf '%s\t%s\t%s\t%s\t%s\n' "$num" "$date" "$author" "$title" "$f"; done
+        done >"$row"
+        cat "$row" >>"$openfiles"
+        [ -s "$openfiles" ] || printf 'none\n' >"$openfiles"
+      fi
+      while IFS=$'\t' read -r num date author title path; do
+        case "$path" in "$term"|*/"$term") ;; *) continue ;; esac
+        grep -q "^| PR | #$num | open, changes this file |" "$tmp" && continue
+        printf '| PR | #%s | open, changes this file | %s | %s | %s |  |\n' "$num" "$date" "$author" "$title" >>"$tmp"
+        items=$((items+1))
+      done <"$openfiles"
       ;;
   esac
 done

{housebroken_cli-0.6.7 → housebroken_cli-0.6.8}/scripts/verify-filed-pr.sh

@@ -97,6 +97,7 @@ view comments '.comments[] | "   " + .author.login + ": " + (.body | gsub("\n";
 
 echo ""
 if [ "$fail" -ne 0 ]; then echo "PR-VERIFY: PROBLEMS ABOVE"
+elif [ "${waiting:-0}" -gt 0 ]; then echo "PR-VERIFY: NOT SETTLED ($waiting workflow run(s) wait for a maintainer to approve them; nothing settles until one does, so move on)"
 elif [ "$unsettled" -ne 0 ]; then echo "PR-VERIFY: NOT SETTLED (CI has not run on this head)"
 else echo "PR-VERIFY: OK"; fi
 [ "$fail" -eq 0 ] && [ "$unsettled" -ne 0 ] && exit 3

{housebroken_cli-0.6.7 → housebroken_cli-0.6.8}/skills/housebroken/SKILL.md

@@ -179,8 +179,11 @@ report it in one line and do not proceed to the next step.
    is missing or older than a day, when the policy printout is missing, older
    than a day or reads BAN, when the policy reads DISCLOSE and the body does
    not disclose, when the body carries a footer, a trailer or a dash
-   character, when `housebroken branch` has not stamped the exact head, or
-   when there is no POST AS IS review of the exact head and body being filed.
+   character or runs past 120 words, when `housebroken branch` has not
+   stamped the exact head against the branch it is filed into (`--base`
+   to both when the project takes pull requests on a branch other than its
+   default), or when there is no POST AS IS review of the exact head and
+   body being filed.
    A refusal is a result; fix the cause.
 
 10. **Watch it land.**

housebroken_cli-0.6.8/src/housebroken/__init__.py

@@ -0,0 +1 @@
+__version__ = "0.6.8"

housebroken_cli-0.6.7/src/housebroken/__init__.py

@@ -1 +0,0 @@
-__version__ = "0.6.7"