Skip to content

[chore]: add code review rules and link them from AGENTS.md - #1481

Merged
sf-tyler-jeong merged 2 commits into
mainfrom
chore/review-rules
Nov 10, 2026
Merged

sf-tyler-jeong merged 2 commits into
mainfrom
chore/review-rules

Conversation

@sf-tyler-jeong

@sf-tyler-jeong sf-tyler-jeong commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Adds REVIEW.md at the repository root with the rules that automated code review should apply to pull requests in this repository, and a ## Code Review Rules section in AGENTS.md that points to it.

Changes

  • REVIEW.md (new):
    • A two-step procedure: first map what the pull request touches to the checks that apply and bind each check to a file or function, then verify each pair against the actual code.
    • A general checklist: correctness and edge cases, behavior changes against the previous code, public surface and compatibility, concurrency and lifecycle in event-loop terms (ordering, callbacks after unmount, late async responses), design, test quality, and claims made in the pull request description.
    • JS / TypeScript checks: configuration wiring, React state/context/hooks, async event pipelines, user interaction, accessibility, public surface (context values, hook returns, provider props, package exports), contract changes behind an unchanged signature (return values, callback arguments, call timing and order, error propagation), and bundling.
    • Reporting rules: scope findings by cause — a defect the diff causes is in scope even when it shows up in a file the diff does not touch, and the comment goes on the line that causes it — state confidence instead of filtering, order findings by impact, and give file:line with a concrete fix direction.
  • AGENTS.md: adds the ## Code Review Rules section below the repository-specific marker. The rest of the file is unchanged.

The rules live in their own file rather than inline in AGENTS.md, so the general agent instructions there stay short.

Backward compatibility

No public API changes — documentation only. Neither file is part of the published package (files in package.json covers release/, dist/, LICENSE, and CHANGELOG.md).

Testing

  • Checked that the root AGENTS.md is the only AGENTS.md in the repository, so no nested file takes precedence over the new section.

Changelogs

  • Added code review rules for automated pull request review (internal)

Add REVIEW.md with the rules automated code review should apply to pull
requests in this repository, and point a Code Review Rules section in
AGENTS.md at it.
@netlify

netlify Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for sendbird-uikit-react ready!

Name Link
🔨 Latest commit 1ec6e50
🔍 Latest deploy log https://app.netlify.com/projects/sendbird-uikit-react/deploys/6af2691d57e9d6ba976976d1
😎 Deploy Preview https://deploy-preview-1481--sendbird-uikit-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@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: 7b818a22fe

ℹ️ 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 thread REVIEW.md Resolved
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-04T07:48:01.646959Z 7b818a2 Manual request
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@sf-tyler-jeong

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Hooray!

Reviewed commit: 7b818a22fe

ℹ️ 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 thread REVIEW.md Resolved
Comment thread REVIEW.md Resolved
Comment thread REVIEW.md Resolved
- Reporting: a defect the diff causes is in scope even when it shows up
  in a file the diff does not touch; comment on the causing diff line.
- JS / TypeScript: compare return values, callback arguments, call
  timing and order, and error propagation against the base revision
  even when the signature is unchanged.
- Concurrency and lifecycle: describe ordering in event-loop terms and
  add callbacks after unmount and late async responses.

@danney-chun danney-chun left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm~

@sf-tyler-jeong
sf-tyler-jeong merged commit d9e2ca6 into main Nov 10, 2026
10 checks passed
@sf-tyler-jeong
sf-tyler-jeong deleted the chore/review-rules branch November 10, 2026 06:12
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