Skip to content

Add CLI tool for generating DSL sample configuration#2

Merged
Pressacco merged 1 commit into
mainfrom
codex/add-command-to-generate-sample-dsl-xml
Nov 8, 2025
Merged

Add CLI tool for generating DSL sample configuration#2
Pressacco merged 1 commit into
mainfrom
codex/add-command-to-generate-sample-dsl-xml

Conversation

@Pressacco

Copy link
Copy Markdown
Member

Summary

  • factor the DSL sample XML into a shared helper so it can be reused
  • add a console project that exposes a generate-dsl command for emitting the sample file
  • document the new utility and wire it into the solution

Testing

  • unable to run dotnet build (dotnet CLI not installed in environment)

Codex Task

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR refactors DSL default constants into a shared static class and introduces a command-line tool for generating DSL configuration files. The analyzer constants (DefaultDslFileName and DefaultDslXml) are extracted from DslTerminologyAnalyzer into a new DslDefaults class, which is then reused by both the analyzer and a newly created console application.

Key changes:

  • Extracted DSL defaults to a shared DslDefaults class for reusability
  • Added BlueDotBrigade.Analyzers.Tool console application with a generate-dsl command
  • Updated documentation with usage examples for the command-line tool

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
DslDefaults.cs New shared class containing default DSL filename and XML content constants
DslTerminologyAnalyzer.cs Refactored to use DslDefaults instead of local constants
Program.cs New command-line tool for generating DSL configuration files
BlueDotBrigade.Analyzers.Tool.csproj Project file for the new tool, linking to DslDefaults.cs
README.md Added documentation for the command-line helper tool
Analyzers_v1.sln Added new tool project with build configurations

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

Comment thread Analyzers_v1.sln
@Pressacco Pressacco merged commit 8261147 into main Nov 8, 2025
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants