docs: Add CLAUDE.md with build, lint, test commands and architecture overview.#2230
docs: Add CLAUDE.md with build, lint, test commands and architecture overview.#2230wraymo wants to merge 1 commit intoy-scope:mainfrom
Conversation
…overview. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
WalkthroughIntroduces a new Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@CLAUDE.md`:
- Line 100: Update the architecture table entry that currently uses
`log-viewer-webui` so it matches the actual submodule name `yscope-log-viewer`;
locate and replace the `log-viewer-webui` token in the CLAUDE.md table row with
`yscope-log-viewer` (ensuring the TypeScript/description cell remains unchanged)
so both the entry at Line 100 and the reference at Line 129 use the canonical
`yscope-log-viewer` name.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
| | `clp-package-utils` | Python | CLI tools for operating the CLP package | | ||
| | `clp-mcp-server` | Python | MCP server exposing CLP search to AI assistants | | ||
| | `webui` | TypeScript/Node.js | Main web interface (client + server) | | ||
| | `log-viewer-webui` | TypeScript | Standalone log viewer (git submodule in webui) | |
There was a problem hiding this comment.
Unify log viewer component naming to avoid contributor confusion.
Line 100 lists log-viewer-webui, while Line 129 refers to yscope-log-viewer as the actual submodule. Please use one canonical name in both places (preferably yscope-log-viewer) to keep the architecture table unambiguous.
Also applies to: 129-129
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@CLAUDE.md` at line 100, Update the architecture table entry that currently
uses `log-viewer-webui` so it matches the actual submodule name
`yscope-log-viewer`; locate and replace the `log-viewer-webui` token in the
CLAUDE.md table row with `yscope-log-viewer` (ensuring the
TypeScript/description cell remains unchanged) so both the entry at Line 100 and
the reference at Line 129 use the canonical `yscope-log-viewer` name.
Description
Adds a
CLAUDE.mdfile at the repo root. This file is automatically loaded by Claude Code when working in this repository, giving the AI assistant immediate context about how to build, lint, and test the project without needing to search through the codebase.The file covers:
task,task core,task rust,task webui, package tars)Checklist
Validation performed
Manually verified all commands against the taskfiles and dev docs in
docs/src/dev-docs/.Summary by CodeRabbit