From ff0d385db82663e98072356ffec4bdba2bf5806a Mon Sep 17 00:00:00 2001
From: Luke Mino-Altherr <luke@comfy.org>
Date: Fri, 21 Nov 2025 11:05:36 -0800
Subject: [PATCH] [bugfix] Fix double-click required after pasting URL in
 upload model dialog (#6801)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

## Summary
Fixed an issue where users had to click twice to continue after pasting
a URL in the upload model dialog - once to blur the input, then again to
click the button.

## Changes
- **What**: Replaced `UrlInput` with plain `InputText` in
`UploadModelUrlInput` to emit value immediately on input instead of only
on blur
- **Cleanup**: Moved URL cleaning/normalization to the `fetchMetadata`
handler, removed unused `disableValidation` prop from `UrlInput`
component

## Review Focus
- URL normalization logic in `useUploadModelWizard.ts`

🤖 Generated with [Claude Code](https://claude.com/claude-code)

┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-6801-bugfix-Fix-double-click-required-after-pasting-URL-in-upload-model-dialog-2b26d73d3650811881aed0cc064efcc7)
by [Unito](https://www.unito.io)

---------

Co-authored-by: Claude <noreply@anthropic.com>
---
 src/components/common/UrlInput.vue                      | 3 ---
 src/platform/assets/components/UploadModelUrlInput.vue  | 7 +++----
 src/platform/assets/composables/useUploadModelWizard.ts | 9 +++++++++
 3 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/src/components/common/UrlInput.vue b/src/components/common/UrlInput.vue
index b0dfbd60f..df40ba0c4 100644
--- a/src/components/common/UrlInput.vue
+++ b/src/components/common/UrlInput.vue
@@ -35,7 +35,6 @@ import { ValidationState } from '@/utils/validationUtil'
 const props = defineProps<{
   modelValue: string
   validateUrlFn?: (url: string) => Promise<boolean>
-  disableValidation?: boolean
 }>()
 
 const emit = defineEmits<{
@@ -102,8 +101,6 @@ const defaultValidateUrl = async (url: string): Promise<boolean> => {
 }
 
 const validateUrl = async (value: string) => {
-  if (props.disableValidation) return
-
   if (validationState.value === ValidationState.LOADING) return
 
   const url = cleanInput(value)
diff --git a/src/platform/assets/components/UploadModelUrlInput.vue b/src/platform/assets/components/UploadModelUrlInput.vue
index 2cd7978e5..bfe534c62 100644
--- a/src/platform/assets/components/UploadModelUrlInput.vue
+++ b/src/platform/assets/components/UploadModelUrlInput.vue
@@ -14,10 +14,10 @@
       <label class="text-sm text-muted mb-0">
         {{ $t('assetBrowser.civitaiLinkLabel') }}
       </label>
-      <UrlInput
+      <InputText
         v-model="url"
         :placeholder="$t('assetBrowser.civitaiLinkPlaceholder')"
-        :disable-validation="true"
+        class="w-full"
       />
       <p v-if="error" class="text-xs text-error">
         {{ error }}
@@ -30,10 +30,9 @@
 </template>
 
 <script setup lang="ts">
+import InputText from 'primevue/inputtext'
 import { computed } from 'vue'
 
-import UrlInput from '@/components/common/UrlInput.vue'
-
 const props = defineProps<{
   modelValue: string
   error?: string
diff --git a/src/platform/assets/composables/useUploadModelWizard.ts b/src/platform/assets/composables/useUploadModelWizard.ts
index 4719d1d54..89a726df9 100644
--- a/src/platform/assets/composables/useUploadModelWizard.ts
+++ b/src/platform/assets/composables/useUploadModelWizard.ts
@@ -62,6 +62,15 @@ export function useUploadModelWizard(modelTypes: Ref<ModelTypeOption[]>) {
   async function fetchMetadata() {
     if (!canFetchMetadata.value) return
 
+    // Clean and normalize URL
+    let cleanedUrl = wizardData.value.url.trim()
+    try {
+      cleanedUrl = new URL(encodeURI(cleanedUrl)).toString()
+    } catch {
+      // If URL parsing fails, just use the trimmed input
+    }
+    wizardData.value.url = cleanedUrl
+
     if (!isCivitaiUrl(wizardData.value.url)) {
       uploadError.value = st(
         'assetBrowser.onlyCivitaiUrlsSupported',