Skip to content

chore: add CLAUDE.md for Claude Code project context#251

Open
LEILEI0628 wants to merge 3 commits intoarana-db:feat/raftfrom
LEILEI0628:chore/add-claude-md-raft
Open

chore: add CLAUDE.md for Claude Code project context#251
LEILEI0628 wants to merge 3 commits intoarana-db:feat/raftfrom
LEILEI0628:chore/add-claude-md-raft

Conversation

@LEILEI0628
Copy link
Copy Markdown

Provide build commands, architecture overview, lint rules, PR title convention, command system guide, and testing instructions.

Provide build commands, architecture overview, lint rules, PR title
convention, command system guide, and testing instructions.

Co-Authored-By: LEILEI <leilei20010628@gmail.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 22, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a5b6c816-9f20-47e2-9389-973594edf580

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@github-actions github-actions bot added ✏️ Feature new feature 📒 Documentation Improvements or additions to documentation labels Mar 22, 2026
Copy link
Copy Markdown
Contributor

@AlexStocks AlexStocks left a comment

Choose a reason for hiding this comment

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

架构和测试文档需要修正

- Fix runtime architecture description: remove "separate tokio runtimes"
  claim, describe async message channel communication accurately
- Fix executor description: "thread pool" → "tokio async task pool"
- Remove misleading CF numeric indices from storage model
- Add crate name examples for unit test command
- Add complete Python integration test steps with server startup

Co-Authored-By: LEILEI <leilei20010628@gmail.com>
CLAUDE.md Outdated

# Run
cargo run --bin kiwi # Run server (debug)
cargo run --release # Run server (release)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

这条命令从仓库根目录跑不通。当前仓库是 virtual workspace,根上没有可执行 package,cargo run --release 会直接失败;如果是想启动服务器,至少要把 server/kiwi 目标写全,比如 cargo run -p server --bin kiwi --release,或者明确说明需要先切到 src/server

…space

Co-Authored-By: LEILEI <leilei20010628@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📒 Documentation Improvements or additions to documentation ✏️ Feature new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants