Add AI tooling config to ignored paths#796
Conversation
Add `.claude/`, `CLAUDE.md`, and `AGENTS.md` to the ignore list. These are configuration files for AI coding tools (Claude Code, Codex, etc.) that may appear in theme repos. Like `.git` and `.svn`, they are tooling artifacts that should not be included in theme validation. Symlinked config files in `.claude/` currently trigger fatal GS030-ASSET-SYM errors even though they are never included in theme zip uploads.
|
ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughThis PR updates Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 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)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. 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 |
Adds
.claude/,CLAUDE.md, andAGENTS.mdto the ignore list inread-theme.js.These are configuration files for AI coding tools (Claude Code, Codex, etc.) that increasingly appear in theme repos. Like
.gitand.svn, they are tooling artifacts that should not be validated as theme files.Currently, a symlinked config file in
.claude/triggers a fatalGS030-ASSET-SYMerror when scanning a local theme directory, even though these files are never included in theme zip uploads.Note
Low Risk
Low risk: only expands the theme file ignore list, affecting which paths are skipped/removed during scanning without changing validation logic.
Overview
Theme scanning now skips AI tooling artifacts by adding
.claude/,CLAUDE.md, andAGENTS.mdto the ignored paths inlib/read-theme.js, preventing these files from being processed (and, when scanning extracted zips in temp dirs, from triggering errors like symlink checks).Reviewed by Cursor Bugbot for commit 8378fcf. Bugbot is set up for automated code reviews on this repo. Configure here.