* Initial agent update and requirements changes to unwind design-poisoning of requirements.

* Feedback from formal reviews.

* More updates from formal reviews.

* Fix lint issues: convert long markdown tables to bullet lists, fix YAML line lengths

Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/aa9db58a-0ce9-40fe-98f5-fc1e4a421394

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

* Address PR review comments: remove duplicate sentence, rename test, update requirement

Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/1b1b544c-5f86-4128-b8b5-ba8e8b6c1fda

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

* Redirect Console.Error in ValidationTests unsupported-extension test

Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/172e7a88-1bed-4e18-a0bf-1d22ac10dfb9

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

---------

Co-authored-by: Malcolm Nixon <Malcolm.Nixon@hiarc.inc>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

Bumps demaconsulting.buildmark from 1.0.0 to 1.1.0
Bumps demaconsulting.reqstream from 1.7.0 to 1.9.0
Bumps demaconsulting.reviewmark from 1.1.0 to 1.2.0
Bumps demaconsulting.sonarmark from 1.4.0 to 1.5.0
Bumps Microsoft.CodeAnalysis.NetAnalyzers from 10.0.202 to 10.0.203
Bumps Microsoft.Extensions.FileSystemGlobbing from 10.0.6 to 10.0.7
Bumps Microsoft.SourceLink.GitHub from 10.0.202 to 10.0.203
Bumps SonarAnalyzer.CSharp from 10.23.0.137933 to 10.24.0.138807

---
updated-dependencies:
- dependency-name: demaconsulting.buildmark
  dependency-version: 1.1.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: demaconsulting.reqstream
  dependency-version: 1.9.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: demaconsulting.reviewmark
  dependency-version: 1.2.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: demaconsulting.sonarmark
  dependency-version: 1.5.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: Microsoft.CodeAnalysis.NetAnalyzers
  dependency-version: 10.0.203
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
- dependency-name: Microsoft.CodeAnalysis.NetAnalyzers
  dependency-version: 10.0.203
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
- dependency-name: Microsoft.Extensions.FileSystemGlobbing
  dependency-version: 10.0.7
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
- dependency-name: Microsoft.SourceLink.GitHub
  dependency-version: 10.0.203
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
- dependency-name: SonarAnalyzer.CSharp
  dependency-version: 10.24.0.138807
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: SonarAnalyzer.CSharp
  dependency-version: 10.24.0.138807
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps Microsoft.NET.Test.Sdk from 18.4.0 to 18.5.1
Bumps MSTest.TestAdapter from 4.2.1 to 4.2.2
Bumps MSTest.TestFramework from 4.2.1 to 4.2.2
Bumps SonarAnalyzer.CSharp from 10.24.0.138807 to 10.25.0.139117
Bumps YamlDotNet from 17.0.1 to 17.1.0

---
updated-dependencies:
- dependency-name: Microsoft.NET.Test.Sdk
  dependency-version: 18.5.1
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: MSTest.TestAdapter
  dependency-version: 4.2.2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
- dependency-name: MSTest.TestFramework
  dependency-version: 4.2.2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
- dependency-name: SonarAnalyzer.CSharp
  dependency-version: 10.25.0.139117
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: SonarAnalyzer.CSharp
  dependency-version: 10.25.0.139117
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: YamlDotNet
  dependency-version: 17.1.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update agents, layout, xUnit, verification, generated, etc.

* Run all 25 formal reviews and fix all identified issues

- Fix design, verification, and requirements docs across all review-sets
- Add missing tests, fix test names, and correct stale documentation
- Split compound requirements into atomic items
- Correct OTS verification docs to reference build.yaml (not build.ps1)
- All 732 tests passing; full lint compliance

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Address PR review thread feedback

Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/56fe4b7d-dadb-469b-97d3-5642a9be964e

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

* Fix build_notes pandoc input paths

Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/7b88860c-b2a6-41b9-9c22-e3d13931e35c

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

* Use PDFsharp in test PDF helper

Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/9ca59781-4fd6-4dee-8d80-7d0f46fd8219

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

* Fix IndexingTests helper import and code_quality generated paths

Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/c82a362e-fb42-4a56-bd05-25fb792a7087

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

* Fix remaining pandoc definitions for generated markdown inputs

Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/fe71784f-8e79-4773-b0d4-593007f3c982

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

* Align docs and traceability with argument handling behavior

Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/8ebeb49b-8a2f-415e-9a5a-9ba04fc46303

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

* Use 'Unknown argument' wording for CLI invalid flags

Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/ecb99268-8127-4f1e-9914-e77bf43a179a

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

---------

Co-authored-by: Copilot <223556219+Copilot@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>

Bumps demaconsulting.buildmark from 1.1.0 to 1.2.2
Bumps demaconsulting.reqstream from 1.9.0 to 1.10.0
Bumps demaconsulting.versionmark from 1.3.0 to 1.4.3
Bumps Polyfill from 10.3.0 to 10.5.1
Bumps xunit.runner.visualstudio from 3.1.1 to 3.1.5

---
updated-dependencies:
- dependency-name: demaconsulting.buildmark
  dependency-version: 1.2.2
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: demaconsulting.reqstream
  dependency-version: 1.10.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: demaconsulting.versionmark
  dependency-version: 1.4.3
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: Polyfill
  dependency-version: 10.5.1
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: xunit.runner.visualstudio
  dependency-version: 3.1.5
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps coverlet.collector from 10.0.0 to 10.0.1
Bumps Microsoft.CodeAnalysis.NetAnalyzers from 10.0.203 to 10.0.300
Bumps Microsoft.Extensions.FileSystemGlobbing from 10.0.7 to 10.0.8
Bumps Microsoft.SourceLink.GitHub from 10.0.203 to 10.0.300

---
updated-dependencies:
- dependency-name: coverlet.collector
  dependency-version: 10.0.1
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
- dependency-name: Microsoft.CodeAnalysis.NetAnalyzers
  dependency-version: 10.0.300
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
- dependency-name: Microsoft.CodeAnalysis.NetAnalyzers
  dependency-version: 10.0.300
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
- dependency-name: Microsoft.Extensions.FileSystemGlobbing
  dependency-version: 10.0.8
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
- dependency-name: Microsoft.SourceLink.GitHub
  dependency-version: 10.0.300
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Changes from reviews.

* More template synchronization and reviews.

* fix: address PR review comments and build-notes metadata check

Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/7d0be210-84ac-42f0-be02-376f60143543

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

* fix: align verification PDF Subject assertion with template metadata pattern

Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/e3ddda55-7ec5-455b-b31e-321121813f7a

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

* fix: align user guide PDF Subject assertion with template pattern

Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/f025d561-3a05-42c0-840a-4aeaf37e7c55

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

* fix: align trace matrix PDF Subject assertion with template

Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/032ae459-6313-4efa-bff8-27e83186c3a7

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

---------

Co-authored-by: Malcolm Nixon <Malcolm.Nixon@hiarc.inc>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>

Bumps Polyfill from 10.5.1 to 10.7.0
Bumps SonarAnalyzer.CSharp from 10.25.0.139117 to 10.26.0.140279
Bumps YamlDotNet from 17.1.0 to 18.0.0

---
updated-dependencies:
- dependency-name: Polyfill
  dependency-version: 10.7.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: Polyfill
  dependency-version: 10.7.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: SonarAnalyzer.CSharp
  dependency-version: 10.26.0.140279
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: SonarAnalyzer.CSharp
  dependency-version: 10.26.0.140279
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: SonarAnalyzer.CSharp
  dependency-version: 10.26.0.140279
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: YamlDotNet
  dependency-version: 18.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: nuget-dependencies
- dependency-name: YamlDotNet
  dependency-version: 18.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: nuget-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps demaconsulting.fileassert from 0.3.0 to 0.4.0
Bumps DemaConsulting.TestResults from 1.7.0 to 1.9.0
Bumps Microsoft.NET.Test.Sdk from 18.5.1 to 18.6.0
Bumps Polyfill from 10.7.0 to 10.8.0
Bumps SonarAnalyzer.CSharp from 10.26.0.140279 to 10.27.0.140913

