From 84e06058353a1bd9d87c09c650654df36940f78a Mon Sep 17 00:00:00 2001 From: Jesper Kristensen Date: Thu, 15 May 2025 21:53:07 +0200 Subject: [PATCH] Cleaning up usage of console log in front end --- src/lib/apis/ollama/index.ts | 2 +- src/lib/apis/users/index.ts | 2 +- .../admin/Functions/FunctionEditor.svelte | 2 +- .../admin/Settings/Documents.svelte | 4 +-- .../components/admin/Settings/General.svelte | 4 +-- .../components/admin/Settings/Images.svelte | 4 +-- .../admin/Settings/Interface.svelte | 2 +- .../components/admin/Settings/Models.svelte | 2 +- .../Models/Manage/ManageOllama.svelte | 32 +++++++++---------- .../admin/Settings/Pipelines.svelte | 4 +-- src/lib/components/admin/Users/Groups.svelte | 2 +- .../admin/Users/Groups/AddGroupModal.svelte | 4 --- .../admin/Users/Groups/EditGroupModal.svelte | 1 - .../admin/Users/UserList/AddUserModal.svelte | 2 +- .../components/channel/MessageInput.svelte | 15 ++++----- src/lib/components/channel/Messages.svelte | 2 +- src/lib/components/channel/Thread.svelte | 2 +- src/lib/components/chat/Artifacts.svelte | 2 +- 18 files changed, 40 insertions(+), 48 deletions(-) diff --git a/src/lib/apis/ollama/index.ts b/src/lib/apis/ollama/index.ts index 74f2914a9..2f6278fe9 100644 --- a/src/lib/apis/ollama/index.ts +++ b/src/lib/apis/ollama/index.ts @@ -408,7 +408,7 @@ export const deleteModel = async (token: string, tagName: string, urlIdx: string return res.json(); }) .then((json) => { - console.log(json); + console.debug(json); return true; }) .catch((err) => { diff --git a/src/lib/apis/users/index.ts b/src/lib/apis/users/index.ts index 48ef343f9..f8ab88ff5 100644 --- a/src/lib/apis/users/index.ts +++ b/src/lib/apis/users/index.ts @@ -343,7 +343,7 @@ export const getAndUpdateUserLocation = async (token: string) => { await updateUserInfo(token, { location: location }); return location; } else { - console.log('Failed to get user location'); + console.info('Failed to get user location'); return null; } }; diff --git a/src/lib/components/admin/Functions/FunctionEditor.svelte b/src/lib/components/admin/Functions/FunctionEditor.svelte index 9b2355fe4..1ef7bddc1 100644 --- a/src/lib/components/admin/Functions/FunctionEditor.svelte +++ b/src/lib/components/admin/Functions/FunctionEditor.svelte @@ -277,7 +277,7 @@ class Pipe: await tick(); if (res) { - console.log('Code formatted successfully'); + console.info('Code formatted successfully'); saveHandler(); } diff --git a/src/lib/components/admin/Settings/Documents.svelte b/src/lib/components/admin/Settings/Documents.svelte index 498ff2c1e..3a3c361cf 100644 --- a/src/lib/components/admin/Settings/Documents.svelte +++ b/src/lib/components/admin/Settings/Documents.svelte @@ -91,7 +91,7 @@ return; } - console.log('Update embedding model attempt:', embeddingModel); + console.debug('Update embedding model attempt:', embeddingModel); updateEmbeddingModelLoading = true; const res = await updateEmbeddingConfig(localStorage.token, { @@ -114,7 +114,7 @@ updateEmbeddingModelLoading = false; if (res) { - console.log('embeddingModelUpdateHandler:', res); + console.debug('embeddingModelUpdateHandler:', res); if (res.status === true) { toast.success($i18n.t('Embedding model set to "{{embedding_model}}"', res), { duration: 1000 * 10 diff --git a/src/lib/components/admin/Settings/General.svelte b/src/lib/components/admin/Settings/General.svelte index 86cc876ef..85f21824f 100644 --- a/src/lib/components/admin/Settings/General.svelte +++ b/src/lib/components/admin/Settings/General.svelte @@ -59,10 +59,10 @@ }; }); - console.log(version); + console.info(version); updateAvailable = compareVersion(version.latest, version.current); - console.log(updateAvailable); + console.info(updateAvailable); }; const updateLdapServerHandler = async () => { diff --git a/src/lib/components/admin/Settings/Images.svelte b/src/lib/components/admin/Settings/Images.svelte index 64fa249dc..003b991a0 100644 --- a/src/lib/components/admin/Settings/Images.svelte +++ b/src/lib/components/admin/Settings/Images.svelte @@ -198,14 +198,14 @@ 2 ); } catch (e) { - console.log(e); + console.error(e); } } requiredWorkflowNodes = requiredWorkflowNodes.map((node) => { const n = config.comfyui.COMFYUI_WORKFLOW_NODES.find((n) => n.type === node.type) ?? node; - console.log(n); + console.debug(n); return { type: n.type, diff --git a/src/lib/components/admin/Settings/Interface.svelte b/src/lib/components/admin/Settings/Interface.svelte index c70259983..53d4f29e4 100644 --- a/src/lib/components/admin/Settings/Interface.svelte +++ b/src/lib/components/admin/Settings/Interface.svelte @@ -90,7 +90,7 @@ } }); - console.log('models', models); + console.debug('models', models); }; diff --git a/src/lib/components/admin/Settings/Models.svelte b/src/lib/components/admin/Settings/Models.svelte index d5b2fc627..5775abf85 100644 --- a/src/lib/components/admin/Settings/Models.svelte +++ b/src/lib/components/admin/Settings/Models.svelte @@ -166,7 +166,7 @@ hidden: !(model?.meta?.hidden ?? false) }; - console.log(model); + console.debug(model); toast.success( model.meta.hidden diff --git a/src/lib/components/admin/Settings/Models/Manage/ManageOllama.svelte b/src/lib/components/admin/Settings/Models/Manage/ManageOllama.svelte index feb5010d2..51f45df5a 100644 --- a/src/lib/components/admin/Settings/Models/Manage/ManageOllama.svelte +++ b/src/lib/components/admin/Settings/Models/Manage/ManageOllama.svelte @@ -66,7 +66,7 @@ const updateModelsHandler = async () => { for (const model of ollamaModels) { - console.log(model); + console.debug(model); updateModelId = model.id; const [res, controller] = await pullModel(localStorage.token, model.id, urlIdx).catch( @@ -114,8 +114,8 @@ } } } - } catch (error) { - console.log(error); + } catch (err) { + console.error(err); } } } @@ -215,13 +215,13 @@ } } } - } catch (error) { - console.log(error); - if (typeof error !== 'string') { - error = error.message; + } catch (err) { + console.error(err); + if (typeof err !== 'string') { + err = err.message; } - toast.error(`${error}`); + toast.error(`${err}`); // opts.callback({ success: false, error, modelName: opts.modelName }); } } @@ -319,8 +319,8 @@ } } } - } catch (error) { - console.log(error); + } catch (err) { + console.error(err); } } } else { @@ -382,9 +382,9 @@ } } } - } catch (error) { - console.log(error); - toast.error(`${error}`); + } catch (err) { + console.error(err); + toast.error(`${err}`); } } } @@ -514,9 +514,9 @@ } } } - } catch (error) { - console.log(error); - toast.error(`${error}`); + } catch (err) { + console.error(err); + toast.error(`${err}`); } } } diff --git a/src/lib/components/admin/Settings/Pipelines.svelte b/src/lib/components/admin/Settings/Pipelines.svelte index e37d39748..635a83df5 100644 --- a/src/lib/components/admin/Settings/Pipelines.svelte +++ b/src/lib/components/admin/Settings/Pipelines.svelte @@ -104,7 +104,7 @@ valves_spec = null; if (PIPELINES_LIST.length > 0) { - console.log(selectedPipelinesUrlIdx); + console.debug(selectedPipelinesUrlIdx); pipelines = await getPipelines(localStorage.token, selectedPipelinesUrlIdx); if (pipelines.length > 0) { @@ -151,7 +151,7 @@ const res = await uploadPipeline(localStorage.token, file, selectedPipelinesUrlIdx).catch( (error) => { - console.log(error); + console.error(error); toast.error('Something went wrong :/'); return null; } diff --git a/src/lib/components/admin/Users/Groups.svelte b/src/lib/components/admin/Users/Groups.svelte index 5847ebbb1..70e5832ea 100644 --- a/src/lib/components/admin/Users/Groups.svelte +++ b/src/lib/components/admin/Users/Groups.svelte @@ -106,7 +106,7 @@ }; const updateDefaultPermissionsHandler = async (group) => { - console.log(group.permissions); + console.debug(group.permissions); const res = await updateUserDefaultPermissions(localStorage.token, group.permissions).catch( (error) => { diff --git a/src/lib/components/admin/Users/Groups/AddGroupModal.svelte b/src/lib/components/admin/Users/Groups/AddGroupModal.svelte index b37da5a18..12cddf398 100644 --- a/src/lib/components/admin/Users/Groups/AddGroupModal.svelte +++ b/src/lib/components/admin/Users/Groups/AddGroupModal.svelte @@ -31,10 +31,6 @@ description = ''; userIds = []; }; - - onMount(() => { - console.log('mounted'); - }); diff --git a/src/lib/components/admin/Users/Groups/EditGroupModal.svelte b/src/lib/components/admin/Users/Groups/EditGroupModal.svelte index 5b6bf6aab..8cc353064 100644 --- a/src/lib/components/admin/Users/Groups/EditGroupModal.svelte +++ b/src/lib/components/admin/Users/Groups/EditGroupModal.svelte @@ -91,7 +91,6 @@ } onMount(() => { - console.log(tabs); selectedTab = tabs[0]; init(); }); diff --git a/src/lib/components/admin/Users/UserList/AddUserModal.svelte b/src/lib/components/admin/Users/UserList/AddUserModal.svelte index 8b3d74c52..cad799eb5 100644 --- a/src/lib/components/admin/Users/UserList/AddUserModal.svelte +++ b/src/lib/components/admin/Users/UserList/AddUserModal.svelte @@ -73,7 +73,7 @@ for (const [idx, row] of rows.entries()) { const columns = row.split(',').map((col) => col.trim()); - console.log(idx, columns); + console.debug(idx, columns); if (idx > 0) { if ( diff --git a/src/lib/components/channel/MessageInput.svelte b/src/lib/components/channel/MessageInput.svelte index d60851fc8..2ad4faa93 100644 --- a/src/lib/components/channel/MessageInput.svelte +++ b/src/lib/components/channel/MessageInput.svelte @@ -80,7 +80,7 @@ const inputFilesHandler = async (inputFiles) => { inputFiles.forEach((file) => { - console.log('Processing file:', { + console.info('Processing file:', { name: file.name, type: file.type, size: file.size, @@ -91,7 +91,7 @@ ($config?.file?.max_size ?? null) !== null && file.size > ($config?.file?.max_size ?? 0) * 1024 * 1024 ) { - console.log('File exceeds max size limit:', { + console.error('File exceeds max size limit:', { fileSize: file.size, maxSize: ($config?.file?.max_size ?? 0) * 1024 * 1024 }); @@ -163,14 +163,14 @@ const uploadedFile = await uploadFile(localStorage.token, file); if (uploadedFile) { - console.log('File upload completed:', { + console.info('File upload completed:', { id: uploadedFile.id, name: fileItem.name, collection: uploadedFile?.meta?.collection_name }); if (uploadedFile.error) { - console.warn('File upload warning:', uploadedFile.error); + console.error('File upload warning:', uploadedFile.error); toast.warning(uploadedFile.error); } @@ -193,7 +193,6 @@ const handleKeyDown = (event: KeyboardEvent) => { if (event.key === 'Escape') { - console.log('Escape'); draggedOver = false; } }; @@ -215,7 +214,6 @@ const onDrop = async (e) => { e.preventDefault(); - console.log(e); if (e.dataTransfer?.files) { const inputFiles = Array.from(e.dataTransfer?.files); @@ -270,7 +268,6 @@ }); onDestroy(() => { - console.log('destroy'); window.removeEventListener('keydown', handleKeyDown); const dropzoneElement = document.getElementById('channel-container'); @@ -479,12 +476,12 @@ } if (e.key === 'Escape') { - console.log('Escape'); + console.info('Escape'); } }} on:paste={async (e) => { e = e.detail.event; - console.log(e); + console.info(e); }} /> diff --git a/src/lib/components/channel/Messages.svelte b/src/lib/components/channel/Messages.svelte index e1bc326b3..9a2bf4cb2 100644 --- a/src/lib/components/channel/Messages.svelte +++ b/src/lib/components/channel/Messages.svelte @@ -51,7 +51,7 @@ {#if !top} { - console.log('visible'); + console.info('visible'); if (!messagesLoading) { loadMoreMessages(); } diff --git a/src/lib/components/channel/Thread.svelte b/src/lib/components/channel/Thread.svelte index 287c7b06a..0b328dc23 100644 --- a/src/lib/components/channel/Thread.svelte +++ b/src/lib/components/channel/Thread.svelte @@ -54,7 +54,7 @@ }; const channelEventHandler = async (event) => { - console.log(event); + console.debug(event); if (event.channel_id === channel.id) { const type = event?.data?.type ?? null; const data = event?.data?.data ?? null; diff --git a/src/lib/components/chat/Artifacts.svelte b/src/lib/components/chat/Artifacts.svelte index b15f38d22..f53e75507 100644 --- a/src/lib/components/chat/Artifacts.svelte +++ b/src/lib/components/chat/Artifacts.svelte @@ -154,7 +154,7 @@ url.pathname + url.search + url.hash ); } else { - console.log('External navigation blocked:', url.href); + console.info('External navigation blocked:', url.href); } } },