@tsrx/core 0.1.22 → 0.1.24

package/package.json

@@ -3,7 +3,7 @@
   "description": "Core compiler infrastructure for TSRX syntax",
   "license": "MIT",
   "author": "Dominic Gannaway",
-  "version": "0.1.22",
+  "version": "0.1.24",
   "type": "module",
   "repository": {
     "type": "git",

package/src/analyze/prune.js

@@ -62,6 +62,14 @@ function get_attribute_value(attribute) {
 	return value?.type === 'JSXExpressionContainer' ? value.expression : value;
 }
 
+/**
+ * @param {AST.Node} node
+ * @returns {boolean}
+ */
+function is_runtime_dynamic_element(node) {
+	return node?.metadata?.runtime_dynamic_element === true;
+}
+
 /**
  * Returns true if node is a DOM element (not a component).
  * @param {AST.Node} node
@@ -73,23 +81,10 @@ function is_element_dom_element(node) {
 		(id.type === 'Identifier' || id.type === 'JSXIdentifier') &&
 		id.name[0].toLowerCase() === id.name[0] &&
 		id.name !== 'children' &&
-		!id.tracked &&
-		!id.dynamic
+		!id.tracked
 	);
 }
 
-/**
- * Returns true if element is dynamic.
- * @param {any} node
- * @returns {boolean}
- */
-function is_element_dynamic(node) {
-	const id = get_element_name(node);
-	return id?.type === 'Identifier' || id?.type === 'JSXIdentifier'
-		? !!(id.tracked || id.dynamic)
-		: false;
-}
-
 // CSS selector constants
 /**
  * @param {number} start
@@ -395,13 +390,11 @@ function get_descendant_elements(node, adjacent_only) {
  * @returns {boolean}
  */
 function can_render_dynamic_content(element, check_classes = false) {
-	if (!is_element_dom_element(element)) {
+	if (is_runtime_dynamic_element(element)) {
 		return true;
 	}
 
-	// Either a dynamic element or component (only can tell at runtime)
-	// But dynamic elements should return false ideally
-	if (is_element_dynamic(element)) {
+	if (!is_element_dom_element(element)) {
 		return true;
 	}
 
@@ -1021,7 +1014,7 @@ function relative_selector_might_apply_to_node(relative_selector, rule, element,
 			}
 
 			case 'TypeSelector': {
-				if (is_element_dynamic(element)) {
+				if (is_runtime_dynamic_element(element)) {
 					break;
 				}
 
@@ -1127,10 +1120,6 @@ export function prune_css(css, element, styleClasses, topScopedClasses) {
 	style_identifier_classes = styleClasses;
 	top_scoped_classes = topScopedClasses;
 
-	if (is_element_dynamic(element)) {
-		element.metadata.scoped = true;
-	}
-
 	/** @type {Visitors<AST.CSS.Node, null>} */
 	const visitors = {
 		Rule(node, context) {

package/src/analyze/validation.js

@@ -9,7 +9,7 @@ import { DIAGNOSTIC_CODES } from '../diagnostics.js';
 export const TSRX_RETURN_STATEMENT_ERROR =
 	'Return statements are not allowed inside TSRX templates. Move the return before the TSRX return value, or use conditional rendering instead.';
 export const TSRX_LOOP_RETURN_ERROR =
-	'Return statements are not allowed inside TSRX template for...of loops. Filter the iterable before rendering or use an @for empty fallback for empty lists.';
+	'Return statements are not allowed inside TSRX template for...of loops. Filter the iterable before rendering or use an @empty fallback for empty lists.';
 export const TSRX_LOOP_BREAK_ERROR =
 	'Break statements are not allowed inside TSRX template for...of loops.';
 export const TSRX_LOOP_CONTINUE_ERROR =

package/src/diagnostics.js

@@ -4,4 +4,5 @@ export const DIAGNOSTIC_CODES = {
 	MISMATCHED_CLOSING_TAG: 'tsrx-mismatched-closing-tag',
 	TEMPLATE_EXPRESSION_TRAILING_SEMICOLON: 'tsrx-template-expression-trailing-semicolon',
 	TEMPLATE_RETURN_STATEMENT: 'tsrx-template-return-statement',
+	FORGOTTEN_STATEMENT_CONTAINER: 'tsrx-forgotten-statement-container',
 };

package/src/index.js

@@ -185,9 +185,7 @@ export {
 	identifier_to_jsx_name,
 	is_bare_render_expression,
 	is_component_jsx_name,
-	is_dynamic_element_id,
 	is_jsx_child,
-	jsx_name_to_expression,
 	set_loc,
 	to_text_expression,
 } from './transform/jsx/ast-builders.js';