Commit Graph

178 Commits

Author SHA1 Message Date
Viktor Hofer 7c8cc3e4ef Add copy_test_files support and clean up test metadata (#69)
* Add copy_test_files support and clean up test metadata

- Add copy_test_files option to eval.yaml setup schema for opt-in
  auto-copy of test fixture files into agent working directory
- Update all msbuild eval.yaml files to enable copy_test_files
- Remove unused README.md, expected-output.md, and eval-test-prompt.txt
  from test directories
- Document setup options in README.md

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Fix copy_test_files to place files at workDir root

Use readdir + per-entry cp instead of cp(dir, dir) which would
nest files under a subdirectory. Addresses PR review feedback.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Reduce parallel skill count to 3 due to quota issues

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2026-02-20 18:10:34 +01:00
Viktor Hofer 0d700bdfb8 Add parallel runs support to skill-validator (#68)
* Add parallel runs support to skill-validator

- Add --parallel-runs flag (default: 8) to control max concurrent runs
- Refactor run loop to use p-limit for controlled parallelism
- All runs within a scenario now execute concurrently (up to limit)
- Significantly reduces validation time with multiple runs

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Add parallel runs support to skill-validator

- Add --parallel-runs flag (default: 1) to control max concurrent runs
- Refactor run loop to use p-limit for controlled parallelism
- All runs within a scenario now execute concurrently (up to limit)
- Add workflow input and env variable (default: 3) for CI
- Add input validation for runs and parallelRuns (clamp to >= 1)
- Show spinner progress for both sequential and parallel modes
- Remove unused imports and variables

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Improve description

* Add DEFAULT_RUNS env variable to workflow

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Add parallel skills, coordinated spinner, and allSettled

- Add --parallel-skills flag to evaluate multiple skills concurrently
- Use single coordinated spinner shared across all parallel skills
- Prefix output with [skill] or [skill/scenario] for disambiguation
  (scenario omitted when eval has only one scenario)
- Use Promise.allSettled so one skill failure doesn't discard others
- Per-scenario runLimit so parallel-runs is scoped per scenario
- Add workflow input and env variable for parallel-skills (default: 3)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Temporarily use Token 2

* Allow skill-validator step to fail without blocking the job

Results are still uploaded and published to the dashboard even when
skills fail validation.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Remove test changes

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2026-02-20 17:34:32 +01:00
Viktor Hofer fa4d72352e Add judge model input to evaluation workflow (#67) 2026-02-20 14:41:51 +01:00
Viktor Hofer 0b85b5c33d Only evaluate changed components in PRs (#66)
* Only evaluate changed components in PRs

Update evaluation workflow to detect which components have changed
in pull requests and only run evaluations for those, similar to
how validate-skills and validate-agents workflows work.

- Add change detection using git diff for PRs
- Add has_components output to skip evaluation when nothing changed
- Add no-changes job to report when no components were modified
- Push/schedule/workflow_dispatch still evaluate all components

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Update validate-skills.yml

* Add validate-agents.yml to workflow paths

* Update workflow to trigger on changes to itself

* Update skill-validator.yml

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2026-02-20 14:31:03 +01:00
Viktor Hofer 7f77fdf29c Reduce quota in evaluation workflow (#65)
Temporarily use an older model and only run a test 3 times instead of 5
2026-02-20 14:17:01 +01:00
Viktor Hofer df1ce04667 Merge pull request #60 from dotnet/DisableCron
Disable cron for now
2026-02-20 11:05:32 +01:00
Viktor Hofer ad23394552 Add discover+matrix to evaluation workflow for multi-component support
Replace hardcoded src/dotnet paths with a discover job that finds all
components under src/ with skills/ and tests/ directories. Each
component is evaluated in parallel via matrix strategy. Artifacts and
benchmark data are now generated per-component.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2026-02-20 10:17:27 +01:00
Viktor Hofer 11bf689e82 Inline skill-validator into eng/skill-validator
Move skill-validator sources and tests from the external
adityamandaleeka/skill-validator repository into eng/skill-validator/.
Update evaluation.yml to build and run from the local copy instead of
checking out the external repo. Add skill-validator.yml CI pipeline
for building and testing the validator itself.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2026-02-20 10:03:31 +01:00
Viktor Hofer e3b9ad2868 Merge branch 'main' into dotnet-msbuild 2026-02-20 09:42:07 +01:00
Aditya Mandaleeka 527d2d5b13 fix: force array output in components.json for dashboard (#59) 2026-02-19 21:40:49 -08:00
Aditya Mandaleeka 6869be2da5 Add git remote origin when initializing gh-pages branch. (#57) 2026-02-19 21:26:14 -08:00
Aditya Mandaleeka 5d78f1d9ed Update skill-validator version. (#56) 2026-02-19 21:11:24 -08:00
Aditya Mandaleeka f82d6a0f02 Include hidden files for uploading results. (#55)
* Add debug step to investigate missing results artifacts

* Fix artifact upload: include hidden files for .skill-validator-results

* Update skill validator commit.
2026-02-19 20:19:49 -08:00
Aditya Mandaleeka ee4c728d54 Change default runs to 5 for statistical significance (#50) 2026-02-19 19:27:57 -08:00
Aditya Mandaleeka 616b81bdd3 Update skill-validator ref with JSON serialization and mkdir fixes (#49) 2026-02-19 19:18:19 -08:00
Aditya Mandaleeka 03223239ce Use correct token. (#48) 2026-02-19 19:01:46 -08:00
Aditya Mandaleeka 88a4fb388b Update skill-validator ref to include CI auth fix (#47) 2026-02-19 18:51:20 -08:00
Aditya Mandaleeka 5d75f75b2a Bump node version to 22. (#46) 2026-02-19 18:36:47 -08:00
Aditya Mandaleeka ae6f4b00a9 Use gh pr comment instead of third party dependency. (#45) 2026-02-19 18:07:54 -08:00
Aditya Mandaleeka e1f7fe3165 Replace eval pipeline with skill-validator. (#44) 2026-02-19 17:27:21 -08:00
Viktor Hofer fd2d4409db Replace peter-evans/create-or-update-comment with gh CLI
The third-party action peter-evans/create-or-update-comment@v4 is not
allowed by repository policy. Replace with inline gh pr comment command
which is pre-installed on GitHub-hosted runners.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2026-02-19 23:59:36 +01:00
Viktor Hofer 5e5b930282 Temporarily dissable skill warnAsError 2026-02-19 23:55:17 +01:00
Viktor Hofer 7d69f47377 Disable reference-exists false-positive test 2026-02-19 23:55:17 +01:00
Viktor Hofer 3362be3cd5 Eval workflow structure (#41)
* Restructure repo to use src/<component> layout

Adopt the same repository structure as dotnet/skills:
- Move skills/ and tests/ under src/dotnet/
- Add plugin.json for the csharp-scripts component
- Add AGENTS.md with plugin versioning instructions
- Update marketplace.json files to point to new paths
- Update CI workflows for new directory structure
- Move contributor guidance from README into CONTRIBUTING.md
- Streamline README to match dotnet/skills format

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Add evaluation workflow and dashboard infrastructure

Add GitHub Actions workflow for skill evaluation with parallel test
execution, response grading, and summary generation. Include:

- CI workflow (.github/workflows/evaluation.yml) with matrix-based
  parallel test runs and configurable parameters
- Evaluation scripts (eng/evaluation/) for invoking Copilot, parsing
  stats, evaluating responses, and generating summaries
- Dashboard (eng/dashboard/) with HTML/JS visualization and benchmark
  data generation script
- Updated CONTRIBUTING.md with evaluation documentation reference

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2026-02-19 23:22:33 +01:00
Viktor Hofer 9c60c3fd44 Merge pull request #40 from ViktorHofer/RepoLayoutRefactoring
Restructure repo to use src/<component> layout
2026-02-19 23:15:30 +01:00
Tim Heuer 6fd0acd2e6 ci: added validator for custom-agents [skip ci] 2026-02-04 17:47:05 -08:00
Tim Heuer 56df391833 ci: fixing workflow for validation [skip ci] 2026-02-04 13:36:14 -08:00
Tim Heuer 41da31e928 feat: add initial marketplace manifest, contributing, readme, and validation workflow 📚 2026-02-04 07:44:22 -08:00