nitrogen 0.33.0 → 0.33.2

package/lib/autolinking/ios/createHybridObjectInitializer.js

@@ -28,13 +28,13 @@ export function createHybridObjectIntializer() {
         if (config?.swift != null) {
             // Autolink a Swift HybridObject!
             containsSwiftObjects = true;
-            const { cppCode, requiredImports, swiftRegistrationClass } = createSwiftHybridObjectRegistration({
+            const { cppCode, requiredImports, swiftRegistrationMethods } = createSwiftHybridObjectRegistration({
                 hybridObjectName: hybridObjectName,
                 swiftClassName: config.swift,
             });
             cppImports.push(...requiredImports);
             cppRegistrations.push(cppCode);
-            swiftRegistrations.push(swiftRegistrationClass);
+            swiftRegistrations.push(swiftRegistrationMethods);
         }
     }
     if (cppRegistrations.length === 0) {
@@ -79,23 +79,6 @@ import NitroModules
 public final class ${autolinkingClassName} {
   public typealias bridge = ${bridgeNamespace}
 
-  private protocol AutolinkedClass {
-    associatedtype T
-    /**
-     * Creates an instance of the Swift class that implements the HybridObject's spec,
-     * and wraps it in a Swift class that can directly interop with C++.
-     *
-     * This is generated by Nitrogen and will initialize the class specified
-     * in the \`"autolinking"\` property of \`nitro.json\`.
-     */
-    static func create() -> T
-    /**
-     * Returns whether this concrete implementation is also
-     * conforming to the \`RecyclableView\` protocol, or not.
-     */
-    static var isRecyclableHybridView: Bool { get }
-  }
-
   ${indent(swiftRegistrations.join('\n\n'), '  ')}
 }
   `.trim();

package/lib/syntax/swift/SwiftCxxBridgedType.js

@@ -310,7 +310,7 @@ export class SwiftCxxBridgedType {
                 switch (language) {
                     case 'swift':
                         return `
-{ () -> ${name.HybridTSpec} in
+{ () -> any ${name.HybridTSpec} in
   let __unsafePointer = ${getFunc}(${cppParameterName})
   let __instance = ${name.HybridTSpecCxx}.fromUnsafe(__unsafePointer)
   return __instance.get${name.HybridTSpec}()

package/lib/syntax/swift/SwiftHybridObjectRegistration.d.ts

@@ -11,10 +11,11 @@ interface Props {
 }
 interface SwiftHybridObjectRegistration {
     cppCode: string;
-    swiftRegistrationClass: string;
+    swiftRegistrationMethods: string;
     requiredImports: SourceImport[];
 }
 export declare function getAutolinkingNamespace(): string;
 export declare function getHybridObjectConstructorCall(hybridObjectName: string): string;
+export declare function getIsRecyclableCall(hybridObjectName: string): string;
 export declare function createSwiftHybridObjectRegistration({ hybridObjectName, swiftClassName, }: Props): SwiftHybridObjectRegistration;
 export {};

package/lib/syntax/swift/SwiftHybridObjectRegistration.js

@@ -10,22 +10,25 @@ export function getAutolinkingNamespace() {
 }
 export function getHybridObjectConstructorCall(hybridObjectName) {
     const namespace = getAutolinkingNamespace();
-    return `${namespace}::${hybridObjectName}::create();`;
+    return `${namespace}::create${hybridObjectName}();`;
+}
+export function getIsRecyclableCall(hybridObjectName) {
+    const namespace = getAutolinkingNamespace();
+    return `${namespace}::is${hybridObjectName}Recyclable();`;
 }
 export function createSwiftHybridObjectRegistration({ hybridObjectName, swiftClassName, }) {
     const { HybridTSpecSwift } = getHybridObjectName(hybridObjectName);
     const type = new HybridObjectType(hybridObjectName, 'swift', [], NitroConfig.current);
     const bridge = new SwiftCxxBridgedType(type);
     return {
-        swiftRegistrationClass: `
-public final class ${hybridObjectName}: AutolinkedClass {
-  public static func create() -> ${bridge.getTypeCode('swift')} {
-    let hybridObject = ${swiftClassName}()
-    return ${indent(bridge.parseFromSwiftToCpp('hybridObject', 'swift'), '    ')}
-  }
-  public static var isRecyclableHybridView: Bool {
-    return ${swiftClassName}.self is any RecyclableView.Type
-  }
+        swiftRegistrationMethods: `
+public static func create${hybridObjectName}() -> ${bridge.getTypeCode('swift')} {
+  let hybridObject = ${swiftClassName}()
+  return ${indent(bridge.parseFromSwiftToCpp('hybridObject', 'swift'), '  ')}
+}
+
+public static func is${hybridObjectName}Recyclable() -> Bool {
+  return ${swiftClassName}.self is any RecyclableView.Type
 }
     `.trim(),
         requiredImports: [

package/lib/views/swift/SwiftHybridViewManager.js

@@ -2,7 +2,7 @@ import { createViewComponentShadowNodeFiles, getViewComponentNames, } from '../C
 import { createFileMetadataString, escapeCppName, } from '../../syntax/helpers.js';
 import { getUmbrellaHeaderName } from '../../autolinking/ios/createSwiftUmbrellaHeader.js';
 import { getHybridObjectName } from '../../syntax/getHybridObjectName.js';
-import { getAutolinkingNamespace, getHybridObjectConstructorCall, } from '../../syntax/swift/SwiftHybridObjectRegistration.js';
+import { getHybridObjectConstructorCall, getIsRecyclableCall, } from '../../syntax/swift/SwiftHybridObjectRegistration.js';
 import { indent } from '../../utils.js';
 import { SwiftCxxBridgedType } from '../../syntax/swift/SwiftCxxBridgedType.js';
 export function createSwiftHybridViewManager(spec) {
@@ -118,7 +118,7 @@ using namespace ${namespace}::views;
 }
 
 + (BOOL)shouldBeRecycled {
-  return ${getAutolinkingNamespace()}::${spec.name}::isRecyclableHybridView();
+  return ${getIsRecyclableCall(spec.name)}
 }
 
 - (void)prepareForRecycle {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nitrogen",
-  "version": "0.33.0",
+  "version": "0.33.2",
   "description": "The code-generator for react-native-nitro-modules.",
   "main": "lib/index",
   "types": "lib/index.d.ts",
@@ -35,7 +35,7 @@
   },
   "dependencies": {
     "chalk": "^5.3.0",
-    "react-native-nitro-modules": "^0.33.0",
+    "react-native-nitro-modules": "^0.33.2",
     "ts-morph": "^27.0.0",
     "yargs": "^18.0.0",
     "zod": "^4.0.5"

package/src/autolinking/ios/createHybridObjectInitializer.ts

@@ -37,14 +37,14 @@ export function createHybridObjectIntializer(): [ObjcFile, SwiftFile] | [] {
     if (config?.swift != null) {
       // Autolink a Swift HybridObject!
       containsSwiftObjects = true
-      const { cppCode, requiredImports, swiftRegistrationClass } =
+      const { cppCode, requiredImports, swiftRegistrationMethods } =
         createSwiftHybridObjectRegistration({
           hybridObjectName: hybridObjectName,
           swiftClassName: config.swift,
         })
       cppImports.push(...requiredImports)
       cppRegistrations.push(cppCode)
-      swiftRegistrations.push(swiftRegistrationClass)
+      swiftRegistrations.push(swiftRegistrationMethods)
     }
   }
 
@@ -93,23 +93,6 @@ import NitroModules
 public final class ${autolinkingClassName} {
   public typealias bridge = ${bridgeNamespace}
 
-  private protocol AutolinkedClass {
-    associatedtype T
-    /**
-     * Creates an instance of the Swift class that implements the HybridObject's spec,
-     * and wraps it in a Swift class that can directly interop with C++.
-     *
-     * This is generated by Nitrogen and will initialize the class specified
-     * in the \`"autolinking"\` property of \`nitro.json\`.
-     */
-    static func create() -> T
-    /**
-     * Returns whether this concrete implementation is also
-     * conforming to the \`RecyclableView\` protocol, or not.
-     */
-    static var isRecyclableHybridView: Bool { get }
-  }
-
   ${indent(swiftRegistrations.join('\n\n'), '  ')}
 }
   `.trim()

package/src/syntax/swift/SwiftCxxBridgedType.ts

@@ -353,7 +353,7 @@ export class SwiftCxxBridgedType implements BridgedType<'swift', 'c++'> {
         switch (language) {
           case 'swift':
             return `
-{ () -> ${name.HybridTSpec} in
+{ () -> any ${name.HybridTSpec} in
   let __unsafePointer = ${getFunc}(${cppParameterName})
   let __instance = ${name.HybridTSpecCxx}.fromUnsafe(__unsafePointer)
   return __instance.get${name.HybridTSpec}()

package/src/syntax/swift/SwiftHybridObjectRegistration.ts

@@ -18,7 +18,7 @@ interface Props {
 
 interface SwiftHybridObjectRegistration {
   cppCode: string
-  swiftRegistrationClass: string
+  swiftRegistrationMethods: string
   requiredImports: SourceImport[]
 }
 
@@ -32,7 +32,12 @@ export function getHybridObjectConstructorCall(
   hybridObjectName: string
 ): string {
   const namespace = getAutolinkingNamespace()
-  return `${namespace}::${hybridObjectName}::create();`
+  return `${namespace}::create${hybridObjectName}();`
+}
+
+export function getIsRecyclableCall(hybridObjectName: string): string {
+  const namespace = getAutolinkingNamespace()
+  return `${namespace}::is${hybridObjectName}Recyclable();`
 }
 
 export function createSwiftHybridObjectRegistration({
@@ -50,15 +55,14 @@ export function createSwiftHybridObjectRegistration({
   const bridge = new SwiftCxxBridgedType(type)
 
   return {
-    swiftRegistrationClass: `
-public final class ${hybridObjectName}: AutolinkedClass {
-  public static func create() -> ${bridge.getTypeCode('swift')} {
-    let hybridObject = ${swiftClassName}()
-    return ${indent(bridge.parseFromSwiftToCpp('hybridObject', 'swift'), '    ')}
-  }
-  public static var isRecyclableHybridView: Bool {
-    return ${swiftClassName}.self is any RecyclableView.Type
-  }
+    swiftRegistrationMethods: `
+public static func create${hybridObjectName}() -> ${bridge.getTypeCode('swift')} {
+  let hybridObject = ${swiftClassName}()
+  return ${indent(bridge.parseFromSwiftToCpp('hybridObject', 'swift'), '  ')}
+}
+
+public static func is${hybridObjectName}Recyclable() -> Bool {
+  return ${swiftClassName}.self is any RecyclableView.Type
 }
     `.trim(),
     requiredImports: [

package/src/views/swift/SwiftHybridViewManager.ts

@@ -11,8 +11,8 @@ import {
 import { getUmbrellaHeaderName } from '../../autolinking/ios/createSwiftUmbrellaHeader.js'
 import { getHybridObjectName } from '../../syntax/getHybridObjectName.js'
 import {
-  getAutolinkingNamespace,
   getHybridObjectConstructorCall,
+  getIsRecyclableCall,
 } from '../../syntax/swift/SwiftHybridObjectRegistration.js'
 import { indent } from '../../utils.js'
 import { SwiftCxxBridgedType } from '../../syntax/swift/SwiftCxxBridgedType.js'
@@ -142,7 +142,7 @@ using namespace ${namespace}::views;
 }
 
 + (BOOL)shouldBeRecycled {
-  return ${getAutolinkingNamespace()}::${spec.name}::isRecyclableHybridView();
+  return ${getIsRecyclableCall(spec.name)}
 }
 
 - (void)prepareForRecycle {

package/lib/syntax/types/Int32Type.d.ts

@@ -1,10 +0,0 @@
-import type { Language } from '../../getPlatformSpecs.js';
-import type { SourceFile, SourceImport } from '../SourceFile.js';
-import type { Type, TypeKind } from './Type.js';
-export declare class Int32Type implements Type {
-    get canBePassedByReference(): boolean;
-    get kind(): TypeKind;
-    getCode(language: Language): string;
-    getExtraFiles(): SourceFile[];
-    getRequiredImports(language: Language): SourceImport[];
-}

package/lib/syntax/types/Int32Type.js

@@ -1,36 +0,0 @@
-export class Int32Type {
-    get canBePassedByReference() {
-        // It's a primitive
-        return false;
-    }
-    get kind() {
-        return 'int32';
-    }
-    getCode(language) {
-        switch (language) {
-            case 'c++':
-                return 'int32_t';
-            case 'swift':
-                return 'Int32';
-            case 'kotlin':
-                return 'Int';
-            default:
-                throw new Error(`Language ${language} is not yet supported for Int32Type!`);
-        }
-    }
-    getExtraFiles() {
-        return [];
-    }
-    getRequiredImports(language) {
-        if (language === 'c++') {
-            return [
-                {
-                    language: language,
-                    name: 'cstdint',
-                    space: 'system',
-                },
-            ];
-        }
-        return [];
-    }
-}