Make React integration work with Deno (#4679)

* Remove removed packages folder * fix * Add a changeset
withastro · Sep 8, 2022 · 5986517 · 5986517
1 parent 9290b24
commit 5986517
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 5 deletions.
diff --git a/.changeset/orange-llamas-play.md b/.changeset/orange-llamas-play.md
@@ -0,0 +1,5 @@
+---
+'@astrojs/react': patch
+---
+
+Prevent decoder from leaking
diff --git a/package.json b/package.json
@@ -32,7 +32,6 @@
     "packages/*",
     "examples/*",
     "examples/component/demo",
-    "examples/component/packages/*",
     "scripts",
     "packages/astro/test/fixtures/component-library-shared",
     "packages/astro/test/fixtures/custom-elements/my-component-lib",

diff --git a/packages/integrations/react/server.js b/packages/integrations/react/server.js
@@ -144,10 +144,17 @@ async function readResult(stream) {
   let result = '';
   const decoder = new TextDecoder('utf-8')
   while (true) {
-    const { done, value } = await reader.read();
-    if (done) {
-      return result;
-    }
+		const { done, value } = await reader.read();
+		if (done) {
+			if(value) {
+				result += decoder.decode(value);
+			} else {
+				// This closes the decoder
+				decoder.decode(new Uint8Array());
+			}
+
+			return result;
+		}
     result += decoder.decode(value, { stream: true });
   }
 }