Drop outdated clang-format from npm (#5191)

* enable logs on expo sample

* repro'

* patch clang-format and run it

* return flags

* return original sample code

* more rollback

* print clang version

* remove deprecated package, add clang-format setup

* try setup clang21

* other strategy to update clang/ more changes for version

* gh action name change

* Update .github/workflows/buildandtest.yml

Author: lucas-zimerman

.github/workflows/buildandtest.yml

@@ -51,6 +51,22 @@ jobs:
           cache-dependency-path: yarn.lock
       - name: Install Dependencies
         run: yarn install
+
+      # Default of ubuntu and apt packages are too old compared to macos packages.
+      # This is required for using a newer version of clang-format.
+      - name: Setup clang-format V20
+        run: |
+          sudo bash -c "$(wget -O - https://apt.llvm.org/llvm.sh)" 20
+          sudo apt-get install -y clang-20 clang-format-20 lld-20 lldb-20
+
+      - name: Set clang-format V20 as default
+        run: |
+          sudo update-alternatives --install /usr/bin/clang clang /usr/bin/clang-20 200
+          sudo update-alternatives --install /usr/bin/clang++ clang++ /usr/bin/clang++-20 200
+          sudo update-alternatives --install /usr/bin/clang-format clang-format /usr/bin/clang-format-20 200
+          clang --version
+          clang-format --version
+
       - name: Lint
         run: yarn lint
 

package.json

@@ -31,7 +31,6 @@
     "@expo/swiftlint": "^0.57.1",
     "@naturalcycles/ktlint": "^1.13.0",
     "@sentry/cli": "2.53.0",
-    "clang-format": "^1.8.0",
     "downlevel-dts": "^0.11.0",
     "google-java-format": "^1.4.0",
     "lerna": "^8.1.8",

packages/core/RNSentryCocoaTester/RNSentryCocoaTesterTests/RNSentry+Test.h

@@ -1,7 +1,6 @@
 #import <RNSentry/RNSentry.h>
 
-@interface
-RNSentry (RNSentryInternal)
+@interface RNSentry (RNSentryInternal)
 
 + (SentryUser *_Nullable)userFrom:(NSDictionary *)userKeys
                     otherUserKeys:(NSDictionary *)userDataKeys;

packages/core/RNSentryCocoaTester/RNSentryCocoaTesterTests/RNSentryDependencyContainerTests.m

@@ -22,7 +22,7 @@ - (void)testRNSentryDependencyContainerInitializesFrameTracker
     OCMStub([(SentryDependencyContainer *)sentryDependencyContainerMock framesTracker])
         .andReturn(frameTrackerMock);
 
-    RNSentryEmitNewFrameEvent emitNewFrameEvent = ^(NSNumber *newFrameTimestampInSeconds) {};
+    RNSentryEmitNewFrameEvent emitNewFrameEvent = ^(NSNumber *newFrameTimestampInSeconds) { };
     [[RNSentryDependencyContainer sharedInstance]
         initializeFramesTrackerListenerWith:emitNewFrameEvent];
     XCTAssertNotNil([[RNSentryDependencyContainer sharedInstance] framesTrackerListener]);

packages/core/RNSentryCocoaTester/RNSentryCocoaTesterTests/RNSentryFramesTrackerListenerTests.m

@@ -46,7 +46,7 @@ - (void)testRNSentryFramesTrackerIsOneTimeListener
     OCMStub([(SentryDependencyContainer *)sentryDependencyContainerMock framesTracker])
         .andReturn(frameTrackerMock);
 
-    RNSentryEmitNewFrameEvent mockEventEmitter = ^(NSNumber *newFrameTimestampInSeconds) {};
+    RNSentryEmitNewFrameEvent mockEventEmitter = ^(NSNumber *newFrameTimestampInSeconds) { };
 
     RNSentryFramesTrackerListener *actualListener = [[RNSentryFramesTrackerListener alloc]
         initWithSentryFramesTracker:[[SentryDependencyContainer sharedInstance] framesTracker]
@@ -66,7 +66,7 @@ - (void)testRNSentryFramesTrackerAddsItselfAsListener
     OCMStub([(SentryDependencyContainer *)sentryDependencyContainerMock framesTracker])
         .andReturn(frameTrackerMock);
 
-    RNSentryEmitNewFrameEvent mockEventEmitter = ^(NSNumber *newFrameTimestampInSeconds) {};
+    RNSentryEmitNewFrameEvent mockEventEmitter = ^(NSNumber *newFrameTimestampInSeconds) { };
 
     RNSentryFramesTrackerListener *actualListener = [[RNSentryFramesTrackerListener alloc]
         initWithSentryFramesTracker:[[SentryDependencyContainer sharedInstance] framesTracker]

packages/core/RNSentryCocoaTester/RNSentryCocoaTesterTests/RNSentryOnDrawReporter+Test.h

@@ -1,8 +1,7 @@
 #import "RNSentryOnDrawReporter.h"
 #import <Foundation/Foundation.h>
 
-@interface
-RNSentryOnDrawReporterView (Testing)
+@interface RNSentryOnDrawReporterView (Testing)
 
 + (instancetype)createWithMockedListener;
 - (RNSentryEmitNewFrameEvent)createEmitNewFrameEvent;

packages/core/RNSentryCocoaTester/RNSentryCocoaTesterTests/RNSentryOnDrawReporter+Test.mm

@@ -28,8 +28,7 @@ - (void)framesTrackerHasNewFrame:(nonnull NSDate *)newFrameDate
 
 @end
 
-@implementation
-RNSentryOnDrawReporterView (Testing)
+@implementation RNSentryOnDrawReporterView (Testing)
 
 + (instancetype)createWithMockedListener
 {

packages/core/RNSentryCocoaTester/RNSentryCocoaTesterTests/RNSentryTests.h

@@ -3,8 +3,7 @@
 
 @class SentryOptions;
 
-@interface
-SentrySDKInternal (PrivateTests)
+@interface SentrySDKInternal (PrivateTests)
 
 + (nullable SentryOptions *)options;
 @end

packages/core/ios/RNSentry+fetchNativeStack.m

@@ -16,8 +16,7 @@
 
 // This method was moved to a new category so we can use `@import Sentry` to use Sentry's Swift
 // classes
-@implementation
-RNSentry (fetchNativeStack)
+@implementation RNSentry (fetchNativeStack)
 
 - (NSDictionary *)fetchNativeStackFramesBy:(NSArray<NSNumber *> *)instructionsAddr
                                symbolicate:(SymbolicateCallbackType)symbolicate

packages/core/ios/RNSentry.h

@@ -28,8 +28,7 @@ typedef int (*SymbolicateCallbackType)(const void *, Dl_info *);
 
 @end
 
-@interface
-RNSentry (fetchNativeStack)
+@interface RNSentry (fetchNativeStack)
 
 - (NSDictionary *_Nonnull)fetchNativeStackFramesBy:(NSArray<NSNumber *> *)instructionsAddr
                                        symbolicate:(SymbolicateCallbackType)symbolicate;