docs: enhance copilot-instructions with commands and workflows

[nanotaboada]

Add comprehensive sections to improve agent efficiency:

• Essential commands for build, test, migrations, and Docker
• Common issues and workarounds (database paths, validation, caching)
• Commit message conventions with enforcement details
• Future evolution section linked to actual GitHub issues

These additions provide agents with actionable information to reduce exploration time and build/command failures when working with the repository for the first time.

https://github.blog/ai-and-ml/github-copilot/5-tips-for-writing-better-custom-instructions-for-copilot/

Summary by CodeRabbit

• Documentation
  • Updated development guidance with essential commands and workflows for building, testing, and database operations.
  • Reorganized future evolution considerations to reference GitHub issues for planned improvements.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

The pull request modifies the GitHub Copilot instructions file by introducing a new "Essential Commands & Workflows" section with development guidance (including Build & Run, Testing, Database Migrations, Docker Operations, and Rate Limiting), duplicating that section, and replacing the "Future Evolution Considerations" section with a consolidated GitHub issues reference.

Changes

Cohort / File(s)	Summary
Copilot Instructions Update .github/copilot-instructions.md	Added "Essential Commands & Workflows" section with concrete development commands and notes; duplicated the section; replaced "Future Evolution Considerations" with a consolidated note directing to GitHub issues while preserving items as linked references.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

• Areas requiring attention:
  • Verify that the duplication of the "Essential Commands & Workflows" section is intentional and not an accidental merge artifact
  • Confirm that the GitHub issues referenced in the "Future Evolution Considerations" replacement are correctly linked

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: enhancing the copilot-instructions documentation with new commands and workflows sections.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feature/custom-instructions-for-copilot

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3b27648 and 22de7c0.

📒 Files selected for processing (1)

• .github/copilot-instructions.md (1 hunks)

🧰 Additional context used

🧠 Learnings (1)

📚 Learning: 2025-08-30T12:45:10.366Z

Learnt from: CR
Repo: nanotaboada/Dotnet.Samples.AspNetCore.WebApi PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-08-30T12:45:10.366Z
Learning: Applies to Data/**/*.cs : Keep EF Core DbContext and database concerns under Data (e.g., PlayerDbContext)

Applied to files:

• .github/copilot-instructions.md

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: Codacy Static Code Analysis
• GitHub Check: Analyze (csharp)

🔇 Additional comments (6)

.github/copilot-instructions.md (6)

253-321: No duplication exists—AI summary is inaccurate.

Verification confirms only one occurrence of "Essential Commands & Workflows" in .github/copilot-instructions.md. The AI-generated summary incorrectly claimed the section was repeated twice. The file is correct as-is.

Likely an incorrect or invalid review comment.

---

317-320: No issues found—documentation is accurate.

Verification confirms the rate limiting configuration in appsettings.json matches the documentation. Both PermitLimit: 60 and WindowSeconds: 60 correctly represent "60 requests per 60 seconds," and QueueLimit: 0 matches the documented "immediate rejection when limit reached."

---

359-367: No issues found - GitHub issue links are valid and accessible.

All six GitHub issues referenced in the Future Evolution section are valid, currently open, and accurately describe the planned enhancements. The links are correct and will remain functional.

---

282-299: All documentation verified as accurate.

The migration script exists at the correct path and behaves exactly as documented:

• ✓ Resets the placeholder database file
• ✓ Runs EF Core migrations
• ✓ Copies from bin/Debug/net8.0/Data/ to Data/
• ✓ Validates dotnet ef CLI tool is installed globally

No issues found.

---

253-268: Port 9000 and endpoint URLs are correctly configured.

Verification confirms that port 9000 is properly configured in .vscode/launch.json via "ASPNETCORE_URLS": "https://localhost:9000", and the documented endpoints (Swagger and health check) are accurate and consistent with the project configuration. The README also confirms this port usage, indicating the documentation is aligned with the actual setup.

---

270-280: Remove or fix the non-existent .runsettings file reference in the coverage command.

The .runsettings file does not exist in the repository. The coverage test command at lines 270-280 references a missing file via --settings .runsettings, which will cause the command to fail. Either create the .runsettings file or remove the --settings .runsettings parameter from the documented command.

⛔ Skipped due to learnings

Learnt from: CR
Repo: nanotaboada/Dotnet.Samples.AspNetCore.WebApi PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-08-30T12:45:10.366Z
Learning: Applies to test/**/*.cs : Write unit tests using xUnit

Learnt from: CR
Repo: nanotaboada/Dotnet.Samples.AspNetCore.WebApi PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-08-30T12:45:10.366Z
Learning: Applies to test/** : Place unit tests under the test/ directory (e.g., Unit/PlayerServiceTests.cs)

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[codacy-production]

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation	Diff coverage
✅ +0.00%	✅ ∅

Coverage variation details

	Coverable lines	Covered lines	Coverage
Common ancestor commit (3b27648)	1169	196	16.77%
Head commit (22de7c0)	1169 (+0)	196 (+0)	16.77% (+0.00%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details

	Coverable lines	Covered lines	Diff coverage
Pull request (#309)	0	0	∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences