Skip to content

chore(ci): [NT-3770] set up Bito agentic code review#388

Open
Elton Muteswa (elton-muteswa-cf) wants to merge 1 commit into
mainfrom
chore/NT-3770-bito-agentic-code-review
Open

chore(ci): [NT-3770] set up Bito agentic code review#388
Elton Muteswa (elton-muteswa-cf) wants to merge 1 commit into
mainfrom
chore/NT-3770-bito-agentic-code-review

Conversation

@elton-muteswa-cf

Copy link
Copy Markdown

Summary

  • Adds .bito.yaml to configure the Bito GitHub App: suggestion_mode: essential, no PR description injection (post_description/post_changelist: false), linter-feedback suppressed (linters_feedback: false), draft PRs excluded, and generated/lock files excluded from review
  • Adds .bito/guidelines/review-posture.txt: instructs Bito to act as a focused tech lead, post only when there is a concrete finding, and never include boilerplate sections (Usage Guide, Review Details)
  • Adds .bito/guidelines/repo-truth-and-boundaries.txt: points Bito at authoritative repo files (AGENTS.md, CLAUDE.md, STYLE_GUIDE.md) and defines blast-radius tiers for high/medium/low-impact changes
  • Adds .bito/guidelines/domain-invariants.txt: SDK-suite-specific review checklist covering public API contracts, test coverage, bundle-size/import discipline, type safety at SDK boundaries, platform-specific invariants, and reference implementation integrity

The approach mirrors how ninetailed-inc/ninetailed-new uses Bito: GitHub App-driven (no workflow step), .bito.yaml for config, plain-text guideline files under .bito/guidelines/ for domain context.

Pre-merge prerequisite

The Bito GitHub App must be installed on this repository before any review will fire. This is an out-of-repo step — install via the GitHub Marketplace or Bito admin console.

Post-merge verification plan

  • Open a substantive PR (new behavior, logic change) and confirm Bito posts a review comment with only actionable findings — no description injection, no boilerplate sections
  • Open a mechanical PR (dependency bump, config tweak) and confirm Bito posts nothing or only real findings
  • Open a draft PR and confirm Bito skips it (exclude_draft_pr: true)
  • Confirm no "Bito Usage Guide" or "Review Details" sections appear in any review comment

Checklist

  • Config-only change; no source, package, test, or workflow files modified
  • .bito.yaml passes Prettier (pnpm format:check)
  • No lint, typecheck, build, or bundle-size checks required (no TypeScript, no package output)
  • Skipped checks: none applicable

🤖 Generated with Claude Code

Adds .bito.yaml and three custom guideline files to configure the Bito
GitHub App to review PRs for substantive concerns the existing pipeline
cannot catch (correctness, test coverage gaps, unsafe patterns, design
fragility) while staying silent on anything ESLint/TypeScript/Prettier
already enforces.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Crossing my fingers 🤞 Maybe we don't actually enable it in the repo until the ISR + Edge support lands 😅

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