ignore noscript when claiming

src/compiler/compile/render_dom/wrappers/Element/index.ts

@@ -277,12 +277,7 @@ export default class ElementWrapper extends Wrapper {
 	render(block: Block, parent_node: Identifier, parent_nodes: Identifier) {
 		const { renderer } = this;
 
-		if (this.node.name === 'noscript') {
-			if (renderer.options.hydratable) {
-				block.chunks.claim.push(b`@claim_noscript(${parent_nodes});`);
-			}
-			return;
-		}
+		if (this.node.name === 'noscript') return;
 
 		if (this.slot_block) {
 			block = this.slot_block;

src/runtime/internal/dom.ts

@@ -184,10 +184,6 @@ export function claim_space(nodes) {
 	return claim_text(nodes, ' ');
 }
 
-export function claim_noscript(nodes) {
-	detach(claim_element(nodes, 'NOSCRIPT', {}, false));
-}
-
 function find_comment(nodes, text, start) {
 	for (let i = start; i < nodes.length; i += 1) {
 		const node = nodes[i];

test/runtime/samples/noscript-removal/_config.js

@@ -1,9 +1,4 @@
 export default {
-	html: `
-	<div>foo</div>
-
-	<div>foo<div>foo</div></div>
-`,
 	ssrHtml: `
 	<noscript>foo</noscript>
 
@@ -11,4 +6,28 @@ export default {
 
 	<div>foo<div>foo<noscript>foo</noscript></div></div>
 	`,
+	test({ assert, target, compileOptions }) {
+		// if created on client side, should not build noscript
+		if (!compileOptions.hydratable) {
+			assert.equal(target.querySelectorAll('noscript').length, 0);
+		}
+
+		// it's okay not to remove the node during hydration
+		// will not be seen by user anyway
+		removeNoScript(target);
+
+		assert.htmlEqual(
+			target.innerHTML,
+			`
+				<div>foo</div>
+				<div>foo<div>foo</div></div>
+			`
+		);
+	}
 };
+
+function removeNoScript(target) {
+	target.querySelectorAll("noscript").forEach(elem => {
+		elem.parentNode.removeChild(elem);
+	});
+}