Skip to content

Commit a7185ae

Browse files
committed
Align skills with Claude skill guide
1 parent 7436da4 commit a7185ae

File tree

26 files changed

+69
-69
lines changed

26 files changed

+69
-69
lines changed

skills/mcaf-adr-writing/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ compatibility: "Requires repository write access; produces Markdown ADRs with Me
5151

5252
## Deliver
5353

54-
- `docs/ADR/ADR-XXXX-<short-title>.md`
54+
- `docs/ADR/ADR-XXXX-short-title.md`
5555
- linked updates to architecture docs when the decision changes boundaries
5656

5757
## Validate

skills/mcaf-dotnet-analyzer-config/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ If analyzer configuration is requested but not structured yet:
5858
2. Create or normalize one repo-root `.editorconfig` with `root = true`.
5959
3. Move rule severity into `.editorconfig` and keep bulk analyzer switches in project or MSBuild config.
6060
4. Add nested `.editorconfig` files only when a subtree really needs different scoped policy.
61-
5. Run `dotnet build <solution-or-project>` and return `status: configured` or `status: improved`.
61+
5. Run `dotnet build SOLUTION_OR_PROJECT` and return `status: configured` or `status: improved`.
6262
6. If the repo intentionally uses another documented analyzer-config ownership model, return `status: not_applicable`.
6363

6464
## Deliver

skills/mcaf-dotnet-archunitnet/SKILL.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,11 +46,11 @@ If `ArchUnitNET` is not configured yet:
4646
1. Detect existing setup:
4747
- `rg -n "TngTech\\.ArchUnitNET" -g '*.csproj' .`
4848
2. Add packages to the architecture test project:
49-
- `dotnet add <test-project>.csproj package TngTech.ArchUnitNET`
49+
- `dotnet add TEST_PROJECT.csproj package TngTech.ArchUnitNET`
5050
- add one framework bridge package: `TngTech.ArchUnitNET.xUnit`, `TngTech.ArchUnitNET.xUnitV3`, `TngTech.ArchUnitNET.MSTestV2`, or `TngTech.ArchUnitNET.TUnit`
5151
3. Add at least one durable boundary rule test.
5252
4. Wire architecture tests into the standard `test` command in `AGENTS.md` and CI.
53-
5. Run `dotnet test <test-project>.csproj` and return `status: configured` or `status: improved`.
53+
5. Run `dotnet test TEST_PROJECT.csproj` and return `status: configured` or `status: improved`.
5454
6. If `NetArchTest` already covers the same boundary policy and no gap exists, return `status: not_applicable`.
5555

5656

skills/mcaf-dotnet-cloc/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ compatibility: "Requires a repository with .NET source files or a Git checkout;
4141
1. Choose the counting mode deliberately:
4242
- `--vcs=git` for repo-respecting counts
4343
- path-based counting for bounded folders
44-
- `--git --diff <base> <head>` for change deltas
44+
- `--git --diff BASE HEAD` for change deltas
4545
2. Prefer `.NET`-relevant views first:
4646
- C# footprint
4747
- test versus production footprint

skills/mcaf-dotnet-code-analysis/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ If first-party .NET code analysis is requested but not configured yet:
6161
- `AnalysisMode` when needed
6262
- warning policy such as `TreatWarningsAsErrors`
6363
4. Keep rule-level severity in the repo-root `.editorconfig`.
64-
5. Run `dotnet build <solution-or-project>` and return `status: configured` or `status: improved`.
64+
5. Run `dotnet build SOLUTION_OR_PROJECT` and return `status: configured` or `status: improved`.
6565
6. If the repo intentionally defers analyzer policy to another documented build layer, return `status: not_applicable`.
6666

6767
## Deliver

skills/mcaf-dotnet-complexity/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ If complexity thresholds are not configured yet:
5757
2. Add severity entries for `CA1502`, `CA1505`, `CA1506`, and `CA1501` in root `.editorconfig`.
5858
3. Add checked-in `CodeMetricsConfig.txt` and include it as `AdditionalFiles` in project props.
5959
4. Keep maintainability limits aligned with `AGENTS.md`.
60-
5. Run `dotnet build <solution-or-project>` and return `status: configured` or `status: improved`.
60+
5. Run `dotnet build SOLUTION_OR_PROJECT` and return `status: configured` or `status: improved`.
6161
6. If policy relies only on `AGENTS.md` limits with no analyzer gate by design, return `status: not_applicable`.
6262

6363

