Skip to content

chore: add CLAUDE.md, clang-tidy, and agent workflow tooling - #419

Merged
16bit-ykiko merged 1 commit into
mainfrom
chore/agent-workflow-and-clang-tidy
Apr 9, 2026
Merged

16bit-ykiko merged 1 commit into
mainfrom
chore/agent-workflow-and-clang-tidy

Conversation

@16bit-ykiko

@16bit-ykiko 16bit-ykiko commented Apr 9, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Add .claude/CLAUDE.md with project guide, C++ coding style, correction patterns from past AI interactions, and pre-commit/pre-PR workflows
  • Add .claude/commands/ with /build, /test, /format slash commands for Claude Code
  • Add .clang-tidy configuration
  • Add scripts/run_clang_tidy.py for parallel clang-tidy with progress reporting (reads CDB, filters to src/ and tests/ only)
  • Add pixi tasks: clang-tidy, lint-cpp, lint-python, lint
  • Add clang-tools dependency for clang-tidy
  • Update .gitignore to track .claude/CLAUDE.md and .claude/commands/

Test plan

  • pixi run format passes
  • pixi run lint-python passes (ruff check)
  • scripts/run_clang_tidy.py correctly filters to project files only

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation

    • Added comprehensive project operating guide with coding standards and best practices.
    • Added command documentation for build, format, and test operations.
  • Chores

    • Added C++ code linting configuration and automation via clang-tidy.
    • Integrated Python code linting checks.
    • Added build dependency for Clang tools.
    • Updated repository configuration to track documentation files.

- Add .claude/CLAUDE.md with project guide, coding style, correction
  patterns from past interactions, and pre-commit/pre-PR workflows
- Add .claude/commands/ for /build, /test, /format slash commands
- Add .clang-tidy configuration for the project
- Add scripts/run_clang_tidy.py for parallel clang-tidy execution
- Add pixi tasks: clang-tidy, lint-cpp, lint-python, lint
- Add clang-tools dependency for clang-tidy
- Update .gitignore to track .claude/CLAUDE.md and .claude/commands/

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Apr 9, 2026 •

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 92224e81-4dd0-45f6-b877-aa0de5543c98

📥 Commits

Reviewing files that changed from the base of the PR and between 3dab2ea and f47f7ff.

⛔ Files ignored due to path filters (1)
  • pixi.lock is excluded by !**/*.lock
📒 Files selected for processing (8)
  • .clang-tidy
  • .claude/CLAUDE.md
  • .claude/commands/build.md
  • .claude/commands/format.md
  • .claude/commands/test.md
  • .gitignore
  • pixi.toml
  • scripts/run_clang_tidy.py

📝 Walkthrough

Walkthrough

The changes introduce linting infrastructure for the C++ project by adding clang-tidy configuration, a parallel-execution Python runner script, pixi task definitions for Python and C++ linting, comprehensive project documentation including coding guidelines, and updated git configuration to track documentation files.

Changes

Cohort / File(s) Summary
Clang-tidy Linting Setup
.clang-tidy, scripts/run_clang_tidy.py
New clang-tidy configuration file defining enabled/disabled checks and naming conventions. Companion Python script runs clang-tidy in parallel across source/test files from CMake's compile_commands.json, reporting per-file status and exit code aggregation.
Pixi Build & Lint Tasks
pixi.toml
Added clang-tools = "==20.1.8" dependency and introduced lint tasks: lint-python (ruff check), lint-cpp (runs parallel clang-tidy), clang-tidy (depends on lint-cpp), and aggregate lint task. All build tasks support optional type parameter (Debug/RelWithDebInfo).
Project Documentation
.claude/CLAUDE.md, .claude/commands/build.md, .claude/commands/format.md, .claude/commands/test.md
Added comprehensive C++ codebase operating guide (CLAUDE.md) covering core patterns, code reuse requirements, style conventions, testing practices, and pre-PR checklists. Added command reference docs for build, format, and test operations with usage examples and parameter specifications.
Git Configuration
.gitignore
Updated to selectively track .claude/CLAUDE.md and .claude/commands/ while ignoring other .claude/ contents.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

Possibly related PRs

  • #335 — Restructures pixi.toml build/task configuration, overlapping with the new clang-tidy and lint task additions in this PR.

Poem

🐰 wiggles nose at the config
Our C++ code now stands pristine,
With clang-tidy's watchful sheen,
Parallel checks run swift and clean,
The finest-linted code you've seen! ✨

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/agent-workflow-and-clang-tidy

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@16bit-ykiko
16bit-ykiko merged commit 342d82a into main Apr 9, 2026
9 of 10 checks passed
@16bit-ykiko
16bit-ykiko deleted the chore/agent-workflow-and-clang-tidy branch April 9, 2026 11:09
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.

1 participant