Skip to content

[#805] Tighten trigger message — RE1/RE2 review only @mentioned PRs #808

Description

@realproject7

Part of #805 (step 3).

Two separable changes — Part A is independent and can land first; Part B depends on #807.

Part A — tighten reviewer scope (INDEPENDENT, no dependency)

The trigger prompt tells RE1/RE2 to "Review open PRs," which drives blanket gh pr list discovery every cycle. In DEFAULT_MESSAGE / per-project trigger_message (server/index.js), change RE1/RE2 to: "review only PRs you were @mentioned on." This cuts reviewer discovery immediately, with no dependency on GITHUB.md.

Part B — point discovery at GITHUB.md (DEPENDS ON #807)

Change all roles to discover issue/PR state from GITHUB.md / GET /api/github-parsed instead of gh pr list. This must not ship before #807 (the file/endpoint must exist). If shipped during rollout, include fallback wording: "if GITHUB.md is absent, fall back to a single gh discovery call."

Risk: Low.

Acceptance criteria


Workflow-risk guardrails (from #805 risk analysis)

  • Reword the trigger message to point at GITHUB.md / /api/github-parsed, no-op quietly on empty, and carry a staleness caveat ("GITHUB.md may be stale — confirm with a direct gh read before merge/review").
  • Verify the per-cycle trigger enumerates the PR number + author so "review only @mentioned PRs" has a concrete target to bind to.
  • Extend the Part B fallback from "absent file" to "absent OR stale > 2 cycles" → fall back to one direct gh read.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    agent/devAssigned to Dev agentenhancementNew feature or request

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions