Approving - this is a clean migration that follows the established sveltekit-superforms patterns used elsewhere in the codebase.

A few minor suggestions to consider:

1. Unused type export: BookmarkFormValues in web-admin/src/features/bookmarks/utils.ts is now unused since its import was removed. Consider deleting it to avoid dead code.

2. Redundant button attributes: The Save button has both onClick={submit} and submitForm/form={formId}. Having both may cause double-submission in edge cases. I'd suggest either:

   • <Button onClick={submit} type="primary">Save</Button> (matching the pattern in OrgNameSettings.svelte)
   • Or <Button type="primary" form={formId} submitForm>Save</Button> (relying on native form submission)

3. Error handling: The onUpdate handler doesn't wrap the mutation calls in try/catch. If the mutations throw, users won't see feedback. Other forms like OrgNameSettings.svelte handle this by catching errors and setting form.errors. Worth considering for consistency.

None of these are blockers.

Developed in collaboration with Claude Code