@pineforge/codegen-pyodide 0.8.0 → 0.9.0

package/pineforge_codegen/codegen/input.py

@@ -206,8 +206,39 @@ class InputHelper:
         default = self._get_input_default(node)
         default_cpp = self._visit_expr(default) if default is not None else "0"
         getter = self._input_type_to_getter(func_name, namespace)
+        # The generic ``input(...)`` overload is typed by its defval in Pine
+        # v6 (an int default yields an int input). The static getter table
+        # cannot see the default, so infer the getter from the default's
+        # literal type here. This matters for TA lengths: ``input(15)`` must
+        # route to ``get_input_int`` so the RMA/EMA ctor receives an int.
+        if func_name == "input" and namespace is None:
+            if isinstance(default, BoolLiteral):
+                getter = "get_input_bool"
+            elif isinstance(default, NumberLiteral):
+                if isinstance(default.value, bool):
+                    getter = "get_input_bool"
+                elif isinstance(default.value, int):
+                    getter = "get_input_int"
+                else:
+                    getter = "get_input_double"
+            elif isinstance(default, StringLiteral):
+                getter = "get_input_string"
+        default_cpp = self._coerce_string_input_default(getter, default_cpp)
         return f'{getter}("{title}", {default_cpp})'
 
+    def _coerce_string_input_default(self, getter: str, default_cpp: str) -> str:
+        """String getters take a ``std::string`` default. An unresolved default
+        (e.g. ``input.string(size.tiny, ...)`` where ``size.tiny`` lowers to
+        ``0``) would pass a null char* and crash ``strlen`` inside the getter,
+        so coerce any non-string default to a safe empty string. (Label size is
+        a visual no-op in the backtest, so the exact default does not affect
+        trading logic.)"""
+        if getter == "get_input_string":
+            stripped = default_cpp.strip()
+            if not stripped.startswith("std::string(") and not stripped.startswith('"'):
+                return 'std::string("")'
+        return default_cpp
+
     def _enforce_enum_declared_before_input_enum(self, node: FuncCall) -> None:
         """Mirror the analyzer: ``input.enum(Enum.member)`` requires Enum to be defined above the call.