Changes in iOS 12.3 may break builds

Description

(please don’t kill me for not following the template 😄- this is more of a “heads up” than a bug report)

In iOS 12.3, it looks like Apple is introducing a new type called symbol to the JavaScriptCore API: (from /Applications/Xcode-beta.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator.sdk/System/Library/Frameworks/JavaScriptCore.framework/Headers/JSValueRef.h)

/*!
@enum JSType
@abstract     A constant identifying the type of a JSValue.
@constant     kJSTypeUndefined  The unique undefined value.
@constant     kJSTypeNull       The unique null value.
@constant     kJSTypeBoolean    A primitive boolean value, one of true or false.
@constant     kJSTypeNumber     A primitive number value.
@constant     kJSTypeString     A primitive string value.
@constant     kJSTypeObject     An object value (meaning that this JSValueRef is a JSObjectRef).
@constant     kJSTypeSymbol     A primitive symbol value.
*/
typedef enum {
    kJSTypeUndefined,
    kJSTypeNull,
    kJSTypeBoolean,
    kJSTypeNumber,
    kJSTypeString,
    kJSTypeObject,
    kJSTypeSymbol API_AVAILABLE(macosx(10.14.4), ios(12.3))
} JSType;

This has certain consequences for Realm: https://github.com/realm/realm-js/blob/91d87fe0a94ae3101b6bb57f33c8d64ed96465d6/src/jsc/jsc_value.hpp#L44-L54

Since the kJSTypeSymbol enum value is not handled in the above switch statement, builds on Xcode 10.2 Beta (10P82s) will fail with this error:

I was able to resolve the error by adding this:

case kJSTypeSymbol: return "symbol";

I’m not sure what side effects that change has so I haven’t made a PR yet.

Version of Realm and Tooling

• Realm JS SDK Version: 2.21.1
• Node or React Native: React Native
• Client OS & Version: Building for iOS 12.2 (so I’m not sure why this issue is occuring with API_AVAILABLE(macosx(10.14.4), ios(12.3)))
• Which debugger for React Native: None

[shivkrishnashah]

Fixed By Adding

case kJSTypeSymbol: return “symbol”;

Example

switch (JSValueGetType(ctx, value)) {
    case kJSTypeNull: return "null";
    case kJSTypeNumber: return "number";
    case kJSTypeObject: return "object";
    case kJSTypeString: return "string";
    case kJSTypeBoolean: return "boolean";
    case kJSTypeUndefined: return "undefined";
    case kJSTypeSymbol: return "symbol";
}

[kneth]

@rajivshah3 Thanks for the update. We are in the process of updating and preparing our CI system with the latest Apple releases (still waiting for next Xcode version).