Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add superpowers plugin (.claude/settings.json) for /writing-plans skill - Add .coderabbit.yaml with path_instructions for all project layers - Update devcontainer.json: install CodeRabbit CLI in postCreateCommand - AGENTS.md: revamp Step 1 with /writing-plans, task granularity, and plan checklist - coding-style.md: add Pre-Implementation Layer Check table and naming timing rule - refactor-plan/instructions.md: add single-responsibility and granularity checks - claude-code.md: add /writing-plans skill, review chain, and milestone template - CONTRIBUTING.md: add AI tools section (Claude Code + CodeRabbit) - Update plan.md: mark all phases complete, add design notes and lessons learned Closes #3292 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Reorganize coding-style.md into grouped sections (Plan Time, Code Structure, Documentation, SvelteKit Patterns, Security, Svelte Patterns, Code Review) for better navigability - Clarify single-responsibility task rule: logical change scope rather than strict single-layer constraint - Make CodeRabbit CLI install non-fatal in devcontainer postCreateCommand - Expand Phase milestone definition and add setup instructions in claude-code.md Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
|
Warning Ignoring CodeRabbit configuration file changes. For security, only the configuration from the base branch is applied for open source repositories. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
🚧 Files skipped from review as they are similar to previous changes (2)
WalkthroughAdds AI-assisted planning and review tooling plus rule updates: superpowers Changes
Sequence Diagram(s)sequenceDiagram
participant Dev as Developer
participant SP as superpowers (/writing-plans)
participant CLI as CodeRabbit CLI
participant Repo as Repository
participant CI as CI
Dev->>SP: Request phased plan (/writing-plans)
SP-->>Dev: Returns 2-5min phased TODOs (milestones)
Dev->>Repo: Commit changes per phase
Dev->>CLI: Run `coderabbit review --plain` at phase boundary
CLI-->>Dev: Outputs severity triage (critical/high)
Repo->>CI: Push PR triggers CI CodeRabbit review
CI-->>Repo: CI review results (final gate)
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~30 minutes Poem
🚥 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)
📝 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: 5
🤖 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/rules/coding-style.md:
- Around line 31-34: Update the naming-conventions bullet for "New files" so it
reflects the actual project conventions instead of the aspirational ones: change
the custom route file example from snake_case (e.g., `user_profile.ts`) to plain
lowercase filenames (e.g., `auth.ts`, `kanban.ts`) and change the directory
convention example from kebab-case (e.g., `user-profile/`) to
underscore-prefixed directories (e.g., `_utils`, `_fixtures`, `_types`); keep
SvelteKit special file guidance as-is (`+page.svelte`, `+page.server.ts`,
`+server.ts`).
In @.claude/settings.json:
- Around line 5-10: The marketplace source under "extraKnownMarketplaces" ->
"superpowers-dev" currently points to a floating git URL; update the "source"
object (the entry with keys "source" and "url") to include a pinned Git ref by
adding a "ref" field set to the appropriate tag (e.g., "v1.0.0") so the "source"
object becomes stable and reproducible.
In @.coderabbit.yaml:
- Around line 44-47: The rules for the services path are self-contradictory:
update the instructions for the path 'src/**/services/**' to remove the circular
"No direct Prisma calls — delegate to service layer" line and replace it with a
clear rule such as "Service functions must return pure values or null; never
Response/json(). Service files may call Prisma directly — avoid database access
in controllers/handlers; keep Prisma usage encapsulated in services." Ensure the
change targets the existing instructions block for 'src/**/services/**' so
reviewers enforce services as the layer that may access Prisma while
controllers/handlers remain DB-free.
- Around line 11-22: The top-level exclusion '!prisma/migrations/**' prevents
the path_instructions for path 'prisma/**' (the migration rules) from ever
running; remove or modify that exclusion so migration files are not ignored
(e.g., delete the '!prisma/migrations/**' entry or replace it with a
non-excluding pattern) so the 'prisma/**' path_instructions (Migration files
must not be edited..., CHECK constraints documented in prisma/ERD.md) can be
enforced against prisma/migrations files.
In @.devcontainer/devcontainer.json:
- Line 28: Update the devcontainer postCreateCommand so the CodeRabbit installer
is pinned by exporting CODERABBIT_VERSION before invoking the installer; modify
the "postCreateCommand" string to set CODERABBIT_VERSION=<pinned-version> (e.g.,
v1.2.3) for the curl | sh step while preserving the existing chained commands
(npm install -g `@anthropic-ai/claude-code` and pnpm install) referenced in the
postCreateCommand entry in .devcontainer/devcontainer.json.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: d6fbfacf-866b-4681-b2fc-e8ded64f362e
📒 Files selected for processing (9)
.claude/rules/coding-style.md.claude/settings.json.claude/skills/refactor-plan/instructions.md.coderabbit.yaml.devcontainer/devcontainer.jsonAGENTS.mdCONTRIBUTING.mddocs/dev-notes/2026-03-20/plan-quality-improvement/plan.mddocs/guides/claude-code.md
- Add dedicated path instruction for prisma/migrations/** to suppress style/formatting noise while still flagging security and data-loss bugs - Remove migrations from global path_filters so the new path_instructions can apply - Fix incorrect path_instructions note for services layer (Prisma calls belong in services, not a violation) - Clarify underscore-prefix convention for helper dirs inside src/routes/ (_utils/, _types/, _fixtures/, _components/) in coding-style.md Co-Authored-By: Claude Sonnet 4.6 <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
1 participant
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.
close #3292
Summary by CodeRabbit
Documentation
Chores