Skip to content

Conversation

@nicdumz
Copy link
Contributor

@nicdumz nicdumz commented Jan 11, 2026

Description

Users may already have yaml.schemas preferences and the extension should not be overwriting user settings.

The code is however free to control keys for extensionUri, as long as it leaves other entries alone.

See also #7080

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-review

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Screen recording or screenshot

N/A

Tests

N/A


Continue Tasks

Status Task Actions
▶️ Queued Update docs on PR View
▶️ Queued Optimize Website Performance View

Powered by Continue


Summary by cubic

Preserve user-defined yaml.schemas settings by merging instead of overwriting when registering the extension’s schema. The extension only adds/updates its own entry keyed by extensionUri and leaves other schema mappings unchanged.

Written for commit a0729bd. Summary will update on new commits.

@nicdumz nicdumz requested a review from a team as a code owner January 11, 2026 15:30
@nicdumz nicdumz requested review from RomneyDa and removed request for a team January 11, 2026 15:30
@continue-staging
Copy link

All Green - Keep your PRs mergeable

Learn more

All Green is an AI agent that automatically:

✅ Addresses code review comments

✅ Fixes failing CI checks

✅ Resolves merge conflicts


Unsubscribe from All Green comments

1 similar comment
@continue
Copy link
Contributor

continue bot commented Jan 11, 2026

All Green - Keep your PRs mergeable

Learn more

All Green is an AI agent that automatically:

✅ Addresses code review comments

✅ Fixes failing CI checks

✅ Resolves merge conflicts


Unsubscribe from All Green comments

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jan 11, 2026
@github-actions
Copy link

github-actions bot commented Jan 11, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@nicdumz
Copy link
Contributor Author

nicdumz commented Jan 11, 2026

I have read the CLA Document and I hereby sign the CLA

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@nicdumz nicdumz force-pushed the do-not-overwrite-settings branch from 9d3e47a to 8790b8e Compare January 11, 2026 15:43
@nicdumz nicdumz changed the title Merge yaml.schemas settings fix: merge yaml.schemas settings Jan 11, 2026
@nicdumz
Copy link
Contributor Author

nicdumz commented Jan 11, 2026

npm error Finding release for v13.0.0-10
npm error GET https://api.github.com/repos/microsoft/ripgrep-prebuilt/releases/tags/v13.0.0-10
npm error Deleting invalid download cache
npm error Downloading ripgrep failed: Error: Request failed: 403

I don't think this error is related to my PR :/

Users may already have `yaml.schemas` preferences and the extension
should not be overwriting user settings.

The code is however free to control keys for `extensionUri`, as long as
it leaves other entries alone.

See also continuedev#7080
@nicdumz nicdumz force-pushed the do-not-overwrite-settings branch from 8790b8e to a0729bd Compare January 11, 2026 16:23
@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Jan 14, 2026
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 14, 2026
@RomneyDa RomneyDa merged commit cfab393 into continuedev:main Jan 14, 2026
55 of 56 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Jan 14, 2026
@github-actions github-actions bot locked and limited conversation to collaborators Jan 14, 2026
@sestinj
Copy link
Contributor

sestinj commented Jan 15, 2026

🎉 This PR is included in version 1.39.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

lgtm This PR has been approved by a maintainer released size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants