Skip to content

Bootstrap BuildMark repository with standard Dema Consulting project structure#1

Merged
Malcolmnixon merged 4 commits intomainfrom
copilot/setup-standard-dema-pattern
Jan 29, 2026
Merged

Bootstrap BuildMark repository with standard Dema Consulting project structure#1
Malcolmnixon merged 4 commits intomainfrom
copilot/setup-standard-dema-pattern

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 29, 2026

Pull Request

Description

Establishes BuildMark repository following the SarifMark pattern: multi-target .NET tool (.NET 8/9/10, C# 12), comprehensive CI/CD pipelines, linting infrastructure, and GitHub templates.

Key Components:

  • Project structure: Console application with --version and --help commands, MSTest project, solution file
  • Linting: cspell, markdownlint, yamllint, editorconfig with file-scoped namespaces and naming conventions
  • CI/CD: Reusable build workflow (quality checks, multi-OS builds, CodeQL, integration tests), push trigger, release workflow with NuGet publish
  • GitHub templates: Issue forms (bug/feature), PR checklist, custom agent specifications (documentation-writer, project-maintainer, software-quality-enforcer)
  • Quality: TreatWarningsAsErrors, Microsoft.CodeAnalysis.NetAnalyzers, SonarAnalyzer.CSharp, SBOM generation
  • License: MIT

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 - Initial repository setup

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

CodeQL security analysis: 0 alerts. All three target frameworks (.NET 8, 9, 10) build and test successfully.

Original prompt

Follow the standard Dema Consulting pattern from https://github.com/demaconsulting/SarifMark as closely as possible (renaming where appropriate) with:

  • DotNet 8/9/10 using C#12
  • cspell
  • editorconfig, markdownlint, yamllint
  • Icon.png
  • src folder with application (just hello world for now)
  • test folder with mstest project (basic for now)
  • Bug/Feature issue templates
  • Pull request template
  • Standard agent files
  • Build pipelines (build.yaml, build_on_push.yaml, release.yaml)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 3 commits January 29, 2026 13:40
…Hub templates

Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copilot AI changed the title [WIP] Add standard Dema Consulting pattern to project Bootstrap BuildMark repository with standard Dema Consulting project structure Jan 29, 2026
Copilot AI requested a review from Malcolmnixon January 29, 2026 13:51
@Malcolmnixon Malcolmnixon marked this pull request as ready for review January 29, 2026 13:58
@Malcolmnixon Malcolmnixon merged commit 354353d into main Jan 29, 2026
10 checks passed
@Malcolmnixon Malcolmnixon deleted the copilot/setup-standard-dema-pattern branch January 29, 2026 16:14
Copilot AI mentioned this pull request Feb 6, 2026
21 tasks
Copilot AI added a commit that referenced this pull request Feb 6, 2026
- Add BuildMark_MarkdownReportGeneration test
- Add BuildMark_GitIntegration test
- Add BuildMark_IssueTracking test
- Add BuildMark_KnownIssuesReporting test
- Tests check for GitHub token and skip gracefully if not available
- These tests satisfy requirements GH-1, GH-2, GH-3, RPT-001-009, PLT-001-005
- Tests run BuildMark on itself for self-validation
Malcolmnixon added a commit that referenced this pull request Feb 7, 2026
* Initial plan

* Add self-validation integration tests for requirements traceability

- Add BuildMark_MarkdownReportGeneration test
- Add BuildMark_GitIntegration test
- Add BuildMark_IssueTracking test
- Add BuildMark_KnownIssuesReporting test
- Tests check for GitHub token and skip gracefully if not available
- These tests satisfy requirements GH-1, GH-2, GH-3, RPT-001-009, PLT-001-005
- Tests run BuildMark on itself for self-validation

* Refactor: Extract GitHub token check to helper method

- Add SkipIfNoGitHubToken() helper to reduce code duplication
- All four self-validation tests now use the shared helper
- Addresses code review feedback

* docs: Fix spell checking issues

- Add 'buildnotes' to cspell dictionary for docs/buildnotes/definition.yaml
- Add 'myterm' as valid example term used in CONTRIBUTING.md
- Add test-results directory to cspell ignore paths to exclude generated files

* feat: Add internal constructor to GitHubGraphQLClient for testability

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

* test: Add unit tests for GitHubGraphQLClient with mocked HttpClient

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

* refactor: Remove conditional skip logic from BuildMark self-validation tests

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

* Remove duplicate BuildMark_* tests from IntegrationTests.cs

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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants