Fix inconsistent vertical scrolling

[Irineu333]

Fixes #15

[Copilot]

Pull Request Overview

This PR refactors the text sample management to use Compose's built-in TextFieldState instead of custom state management. The migration eliminates redundant state synchronization code and leverages Compose's native undo/redo functionality.

Key changes:

• Replaced TextSampleRepository with SampleRepository using TextFieldState for text state management
• Migrated TextEditor component to use TextFieldState directly instead of value/onChange pattern
• Updated onTextLayout callback to work with TextFieldState2 API (receives lambda returning TextLayoutResult)

Reviewed Changes

Copilot reviewed 20 out of 20 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
MatcherAction.kt	Removed obsolete UpdateText action since state is now managed directly via TextFieldState
MatcherViewModel.kt	Updated to use SampleRepository and expose sampleField instead of managing text updates via actions
MatcherScreen.kt	Simplified TextEditor usage by passing TextFieldState directly instead of value/onChange callbacks
TextEditor.*.kt	Migrated to TextFieldState parameter and updated onTextLayout to receive lambda for TextFieldState2 compatibility
SampleRepository.kt	New interface exposing TextFieldState and flows for text/history instead of combined state flow
SampleRepositoryImpl.kt	New implementation using TextFieldState with built-in undo/redo support
PatternStateRepository.kt	Added cleanUpdate method and updated clear signature for consistency
SalvageManagerImpl.kt	Updated to work with new repository interfaces and direct text access
Opened.kt	Changed to accept CharSequence instead of SampleState for simplified comparison
Pattern.kt	Updated sample field type to CharSequence for compatibility with TextFieldState
PatternsSqlDelightDataSource.kt	Added .toString() conversion when persisting CharSequence to database
Inputs.kt	Removed text field as it's no longer needed in the model
HistoryState.kt	Added constructor accepting UndoState for compatibility with TextFieldState
TextLayoutResult.kt	Added bounds checking to prevent index out of bounds errors

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.