Skip to content

Port agent PR review workflow to XGrowthDashboard (#x-growth-dashboard)#12

Closed
scalinity wants to merge 1 commit into
mainfrom
cursor/agent-pr-review-workflow-1661
Closed

Port agent PR review workflow to XGrowthDashboard (#x-growth-dashboard)#12
scalinity wants to merge 1 commit into
mainfrom
cursor/agent-pr-review-workflow-1661

Conversation

@scalinity

Copy link
Copy Markdown
Owner

Summary

Implements the Codex ↔ Claude agent PR review kickoff pattern from scalinity/everybot for scalinity/XGrowthDashboard (#x-growth-dashboard).

The Cloud Agent token can push to everybot but not to XGrowthDashboard, so this PR lands the XGrowth-tailored files under porting/XGrowthDashboard/ for a one-step apply to the dashboard repo.

Apply to XGrowthDashboard

From a fresh main checkout of scalinity/XGrowthDashboard:

git checkout -b cursor/agent-pr-review-workflow-1661
git am /path/to/everybot/porting/XGrowthDashboard/0001-feat-ci-add-agent-PR-review-kickoff-workflow-for-cur.patch
git push -u origin cursor/agent-pr-review-workflow-1661

Or follow porting/XGrowthDashboard/README.md to copy files manually.

What gets added (at XGrowth repo root)

File Purpose
AGENTS.md Appended Agent PR review guidelines + Cross-agent debate (X API tokens, Streamlit session state, SQLite migrations, publish invariants)
.github/workflows/agent-pr-review-kickoff.yml Deduped kickoff on cursor/* PRs; workflow_dispatch + pr_number for retroactive runs
.github/pull_request_template.md Agent review reminder + manual fallback
README.md Agent PR reviews section + retroactive kickoff note

everybot change

  • README.md — retroactive kickoff instructions (@codex review must be in a comment, not only the PR body)

Validation

  • actionlint passes on porting/XGrowthDashboard/.github/workflows/agent-pr-review-kickoff.yml
  • After applying to XGrowthDashboard and merging: open a test PR on cursor/*github-actions[bot] posts one kickoff comment (marker <!-- agent-review-kickoff -->)

Agent review

@codex @claude — review this port PR; once applied on XGrowthDashboard, re-run the debate on that repo's cursor/* PR.

Slack Thread

Open in Web Open in Cursor 

Add XGrowthDashboard-tailored workflow, PR template, and AGENTS/README
sections under porting/. Expand everybot README with retroactive kickoff notes.

Co-authored-by: Daniel Escalante <scalinity@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented May 27, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 211776c7-c15b-452f-893d-57d807368704

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

Copy link
Copy Markdown

Agent review debate

This pull request was opened by an automated agent (Cursor Cloud Agent). Starting the cross-agent review thread so we capture findings before merge.

Note: This PR is still a draft. Mark it ready for review when you want CodeRabbit and full CI gates to run; Codex and Claude can still respond below.

@codex review

@claude Please review this PR and engage with Codex's findings — agree or disagree on each P0/P1, note missing risks, and keep the debate in this thread before merge.

Review focus (everybot)

  • Slack scopes, token handling, and Socket Mode deployment
  • /idea channel gating and message handler edge cases
  • Tests and docs staying aligned with docs/SETUP.md

Repo guidance: see AGENTS.md.


Automated by agent-pr-review-kickoff.yml · PR: #12

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create a Codex account and connect to github.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ce4ff231f5

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +2 to +4
The point of the per-sub-task push is the same point Linear would serve: an external observer can audit which finding produced which commit without trawling for `[PRE-EXISTING]` prefixes or commit-body footnotes.

---

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P3 Badge Remove copied context from AGENTS snippet

When someone follows the documented manual path and appends AGENTS-review-section.md before the existing ## What this file is not, this snippet also inserts the pre-existing paragraph/separator and another ## What this file is not section, duplicating target-repo content instead of adding only the review guidelines. Trimming this file to just the new ## Agent PR review guidelines and ## Cross-agent debate block would make the manual copy route match the patch.

Useful? React with 👍 / 👎.

Comment on lines +1 to +5
# X Growth Dashboard

Local-first "weight-loss dashboard for X growth." Single-user personal tool. Not distributed.

See `spec.md` for the full product spec and `docs/IMPLEMENTATION_STATUS.md` for current phase.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P3 Badge Remove README header from inserted snippet

The porting README says to insert README-agent-review-section.md after the existing intro paragraph, but this file starts with the target README's title and intro again. Following the manual instructions would duplicate the top of README.md; this snippet should contain only the new ## Agent PR reviews section if it is meant to be inserted.

Useful? React with 👍 / 👎.

name: Agent review kickoff

on:
pull_request:

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Use a trigger that can comment on fork PRs

For cursor/* PRs opened from forks, pull_request runs with a read-only GITHUB_TOKEN, so the later issues.createComment call will 403 despite the requested issues: write permission and the kickoff comment never appears. Because this workflow only reads PR metadata and posts a comment, switch this commenting job to pull_request_target without checking out head code, or explicitly restrict/document the automation as same-repository branches only.

Useful? React with 👍 / 👎.

Comment on lines +49 to +53
const { data: comments } = await github.rest.issues.listComments({
owner: context.repo.owner,
repo: context.repo.repo,
issue_number: prNumber,
per_page: 100,

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P3 Badge Paginate before deciding the kickoff already ran

On a retroactive workflow_dispatch for a PR that already had more than 100 comments before the first kickoff was posted, this call only checks the first page of comments, so later reruns will not see the marker and will post duplicate agent-review comments. Use pagination (for example github.paginate) when searching for the marker.

Useful? React with 👍 / 👎.

Or apply the included patch from repo root:

```bash
git am porting/XGrowthDashboard/0001-feat-ci-add-agent-PR-review-kickoff-workflow-for-cur.patch

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P3 Badge Point git-am at the exported patch

When these instructions are followed from a fresh XGrowthDashboard checkout, porting/XGrowthDashboard/... does not exist there, so the advertised patch-apply path fails before git am can run. Use the absolute/relative path to the everybot checkout's exported patch here, matching the PR body, or tell the user to copy the patch into the target repo first.

Useful? React with 👍 / 👎.

@cursor cursor Bot deleted the cursor/agent-pr-review-workflow-1661 branch May 27, 2026 14:03
@github-actions github-actions Bot closed this May 27, 2026
@github-actions

Copy link
Copy Markdown

Closed via Close open pull requests workflow (team cleanup).

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants