Add CodeRabbit config focused on quality, memory, and test coverage

[ccgsnet]

No description provided.

[coderabbitai]

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: 71522bbb-b5a0-437c-a2fd-2c42a42c24bd

Commits

Reviewing files that changed from the base of the PR and between 536bf08 and c0b26d7.

Files selected for processing (1)

• .coderabbit.yaml

🚧 Files skipped from review as they are similar to previous changes (1)

• .coderabbit.yaml

---

• Configuration/docs only (no new production logic): Adds a .coderabbit.yaml CodeRabbit workflow plus .github/copilot-instructions.md coding standards; the PR itself doesn’t introduce correctness, thread-safety, or error-handling changes.
• Quality risk coverage enforced for future PRs: Enables multiple static analysis tools (clang/cppcheck/clippy/flake8/ruff/shellcheck + github-checks) and sets a CodeRabbit “chill” review profile that explicitly calls out correctness, thread safety, and error handling.
• Hot-path memory/allocation focus: Review instructions prioritize flagging unnecessary copies of large objects (e.g., Properties/vectors/strings/maps), heap allocations while holding api_mutex, missing reserve(), shared_ptr cycle/use-after-free risks, and broader memory/performance issues.
• Behavior-change test matching gate: Adds a pre-merge custom check (“Tests for behavior changes”) that errors when production logic under src/ or 3rd_party_slots/ changes behavior without accompanying test updates (C++ src/tests/cpp/*_test.cc, Python tests under 3rd_party_slots/python_client/, Rust crate tests) or without justification in the PR description.

Walkthrough

Adds .coderabbit.yaml configuring review profile, static analysis tools (clang, cppcheck, clippy, flake8, ruff, shellcheck), pre-merge gating, per-path review instructions, and knowledge base settings. Adds .github/copilot-instructions.md with per-language coding standards and an explicit list of reviewer constraints for the DAS repository.

Changes

AI Reviewer and Coding Standards Configuration

Layer / File(s)	Summary
CodeRabbit profile, tools, and pre-merge gates .coderabbit.yaml	Sets review language, focus, and tone; enables clang, cppcheck, clippy, flake8, ruff, shellcheck, and GitHub checks with a long CI timeout; adds a custom "Tests for behavior changes" pre-merge gate that errors when production logic changes without test updates or justification; configures unit-test automation, base branches, and path filters to ignore generated and lock files.
Per-path review instructions and knowledge base .coderabbit.yaml	Defines path_instructions rulesets enforcing formatting and test-coverage priorities for C++ sources and tests, Python client code and tests, Rust client code, GitHub workflow files, and config directories; enables knowledge_base guidance and sets chat.auto_reply to true.
Copilot coding standards and reviewer constraints .github/copilot-instructions.md	Introduces project context, general principles, C++ formatting and namespace/error-handling patterns, Python and Rust conventions, config preservation rules, and an explicit list of suggestions automated reviewers must not make.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	No description was provided by the author, making it impossible to evaluate relevance to the changeset.	Add a pull request description explaining the purpose of the CodeRabbit configuration and how it benefits the project.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main objective of adding CodeRabbit configuration with focus on quality, memory, and test coverage.
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.
Tests For Behavior Changes	✅ Passed	PR only touches CI configuration (.coderabbit.yaml) and documentation (.github/copilot-instructions.md), with no production code changes in src/ or 3rd_party_slots/.

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

Finishing Touches

Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch coderabbit

---

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

[coderabbitai]

Actionable comments posted: 1

Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.coderabbit.yaml:
- Around line 44-54: The test requirement check is currently not enforced
because mode is set to warning and request_changes_workflow is set to false. To
make the check actually block merges when the FAIL criteria are triggered,
change the mode value from warning to error in the instructions section, and
change request_changes_workflow from false to true at the top level of the
configuration. This combination will ensure that PRs with production logic
changes and missing test updates cannot be merged without addressing the check.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

Review info

Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: ccf0b7a2-0477-458b-8293-fc4047c9adc6

Commits

Reviewing files that changed from the base of the PR and between 79d3c78 and 536bf08.

Files selected for processing (2)

• .coderabbit.yaml
• .github/copilot-instructions.md