Skip to content

Replace agents with TemplateDotNetTool agent structure#51

Merged
Malcolmnixon merged 5 commits intomainfrom
copilot/replace-agents-from-template-dotnet-tool
Feb 11, 2026
Merged

Replace agents with TemplateDotNetTool agent structure#51
Malcolmnixon merged 5 commits intomainfrom
copilot/replace-agents-from-template-dotnet-tool

Conversation

Copy link
Contributor

Copilot AI commented Feb 11, 2026

Pull Request

Description

Aligns SarifMark's agent structure with TemplateDotNetTool template, replacing 3 monolithic agents with 6 specialized agents that separate concerns:

Removed:

  • documentation-writer.md, project-maintainer.md, software-quality-enforcer.md (overlapping responsibilities)

Added:

  • requirements-agent.md - Requirements development and test coverage strategy
  • technical-writer.md - Documentation accuracy and regulatory best practices
  • repo-consistency-agent.md - Template drift detection (uses inline URLs for better AI context)
  • code-quality-agent.md - Linting, static analysis, security scanning
  • software-developer.md - Production code and self-validation tests (literate programming)
  • test-developer.md - Unit/integration tests (AAA pattern)

All agents customized for SarifMark (replacing TemplateTool → SarifMark) except repo-consistency-agent.md which intentionally preserves TemplateDotNetTool references as its purpose is maintaining consistency with that template.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code quality improvement

Related Issues

Pre-Submission Checklist

Before submitting this pull request, ensure you have completed the following:

Build and Test

  • Code builds successfully: dotnet build --configuration Release
  • All tests pass: dotnet test --configuration Release
  • Code produces zero warnings

Code Quality

  • Code formatting is correct: dotnet format --verify-no-changes
  • New code has appropriate XML documentation comments
  • Static analyzer warnings have been addressed

Quality Checks

Please run the following checks before submitting:

  • Spell checker passes: cspell "**/*.{md,cs}"
  • Markdown linter passes: markdownlint "**/*.md"
  • YAML linter passes: yamllint '**/*.{yml,yaml}'

Testing

  • Added unit tests for new functionality
  • Updated existing tests if behavior changed
  • All tests follow the AAA (Arrange, Act, Assert) pattern
  • Test coverage is maintained or improved

Documentation

  • Updated README.md (if applicable)
  • Updated ARCHITECTURE.md (if applicable)
  • Added code examples for new features (if applicable)
  • Updated requirements.yaml (if applicable)

Additional Notes

Added buildnotes to .cspell.json dictionary. The repo-consistency-agent.md file uses inline URLs (<https://...>) rather than reference-style links to preserve better context for AIs reading sections of the file.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 3 commits February 11, 2026 12:36
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
…t.md

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copilot AI changed the title [WIP] Replace repository agents with TemplateDotNetTool agents Replace agents with TemplateDotNetTool agent structure Feb 11, 2026
Copilot AI requested a review from Malcolmnixon February 11, 2026 12:41
@@ -0,0 +1,124 @@
---
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's not use the link-reference format here. While normal markdown viewers would have no problem I'm concerned about the context of AIs as they read sections of the file.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reverted to inline URL format in commit 9e03e54. The file now uses <https://github.com/demaconsulting/TemplateDotNetTool> inline URLs and plain "TemplateDotNetTool" text instead of reference-style links.

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copilot AI added a commit that referenced this pull request Mar 4, 2026
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Malcolmnixon added a commit that referenced this pull request Mar 4, 2026
* Initial plan

* Apply template updates from PRs #31-#51

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

* Apply semantic requirement IDs from template PR #50

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants