From 71009bddc73865133f7d29fffc4e9a2bfad98ad2 Mon Sep 17 00:00:00 2001 From: Jakob Schlanstedt Date: Tue, 28 Oct 2025 18:30:48 +0100 Subject: [PATCH] refactor(note-create): simplify createNote switch to equivalent small ifs --- apps/client/src/services/note_create.ts | 25 +++++++------------------ 1 file changed, 7 insertions(+), 18 deletions(-) diff --git a/apps/client/src/services/note_create.ts b/apps/client/src/services/note_create.ts index 906afec57..56c1cd880 100644 --- a/apps/client/src/services/note_create.ts +++ b/apps/client/src/services/note_create.ts @@ -128,25 +128,14 @@ async function createNote( resolvedOptions = maybeResolvedOptions; } - switch (resolvedOptions.target) { - case "into": - return await createNoteAtNote("into", {...options} as CreateNoteAtUrlOpts); - - case "before": - return await createNoteAtNote("before", resolvedOptions as CreateNoteBeforeUrlOpts); - - case "after": - return await createNoteAtNote("after", resolvedOptions as CreateNoteAfterUrlOpts); - - case "inbox": - return await createNoteIntoInbox(resolvedOptions as CreateNoteIntoInboxOpts); - - default: { - console.warn("[createNote] Unknown target:", options.target, resolvedOptions); - toastService.showMessage("Unknown note creation target."); // optional - return { note: null, branch: undefined }; - } + if (resolvedOptions.target === "inbox") { + return createNoteIntoInbox(resolvedOptions as CreateNoteIntoInboxOpts); } + + return createNoteAtNote( + resolvedOptions.target as "into" | "after" | "before", + resolvedOptions as CreateNoteAtUrlOpts + ); } async function promptForType(