Repository navigation
Support native Docker evals with Vally 0.21 - #17
Draft
ronniegeraghty wants to merge 1 commit into
Draft
ronniegeraghty wants to merge 1 commit into
ronniegeraghty wants to merge 1 commit into
Conversation
Document the explicit native Docker backend without changing execution defaults. Keep registry-specific tarball endpoints out of npm's generated lockfile. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
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
Add the supported Vally 0.21 native Docker eval path without changing execution defaults or deciding whether Docker should become mandatory.
Changes
@microsoft/vally-clito stable0.21.0; keep the explicit host Copilot1.0.98pin and marketplace remote pins unchanged.omit-lockfile-registry-resolvedoption. Removing 218 registry-specific endpoints preserved every version, integrity hash, and other lock entry field from the initial upgrade.Verification
npm ci,npm run lint(6 skills and 7 eval definitions), local catalog consistency,git diff --check, and 51 retained model-free script tests passed.npm ciused the existing configured development registry. This does not establish independent public-registry availability.copilot-sdkrun executed both unchangedpr-descriptioncases and their native deterministic graders: one worker, one attempt per case, zero retries, normal task network, offline grading.Both task executions and both grading executions completed without execution or grading errors. The eval verdict still failed (1/2 cases, 79.17% score against the unchanged 100% threshold); no skill, grader, model, or threshold was changed to obtain a pass.
Runtime observations: core
0.21.0, SDK1.0.31, actual Copilot runtime1.0.109, selected modelclaude-haiku-5.5and response modelclaude-haiku-5-5. The runtime listed 15 available skills, including the target, but neither case recorded a skill activation. Recorded usage was 68,909 tokens across three model calls. The SDK's managed runtime uses--no-auto-update; this container runtime is distinct from the preserved host Copilot pin.Run IDs:
native-docker-21-custom-pr-description-20261111andnative-docker-21-corrected-mock-20261111. Post-run container and volume inventories were empty.Risks / Rollback
sha256:654c029a370cb5cb0bce6cea326fc47cdeda3eaa67e89f20ff3f02566f98723d; it was never tagged as Vally's canonical default cache image and is not published by this PR.