reviewloop 0.3.2__tar.gz → 0.3.4__tar.gz

{reviewloop-0.3.2 → reviewloop-0.3.4}/.claude/skills/reviewloop/SKILL.md

@@ -10,9 +10,20 @@ allowed-tools:
 
 ## Autonomous Review Loop
 
+**CRITICAL: Never merge a PR unless the user explicitly asks you to.**
+
 This is an automated review loop. Follow these steps:
 
-### 1. Wait for CI to complete
+### 1. Prepare branch and PR
+
+Before starting the review cycle, ensure all changes are committed, pushed, and a PR exists.
+
+1. Verify you are NOT on the default branch. If on the default branch, inform the user and stop.
+2. If there are uncommitted changes (staged or unstaged), commit them with a descriptive message. Skip if the working directory is clean.
+3. Push commits to the remote. If no upstream tracking branch exists, push with the `-u` flag. Skip if already up-to-date.
+4. Check if a PR exists for the current branch (`gh pr view`). If none exists, create one with `gh pr create --fill`.
+
+### 2. Wait for CI to complete
 
 Run `.claude/skills/reviewloop/scripts/review-wait.sh` to poll for CI completion.
 
@@ -21,7 +32,7 @@ Run `.claude/skills/reviewloop/scripts/review-wait.sh` to poll for CI completion
 - Times out after 10 minutes (configurable with `--timeout=SECONDS`)
 - Exits immediately if no CI is in progress
 
-### 2. Fetch and address comments
+### 3. Fetch and address comments
 
 Once CI completes:
 
@@ -32,14 +43,18 @@ Once CI completes:
 5. For inline threads: resolve via the `gh api graphql` mutation shown in the file (whether addressed or rejected)
 6. For general review comments: react with thumbs-up via the mutation shown in the file to mark as processed (even if no action was needed)
 
-### 3. Verify and commit
+### 4. Verify and commit
 
 1. Run `.claude/skills/reviewloop/scripts/review-comments.sh` again to verify no unresolved comments remain
 2. Run your project's linting and type-checking commands to ensure code quality
 3. Commit and push changes
 
-### 4. Loop
+### 5. Loop
+
+Go back to step 2 - wait for the next review triggered by the push.
 
-Go back to step 1 - wait for the next review triggered by the push.
+**Exit condition**: When `.claude/skills/reviewloop/scripts/review-comments.sh` shows no comments, provide a summary with:
 
-**Exit condition**: When `.claude/skills/reviewloop/scripts/review-comments.sh` shows no comments. Give a brief high-level summary of what was addressed.
+1. **PR link**: The full clickable URL of the PR (from step 1 or via `gh pr view --json url -q '.url'`)
+2. **Overall PR summary**: A brief high-level summary of what the PR accomplishes overall
+3. **Review loop changes**: A summary of what was specifically changed during the review loop to address reviewer comments. If no changes were made during the review loop, state that no changes were needed.

{reviewloop-0.3.2/src/reviewloop_cli/templates → reviewloop-0.3.4/.claude/skills/reviewloop}/scripts/review-comments.sh

