Incomplete Svelte 4 → 5 Migration: Legacy syntax still in use

[rohanshrma222]

What

The codebase uses Svelte 5 ("svelte": "^5.45.5") but still contains legacy Svelte 4 syntax in multiple files. This inconsistency can cause confusion and potential bugs during the migration.

Steps to reproduce the behavior:

1. Search for on:click or onMount in the codebase
2. Observe multiple files still using Svelte 4 event handlers and lifecycle hooks

Expected behavior

All files should use Svelte 5 syntax:

• on:click → onclick
• onMount → $effect
• onDestroy → $effect (with cleanup function)

Screenshots

Platform (Desktop, Mobile, Hunger Games)

• OS:
• Platform

Part of

[VaiTon]

onMount is still ok in svelte 5. Where did you find it is not?

[rohanshrma222]

onMount is still ok in svelte 5. Where did you find it is not?

It was an optional change,i did that just for consistency.

[rohanshrma222]

Would you want me to revert it?

[VaiTon]

@rohanshrma222 I want to know where did you read about onMount being deprecated

[rohanshrma222]

I didn’t find any official source saying onMount is deprecated. I assumed it should be migrated to $effect while updating lifecycle patterns for consistency.

[VaiTon]

I assumed it should be migrated to $effect while updating lifecycle patterns for consistency.

I find it hard to believe :)