Prevent type checking define:vars scripts

[lilnasy]

Changes

• Closes Typescript warning when using define:vars on script tag #711

Testing

Manually tested against this snippet:

---
export interface Props {
  server: string;
}

const { server } = Astro.props;
---

<div id="waline"></div>
<script>
  import { init } from "https://unpkg.com/@waline/client@v2/dist/waline.mjs";
  const x = 43
  init({
    el: "#waline",
    serverURL,
  });
</script>
<script define:vars={{ serverURL: server }} type="module">
  import { init } from "https://unpkg.com/@waline/client@v2/dist/waline.mjs";
  const x = 42
  init({
    el: "#waline",
    serverURL,
  });
</script>
<script is:inline>
	const x = 43
</script>
<script is:inline>
	const x = 44
</script>

Docs

Does not affect usage

[changeset-bot]

🦋 Changeset detected

Latest commit: 042bb0d

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

This PR includes changesets to release 3 packages

Name	Type
@astrojs/language-server	Patch
@astrojs/check	Patch
astro-vscode	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

[lilnasy]

the fix is here, the rest is clerical

[Princesseuh]

Just a comment on the changeset, looks great otherwise. Thank you!

[Princesseuh]

Please add the VS Code package and the Astro check ones to this changeset so they also get a release for this

[lilnasy]

Wouldn't their semver ranges include this release?