Skip to content

Fix metadata-loader tests for updated schema contract#518

Merged
hotlong merged 2 commits into
mainfrom
copilot/create-patch-again
Feb 5, 2026
Merged

Fix metadata-loader tests for updated schema contract#518
hotlong merged 2 commits into
mainfrom
copilot/create-patch-again

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 4, 2026

MetadataLoaderContractSchema now requires protocol and capabilities fields, but test fixtures were not updated.

Changes

  • Added protocol field to test contracts ('file' and 'http')
  • Added capabilities object with read/write/watch/list flags to both test cases
  • Added assertions to verify protocol field parsing
const contract = {
  name: 'filesystem',
  protocol: 'file',  // Added
  capabilities: {     // Added
    read: true,
    write: true,
    watch: false,
    list: true,
  },
  supportedFormats: ['json', 'yaml', 'typescript'] as const,
};

2 failing tests → all 3064 tests passing.

Original prompt

发一个补丁


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
spec Ready Ready Preview, Comment Feb 4, 2026 5:03pm

Request Review

… tests

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
Copilot AI changed the title [WIP] Create a patch for the existing code Fix metadata-loader tests for updated schema contract Feb 4, 2026
Copilot AI requested a review from hotlong February 4, 2026 16:56
@hotlong hotlong marked this pull request as ready for review February 5, 2026 01:36
Copilot AI review requested due to automatic review settings February 5, 2026 01:36
@hotlong hotlong merged commit c733b79 into main Feb 5, 2026
12 of 13 checks passed
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request fixes failing tests in the metadata-loader test suite by updating test fixtures to conform to the updated MetadataLoaderContractSchema which now requires protocol and capabilities fields.

Changes:

  • Added required protocol field ('file' and 'http') to both test contract fixtures
  • Added required capabilities object with read/write/watch/list boolean flags to both test cases
  • Added assertions to verify the protocol field is correctly parsed

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants