From ad90896958906e7d96bd059506660c9ad9e228f4 Mon Sep 17 00:00:00 2001
From: Reece McDonald <mcdonald.reece@heb.com>
Date: Thu, 23 Feb 2023 17:14:48 -0600
Subject: [PATCH 1/2] [Lit] render DSD attributes based on `shadowRootOptions`
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

## Changes

- Update `@astrojs/lit`’s `server.js` to properly render elements with `delegatesFocus: false` set in their `shadowRootOptions`.
- Logic is based on `@lit-labs/ssr` [latest implementation as found here](https://github.com/lit/lit/blob/b0c3f82ef0f97326a205e77e7e1043b75a5cc53f/packages/labs/ssr/src/lib/render-value.ts#L738)

## Testing

A test was added to ensure an element with `delegatesFocus` set to true has this attribute properly included in the rendered static markup.
---
 packages/integrations/lit/server.js           |  6 +++++-
 packages/integrations/lit/test/server.test.js | 11 +++++++++++
 2 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/packages/integrations/lit/server.js b/packages/integrations/lit/server.js
index da571466f498..aa91d1ea8c65 100644
--- a/packages/integrations/lit/server.js
+++ b/packages/integrations/lit/server.js
@@ -62,7 +62,11 @@ function* render(Component, attrs, slots) {
 	yield `>`;
 	const shadowContents = instance.renderShadow({});
 	if (shadowContents !== undefined) {
-		yield '<template shadowroot="open" shadowrootmode="open">';
+		const { mode = 'open', delegatesFocus } = instance.shadowRootOptions ?? {};
+    // `delegatesFocus` is intentionally allowed to coerce to boolean to
+    // match web platform behavior.
+    const delegatesfocusAttr = delegatesFocus ? ' shadowrootdelegatesfocus' : '';
+		yield `<template shadowroot="${mode}" shadowrootmode="${mode}"${delegatesfocusAttr}>`;
 		yield* shadowContents;
 		yield '</template>';
 	}
diff --git a/packages/integrations/lit/test/server.test.js b/packages/integrations/lit/test/server.test.js
index 439c0877da72..af1f99c71162 100644
--- a/packages/integrations/lit/test/server.test.js
+++ b/packages/integrations/lit/test/server.test.js
@@ -83,4 +83,15 @@ describe('renderToStaticMarkup', () => {
 		expect($(tagName).attr('attr1')).to.equal(attr1);
 		expect($(`${tagName} template`).text()).to.contain(`Hello ${prop1}`);
 	});
+
+	it('should render DSD attributes based on shadowRootOptions', async () => {
+		const tagName = 'lit-component';
+		customElements.define(tagName, class extends LitElement {
+			static shadowRootOptions = {...LitElement.shadowRootOptions, delegatesFocus: true};
+		});
+		const render = await renderToStaticMarkup(tagName);
+		expect(render).to.deep.equal({
+			html: `<${tagName}><template shadowroot=\"open\" shadowrootmode=\"open\" shadowrootdelegatesfocus><!--lit-part--><!--/lit-part--></template></${tagName}>`,
+		});
+	});
 });

From 6fc4f22e5a34192f3697e2850971a720af75fe88 Mon Sep 17 00:00:00 2001
From: Reece McDonald <mcdonald.reece@heb.com>
Date: Thu, 23 Feb 2023 17:24:30 -0600
Subject: [PATCH 2/2] chore: add changeset

---
 .changeset/old-fireants-allow.md | 5 +++++
 1 file changed, 5 insertions(+)
 create mode 100644 .changeset/old-fireants-allow.md

diff --git a/.changeset/old-fireants-allow.md b/.changeset/old-fireants-allow.md
new file mode 100644
index 000000000000..4857a7a7cdad
--- /dev/null
+++ b/.changeset/old-fireants-allow.md
@@ -0,0 +1,5 @@
+---
+'@astrojs/lit': patch
+---
+
+Render DSD attributes based on `shadowRootOptions`