Skip to content

chore: update to vally 0.7.0 - #2759

Merged
JasonYeMSFT (JasonYeMSFT) merged 1 commit into
microsoft:mainfrom
JasonYeMSFT:chuye/update-vally
Jul 6, 2026
Merged

chore: update to vally 0.7.0#2759
JasonYeMSFT (JasonYeMSFT) merged 1 commit into
microsoft:mainfrom
JasonYeMSFT:chuye/update-vally

Conversation

@JasonYeMSFT

@JasonYeMSFT JasonYeMSFT (JasonYeMSFT) commented Jul 6, 2026

Copy link
Copy Markdown
Member

Description

Update Vally to version 0.7.0. Vally 0.7.0 removed the buggy reference linter that raises false positive issues for orphaned reference files. This will remove the lint error we get for azure-compute in PR checks.

Checklist

  • Tests pass locally (cd tests && npm test)
  • Title has one of the prefixes: fix:, feat:, feature:, chore:, misc:, test:, eval:
  • If modifying skill descriptions: verified routing correctness with integration tests (In tests/, npm run test:integration -- <skill> or npm run test:vally -- --skill <skill>)

Related Issues

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the test harness’ Vally dependency to @microsoft/vally-cli@^0.7.0 and aligns the custom graders’ metadata with the newer Vally grader schema.

Changes:

  • Bump @microsoft/vally-cli from ^0.6.0 to ^0.7.0 (and refresh the lockfile).
  • Update custom grader metadata to use the new behavior fields (requiresLlmClient, requiresWorkspace) and remove deprecated metadata fields / checks.
Show a summary per file
File Description
tests/vally/json-grader.ts Updates grader metadata schema for Vally 0.7 and removes the workspaceStatus-based guard.
tests/vally/java-upgrade-grader.ts Updates grader metadata schema for Vally 0.7 and removes the workspaceStatus-based guard.
tests/package.json Bumps @microsoft/vally-cli dependency to ^0.7.0.
tests/package-lock.json Lockfile refresh reflecting Vally 0.7.0 and transitive dependency updates.

Review details

Files not reviewed (1)
  • tests/package-lock.json: Generated file
  • Files reviewed: 3/4 changed files
  • Comments generated: 0
  • Review effort level: Low

@jongio Jon Gallant (jongio) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only two files in the repo implement GraderMetadata (json-grader.ts and java-upgrade-grader.ts), and both are updated here. No missed siblings for the 0.7.0 schema migration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants