Repository navigation
Conversation
Co-Authored-By: Claude Opus 5.5 <[email protected]>
Gitignore the per-user .claude/settings.local.json and .claude/launch.json, which Biome otherwise formats and fails on, and exclude Markdown from Biome, which has no handler for it and warned on every SKILL.md. Co-Authored-By: Claude Opus 5.5 <[email protected]>
|
Outcome: CI passes (check, toolchain) and the PR is ready for review. One open decision: keep (Written by Claude Opus 5.5) |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
No actionable comments were generated in the recent review.
 |
| Check name | Status | Explanation |
|---|---|---|
| Title check |  |
The title "Pass Biome with Claude Code's local files present" matches the main change: the PR gitignores Claude Code per-user files and changes Biome config so that just check passes with those files … |
| Description check | |
The description states that the PR completes the biome-local quest so that just check passes with Claude Code's per-user files present. It lists the .gitignore additions for .claude/settings.local.jso… |
| Docstring Coverage | |
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
Full details: Title check
-
Explanation
The title "Pass Biome with Claude Code's local files present" matches the main change: the PR gitignores Claude Code per-user files and changes Biome config so that just check passes with those files present. It is specific and conveys the primary intent.
Full details: Description check
-
Explanation
The description states that the PR completes the biome-local quest so that just check passes with Claude Code's per-user files present. It lists the .gitignore additions for .claude/settings.local.json and .claude/launch.json, the Biome files.includes change adding !**/*.md, and the deletion of the quest and its README link. It also explains the launch.json decision and the verification steps. This matches the changeset.
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)
Finishing Touches
-
✨ Simplify code
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
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 help to get the list of available commands.
Completes the
quest/m0/biome-local.mdquest:just checknow passes on a checkout that has Claude Code's per-user files, and Biome no longer warns about files it can't handle..claude/settings.local.jsonand.claude/launch.json. When they're untracked, Biome formats them and fails. Biome reads the repo's.gitignorebut not a globalcore.excludesFile, so a personal global ignore doesn't help.!**/*.mdto Biome'sfiles.includes. Biome has no Markdown handler, so every trackedSKILL.mdprinted afiles/missingHandlerwarning.quest/m0/README.md.Decision for review
The quest's Plan only named
settings.local.json. I also ignored.claude/launch.json, the Claude desktop app's preview config. It has the same problem, and the maintainer's checkout has an untracked copy containing machine-specific paths (cwd, ports). If you'd rather share a launch config across contributors, drop that line and commit a.claude/launch.jsonthat passes Biome.Verification
Before the fix, with both files present in the worktree,
bunx biome checkreported 2 format errors (launch.json,settings.local.json) and 10files/missingHandlerwarnings. After the fix, with the same files still present,quest check,just fix, andjust checkall pass with no warnings.(Written by Claude Opus 5.5)
🤖 Generated with Claude Code