docs(copilot): unify instructions as canonical single-file format#387
docs(copilot): unify instructions as canonical single-file format#387nanotaboada merged 2 commits intomasterfrom
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. WalkthroughThis pull request restructures the Copilot instructions documentation file by reorganizing sections, expanding guidance on workflows and conventions, and deletes the AGENTS.md file entirely, removing centralized developer-facing operational instructions. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~5 minutes Possibly related PRs
🚥 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)
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 |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.github/copilot-instructions.md:
- Line 109: The "Add an endpoint" workflow omits updating AutoMapper profiles
and repository methods; update the workflow text to include steps for
adding/updating the AutoMapper Profiles/ (e.g., create mapping for the new DTO
in Profiles/) and for adding necessary data-access logic in Repositories/ (e.g.,
add repository method(s) used by the new service), so the sequence becomes: add
DTO in Models/, update Profiles/ (AutoMapper), add repository method(s) in
Repositories/, add service method in Services/, add controller action in
Controllers/, add validator in Validators/, add tests, then run pre-commit
checks.
- Line 94: Remove the obsolete mention of azure-pipelines.yml from the CI/CD
list and update the "Add an endpoint" workflow guidance to also remind
contributors to update the AutoMapper profile (specifically the
PlayerMappingProfile class) when new DTOs or entity↔DTO mappings are introduced;
reference AutoMapper and PlayerMappingProfile so reviewers remember to add
mapping entries whenever endpoints add or change DTOs.
Consolidate .github/copilot-instructions.md and AGENTS.md into a single always-on file with a consistent section order (Overview, Tech Stack, Structure, Coding Guidelines, Commands, Agent Mode). - Replace two-file system with one canonical copilot-instructions.md - Absorb AGENTS.md content (autonomy levels, workflows) into Agent Mode Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
df455bd to
611b5cf
Compare
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|



Consolidate .github/copilot-instructions.md and AGENTS.md into a single always-on file with a consistent section order (Overview, Tech Stack, Structure, Coding Guidelines, Commands, Agent Mode).
Summary by CodeRabbit