Additional agent cleanups and synchronizations.#78
Conversation
There was a problem hiding this comment.
Pull request overview
Documentation-focused synchronization of agent/standards guidance plus a small expansion of the lint scripts to include .NET formatting verification.
Changes:
- Add
dotnet format --verify-no-changesto bothlint.shandlint.bat. - Standardize agent reporting filename placeholders and tighten/normalize agent report template fields.
- Refresh standards docs (project paths, ReviewMark examples, documentation layout) and adjust cspell wordlist.
Reviewed changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| lint.sh | Adds .NET formatting verification to the lint suite. |
| lint.bat | Adds .NET formatting verification to the lint suite (Windows). |
| AGENTS.md | Updates project structure examples and agent log filename convention. |
| .github/standards/technical-documentation.md | Updates documentation folder layout example (design + reqstream structure). |
| .github/standards/reviewmark-usage.md | Updates ReviewMark needs-review and example paths to newer project layout placeholders. |
| .github/standards/design-documentation.md | Clarifies subsystem folder naming guidance. |
| .github/standards/csharp-testing.md | Fixes “Anti-patterns” wording consistency. |
| .github/agents/repo-consistency.agent.md | Standardizes agent log filename placeholder and report template fields. |
| .github/agents/quality.agent.md | Standardizes agent log filename placeholder and report template fields. |
| .github/agents/implementation.agent.md | Standardizes agent log filename placeholder and report template fields. |
| .github/agents/developer.agent.md | Standardizes agent log filename placeholder and report template fields. |
| .github/agents/code-review.agent.md | Updates review report filename placeholder and report template fields. |
| .cspell.yaml | Updates allowed technical terms (adds venv, removes some entries). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 14 out of 14 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Agent-Logs-Url: https://github.com/demaconsulting/TemplateDotNetLibrary/sessions/2c5eac44-55f7-4f5d-809a-2fb03886399f Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Pull Request
Description
Additional agent cleanups and synchronizations, including updating
.reviewmark.yamlto addrequirements.yamlto theneeds-reviewpatterns to align with the guidance in.github/standards/reviewmark-usage.md.Type of Change
Related Issues
N/A
Pre-Submission Checklist
Before submitting this pull request, ensure you have completed the following:
Build and Test
dotnet build --configuration Releasedotnet test --configuration ReleaseCode Quality
dotnet format --verify-no-changesQuality Checks
Please run the following checks before submitting:
./lint.sh(Unix/macOS) orcmd /c lint.bat/./lint.bat(Windows)Testing
Documentation
Additional Notes
N/A