Skip to content

chore: sync master with latest branch state#27

Merged
tomcasaburi merged 3 commits intomasterfrom
codex/master-sync-20260311
Mar 11, 2026
Merged

chore: sync master with latest branch state#27
tomcasaburi merged 3 commits intomasterfrom
codex/master-sync-20260311

Conversation

@tomcasaburi
Copy link
Member

@tomcasaburi tomcasaburi commented Mar 11, 2026

Updates with the current local master state that already includes the latest remote commits plus the remaining local branch work. This PR is only to satisfy protected-branch rules and cleanly land the repository back onto .


Note

Low Risk
Low-risk changes limited to repo AI workflow metadata and a test server script; main risk is minor CI/test breakage if the createComment option rename doesn’t match the installed @plebbit/plebbit-js version.

Overview
Adds a new review-and-merge-pr agent definition to both .codex/ and .cursor/ so the workflow is available across toolchains.

Updates the task-router guidance in AGENTS.md and adjusts the test server’s comment publishing to use communityAddress instead of subplebbitAddress when calling createComment.

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

Summary by CodeRabbit

  • New Features

    • Added Review and Merge PR agent configuration for automated workflow management.
  • Documentation

    • Updated workflow alignment guidance in agent documentation.
  • Chores

    • Renamed API parameter for improved clarity in community address references.

@coderabbitai
Copy link

coderabbitai bot commented Mar 11, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 882ee24b-a27c-4efc-8d5c-64572d4988ed

📥 Commits

Reviewing files that changed from the base of the PR and between 614b909 and 9fb12de.

📒 Files selected for processing (4)
  • .codex/skills/review-and-merge-pr/agents/openai.yaml
  • .cursor/skills/review-and-merge-pr/agents/openai.yaml
  • AGENTS.md
  • test/test-server/index.js

📝 Walkthrough

Walkthrough

This PR adds new agent configuration files for a "Review and Merge PR" agent in both Codex and Cursor directories, updates the AGENTS.md documentation table, and renames a property in a test file from subplebbitAddress to communityAddress.

Changes

Cohort / File(s) Summary
Agent Configuration Files
.codex/skills/review-and-merge-pr/agents/openai.yaml, .cursor/skills/review-and-merge-pr/agents/openai.yaml
New YAML configuration files defining a "Review and Merge PR" agent with display name, short description, and default prompt for PR review and merge workflows.
Documentation
AGENTS.md
Added a new row to the Task Router table describing keeping Codex/Cursor workflow copies aligned with condensed wording.
Test File
test/test-server/index.js
Renamed property in plebbit2.createComment call from subplebbitAddress to communityAddress.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Poem

🐰 Hop along with configs new and bright,
Agent files in Codex and Cursor aligned just right,
Properties renamed with a careful touch,
Documentation updated—oh, we love it so much!
Review and merge with agent care! ✨

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch codex/master-sync-20260311

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

@tomcasaburi tomcasaburi merged commit 13a079b into master Mar 11, 2026
1 of 7 checks passed
@tomcasaburi tomcasaburi deleted the codex/master-sync-20260311 branch March 11, 2026 12:11
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

title: "comment title",
content: "comment content",
subplebbitAddress: signer.address,
communityAddress: signer.address,
Copy link

Choose a reason for hiding this comment

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

Test server uses unsupported property name for plebbit-js

High Severity

The test server calls plebbit2.createComment() directly with communityAddress instead of the original subplebbitAddress. Unlike the application code in accounts-actions.ts, which wraps options through withLegacySubplebbitAddress() to ensure both property names are present, this call goes straight to plebbit-js without the compat layer. Since plebbit-js at the pinned commit hasn't been rebranded (evidenced by createSubplebbit still being used on line 45 and the compat layer's entire existence), it likely doesn't recognize communityAddress, causing test comment publishing to fail silently or error.

Fix in Cursor Fix in Web

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.

1 participant