Repository navigation
Conversation
…arity Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🔵 Needs a closer look
The runtime upgrades and run-lock recovery changes introduce cross-host compatibility, cancellation, and executor-fallback interactions that require maintainer review beyond mechanical consistency checks.
No open issues 🎉
What changed in this PR
Updates the marketplace’s optional Evals plugin pin and strengthens repository validation without changing local plugin behavior.
Changes:
- Synchronizes Evals v0.24.6 references across the catalog, shared eval environment, and README.
- Adds validation workflow concurrency controls and a 15-minute job timeout.
- Adds regression coverage for plugins-only marketplaces and catalogs without remote entries.
| File | Description |
|---|---|
| README.md | Updates upstream setup and license links to the new Evals pin. |
| marketplace.json | Pins the remote Evals plugin to v0.24.6 and its full commit SHA. |
| .vally.yaml | Aligns shared eval environment setup with the catalog pin. |
| .github/workflows/validate.yml | Adds concurrency controls and a validation job timeout. |
| .github/scripts/lint.test.mjs | Tests plugins-only linting and remote-pin checks with no remote entries. |
Review effort: Balanced (dynamic)
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.
Summary
Consolidated
repo-maintainandupdate-remote-pinspass across workflow concurrency/timeout hygiene, updating the remoteevalplugin pin tocopilot-evals-v0.24.6(5c3d5213f88149fe1e5476ef029277fe56fd42e4), script branch coverage, and strict Vally 0.21.0 skill/eval lint checks.Phase Breakdown
71cf8c4concurrency(cancel-in-progress: ${{ github.event_name == 'pull_request' }}) andtimeout-minutes: 15to.github/workflows/validate.ymlwhile preserving SHA-pinnedactions/checkoutandactions/setup-node(node-version: 24,persist-credentials: false,cache: npm).9fc27c1eval(github/evals,packages/plugin) remote pin inmarketplace.json,.vally.yaml, andREADME.mdfromcopilot-evals-v0.24.0(34ddb5803aeaee8330ab70c19ad1a3f37fb77ded) tocopilot-evals-v0.24.6(5c3d5213f88149fe1e5476ef029277fe56fd42e4,version: "0.24.6").package-lock.jsonintegrity hashes without registry tarball URLs..github/scripts/eval-report.mjswhile #15 is open.1b226f1.github/scripts/lint.test.mjscoveringplugins-only marketplaces and catalogs without remote entries (95.20%line /81.30%branch /100%function coverage across.github/scripts/*.mjs).npm run lint(6/6skills +7/7evals clean undervally lint --strict), Agent Plugins schema validation (3/3manifests valid),check-local-catalog.mjs,check-remote-pins.mjs, andnpm run eval:models -- --dry-run(14/14plans) passing with zero model calls.Upstream
evalRemote Pin Review (update-remote-pins)copilot-evals-v0.24.0@34ddb5803aeaee8330ab70c19ad1a3f37fb77ded(version: "0.24.0")copilot-evals-v0.24.6@5c3d5213f88149fe1e5476ef029277fe56fd42e4(version: "0.24.6",packages/plugin/plugin.json)34ddb58...5c3d5213):.run-lockmarkers (skills/run/SKILL.mdclarifies recovery files are retained and stale.run-lockmarkers neither block Native Run / standalone Vally nor authorize executor fallback).extension.mjs,web/app.mjs,workbench.mjs,dist/mcp/server.mjs) and Windowsx64/arm64native helpers.mcp.json,copilot-extension.json,THIRD-PARTY-NOTICES.txt,LICENSE(GitHub Limited Use Source License), hooks (none), network destinations, and public repository access (github/evalsis public; no private-repository token required) are unchanged.Repo Health Score:
94/100 (A)->97/100 (A+)(+3)Validate&CodeQLgreen onmain; 40-char SHA pins; added PRconcurrency+timeout-minutesinvalidate.yml560/560script & recipe tests passing;95.20%line /81.30%branch /100%function coverage@microsoft/vally-cli@0.21.0(>= 7d, MIT);evalremote pin synced tov0.24.6;@github/copilot(SEE LICENSE IN LICENSE.md) left for maintainer disposition0Dependabot/CodeQL findings; Private Reporting enabled;package-lock.jsonfree of registry URLs0broken local links;README.mdevallinks synced tocopilot-evals-v0.24.6/5c3d5213vally 0.21.0--strictclean across6skills &7evals;evals.ymlmanual-only withenvironment: evals>24hwithout comments)Verification
npm ci --no-audit --no-fundVALLY_TELEMETRY_OPTOUT=1 npm run lintnode .github/scripts/check-local-catalog.mjsnode .github/scripts/check-remote-pins.mjsbash .github/scripts/validate-schema.shnpm run eval:models -- --dry-runnode --test .github/scripts/lint.test.mjs .github/scripts/validation.test.mjs .github/scripts/eval-report.test.mjs .github/scripts/docker-defaults.test.mjs