react-native-vroom-chart 0.5.0 → 0.7.0

package/cpp/VroomChartHostObject.cpp

@@ -1,6 +1,8 @@
 #include "VroomChartHostObject.h"
 
 #include <cstring>
+#include <string>
+#include <vector>
 
 #include "chart_internal.h"
 #include "vroom/vroom_chart.h"
@@ -27,7 +29,7 @@ ChartHostObject::~ChartHostObject() {
 std::vector<jsi::PropNameID> ChartHostObject::getPropertyNames(
     jsi::Runtime& rt) {
   std::vector<jsi::PropNameID> out;
-  out.reserve(24);
+  out.reserve(29);
   out.push_back(jsi::PropNameID::forAscii(rt, "setCandles"));
   out.push_back(jsi::PropNameID::forAscii(rt, "setSize"));
   out.push_back(jsi::PropNameID::forAscii(rt, "setColor"));
@@ -52,10 +54,78 @@ std::vector<jsi::PropNameID> ChartHostObject::getPropertyNames(
   out.push_back(jsi::PropNameID::forAscii(rt, "setOverlays"));
   out.push_back(jsi::PropNameID::forAscii(rt, "setVWAP"));
   out.push_back(jsi::PropNameID::forAscii(rt, "setBollinger"));
+  out.push_back(jsi::PropNameID::forAscii(rt, "setVolume"));
+  out.push_back(jsi::PropNameID::forAscii(rt, "setVolumeCollapse"));
+  out.push_back(jsi::PropNameID::forAscii(rt, "coordAt"));
+  out.push_back(jsi::PropNameID::forAscii(rt, "setPriceLines"));
+  out.push_back(jsi::PropNameID::forAscii(rt, "hitTestPriceLine"));
+  out.push_back(jsi::PropNameID::forAscii(rt, "setPriceLineHover"));
+  out.push_back(jsi::PropNameID::forAscii(rt, "setPriceLineDrag"));
   out.push_back(jsi::PropNameID::forAscii(rt, "render"));
   return out;
 }
 
+#if defined(__ANDROID__)
+// Android only: RN-Skia is compiled into its own librnskia.so, separate from
+// libvroomchart.so. Constructing a RNSkia::JsiSkPicture directly here (as iOS
+// does, below) gives it a vtable/typeinfo from *our* .so; when RN-Skia's own
+// compiled code later consumes it (e.g. Convertor.h's
+// `getPropertyValue<sk_sp<SkPicture>>`, which does
+// `value.asObject(rt).asHostObject<JsiSkPicture>(rt)` — a dynamic_pointer_cast
+// under the hood), the cast fails cross-.so with "Object is not a HostObject
+// of desired type", because the object's *runtime* type was never actually
+// compiled inside librnskia.so. iOS statically links everything into one
+// binary, so no such mismatch exists there.
+//
+// The fix: build the picture through RN-Skia's own public JS API instead
+// (`Skia.Picture.MakePicture(bytes)`, the same call `require(...).png`-style
+// static pictures use), so the resulting JsiSkPicture is genuinely
+// constructed by librnskia.so's own code. This costs a serialize +
+// re-parse of the picture's draw ops per call — real overhead on a gesture
+// hot path — but is the only ABI-safe option found so far. Revisit if
+// profiling shows this mattering (e.g. a merged-.so build, or a lighter
+// bridge that avoids the round trip).
+#include "include/core/SkData.h"
+
+namespace {
+class SkDataMutableBuffer : public facebook::jsi::MutableBuffer {
+ public:
+  explicit SkDataMutableBuffer(sk_sp<SkData> data) : data_(std::move(data)) {}
+  size_t size() const override { return data_->size(); }
+  uint8_t* data() override {
+    return const_cast<uint8_t*>(
+        static_cast<const uint8_t*>(data_->data()));
+  }
+
+ private:
+  sk_sp<SkData> data_;
+};
+}  // namespace
+
+static facebook::jsi::Value wrapPicture(facebook::jsi::Runtime& rt,
+                                       const sk_sp<SkPicture>& pic) {
+  if (!pic) return facebook::jsi::Value::null();
+  sk_sp<SkData> serialized = pic->serialize();
+  if (!serialized) return facebook::jsi::Value::null();
+
+  auto buffer = std::make_shared<SkDataMutableBuffer>(std::move(serialized));
+  jsi::ArrayBuffer arrayBuffer(rt, buffer);
+
+  auto skiaApi = rt.global().getProperty(rt, "SkiaApi");
+  if (!skiaApi.isObject()) return facebook::jsi::Value::null();
+  auto pictureFactory = skiaApi.asObject(rt).getProperty(rt, "Picture");
+  if (!pictureFactory.isObject()) return facebook::jsi::Value::null();
+  auto makePicture =
+      pictureFactory.asObject(rt).getPropertyAsFunction(rt, "MakePicture");
+
+  // Mirrors JsiSkPictureFactory::MakePicture's expected argument shape: an
+  // object with a `.buffer` property holding the ArrayBuffer (matching a
+  // Uint8Array-like value; the factory ignores everything else about it).
+  jsi::Object arg(rt);
+  arg.setProperty(rt, "buffer", arrayBuffer);
+  return makePicture.call(rt, arg);
+}
+#else
 // Shared helper: wraps a fresh picture for return to JS, with memory pressure
 // reported to Hermes so GC keeps up under gesture-rate churn.
 static facebook::jsi::Value wrapPicture(facebook::jsi::Runtime& rt,
@@ -66,6 +136,7 @@ static facebook::jsi::Value wrapPicture(facebook::jsi::Runtime& rt,
   return JSI_CREATE_HOST_OBJECT_WITH_MEMORY_PRESSURE(rt, host,
                                                      /*context=*/nullptr);
 }
+#endif
 
 jsi::Value ChartHostObject::get(jsi::Runtime& rt,
                                 const jsi::PropNameID& propName) {
@@ -450,46 +521,106 @@ jsi::Value ChartHostObject::get(jsi::Runtime& rt,
   }
 
   if (name == "setRSI") {
-    // setRSI(enabled, period, upperBand, lowerBand, maEnabled, maPeriod) —
-    // configures the RSI pane. No render; the next render() picks it up.
+    // setRSI({enabled, period, upperBand, lowerBand, maPeriod, maKind,
+    // maVisible, lineColor, lineWidth, lineVisible, maColor, maWidth,
+    // bandColor, bandsVisible}) — configures the RSI pane. No render; the next
+    // render() picks it up.
     return jsi::Function::createFromHostFunction(
         rt,
         jsi::PropNameID::forAscii(rt, "setRSI"),
-        6,
-        [this](jsi::Runtime& /*rt2*/,
+        1,
+        [this](jsi::Runtime& rt2,
                const jsi::Value& /*thisVal*/,
                const jsi::Value* args,
                size_t count) -> jsi::Value {
-          if (count < 6) return jsi::Value::undefined();
-          const bool enabled = args[0].asBool();
-          const int period = static_cast<int>(args[1].asNumber());
-          const double upper = args[2].asNumber();
-          const double lower = args[3].asNumber();
-          const bool ma_enabled = args[4].asBool();
-          const int ma_period = static_cast<int>(args[5].asNumber());
-          vroom_chart_set_rsi(chart_, enabled, period, upper, lower, ma_enabled,
-                              ma_period);
+          if (count < 1 || !args[0].isObject()) return jsi::Value::undefined();
+          auto s = args[0].asObject(rt2);
+          VroomRSI cfg{};
+          cfg.enabled = s.getProperty(rt2, "enabled").asBool() ? 1 : 0;
+          cfg.period = static_cast<int32_t>(
+              s.getProperty(rt2, "period").asNumber());
+          cfg.upper_band = s.getProperty(rt2, "upperBand").asNumber();
+          cfg.lower_band = s.getProperty(rt2, "lowerBand").asNumber();
+          cfg.ma_period = static_cast<int32_t>(
+              s.getProperty(rt2, "maPeriod").asNumber());
+          cfg.ma_kind = static_cast<int32_t>(
+              s.getProperty(rt2, "maKind").asNumber());
+          cfg.ma_visible = s.getProperty(rt2, "maVisible").asBool() ? 1 : 0;
+          cfg.line_color = static_cast<uint32_t>(
+              s.getProperty(rt2, "lineColor").asNumber());
+          cfg.line_width = static_cast<float>(
+              s.getProperty(rt2, "lineWidth").asNumber());
+          cfg.line_visible = s.getProperty(rt2, "lineVisible").asBool() ? 1 : 0;
+          cfg.ma_color = static_cast<uint32_t>(
+              s.getProperty(rt2, "maColor").asNumber());
+          cfg.ma_width = static_cast<float>(
+              s.getProperty(rt2, "maWidth").asNumber());
+          cfg.band_color = static_cast<uint32_t>(
+              s.getProperty(rt2, "bandColor").asNumber());
+          cfg.bands_visible =
+              s.getProperty(rt2, "bandsVisible").asBool() ? 1 : 0;
+          vroom_chart_set_rsi(chart_, &cfg);
           return jsi::Value::undefined();
         });
   }
 
   if (name == "setMACD") {
-    // setMACD(enabled, fast, slow, signal) — configures the MACD pane. No
-    // render; the next render() picks it up.
+    // setMACD({enabled, fast, slow, signal, source, maKind, signalMaKind,
+    // lineColor, lineWidth, lineVisible, signalColor, signalWidth,
+    // signalVisible, histVisible, histUpColor, histUpFadingColor,
+    // histDownColor, histDownFadingColor, zeroColor, zeroVisible}) — configures
+    // the MACD pane. No render; the next render() picks it up.
     return jsi::Function::createFromHostFunction(
         rt,
         jsi::PropNameID::forAscii(rt, "setMACD"),
-        4,
-        [this](jsi::Runtime& /*rt2*/,
+        1,
+        [this](jsi::Runtime& rt2,
                const jsi::Value& /*thisVal*/,
                const jsi::Value* args,
                size_t count) -> jsi::Value {
-          if (count < 4) return jsi::Value::undefined();
-          const bool enabled = args[0].asBool();
-          const int fast = static_cast<int>(args[1].asNumber());
-          const int slow = static_cast<int>(args[2].asNumber());
-          const int signal = static_cast<int>(args[3].asNumber());
-          vroom_chart_set_macd(chart_, enabled, fast, slow, signal);
+          if (count < 1 || !args[0].isObject()) return jsi::Value::undefined();
+          auto s = args[0].asObject(rt2);
+          VroomMACD cfg{};
+          cfg.enabled = s.getProperty(rt2, "enabled").asBool() ? 1 : 0;
+          cfg.fast = static_cast<int32_t>(
+              s.getProperty(rt2, "fast").asNumber());
+          cfg.slow = static_cast<int32_t>(
+              s.getProperty(rt2, "slow").asNumber());
+          cfg.signal = static_cast<int32_t>(
+              s.getProperty(rt2, "signal").asNumber());
+          cfg.source = static_cast<int32_t>(
+              s.getProperty(rt2, "source").asNumber());
+          cfg.ma_kind = static_cast<int32_t>(
+              s.getProperty(rt2, "maKind").asNumber());
+          cfg.signal_ma_kind = static_cast<int32_t>(
+              s.getProperty(rt2, "signalMaKind").asNumber());
+          cfg.line_color = static_cast<uint32_t>(
+              s.getProperty(rt2, "lineColor").asNumber());
+          cfg.line_width = static_cast<float>(
+              s.getProperty(rt2, "lineWidth").asNumber());
+          cfg.line_visible =
+              s.getProperty(rt2, "lineVisible").asBool() ? 1 : 0;
+          cfg.signal_color = static_cast<uint32_t>(
+              s.getProperty(rt2, "signalColor").asNumber());
+          cfg.signal_width = static_cast<float>(
+              s.getProperty(rt2, "signalWidth").asNumber());
+          cfg.signal_visible =
+              s.getProperty(rt2, "signalVisible").asBool() ? 1 : 0;
+          cfg.hist_visible =
+              s.getProperty(rt2, "histVisible").asBool() ? 1 : 0;
+          cfg.hist_up_color = static_cast<uint32_t>(
+              s.getProperty(rt2, "histUpColor").asNumber());
+          cfg.hist_up_fading_color = static_cast<uint32_t>(
+              s.getProperty(rt2, "histUpFadingColor").asNumber());
+          cfg.hist_down_color = static_cast<uint32_t>(
+              s.getProperty(rt2, "histDownColor").asNumber());
+          cfg.hist_down_fading_color = static_cast<uint32_t>(
+              s.getProperty(rt2, "histDownFadingColor").asNumber());
+          cfg.zero_color = static_cast<uint32_t>(
+              s.getProperty(rt2, "zeroColor").asNumber());
+          cfg.zero_visible =
+              s.getProperty(rt2, "zeroVisible").asBool() ? 1 : 0;
+          vroom_chart_set_macd(chart_, &cfg);
           return jsi::Value::undefined();
         });
   }
@@ -533,21 +664,26 @@ jsi::Value ChartHostObject::get(jsi::Runtime& rt,
   }
 
   if (name == "setVWAP") {
-    // setVWAP(enabled, resetOffsetMin, color, width) — session VWAP overlay.
+    // setVWAP({enabled, resetOffsetMin, color, width}) — session VWAP overlay.
     return jsi::Function::createFromHostFunction(
         rt,
         jsi::PropNameID::forAscii(rt, "setVWAP"),
-        4,
-        [this](jsi::Runtime& /*rt2*/,
+        1,
+        [this](jsi::Runtime& rt2,
                const jsi::Value& /*thisVal*/,
                const jsi::Value* args,
                size_t count) -> jsi::Value {
-          if (count < 4) return jsi::Value::undefined();
-          const bool enabled = args[0].asBool();
-          const int reset = static_cast<int>(args[1].asNumber());
-          const uint32_t color = static_cast<uint32_t>(args[2].asNumber());
-          const float width = static_cast<float>(args[3].asNumber());
-          vroom_chart_set_vwap(chart_, enabled, reset, color, width);
+          if (count < 1 || !args[0].isObject()) return jsi::Value::undefined();
+          auto s = args[0].asObject(rt2);
+          VroomVWAP cfg{};
+          cfg.enabled = s.getProperty(rt2, "enabled").asBool() ? 1 : 0;
+          cfg.reset_offset_min = static_cast<int32_t>(
+              s.getProperty(rt2, "resetOffsetMin").asNumber());
+          cfg.color = static_cast<uint32_t>(
+              s.getProperty(rt2, "color").asNumber());
+          cfg.width = static_cast<float>(
+              s.getProperty(rt2, "width").asNumber());
+          vroom_chart_set_vwap(chart_, &cfg);
           return jsi::Value::undefined();
         });
   }
@@ -598,6 +734,205 @@ jsi::Value ChartHostObject::get(jsi::Runtime& rt,
         });
   }
 
+  if (name == "setVolume") {
+    // setVolume({enabled, heightFrac, opacity, radiusPx, upColor, downColor})
+    // — the volume bars. Negative floats / zero colors inherit the theme. No
+    // render; the next render() picks it up.
+    return jsi::Function::createFromHostFunction(
+        rt,
+        jsi::PropNameID::forAscii(rt, "setVolume"),
+        1,
+        [this](jsi::Runtime& rt2,
+               const jsi::Value& /*thisVal*/,
+               const jsi::Value* args,
+               size_t count) -> jsi::Value {
+          if (count < 1 || !args[0].isObject()) return jsi::Value::undefined();
+          auto s = args[0].asObject(rt2);
+          VroomVolume cfg{};
+          cfg.enabled = s.getProperty(rt2, "enabled").asBool() ? 1 : 0;
+          cfg.height_frac = static_cast<float>(
+              s.getProperty(rt2, "heightFrac").asNumber());
+          cfg.opacity = static_cast<float>(
+              s.getProperty(rt2, "opacity").asNumber());
+          cfg.radius_px = static_cast<float>(
+              s.getProperty(rt2, "radiusPx").asNumber());
+          cfg.up_color = static_cast<uint32_t>(
+              s.getProperty(rt2, "upColor").asNumber());
+          cfg.down_color = static_cast<uint32_t>(
+              s.getProperty(rt2, "downColor").asNumber());
+          vroom_chart_set_volume(chart_, &cfg);
+          return jsi::Value::undefined();
+        });
+  }
+
+  if (name == "setVolumeCollapse") {
+    // setVolumeCollapse(t, easing) — staggered collapse of the volume bars. `t`
+    // is LINEAR progress; the core eases each bar over its own window.
+    return jsi::Function::createFromHostFunction(
+        rt,
+        jsi::PropNameID::forAscii(rt, "setVolumeCollapse"),
+        2,
+        [this](jsi::Runtime& /*rt2*/,
+               const jsi::Value& /*thisVal*/,
+               const jsi::Value* args,
+               size_t count) -> jsi::Value {
+          if (count < 2) return jsi::Value::undefined();
+          vroom_chart_set_volume_collapse(
+              chart_,
+              static_cast<float>(args[0].asNumber()),
+              static_cast<int32_t>(args[1].asNumber()));
+          return jsi::Value::undefined();
+        });
+  }
+
+  if (name == "coordAt") {
+    // coordAt(x, y) -> { timeMs, price } | null. The continuous data coordinate
+    // at a pixel — not snapped to a candle slot. Null when there are no candles
+    // or the viewport is degenerate. No rendering.
+    return jsi::Function::createFromHostFunction(
+        rt,
+        jsi::PropNameID::forAscii(rt, "coordAt"),
+        2,
+        [this](jsi::Runtime& rt2,
+               const jsi::Value& /*thisVal*/,
+               const jsi::Value* args,
+               size_t count) -> jsi::Value {
+          if (count < 2) return jsi::Value::null();
+          VroomCoord c{};
+          if (!vroom_chart_coord_at(chart_,
+                                    static_cast<float>(args[0].asNumber()),
+                                    static_cast<float>(args[1].asNumber()), &c)) {
+            return jsi::Value::null();
+          }
+          jsi::Object obj(rt2);
+          obj.setProperty(rt2, "timeMs", static_cast<double>(c.time_ms));
+          obj.setProperty(rt2, "price", c.price);
+          return obj;
+        });
+  }
+
+  if (name == "setPriceLines") {
+    // setPriceLines({ lines: [{ price, color, width, lineStyle, text, quantity,
+    // flags }, ...], bodyBg, fontSizePx, lineLengthFrac, align, hoverBoost }) —
+    // replaces the full set of price status lines. No render; the next render()
+    // picks it up.
+    return jsi::Function::createFromHostFunction(
+        rt,
+        jsi::PropNameID::forAscii(rt, "setPriceLines"),
+        1,
+        [this](jsi::Runtime& rt2,
+               const jsi::Value& /*thisVal*/,
+               const jsi::Value* args,
+               size_t count) -> jsi::Value {
+          if (count < 1 || !args[0].isObject()) return jsi::Value::undefined();
+          auto cfg = args[0].asObject(rt2);
+          auto lines_val = cfg.getProperty(rt2, "lines");
+          if (!lines_val.isObject()) return jsi::Value::undefined();
+          auto lines_obj = lines_val.asObject(rt2);
+          if (!lines_obj.isArray(rt2)) return jsi::Value::undefined();
+          auto arr = lines_obj.asArray(rt2);
+          const size_t len = arr.size(rt2);
+          std::vector<VroomPriceLine> lines(len);
+          // Label storage, kept alive until set_price_lines has copied it.
+          std::vector<std::string> texts(len);
+          std::vector<std::string> quantities(len);
+          for (size_t i = 0; i < len; ++i) {
+            auto l = arr.getValueAtIndex(rt2, i).asObject(rt2);
+            lines[i].price = l.getProperty(rt2, "price").asNumber();
+            lines[i].color = static_cast<uint32_t>(
+                l.getProperty(rt2, "color").asNumber());
+            lines[i].width = static_cast<float>(
+                l.getProperty(rt2, "width").asNumber());
+            lines[i].line_style = static_cast<int32_t>(
+                l.getProperty(rt2, "lineStyle").asNumber());
+            texts[i] = l.getProperty(rt2, "text").asString(rt2).utf8(rt2);
+            quantities[i] =
+                l.getProperty(rt2, "quantity").asString(rt2).utf8(rt2);
+            lines[i].text = texts[i].c_str();
+            lines[i].quantity = quantities[i].c_str();
+            lines[i].flags = static_cast<int32_t>(
+                l.getProperty(rt2, "flags").asNumber());
+          }
+          VroomPriceLineStyle style{};
+          style.body_bg = static_cast<uint32_t>(
+              cfg.getProperty(rt2, "bodyBg").asNumber());
+          style.font_size_px = static_cast<float>(
+              cfg.getProperty(rt2, "fontSizePx").asNumber());
+          style.line_length_frac = static_cast<float>(
+              cfg.getProperty(rt2, "lineLengthFrac").asNumber());
+          style.align = static_cast<int32_t>(
+              cfg.getProperty(rt2, "align").asNumber());
+          style.hover_boost = static_cast<float>(
+              cfg.getProperty(rt2, "hoverBoost").asNumber());
+          vroom_chart_set_price_lines(chart_, lines.data(), lines.size(), &style);
+          return jsi::Value::undefined();
+        });
+  }
+
+  if (name == "hitTestPriceLine") {
+    // hitTestPriceLine(x, y) -> { index, part } | null. `part` is 0 for the line
+    // or its label body (the drag target) and 1 for the close button. Cheap
+    // enough to call at gesture rate — no rendering.
+    return jsi::Function::createFromHostFunction(
+        rt,
+        jsi::PropNameID::forAscii(rt, "hitTestPriceLine"),
+        2,
+        [this](jsi::Runtime& rt2,
+               const jsi::Value& /*thisVal*/,
+               const jsi::Value* args,
+               size_t count) -> jsi::Value {
+          if (count < 2) return jsi::Value::null();
+          int32_t index = -1, part = -1;
+          if (!vroom_chart_hit_test_price_line(
+                  chart_, static_cast<float>(args[0].asNumber()),
+                  static_cast<float>(args[1].asNumber()), &index, &part)) {
+            return jsi::Value::null();
+          }
+          jsi::Object obj(rt2);
+          obj.setProperty(rt2, "index", index);
+          obj.setProperty(rt2, "part", part);
+          return obj;
+        });
+  }
+
+  if (name == "setPriceLineHover") {
+    // setPriceLineHover(index, part) — highlight a price line's segment; -1
+    // clears. No hover on touch, so this exists for parity/pointer devices.
+    return jsi::Function::createFromHostFunction(
+        rt,
+        jsi::PropNameID::forAscii(rt, "setPriceLineHover"),
+        2,
+        [this](jsi::Runtime& /*rt2*/,
+               const jsi::Value& /*thisVal*/,
+               const jsi::Value* args,
+               size_t count) -> jsi::Value {
+          if (count < 2) return jsi::Value::undefined();
+          vroom_chart_set_price_line_hover(
+              chart_, static_cast<int32_t>(args[0].asNumber()),
+              static_cast<int32_t>(args[1].asNumber()));
+          return jsi::Value::undefined();
+        });
+  }
+
+  if (name == "setPriceLineDrag") {
+    // setPriceLineDrag(index, price) — live drag preview; index -1 ends it. The
+    // committed price is untouched: restate setPriceLines to apply a move.
+    return jsi::Function::createFromHostFunction(
+        rt,
+        jsi::PropNameID::forAscii(rt, "setPriceLineDrag"),
+        2,
+        [this](jsi::Runtime& /*rt2*/,
+               const jsi::Value& /*thisVal*/,
+               const jsi::Value* args,
+               size_t count) -> jsi::Value {
+          if (count < 2) return jsi::Value::undefined();
+          vroom_chart_set_price_line_drag(
+              chart_, static_cast<int32_t>(args[0].asNumber()),
+              args[1].asNumber());
+          return jsi::Value::undefined();
+        });
+  }
+
   if (name == "render") {
     return jsi::Function::createFromHostFunction(
         rt,

package/cpp/VroomJsiInstaller.cpp

@@ -40,7 +40,17 @@ static void ensureAxisTypeface(jsi::Runtime& runtime) {
   if (!ctx) return;
   auto mgr = ctx->createFontMgr();
   if (!mgr) return;
-  auto tf = mgr->matchFamilyStyle(nullptr, SkFontStyle());  // system default
+
+  // A null family name is meant to request "the platform default" — CoreText
+  // (iOS) honors that, but Android's SkFontMgr_New_Android does not: it
+  // returns null unless given an actual family name, even though its
+  // underlying font set (sans-serif, arial, ...) is perfectly populated. Fall
+  // back to "sans-serif" (Android's standard generic-family alias) whenever
+  // the null-family lookup comes back empty.
+  auto tf = mgr->matchFamilyStyle(nullptr, SkFontStyle());
+  if (!tf) {
+    tf = mgr->matchFamilyStyle("sans-serif", SkFontStyle());
+  }
   if (!tf) return;
   vroom::set_axis_typeface(tf);
   done = true;

package/cpp/VroomSkiaContext.cpp

@@ -12,13 +12,21 @@ std::shared_ptr<RNSkia::RNSkPlatformContext> getRNSkContext(
   if (!obj.isHostObject(runtime)) return nullptr;
   auto host = obj.asHostObject(runtime);
 
-  auto base = std::dynamic_pointer_cast<RNSkia::JsiSkHostObject>(host);
-  if (!base) return nullptr;
-
-  // The accessor class adds no fields; static_cast is safe even though the
-  // runtime type is JsiSkApi (a different JsiSkHostObject subclass). We only
-  // call the inherited non-virtual getContext() through this view.
-  auto* accessor = static_cast<HostObjectAccessor*>(base.get());
+  // `global.SkiaApi` is always installed by RN-Skia's own
+  // RNSkManager::installBindings() as exactly a `JsiSkApi` host object — never
+  // any other JsiSkHostObject subclass — so this cast is safe as a plain
+  // static_cast, without any RTTI check. We deliberately avoid
+  // dynamic_pointer_cast here: on Android, RN-Skia is compiled into its own
+  // librnskia.so, separate from libvroomchart.so, so a `JsiSkApi` constructed
+  // by librnskia.so's code carries typeinfo from that .so; a dynamic_cast
+  // performed here (in libvroomchart.so) sees an unmerged RTTI record for
+  // "the same" class and always fails, even though the object is perfectly
+  // valid to use. (Same underlying issue as the JsiSkPicture cross-.so cast
+  // documented in VroomChartHostObject.cpp — but there RN-Skia's own compiled
+  // code does the cast on an object *we* construct, so we can't avoid it;
+  // here *we* do the cast on an object *they* construct, so we can just skip
+  // the runtime check we don't need.)
+  auto* accessor = static_cast<HostObjectAccessor*>(host.get());
   return accessor->getContext();
 }