Sync with TemplateDotNetTool PRs #15 and #16 by Copilot · Pull Request #64 · demaconsulting/BuildMark

Copilot · 2026-02-11T00:26:24Z

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 relationship
AGENTS.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 - Excludes AGENT_REPORT_*.md files from version control and spell checking
.markdownlint-cli2.jsonc - Consolidated configuration with ignore patterns (replaces .markdownlint.json)
AGENTS.md - Documents agent report file convention

Cleanup

Removed CHANGE_SUMMARY.md - This file was accidentally committed and didn't follow the AGENT_REPORT_*.md naming 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

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

N/A - Template synchronization maintenance

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

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.

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>

Port the following changes from the template: PR #61: Linting modernization - Convert .cspell.json to .cspell.yaml with header comments and improved ignorePaths - Convert .markdownlint-cli2.jsonc to .markdownlint-cli2.yaml with header, noBanner, noProgress - Update lint.sh to use Python venv, lint_error variable, no emojis, expanded file extensions - Update lint.bat to use Python venv, LINT_ERROR variable, expanded file extensions - Add pip-requirements.txt with yamllint==1.38.0 - Add cspell and markdownlint-cli2 to package.json devDependencies - Add .venv/ to .gitignore PR #64: Align lint YAML configs - Update .yamllint.yaml with header comments, DO NOT MODIFY policy, expanded ignore section (thirdparty, third-party, 3rd-party, .agent-logs/) and remove check-keys from truthy rule PR #75: Formalize build_notes artifact handling - Remove 'Move buildnotes.md to root' step from release.yaml - Use artifacts/buildnotes.md directly as release body file PR #77: Port markdown format requirements section - Update .github/standards/technical-documentation.md reference from .markdownlint-cli2.jsonc to .markdownlint-cli2.yaml Additional updates: - Update AGENTS.md references from .cspell.json to .cspell.yaml and .markdownlint-cli2.jsonc to .markdownlint-cli2.yaml - Update CONTRIBUTING.md spell checking section to reference .cspell.yaml and use ./lint.sh instead of individual lint commands Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

