Fix Svelte 5 ownership and state_proxy_equality_mismatch warnings

[myieye]

• Remove redundant === check in StompGuard watch callback that triggered \state_proxy_equality_mismatch\ warning (the check was just an optimization, not a correctness requirement)
• Use bind:entry to establish proper ownership for EntryEditor and prevent warnings.

[github-actions]

UI unit Tests

  1 files  ±0   54 suites  ±0   25s ⏱️ ±0s
140 tests ±0  140 ✅ ±0  0 💤 ±0  0 ❌ ±0 
207 runs  ±0  207 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 4c7e50a. ± Comparison against base commit 3f62d45.

♻️ This comment has been updated with latest results.

[argos-ci]

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build	Status	Details	Updated (UTC)
default (Inspect)	✅ No changes detected	-	Mar 10, 2026, 1:21 PM

[github-actions]

C# Unit Tests

162 tests  ±0   162 ✅ ±0   19s ⏱️ -1s
 23 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 4c7e50a. ± Comparison against base commit 3f62d45.

♻️ This comment has been updated with latest results.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: ec3142a0-b941-4e25-93d2-86975e779be0

📥 Commits

Reviewing files that changed from the base of the PR and between 3f62d45 and 4c7e50a.

📒 Files selected for processing (3)

• frontend/viewer/src/lib/components/stomp/stomp-guard.svelte.ts
• frontend/viewer/src/lib/entry-editor/EditEntryDialog.svelte
• frontend/viewer/src/project/browse/EntryView.svelte

---

📝 Walkthrough

Walkthrough

This pull request modifies three files to implement two-way data binding for entry values across components and removes an early-return optimization from the stomp-guard watch callback. The changes convert entry propagation from static props to bound references, enabling synchronization between parent and child components.

Changes

Cohort / File(s)	Summary
Entry Component Binding frontend/viewer/src/lib/entry-editor/EditEntryDialog.svelte, frontend/viewer/src/project/browse/EntryView.svelte	Converted from passing entry as a static prop to two-way binding (bind:entry). EntryView also changed entry variable from const to let to support mutability required for binding.
Stomp Guard Watch Callback frontend/viewer/src/lib/components/stomp/stomp-guard.svelte.ts	Removed early-return optimization that bypassed processing when newParentValue equals current _value, allowing the callback to always execute subsequent logic.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• sillsdev/languageforge-lexbox#2089: Modifies EntryView.svelte and changes entry value propagation to use mutable/bound entry with updated persistence source handling.
• sillsdev/languageforge-lexbox#1768: Modifies the same stomp-guard watch callback behavior and implementation.
• sillsdev/languageforge-lexbox#1851: Changes the stomp-guard implementation with isDirty getter addition, affecting the same guard logic.

Suggested reviewers

• hahn-kev
• imnasnainaec

Poem

🐰 Two-way bindings dance and sway,
From parent down to child's display,
No more one-way static paths—
The stomp-guard optimization hath
Been trimmed to let the flow run true! 🎉

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch claude/fix-entry-editor-bindings-A4ZFF

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}