skills/mcaf-dotnet-coverlet/SKILL.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -55,13 +55,13 @@ If coverage is not configured yet:
5555
- `dotnet tool list --global`
5656
- `command -v coverlet`
5757
2. Install exactly one driver path:
58-
- VSTest collector: `dotnet add <test-project>.csproj package coverlet.collector`
59-
- MSBuild driver: `dotnet add <test-project>.csproj package coverlet.msbuild`
58+
- VSTest collector: `dotnet add TEST_PROJECT.csproj package coverlet.collector`
59+
- MSBuild driver: `dotnet add TEST_PROJECT.csproj package coverlet.msbuild`
6060
- Console tool: `dotnet new tool-manifest` (if missing) and `dotnet tool install coverlet.console`
6161
3. Record one concrete local and CI command in `AGENTS.md`:
62-
- collector: `dotnet test <test-project>.csproj --collect:"XPlat Code Coverage"`
63-
- msbuild: `dotnet test <test-project>.csproj /p:CollectCoverage=true /p:CoverletOutputFormat=cobertura`
64-
- console: `dotnet tool run coverlet <test-assembly.dll> --target "dotnet" --targetargs "test <test-project>.csproj --no-build"`
62+
- collector: `dotnet test TEST_PROJECT.csproj --collect:"XPlat Code Coverage"`
63+
- msbuild: `dotnet test TEST_PROJECT.csproj /p:CollectCoverage=true /p:CoverletOutputFormat=cobertura`
64+
- console: `dotnet tool run coverlet TEST_ASSEMBLY.dll --target "dotnet" --targetargs "test TEST_PROJECT.csproj --no-build"`
6565
4. Run the chosen command once and return `status: configured` or `status: improved`.
6666
5. If another coverage engine already owns coverage for this repo, return `status: not_applicable`.
6767

skills/mcaf-dotnet-features/SKILL.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,11 +59,11 @@ If the requested C# feature depends on SDK or language support the repo does not
5959

6060
1. Detect current state:
6161
- `dotnet --list-sdks`
62-
- `rg -n "<TargetFramework>|<LangVersion>|<TargetFrameworks>" -g '*.csproj' -g 'Directory.Build.*' .`
62+
- `rg -n "TargetFramework|LangVersion|TargetFrameworks" -g '*.csproj' -g 'Directory.Build.*' .`
6363
2. Confirm whether the repo wants to stay on the current stable language level or intentionally upgrade.
6464
3. If the feature requires a newer supported SDK or target framework, upgrade the repo toolchain deliberately instead of relying on the local machine by accident.
6565
4. If the repo needs explicit `LangVersion`, record it in project or shared MSBuild config.
66-
5. Run `dotnet build <solution-or-project>` after the feature or toolchain change and return `status: configured` or `status: improved`.
66+
5. Run `dotnet build SOLUTION_OR_PROJECT` after the feature or toolchain change and return `status: configured` or `status: improved`.
6767
6. If the repo intentionally stays below the required language level, return `status: not_applicable`.
6868

6969
## Deliver

skills/mcaf-dotnet-features/references/csharp-modern-features.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,13 @@ Use this reference when modernizing C# code without breaking the repo's target f
88
- `.NET 9` maps to `C# 13`
99
- `.NET 10` maps to `C# 14`
1010
- newer language versions than the target framework supports are not supported
11-
- do not set `<LangVersion>latest</LangVersion>` because it makes builds machine-dependent
12-
- use `<LangVersion>preview</LangVersion>` only when the repo intentionally opts into preview
11+
- do not set `LangVersion=latest` because it makes builds machine-dependent
12+
- use `LangVersion=preview` only when the repo intentionally opts into preview
1313

1414
Useful checks:
1515

1616
```bash
17-
rg -n "<TargetFramework|<TargetFrameworks|<LangVersion" -g '*.csproj' -g 'Directory.Build.*' .
17+
rg -n "TargetFramework|TargetFrameworks|LangVersion" -g '*.csproj' -g 'Directory.Build.*' .
1818
```
1919

2020
If the repo's current language version is unclear, the compiler can reveal it:

skills/mcaf-dotnet-format/SKILL.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ compatibility: "Requires a .NET SDK-based repository; respects the repo's `AGENT
3838
## Workflow
3939

4040
1. Prefer the SDK-provided `dotnet format` command instead of inventing custom format scripts.
41-
2. Start with verify mode in CI: `dotnet format <target> --verify-no-changes`.
41+
2. Start with verify mode in CI: `dotnet format TARGET --verify-no-changes`.
4242
3. Use narrower subcommands only when the repo needs them:
4343
- `whitespace`
4444
- `style`
@@ -56,7 +56,7 @@ If `dotnet format` is requested but not available yet:
5656
2. Treat `dotnet format` as SDK-provided, not as a separate repo-local tool by default.
5757
3. If the command is missing, install or upgrade to a supported .NET SDK, then recheck `dotnet format --version`.
5858
4. Add explicit local and CI commands to `AGENTS.md`, usually:
59-
- `dotnet format <target> --verify-no-changes`
59+
- `dotnet format TARGET --verify-no-changes`
6060
5. Run the chosen command once and return `status: configured` or `status: improved`.
6161
6. If the repo intentionally uses only `CSharpier` for formatting ownership, return `status: not_applicable`.
6262

0 commit comments

Comments
 (0)