---
updated-dependencies:
- dependency-name: demaconsulting.fileassert
  dependency-version: 0.4.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: DemaConsulting.TestResults
  dependency-version: 1.9.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: DemaConsulting.TestResults
  dependency-version: 1.9.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: Microsoft.NET.Test.Sdk
  dependency-version: 18.6.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: Microsoft.NET.Test.Sdk
  dependency-version: 18.6.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: Polyfill
  dependency-version: 10.8.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: Polyfill
  dependency-version: 10.8.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: SonarAnalyzer.CSharp
  dependency-version: 10.27.0.140913
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: SonarAnalyzer.CSharp
  dependency-version: 10.27.0.140913
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: SonarAnalyzer.CSharp
  dependency-version: 10.27.0.140913
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps demaconsulting.pandoctool from 3.9.0.2 to 3.10.0
Bumps demaconsulting.weasyprinttool from 68.1.0 to 69.0.0
Bumps Polyfill from 10.8.0 to 10.8.1

---
updated-dependencies:
- dependency-name: demaconsulting.pandoctool
  dependency-version: 3.10.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: demaconsulting.weasyprinttool
  dependency-version: 69.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: nuget-dependencies
- dependency-name: Polyfill
  dependency-version: 10.8.1
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
- dependency-name: Polyfill
  dependency-version: 10.8.1
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps demaconsulting.buildmark from 1.2.2 to 1.3.0
Bumps Microsoft.CodeAnalysis.NetAnalyzers from 10.0.300 to 10.0.301
Bumps Microsoft.Extensions.FileSystemGlobbing from 10.0.8 to 10.0.9
Bumps Polyfill from 10.8.1 to 10.10.0

---
updated-dependencies:
- dependency-name: demaconsulting.buildmark
  dependency-version: 1.3.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: Microsoft.CodeAnalysis.NetAnalyzers
  dependency-version: 10.0.301
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
- dependency-name: Microsoft.CodeAnalysis.NetAnalyzers
  dependency-version: 10.0.301
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
- dependency-name: Microsoft.CodeAnalysis.NetAnalyzers
  dependency-version: 10.0.301
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
- dependency-name: Microsoft.Extensions.FileSystemGlobbing
  dependency-version: 10.0.9
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
- dependency-name: Microsoft.Extensions.FileSystemGlobbing
  dependency-version: 10.0.9
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-dependencies
- dependency-name: Polyfill
  dependency-version: 10.10.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
- dependency-name: Polyfill
  dependency-version: 10.10.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps demaconsulting.fileassert from 0.4.0 to 0.5.1

