Update lit-ssr dependency

.changeset/gold-windows-fly.md

@@ -0,0 +1,5 @@
+---
+'@astrojs/lit': patch
+---
+
+Bump deps to lit-ssr 3.0 and polyfill supoprt for template[shadowrootmode] support. Also fix breakage caused by [email protected] hydration behavior requiring lit-ssr 3.0

examples/framework-lit/package.json

@@ -12,8 +12,8 @@
   },
   "dependencies": {
     "astro": "^2.1.8",
-    "lit": "^2.2.5",
+    "lit": "^2.7.0",
     "@astrojs/lit": "^1.3.0",
-    "@webcomponents/template-shadowroot": "^0.1.0"
+    "@webcomponents/template-shadowroot": "^0.2.1"
   }
 }

packages/astro/e2e/fixtures/lit-component/package.json

@@ -4,8 +4,8 @@
   "private": true,
   "dependencies": {
     "@astrojs/lit": "workspace:*",
-    "@webcomponents/template-shadowroot": "^0.1.0",
+    "@webcomponents/template-shadowroot": "^0.2.1",
     "astro": "workspace:*",
-    "lit": "^2.2.5"
+    "lit": "^2.7.0"
   }
 }

packages/astro/e2e/fixtures/multiple-frameworks/package.json

