DRY pass #2, batch 8: stale comments (#5372)

useNoteList no longer names a Search view; the desktop adapter's M10.7 plan
gave way to sync under the local core; local.ts's sharing comment sat above
settings; AccountList and password_resets still said there was no mail path;
NoteEditor kept an orphan checklist-flag comment, a textarea comment from
before blocks, and the link-preview comment above the file picker; the
serialize docstring's growth plan is now what it holds.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

frontend/src/adapters/index.ts

@@ -2,9 +2,8 @@
 // `api/client` in stores/views.
 //
 // The desktop shell runs fully offline against the on-device SQLite core
-// (`local.ts`, over Tauri `invoke`); the web build uses REST. A server-connected
-// desktop (opt-in sync) comes in M10.7 — at which point this becomes
-// `isDesktop() && !hasConfiguredServer() ? local : rest`.
+// (`local.ts`, over Tauri `invoke`); the web build uses REST. A desktop linked to a
+// server still reads and writes locally: the core syncs underneath.
 
 import { isDesktop } from "../desktop/bridge";
 import { rest } from "./rest";

frontend/src/adapters/local.ts

@@ -84,14 +84,15 @@ export const local: Repo = {
     titles: () => invoke<TitleEntry[]>("notes_titles"),
   },
 
-  // Sharing is between accounts on a server, so these go to the linked one (#5175).
-  // Unlinked, each rejects with the core's explanation, which the dialog shows.
+  // The server's admin settings aren't reached from the desktop, linked or not.
   settings: {
     list: () => Promise.reject<ServerSetting[]>(new Error(NEEDS_SERVER)),
     save: () => Promise.reject<ServerSetting[]>(new Error(NEEDS_SERVER)),
     sendTestEmail: () => Promise.reject<{ to: string }>(new Error(NEEDS_SERVER)),
   },
 
+  // Sharing is between accounts on a server, so these go to the linked one (#5175).
+  // Unlinked, each rejects with the core's explanation, which the dialog shows.
   shares: {
     directory: () => invoke<Directory>("shares_directory"),
     list: (noteId) => invoke<NoteShare[]>("shares_list", { noteId }),

frontend/src/components/AccountList.vue

@@ -7,7 +7,8 @@ import OneTimeLink from "./OneTimeLink.vue";
 import { useRowAction, useLoad } from "../composables/useAction";
 
 // Admin: the accounts on this instance, and a password reset link for any of them
-// (#5173). There is no mail path, so the admin hands the link over.
+// (#5173). The admin hands the link over; a person can also mail one to themselves
+// when the server can send email (#5266).
 
 interface Account {
   id: string;

frontend/src/components/NoteEditor.vue

@@ -47,9 +47,6 @@ function setBody(text: string): void {
   blocks.value = splitBlocks(text);
 }
 const labelList = ref<NoteLabel[]>(props.note ? [...props.note.labels] : []);
-// Whether this editor is showing the checklist. A note HAS a checklist (M13 step 2)
-// rather than BEING one, so this is a view flag, not a property of the note: it turns
-// on when the note already carries items, and when someone asks for one.
 const saving = ref(false);
 const root = ref<HTMLElement | null>(null);
 // One element per block, keyed by the block's id — the only thing about a block that
@@ -489,12 +486,6 @@ async function uploadFile(file: File) {
   }
 }
 
-// ---- link previews ----
-//
-// Nothing to trigger any more: the server unfurls a note's URLs in the background
-// after each save (`unfurl_queue.py`) and the preview arrives on a later read. What
-// is left here is removing one you don't want — the editor is the only place with
-// room to offer that, and the card deliberately doesn't.
 async function onFileChange(e: Event) {
   const input = e.target as HTMLInputElement;
   const file = input.files?.[0];
@@ -638,7 +629,10 @@ function revPreview(rev: NoteRevision): string {
           device until you remove it.
         </p>
 
-        <!-- Fetched automatically after each save; removable here and nowhere else. -->
+        <!-- The server unfurls a note's URLs in the background after each save
+             (`unfurl_queue.py`) and the preview arrives on a later read. Removing one
+             happens here and nowhere else: the editor has room to offer it, and the
+             card deliberately doesn't. -->
         <div v-if="liveNote.previews.length" class="flex flex-col gap-2">
           <LinkPreview
             v-for="p in liveNote.previews"
@@ -698,8 +692,8 @@ function revPreview(rev: NoteRevision): string {
             />
           </template>
         </div>
-        <!-- No checklist component. The items are lines of the textarea above, which
-             is what lets a list sit between two paragraphs (M304). -->
+        <!-- No checklist component. The items are blocks above, each one line of the
+             body, which is what lets a list sit between two paragraphs (M304). -->
 
         <div v-if="labelList.length" class="flex flex-wrap gap-1.5 pt-1">
           <span

frontend/src/composables/useNoteList.ts

@@ -2,12 +2,12 @@ import { ref, type Ref } from "vue";
 import type { Note } from "../stores/notes";
 import { errorMessage } from "../api/errors";
 
-// The load-a-list-of-notes scaffold every recall view (Search, Timeline, Reminders)
+// The load-a-list-of-notes scaffold every recall view (Timeline, Reminders)
 // hand-rolled: an items ref + loading/error flags + a load() that fills them and never
 // leaves a half-state. The view supplies just the fetcher; wire the returned refs into
 // <AsyncState> and pass `items` to <EmptyState>/cards.
 //
-// `fetcher` returns the notes to show (or [] — e.g. an empty search box). A thrown
+// `fetcher` returns the notes to show. A thrown
 // ApiError's `.error` becomes the message; otherwise `fallbackError` is shown.
 export function useNoteList(
   fetcher: () => Promise<Note[]>,