Skip to content

Add .NET refactoring skill #873

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
AbhitejJohn merged 57 commits into from
Sep 15, 2026
Merged

Add .NET refactoring skill #873

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
57 commits
Select commit Hold shift + click to select a range
e16a66d
Add csharp-refactoring and dotnet-breaking-changes dotnet skills
AbhitejJohn Jul 7, 2026
2c44302
Add eval tests for csharp-refactoring and dotnet-breaking-changes skills
AbhitejJohn Jul 7, 2026
3bf4a19
minor changes to description to increase calling chance.
webreidi Jul 8, 2026
9433785
Trim csharp-refactoring SKILL.md to lean v3 (proportional rigor)
AbhitejJohn Jul 21, 2026
0df1a51
Trim dotnet-breaking-changes SKILL.md to lean v3 (proportional rigor)
AbhitejJohn Jul 22, 2026
674b386
Drop LangVersion=latest from Billing fixtures for reproducibility
AbhitejJohn Jul 23, 2026
ef9e8d3
Migrate csharp-refactoring and dotnet-breaking-changes evals to the V…
AbhitejJohn Jul 23, 2026
7ae020c
Fix eval grader config: file-contains/file-not-contains use 'value' n…
AbhitejJohn Jul 23, 2026
6fdd84e
Update SKILL.md with C# refactoring guidelines
webreidi Sep 9, 2026
2355d16
Fix typo in source-generation.md regarding partial types
webreidi Sep 9, 2026
3d98ee2
Split dotnet breaking changes into separate branch
webreidi Sep 9, 2026
38c1c93
Refine typical workflow instructions for refactoring
webreidi Sep 9, 2026
7ba34bd
Update regex pattern for output matching in eval.yaml
webreidi Sep 9, 2026
bbce5c7
Update CODEOWNERS and known-domains.txt for improved ownership and do…
webreidi Sep 9, 2026
a7af348
Refactor csharp-refactoring documentation and evaluation tests for cl…
webreidi Sep 10, 2026
9f3d643
Remove VSCode settings file to streamline project configuration
webreidi Sep 10, 2026
6eb6660
Refactor csharp-refactoring skill documentation and evaluation tests …
webreidi Sep 10, 2026
d6a07d5
Add csharp-refactoring and dotnet-breaking-changes dotnet skills
AbhitejJohn Jul 7, 2026
7900815
Add eval tests for csharp-refactoring and dotnet-breaking-changes skills
AbhitejJohn Jul 7, 2026
becad34
minor changes to description to increase calling chance.
webreidi Jul 8, 2026
a3806f7
Trim csharp-refactoring SKILL.md to lean v3 (proportional rigor)
AbhitejJohn Jul 21, 2026
ab9795f
Trim dotnet-breaking-changes SKILL.md to lean v3 (proportional rigor)
AbhitejJohn Jul 22, 2026
003eb76
Drop LangVersion=latest from Billing fixtures for reproducibility
AbhitejJohn Jul 23, 2026
2bb7e36
Migrate csharp-refactoring and dotnet-breaking-changes evals to the V…
AbhitejJohn Jul 23, 2026
671c94f
Fix eval grader config: file-contains/file-not-contains use 'value' n…
AbhitejJohn Jul 23, 2026
f437606
Strengthen refactoring skill evaluations
AbhitejJohn Sep 10, 2026
24d0399
Remove dotnet breaking changes work from refactoring branch
webreidi Sep 10, 2026
a1f0688
Update SKILL.md to refine refactor and feature definitions
webreidi Sep 10, 2026
598a59d
Rename AppSettingsHelper.ParseIntSetting to ParseIntegerSetting
webreidi Sep 10, 2026
6019331
Update Coupons.g.cs.template inclusion in project file
webreidi Sep 10, 2026
61ae0e1
Refactor graders in eval.yaml to improve consolidation checks for Con…
webreidi Sep 10, 2026
db427d3
Update test configuration to check for 'Passed!' output
webreidi Sep 10, 2026
e88d000
Update description in SKILL.md to clarify usage for nullable referenc…
webreidi Sep 10, 2026
6034e9a
Fix stdout_contains syntax in eval.yaml
webreidi Sep 10, 2026
aa64dba
Update eval.yaml to include CollisionExamples.cs
webreidi Sep 10, 2026
0f4fa51
Refactor Coupons class methods for clarity and consistency
webreidi Sep 10, 2026
c9bde3d
Fix formatting of Constants.g.cs
webreidi Sep 10, 2026
5c80132
Fix csharp refactoring eval fixtures
AbhitejJohn Sep 10, 2026
e03bcfb
Harden csharp refactoring eval graders
AbhitejJohn Sep 10, 2026
a9b8c41
Add refactoring dormancy coverage
AbhitejJohn Sep 10, 2026
040d33f
Remove redundant refactoring eval checks
AbhitejJohn Sep 10, 2026
31290d2
Add file-contains check for Shipped.txt
webreidi Sep 10, 2026
c797c40
Refactor ConfigReader body extraction method
webreidi Sep 10, 2026
d450573
Scope ConfigReader grader to its class
AbhitejJohn Sep 10, 2026
d1a55cc
Enhance csharp-refactoring skill documentation and tests
webreidi Sep 11, 2026
31ca050
Refactor operation catalog and update evaluation tests for method ren…
webreidi Sep 11, 2026
72fc029
Update experiment name in evaluation workflow and enhance csharp-refa…
webreidi Sep 11, 2026
7f0b539
Update SKILL.md with mandatory gate clarification
webreidi Sep 11, 2026
63c0d73
Enhance csharp-refactoring skill documentation with new decision guid…
webreidi Sep 11, 2026
2c4ed80
Refine mandatory gate instructions in csharp-refactoring skill docume…
webreidi Sep 14, 2026
d016cf9
Fix compatibility test logic in csharp-refactoring evaluation to excl…
webreidi Sep 14, 2026
11870a3
Revise refactoring and test failure guidelines
webreidi Sep 14, 2026
3c8cab0
Update SKILL.md with clearer refactoring guidelines
webreidi Sep 14, 2026
3b8d365
Harden csharp refactoring evaluation
AbhitejJohn Sep 15, 2026
f152c7a
Preserve refactoring validation guidance after rebase
AbhitejJohn Sep 15, 2026
1f73f15
Fix evaluation workflow expression limit
AbhitejJohn Sep 15, 2026
f4f0ed3
Address refactoring PR review feedback
AbhitejJohn Sep 15, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .github/CODEOWNERS
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@
# dotnet (common everyday C#/.NET)
/plugins/dotnet/skills/setup-local-sdk/ @dotnet/roslyn-ide @dotnet/skills-csharp-language-reviewers
/tests/dotnet/setup-local-sdk/ @dotnet/roslyn-ide @dotnet/skills-csharp-language-reviewers
/plugins/dotnet/skills/csharp-refactoring/ @dotnet/roslyn-ide @dotnet/skills-csharp-language-reviewers
/tests/dotnet/csharp-refactoring/ @dotnet/roslyn-ide @dotnet/skills-csharp-language-reviewers
/plugins/dotnet/lsp.json @dotnet/roslyn-ide @dotnet/skills-csharp-language-reviewers

# dotnet-advanced (advanced .NET development skills)
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/evaluation-workflow-tests.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ on:
- ".github/workflows/evaluation-workflow-tests.yml"
- "eng/evaluation-tools/**"
- "eng/evaluation/test_token_failover.py"
- "eng/evaluation/find-targets.ps1"
- "eng/evaluation/path-safety.ps1"
- "eng/dashboard/**"
- "eng/vally-adapter/**"
Expand All @@ -20,6 +21,7 @@ on:
- ".github/workflows/evaluation-workflow-tests.yml"
- "eng/evaluation-tools/**"
- "eng/evaluation/test_token_failover.py"
- "eng/evaluation/find-targets.ps1"
- "eng/evaluation/path-safety.ps1"
- "eng/dashboard/**"
- "eng/vally-adapter/**"
Expand Down
610 changes: 8 additions & 602 deletions .github/workflows/evaluation.yml
View file
Open in desktop

Large diffs are not rendered by default.

Loading
Loading