V0.9.0/dotnet remote testing - #36
Merged
Merged
Conversation
New skill provides deterministic remote testing of .NET projects in Docker containers using Microsoft's official SDK images. Supports testenvironments.json configuration or zero-config discovery from official release metadata. Orchestrates container setup, NuGet cache management, test execution, and result parsing transparently behind a single reusable runner script.
Introduces run-skill-benchmark.ps1 as the preferred local entry point for skill benchmarking. Manages a single persistent temp workspace, shares benchmark-scoped caches, stages fixtures once, enforces bounded parallelism and per-run timeouts, and prewarms expensive resolver work. Still delegates aggregation and static review generation to the installed Anthropic skill-creator copy.
Resolves NuGet package versions deterministically and now adds two new capabilities: SHA256-based caching that reuses previous resolution results within a benchmark scope, and structured tracing that logs role, frameworks, packages, candidate limits, cache hit status, and execution duration. Caching is controlled by DOTNET_TEST_RESOLVER_CACHE_DIR; tracing by DOTNET_TEST_RESOLVER_TRACE_FILE; maximum candidates by DOTNET_TEST_MAXIMUM_CANDIDATES. Test harness enhanced to verify cache behavior and validate timing instrumentation.
Update AGENTS.md with guidance to prefer the repo-owned run-skill-benchmark.ps1 runner for local skill benchmarking. Update CONTRIBUTING.md with entry point documentation and examples. Update README.md to add dotnet-remote-testing to the skills list, document the benchmark runner as the preferred local workflow, and explain structured result parsing and failure classification.
Greptile SummaryThe PR adds the
Confidence Score: 5/5The PR appears safe to merge because no blocking failure remains from the previously reported issues. No blocking failure remains. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart LR
A[Skill eval definitions] --> B[Benchmark runner]
B --> C[Temporary workspace]
B --> D[Paired candidate and baseline runs]
D --> E[Grading]
E --> F[benchmark.json]
F --> G[Static review viewer]
H[Remote testing request] --> I[Environment discovery]
I --> J[Docker image resolution]
J --> K[Isolated restore, build, and test]
K --> L[Structured results and cleanup]
Reviews (3): Last reviewed commit: "updated changelog" | Re-trigger Greptile |
The remote test runner can now discover and use cached Microsoft release metadata when the network is unavailable or when offline mode is explicitly requested. Release metadata is cached outside the repo for offline reuse, and the form now exposes the computed project target as a recommended selectable option alongside a custom path entry. A new eval case validates offline discovery with a cached releases-index.cache.json fixture.
The benchmark runner now generates a POSIX shell script shim alongside the Windows batch file shim to support cross-platform dotnet execution. The Unix shim is marked executable on non-Windows platforms, and the test harness validates both the script's delegation to the logging wrapper and its executable bit on Unix systems.
Updated the dotnet-remote-testing feature description in README.md to highlight offline-safe discovery: successful release metadata is cached outside the repo for offline reuse, and the form now exposes the exact runner-computed target as a recommended option.
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.
This pull request updates documentation to clarify and standardize the workflow for running skill evaluation benchmarks, emphasizing the use of the repo-owned PowerShell runner script. The changes improve instructions for both new and existing contributors, ensuring consistency and reliability in skill benchmarking and review.
Standardization of skill benchmarking workflow:
scripts/run-skill-benchmark.ps1PowerShell script, which manages temp workspaces, caches, parallelism, timeouts, and artifact generation, while delegating aggregation and review to Anthropic's installed tools. [1] [2] [3]README.mdandCONTRIBUTING.md, including details on the artifacts produced and the review flow.Documentation improvements:
dotnet-remote-testingskill vianpx skills addin the skill installation section.