---
updated-dependencies:
- dependency-name: demaconsulting.fileassert
  dependency-version: 0.5.1
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps the github-actions-dependencies group with 1 update: [actions/checkout](https://github.com/actions/checkout).


Updates `actions/checkout` from 6 to 7
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v6...v7)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions-dependencies
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat: add context files — global and per-review-set (#78)

- Add optional top-level context: list to .reviewmark.yaml
- Add optional per-review-set context: list to .reviewmark.yaml
- Context entries use the same glob-pattern syntax as paths:
- Context files excluded from fingerprint computation
- Context files excluded from needs-review coverage enforcement
- --elaborate output includes Context subsection with [global]/[local] labels
- Update requirements, design, verification, README, user guide, and tests

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* docs: clarify context does not satisfy needs-review coverage (#78)

* fix: sanitize null context entries and update user guide field counts

- Filter null/whitespace entries from global and per-review-set context
  lists in BuildConfiguration() to prevent GlobMatcher throwing
- Emit lint warning for whitespace-only context entries (consistent with
  paths: handling)
- Update user guide: three top-level keys -> four, three fields -> four

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* fix: uniform null/whitespace sanitization across all YAML list fields

- Filter null/whitespace entries from needs-review, paths, global context,
  and per-review-set context in BuildConfiguration() consistently
- Add ValidateTopLevel() lint validator for needs-review and global context
  whitespace entries (warning severity)
- Add lint warning for individual whitespace-only paths entries in ValidateReviews()
- Add tests for all three new lint warning paths

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Malcolm Nixon <Malcolm.Nixon@hiarc.inc>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* fix: remove [global]/[local] labels from context output; add README global context example

- Context files in --elaborate output are now listed as plain paths
  without [global]/[local] scope prefixes
- Global context files still appear before local context files
- Add top-level context: example to README .reviewmark.yaml schema
- Update tests and docs to match new plain-path output

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: remove [global]/[local] labels from requirements, design, and verification docs

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: align context comment positions in README.md example

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: improve README.md example YAML comment style

- Move block-level evidence-source and context comments to own lines
- Remove redundant inline comment from per-review-set context:
- Normalise inline comment alignment to column 26

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: strengthen context elaboration tests

- Assert exact list-item format (- \path\) not just substring contains
- Assert DoesNotContain [global]/[local] labels in both context tests
- Add GlobalContextBeforeLocalContext test to verify ordering

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Malcolm Nixon <Malcolm.Nixon@hiarc.inc>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* feat: suppress context files that are also under review in elaboration output

When a file appears in both the context: list and the paths: list for a
given review set, it is suppressed from the Context subsection and shown
only in the Files subsection. This prevents the same file from being
presented with contradictory purpose in a single elaboration output.

- Update ElaborateReviewSet() to subtract files-under-review from context
- Add requirement ReviewMark-Config-ContextDeduplication
- Add design note for the deduplication step
- Add tests for suppression and non-suppression cases
- Add verification entries

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* feat: suppress context files that are also under review in elaboration output

When a file appears in both the context: list and the paths: list for a
given review set, it is suppressed from the Context subsection and shown
only in the Files subsection. This prevents the same file from being
presented with contradictory purpose in a single elaboration output.

- Update ElaborateReviewSet() to subtract files-under-review from context
- Add requirement ReviewMark-Config-ContextDeduplication
- Add design note for the deduplication step
- Add tests for suppression and non-suppression cases
- Add verification entries

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Malcolm Nixon <Malcolm.Nixon@hiarc.inc>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

…82)

* feat: per-review-set context exclusion patterns can suppress global context files

Global and per-review-set context patterns are now resolved as a single
ordered sequence. This allows per-review-set exclusion patterns (! prefix)
to suppress files added by the global context, using GlobMatcher's existing
ordered include/exclude semantics.

- Simplify ElaborateReviewSet() to use a single combined GlobMatcher call
- Remove now-redundant Distinct() call (GlobMatcher deduplicates internally)
- Add requirement ReviewMark-Config-ContextExclusionPatterns
- Add design note for combined ordered context resolution
- Add tests for exclusion and isolation across review sets
- Add verification entries

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: remove duplicate Requirements Coverage heading in verification doc

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: replace global-before-local ordering guarantee with lexicographic sort

Context files are sorted lexicographically (consistent with Files), not
ordered by source (global before local). The ordering guarantee was
undocumented intent that was never reliably preserved by GlobMatcher's
sorted output. Removing it simplifies the implementation and aligns
context with the Files section behaviour.

- Rename GlobalContextBeforeLocalContext test to BothGlobalAndLocalContextAppear
- Replace index-comparison ordering assertion with order-agnostic presence checks
- Update requirement, design, and verification docs to reflect lexicographic sort

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: correct spelling behaviour -> behavior in requirements YAML

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Remove lexicographic-order claim from context requirement and design

Ordering is an implementation detail of GlobMatcher, not a user requirement.
Removed the 'in lexicographic order (consistent with the Files section)'
language from ReviewMark-Config-ContextInElaboration and the corresponding
design description.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Malcolm Nixon <Malcolm.Nixon@hiarc.inc>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Constrain review-set paths to needs-review file set

Review set 'paths' file resolution is now intersected with the
needs-review file set. This prevents build artifacts (bin/, obj/)
from being included in review sets when broad glob patterns are
used, which was causing fingerprints to change on every build and
reviews to go spuriously stale.

- ReviewSet.GetFiles/GetFingerprint accept an optional constraint set
- ReviewMarkConfiguration passes the needs-review set as the constraint
  in PublishReviewPlan, PublishReviewReport, and ElaborateReviewSet
- When needs-review is absent/empty, no constraint is applied
- context patterns remain unconstrained (reference material, not files
  under review)

New requirements, design, verification, and user guide documentation
added to cover the constraint behavior.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Fix review findings from formal review of feature branch

- Requirement ReviewMark-Config-PathsConstrainedToNeedsReview: change
  'strict subset' to 'subset' (review set may equal full needs-review set)
- User guide examples: add tests/**/*.cs to needs-review in two examples
  so all paths entries are covered under the new constraint
- Design review-mark-configuration.md: add GetNeedsReviewFiles() to Key
  Methods section
- Design review-mark.md: correct Load() description; glob resolution
  occurs in the generation/reporting methods, not Load()

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Malcolm Nixon <Malcolm.Nixon@hiarc.inc>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>