Fixed bug overwriting imported or prefilled component details.

Author: dterefe

DUUIWeb/src/lib/svelte/components/ComponentPopup.svelte

@@ -29,25 +29,61 @@
 	const importComponent = async () => {
 		if (!files) return
 		const file: File = files[0]
-		async function parseJsonFile(file) {
+
+		async function parseJsonFile(file: File): Promise<any> {
 			return new Promise((resolve, reject) => {
 				const fileReader = new FileReader()
-				fileReader.onload = (event) => resolve(JSON.parse(event.target.result))
+				fileReader.onload = (event) => {
+					try {
+						const text = (event.target?.result || '') as string
+						resolve(JSON.parse(text))
+					} catch (err) {
+						reject(err)
+					}
+				}
 				fileReader.onerror = (error) => reject(error)
 				fileReader.readAsText(file)
 			})
 		}
+
 		const result = await parseJsonFile(file)
 		if (!result) return
 
-		let component = blankComponent($currentPipelineStore.oid, index)
+		// Support both plain component JSON and pipeline JSON (with a components array).
+		const source: any =
+			result && Array.isArray(result.components) && result.components.length > 0
+				? result.components[0]
+				: result
+
+		const base = blankComponent($currentPipelineStore.oid, index)
+
+		const mergedOptions = {
+			...base.options,
+			...(source.options || {}),
+			registry_auth: {
+				...base.options.registry_auth,
+				...(source.options?.registry_auth || {})
+			}
+		}
+
+		const component: DUUIComponent = {
+			...base,
+			...source,
+			// Ensure required fields always exist and keep sensible defaults
+			name: source.name ?? base.name,
+			driver: source.driver ?? base.driver,
+			target: source.target ?? base.target,
+			options: mergedOptions,
+			parameters: source.parameters ?? base.parameters,
+			id: uuidv4()
+		}
 
 		drawerStore.open({
 			id: 'component',
 			...componentDrawerSettings,
 			meta: {
-				component: { ...component, ...result, id: uuidv4() },
-				inEditor: inEditor,
+				component,
+				inEditor,
 				creating: true
 			}
 		})

DUUIWeb/src/lib/svelte/components/Drawer/ComponentDrawer.svelte

@@ -410,16 +410,23 @@
 
 							if (isEmpty(registryEntry.name)) return;
 
-							component.name = registryEntry.name
-							component.target = registryEntry.registry_url
+							// Build a suggested target from the registry entry.
+							// If the registry URL is missing, fall back to name:tag.
+							let suggestedTarget = registryEntry.registry_url
 								? `${registryEntry.registry_url}${registryEntry.name}:${entryMetadata.tag}`
-								: "";
-							
-							if (component.target.startsWith('https://')) {
-								component.target = component.target.replace(/^https?:\/\//, '');
+								: `${registryEntry.name}:${entryMetadata.tag}`;
+
+							if (suggestedTarget.startsWith('https://')) {
+								suggestedTarget = suggestedTarget.replace(/^https?:\/\//, '');
+							}
+
+							// Only overwrite the existing target if it is empty.
+							if (!component.target) {
+								component.name = registryEntry.name
+								component.target = suggestedTarget;
+								component.description = entryMetadata.description
 							}
 
-							component.description = entryMetadata.description
 						}
 					}
 				/>