Improve coderabbit config

[ykmr1224]

Description

• Improve CodeRabbit config based on recent PR
• Removed General review instructions since it was not valid config in coderabbit.yml

Related Issues

n/a

Check List

• New functionality includes testing.
• New functionality has been documented.
• New functionality has javadoc added.
• New functionality has a user manual doc added.
• New PPL command checklist all confirmed.
• API changes companion pull request created.
• Commits are signed per the DCO using --signoff or -s.
• Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[coderabbitai]

📝 Walkthrough

Summary by CodeRabbit

• Configuration Updates

  • Added path_filters and drafts flag to control which PRs/files are auto-reviewed and to skip draft PRs
  • New chat.art toggle to disable ASCII art

• Knowledge Base

  • Enabled web search and localized knowledge scopes with per-project guideline links

• Documentation

  • Switched docs to Markdown and relaxed language-identifier checks

• Tests

  • Expanded testing requirements and test-data guidance (NDJSON, integration tests); avoid flagging inline/shared test data

• Tone

  • Centralized REVIEW_GUIDELINES referenced; added review tone and concise-feedback rules

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Walkthrough

Replaces and centralizes review guidance: updates .coderabbit.yaml to add path_filters and drafts, remove global instructions, reference .rules/REVIEW_GUIDELINES.md for file-type rules, change docs pattern to docs/**/*.md, add chat/art toggle and a knowledge_base configuration, and expands/rewrites REVIEW_GUIDELINES.md.

Changes

Cohort / File(s)	Summary
CodeRabbit config \.coderabbit\.yaml	Adds path_filters under reviews, drafts under auto_review; removes global instructions; switches many per-path blocks to reference centralized guidelines; changes docs path to docs/**/*.md; adds chat.art flag and knowledge_base config (web_search, local scopes, code_guidelines file pattern).
Centralized guidelines \.rules/REVIEW_GUIDELINES.md	Rewrites and expands general and Java standards, testing requirements, parser/grammar guidance, review tone/focus, and code quality principles; adds explicit test expectations, resource management and maintainability guidance.
Paths & test/data handling \.coderabbit\.yaml (paths)	Replaces many bespoke file-specific checks with references to REVIEW_GUIDELINES.md; broadens allowed doc/test-data formats (NDJSON, .md docs) and clarifies rules about inline test data and test helpers.

Sequence Diagram(s)

(omitted)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• Update CodeRabbit instructions #4962 — Modifies .coderabbit.yaml review instruction structure and path-specific guidance (strong overlap).
• Adjust CodeRabbit review config #4901 — Updates .coderabbit.yaml including chat/art settings and path-specific instructions (overlapping config changes).

Suggested reviewers

• anirudha
• LantaoJin
• penghuo
• ps48
• yuancu
• kavithacm
• derek-ho
• joshuali925
• GumpacG
• Swiddis
• forestmvey
• noCharger
• YANG-DB
• mengweieric
• Yury-Fridlyand
• MaxKsyunz
• vamsimanohar
• acarbonetto
• seankao-az
• qianheng-aws

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'Improve coderabbit config' accurately describes the main changes in the PR, which primarily involve improvements to the .coderabbit.yaml configuration file.
Description check	✅ Passed	The description is related to the changeset, explaining that the PR improves CodeRabbit config and removes invalid configuration. It aligns with the actual changes made to .coderabbit.yaml and .rules/REVIEW_GUIDELINES.md.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

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

[dai-chen]

Just curious are these new tenets collected / summarized from somewhere?

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In @.coderabbit.yaml:
- Around line 161-172: The tone_instructions entry is currently an array and
misplaced; change it to a single top-level string under the key
tone_instructions (not nested under reviews) and condense the content to one
clear sentence <=250 characters; update the existing array value (referenced as
tone_instructions) to a single-line string that summarizes the most important
directives (conciseness, focus on functional issues, avoid repeated suggestions)
so the schema accepts it.

[RyanL1997]

Hi @ykmr1224 , thanks for the change.