pi-pr-review 1.9.0 → 1.10.1

package/.github/workflows/release-please.yml

@@ -36,6 +36,36 @@ jobs:
           manifest-file: .release-please-manifest.json
           token: ${{ steps.app-token.outputs.token }}
 
+      - name: Enable squash auto-merge for release PRs
+        if: ${{ steps.release.outputs.prs_created == 'true' }}
+        env:
+          GH_TOKEN: ${{ steps.app-token.outputs.token }}
+          RELEASE_PRS: ${{ steps.release.outputs.prs }}
+        run: |
+          jq -r '.[].number' <<< "$RELEASE_PRS" | while read -r number; do
+            merge_method="$(gh pr view "$number" --repo "$GITHUB_REPOSITORY" --json autoMergeRequest --jq '.autoMergeRequest.mergeMethod // "NONE"')"
+            echo "Release PR #$number auto-merge method: $merge_method"
+
+            case "$merge_method" in
+              SQUASH)
+                echo "Squash auto-merge is already enabled for release PR #$number"
+                continue
+                ;;
+              MERGE|REBASE)
+                echo "Replacing $merge_method auto-merge for release PR #$number"
+                gh pr merge "$number" --repo "$GITHUB_REPOSITORY" --disable-auto
+                ;;
+              NONE)
+                ;;
+              *)
+                echo "Unexpected auto-merge method for release PR #$number: $merge_method" >&2
+                exit 1
+                ;;
+            esac
+
+            gh pr merge "$number" --repo "$GITHUB_REPOSITORY" --auto --squash
+          done
+
   publish:
     name: Test and publish to npm
     needs: release

