jsxpression 0.1.38 → 0.1.41

package/src/evaluate/evaluate.test.ts

@@ -60,7 +60,7 @@ describe("evaluate", () => {
       },
     });
 
-    expect(result.props!.value).toBe(15); // floor(5 * 3.14159...) = 15
+    expect(result.props.value).toBe(15); // floor(5 * 3.14159...) = 15
   });
 
   it("should throw EvaluationError for unknown component", () => {
@@ -116,7 +116,7 @@ describe("evaluate", () => {
     expect(result.type).toBe("Outer");
     expect(result.props).toEqual({ x: 10 });
     expect(result.children).toHaveLength(1);
-    const innerNode = result.children[0] as Node;
+    const innerNode = result.children[0];
     expect(innerNode.type).toBe("Inner");
     expect(innerNode.props).toEqual({ y: 20 });
     expect(innerNode.children).toEqual(["Text"]);
@@ -159,7 +159,7 @@ describe("evaluate", () => {
       },
     });
 
-    expect(result.props!.msg).toBe("visible");
+    expect(result.props.msg).toBe("visible");
   });
 
   it("should evaluate array methods on props", () => {
@@ -176,7 +176,7 @@ describe("evaluate", () => {
       },
     });
 
-    expect(result.props!.values).toEqual([6, 8, 10]);
+    expect(result.props.values).toEqual([6, 8, 10]);
   });
 
   it("should handle runtime errors gracefully", () => {
@@ -208,7 +208,7 @@ describe("evaluate", () => {
       },
     });
 
-    expect(result.props!.msg).toBe("Anonymous");
+    expect(result.props.msg).toBe("Anonymous");
   });
 
   it("should flatten and filter children correctly", () => {
@@ -269,7 +269,7 @@ describe("evaluate", () => {
       }
     );
 
-    expect(result.props!.values).toBe("6, 8, 10");
+    expect(result.props.values).toBe("6, 8, 10");
   });
 
   it("should pass correct parameters to custom createElement", () => {
@@ -295,7 +295,7 @@ describe("evaluate", () => {
       components: { Test: TestComponent },
     });
 
-    expect(result.props!.msg).toBe("Hello World");
+    expect(result.props.msg).toBe("Hello World");
   });
 
   it("should handle numeric operations", () => {
@@ -310,7 +310,7 @@ describe("evaluate", () => {
       components: { Test: TestComponent },
     });
 
-    expect(result.props!.value).toBe(11); // 5 + (3 * 2)
+    expect(result.props.value).toBe(11); // 5 + (3 * 2)
   });
 
   it("should handle data access in children", () => {
@@ -340,7 +340,7 @@ describe("evaluate", () => {
         components: { Test: TestComponent },
       });
 
-      expect(result.props!.value).toBe(42);
+      expect(result.props.value).toBe(42);
     });
 
     it("should evaluate const with computed expression", () => {
@@ -355,7 +355,7 @@ describe("evaluate", () => {
         components: { Test: TestComponent },
       });
 
-      expect(result.props!.value).toBe(8); // (3+1)*2
+      expect(result.props.value).toBe(8); // (3+1)*2
     });
 
     it("should evaluate const referencing data", () => {
@@ -370,7 +370,7 @@ describe("evaluate", () => {
         components: { Test: TestComponent },
       });
 
-      expect(result.props!.msg).toBe("Hello World");
+      expect(result.props.msg).toBe("Hello World");
     });
 
     it("should evaluate const with computed array", () => {
@@ -388,7 +388,7 @@ describe("evaluate", () => {
         }
       );
 
-      expect(result.props!.count).toBe(3);
+      expect(result.props.count).toBe(3);
     });
   });
 
@@ -425,7 +425,7 @@ describe("evaluate", () => {
         }
       );
 
-      expect(result.props!.result).toBe(10);
+      expect(result.props.result).toBe(10);
     });
 
     it("should evaluate nested user-defined components", () => {
@@ -450,8 +450,8 @@ describe("evaluate", () => {
 
       expect(result.type).toBe("Div");
       expect(result.children).toHaveLength(1);
-      expect((result.children[0] as Node).type).toBe("Span");
-      expect((result.children[0] as Node).children).toEqual(["hi"]);
+      expect(result.children[0].type).toBe("Span");
+      expect(result.children[0].children).toEqual(["hi"]);
     });
 
     it("should evaluate function with children", () => {
@@ -476,7 +476,7 @@ describe("evaluate", () => {
 
       expect(result.type).toBe("Div");
       expect(result.children).toHaveLength(1);
-      expect((result.children[0] as Node).type).toBe("Span");
+      expect(result.children[0].type).toBe("Span");
     });
 
     it("should evaluate function with default parameter values", () => {
@@ -530,7 +530,7 @@ describe("evaluate", () => {
         }
       );
 
-      expect(result.props!.result).toBe(12);
+      expect(result.props.result).toBe(12);
     });
 
     it("should evaluate function used in map with const", () => {
@@ -571,7 +571,7 @@ describe("evaluate", () => {
         components: { Test: TestComponent },
       });
 
-      expect(result.props!.value).toBe("42");
+      expect(result.props.value).toBe("42");
     });
 
     it("should evaluate String() with boolean", () => {
@@ -586,7 +586,7 @@ describe("evaluate", () => {
         components: { Test: TestComponent },
       });
 
-      expect(result.props!.value).toBe("true");
+      expect(result.props.value).toBe("true");
     });
   });
 });

package/src/evaluate/evaluate.ts

