@shopify/react-native-skia 2.0.0-next.4 → 2.0.0

package/apple/RNSkApplePlatformContext.h

@@ -26,7 +26,8 @@ public:
 #if !TARGET_OS_OSX
       : RNSkPlatformContext(jsCallInvoker, [[UIScreen mainScreen] scale]) {
 #else
-      : RNSkPlatformContext(jsCallInvoker, [[NSScreen mainScreen] backingScaleFactor]) {
+      : RNSkPlatformContext(jsCallInvoker,
+                            [[NSScreen mainScreen] backingScaleFactor]) {
 #endif // !TARGET_OS_OSX
 
     // Create screenshot manager

package/apple/RNSkApplePlatformContext.mm

@@ -173,7 +173,8 @@ const TextureInfo RNSkApplePlatformContext::getTexture(sk_sp<SkImage> image) {
   return result;
 }
 
-const TextureInfo RNSkApplePlatformContext::getTexture(sk_sp<SkSurface> surface) {
+const TextureInfo
+RNSkApplePlatformContext::getTexture(sk_sp<SkSurface> surface) {
   GrBackendTexture texture = SkSurfaces::GetBackendTexture(
       surface.get(), SkSurfaces::BackendHandleAccess::kFlushRead);
   TextureInfo result;
@@ -195,7 +196,7 @@ RNSkApplePlatformContext::createVideo(const std::string &url) {
 
 std::shared_ptr<WindowContext>
 RNSkApplePlatformContext::makeContextFromNativeSurface(void *surface, int width,
-                                                     int height) {
+                                                       int height) {
 #if defined(SK_GRAPHITE)
   return DawnContext::getInstance().MakeWindow(surface, width, height);
 #else
@@ -209,7 +210,7 @@ void RNSkApplePlatformContext::raiseError(const std::exception &err) {
 }
 
 sk_sp<SkSurface> RNSkApplePlatformContext::makeOffscreenSurface(int width,
-                                                              int height) {
+                                                                int height) {
 #if defined(SK_GRAPHITE)
   return DawnContext::getInstance().MakeOffscreen(width, height);
 #else
@@ -217,7 +218,8 @@ sk_sp<SkSurface> RNSkApplePlatformContext::makeOffscreenSurface(int width,
 #endif
 }
 
-sk_sp<SkImage> RNSkApplePlatformContext::makeImageFromNativeBuffer(void *buffer) {
+sk_sp<SkImage>
+RNSkApplePlatformContext::makeImageFromNativeBuffer(void *buffer) {
 #if defined(SK_GRAPHITE)
   return DawnContext::getInstance().MakeImageFromBuffer(buffer);
 #else

package/apple/RNSkAppleView.h

@@ -2,9 +2,9 @@
 
 #import <memory>
 
+#import "RNSkApplePlatformContext.h"
 #import "RNSkMetalCanvasProvider.h"
 #import "RNSkView.h"
-#import "RNSkApplePlatformContext.h"
 
 class RNSkBaseAppleView {
 public:

package/apple/RNSkAppleView.mm

@@ -1,8 +1,8 @@
 #import <memory>
 
+#import "RNSkApplePlatformContext.h"
 #import "RNSkMetalCanvasProvider.h"
 #import "RNSkView.h"
-#import "RNSkApplePlatformContext.h"
 
 class RNSkBaseAppleView {
 public:

package/apple/SkiaManager.mm

@@ -41,7 +41,8 @@ static __weak SkiaManager *sharedInstance = nil;
       // Create the RNSkiaManager (cross platform)
       _skManager = std::make_shared<RNSkia::RNSkManager>(
           jsRuntime, jsInvoker,
-          std::make_shared<RNSkia::RNSkApplePlatformContext>(bridge, jsInvoker));
+          std::make_shared<RNSkia::RNSkApplePlatformContext>(bridge,
+                                                             jsInvoker));
     }
   }
   return self;

package/apple/SkiaUIView.h

@@ -10,8 +10,8 @@
 #import <React/RCTUIKit.h>
 #endif // !TARGET_OS_OSX
 
-#import "RNSkManager.h"
 #import "RNSkAppleView.h"
+#import "RNSkManager.h"
 #import "SkiaManager.h"
 
 #if RCT_NEW_ARCH_ENABLED

package/cpp/api/JsiSkCanvas.h

@@ -243,12 +243,12 @@ public:
 
     auto jsiPoints = arguments[1].asObject(runtime).asArray(runtime);
     auto pointsSize = jsiPoints.size(runtime);
-    
+
     // Check if we have at least one point
     if (pointsSize == 0) {
       throw std::invalid_argument("Points array must not be empty");
     }
-    
+
     points.reserve(pointsSize);
 
     for (int i = 0; i < pointsSize; i++) {
@@ -280,12 +280,13 @@ public:
 
     auto jsiCubics = arguments[0].asObject(runtime).asArray(runtime);
     auto cubicsSize = jsiCubics.size(runtime);
-    
+
     // Validate cubic points - must be exactly 12 points
     if (cubicsSize != 12) {
-      throw std::invalid_argument("Cubic points array must contain exactly 12 points");
+      throw std::invalid_argument(
+          "Cubic points array must contain exactly 12 points");
     }
-    
+
     cubics.reserve(cubicsSize);
     for (int i = 0; i < cubicsSize; i++) {
       std::shared_ptr<SkPoint> point = JsiSkPoint::fromValue(
@@ -296,12 +297,13 @@ public:
     if (count >= 2 && !arguments[1].isNull() && !arguments[1].isUndefined()) {
       auto jsiColors = arguments[1].asObject(runtime).asArray(runtime);
       auto colorsSize = jsiColors.size(runtime);
-      
+
       // Validate colors array - must be exactly 4 colors
       if (colorsSize != 4) {
-        throw std::invalid_argument("Colors array must contain exactly 4 colors");
+        throw std::invalid_argument(
+            "Colors array must contain exactly 4 colors");
       }
-      
+
       colors.reserve(colorsSize);
       for (int i = 0; i < colorsSize; i++) {
         SkColor color = JsiSkColor::fromValue(
@@ -313,12 +315,13 @@ public:
     if (count >= 3 && !arguments[2].isNull() && !arguments[2].isUndefined()) {
       auto jsiTexs = arguments[2].asObject(runtime).asArray(runtime);
       auto texsSize = jsiTexs.size(runtime);
-      
+
       // Validate textures array - must be exactly 4 points
       if (texsSize != 4) {
-        throw std::invalid_argument("Texture coordinates array must contain exactly 4 points");
+        throw std::invalid_argument(
+            "Texture coordinates array must contain exactly 4 points");
       }
-      
+
       texs.reserve(texsSize);
       for (int i = 0; i < texsSize; i++) {
         auto point = JsiSkPoint::fromValue(
@@ -330,9 +333,8 @@ public:
     auto paint =
         count >= 4 ? JsiSkPaint::fromValue(runtime, arguments[4]) : nullptr;
     auto blendMode = static_cast<SkBlendMode>(arguments[3].asNumber());
-    _canvas->drawPatch(cubics.data(), colors.empty() ? nullptr : colors.data(), 
-                       texs.empty() ? nullptr : texs.data(), blendMode,
-                       *paint);
+    _canvas->drawPatch(cubics.data(), colors.empty() ? nullptr : colors.data(),
+                       texs.empty() ? nullptr : texs.data(), blendMode, *paint);
     return jsi::Value::undefined();
   }
 
@@ -389,12 +391,13 @@ public:
 
     std::vector<SkGlyphID> glyphs;
     int glyphsSize = static_cast<int>(jsiGlyphs.size(runtime));
-    
+
     // Validate that glyphs and positions arrays have the same size
     if (glyphsSize != pointsSize) {
-      throw std::invalid_argument("Glyphs and positions arrays must have the same length");
+      throw std::invalid_argument(
+          "Glyphs and positions arrays must have the same length");
     }
-    
+
     glyphs.reserve(glyphsSize);
     for (int i = 0; i < glyphsSize; i++) {
       glyphs.push_back(jsiGlyphs.getValueAtIndex(runtime, i).asNumber());
@@ -553,22 +556,24 @@ public:
           runtime, rects.getValueAtIndex(runtime, i).asObject(runtime));
       skRects.push_back(*rect.get());
     }
-    
+
     // Validate transforms and rects have the same size
     if (xformsSize != rectsSize) {
-      throw std::invalid_argument("Transforms and rects arrays must have the same length");
+      throw std::invalid_argument(
+          "Transforms and rects arrays must have the same length");
     }
 
     std::vector<SkColor> colors;
     if (count > 5 && !arguments[5].isUndefined()) {
       auto colorsArray = arguments[5].asObject(runtime).asArray(runtime);
       int colorsSize = static_cast<int>(colorsArray.size(runtime));
-      
+
       // Validate colors array matches the size of sprites and transforms
       if (colorsSize != rectsSize) {
-        throw std::invalid_argument("Colors array must have the same length as rects/transforms");
+        throw std::invalid_argument(
+            "Colors array must have the same length as rects/transforms");
       }
-      
+
       colors.reserve(colorsSize);
       for (int i = 0; i < colorsSize; i++) {
         // Convert from [r,g,b,a] in [0,1] to SkColor
@@ -593,9 +598,8 @@ public:
       sampling = SamplingOptionsFromValue(runtime, arguments[6]);
     }
     _canvas->drawAtlas(atlas.get(), xforms.data(), skRects.data(),
-                       colors.empty() ? nullptr : colors.data(), 
-                       skRects.size(), blendMode, sampling,
-                       nullptr, paint.get());
+                       colors.empty() ? nullptr : colors.data(), skRects.size(),
+                       blendMode, sampling, nullptr, paint.get());
 
     return jsi::Value::undefined();
   }

package/cpp/api/recorder/Drawings.h

@@ -639,12 +639,14 @@ public:
   void draw(DrawingCtx *ctx) {
     // Validate colors array has exactly 4 colors if provided
     if (props.colors.has_value() && props.colors.value().size() != 4) {
-      throw std::invalid_argument("Colors array for patch must have exactly 4 colors");
+      throw std::invalid_argument(
+          "Colors array for patch must have exactly 4 colors");
     }
-    
+
     // Validate texture array has exactly 4 points if provided
     if (props.texture.has_value() && props.texture.value().size() != 4) {
-      throw std::invalid_argument("Texture coordinates array for patch must have exactly 4 points");
+      throw std::invalid_argument(
+          "Texture coordinates array for patch must have exactly 4 points");
     }
 
     // Determine default blend mode based on presence of colors
@@ -686,15 +688,6 @@ public:
   }
 
   void draw(DrawingCtx *ctx) {
-    // Validate array sizes
-    if (props.colors.has_value() && props.colors.value().size() != props.vertices.size()) {
-      throw std::invalid_argument("Colors array must have the same size as vertices array");
-    }
-    
-    if (props.textures.has_value() && props.textures.value().size() != props.vertices.size()) {
-      throw std::invalid_argument("Textures array must have the same size as vertices array");
-    }
-
     // Create vertices using MakeCopy
     auto vertices = SkVertices::MakeCopy(
         props.mode, static_cast<int>(props.vertices.size()),
@@ -930,14 +923,17 @@ public:
     if (props.image) {
       // Validate transforms and sprites have the same size
       if (props.transforms.size() != props.sprites.size()) {
-        throw std::invalid_argument("transforms and sprites arrays must have the same length");
+        throw std::invalid_argument(
+            "transforms and sprites arrays must have the same length");
       }
-      
+
       // Validate colors array matches if provided
-      if (props.colors.has_value() && props.colors.value().size() != props.transforms.size()) {
-        throw std::invalid_argument("colors array must have the same length as transforms/sprites");
+      if (props.colors.has_value() &&
+          props.colors.value().size() != props.transforms.size()) {
+        throw std::invalid_argument(
+            "colors array must have the same length as transforms/sprites");
       }
-      
+
       auto colors =
           props.colors.has_value() ? props.colors.value().data() : nullptr;
       auto blendMode = props.blendMode.value_or(SkBlendMode::kDstOver);

package/cpp/api/recorder/Shaders.h

@@ -239,16 +239,13 @@ public:
   }
 
   void pushShader(DrawingCtx *ctx) {
-    // Validate colors array has at least 2 colors
-    if (props.colors.size() < 2) {
-      throw std::invalid_argument("Colors array must have at least 2 colors");
-    }
-    
     // Validate positions array matches colors array in size
-    if (props.positions.has_value() && props.positions.value().size() != props.colors.size()) {
-      throw std::invalid_argument("Positions array must have the same size as colors array");
+    if (props.positions.has_value() &&
+        props.positions.value().size() != props.colors.size()) {
+      throw std::invalid_argument(
+          "Positions array must have the same size as colors array");
     }
-    
+
     SkMatrix m3 = processTransform(props.matrix, props.transform, props.origin);
     const SkPoint pts[2] = {props.start, props.end};
     auto shader = SkGradientShader::MakeLinear(
@@ -286,16 +283,13 @@ public:
   }
 
   void pushShader(DrawingCtx *ctx) {
-    // Validate colors array has at least 2 colors
-    if (props.colors.size() < 2) {
-      throw std::invalid_argument("Colors array must have at least 2 colors");
-    }
-    
     // Validate positions array matches colors array in size
-    if (props.positions.has_value() && props.positions.value().size() != props.colors.size()) {
-      throw std::invalid_argument("Positions array must have the same size as colors array");
+    if (props.positions.has_value() &&
+        props.positions.value().size() != props.colors.size()) {
+      throw std::invalid_argument(
+          "Positions array must have the same size as colors array");
     }
-    
+
     SkMatrix m3 = processTransform(props.matrix, props.transform, props.origin);
     auto shader = SkGradientShader::MakeRadial(
         props.center, props.radius, props.colors.data(),
@@ -334,16 +328,13 @@ public:
   }
 
   void pushShader(DrawingCtx *ctx) {
-    // Validate colors array has at least 2 colors
-    if (props.colors.size() < 2) {
-      throw std::invalid_argument("Colors array must have at least 2 colors");
-    }
-    
     // Validate positions array matches colors array in size
-    if (props.positions.has_value() && props.positions.value().size() != props.colors.size()) {
-      throw std::invalid_argument("Positions array must have the same size as colors array");
+    if (props.positions.has_value() &&
+        props.positions.value().size() != props.colors.size()) {
+      throw std::invalid_argument(
+          "Positions array must have the same size as colors array");
     }
-    
+
     SkMatrix m3 = processTransform(props.matrix, props.transform, props.origin);
     auto shader = SkGradientShader::MakeSweep(
         props.center.x(), props.center.y(), props.colors.data(),
@@ -385,16 +376,14 @@ public:
   }
 
   void pushShader(DrawingCtx *ctx) {
-    // Validate colors array has at least 2 colors
-    if (props.colors.size() < 2) {
-      throw std::invalid_argument("Colors array must have at least 2 colors");
-    }
-    
+
     // Validate positions array matches colors array in size
-    if (props.positions.has_value() && props.positions.value().size() != props.colors.size()) {
-      throw std::invalid_argument("Positions array must have the same size as colors array");
+    if (props.positions.has_value() &&
+        props.positions.value().size() != props.colors.size()) {
+      throw std::invalid_argument(
+          "Positions array must have the same size as colors array");
     }
-    
+
     SkMatrix m3 = processTransform(props.matrix, props.transform, props.origin);
     auto shader = SkGradientShader::MakeTwoPointConical(
         props.start, props.startRadius, props.end, props.endRadius,

package/package.json

@@ -8,7 +8,7 @@
     "setup-skia-web": "scripts/setup-canvaskit.js"
   },
   "title": "React Native Skia",
-  "version": "2.0.0-next.4",
+  "version": "2.0.0",
   "description": "High-performance React Native Graphics using Skia",
   "main": "lib/module/index.js",
   "react-native": "src/index.ts",
@@ -155,6 +155,5 @@
         }
       ]
     ]
-  },
-  "stableVersion": "0.0.0"
+  }
 }