[PR #2] [MERGED] chore: add CodeRabbit configuration #11

New issue

Closed

opened 2026-10-31 22:03:50 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-31 22:03:50 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Comfy-Org/comfy-typescript-sdk/pull/2
Author: @wei-hai
Created: 7/15/2026
Status: ✅ Merged
Merged: 7/15/2026
Merged by: @wei-hai

Base: main ← Head: chore/coderabbit

📝 Commits (1)

fcba56a chore: add CodeRabbit configuration

📊 Changes

1 file changed (+56 additions, -0 deletions)

View changed files

➕ .coderabbit.yaml (+56 -0)

📄 Description

Summary

Adds a top-level .coderabbit.yaml so this repo gets automated PR review, matching the convention already used by the org's other repos (ComfyUI, ComfyUI_frontend, cloud).

English review comments, "assertive" profile, request-changes workflow on, poem/fortune off.
Auto-review enabled on non-draft PRs; PRs titled WIP / DO NOT MERGE are skipped.
No CodeRabbit built-in linter is disabled here (CodeRabbit has no built-in oxlint/oxfmt tool to conflict with); review instructions point it at public-API stability, type safety, and async/error handling since this repo's own CI already runs oxlint, oxfmt --check, tsc, and vitest.
Path-specific guidance for src/** (this is a published SDK, so flag breaking changes to exported types/functions, no any) and src/**/*.test.ts (avoid change-detector tests).
Secret scanning (gitleaks) and GitHub-checks integration turned on.

Test plan

Validated the YAML parses (python3 -c "import yaml; yaml.safe_load(...)")
Confirmed no internal-only references (Linear/Notion/Slack/Datadog/internal URLs) in the file
Confirm CodeRabbit picks up the config and reviews the next PR opened against this branch/repo

🤖 Generated with Claude Code

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Comfy-Org/comfy-typescript-sdk/pull/2 **Author:** [@wei-hai](https://github.com/wei-hai) **Created:** 7/15/2026 **Status:** ✅ Merged **Merged:** 7/15/2026 **Merged by:** [@wei-hai](https://github.com/wei-hai) **Base:** `main` ← **Head:** `chore/coderabbit` --- ### 📝 Commits (1) - [`fcba56a`](https://github.com/Comfy-Org/comfy-typescript-sdk/commit/fcba56af210cfbc6ac3b52e940c53b0d5bf9399d) chore: add CodeRabbit configuration ### 📊 Changes **1 file changed** (+56 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `.coderabbit.yaml` (+56 -0) </details> ### 📄 Description ## Summary Adds a top-level `.coderabbit.yaml` so this repo gets automated PR review, matching the convention already used by the org's other repos (ComfyUI, ComfyUI_frontend, cloud). - English review comments, "assertive" profile, request-changes workflow on, poem/fortune off. - Auto-review enabled on non-draft PRs; PRs titled `WIP` / `DO NOT MERGE` are skipped. - No CodeRabbit built-in linter is disabled here (CodeRabbit has no built-in oxlint/oxfmt tool to conflict with); review instructions point it at public-API stability, type safety, and async/error handling since this repo's own CI already runs `oxlint`, `oxfmt --check`, `tsc`, and `vitest`. - Path-specific guidance for `src/**` (this is a published SDK, so flag breaking changes to exported types/functions, no `any`) and `src/**/*.test.ts` (avoid change-detector tests). - Secret scanning (`gitleaks`) and GitHub-checks integration turned on. ## Test plan - [x] Validated the YAML parses (`python3 -c "import yaml; yaml.safe_load(...)"`) - [x] Confirmed no internal-only references (Linear/Notion/Slack/Datadog/internal URLs) in the file - [ ] Confirm CodeRabbit picks up the config and reviews the next PR opened against this branch/repo 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-31 22:03:50 +00:00