jonah-fleet 1.35.0 → 1.35.1

package/CHANGELOG.md

@@ -5,6 +5,19 @@ All notable changes to `jonah-fleet` will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.35.1](https://github.com/Jonah-Projects/jonah-fleet/compare/v1.35.0...v1.35.1) (2026-10-10)
+
+
+### Bug Fixes
+
+* **workflows:** omit issue_comment triggers and document quota guard ([#672](https://github.com/Jonah-Projects/jonah-fleet/issues/672)) ([2c27bd1](https://github.com/Jonah-Projects/jonah-fleet/commit/2c27bd19e44508e39a6e0bb68951c5fd32beec96))
+
+## [Unreleased] - 2026-10-09
+
+### Changed
+
+* **workflows:** omit unfiltered `issue_comment` triggers from `trigger-review-routine.yml` to conserve Actions runner quota and prevent comment-loop thrash
+
 ## [1.35.0](https://github.com/Jonah-Projects/jonah-fleet/compare/v1.34.0...v1.35.0) (2026-10-09)
 
 
@@ -45,7 +58,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Features
 
 * **daemon,telemetry:** halt daemon execution when plan quota drops below 20% in either 5h or 7-day windows, with `--full-burn` CLI flag, environment variable, manifest configuration, and interactive hotkey override
-
 ## [Unreleased] - 2026-10-03
 
 ### Features

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jonah-fleet",
-  "version": "1.35.0",
+  "version": "1.35.1",
   "description": "Standalone autonomous agent fleet with Symphony orchestration, claim protocols, and continuous improvement loops",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/templates/prompts/ORCHESTRATION.md

@@ -18,7 +18,7 @@ Dispatch is both **scheduled** and **event-driven**. All routines run as ephemer
 
 - **Scheduled cron sweeps**: Autowork runs periodically (`autowork-cron.yml`), complemented by prompt optimization (`prompt-optimizer-cron.yml`), issues housekeeping (`issues-housekeeping-cron.yml`), dependency security checks (`dependency-check-cron.yml`), analytics review (`analytics-review-cron.yml`), product planning (`product-planning-cron.yml`), and design review (`design-review-cron.yml`).
 - **Event-driven & manual triggers**: GitHub Actions workflows fire routines on events and interactive commands so work starts within seconds instead of waiting for scheduled ticks:
-  - `trigger-review-routine.yml` fires Peer Review automatically when a PR is marked ready for review, updated, or review is requested (`ready_for_review`, `opened`, `reopened`, `synchronize`, `review_requested`). It can also be manually (re)triggered via `workflow_dispatch` (with optional `pr_number` for Targeted mode or blank for Scan mode) or by commenting `/review`, `/peer-review`, `/retrigger`, or `/re-review` on any open pull request.
+  - `trigger-review-routine.yml` fires Peer Review automatically when a PR is marked ready for review, updated, or review is requested (`ready_for_review`, `opened`, `reopened`, `synchronize`, `review_requested`). It can also be manually (re)triggered via `workflow_dispatch` (with optional `pr_number` for Targeted mode or blank for Scan mode). Note: Comment-based triggers are omitted to conserve Actions runner quota and prevent comment-loop thrash.
   - `trigger-autowork-on-merge.yml` fires Autowork in **Targeted mode** when a PR merges to `main` and unblocks the next unit of chained work.
   - `trigger-autowork-on-bug.yml` fires Autowork when an issue becomes a high-priority bug.
   - `trigger-autowork-on-assign.yml` fires Autowork in **Targeted mode** when an issue is assigned to a designated bot account or persona (`vars.AGENT_BOT_LOGIN`).
@@ -229,8 +229,8 @@ How the fleet guarantees continuous review throughput, recovers from transient A
 
 1. **Failure Trapping & Transparency**: When a peer-review workflow session encounters an execution failure (e.g. LLM API rate limit / quota exhaustion or runner timeout), `trigger-review-routine.yml` traps the failure and posts an informational status notice on the PR. Failures are made immediately visible on the PR timeline rather than silently failing in the background.
 2. **Periodic Scan Sweep (Watchdog)**: `trigger-review-routine.yml` runs a scheduled 2-hour cron sweep (`cron: '45 */2 * * *'`) in Scan mode. Any open PR in `ready_for_review` state that was orphaned due to a transient API rate limit or missed event trigger is automatically picked up, evaluated, and resolved.
-3. **Interactive Re-triggering**: Any team member or author can immediately re-dispatch review by commenting `/review`, `/peer-review`, `/retrigger`, or `/re-review` on any open pull request, or manually triggering `trigger-review-routine.yml` via `workflow_dispatch`.
-4. **Autowork Phase 1 Watchdog**: During Phase 1 convergence, Autowork actively identifies open ready PRs that have received no review feedback for $>2$ hours, re-triggering review via draft toggle or `/review` comment before picking up new work. Review re-triggering is strictly conditioned on all CI checks having passed (`conclusion: SUCCESS`, `mergeStateStatus: CLEAN`) and is strictly prohibited if checks are in-progress or awaiting approval (`ACTION_REQUIRED`).
+3. **Interactive Re-triggering**: Any team member or author can immediately re-dispatch review by manually triggering `trigger-review-routine.yml` via `workflow_dispatch` (with optional `pr_number` for Targeted mode or blank for Scan mode) or by re-requesting review on GitHub. Unfiltered comment triggers are omitted to guard runner quota.
+4. **Autowork Phase 1 Watchdog**: During Phase 1 convergence, Autowork actively identifies open ready PRs that have received no review feedback for $>2$ hours, re-triggering review via draft toggle (`gh pr ready <PR> --undo && gh pr ready <PR>`) before picking up new work. Review re-triggering is strictly conditioned on all CI checks having passed (`conclusion: SUCCESS`, `mergeStateStatus: CLEAN`) and is strictly prohibited if checks are in-progress or awaiting approval (`ACTION_REQUIRED`).
 
 ---
 

package/templates/prompts/autowork.md

@@ -141,9 +141,9 @@ _Generated by [Antigravity](${GITHUB_SERVER_URL}/${GITHUB_REPOSITORY}/actions/ru
          - *Option B*: [Pros / Cons]
          - **Recommended Path**: [Agent recommendation]
        ```
-     3c. **Orphaned Ready PR Recovery**: If an open PR authored by this routine is `ready_for_review`, has passing CI, no unaddressed review comments, and has received no review activity for over 2 hours (e.g. because peer review crashed or encountered quota limits), kickstart the review routine by posting `/review` comment or toggling draft and ready (`gh pr ready <PR> --undo && gh pr ready <PR>`).
+     3c. **Orphaned Ready PR Recovery**: If an open PR authored by this routine is `ready_for_review`, has passing CI, no unaddressed review comments, and has received no review activity for over 2 hours (e.g. because peer review crashed or encountered quota limits), kickstart the review routine by toggling draft and ready (`gh pr ready <PR> --undo && gh pr ready <PR>`).
    - **Passing CI Verification Gate**: Verify via `gh pr view <PR> --json statusCheckRollup,mergeStateStatus` that all required and existing checks have completed with `conclusion: "SUCCESS"` and `mergeStateStatus` is `CLEAN` (neither `UNSTABLE`, `BLOCKED`, nor `DIRTY`).
-   - **Unapproved/Pending Workflow Invariant**: NEVER post `/review` or toggle draft state if checks are in-progress, failing, or awaiting approval (`conclusion: "ACTION_REQUIRED"`). Doing so creates an infinite comment storm while workflows remain paused awaiting human permissions.
+   - **Unapproved/Pending Workflow Invariant**: NEVER toggle draft state if checks are in-progress, failing, or awaiting approval (`conclusion: "ACTION_REQUIRED"`). Doing so creates redundant review triggers while workflows remain paused awaiting human permissions.
 4. **Check open issues with merged PRs / work done — close them**:
    - For open issues, check if recent git history or merged PRs reference the issue (e.g. `git log -n 50 --grep="#<ISSUE_NUMBER>"` or `gh pr list --state merged --search "<ISSUE_NUMBER>" --limit 5`).
    - If a merged PR or commit on `main` resolved the issue, close it immediately:

package/templates/workflows/trigger-review-routine.yml

@@ -12,8 +12,6 @@ on:
         description: 'Optional PR number to review in Targeted mode (leave blank for Scan mode)'
         required: false
         type: string
-  issue_comment:
-    types: [created]
 
 jobs:
   trigger-routine:
@@ -23,21 +21,9 @@ jobs:
        github.event.pull_request.head.repo.full_name == github.repository &&
        !startsWith(github.event.pull_request.head.ref, 'release-please--')) ||
       (github.event_name == 'schedule') ||
-      (github.event_name == 'workflow_dispatch') ||
-      (github.event_name == 'issue_comment' &&
-       github.event.issue.pull_request != null &&
-       !endsWith(github.event.comment.user.login, '[bot]') &&
-       github.event.comment.user.type != 'Bot' &&
-       (startsWith(github.event.comment.body, '/review') ||
-        startsWith(github.event.comment.body, '/peer-review') ||
-        startsWith(github.event.comment.body, '/retrigger') ||
-        startsWith(github.event.comment.body, '/re-review') ||
-        contains(github.event.comment.body, '/review') ||
-        contains(github.event.comment.body, '/peer-review') ||
-        contains(github.event.comment.body, '/retrigger') ||
-        contains(github.event.comment.body, '/re-review')))
+      (github.event_name == 'workflow_dispatch')
     concurrency:
-      group: review-routine-pr-${{ github.event.pull_request.number || github.event.issue.number || inputs.pr_number || 'scan' }}
+      group: review-routine-pr-${{ github.event.pull_request.number || inputs.pr_number || 'scan' }}
       cancel-in-progress: true
     runs-on: ubuntu-latest
     timeout-minutes: 65
@@ -52,7 +38,7 @@ jobs:
         id: guard
         env:
           GH_TOKEN: ${{ github.token }}
-          PR_NUM: ${{ github.event.pull_request.number || github.event.issue.number || inputs.pr_number }}
+          PR_NUM: ${{ github.event.pull_request.number || inputs.pr_number }}
           EVENT_NAME: ${{ github.event_name }}
           REPOSITORY: ${{ github.repository }}
         run: |
@@ -232,7 +218,7 @@ jobs:
         id: run-issue
         env:
           GH_TOKEN: ${{ secrets.GH_PAT || github.token }}
-          PR_ARG: ${{ github.event.pull_request.number || github.event.issue.number || inputs.pr_number }}
+          PR_ARG: ${{ github.event.pull_request.number || inputs.pr_number }}
           EVENT_NAME: ${{ github.event_name }}
           RUN_ID: ${{ github.run_id }}
           SERVER_URL: ${{ github.server_url }}
@@ -275,8 +261,8 @@ jobs:
           ANTIGRAVITY_API_KEY: ${{ secrets.GEMINI_API_KEY }}
           GH_TOKEN: ${{ secrets.GH_PAT || github.token }}
           ROUTINE: peer-review
-          PR_NUMBER: ${{ github.event.pull_request.number || github.event.issue.number || inputs.pr_number }}
-          PR_URL: ${{ github.event.pull_request.html_url || github.event.issue.html_url }}
+          PR_NUMBER: ${{ github.event.pull_request.number || inputs.pr_number }}
+          PR_URL: ${{ github.event.pull_request.html_url }}
           ROUTINE_ISSUE_NUMBER: ${{ steps.run-issue.outputs.issue_number }}
           MODEL: ${{ steps.routine-config.outputs.model || 'gemini-3.8-flash-high' }}
           PRINT_TIMEOUT: ${{ steps.routine-config.outputs.print_timeout || '55m' }}
@@ -322,7 +308,7 @@ jobs:
         if: failure() && steps.guard.outputs.skip != 'true'
         env:
           GH_TOKEN: ${{ secrets.GH_PAT || github.token }}
-          PR_NUMBER: ${{ github.event.pull_request.number || github.event.issue.number || inputs.pr_number }}
+          PR_NUMBER: ${{ github.event.pull_request.number || inputs.pr_number }}
           ROUTINE_ISSUE_NUMBER: ${{ steps.run-issue.outputs.issue_number }}
           GITHUB_SERVER_URL: ${{ github.server_url }}
           GITHUB_REPOSITORY: ${{ github.repository }}
@@ -333,7 +319,7 @@ jobs:
             if [ -n "$ROUTINE_ISSUE_NUMBER" ]; then
               LOG_REF=" · [Run Log #${ROUTINE_ISSUE_NUMBER}](${GITHUB_SERVER_URL}/${GITHUB_REPOSITORY}/issues/${ROUTINE_ISSUE_NUMBER})"
             fi
-            gh pr comment "$PR_NUMBER" --body "⚠️ **Peer Review Routine Notice**: The automated review session encountered an execution error (e.g., API rate/quota limit or runner interruption)${LOG_REF}. The review will be retried automatically on the next scheduled scan sweep or can be re-triggered immediately by commenting \`/review\` on this PR.
+            gh pr comment "$PR_NUMBER" --body "⚠️ **Peer Review Routine Notice**: The automated review session encountered an execution error (e.g., API rate/quota limit or runner interruption)${LOG_REF}. The review will be retried automatically on the next scheduled scan sweep or can be re-triggered manually via workflow dispatch on this PR.
 
           _Generated by [Antigravity](${GITHUB_SERVER_URL}/${GITHUB_REPOSITORY}/actions/runs/${GITHUB_RUN_ID})_" || true
           fi
@@ -376,7 +362,7 @@ jobs:
           RUN_ID: ${{ github.run_id }}
           SERVER_URL: ${{ github.server_url }}
           REPOSITORY: ${{ github.repository }}
-          PR_NUMBER: ${{ github.event.pull_request.number || github.event.issue.number || inputs.pr_number }}
+          PR_NUMBER: ${{ github.event.pull_request.number || inputs.pr_number }}
           QUOTA_PAUSED: ${{ env.QUOTA_PAUSED }}
           QUOTA_RESET_INFO: ${{ env.QUOTA_RESET_INFO }}
         run: |