@@ -12,8 +12,8 @@
     "astro": "workspace:*"
   },
   "dependencies": {
-    "@webcomponents/template-shadowroot": "^0.1.0",
-    "lit": "^2.2.5",
+    "@webcomponents/template-shadowroot": "^0.2.1",
+    "lit": "^2.7.0",
     "preact": "^10.7.3",
     "react": "^18.1.0",
     "react-dom": "^18.1.0",

packages/astro/e2e/lit-component.test.js

@@ -8,10 +8,6 @@ const test = testFactory({
 // TODO: configure playwright to handle web component APIs
 // https://github.com/microsoft/playwright/issues/14241
 test.describe('Lit components', () => {
-	test.beforeAll(() => {
-		delete globalThis.window;
-	});
-
 	test.describe('Development', () => {
 		let devServer;
 		const t = test.extend({});
@@ -158,7 +154,6 @@ test.describe('Lit components', () => {
 		const t = test.extend({});
 
 		t.beforeAll(async ({ astro }) => {
-			delete globalThis.window;
 			// Playwright's Node version doesn't have these functions, so stub them.
 			process.stdout.clearLine = () => {};
 			process.stdout.cursorTo = () => {};

packages/astro/test/fixtures/lit-element/package.json

@@ -4,8 +4,8 @@
   "private": true,
   "dependencies": {
     "@astrojs/lit": "workspace:*",
-    "@webcomponents/template-shadowroot": "^0.1.0",
+    "@webcomponents/template-shadowroot": "^0.2.1",
     "astro": "workspace:*",
-    "lit": "^2.2.5"
+    "lit": "^2.7.0"
   }
 }

packages/astro/test/ssr-lit.test.js

@@ -25,7 +25,6 @@ describe('Lit integration in SSR', () => {
 	}
 
 	it('Is able to load', async () => {
-		delete globalThis.window; // On Windows this results in `ReferenceError: window is not defined`
 		const html = await fetchHTML('/');
 		const $ = cheerioLoad(html);
 		expect($('#win').text()).to.equal('function');

packages/integrations/lit/package.json

@@ -34,20 +34,21 @@
     "test": "mocha"
   },
   "dependencies": {
-    "@lit-labs/ssr": "^2.2.0",
+    "@lit-labs/ssr": "^3.1.0",
+    "@lit-labs/ssr-dom-shim": "^1.1.0",
     "parse5": "^7.1.2"
   },
   "devDependencies": {
     "astro": "workspace:*",
     "astro-scripts": "workspace:*",
     "chai": "^4.3.6",
     "cheerio": "^1.0.0-rc.11",
-    "lit": "^2.2.5",
+    "lit": "^2.7.0",
     "mocha": "^9.2.2",
     "sass": "^1.52.2"
   },
   "peerDependencies": {
-    "@webcomponents/template-shadowroot": "^0.1.0",
-    "lit": "^2.1.3"
+    "@webcomponents/template-shadowroot": "^0.2.1",
+    "lit": "^2.7.0"
   }
 }

packages/integrations/lit/server-shim.js

@@ -1,20 +1,28 @@
-import { installWindowOnGlobal } from '@lit-labs/ssr/lib/dom-shim.js';
+import {customElements as litCE} from '@lit-labs/ssr-dom-shim';
 
-if (typeof fetch === 'function') {
-	const _fetch = fetch;
-	installWindowOnGlobal();
-	globalThis.fetch = window.fetch = _fetch;
-} else {
-	installWindowOnGlobal();
+// Something at build time injects document.currentScript = undefined instead of
+// document.currentScript = null. This causes Sass build to fail because it
+// seems to be expecting `=== null`. This set to `undefined` doesn't seem to be
+// caused by Lit and only happens at build / test time, but not in dev or
+// preview time.
+if (globalThis.document) {
+	document.currentScript = null;
 }
 
-window.global = window;
-document.getElementsByTagName = () => [];
-// See https://github.com/lit/lit/issues/2393
-document.currentScript = null;
+// Astro seems to have a DOM shim and the only real difference that we need out
+// of the Lit DOM shim is that the Lit DOM shim does something that triggers
+// ReactiveElement.finalize() to be called. So this is the only thing we will
+// re-shim since Lit will try to respect other global DOM shims.
+globalThis.customElements = litCE;
 
-const ceDefine = customElements.define;
+const litCeDefine = customElements.define;
+
+// We need to patch customElements.define to keep track of the tagName on the
+// class itself so that we can transform JSX custom element class definintion to
+// a DSD string on the server, because there is no way to get the tagName from a
+// CE class otherwise. Not an issue on client:only because the browser supports
+// appending a class instance directly to the DOM.
 customElements.define = function (tagName, Ctr) {
 	Ctr[Symbol.for('tagName')] = tagName;
-	return ceDefine.call(this, tagName, Ctr);
+	return litCeDefine.call(this, tagName, Ctr);
 };

packages/integrations/lit/server.js

@@ -1,5 +1,4 @@
 import './server-shim.js';
-import '@lit-labs/ssr/lib/render-lit-html.js';
 import { LitElementRenderer } from '@lit-labs/ssr/lib/lit-element-renderer.js';
 import * as parse5 from 'parse5';
 

packages/integrations/lit/test/server.test.js

@@ -1,6 +1,8 @@
 import { expect } from 'chai';
-import server from '../server.js';
 import { LitElement, html } from 'lit';
+// Must come after lit import because @lit/reactive-element defines
+// globalThis.customElements which the server shim expects to be defined.
+import server from '../server.js';
 import * as cheerio from 'cheerio';
 
 const { check, renderToStaticMarkup } = server;
@@ -12,6 +14,10 @@ describe('check', () => {
 
 	it('should be false with a registered non-lit component', async () => {
 		const tagName = 'non-lit-component';
+		// Lit no longer shims HTMLElement globally, so we need to do it ourselves.
+		if (!globalThis.HTMLElement) {
+			globalThis.HTMLElement = class {};
+		}
 		customElements.define(tagName, class TestComponent extends HTMLElement {});
 		expect(await check(tagName)).to.equal(false);
 	});
@@ -85,7 +91,7 @@ describe('renderToStaticMarkup', () => {
 	});
 
 	it('should render DSD attributes based on shadowRootOptions', async () => {
-		const tagName = 'lit-component';
+		const tagName = 'shadow-root-options-component';
 		customElements.define(
 			tagName,
 			class extends LitElement {