Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: demaconsulting/ReviewMark
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 0.1.0
Choose a base ref
...
head repository: demaconsulting/ReviewMark
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 1.0.0
Choose a head ref
  • 19 commits
  • 122 files changed
  • 5 contributors

Commits on Mar 16, 2026

  1. Bump the nuget-dependencies group with 7 updates (#23)

    Bumps DemaConsulting.TestResults from 1.5.0 to 1.6.0
    Bumps dotnet-sonarscanner from 11.1.0 to 11.2.0
    Bumps Microsoft.CodeAnalysis.NetAnalyzers from 10.0.103 to 10.0.201
    Bumps Microsoft.Extensions.FileSystemGlobbing from 10.0.3 to 10.0.5
    Bumps Microsoft.SourceLink.GitHub from 10.0.103 to 10.0.201
    Bumps Polyfill from 9.18.0 to 9.22.0
    Bumps SonarAnalyzer.CSharp from 10.20.0.135146 to 10.21.0.135717
    
    ---
    updated-dependencies:
    - dependency-name: DemaConsulting.TestResults
      dependency-version: 1.6.0
      dependency-type: direct:production
      update-type: version-update:semver-minor
      dependency-group: nuget-dependencies
    - dependency-name: dotnet-sonarscanner
      dependency-version: 11.2.0
      dependency-type: direct:production
      update-type: version-update:semver-minor
      dependency-group: nuget-dependencies
    - dependency-name: Microsoft.CodeAnalysis.NetAnalyzers
      dependency-version: 10.0.201
      dependency-type: direct:production
      update-type: version-update:semver-patch
      dependency-group: nuget-dependencies
    - dependency-name: Microsoft.CodeAnalysis.NetAnalyzers
      dependency-version: 10.0.201
      dependency-type: direct:production
      update-type: version-update:semver-patch
      dependency-group: nuget-dependencies
    - dependency-name: Microsoft.Extensions.FileSystemGlobbing
      dependency-version: 10.0.5
      dependency-type: direct:production
      update-type: version-update:semver-patch
      dependency-group: nuget-dependencies
    - dependency-name: Microsoft.SourceLink.GitHub
      dependency-version: 10.0.201
      dependency-type: direct:production
      update-type: version-update:semver-patch
      dependency-group: nuget-dependencies
    - dependency-name: Polyfill
      dependency-version: 9.22.0
      dependency-type: direct:production
      update-type: version-update:semver-minor
      dependency-group: nuget-dependencies
    - dependency-name: SonarAnalyzer.CSharp
      dependency-version: 10.21.0.135717
      dependency-type: direct:production
      update-type: version-update:semver-minor
      dependency-group: nuget-dependencies
    - dependency-name: SonarAnalyzer.CSharp
      dependency-version: 10.21.0.135717
      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>
    dependabot[bot] authored Mar 16, 2026
    Configuration menu
    Copy the full SHA
    140041d View commit details
    Browse the repository at this point in the history

Commits on Mar 24, 2026

  1. Bump the nuget-dependencies group with 1 update (#24)

    Bumps coverlet.collector from 8.0.0 to 8.0.1
    
    ---
    updated-dependencies:
    - dependency-name: coverlet.collector
      dependency-version: 8.0.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>
    dependabot[bot] authored Mar 24, 2026
    Configuration menu
    Copy the full SHA
    a84765c View commit details
    Browse the repository at this point in the history
  2. Bring in latest repo-consistency agent and apply template improvement…

    …s from TemplateDotNetTool (#25)
    
    * Initial plan
    
    * Bring in latest repo-consistency agent and template improvements from TemplateDotNetTool
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/1ac7f038-065a-41da-8e89-1a36e85c95fa
    
    * Apply remaining missed template improvements: file renames, yaml format conversions, lint script rewrite
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/42beae80-b9d8-4be1-ad38-c556f57170b4
    
    * Fix linting issues: wrap long lines in CONTRIBUTING.md, update workflow, add spell check policy to AGENTS.md
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/3150f0bf-4a5f-4a48-a324-6ed80d75870a
    
    * Update .cspell.json references to .cspell.yaml in contributing guide and technical-writer agent
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/81812042-1ec3-4213-a9b5-de4ea286f9c1
    
    ---------
    
    Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Copilot and Malcolmnixon authored Mar 24, 2026
    Configuration menu
    Copy the full SHA
    c34b46a View commit details
    Browse the repository at this point in the history

Commits on Mar 25, 2026

  1. feat: Add --lint support for validating definition files (#27)

    * Initial plan
    
    * feat: Add --lint support for validating definition files
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/a75bab3c-44b6-4dcf-8da7-eb85c41f6ec3
    
    * refactor: address review feedback - TestDirectory helper, error message assertions, corrupted YAML tests
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/245f177b-039d-4058-a8eb-3fce0818cd47
    
    * feat: improve lint error quality - filename/line numbers, includes support, and stronger test assertions
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/7318deb4-3700-47ba-b4b8-b7c3e37c4b4e
    
    * revert: remove includes feature - reviewmark uses a single definition file
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/25f4d0e6-fdf3-493b-803c-d19cfb0967fd
    
    * refactor: accumulate all lint errors in one pass via ReviewMarkConfiguration.Lint
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/0745bda8-94ca-4460-a7b2-c361d77474c0
    
    * fix: clarify case-sensitivity intent and tighten Lint unit test count assertion
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/0745bda8-94ca-4460-a7b2-c361d77474c0
    
    * fix: dispose Context before reading log file, consistent error messages, TestDirectory safety, shared type validator
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/206afe15-d999-4902-a3d6-381cc23061cb
    
    * docs: add --lint to README and user guide
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/a5fb2a08-3a4d-4a2e-908c-d95bbe84dae0
    
    * fix: correct British spelling 'recognised' to American 'recognized'
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/2712fd02-b8b6-44bc-99bf-2a973437ffbb
    
    * feat: change lint output to [location]: [severity]: [issue] format
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/3576c6ae-5203-472e-b193-49f8d6e46feb
    
    ---------
    
    Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Copilot and Malcolmnixon authored Mar 25, 2026
    Configuration menu
    Copy the full SHA
    4beb01d View commit details
    Browse the repository at this point in the history

Commits on Mar 26, 2026

  1. Add 'none' evidence source to project setup (#29)

    * Initial plan
    
    * Add 'none' evidence source type that returns an empty index
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/75a0f826-6b34-4a3a-b44e-45546b78f006
    
    * Complete 'none' evidence source: tests, docs, requirements
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/a8d3447c-9cdf-471f-9314-e21a1eb99b7d
    
    * Update src/DemaConsulting.ReviewMark/ReviewMarkConfiguration.cs
    
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    
    * Address review feedback: update guide.md intro and README comment for 'none' source
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/729531fc-fd47-41a4-ae10-68d50821e6f1
    
    * Update src/DemaConsulting.ReviewMark/ReviewMarkConfiguration.cs
    
    Co-authored-by: Copilot <175728472+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>
    Co-authored-by: Malcolm Nixon <Malcolm.nixon@gmail.com>
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    4 people authored Mar 26, 2026
    Configuration menu
    Copy the full SHA
    dada883 View commit details
    Browse the repository at this point in the history

Commits on Mar 28, 2026

  1. Sync with TemplateDotNetTool template PRs (#58, #65, #66, #67, #70, #…

    …75) (#30)
    
    * Initial plan
    
    * Apply template repo changes from TemplateDotNetTool PRs
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/4f05d485-5375-4cca-952d-ddacf934b681
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Improve .reviewmark.yaml review-sets with requirements and design context
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/45abfc5f-35fa-4006-b4f7-b799bd49c895
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Split requirements.yaml into per-software-item files under docs/reqstream/ and update .reviewmark.yaml
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/05c02ae5-9340-45f7-bc6a-bccbaa4d5dc3
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Update .github/agents/technical-writer.agent.md
    
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    
    * Update docs/code_review_report/introduction.md
    
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    
    * Fix build-docs ReviewMark step to use global 'reviewmark' tool instead of dotnet run
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/5a482424-635c-4f04-afff-9a32e7b57c52
    
    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>
    4 people authored Mar 28, 2026
    Configuration menu
    Copy the full SHA
    61697ee View commit details
    Browse the repository at this point in the history
  2. Add software design documentation, requirements files, and expand rev…

    …iew-sets (#31)
    
    * Initial plan
    
    * Add design documentation and update review-sets in .reviewmark.yaml
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/a5158598-c9e7-4185-a00d-0b6c7069931e
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Address PR review feedback: add requirements files, ValidationTests, fix review-sets and diagram
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/11aefa92-cb5d-46a4-ad14-fd7c21807456
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Rename requirements files per convention and split OTS into per-component files
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/041a3a1a-f5b2-4bec-b0d4-135c272431e2
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Add unit-context/program requirements, rename Index.cs to ReviewIndex.cs, convert diagrams to text
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/2cd94fd3-6efa-4e14-87c6-aaa5861db6d6
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Add ReviewEvidence record documentation to review-index.md
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/5bc38306-41ce-46fc-ab70-c8424822b0b7
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Fix requirements test linkages to match actual test method names
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/69ec20b8-a181-4b63-b73e-400ec7abd055
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Update docs/design/review-mark-configuration.md
    
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    
    * Update docs/design/program.md
    
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    
    * Update docs/reqstream/unit-program.yaml
    
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    
    * Update docs/design/review-index.md
    
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    
    * Update docs/design/review-index.md
    
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    
    * Update docs/design/context.md
    
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    
    * Update docs/design/system.md
    
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    
    * Update docs/design/definition.yaml
    
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    
    * Update docs/reqstream/unit-path-helpers.yaml
    
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    
    * Update docs/design/review-mark-configuration.md
    
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    
    * Revise design docs to focus on WHAT/WHY rather than implementation HOW
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/1632c6d8-1a27-4cfa-a722-72ae2f98ecd2
    
    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>
    4 people authored Mar 28, 2026
    Configuration menu
    Copy the full SHA
    168e65f View commit details
    Browse the repository at this point in the history

Commits on Mar 30, 2026

  1. Apply TemplateDotNetTool PRs #76, #77, and #78 - Standards-driven age…

    …nt restructure and docs reorganization (#34)
    
    * Initial plan
    
    * Apply TemplateDotNetTool PR #76 - Agent and lint cleanup
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/5e347cdf-1fe1-49e2-81c0-7f3ad8fa994c
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Apply TemplateDotNetTool PR #77 - Add Markdown format requirements to technical-documentation.md
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/80fe8d02-e5bc-44a7-ab7f-d0cec390ce31
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Rename docs/guide to docs/user_guide and guide.md to introduction.md
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/c4adc73c-69b1-4b10-b8e5-c37e5d901ffa
    
    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>
    Copilot and Malcolmnixon authored Mar 30, 2026
    Configuration menu
    Copy the full SHA
    e07fa7f View commit details
    Browse the repository at this point in the history
  2. Bump demaconsulting.pandoctool from 3.9.0 to 3.9.0.2 (#36)

    ---
    updated-dependencies:
    - dependency-name: demaconsulting.pandoctool
      dependency-version: 3.9.0.2
      dependency-type: direct:production
      update-type: version-update:semver-patch
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    dependabot[bot] authored Mar 30, 2026
    Configuration menu
    Copy the full SHA
    7980360 View commit details
    Browse the repository at this point in the history
  3. Bump the nuget-dependencies group with 5 updates (#35)

    Bumps demaconsulting.reqstream from 1.4.1 to 1.5.0
    Bumps demaconsulting.sonarmark from 1.2.0 to 1.3.0
    Bumps demaconsulting.versionmark from 1.0.0 to 1.1.0
    Bumps Polyfill from 9.22.0 to 9.23.0
    Bumps SonarAnalyzer.CSharp from 10.21.0.135717 to 10.22.0.136894
    
    ---
    updated-dependencies:
    - dependency-name: demaconsulting.reqstream
      dependency-version: 1.5.0
      dependency-type: direct:production
      update-type: version-update:semver-minor
      dependency-group: nuget-dependencies
    - dependency-name: demaconsulting.sonarmark
      dependency-version: 1.3.0
      dependency-type: direct:production
      update-type: version-update:semver-minor
      dependency-group: nuget-dependencies
    - dependency-name: demaconsulting.versionmark
      dependency-version: 1.1.0
      dependency-type: direct:production
      update-type: version-update:semver-minor
      dependency-group: nuget-dependencies
    - dependency-name: Polyfill
      dependency-version: 9.23.0
      dependency-type: direct:production
      update-type: version-update:semver-minor
      dependency-group: nuget-dependencies
    - dependency-name: SonarAnalyzer.CSharp
      dependency-version: 10.22.0.136894
      dependency-type: direct:production
      update-type: version-update:semver-minor
      dependency-group: nuget-dependencies
    - dependency-name: SonarAnalyzer.CSharp
      dependency-version: 10.22.0.136894
      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>
    dependabot[bot] authored Mar 30, 2026
    Configuration menu
    Copy the full SHA
    392113f View commit details
    Browse the repository at this point in the history

Commits on Mar 31, 2026

  1. Restructure ReviewMark into subsystem folders following ReqStream tem…

    …plate pattern (#37)
    
    * Initial plan
    
    * Restructure source into subsystem folder layout
    
    - Move source files into Cli/, Configuration/, Indexing/, SelfTest/ subsystem folders
    - Move test files into matching subsystem folders
    - Move design docs into matching subsystem folders under docs/design/
    - Update namespaces: DemaConsulting.ReviewMark.{Cli,Configuration,Indexing,SelfTest}
    - Add using directives to cross-subsystem consumers
    - Update docs/design/definition.yaml with new input-file paths
    - Rewrite docs/design/introduction.md with subsystem-aware content
    - Create docs/reqstream/subsystem-indexing.yaml
    - Create docs/reqstream/subsystem-self-test.yaml
    - Update requirements.yaml to include new subsystem requirement files
    - Update .reviewmark.yaml with new review-sets and updated paths
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Fix lint errors: correct import ordering and yaml line length
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Use glob pattern for design docs in ReviewMark-Design review-set
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/4ae917d8-dada-479b-a86a-e2ef2dd74fde
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Reorganize docs/reqstream into subsystem subdirectories mirroring docs/design
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/20e6ebb2-c034-4d2d-bea2-2f9a1d21613d
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Update .reviewmark.yaml
    
    Co-authored-by: Copilot <175728472+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>
    Co-authored-by: Malcolm Nixon <Malcolm.nixon@gmail.com>
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    4 people authored Mar 31, 2026
    Configuration menu
    Copy the full SHA
    c38d2dd View commit details
    Browse the repository at this point in the history
  2. Formal review of all 13 review-sets; apply findings (#38)

    * Initial plan
    
    * fix: address code review findings from formal review of all review-sets
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/78f46014-06de-48bf-85c7-0edbb7983736
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * fix: correct status coverage gaps in user guide and system requirements
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/3a13bcb6-e2e3-4c73-9f48-566e4c5c8f96
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Update docs/design/cli/context.md
    
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    
    * fix: apply PR review suggestions — rephrase location doc and expand enforce justification
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/5adf8c75-f585-4404-b78d-e5314af3906c
    
    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>
    4 people authored Mar 31, 2026
    Configuration menu
    Copy the full SHA
    539dfcf View commit details
    Browse the repository at this point in the history

Commits on Apr 2, 2026

  1. Sync AGENTS.md, agent files, and standards from TemplateDotNetTool (#39)

    - Updated AGENTS.md with latest template content
    - Updated .github/agents/*.md (code-review, developer, implementation, quality, repo-consistency)
    - Updated .github/standards/*.md (csharp-testing, reqstream-usage, reviewmark-usage, technical-documentation)
    - Added .github/standards/design-documentation.md (new in template)
    
    Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Copilot and Malcolmnixon authored Apr 2, 2026
    Configuration menu
    Copy the full SHA
    f40bd3b View commit details
    Browse the repository at this point in the history
  2. feat: unified Load mechanism with linting support and ReportIssues he…

    …lper (#41)
    
    * Initial plan
    
    * feat: implement LoadWithLinting unified load+lint mechanism
    
    - Add LintSeverity enum, LintIssue record, ReviewMarkLoadResult record
    - Add ReviewMarkConfiguration.LoadWithLinting() as the unified mechanism
    - Refactor Load() and Lint() to delegate to LoadWithLinting()
    - Update Program.RunLintLogic and RunDefinitionLogic to use LoadWithLinting
    - Add 6 new tests (167 total passing)
    - Update design docs and requirements
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/72f2e0f3-5b69-43b1-bfe0-820655e63846
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * feat: rename LoadWithLinting→Load, remove old Load/Lint wrappers, add ReportIssues(Context)
    
    - Rename LoadWithLinting to Load (the one clean API for all callers)
    - Remove old exception-throwing Load and string-returning Lint wrappers
    - Add ReviewMarkLoadResult.ReportIssues(Context) helper — eliminates the
      duplicated foreach loops in RunLintLogic and RunDefinitionLogic
    - Update all tests: old exception/string-list tests migrated to result API,
      LoadWithLinting_* tests renamed to Load_*, new ReportIssues test added
    - Update design doc and requirements
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/6ea03d7a-aae2-4774-adad-ac3f8c98c0de
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * fix: update reqstream test names after Load/Lint rename
    
    The previous rename of test methods broke the reqstream traceability:
    - ReviewMarkConfiguration_Load_InvalidYaml_ErrorIncludesFilenameAndLine
      → ReviewMarkConfiguration_Load_InvalidYaml_ReturnsNullConfigWithErrorIssue
    - ReviewMarkConfiguration_Load_MissingEvidenceSource_ErrorIncludesFilename
      → ReviewMarkConfiguration_Load_MissingEvidenceSource_ReturnsNullConfigWithErrorIssue
    - ReviewMarkConfiguration_Lint_MultipleErrors_ReturnsAll
      → ReviewMarkConfiguration_Load_MultipleErrors_ReturnsAllIssues
    - ReviewMarkConfiguration_Lint_NoneEvidenceSource_NoErrors
      → ReviewMarkConfiguration_Load_NoneEvidenceSource_NoIssues
    
    Updated in subsystem-cli.yaml and unit-review-index.yaml to satisfy
    ReviewMark-Cmd-Lint and ReviewMark-EvidenceSource-None requirements.
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/ccf913ed-b0f1-4487-8964-1c5468d72010
    
    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>
    Copilot and Malcolmnixon authored Apr 2, 2026
    Configuration menu
    Copy the full SHA
    cb37c3d View commit details
    Browse the repository at this point in the history

Commits on Apr 3, 2026

  1. restructure: move design and reqstream files into review-mark system …

    …folder (#43)
    
    * Binary-copy updated files from TemplateDotNetTool
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/2f33b6eb-2560-4958-a7a8-8bc20814d740
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * restructure: move design and reqstream files into review-mark system folder
    
    - Move docs/design files into docs/design/review-mark/ system folder
    - Move docs/reqstream files into docs/reqstream/review-mark/ system folder
    - Rename reqstream files to remove subsystem-/unit- prefixes
    - Create missing subsystem overview design files (cli.md, configuration.md, indexing.md, self-test.md)
    - Update requirements.yaml includes to new paths
    - Update .reviewmark.yaml: new paths, renamed IDs, add ReviewMark-Configuration subsystem
    - Update docs/design/introduction.md Folder Layout section
    - Update docs/design/definition.yaml resource-path and input-files
    - Complies with updated .github/standards
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Update docs/design/review-mark/cli/cli.md
    
    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
    
    * fix: split configuration requirements, add tests to subsystem reviews, fix lint issues
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/c5dea3ca-a569-475d-acf6-c15f9ca66bc0
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * refine: use class-scoped titles for unit requirements
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/c5dea3ca-a569-475d-acf6-c15f9ca66bc0
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * sync: update PathHelpers to use post-combine containment check from template
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/3983415c-b3a4-43dc-8d11-0ddd968e0198
    
    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>
    4 people authored Apr 3, 2026
    Configuration menu
    Copy the full SHA
    8be65f1 View commit details
    Browse the repository at this point in the history
  2. Fix documentation, requirements, and subsystem test suite issues iden…

    …tified in formal review (#44)
    
    * fix: address formal review issues in documentation and requirements files
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Add subsystem test suites and requirements children links
    
    - Add CliTests.cs: subsystem integration tests for CLI (Context + Program)
    - Add ConfigurationTests.cs: subsystem integration tests for Configuration
    - Add IndexingTests.cs: subsystem integration tests for Indexing
    - Add SelfTestTests.cs: subsystem integration tests for SelfTest
    - Update cli.yaml: add children links and new subsystem test references
    - Update configuration.yaml: add children links and new subsystem test references
    - Update indexing.yaml: add children links and new subsystem test references
    - Update self-test.yaml: add children links and new subsystem test references
    - Update .reviewmark.yaml: update subsystem review-sets to include new test files
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * fix: add 'selftest' to cspell word list
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * fix: shorten MD013-violating table row in review-mark-configuration.md
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/4e67c1ba-0bd1-4537-b878-2b85e42f4d22
    
    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>
    Copilot and Malcolmnixon authored Apr 3, 2026
    Configuration menu
    Copy the full SHA
    f658cc5 View commit details
    Browse the repository at this point in the history

Commits on Apr 4, 2026

  1. Fix review issues: I/O error handling in Program.cs, console color in…

    … Context.cs, missing test refs in cli.yaml (#45)
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/ac56b6a1-fc95-41b9-8f21-2a884f7253a0
    
    Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    Copilot and Malcolmnixon authored Apr 4, 2026
    Configuration menu
    Copy the full SHA
    69d9205 View commit details
    Browse the repository at this point in the history

Commits on Apr 5, 2026

  1. Add IntegrationTest_* and Cli_* tests; update reqstream YAML for test…

    …-linkage compliance (#46)
    
    * Sync .github/agents and .github/standards from TemplateDotNetTool
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/5fe50d02-c30f-45d3-a137-438007a8d9b6
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Add IntegrationTest_* and Cli_* test methods; update reqstream YAML for test-linkage compliance
    
    - Add 7 new IntegrationTest_* methods to IntegrationTests.cs covering:
      ReviewPlanGeneration, ReviewReportGeneration, Enforce, IndexScan,
      WorkingDirectoryOverride, Elaborate, Lint
    
    - Add 13 new Cli_* methods to CliTests.cs covering:
      ResultsFlag, LogFlag, ErrorOutput, InvalidArgs, ExitCode, DefinitionFlag,
      PlanFlag, ReportFlag, EnforceFlag, DirFlag, ElaborateFlag, LintFlag, IndexFlag
    
    - Update docs/reqstream/review-mark/review-mark.yaml: system requirements now
      use only IntegrationTest_* tests and add children: links to subsystem IDs
    
    - Update docs/reqstream/review-mark/cli/cli.yaml: subsystem requirements now
      use only Cli_* tests; add new requirements for all CLI flags
    
    - Update docs/reqstream/review-mark/configuration/configuration.yaml: subsystem
      requirements use only Configuration_* tests
    
    - Update docs/reqstream/review-mark/indexing/indexing.yaml: subsystem requirements
      use only Indexing_* tests
    
    - Update docs/reqstream/review-mark/self-test/self-test.yaml: subsystem requirements
      use only SelfTest_* tests
    
    - Update docs/reqstream/review-mark/configuration/review-mark-configuration.yaml:
      remove Program_* test from ReviewMark-Config-Loading
    
    - Update docs/reqstream/review-mark/indexing/review-index.yaml: remove
      ReviewMarkConfiguration_* tests from ReviewMark-EvidenceSource-None
    
    - Update .reviewmark.yaml: add Purpose review; update ReviewMark-Architecture,
      ReviewMark-Design, ReviewMark-AllRequirements; remove user_guide and
      TestDirectory.cs from ReviewMark-Program review
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * Fix review set spelling: 'review set' -> 'review-set' in test comments
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * fix: address quality findings - fix cspell wdir, add missing system reqs, reorder review-sets
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * fix: dispose context before reading log file in Cli_LogFlag_WritesOutputToFile
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/faf4bc52-4fa8-491e-a753-863b18c1e6a6
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * fix: apply all PR review feedback - temp file leaks, ErrorOutput, depth tests, traceability tests
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/bb1deea5-108a-46ac-9875-9199ef6993e9
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * refactor: simplify GetMethod call and use specific heading text in depth assertions
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/bb1deea5-108a-46ac-9875-9199ef6993e9
    
    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>
    Copilot and Malcolmnixon authored Apr 5, 2026
    Configuration menu
    Copy the full SHA
    6b61889 View commit details
    Browse the repository at this point in the history
  2. fix: formal review fixes — results file directory creation and requir…

    …ements scope cleanup (#47)
    
    * fix: remove self-validation test refs from CLI/Configuration requirements; fix results file directory creation
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/0c41f310-04ab-4dc1-8f1e-c297f19312b5
    
    Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
    
    * refactor: use TestDirectory helper in Validation_Run_WithResultsFileInNewDirectory_CreatesDirectory test
    
    Agent-Logs-Url: https://github.com/demaconsulting/ReviewMark/sessions/4b5d1e5d-5412-46c0-a53c-0dc783a2cf7f
    
    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>
    3 people authored Apr 5, 2026
    Configuration menu
    Copy the full SHA
    4bfdc01 View commit details
    Browse the repository at this point in the history
Loading