fix(ios): enable JSContext to be inspectable (#13812)

* fix(ios): enable JSContext to be inspectable In iOS 16.4 the default state for whether a JSContext is inspectable became false and the owner of the context must explicitly set it to be inspectable. This commit enables the JSContext to be inspectable and I have verified this works for both Safari and VS Code debugging. However, for security concerns it's probably best that we disable this in production I assume. How to do that, I have no idea. It probably also needs to be guarded to only be set on iOS 16.4+ Refs https://webkit.org/blog/13966/webkit-features-in-safari-16-4/https://webkit.org/blog/13936/enabling-the-inspection-of-web-content-in-apps/ * refactor: guard usage of JSGlobalContextSetInspectable * refactor: use different guard * refactor: use YES instead of true, add production check * fix(ios): add prod-check to debugger handler --------- Co-authored-by: Hans Knöchel <h.knoechel@lambus.com>

Author: ewanharris

iphone/TitaniumKit/TitaniumKit/Sources/API/TiApp.m

@@ -248,6 +248,15 @@ - (void)booted:(id)bridge
  }
  [_queuedApplicationSelectors removeAllObjects];
  }
+
+// Ensure that the JSContext is debuggable during development
+#if __IPHONE_OS_VERSION_MAX_ALLOWED >= 160400
+BOOL isProduction = [TiSharedConfig.defaultConfig.applicationDeployType isEqualToString:@"production"];
+
+if (!isProduction) {
+JSGlobalContextSetInspectable([[(KrollBridge *)bridge krollContext] context], YES);
+ }
+#endif
  }
 }
 

iphone/TitaniumKit/TitaniumKit/Sources/Misc/TiSharedConfig.h

@@ -51,7 +51,7 @@ NS_ASSUME_NONNULL_BEGIN
 @property (nonatomic, strong, nullable) NSString *applicationDescription;
 
 /**
- Deploy type of the application, e.g. "development", currently **unused**.
+ Deploy type of the application, e.g. "development", "test" or "production"
 */
 @property (nonatomic, strong, nullable) NSString *applicationDeployType;