Run project agents: repo consistency, code quality, requirements, docs, and code improvements#141
Merged
Malcolmnixon merged 9 commits intomainfrom Feb 19, 2026
Merged
Conversation
- Update .editorconfig with modern C# code style rules - Add end_of_line = lf setting - Add file-scoped namespace enforcement - Add expression-bodied member preferences - Simplify and reorganize settings to match template - Change braces preference to suggestion for gradual adoption - Restructure .cspell.json to match template format - Alphabetically sort and merge word lists - Add missing template words - Preserve SpdxTool-specific terms - Simplify ignorePaths - Enhance .gitignore patterns - Add .idea/ directory for JetBrains Rider - Add comprehensive documentation build artifacts - Add test results patterns - Add VersionMark captures and temporary files - Add SBOM configuration to csproj - Enable SBOM generation - Add Microsoft.Sbom.Targets package - Add InternalsVisibleTo for test project - Add convenience build and lint scripts - Create build.sh and build.bat - Create lint.sh and lint.bat - Support both Unix/Linux and Windows platforms All changes verified with successful build and test execution. No breaking changes introduced.
- Add declaredat (SPDX metadata field) - Add hotspots (security terminology) - Add sandboxed (security terminology)
- Add UTF-8 BOM to files missing it - Add final newlines to all source files - Fix whitespace formatting issues - Change charset to utf-8-bom to match existing code style All files now pass dotnet format --verify-no-changes
…atterns Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
…alidate commands Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
…atterns Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Run repo consistency agent and improve other agents
Run project agents: repo consistency, code quality, requirements, docs, and code improvements
Feb 19, 2026
Malcolmnixon
requested changes
Feb 19, 2026
…ions; remove agent report files from tracking Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
Malcolmnixon
approved these changes
Feb 19, 2026
This was referenced Feb 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request
Description
Ran all project agents against the repository to align with TemplateDotNetTool patterns and apply cross-cutting improvements.
Repo Consistency
.editorconfig: Simplified and aligned with template (file-scoped namespaces, expression-bodied members, CA diagnostic suppressions, enforced braces as warning).cspell.json: Merged and alphabetized word list.gitignore: Added JetBrains, doc build artifacts, test results, temp file patternscsproj: Added SBOM generation viaMicrosoft.Sbom.Targets,InternalsVisibleTofor test projectbuild.sh/build.batandlint.sh/lint.batscriptsBrace Enforcement
csharp_prefer_bracesfromtrue:suggestiontotrue:warningin.editorconfigif/else/for/foreach/whilestatements across the entire codebase to resolve all IDE0011 violationsSelf-Validation Coverage (56% → 81%)
Four new self-validation tests covering previously untested commands:
ValidateDiagram.cs— Mermaid diagram generationValidateHash.cs— SHA256 hash generation/verificationValidateToMarkdown.cs— Markdown summary generationValidateBasic.cs— Basic SPDX document validationDocumentation
CONTRIBUTING.md,SECURITY.md,docs/guide/guide.md,docs/spdx-tool-and-sbom-tool.md,docs/spdx-tool-command-line.mddeclaredat→declared atindocs/spdx-tool-workflow-files.mdType of Change
Related Issues
Pre-Submission Checklist
Before submitting this pull request, ensure you have completed the following:
Build and Test
dotnet build --configuration Releasedotnet test --configuration ReleaseCode Quality
dotnet format --verify-no-changesQuality Checks
Please run the following checks before submitting:
cspell "**/*.{md,cs}"markdownlint "**/*.md"yamllint '**/*.{yml,yaml}'Testing
Documentation
Additional Notes
Microsoft.Sbom.Targets 4.1.5checked against GitHub Advisory DB — no known vulnerabilitiesAGENT_REPORT_*.md) are excluded via.gitignoreand not tracked in the repository🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.