Fix rendering of known issues in build output sections

[Copilot]

Pull Request

Description

Known issues were being routed through the same section rules as commit-linked items, causing open bugs to appear under sections like ## Bugs Fixed even when no bugs were resolved in the commit range.

Root cause: All 3 repo connectors (GitHub, AzureDevOps, Mock) concatenated knownIssues into allItems before calling ApplyRules(), making known issues indistinguishable from commit-linked work items during routing.

Fix:

• Removed .Concat(knownIssues) from the items passed to ApplyRules() in all 3 connectors — known issues now bypass the routing pipeline entirely
• When --include-known-issues is passed, a dedicated ## Known Issues section is appended after all routed sections, so known issues are never visually confused with items that were actually resolved in the build

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

Pre-Submission Checklist

Before submitting this pull request, ensure you have completed the following:

Build and Test

• Code builds successfully and all tests pass: pwsh ./build.ps1
• Code produces zero warnings

Code Quality

• New code has appropriate XML documentation comments
• Static analyzer warnings have been addressed

Quality Checks

Please run the following checks before submitting:

• All linters pass: pwsh ./lint.ps1

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 docs/ documentation (if applicable)
• Added code examples for new features (if applicable)
• Updated requirements.yaml (if applicable)

Additional Notes

Changes include:

• 3 connector fixes (GitHub, AzureDevOps, Mock): removed known issues from ApplyRules() input so they are excluded from the routing pipeline entirely
• BuildInformation.cs: AppendKnownIssuesSection() is now called after all routed sections when --include-known-issues is active, rendering known issues in a clearly separate ## Known Issues section
• 5 new tests: 4 in BuildInformationTests.cs covering routed + known-issues combinations; 1 in MockRepoConnectorTests.cs verifying no known issue ID leaks into routed sections
• Docs: new BuildMark-BuildInformation-RoutedSections requirement, updated design doc, refreshed verification scenarios

All 545 tests pass.

[Malcolmnixon]

@copilot The implementation agent froze for 30 minutes without doing anything. Please try implementing this fix again.

[Malcolmnixon]

@copilot Please continue

[Copilot]

@copilot Please continue

The fix was already completed in the previous session (commits 22b6ab4–70ae2d7). All 545 tests pass and the build is clean.

Summary of what was implemented:

• Known issues are now excluded from the ApplyRules() routing pipeline in all 3 connectors (GitHub, AzureDevOps, Mock)
• When --include-known-issues is passed, known issues appear in a dedicated ## Known Issues section after all routed sections — never mixed into sections like ## Bugs Fixed
• 5 new tests added; requirements, design, and verification docs updated

[Copilot]

Pull request overview

Fixes misrouting of “known issues” into commit-linked routed sections (e.g., showing open bugs under “Bugs Fixed”) by ensuring known issues bypass the routing pipeline and are rendered in a dedicated section when requested.

Changes:

• Excludes knownIssues from the item set passed into ApplyRules() across GitHub, Azure DevOps, and Mock connectors.
• Updates BuildInformation.ToMarkdown() so the ## Known Issues section (when enabled) is appended after routed sections.
• Adds/updates unit tests and requirement/verification docs to cover routed-sections + known-issues combinations.

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
test/DemaConsulting.BuildMark.Tests/RepoConnectors/Mock/MockRepoConnectorTests.cs	Adds a regression test ensuring known issues do not appear in any routed section output.
test/DemaConsulting.BuildMark.Tests/BuildNotes/BuildInformationTests.cs	Adds tests verifying Known Issues rendering behavior in routed mode (flag on/off, ordering).
src/DemaConsulting.BuildMark/RepoConnectors/Mock/MockRepoConnector.cs	Stops including known issues in routing input; documents the separation.
src/DemaConsulting.BuildMark/RepoConnectors/GitHub/GitHubRepoConnector.cs	Stops including known issues in routing input; documents the separation.
src/DemaConsulting.BuildMark/RepoConnectors/AzureDevOps/AzureDevOpsRepoConnector.cs	Stops including known issues in routing input; documents the separation.
src/DemaConsulting.BuildMark/BuildNotes/BuildInformation.cs	Appends Known Issues after routed sections when includeKnownIssues is enabled.
docs/verification/build-mark/repo-connectors/mock/mock-repo-connector.md	Updates verification scenarios/count to include the new routing/known-issues test.
docs/verification/build-mark/build-notes/build-information.md	Adds verification scenarios for Known Issues rendering in routed mode.
docs/reqstream/build-mark/repo-connectors/mock/mock-repo-connector.yaml	Adds the new Mock connector test to requirement traceability.
docs/reqstream/build-mark/build-notes/build-information.yaml	Adds the new BuildInformation tests to requirement traceability.
docs/design/build-mark/build-notes/build-information.md	Updates design documentation to reflect that known issues are excluded from routing and appended after routed sections.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.