feat: add astro support for VSCode extension

[XiNiHa]

Adds Astro syntax grammar support, language server parsing support, and misc wirings.

[changeset-bot]

🦋 Changeset detected

Latest commit: deccc6a

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

This PR includes changesets to release 4 packages

Name	Type
graphql-language-service-server	Minor
vscode-graphql	Minor
vscode-graphql-syntax	Minor
graphql-language-service-cli	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

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: XiNiHa / name: Cosmo Shin (신의하) (198ce79, 48ed2e3, c21038c, deccc6a)

[acao]

awesome work! you even covered the esbuild script, very comprehensive, thank you!

[acao]

@XiNiHa just a few issues with dependencies, spelling and linting to clear up it looks like

[acao]

I'm about to merge a refactor for all of this that you will appreciate i'm sure, though it will cause a merge conflict

[acao]

@XiNiHa enjoy! let me know if you want me to redo it

[XiNiHa]

Sorry for being late to fix things up ;( I'll try to look at stuffs around this weekend

[acao]

@XiNiHa no worries, take your time!

also I improved the grammars a bit. let me know if you want to pair on this or have any questions.

[XiNiHa]

The language server doesn't work due to ardatan/graphql-tools#5781 not being merged yet.

[acao]

@XiNiHa we have our own code parsing currently, but graphql-tools are also parsing files on their own, we don't use that result

[acao]

two things!

1. can you add astro tests in findGraphQLParser-test.js
2. I'm not sure why the graphql-language-service-server tests are failing for a missing astrojs-compiler-sync, even though it is added to package.json and seems to be reflected in yarn. can you try running yarn install again to update the lockfile?

[XiNiHa]

I'm not sure why the graphql-language-service-server tests are failing for a missing astrojs-compiler-sync

Looks like a typo in the package.json of astrojs-compiler-sync, will make a PR on there

[acao]

awesome work!

sidenote: i think the issue with svelte2tsx is twofold - the language server needs to specify typescript as a dependency so that it works in non vscode contexts, and with vscode, we need to upgrade vsce as you have done and mark typescript as external. if we try to make it internal to the esbuild bundle it breaks as well. i can show you how to debug definitively - the vscode extension runner does not reflect the esbuild bundled state

[codecov]

Codecov Report

Attention: 20 lines in your changes are missing coverage. Please review.

Comparison is base (88ae243) 55.94% compared to head (deccc6a) 55.74%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3475      +/-   ##
==========================================
- Coverage   55.94%   55.74%   -0.21%     
==========================================
  Files         113      114       +1     
  Lines        5291     5312      +21     
  Branches     1441     1444       +3     
==========================================
+ Hits         2960     2961       +1     
- Misses       1903     1923      +20     
  Partials      428      428              

Files	Coverage Δ
...s/graphql-language-service-server/src/constants.ts	100.00% <ø> (ø)
...hql-language-service-server/src/findGraphQLTags.ts	82.71% <ø> (ø)
...aphql-language-service-server/src/parsers/astro.ts	4.76% <4.76%> (ø)

[acao]

@XiNiHa this looks great! I will go ahead and merge, and then test the .vsix bundle manually just to be 100% certain. I think we can merge this and fix the svelte parsing in a single release

[acao]

I will take care of adding the tests in a follow up PR, but I might ping you later to make sure I've covered all the cases