Skip to content

chore: Add a cursorbot workflow#24799

Merged
dan437 merged 1 commit intomainfrom
cursorbot
Jan 16, 2026
Merged

chore: Add a cursorbot workflow#24799
dan437 merged 1 commit intomainfrom
cursorbot

Conversation

@dan437
Copy link
Copy Markdown
Contributor

@dan437 dan437 commented Jan 16, 2026

Description

When a ticket is assigned to metamaskbot (cursorbot in the future), it will open a PR.

Changelog

CHANGELOG entry: null

Related issues

Fixes:

Manual testing steps

  1. Assign a ticket to metamaskbot
  2. See that a PR was opened after some time

Screenshots/Recordings

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

Note

Introduces automated Cursor-driven issue implementation and consolidates Cursor setup as a reusable action.

  • New composite action /.github/actions/cursor-cli-setup to configure Cursor permissions (read-only/read-write), install CLI, and output base64-encoded issue data
  • New workflow /.github/workflows/cursorbot.yml triggers on issues: assigned to metamaskbot; creates branch, runs implementation prompt via cursor-agent, commits/pushes changes, and opens a draft PR (or comments if no changes)
  • Refactor /.github/workflows/cursor-issue-analysis.yml (v0.4.0): replaces inline setup with the new action and wires its issue-content output into analysis
  • New prompt /.github/cursor/prompts/implement-issue.md defining implementation guidelines and required output

Scope/risk: adds automation with repo write permissions and scripted commits/PR creation; Cursor CLI permissions restrict sensitive file access.

Written by Cursor Bugbot for commit 43dadf4. This will update automatically on new commits. Configure here.

Signed-off-by: dan437 <80175477+dan437@users.noreply.github.com>
@dan437 dan437 requested a review from a team as a code owner January 16, 2026 14:26
@github-actions
Copy link
Copy Markdown
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbot metamaskbot added the team-confirmations Push issues to confirmations team label Jan 16, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🔍 Smart E2E Test Selection

  • Selected E2E tags: None (no tests recommended)
  • Risk Level: low
  • AI Confidence: 98%
click to see 🤖 AI reasoning details

All 4 changed files are GitHub Actions/Workflows configuration files located in the .github/ directory:

  1. .github/actions/cursor-cli-setup/action.yml - A reusable GitHub Action for setting up Cursor CLI
  2. .github/cursor/prompts/implement-issue.md - A prompt template for Cursor AI
  3. .github/workflows/cursor-issue-analysis.yml - A workflow for AI-powered issue analysis
  4. .github/workflows/cursorbot.yml - A workflow for automated issue implementation

These changes are purely CI/CD automation infrastructure for integrating Cursor AI into the development workflow. They:

  • Do NOT modify any application code (no changes to app/, src/, or any TypeScript/JavaScript source files)
  • Do NOT modify any E2E test code or infrastructure (no changes to e2e/ directory)
  • Do NOT affect any core modules, controllers, Engine, or critical paths
  • Do NOT impact any user-facing features or wallet functionality
  • Are completely isolated to GitHub automation

Since these changes have zero impact on the MetaMask Mobile application or its E2E test infrastructure, no E2E tests need to be run.

View GitHub Actions results

2. Key implementation decisions
3. Any potential risks or considerations for reviewers

Do NOT create the pull request yourself - the workflow will handle that.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

That uppercase NOT made me laugh, like it's not gonna obey your command 😄

@dan437 dan437 enabled auto-merge January 16, 2026 14:42
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@NicolasMassart NicolasMassart left a comment

Choose a reason for hiding this comment

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

Nice workflow.
Not critical but there's a lot of places where commands could fail silently and let the workflow continue just to fail later.
You could stop it early by adding error checks for gh issue view, cursor-agent execution, and git push as well as error handling for base64 encoding operations, branch creation or install Cursor CLI.
The current implem works, but if anything fails, it will not really be easy to debug

@dan437 dan437 added this pull request to the merge queue Jan 16, 2026
Merged via the queue into main with commit 6a2dd30 Jan 16, 2026
78 checks passed
@dan437 dan437 deleted the cursorbot branch January 16, 2026 15:58
@github-actions github-actions bot locked and limited conversation to collaborators Jan 16, 2026
@metamaskbot metamaskbot added the release-7.63.0 Issue or pull request that will be included in release 7.63.0 label Jan 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-7.63.0 Issue or pull request that will be included in release 7.63.0 size-M team-confirmations Push issues to confirmations team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants