fix(bindgen): duplicate JsonCompatible imports for inputs, outputs

InsightSoftwareConsortium · Sep 26, 2023 · 4e38ec5 · 4e38ec5
1 parent dc86da5
commit 4e38ec5
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 9 deletions.
diff --git a/src/bindgen/typescript/function-module.js b/src/bindgen/typescript/function-module.js
@@ -39,7 +39,6 @@ function functionModule (srcOutputDir, forNode, interfaceJson, modulePascalCase,
   let functionContent = `import {\n`
   const usedInterfaceTypes = new Set()
   let needMountDirs = false
-  let needJsonCompatible = false
   const pipelineComponents = ['inputs', 'outputs', 'parameters']
   pipelineComponents.forEach((pipelineComponent) => {
     interfaceJson[pipelineComponent].forEach((value) => {
@@ -49,10 +48,6 @@ function functionModule (srcOutputDir, forNode, interfaceJson, modulePascalCase,
           needMountDirs = true
           return
         }
-        if (interfaceType === 'JsonCompatible' && pipelineComponent === 'inputs') {
-          needJsonCompatible = true
-          return
-        }
         if (!(pipelineComponent === 'inputs' && interfaceType === 'BinaryStream')) {
           usedInterfaceTypes.add(interfaceType)
         }
@@ -65,9 +60,6 @@ function functionModule (srcOutputDir, forNode, interfaceJson, modulePascalCase,
   functionContent += '  InterfaceTypes,\n'
   functionContent += '  PipelineOutput,\n'
   functionContent += '  PipelineInput,\n'
-  if (needJsonCompatible) {
-    functionContent += '  JsonCompatible,\n'
-  }
   if (forNode) {
     functionContent += '  runPipelineNode\n'
   } else {

diff --git a/src/bindgen/typescript/types-require-import.js b/src/bindgen/typescript/types-require-import.js
@@ -1,3 +1,3 @@
-const typesRequireImport = ['Image', 'Mesh', 'PolyData', 'TextFile', 'BinaryFile', 'TextFile', 'BinaryFile']
+const typesRequireImport = ['Image', 'Mesh', 'PolyData', 'TextFile', 'BinaryFile', 'TextFile', 'BinaryFile', 'JsonCompatible']
 
 export default typesRequireImport