From 2b40e5094273fe64a7c624e863333ddae9d03520 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 3 Apr 2026 00:25:46 +0000 Subject: [PATCH 1/3] feat: comply with updated standards for design docs, reqstream, and reviewmark Agent-Logs-Url: https://github.com/demaconsulting/SarifMark/sessions/b51e81ae-3c8e-484e-8e6a-3236e2ef586c Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> --- .reviewmark.yaml | 28 +++++++++++++++---- docs/design/system.md | 2 +- docs/reqstream/{ => ots}/ots-software.yaml | 0 .../{sarifmark-system.yaml => system.yaml} | 0 requirements.yaml | 4 +-- 5 files changed, 25 insertions(+), 9 deletions(-) rename docs/reqstream/{ => ots}/ots-software.yaml (100%) rename docs/reqstream/{sarifmark-system.yaml => system.yaml} (100%) diff --git a/.reviewmark.yaml b/.reviewmark.yaml index 196c965..33128a5 100644 --- a/.reviewmark.yaml +++ b/.reviewmark.yaml @@ -31,15 +31,10 @@ reviews: - id: SarifMark-System title: Review of SarifMark System paths: - - "docs/reqstream/sarifmark-system.yaml" + - "docs/reqstream/system.yaml" - "docs/reqstream/platform-requirements.yaml" - - "docs/reqstream/ots-software.yaml" - "docs/design/introduction.md" - "docs/design/system.md" - - "docs/reqstream/unit-program.yaml" - - "docs/design/program.md" - - "src/**/Program.cs" - - "test/**/ProgramTests.cs" - "test/**/IntegrationTests.cs" - "test/**/Runner.cs" - "test/**/AssemblyInfo.cs" @@ -54,6 +49,17 @@ reviews: - "requirements.yaml" - "docs/reqstream/**/*.yaml" + # --------------------------------------------------------------------------- + # Design Review + # --------------------------------------------------------------------------- + + - id: SarifMark-Design + title: Review of SarifMark Design Documentation + paths: + - "docs/reqstream/system.yaml" + - "docs/reqstream/platform-requirements.yaml" + - "docs/design/**/*.md" + # --------------------------------------------------------------------------- # Subsystem Reviews # --------------------------------------------------------------------------- @@ -150,3 +156,13 @@ reviews: - "docs/design/utilities/path-helpers.md" - "src/**/Utilities/PathHelpers.cs" - "test/**/Utilities/PathHelpersTests.cs" + + # --------------------------------------------------------------------------- + # OTS Review + # --------------------------------------------------------------------------- + + - id: SarifMark-OTS + title: Review of SarifMark OTS Software + paths: + - "docs/reqstream/ots/ots-software.yaml" + - "test/**/IntegrationTests.cs" diff --git a/docs/design/system.md b/docs/design/system.md index 16c8286..8eb1f19 100644 --- a/docs/design/system.md +++ b/docs/design/system.md @@ -82,7 +82,7 @@ subsystem is a stateless helper used by `SelfTest` for path construction. ## System Requirements -System-level requirements are captured in `docs/reqstream/sarifmark-system.yaml` +System-level requirements are captured in `docs/reqstream/system.yaml` and are validated through integration tests that exercise the published dotnet DLL end-to-end across the supported platforms. diff --git a/docs/reqstream/ots-software.yaml b/docs/reqstream/ots/ots-software.yaml similarity index 100% rename from docs/reqstream/ots-software.yaml rename to docs/reqstream/ots/ots-software.yaml diff --git a/docs/reqstream/sarifmark-system.yaml b/docs/reqstream/system.yaml similarity index 100% rename from docs/reqstream/sarifmark-system.yaml rename to docs/reqstream/system.yaml diff --git a/requirements.yaml b/requirements.yaml index 2e25bbb..3855d11 100644 --- a/requirements.yaml +++ b/requirements.yaml @@ -13,7 +13,7 @@ # --- includes: - - docs/reqstream/sarifmark-system.yaml + - docs/reqstream/system.yaml - docs/reqstream/cli/subsystem-cli.yaml - docs/reqstream/cli/unit-context.yaml - docs/reqstream/sarif/subsystem-sarif.yaml @@ -25,5 +25,5 @@ includes: - docs/reqstream/utilities/subsystem-utilities.yaml - docs/reqstream/utilities/unit-path-helpers.yaml - docs/reqstream/platform-requirements.yaml - - docs/reqstream/ots-software.yaml + - docs/reqstream/ots/ots-software.yaml - docs/reqstream/unit-program.yaml From 591bfa36ffd5a452630b5ef90573c342c5d59de8 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 3 Apr 2026 00:29:49 +0000 Subject: [PATCH 2/3] feat: remove SarifMark-OTS review set from reviewmark config Agent-Logs-Url: https://github.com/demaconsulting/SarifMark/sessions/91065279-41b9-415a-8317-81fb74bf0b57 Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> --- .reviewmark.yaml | 9 --------- 1 file changed, 9 deletions(-) diff --git a/.reviewmark.yaml b/.reviewmark.yaml index 33128a5..b200d65 100644 --- a/.reviewmark.yaml +++ b/.reviewmark.yaml @@ -157,12 +157,3 @@ reviews: - "src/**/Utilities/PathHelpers.cs" - "test/**/Utilities/PathHelpersTests.cs" - # --------------------------------------------------------------------------- - # OTS Review - # --------------------------------------------------------------------------- - - - id: SarifMark-OTS - title: Review of SarifMark OTS Software - paths: - - "docs/reqstream/ots/ots-software.yaml" - - "test/**/IntegrationTests.cs" From f9a06cf00a96d482c58b9b49d2a56d80e7c9d2f4 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 3 Apr 2026 00:48:03 +0000 Subject: [PATCH 3/3] fix: remove trailing blank line in .reviewmark.yaml to pass yamllint Agent-Logs-Url: https://github.com/demaconsulting/SarifMark/sessions/5f7456a2-2f58-4358-a249-14aaf026aac1 Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com> --- .reviewmark.yaml | 1 - 1 file changed, 1 deletion(-) diff --git a/.reviewmark.yaml b/.reviewmark.yaml index b200d65..7c243cd 100644 --- a/.reviewmark.yaml +++ b/.reviewmark.yaml @@ -156,4 +156,3 @@ reviews: - "docs/design/utilities/path-helpers.md" - "src/**/Utilities/PathHelpers.cs" - "test/**/Utilities/PathHelpersTests.cs" -