docs(skills): add multi-entity naming and config-driven enum rules#3254
Conversation
There was a problem hiding this comment.
Pull request overview
Updates internal Claude skills documentation to clarify naming for multi-entity modules and to encourage config-driven enums when scaffolding new modules.
Changes:
- Document multi-entity module file naming (module-prefixed filenames even when modeling multiple entities).
- Add guidance to
/create-moduleon defining enum-like business values in module config and referencing them from models/schemas.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| .claude/skills/naming/SKILL.md | Adds “multi-entity modules” naming guidance and a generalized {module}.{entity}... pattern. |
| .claude/skills/create-module/SKILL.md | Adds a “config-driven enums” subsection describing where to define and how to reference business enums. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (2)
WalkthroughDocumentation updates to Claude skills modules: correcting header formatting in create-module skill documentation and updating a reference path from relative to absolute format, plus adding a mapping example to naming skill documentation. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes ✨ 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 Tip You can customize the tone of the review comments and chat replies.Configure the |
- Use existing organizations module examples instead of hypothetical billing - Fix file path reference to modules/users/models/users.schema.js - Renumber orphan 4b as sub-section under step 4
…-skills-naming-conventions
Summary
/namingskill (files must be prefixed by module name, not entity name)/create-moduleskill (business values must come from module config, not hardcoded)Test plan
Summary by CodeRabbit