Skip to content

🔧 config(coderabbit): adopt agent-voice tone and SYP best practices#15

Merged
scttbnsn merged 1 commit into
mainfrom
config/coderabbit-voice
Jul 5, 2026
Merged

🔧 config(coderabbit): adopt agent-voice tone and SYP best practices#15
scttbnsn merged 1 commit into
mainfrom
config/coderabbit-voice

Conversation

@scttbnsn

@scttbnsn scttbnsn commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Second pass pulling in the evolved config the SYP repos converged on:

  • tone_instructions: review comments written for an AI coding agent, terse, problem+fix, actionable only
  • high_level_summary_instructions: emoji changelog plus flat concern list, formatted for Claude Code to consume
  • Walkthrough noise stripped: collapsed walkthrough, no sequence diagrams, no effort estimate, no changed-files table, no fortune
  • auto_incremental_review on, drafts skipped, WIP/DO NOT MERGE titles skipped
  • Explicit tool enables for this stack (gitleaks, actionlint, markdownlint, yamllint, shellcheck, hadolint, ast-grep essential rules, github-checks) with languagetool off
  • pre_merge_checks: docstring/title/description gates off, linked-issue assessment as warning
  • Finishing touches (docstrings, unit tests) available on demand

✨ Added

  • Expanded .coderabbit.yaml with agent-focused review guidance and terse, action-oriented tone instructions.
  • Enabled additional workflow/engine controls for AI-agent usage and auto-review behavior.
  • Added tools support for stack-specific checks: gitleaks, actionlint, markdownlint, yamllint, shellcheck, hadolint, ast-grep essential rules, and github-checks.
  • Added finishing-touch controls for docstrings and unit_tests.
  • Added pre_merge_checks configuration with docstrings/title/description modes and linked-issue assessment as a warning.
  • Set chat.art: false.

🔧 Changed

  • Reworked the review summary format to use an emoji changelog plus a flat concern list for Claude Code.
  • Updated review-status toggles and auto-incremental review behavior.
  • Added ignore_title_keywords entries to skip draft/WIP/DO NOT MERGE-style reviews.
  • Disabled languagetool.
  • Relaxed pre-merge checks for docstrings, title, and description.

🗑️ Removed

  • Collapsed walkthrough content.
  • Sequence diagrams.
  • Effort estimates.
  • Changed-files tables.
  • Fortune text.
  • The previous shorter language/reviews config block.

⚠️ Breaking

  • None noted.

  • Verify the new skip-keyword list covers all title variants used by the team.

  • Confirm the enabled toolchain matches the repo’s CI/runtime environment.

  • Check that disabling languagetool is intentional for all review paths.

@vercel

vercel Bot commented Jul 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
rolester-website Ready Ready Preview, Comment Jul 5, 2026 11:49pm

@coderabbitai

coderabbitai Bot commented Jul 5, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: a8e1c1ca-c96f-4456-ab30-41ad381d6b3d

📥 Commits

Reviewing files that changed from the base of the PR and between ac83325 and 0a7f0bb.

📒 Files selected for processing (1)
  • .coderabbit.yaml

📝 Walkthrough

Compact metadata: Config-only PR modifying `.coderabbit.yaml` (+84/-5). No code, no exported entities affected.

Walkthrough

This PR updates `.coderabbit.yaml`. It replaces the prior compact reviews block with an expanded configuration adding `tone_instructions`, workflow controls (`enable_prompt_for_ai_agents`, `abort_on_close`), incremental auto-review behavior, and an expanded `ignore_title_keywords` list. It adds `finishing_touches` (docstrings, unit_tests), `pre_merge_checks` (modes/off settings, warning level), and a `tools` block enabling linters/security checks (github-checks timeout, ast-grep essential_rules, languagetool disabled). It also sets `chat.art: false`.

Changes

Area Change
Reviews config Added tone_instructions, workflow toggles, ignore_title_keywords
Finishing touches Enabled docstrings, unit_tests generation
Pre-merge checks Added modes/off settings for docstrings, title, description, issue assessment warning
Tools Enabled linters/security tools with specific options
Chat Set art to false

No code paths, contracts, or exported entities touched — nothing to flag on drift or DRY here since this is pure YAML config.

Sequence Diagram(s)

Not applicable — configuration-only change with no runtime flow.

Estimated code review effort: Low-Medium (config diff, verify key names/values against CodeRabbit schema).

Related issues: None referenced.

Related PRs: None referenced.

Suggested labels: `config`, `documentation`

Suggested reviewers: Repo maintainer familiar with CodeRabbit config schema.

Poem

A rabbit hopped into the YAML den,
Turned knobs for tone, then turned again,
Tools now guard the pull request gate,
Chat art off — no ASCII to await,
Config done, no code to debate.
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch config/coderabbit-voice

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

@biggest-littlest biggest-littlest left a comment

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.

Config-only change, checked against the CodeRabbit v2 schema.

@ALARGECOMPANY ALARGECOMPANY left a comment

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.

Config-only change, checked against the CodeRabbit v2 schema.

@scttbnsn scttbnsn merged commit cb9877f into main Jul 5, 2026
3 of 6 checks passed
@scttbnsn scttbnsn deleted the config/coderabbit-voice branch July 5, 2026 23:47
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.

3 participants