BridgeJS: Register parameter names in scope instead of reserveNames

Author: krodak

Plugins/BridgeJS/Sources/BridgeJSLink/BridgeJSLink.swift

@@ -1219,7 +1219,8 @@ public struct BridgeJSLink {
                 loweringFragment.parameters.count == 1,
                 "Lowering fragment should have exactly one parameter to lower"
             )
-            let loweredValues = try loweringFragment.printCode([param.name], context)
+            let paramName = scope.variable(param.name)
+            let loweredValues = try loweringFragment.printCode([paramName], context)
             parameterForwardings.append(contentsOf: loweredValues)
         }
 

Plugins/BridgeJS/Sources/BridgeJSLink/JSGlueGen.swift

@@ -90,11 +90,6 @@ final class JSGlueVariableScope {
         JSGlueVariableScope(intrinsicRegistry: intrinsicRegistry)
     }
 
-    fileprivate func reserveNames(_ names: [String]) {
-        for name in names {
-            variables.insert(name)
-        }
-    }
 }
 
 extension JSGlueVariableScope {
@@ -738,7 +733,6 @@ struct IntrinsicJSFragment: Sendable {
             printCode: { arguments, context in
                 let (scope, printer) = (context.scope, context.printer)
                 let value = arguments[0]
-                scope.reserveNames(arguments)
                 let isSomeVar = scope.variable("isSome")
                 printer.write("const \(isSomeVar) = \(kind.presenceCheck(value: value));")