fix(starter): fix up vscode settings merge 🐼

[gioboa]

Close #5280

base/.vscode/settings.json is a JSON5 file, we need to use json5 lib to parse it.
Right now we have an error because in the Qwik CLI, we're trying to parse that file with JSON.parse; this will fallback to the catch branch and overwrite the entire file.

[changeset-bot]

🦋 Changeset detected

Latest commit: 1e4ab47

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages

Name	Type
create-qwik	Patch
@builder.io/qwik	Patch
@builder.io/qwik-city	Patch
eslint-plugin-qwik	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@builder.io/qwik@7858

npm i https://pkg.pr.new/@builder.io/qwik-city@7858

npm i https://pkg.pr.new/eslint-plugin-qwik@7858

npm i https://pkg.pr.new/create-qwik@7858

commit: c11e2be

[github-actions]

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name	Status	Preview	Last Commit
qwik-docs	✅ Ready (View Log)	Visit Preview	792c74e

[wmertens]

Actually, the bug is on our side. The settings file is JSON5, meaning it supports comments, and we should parse it as such. We can use https://www.npmjs.com/package/json5-writer to handle json files in a way that preserves comments.

[gioboa]

Actually, the bug is on our side. The settings file is JSON5, meaning it supports comments, and we should parse it as such. We can use https://www.npmjs.com/package/json5-writer to handle json files in a way that preserves comments.

Fixed

[gioboa]

JSON5.parse is working fine but it's stripping the comments 🤔
On the other hand it is solving the main issue

[gioboa]

with @croct/json5-parser is working better and it's keeping the comments too.

[wmertens]