nitrogen 0.36.1 → 0.36.3

package/lib/syntax/kotlin/KotlinFunction.js

@@ -16,6 +16,12 @@ export function createKotlinFunction(functionType) {
         .getRequiredImports('kotlin')
         .map((i) => `import ${i.name}`)
         .filter(isNotDuplicate);
+    const funcAnnotations = [];
+    if (!functionType.isSync) {
+        // Async functions immediately schedule calls on a Dispatcher,
+        // we can make those @FastNative as they are not doing any JNI allocations
+        funcAnnotations.push('@FastNative');
+    }
     const kotlinCode = `
 ${createFileMetadataString(`${name}.kt`)}
 
@@ -73,7 +79,7 @@ class ${name}_cxx: ${name} {
   override fun invoke(${kotlinParams.join(', ')}): ${kotlinReturnType}
     = invoke_cxx(${kotlinParamsForward.join(',')})
 
-  @FastNative
+  ${funcAnnotations.join('\n')}
   private external fun invoke_cxx(${kotlinParams.join(', ')}): ${kotlinReturnType}
 }
 

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

@@ -7,6 +7,7 @@ export interface GetFunctionCodeOptions extends GetCodeOptions {
 export declare class FunctionType implements Type {
     readonly returnType: Type;
     readonly parameters: NamedType[];
+    readonly isSync: boolean;
     constructor(returnType: Type, parameters: NamedType[], isSync?: boolean);
     get specializationName(): string;
     get jsName(): string;

package/lib/syntax/types/FunctionType.js

@@ -5,6 +5,7 @@ import { PromiseType } from './PromiseType.js';
 export class FunctionType {
     returnType;
     parameters;
+    isSync;
     constructor(returnType, parameters, isSync = false) {
         if (returnType.kind === 'void' || isSync) {
             // void callbacks are async, but we don't care about the result.
@@ -15,6 +16,7 @@ export class FunctionType {
             this.returnType = new PromiseType(returnType);
         }
         this.parameters = parameters;
+        this.isSync = isSync;
         if (isSync && returnType.kind === 'void') {
             throw new Error(`Function \`${this.jsName}\` cannot be sync (\`Sync<...>\`) AND return \`void\`, as this is ambiguous. ` +
                 `Either return a value (even if it's just a \`boolean\`) to keep it sync, or make it async.`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nitrogen",
-  "version": "0.36.1",
+  "version": "0.36.3",
   "description": "Code generator for React Native Nitro Modules that turns TypeScript specs into C++, Swift, and Kotlin bindings.",
   "main": "lib/index",
   "types": "lib/index.d.ts",
@@ -35,7 +35,7 @@
   },
   "dependencies": {
     "chalk": "^5.3.0",
-    "react-native-nitro-modules": "^0.36.1",
+    "react-native-nitro-modules": "^0.36.3",
     "ts-morph": "^28.0.0",
     "yargs": "^18.0.0",
     "zod": "^4.4.3"

package/src/syntax/kotlin/KotlinFunction.ts

@@ -24,6 +24,12 @@ export function createKotlinFunction(functionType: FunctionType): SourceFile[] {
     .getRequiredImports('kotlin')
     .map((i) => `import ${i.name}`)
     .filter(isNotDuplicate)
+  const funcAnnotations: string[] = []
+  if (!functionType.isSync) {
+    // Async functions immediately schedule calls on a Dispatcher,
+    // we can make those @FastNative as they are not doing any JNI allocations
+    funcAnnotations.push('@FastNative')
+  }
 
   const kotlinCode = `
 ${createFileMetadataString(`${name}.kt`)}
@@ -82,7 +88,7 @@ class ${name}_cxx: ${name} {
   override fun invoke(${kotlinParams.join(', ')}): ${kotlinReturnType}
     = invoke_cxx(${kotlinParamsForward.join(',')})
 
-  @FastNative
+  ${funcAnnotations.join('\n')}
   private external fun invoke_cxx(${kotlinParams.join(', ')}): ${kotlinReturnType}
 }
 

package/src/syntax/types/FunctionType.ts

@@ -12,6 +12,7 @@ export interface GetFunctionCodeOptions extends GetCodeOptions {
 export class FunctionType implements Type {
   readonly returnType: Type
   readonly parameters: NamedType[]
+  readonly isSync: boolean
 
   constructor(returnType: Type, parameters: NamedType[], isSync = false) {
     if (returnType.kind === 'void' || isSync) {
@@ -22,6 +23,7 @@ export class FunctionType implements Type {
       this.returnType = new PromiseType(returnType)
     }
     this.parameters = parameters
+    this.isSync = isSync
 
     if (isSync && returnType.kind === 'void') {
       throw new Error(