@@ -48,7 +48,7 @@ export function evaluate<T = any>(source: string, schema: Schema, options: Evalu
   const h = createH(components, createElement || defaultCreateElement, schema);
   const frozen = deepFreezeData(structuredClone(data));
   try {
-    // eslint-disable-next-line @typescript-eslint/no-implied-eval, no-new-func
+    // oxlint-disable-next-line typescript/no-implied-eval, no-new-func
     return new Function("h", "Math", ...dataKeys, ...functionKeys, source)(
       h,
       Math,
@@ -88,7 +88,7 @@ function createH(components: ComponentDict, createElement: CreateElement, schema
 
     if (!Component) {
       if (isElementAllowed(schema, type)) {
-        const defaultComponent = (props: PropsDict): any => (canHaveChildren(schema, type) ? props.children : null);
+        const defaultComponent = (props: PropsDict): any => (canHaveChildren(schema, type) ? props["children"] : null);
         return createElement(defaultComponent, props, ...children.flat().filter(Boolean));
       }
 

package/src/index.ts

@@ -3,7 +3,7 @@ export {
   type PropertySchema,
   type FunctionSchema,
   serializePropertySchemaToJson,
-  deserializePropertySchemaFromJson
+  deserializePropertySchemaFromJson,
 } from "./schema.js";
 
 export { ParseError } from "./parse/index.js";

package/src/render.test.ts

@@ -248,13 +248,16 @@ describe("render", () => {
     ).toThrow(ParseError);
   });
 
-  it("should propagate AnalysisError for disallowed identifiers", () => {
+  it("should propagate EvaluationError for disallowed identifiers (analyzeIdentifiers disabled)", () => {
     const TestComponent = (): TestNode => ({
       type: "Test",
       props: {},
       children: [],
     });
 
+    // analyzeIdentifiers is currently disabled in analyze.ts, so bare property access on an
+    // undeclared global like `window` is no longer caught at analysis time — it now fails at
+    // evaluation time instead, once the identifier turns out to be undefined at runtime.
     expect(() =>
       render(
         "<Text>{window.location}</Text>",
@@ -269,7 +272,7 @@ describe("render", () => {
           components: { Text: TestComponent },
         }
       )
-    ).toThrow(AnalysisError);
+    ).toThrow(EvaluationError);
   });
 
   it("should propagate AnalysisError for unknown JSX elements", () => {

package/src/render.ts

@@ -1,7 +1,8 @@
 import { compile as compileInternal } from "./compile/index.js";
-import { evaluate, EvaluateOptions } from "./evaluate/evaluate.js";
+import type { EvaluateOptions } from "./evaluate/evaluate.js";
+import { evaluate } from "./evaluate/evaluate.js";
 import { parse } from "./parse/index.js";
-import { Schema } from "./schema.js";
+import type { Schema } from "./schema.js";
 import { analyze, AnalysisError } from "./analyze/index.js";
 
 /**
@@ -71,7 +72,7 @@ export function render<T = any>(
     throw AnalysisError.fromReport(report);
   }
 
-  return evaluate(compileInternal(ast), schema, options) as T;
+  return evaluate(compileInternal(ast), schema, options);
 }
 
 export function compile<T = any>(source: string, schema: Schema, { minSeverity = 3 }: RenderOptions<T> = {}): string {

package/src/schema.ts

@@ -161,7 +161,7 @@ export function deserializePropertySchemaFromJson(json: string): PropertySchema
     return undefined;
   }
   const validated = zodSchema.safeParse(parsed);
-  if(!validated.success) {
+  if (!validated.success) {
     return undefined;
   }
   return validated.data;

package/src/traverse.ts

@@ -103,55 +103,54 @@ const tsNodeTypes = [
 
 const tsWalkers: Record<string, (node: any, state: any, c: any) => void> = {};
 for (const nodeType of tsNodeTypes) {
-  // eslint-disable-next-line @typescript-eslint/no-empty-function
   tsWalkers[nodeType] = () => {};
 }
 
 const jsxWalkers: ExtendedRecursiveVisitors = {
   JSXElement(node, state, visit) {
-    const element = node as JSXElement;
+    const element = node;
     visit(element.openingElement, state);
     for (const child of element.children) {
       if (isJsxText(child) || isJsxEmptyExpression(child)) {
         continue;
       }
-      visit(child as any, state);
+      visit(child, state);
     }
   },
 
   JSXFragment(node, state, visit) {
-    const fragment = node as JSXFragment;
+    const fragment = node;
     for (const child of fragment.children) {
       if (isJsxText(child) || isJsxEmptyExpression(child)) {
         continue;
       }
-      visit(child as any, state);
+      visit(child, state);
     }
   },
 
   JSXOpeningElement(node, state, visit) {
-    const opening = node as JSXOpeningElement;
+    const opening = node;
     for (const attr of opening.attributes) {
-      visit(attr as any, state);
+      visit(attr, state);
     }
   },
 
   JSXAttribute(node, state, visit) {
-    const attr = node as JSXAttribute;
+    const attr = node;
     if (attr.value) {
-      visit(attr.value as any, state);
+      visit(attr.value, state);
     }
   },
 
   JSXSpreadAttribute(node, state, visit) {
-    const spread = node as JSXSpreadAttribute;
-    visit(spread.argument as any, state);
+    const spread = node;
+    visit(spread.argument, state);
   },
 
   JSXExpressionContainer(node, state, visit) {
-    const container = node as JSXExpressionContainer;
+    const container = node;
     if (!isJsxEmptyExpression(container.expression)) {
-      visit(container.expression as any, state);
+      visit(container.expression, state);
     }
   },
 };

package/src/validate.ts

@@ -1,6 +1,6 @@
 import { analyze, AnalysisError } from "./analyze/index.js";
 import { parse, ParseError } from "./parse/index.js";
-import { Schema } from "./schema.js";
+import type { Schema } from "./schema.js";
 
 export type ValidateResult = { ok: true } | { ok: false; error: AnalysisError | ParseError };