Sync with TemplateDotNetTool PRs #15 and #16#64
Merged
Malcolmnixon merged 4 commits intomainfrom Feb 11, 2026
Merged
Conversation
Changes from PR #15: Clarify asymmetric requirements-test relationship - Updated agent documentation to clarify all requirements MUST link to tests - Clarified that NOT all tests need to link to requirements - Tests may exist for corner cases, design validation, failure scenarios, etc. - Updated requirements-agent.md, test-developer.md, and AGENTS.md Changes from PR #16: Add agent report file exclusions and consolidate markdownlint config - Established AGENT_REPORT_*.md naming convention for temporary agent report files - Added AGENT_REPORT_*.md exclusions to .gitignore - Added AGENT_REPORT_*.md to .cspell.json ignorePaths - Replaced .markdownlint.json with .markdownlint-cli2.jsonc that includes ignore patterns - Added documentation in AGENTS.md for agent report file guidelines All changes maintain BuildMark-specific functionality while aligning with template best practices.
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Update repo based on review of recent PRs
Sync with TemplateDotNetTool PRs #15 and #16
Feb 11, 2026
Malcolmnixon
requested changes
Feb 11, 2026
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Malcolmnixon
approved these changes
Feb 11, 2026
This was referenced Feb 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request
Description
Aligns BuildMark with TemplateDotNetTool improvements from PRs #15 and #16, covering requirements-test relationships and agent workflow infrastructure.
PR #15: Asymmetric Requirements-Test Relationship
Clarifies that requirements MUST link to tests (CI-enforced), but not all tests need requirements. Updated agent documentation:
.github/agents/requirements-agent.md- Added explicit guidance on unlinked tests (regression, edge cases, internal utilities, exploratory).github/agents/test-developer.md- New "Tests and Requirements" section explaining the asymmetric relationshipAGENTS.md- Documented BuildMark-specific test naming (BuildMark_*for self-validation)PR #16: Agent Report Files & Markdownlint Consolidation
Standardizes agent inter-communication and simplifies linting:
.gitignore,.cspell.json- ExcludesAGENT_REPORT_*.mdfiles from version control and spell checking.markdownlint-cli2.jsonc- Consolidated configuration with ignore patterns (replaces.markdownlint.json)AGENTS.md- Documents agent report file conventionCleanup
CHANGE_SUMMARY.md- This file was accidentally committed and didn't follow theAGENT_REPORT_*.mdnaming convention. Per the documentation updates, agent report files should not be committed.All changes are documentation and configuration only. No production code modified.
Type of Change
Related Issues
N/A - Template synchronization maintenance
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:
cspell "**/*.{md,cs}"markdownlint "**/*.md"yamllint '**/*.{yml,yaml}'Testing
Documentation
Additional Notes
All 127 tests pass. Zero build warnings. Zero linting errors. Configuration changes are backward compatible.
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.