@@ -26,7 +26,7 @@ get_repo_info() {
 }
 
 get_current_user() {
-    gh api user -q '.login' 2>&1 || die "Failed to get current user"
+    gh api user -q '.login' 2>&1 | tr -d '\r' || die "Failed to get current user"
 }
 
 run_graphql() {
@@ -63,8 +63,8 @@ main() {
 
     local repo_info owner repo pr_number current_user
     repo_info=$(get_repo_info)
-    owner=$(echo "$repo_info" | jq -r '.owner.login')
-    repo=$(echo "$repo_info" | jq -r '.name')
+    owner=$(echo "$repo_info" | jq -r '.owner.login' | tr -d '\r')
+    repo=$(echo "$repo_info" | jq -r '.name' | tr -d '\r')
     pr_number=$(get_pr_number)
     current_user=$(get_current_user)
 
@@ -135,7 +135,7 @@ EOF
             while IFS= read -r thread; do
                 print_thread "$thread" "$is_first"
                 is_first=false
-            done < <(echo "$unresolved_threads" | jq -c '.[]')
+            done < <(echo "$unresolved_threads" | jq -c '.[]' | tr -d '\r')
         fi
 
         if [[ "$review_count" -gt 0 ]]; then
@@ -149,7 +149,7 @@ EOF
             while IFS= read -r review; do
                 print_review "$review" "$is_first"
                 is_first=false
-            done < <(echo "$review_bodies" | jq -c '.[]')
+            done < <(echo "$review_bodies" | jq -c '.[]' | tr -d '\r')
         fi
     } > .reviews/prComments.md
 

{reviewloop-0.3.2/src/reviewloop_cli/templates → reviewloop-0.3.4/.claude/skills/reviewloop}/scripts/review-wait.sh

@@ -32,7 +32,7 @@ get_ci_state() {
     checks_json=$(gh pr checks "$pr_number" --json name,state 2>&1) || die "Failed to fetch PR checks: $checks_json"
 
     # Get all check states
-    states=$(echo "$checks_json" | jq -r '[.[] | .state] | if length == 0 then "" else .[] end' 2>/dev/null)
+    states=$(echo "$checks_json" | jq -r '[.[] | .state] | if length == 0 then "" else .[] end' 2>/dev/null | tr -d '\r')
 
     [[ -z "$states" ]] && return
 

{reviewloop-0.3.2 → reviewloop-0.3.4}/.github/workflows/release.yml

@@ -38,7 +38,7 @@ jobs:
             python -c "from importlib.metadata import version; print(version('reviewloop'))"
 
       - name: Store distributions
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v6
         with:
           name: python-package-distributions
           path: dist/
@@ -55,7 +55,7 @@ jobs:
       actions: read
     steps:
       - name: Download distributions
-        uses: actions/download-artifact@v4
+        uses: actions/download-artifact@v7
         with:
           name: python-package-distributions
           path: dist/
@@ -78,7 +78,7 @@ jobs:
       actions: read
     steps:
       - name: Download distributions
-        uses: actions/download-artifact@v4
+        uses: actions/download-artifact@v7
         with:
           name: python-package-distributions
           path: dist/
@@ -100,7 +100,7 @@ jobs:
       - uses: actions/checkout@v6
 
       - name: Download distributions
-        uses: actions/download-artifact@v4
+        uses: actions/download-artifact@v7
         with:
           name: python-package-distributions
           path: dist/

{reviewloop-0.3.2 → reviewloop-0.3.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: reviewloop
-Version: 0.3.2
+Version: 0.3.4
 Summary: Autonomous review loop - automates the review-fix-push cycle
 Project-URL: Homepage, https://github.com/fabianboth/autonomous-review-loop
 Project-URL: Repository, https://github.com/fabianboth/autonomous-review-loop.git

{reviewloop-0.3.2 → reviewloop-0.3.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "reviewloop"
-version = "0.3.2"
+version = "0.3.4"
 description = "Autonomous review loop - automates the review-fix-push cycle"
 readme = "README_PYPI.md"
 license = "MIT"

{reviewloop-0.3.2 → reviewloop-0.3.4}/src/reviewloop_cli/templates/SKILL.md

@@ -10,9 +10,20 @@ allowed-tools:
 
 ## Autonomous Review Loop
 
+**CRITICAL: Never merge a PR unless the user explicitly asks you to.**
+
 This is an automated review loop. Follow these steps:
 
-### 1. Wait for CI to complete
+### 1. Prepare branch and PR
+
+Before starting the review cycle, ensure all changes are committed, pushed, and a PR exists.
+
+1. Verify you are NOT on the default branch. If on the default branch, inform the user and stop.
+2. If there are uncommitted changes (staged or unstaged), commit them with a descriptive message. Skip if the working directory is clean.
+3. Push commits to the remote. If no upstream tracking branch exists, push with the `-u` flag. Skip if already up-to-date.
+4. Check if a PR exists for the current branch (`gh pr view`). If none exists, create one with `gh pr create --fill`.
+
+### 2. Wait for CI to complete
 
 Run `.claude/skills/reviewloop/scripts/review-wait.sh` to poll for CI completion.
 
@@ -21,7 +32,7 @@ Run `.claude/skills/reviewloop/scripts/review-wait.sh` to poll for CI completion
 - Times out after 10 minutes (configurable with `--timeout=SECONDS`)
 - Exits immediately if no CI is in progress
 
-### 2. Fetch and address comments
+### 3. Fetch and address comments
 
 Once CI completes:
 
@@ -32,14 +43,18 @@ Once CI completes:
 5. For inline threads: resolve via the `gh api graphql` mutation shown in the file (whether addressed or rejected)
 6. For general review comments: react with thumbs-up via the mutation shown in the file to mark as processed (even if no action was needed)
 
-### 3. Verify and commit
+### 4. Verify and commit
 
 1. Run `.claude/skills/reviewloop/scripts/review-comments.sh` again to verify no unresolved comments remain
 2. Run your project's linting and type-checking commands to ensure code quality
 3. Commit and push changes
 
-### 4. Loop
+### 5. Loop
+
+Go back to step 2 - wait for the next review triggered by the push.
 
-Go back to step 1 - wait for the next review triggered by the push.
+**Exit condition**: When `.claude/skills/reviewloop/scripts/review-comments.sh` shows no comments, provide a summary with:
 
-**Exit condition**: When `.claude/skills/reviewloop/scripts/review-comments.sh` shows no comments. Give a brief high-level summary of what was addressed.
+1. **PR link**: The full clickable URL of the PR (from step 1 or via `gh pr view --json url -q '.url'`)
+2. **Overall PR summary**: A brief high-level summary of what the PR accomplishes overall
+3. **Review loop changes**: A summary of what was specifically changed during the review loop to address reviewer comments. If no changes were made during the review loop, state that no changes were needed.

{reviewloop-0.3.2/.claude/skills/reviewloop → reviewloop-0.3.4/src/reviewloop_cli/templates}/scripts/review-comments.sh

@@ -26,7 +26,7 @@ get_repo_info() {
 }
 
 get_current_user() {
-    gh api user -q '.login' 2>&1 || die "Failed to get current user"
+    gh api user -q '.login' 2>&1 | tr -d '\r' || die "Failed to get current user"
 }
 
 run_graphql() {
@@ -63,8 +63,8 @@ main() {
 
     local repo_info owner repo pr_number current_user
     repo_info=$(get_repo_info)
-    owner=$(echo "$repo_info" | jq -r '.owner.login')
-    repo=$(echo "$repo_info" | jq -r '.name')
+    owner=$(echo "$repo_info" | jq -r '.owner.login' | tr -d '\r')
+    repo=$(echo "$repo_info" | jq -r '.name' | tr -d '\r')
     pr_number=$(get_pr_number)
     current_user=$(get_current_user)
 
@@ -135,7 +135,7 @@ EOF
             while IFS= read -r thread; do
                 print_thread "$thread" "$is_first"
                 is_first=false
-            done < <(echo "$unresolved_threads" | jq -c '.[]')
+            done < <(echo "$unresolved_threads" | jq -c '.[]' | tr -d '\r')
         fi
 
         if [[ "$review_count" -gt 0 ]]; then
@@ -149,7 +149,7 @@ EOF
             while IFS= read -r review; do
                 print_review "$review" "$is_first"
                 is_first=false
-            done < <(echo "$review_bodies" | jq -c '.[]')
+            done < <(echo "$review_bodies" | jq -c '.[]' | tr -d '\r')
         fi
     } > .reviews/prComments.md
 

{reviewloop-0.3.2/.claude/skills/reviewloop → reviewloop-0.3.4/src/reviewloop_cli/templates}/scripts/review-wait.sh

@@ -32,7 +32,7 @@ get_ci_state() {
     checks_json=$(gh pr checks "$pr_number" --json name,state 2>&1) || die "Failed to fetch PR checks: $checks_json"
 
     # Get all check states
-    states=$(echo "$checks_json" | jq -r '[.[] | .state] | if length == 0 then "" else .[] end' 2>/dev/null)
+    states=$(echo "$checks_json" | jq -r '[.[] | .state] | if length == 0 then "" else .[] end' 2>/dev/null | tr -d '\r')
 
     [[ -z "$states" ]] && return
 

{reviewloop-0.3.2 → reviewloop-0.3.4}/uv.lock

@@ -86,7 +86,7 @@ wheels = [
 
 [[package]]
 name = "reviewloop"
-version = "0.3.1"
+version = "0.3.4"
 source = { editable = "." }
 dependencies = [
     { name = "readchar" },