Update lit-ssr dependency

[e111077]

fixes #6670

Changes

• Updates lit-ssr dependency to match that of lit@^2.7.0's requirements
• Update integration with new lit-ssr changes (no global window)

Testing

Usual test run – tests updated

Docs

Should not require behavior change

[changeset-bot]

🦋 Changeset detected

Latest commit: 4b9a03e

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

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

[matthewp]

No longer needs to shim the window?

[e111077]

All of the shims needed for lit are baked into Lit's node build. I just pushed a commit though because we do need to actually re-shim only customElements.define and HTMLElement because our baked-in shims in ReactiveElement's node build is forgiving and tries to call anything anyone else has shimmed, but we have a small hack we are relying on so we re-shim that

[augustjk]

This part of the readme should be updated

astro/packages/integrations/lit/README.md

Lines 123 to 142 in fc71c3f

	### Browser globals
	The Lit integration's SSR works by adding a few browser global properties to the global environment. Some of the properties it adds includes `window`, `document`, and `location`.
	These globals *can* interfere with other libraries that might use the existence of these variables to detect that they are running in the browser, when they are actually running in the server. This can cause bugs with these libraries.
	Because of this, the Lit integration might not be compatible with these types of libraries. One thing that can help is changing the order of integrations when Lit is interfering with other integrations:
	```diff
	import { defineConfig } from 'astro/config';
	import vue from '@astrojs/vue';
	import lit from '@astrojs/lit';
	export default defineConfig({
	- integrations: [vue(), lit()]
	+ integrations: [lit(), vue()]
	});
	```
	The correct order might be different depending on the underlying cause of the problem. This is not guaranteed to fix every issue however, and some libraries cannot be used if you are using the Lit integration because of this.

[e111077]

@matthewp would also like to note that we have to re-shim HTMLElement because Astro's DOM shim doesn't seem to do anything when you do

element.setAttribute('foo', 'bar')
element.setAttribute('baz', 'quux')
element.getAttribute('foo') // bar
element.attributes // ['foo', 'baz']

https://github.com/withastro/astro/blob/main/packages/webapi/src/lib/Element.ts#L25-L32

I wonder WYT about implementing that in Astro?

[e111077]

You also might want to squash merge this one 🙈

[matthewp]

Thanks! I'm not sure how we feel about having the DOM shim in long term. Sort of feel like the renderer should bring its own if needed. So I think what you are doing here is fine for now, thanks!

[matthewp]

@augustjk is this ok with you now? can you give your 👍 if you think this is good to merge?

[augustjk]

Looking good, thank you!!