Skip to content

Align repository with TemplateDotNetTool standards#52

Merged
Malcolmnixon merged 4 commits intomainfrom
copilot/fix-repo-consistency-issues
Feb 11, 2026
Merged

Align repository with TemplateDotNetTool standards#52
Malcolmnixon merged 4 commits intomainfrom
copilot/fix-repo-consistency-issues

Conversation

Copy link
Contributor

Copilot AI commented Feb 11, 2026

Pull Request

Description

Repo-consistency-agent identified drift from TemplateDotNetTool template patterns. This PR brings configuration files and documentation into alignment with template standards while preserving SarifMark-specific customizations.

Configuration Updates

  • .gitignore: Replaced with comprehensive template version (39→105 lines) covering IDE-specific files, build artifacts, and Python cache patterns
  • .markdownlint: Migrated from .markdownlint.json to .markdownlint-cli2.jsonc with structured config/ignores sections
  • .cspell.json: Added common DEMA project words (CodeQL, mermaid, slnx) and AGENT_REPORT_*.md to ignorePaths
  • lint.sh & lint.bat: Removed redundant node_modules exclusion (now handled by markdownlint config)

Documentation Updates

  • AGENTS.md: Added "Available Specialized Agents" section for discoverability
  • .github/agents/technical-writer.md: Documented inline link exception for AI agent markdown files

Critical Fix

All ignore configurations now properly exclude agent-generated reports (AGENT_REPORT_*.md) from version control, preventing accidental commits of temporary analysis files.

Type of Change

  • Code quality improvement
  • Documentation update

Pre-Submission Checklist

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

  • 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

Changes are configuration and documentation only - no code modifications. All 68 tests pass, build produces zero warnings, and all linters pass. Both Unix (lint.sh) and Windows (lint.bat) linting scripts are now consistent.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits February 11, 2026 13:09
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] Review repository consistency and fix findings Align repository with TemplateDotNetTool standards Feb 11, 2026
Copilot AI requested a review from Malcolmnixon February 11, 2026 13:12
@Malcolmnixon
Copy link
Contributor

@copilot You updated lint.sh but not lint.bat

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

Copilot AI commented Feb 11, 2026

@copilot You updated lint.sh but not lint.bat

Fixed in commit 34a4134. Removed the redundant "#node_modules" parameter from lint.bat to match the lint.sh changes.

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