docs: add per-module .claude/CLAUDE.md and redirect AGENTS.md#1466
Merged
goldmedal merged 3 commits intoCanner:mainfrom Mar 20, 2026
Merged
docs: add per-module .claude/CLAUDE.md and redirect AGENTS.md#1466goldmedal merged 3 commits intoCanner:mainfrom
goldmedal merged 3 commits intoCanner:mainfrom
Conversation
Create module-specific .claude/CLAUDE.md for wren-core, wren-core-base, wren-core-py, and wren-core-legacy with build commands, architecture, and conventions. Add AGENTS.md redirects in all modules (including ibis-server and mcp-server) pointing to .claude/CLAUDE.md. Simplify root AGENTS.md to an index of module paths. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
github-actions
bot
added
documentation
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✅ Files skipped from review due to trivial changes (3)
📝 WalkthroughWalkthroughThis PR replaces the repository-level AGENTS.md with a pointer stub and adds module-scoped Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@ibis-server/.claude/CLAUDE.md`:
- Around line 7-15: The markdown has unlabeled/incorrect fenced code blocks and
missing blank lines around the Runtime table causing MD040/MD058 failures;
update each affected fenced block (the REST client block starting with "REST
client (MCP server / Wren Studio)" and the connector list block containing
"postgres mysql mssql ..." ) to use a language identifier (use "text" as
shown) and ensure there is a blank line immediately before and after each
triple-backtick fence, and add a blank line before and after the "**Runtime:**"
table so the table is separated by empty lines from surrounding text; no code
changes beyond replacing ``` with ```text and inserting the needed blank lines.
In `@mcp-server/.claude/CLAUDE.md`:
- Around line 7-12: The fenced code block that starts with "AI Agent
(Claude/Cline/Cursor)" is missing a language tag and triggers markdownlint
MD040; update the opening triple-backticks to include a language (e.g., ```text)
so the block becomes a labeled code fence and passes linting, leaving the block
content unchanged.
In `@wren-core/.claude/CLAUDE.md`:
- Line 27: Update the test command that currently shows "cargo test --lib
--tests --bins" to explicitly set the required stack size by prefixing it with
the environment variable RUST_MIN_STACK=8388608 (i.e., use
RUST_MIN_STACK=8388608 cargo test --lib --tests --bins) so the stack-size
requirement is not missed when copy-pasting; modify the line containing the
command string to include that env var and keep the existing comment if desired.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 8b48f5b2-dbac-49b8-96b4-2d314d705e7d
📒 Files selected for processing (13)
AGENTS.mdibis-server/.claude/CLAUDE.mdibis-server/AGENTS.mdmcp-server/.claude/CLAUDE.mdmcp-server/AGENTS.mdwren-core-base/.claude/CLAUDE.mdwren-core-base/AGENTS.mdwren-core-legacy/.claude/CLAUDE.mdwren-core-legacy/AGENTS.mdwren-core-py/.claude/CLAUDE.mdwren-core-py/AGENTS.mdwren-core/.claude/CLAUDE.mdwren-core/AGENTS.md
- Add `text` language tag to unlabeled code fences (MD040) - Add blank line before Runtime table (MD058) - Make RUST_MIN_STACK explicit in wren-core test command Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
wwwy3y3
approved these changes
Mar 20, 2026
nhaluc1005
pushed a commit
to nhaluc1005/text2sql-practice
that referenced
this pull request
Apr 3, 2026
…#1466) Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.claude/CLAUDE.mdfor each module (wren-core,wren-core-base,wren-core-py,wren-core-legacy,ibis-server,mcp-server) with module-specific build commands, architecture, and conventionsAGENTS.mdredirects in all modules pointing to.claude/CLAUDE.mdAGENTS.mdto an index of module pathsTest plan
.claude/CLAUDE.mdis picked up correctly when working within its moduleAGENTS.mdredirects are clear and point to the right files🤖 Generated with Claude Code
Summary by CodeRabbit