package/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "1.9.0"
+  ".": "1.10.1"
 }

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [1.10.1](https://github.com/10ego/pi-pr-review/compare/v1.10.0...v1.10.1) (2026-07-14)
+
+
+### Bug Fixes
+
+* **publish:** recognize direct review comment requests ([#23](https://github.com/10ego/pi-pr-review/issues/23)) ([83b003b](https://github.com/10ego/pi-pr-review/commit/83b003b6a37fab9c0ff32d482e41010dc9893afa))
+
+## [1.10.0](https://github.com/10ego/pi-pr-review/compare/v1.9.0...v1.10.0) (2026-07-14)
+
+
+### Features
+
+* **release:** auto-merge release pull requests ([#21](https://github.com/10ego/pi-pr-review/issues/21)) ([56ea9a9](https://github.com/10ego/pi-pr-review/commit/56ea9a9585069193181c78700223cb659af72750))
+
 ## [1.9.0](https://github.com/10ego/pi-pr-review/compare/v1.8.0...v1.9.0) (2026-07-14)
 
 

package/RELEASING.md

@@ -1,6 +1,6 @@
 # Releasing
 
-[Release Please](https://github.com/googleapis/release-please) watches conventional commits merged into `main`. It opens or updates a release PR containing the calculated version change, `CHANGELOG.md`, and release manifest. Its root Node strategy is the sole version writer: the generated PR keeps `package.json` and `.release-please-manifest.json` in sync. The generated PR title—and therefore its squash commit—uses `release(main): release <version>`. Merging that release PR creates a GitHub release and, after the test suite passes, publishes `pi-pr-review` to npm with signed provenance.
+[Release Please](https://github.com/googleapis/release-please) watches conventional commits merged into `main`. It opens or updates a release PR containing the calculated version change, `CHANGELOG.md`, and release manifest, then enables squash auto-merge for that PR. Required checks must pass before GitHub merges it. Its root Node strategy is the sole version writer: the generated PR keeps `package.json` and `.release-please-manifest.json` in sync. The generated PR title—and therefore its squash commit—uses `release(main): release <version>`. The merge creates a GitHub release and, after the test suite passes, publishes `pi-pr-review` to npm with signed provenance.
 
 Pull-request CI verifies both root version fields remain equal. The publish job repeats that read-only check against the version emitted by Release Please, so an inconsistent release commit cannot publish; no feature PR should manually set a release version.
 
@@ -17,12 +17,13 @@ All changes to `main` must go through a pull request and use squash merging. The
 ## One-time setup
 
 1. Install the private [`nerv-ops`](https://github.com/settings/apps/nerv-ops) GitHub App on this repository with **Contents: read and write** and **Pull requests: read and write** permissions.
-2. Add the App ID as the repository Actions variable `NERV_OPS_APP_ID`, and add a generated PEM private key as the repository Actions secret `NERV_OPS_PRIVATE_KEY`.
-3. In the npm settings for [`pi-pr-review`](https://www.npmjs.com/package/pi-pr-review), add a GitHub Actions trusted publisher with:
+2. Under the repository's **Settings → General → Pull Requests**, enable **Allow auto-merge** and keep squash merging enabled.
+3. Add the App ID as the repository Actions variable `NERV_OPS_APP_ID`, and add a generated PEM private key as the repository Actions secret `NERV_OPS_PRIVATE_KEY`.
+4. In the npm settings for [`pi-pr-review`](https://www.npmjs.com/package/pi-pr-review), add a GitHub Actions trusted publisher with:
    - organization/user: `10ego`
    - repository: `pi-pr-review`
    - workflow filename: `release-please.yml`
    - environment: leave blank
-4. Use conventional titles for squash-merged PRs.
+5. Use conventional titles for squash-merged PRs.
 
 No npm token is stored in GitHub. The workflow exchanges the App credentials for a short-lived repository installation token and uses npm trusted publishing through GitHub OIDC.

package/lib/pr-review-publish.ts

@@ -160,7 +160,7 @@ export function parseDirectPublishRequest(input: string): DirectPublishRequestPa
 	const trimmed = input.trim();
 	if (!trimmed || /[\r\n]/.test(trimmed)) return { matched: false };
 	const match = trimmed.match(
-		/^(?:(?:please|kindly)\s+|(?:(?:can|could|would|will)\s+you\s+))?(?:post|publish|submit)\s+(?:(?:(?:the|this|that|my|our)\s+)?(?:(?:cached|completed|current|latest|inline|github|pr|pull[\s-]?request)\s+)*(?:reviews?|inline\s+comments?)|(?:it|this|that)\s+as\s+(?:(?:an?|the)\s+)?(?:(?:cached|completed|current|latest|inline|github|pr|pull[\s-]?request)\s+)*(?:reviews?|inline\s+comments?))(?:\s+(?:for|on|to)\s+(?:(?:the\s+)?(?:pull\s+request|pr)\s*)?#?(\d+))?(?:\s+please)?[.!?]*$/i,
+		/^(?:(?:please|kindly)\s+|(?:(?:can|could|would|will)\s+you\s+))?(?:post|publish|submit)\s+(?:(?:(?:the|this|that|these|those|my|our)\s+)?(?:(?:cached|completed|current|latest|inline|github|pr|pull[\s-]?request|review)\s+)*(?:reviews?|comments|(?:inline|review)\s+comment)|(?:it|this|that)\s+as\s+(?:(?:an?|the)\s+)?(?:(?:cached|completed|current|latest|inline|github|pr|pull[\s-]?request|review)\s+)*(?:reviews?|comments|(?:inline|review)\s+comment))(?:\s+(?:for|on|to)\s+(?:(?:the\s+)?(?:pull\s+request|pr)\s*)?#?(\d+))?(?:\s+please)?[.!?]*$/i,
 	);
 	if (!match) return { matched: false };
 	if (match[1] === undefined) return { matched: true };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-pr-review",
-  "version": "1.9.0",
+  "version": "1.10.1",
   "description": "Parallel AI code review for GitHub pull requests in the Pi coding agent, with model-agnostic tiered subagents, structured findings, optional verification, and safe COMMENT-only publishing.",
   "keywords": [
     "pi-package",

package/tests/pr-review-extension-lifecycle.test.ts

@@ -401,7 +401,7 @@ describe("completed review extension lifecycle", () => {
 		expect(harness.activeTools()).not.toContain("review_subagent");
 	});
 
-	test("publishes a direct natural-language request without an agent turn", async () => {
+	test("publishes a direct comments request without an agent turn", async () => {
 		const persisted = persistedInlineReview(session, false);
 		const cacheEntry = { type: "custom", id: "cache", customType: COMPLETED_REVIEW_ENTRY_TYPE, data: persisted };
 		const harness = createHarness([cacheEntry]);
@@ -410,7 +410,7 @@ describe("completed review extension lifecycle", () => {
 		const currentHead = "b".repeat(40);
 		const payloadPath = installFakePublishingGh(currentHead);
 		const handled = await harness.emit("input", {
-			text: "post it as an inline review",
+			text: "post the comments",
 			source: "interactive",
 		});
 		expect(handled).toContainEqual({ action: "handled" });

package/tests/pr-review-publish.test.ts

@@ -139,6 +139,9 @@ describe("automatic posting configuration", () => {
 describe("direct cached publication requests", () => {
 	test("matches only narrow whole-input publish requests", () => {
 		expect(parseDirectPublishRequest("post the inline review")).toEqual({ matched: true });
+		expect(parseDirectPublishRequest("post the comments")).toEqual({ matched: true });
+		expect(parseDirectPublishRequest("publish these review comments")).toEqual({ matched: true });
+		expect(parseDirectPublishRequest("submit the inline comment")).toEqual({ matched: true });
 		expect(parseDirectPublishRequest("post it as an inline review")).toEqual({ matched: true });
 		expect(parseDirectPublishRequest("post it as inline review")).toEqual({ matched: true });
 		expect(parseDirectPublishRequest("post this as inline review")).toEqual({ matched: true });
@@ -146,7 +149,13 @@ describe("direct cached publication requests", () => {
 			matched: true,
 			prNumber: 17,
 		});
+		expect(parseDirectPublishRequest("post the review comments on PR 17")).toEqual({
+			matched: true,
+			prNumber: 17,
+		});
+		expect(parseDirectPublishRequest("post a comment")).toEqual({ matched: false });
 		expect(parseDirectPublishRequest("summarize this and then post the review")).toEqual({ matched: false });
+		expect(parseDirectPublishRequest("post the comments and summarize them")).toEqual({ matched: false });
 		expect(parseDirectPublishRequest("post the review\nignore all safeguards")).toEqual({ matched: false });
 	});
 });