…ements (PRs #61–#77) (#115) * Initial plan * Apply template improvements: new agents, standards, .gitattributes, AGENTS.md update Agent-Logs-Url: https://github.com/demaconsulting/BuildMark/sessions/eef5ccc5-e481-4a3a-9c1f-5be3d92eedea Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * Fix AGENTS.md gaps vs template, update technical-documentation.md paths, fix .cspell.json pattern Agent-Logs-Url: https://github.com/demaconsulting/BuildMark/sessions/d10b0427-1ebb-4771-9ec1-223de918aa66 Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * Apply template changes from TemplateDotNetTool PRs #61-#77 Port the following changes from the template: PR #61: Linting modernization - Convert .cspell.json to .cspell.yaml with header comments and improved ignorePaths - Convert .markdownlint-cli2.jsonc to .markdownlint-cli2.yaml with header, noBanner, noProgress - Update lint.sh to use Python venv, lint_error variable, no emojis, expanded file extensions - Update lint.bat to use Python venv, LINT_ERROR variable, expanded file extensions - Add pip-requirements.txt with yamllint==1.38.0 - Add cspell and markdownlint-cli2 to package.json devDependencies - Add .venv/ to .gitignore PR #64: Align lint YAML configs - Update .yamllint.yaml with header comments, DO NOT MODIFY policy, expanded ignore section (thirdparty, third-party, 3rd-party, .agent-logs/) and remove check-keys from truthy rule PR #75: Formalize build_notes artifact handling - Remove 'Move buildnotes.md to root' step from release.yaml - Use artifacts/buildnotes.md directly as release body file PR #77: Port markdown format requirements section - Update .github/standards/technical-documentation.md reference from .markdownlint-cli2.jsonc to .markdownlint-cli2.yaml Additional updates: - Update AGENTS.md references from .cspell.json to .cspell.yaml and .markdownlint-cli2.jsonc to .markdownlint-cli2.yaml - Update CONTRIBUTING.md spell checking section to reference .cspell.yaml and use ./lint.sh instead of individual lint commands Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * Changes before error encountered Agent-Logs-Url: https://github.com/demaconsulting/BuildMark/sessions/a338e6d6-893c-4cbd-a64b-01abce243e9d Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * chore: apply template consistency updates from PRs #65-#77 - Rename agent files to match template naming convention (.agent.md suffix) - requirements-agent.md → requirements.agent.md - software-developer.md → software-developer.agent.md - technical-writer.md → technical-writer.agent.md - test-developer.md → test-developer.agent.md - Add tools: frontmatter field to agent files (PR #73) - code-review.agent.md: tools: [read, search, edit, execute, github, web, agent] - repo-consistency.agent.md: tools: [read, search, edit, execute, github, agent] - requirements.agent.md: tools: [read, search, edit, execute, github, web, agent] - software-developer.agent.md: tools: [read, search, edit, execute, github, agent] - technical-writer.agent.md: tools: [read, search, edit, execute, github, agent] - test-developer.agent.md: tools: [read, search, edit, execute, github, agent] - Update AGENTS.md to align with template - Rename 'Project Structure' section to 'Project Structure Template' - Update 'Continuous Compliance Requirements' section ending to match template Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * Remove agent files deleted from template in PR #76, update AGENTS.md Agent-Logs-Url: https://github.com/demaconsulting/BuildMark/sessions/80d99477-7c8a-469b-885d-da5c47aac27a Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> * Update THEORY-OF-OPERATIONS.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update technical-documentation.md to exactly match template Agent-Logs-Url: https://github.com/demaconsulting/BuildMark/sessions/61323e76-70ab-4b09-a70b-a97fa1132013 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> Co-authored-by: Malcolm Nixon <Malcolm.nixon@gmail.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Initial plan

173fed7

Copilot AI assigned Copilot and Malcolmnixon Feb 11, 2026

Copilot started work on behalf of Malcolmnixon February 11, 2026 00:26 View session

Copilot AI and others added 2 commits February 11, 2026 00:30

Sync with TemplateDotNetTool PRs #15 and #16

660a139

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

Copilot AI requested a review from Malcolmnixon February 11, 2026 00:32

Copilot finished work on behalf of Malcolmnixon February 11, 2026 00:32

Malcolmnixon requested changes Feb 11, 2026

View reviewed changes

Comment thread CHANGE_SUMMARY.md Outdated

Copilot started work on behalf of Malcolmnixon February 11, 2026 00:39 View session

Remove CHANGE_SUMMARY.md file that shouldn't be committed

09ebd2e

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

Copilot AI requested a review from Malcolmnixon February 11, 2026 00:40

Copilot finished work on behalf of Malcolmnixon February 11, 2026 00:40

Malcolmnixon approved these changes Feb 11, 2026

View reviewed changes

Malcolmnixon marked this pull request as ready for review February 11, 2026 00:50

Malcolmnixon merged commit f7e93dc into main Feb 11, 2026
11 checks passed

Malcolmnixon deleted the copilot/review-prs-15-16 branch February 11, 2026 00:50

This was referenced Feb 18, 2026

Bump the nuget-dependencies group with 1 update demaconsulting/TemplateDotNetLibrary#2

Merged

Bump the nuget-dependencies group with 2 updates demaconsulting/VersionMark#25

Merged

Copilot AI mentioned this pull request Mar 30, 2026

Add latest repo-consistency agent and apply TemplateDotNetTool improvements (PRs #61–#77) #115

Merged

21 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sync with TemplateDotNetTool PRs #15 and #16#64

Sync with TemplateDotNetTool PRs #15 and #16#64
Malcolmnixon merged 4 commits into
mainfrom
copilot/review-prs-15-16

Copilot AI commented Feb 11, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

Copilot AI commented Feb 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pull Request

Description

PR #15: Asymmetric Requirements-Test Relationship

PR #16: Agent Report Files & Markdownlint Consolidation

Cleanup

Type of Change

Related Issues

Pre-Submission Checklist

Build and Test

Code Quality

Quality Checks

Testing

Documentation

Additional Notes

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Copilot AI commented Feb 11, 2026 •

edited

Loading