@shortkitsdk/react-native 0.1.0 → 0.2.0

package/android/src/main/java/com/shortkit/reactnative/ShortKitFeedView.kt

@@ -2,6 +2,8 @@ package com.shortkit.reactnative
 
 import android.content.Context
 import android.content.ContextWrapper
+import android.os.Handler
+import android.os.Looper
 import android.util.Log
 import android.view.View
 import android.view.ViewGroup
@@ -36,6 +38,7 @@ class ShortKitFeedView(context: Context) : FrameLayout(context) {
 
     private var feedFragment: ShortKitFeedFragment? = null
     private var fragmentContainerId: Int = View.generateViewId()
+    private val handler = Handler(Looper.getMainLooper())
 
     init {
         id = fragmentContainerId
@@ -54,9 +57,12 @@ class ShortKitFeedView(context: Context) : FrameLayout(context) {
     /** Overlay for the upcoming cell (nativeID="overlay-next"). */
     private var nextOverlayView: View? = null
 
-    /** The page index from which the current scroll gesture started. */
+    /** The page index used for overlay transform calculations. */
     private var currentPage: Int = 0
 
+    /** Deferred page update to avoid flashing stale metadata. */
+    private var pageUpdateRunnable: Runnable? = null
+
     // -----------------------------------------------------------------------
     // Lifecycle
     // -----------------------------------------------------------------------
@@ -145,6 +151,8 @@ class ShortKitFeedView(context: Context) : FrameLayout(context) {
     }
 
     private fun teardownScrollTracking() {
+        pageUpdateRunnable?.let { handler.removeCallbacks(it) }
+        pageUpdateRunnable = null
         pageChangeCallback?.let { viewPager?.unregisterOnPageChangeCallback(it) }
         pageChangeCallback = null
         viewPager = null
@@ -162,6 +170,41 @@ class ShortKitFeedView(context: Context) : FrameLayout(context) {
         val cellHeight = height.toFloat()
         if (cellHeight <= 0) return
 
+        // Detect page change, but DEFER updating currentPage.
+        //
+        // Why: when the scroll settles on a new page, overlay-current still
+        // shows the OLD page's metadata (React hasn't processed OVERLAY_ACTIVATE
+        // yet). If we update currentPage immediately, delta snaps to 0 and
+        // overlay-current becomes visible with stale data.
+        //
+        // By deferring ~80ms, overlay-next (which already shows the correct
+        // data via NextOverlayProvider) stays visible at y=0 while React
+        // processes the state update.
+        if (positionOffset == 0f) {
+            if (position != currentPage && pageUpdateRunnable == null) {
+                val targetPage = position
+                val runnable = Runnable {
+                    currentPage = targetPage
+                    pageUpdateRunnable = null
+                    // Reapply overlay transforms now that currentPage is updated.
+                    // Without this, overlay-next (static NextOverlayProvider state)
+                    // stays visible at y=0 while overlay-current (live state) stays
+                    // hidden — no scroll event fires to trigger handleScrollOffset.
+                    val h = height.toFloat()
+                    currentOverlayView?.translationY = 0f
+                    nextOverlayView?.translationY = h
+                }
+                pageUpdateRunnable = runnable
+                handler.postDelayed(runnable, 80)
+            }
+        } else if (pageUpdateRunnable != null) {
+            // User is scrolling again — apply pending update immediately
+            // so transforms stay aligned for the new gesture.
+            handler.removeCallbacks(pageUpdateRunnable!!)
+            pageUpdateRunnable = null
+            currentPage = position
+        }
+
         // positionOffset is 0.0 when settled, approaches 1.0 during forward scroll.
         // position is the index of the page currently filling most of the screen.
         val delta: Float = if (position >= currentPage) {
@@ -172,11 +215,6 @@ class ShortKitFeedView(context: Context) : FrameLayout(context) {
             -(cellHeight - positionOffsetPixels.toFloat())
         }
 
-        // Update currentPage when the scroll settles
-        if (positionOffset == 0f) {
-            currentPage = position
-        }
-
         // Find the overlay views if not cached
         if (currentOverlayView == null || nextOverlayView == null) {
             findOverlayViews()
@@ -196,23 +234,40 @@ class ShortKitFeedView(context: Context) : FrameLayout(context) {
     }
 
     /**
-     * Find the sibling RN overlay views by nativeID. In the Fabric hierarchy,
-     * the ShortKitFeedView and the overlay containers are children of the
-     * same parent (wrapped by ShortKitFeed.tsx with overflow: hidden).
+     * Find the sibling RN overlay views by nativeID.
+     *
+     * In Fabric interop, this view may be wrapped in an intermediate
+     * ViewGroup, so `getParent()` may not be the React `<View>` container
+     * from ShortKitFeed.tsx. We walk up the ancestor chain until we find
+     * the overlays.
      */
     private fun findOverlayViews() {
-        val parent = parent as? ViewGroup ?: return
-
-        for (i in 0 until parent.childCount) {
-            val sibling = parent.getChildAt(i)
-            if (sibling === this) continue
-            if (currentOverlayView == null) {
-                currentOverlayView = ReactFindViewUtil.findView(sibling, "overlay-current")
-            }
-            if (nextOverlayView == null) {
-                nextOverlayView = ReactFindViewUtil.findView(sibling, "overlay-next")
+        var ancestor = parent as? ViewGroup ?: return
+        while (true) {
+            for (i in 0 until ancestor.childCount) {
+                val child = ancestor.getChildAt(i)
+                if (child === this || isOwnAncestor(child)) continue
+
+                if (currentOverlayView == null) {
+                    currentOverlayView = ReactFindViewUtil.findView(child, "overlay-current")
+                }
+                if (nextOverlayView == null) {
+                    nextOverlayView = ReactFindViewUtil.findView(child, "overlay-next")
+                }
             }
+            if (currentOverlayView != null && nextOverlayView != null) return
+            ancestor = ancestor.parent as? ViewGroup ?: return
+        }
+    }
+
+    /** Check if the given view is an ancestor of this view. */
+    private fun isOwnAncestor(view: View): Boolean {
+        var current: ViewParent? = parent
+        while (current != null) {
+            if (current === view) return true
+            current = current.parent
         }
+        return false
     }
 
     /** Recursively find the first [ViewPager2] in a view hierarchy. */

package/android/src/main/java/com/shortkit/reactnative/ShortKitModule.kt

@@ -11,12 +11,8 @@ import com.shortkit.FeedConfig
 import com.shortkit.FeedHeight
 import com.shortkit.FeedTransitionPhase
 import com.shortkit.JsonValue
-import com.shortkit.OverlayActionDelegate
 import com.shortkit.ShortKit
-import com.shortkit.ShortKitDelegate
-import com.shortkit.ShortKitError
 import com.shortkit.ShortKitPlayer
-import com.shortkit.SurveyOption
 import com.shortkit.VideoOverlayMode
 import com.shortkit.CarouselOverlayMode
 import com.shortkit.SurveyOverlayMode
@@ -30,8 +26,7 @@ import org.json.JSONArray
 import org.json.JSONObject
 
 class ShortKitModule(reactContext: ReactApplicationContext) :
-    NativeShortKitModuleSpec(reactContext),
-    ShortKitDelegate {
+    NativeShortKitModuleSpec(reactContext) {
 
     companion object {
         const val NAME = "ShortKitModule"
@@ -116,9 +111,6 @@ class ShortKitModule(reactContext: ReactApplicationContext) :
             clientAppVersion = clientAppVersion,
             customDimensions = dims
         )
-        sdk.delegate = this
-        sdk.overlayActionDelegate = overlayDelegate
-
         this.shortKit = sdk
         shared = this
 
@@ -219,102 +211,6 @@ class ShortKitModule(reactContext: ReactApplicationContext) :
         shortKit?.player?.setMaxBitrate(bitrate.toInt())
     }
 
-    // -----------------------------------------------------------------------
-    // ShortKitDelegate
-    // -----------------------------------------------------------------------
-
-    override fun onError(error: ShortKitError) {
-        val params = Arguments.createMap().apply {
-            when (error) {
-                is ShortKitError.NetworkError -> {
-                    putString("code", "network_error")
-                    putString("message", error.cause?.localizedMessage ?: "Network error")
-                }
-                is ShortKitError.ApiError -> {
-                    putString("code", "api_error")
-                    putString("message", error.message ?: "API error")
-                }
-                is ShortKitError.PlayerError -> {
-                    putString("code", error.code)
-                    putString("message", error.message ?: "Player error")
-                }
-                is ShortKitError.ConfigError -> {
-                    putString("code", "config_error")
-                    putString("message", error.message ?: "Config error")
-                }
-                is ShortKitError.AuthError -> {
-                    putString("code", "auth_error")
-                    putString("message", "Invalid API key")
-                }
-            }
-        }
-        sendEvent("onError", params)
-    }
-
-    override fun onShareTapped(item: ContentItem) {
-        val params = Arguments.createMap().apply {
-            putString("item", serializeContentItemToJSON(item))
-        }
-        sendEvent("onShareTapped", params)
-    }
-
-    override fun onSurveyResponse(surveyId: String, option: SurveyOption) {
-        val params = Arguments.createMap().apply {
-            putString("surveyId", surveyId)
-            putString("optionId", option.id)
-            putString("optionText", option.text)
-        }
-        sendEvent("onSurveyResponse", params)
-    }
-
-    // We implement the optional delegate methods as no-ops; events are handled
-    // via StateFlow/SharedFlow subscriptions instead.
-    override fun onFeedReady() {}
-    override fun onContentChanged(item: ContentItem) {}
-    override fun onFeedEmpty() {}
-
-    // -----------------------------------------------------------------------
-    // OverlayActionDelegate (separate object to avoid signature clash with
-    // ShortKitDelegate.onShareTapped)
-    // -----------------------------------------------------------------------
-
-    private val overlayDelegate = object : OverlayActionDelegate {
-        override fun onReadMoreTapped(item: ContentItem) {
-            val params = Arguments.createMap().apply {
-                putString("item", serializeContentItemToJSON(item))
-            }
-            sendEvent("onArticleTapped", params)
-        }
-
-        override fun onShareTapped(item: ContentItem) {
-            val params = Arguments.createMap().apply {
-                putString("item", serializeContentItemToJSON(item))
-            }
-            sendEvent("onOverlayShareTapped", params)
-        }
-
-        override fun onCommentTapped(item: ContentItem) {
-            val params = Arguments.createMap().apply {
-                putString("item", serializeContentItemToJSON(item))
-            }
-            sendEvent("onCommentTapped", params)
-        }
-
-        override fun onSaveTapped(item: ContentItem) {
-            val params = Arguments.createMap().apply {
-                putString("item", serializeContentItemToJSON(item))
-            }
-            sendEvent("onSaveTapped", params)
-        }
-
-        override fun onLikeTapped(item: ContentItem) {
-            val params = Arguments.createMap().apply {
-                putString("item", serializeContentItemToJSON(item))
-            }
-            sendEvent("onLikeTapped", params)
-        }
-    }
-
     // -----------------------------------------------------------------------
     // Overlay lifecycle events (called by Fabric view)
     // -----------------------------------------------------------------------

package/ios/ShortKitBridge.swift

@@ -6,7 +6,7 @@ import ShortKit
 ///
 /// Holds the `ShortKit` instance, subscribes to all Combine publishers on
 /// `ShortKitPlayer`, and forwards events to JS via the delegate protocol.
-@objc public class ShortKitBridge: NSObject, ShortKitDelegate {
+@objc public class ShortKitBridge: NSObject {
 
     // MARK: - Shared instance (accessed by Fabric view manager in Task 13)
 
@@ -41,7 +41,6 @@ import ShortKit
             clientAppVersion: clientAppVersion,
             customDimensions: dimensions
         )
-        sdk.delegate = self
         self.shortKit = sdk
 
         ShortKitBridge.shared = self
@@ -141,33 +140,6 @@ import ShortKit
         shortKit?.player.setMaxBitrate(Int(bitrate))
     }
 
-    // MARK: - ShortKitDelegate
-
-    public func shortKit(_ shortKit: ShortKit, didEncounterError error: ShortKitError) {
-        let body: [String: Any]
-        switch error {
-        case .networkError(let underlying):
-            body = ["code": "network_error", "message": underlying.localizedDescription]
-        case .playbackError(let code, let message):
-            body = ["code": code, "message": message]
-        case .authError:
-            body = ["code": "auth_error", "message": "Invalid API key"]
-        }
-        emitOnMain("onError", body: body)
-    }
-
-    public func shortKit(_ shortKit: ShortKit, didTapShareFor item: ContentItem) {
-        emitOnMain("onShareTapped", body: ["item": serializeContentItemToJSON(item)])
-    }
-
-    public func shortKit(_ shortKit: ShortKit, didRespondToSurvey surveyId: String, with option: SurveyOption) {
-        emitOnMain("onSurveyResponse", body: [
-            "surveyId": surveyId,
-            "optionId": option.id,
-            "optionText": option.text
-        ])
-    }
-
     // MARK: - Combine Subscriptions
 
     private func subscribeToPublishers(_ player: ShortKitPlayer) {
@@ -335,31 +307,6 @@ import ShortKit
         emitOnMain(name, body: body)
     }
 
-    /// Emit an article tap event.
-    public func emitArticleTapped(_ item: ContentItem) {
-        emitOnMain("onArticleTapped", body: ["item": serializeContentItemToJSON(item)])
-    }
-
-    /// Emit a comment tap event.
-    public func emitCommentTapped(_ item: ContentItem) {
-        emitOnMain("onCommentTapped", body: ["item": serializeContentItemToJSON(item)])
-    }
-
-    /// Emit an overlay share tap event.
-    public func emitOverlayShareTapped(_ item: ContentItem) {
-        emitOnMain("onOverlayShareTapped", body: ["item": serializeContentItemToJSON(item)])
-    }
-
-    /// Emit a save tap event.
-    public func emitSaveTapped(_ item: ContentItem) {
-        emitOnMain("onSaveTapped", body: ["item": serializeContentItemToJSON(item)])
-    }
-
-    /// Emit a like tap event.
-    public func emitLikeTapped(_ item: ContentItem) {
-        emitOnMain("onLikeTapped", body: ["item": serializeContentItemToJSON(item)])
-    }
-
     // MARK: - Content Item Serialization
 
     /// Serialize a ContentItem to a JSON string for bridge transport.

package/ios/ShortKitFeedView.swift

@@ -31,8 +31,10 @@ import ShortKit
     private weak var currentOverlayView: UIView?
     /// Overlay for the upcoming cell (nativeID="overlay-next").
     private weak var nextOverlayView: UIView?
-    /// The page index from which the current scroll gesture started.
+    /// The page index used for overlay transform calculations.
     private var currentPage: Int = 0
+    /// Deferred page update to avoid flashing stale metadata.
+    private var pageUpdateWorkItem: DispatchWorkItem?
 
     // MARK: - Lifecycle
 
@@ -86,6 +88,8 @@ import ShortKit
     }
 
     private func removeFeedViewController() {
+        pageUpdateWorkItem?.cancel()
+        pageUpdateWorkItem = nil
         scrollObservation?.invalidate()
         scrollObservation = nil
         currentOverlayView?.transform = .identity
@@ -106,7 +110,6 @@ import ShortKit
 
     private func setupScrollTracking(_ feedVC: ShortKitFeedViewController) {
         guard let scrollView = findScrollView(in: feedVC.view) else {
-            NSLog("[ShortKitFeedView] Could not find scroll view in feed VC hierarchy.")
             return
         }
 
@@ -120,14 +123,47 @@ import ShortKit
         guard cellHeight > 0 else { return }
 
         let offset = scrollView.contentOffset.y
-        let delta = offset - CGFloat(currentPage) * cellHeight
 
-        // Update currentPage when the scroll settles near a page boundary
+        // Detect page change, but DEFER updating currentPage.
+        //
+        // Why: when the scroll settles on a new page, overlay-current still
+        // shows the OLD page's metadata (React hasn't processed OVERLAY_ACTIVATE
+        // yet). If we update currentPage immediately, delta snaps to 0 and
+        // overlay-current becomes visible with stale data.
+        //
+        // By deferring ~80ms, overlay-next (which already shows the correct
+        // data via NextOverlayProvider) stays visible at y=0 while React
+        // processes the state update. After the delay, overlay-current has
+        // been re-rendered with correct data and takes over seamlessly.
         let nearestPage = Int(round(offset / cellHeight))
         if abs(offset - CGFloat(nearestPage) * cellHeight) < 1.0 {
+            if nearestPage != currentPage && pageUpdateWorkItem == nil {
+                let targetPage = nearestPage
+                let workItem = DispatchWorkItem { [weak self] in
+                    guard let self else { return }
+                    self.currentPage = targetPage
+                    self.pageUpdateWorkItem = nil
+                    // Reapply overlay transforms now that currentPage is updated.
+                    // Without this, overlay-next (static NextOverlayProvider state)
+                    // stays visible at y=0 while overlay-current (live state) stays
+                    // hidden — no scroll event fires to trigger handleScrollOffset.
+                    let h = self.bounds.height
+                    self.currentOverlayView?.transform = .identity
+                    self.nextOverlayView?.transform = CGAffineTransform(translationX: 0, y: h)
+                }
+                pageUpdateWorkItem = workItem
+                DispatchQueue.main.asyncAfter(deadline: .now() + 0.08, execute: workItem)
+            }
+        } else if let workItem = pageUpdateWorkItem {
+            // User is scrolling again — apply pending update immediately
+            // so transforms stay aligned for the new gesture.
+            workItem.cancel()
+            pageUpdateWorkItem = nil
             currentPage = nearestPage
         }
 
+        let delta = offset - CGFloat(currentPage) * cellHeight
+
         // Find the overlay views if not cached
         if currentOverlayView == nil || nextOverlayView == nil {
             findOverlayViews()
@@ -146,28 +182,39 @@ import ShortKit
         }
     }
 
-    /// Find the sibling RN overlay views by nativeID (mapped to
-    /// accessibilityIdentifier on iOS). In the Fabric hierarchy, the
-    /// ShortKitFeedView and the overlay containers are children of the
-    /// same parent (wrapped by ShortKitFeed.tsx with overflow: hidden).
+    /// Find the sibling RN overlay views by nativeID.
+    ///
+    /// In Fabric interop, `ShortKitFeedView` (a Paper-style `RCTViewManager`
+    /// view) is wrapped in an intermediate `RCTViewComponentView`. So
+    /// `self.superview` is that wrapper — not the React `<View>` container
+    /// rendered by ShortKitFeed.tsx. We walk up the ancestor chain until we
+    /// find the overlays.
     private func findOverlayViews() {
-        guard let parent = superview else { return }
-
-        // The nativeID views may be nested inside intermediate Fabric
-        // wrapper views, so we search recursively within siblings.
-        for sibling in parent.subviews where sibling !== self {
-            if let found = findView(withNativeID: "overlay-current", in: sibling) {
-                currentOverlayView = found
-            }
-            if let found = findView(withNativeID: "overlay-next", in: sibling) {
-                nextOverlayView = found
+        var ancestor: UIView? = superview
+        while let container = ancestor {
+            for child in container.subviews {
+                // Skip subtrees that contain ourselves
+                guard !self.isDescendant(of: child) else { continue }
+
+                if currentOverlayView == nil {
+                    currentOverlayView = findView(withNativeID: "overlay-current", in: child)
+                }
+                if nextOverlayView == nil {
+                    nextOverlayView = findView(withNativeID: "overlay-next", in: child)
+                }
             }
+            if currentOverlayView != nil && nextOverlayView != nil { return }
+            ancestor = container.superview
         }
     }
 
-    /// Recursively find a view by its accessibilityIdentifier (nativeID).
+    /// Recursively find a view by its React Native `nativeID` prop.
+    /// In Fabric, this is stored on `RCTViewComponentView.nativeId`,
+    /// not `accessibilityIdentifier`.
     private func findView(withNativeID nativeID: String, in view: UIView) -> UIView? {
-        if view.accessibilityIdentifier == nativeID {
+        if view.responds(to: Selector(("nativeId"))),
+           let rnNativeId = view.value(forKey: "nativeId") as? String,
+           rnNativeId == nativeID {
             return view
         }
         for subview in view.subviews {

package/ios/ShortKitModule.mm

@@ -57,11 +57,6 @@ RCT_EXPORT_MODULE(ShortKitModule)
     @"onError",
     @"onShareTapped",
     @"onSurveyResponse",
-    @"onArticleTapped",
-    @"onCommentTapped",
-    @"onOverlayShareTapped",
-    @"onSaveTapped",
-    @"onLikeTapped",
     @"onOverlayConfigure",
     @"onOverlayActivate",
     @"onOverlayReset",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@shortkitsdk/react-native",
-  "version": "0.1.0",
+  "version": "0.2.0",
   "description": "ShortKit React Native SDK — short-form video feed",
   "react-native": "src/index",
   "source": "src/index",

package/src/ShortKitFeed.tsx

@@ -29,11 +29,6 @@ export function ShortKitFeed(props: ShortKitFeedProps) {
     onLoop,
     onFeedTransition,
     onFormatChange,
-    onArticleTapped,
-    onCommentTapped,
-    onOverlayShareTapped,
-    onSaveTapped,
-    onLikeTapped,
   } = props;
 
   const context = useContext(ShortKitContext);
@@ -114,51 +109,6 @@ export function ShortKitFeed(props: ShortKitFeedProps) {
       );
     }
 
-    if (onArticleTapped) {
-      subscriptions.push(
-        NativeShortKitModule.onArticleTapped((event) => {
-          const item = deserializeContentItem(event.item);
-          if (item) onArticleTapped(item);
-        }),
-      );
-    }
-
-    if (onCommentTapped) {
-      subscriptions.push(
-        NativeShortKitModule.onCommentTapped((event) => {
-          const item = deserializeContentItem(event.item);
-          if (item) onCommentTapped(item);
-        }),
-      );
-    }
-
-    if (onOverlayShareTapped) {
-      subscriptions.push(
-        NativeShortKitModule.onOverlayShareTapped((event) => {
-          const item = deserializeContentItem(event.item);
-          if (item) onOverlayShareTapped(item);
-        }),
-      );
-    }
-
-    if (onSaveTapped) {
-      subscriptions.push(
-        NativeShortKitModule.onSaveTapped((event) => {
-          const item = deserializeContentItem(event.item);
-          if (item) onSaveTapped(item);
-        }),
-      );
-    }
-
-    if (onLikeTapped) {
-      subscriptions.push(
-        NativeShortKitModule.onLikeTapped((event) => {
-          const item = deserializeContentItem(event.item);
-          if (item) onLikeTapped(item);
-        }),
-      );
-    }
-
     return () => {
       for (const sub of subscriptions) {
         sub.remove();
@@ -171,11 +121,6 @@ export function ShortKitFeed(props: ShortKitFeedProps) {
     onLoop,
     onFeedTransition,
     onFormatChange,
-    onArticleTapped,
-    onCommentTapped,
-    onOverlayShareTapped,
-    onSaveTapped,
-    onLikeTapped,
   ]);
 
   // ---------------------------------------------------------------------------

package/src/ShortKitProvider.tsx

@@ -68,7 +68,7 @@ type Action =
   | { type: 'CUE'; payload: { text: string; startTime: number; endTime: number } | null }
   | { type: 'PREFETCH_COUNT'; payload: number }
   | { type: 'OVERLAY_CONFIGURE'; payload: ContentItem }
-  | { type: 'OVERLAY_ACTIVATE' }
+  | { type: 'OVERLAY_ACTIVATE'; payload: ContentItem }
   | { type: 'OVERLAY_RESET' }
   | { type: 'OVERLAY_FADE_OUT' }
   | { type: 'OVERLAY_RESTORE' }
@@ -111,7 +111,7 @@ function reducer(state: State, action: Action): State {
     case 'OVERLAY_CONFIGURE':
       return { ...state, nextItem: action.payload };
     case 'OVERLAY_ACTIVATE':
-      return { ...state, currentItem: state.nextItem, isActive: true };
+      return { ...state, currentItem: action.payload, isActive: true };
     case 'OVERLAY_RESET':
       // Don't set isActive = false — the overlay stays mounted during
       // transitions. In the native SDK each cell has its own overlay
@@ -338,8 +338,11 @@ export function ShortKitProvider({
     );
 
     subscriptions.push(
-      NativeShortKitModule.onOverlayActivate((_event) => {
-        dispatch({ type: 'OVERLAY_ACTIVATE' });
+      NativeShortKitModule.onOverlayActivate((event) => {
+        const item = deserializeContentItem(event.item);
+        if (item) {
+          dispatch({ type: 'OVERLAY_ACTIVATE', payload: item });
+        }
       }),
     );
 

package/src/specs/NativeShortKitModule.ts

@@ -92,26 +92,6 @@ type SurveyResponseEvent = Readonly<{
   optionText: string;
 }>;
 
-type ArticleTappedEvent = Readonly<{
-  item: string; // JSON-serialized ContentItem
-}>;
-
-type CommentTappedEvent = Readonly<{
-  item: string; // JSON-serialized ContentItem
-}>;
-
-type OverlayShareTappedEvent = Readonly<{
-  item: string; // JSON-serialized ContentItem
-}>;
-
-type SaveTappedEvent = Readonly<{
-  item: string; // JSON-serialized ContentItem
-}>;
-
-type LikeTappedEvent = Readonly<{
-  item: string; // JSON-serialized ContentItem
-}>;
-
 type OverlayConfigureEvent = Readonly<{
   item: string; // JSON-serialized ContentItem
 }>;
@@ -184,11 +164,6 @@ export interface Spec extends TurboModule {
   readonly onError: EventEmitter<ErrorEvent>;
   readonly onShareTapped: EventEmitter<ShareTappedEvent>;
   readonly onSurveyResponse: EventEmitter<SurveyResponseEvent>;
-  readonly onArticleTapped: EventEmitter<ArticleTappedEvent>;
-  readonly onCommentTapped: EventEmitter<CommentTappedEvent>;
-  readonly onOverlayShareTapped: EventEmitter<OverlayShareTappedEvent>;
-  readonly onSaveTapped: EventEmitter<SaveTappedEvent>;
-  readonly onLikeTapped: EventEmitter<LikeTappedEvent>;
   readonly onOverlayConfigure: EventEmitter<OverlayConfigureEvent>;
   readonly onOverlayActivate: EventEmitter<OverlayActivateEvent>;
   readonly onOverlayReset: EventEmitter<OverlayResetEvent>;

package/src/types.ts

@@ -127,11 +127,6 @@ export interface ShortKitFeedProps {
   onLoop?: (event: LoopEvent) => void;
   onFeedTransition?: (event: FeedTransitionEvent) => void;
   onFormatChange?: (event: FormatChangeEvent) => void;
-  onArticleTapped?: (item: ContentItem) => void;
-  onCommentTapped?: (item: ContentItem) => void;
-  onOverlayShareTapped?: (item: ContentItem) => void;
-  onSaveTapped?: (item: ContentItem) => void;
-  onLikeTapped?: (item: ContentItem) => void;
 }
 
 // --- Hook Return Types ---