Skip to content

fix(ui): add Mode=OneWay to all Run.Text bindings#21

Merged
LXBStudioLLC merged 1 commit into
mainfrom
fix/run-binding-mode-oneway
Jul 5, 2026
Merged

fix(ui): add Mode=OneWay to all Run.Text bindings#21
LXBStudioLLC merged 1 commit into
mainfrom
fix/run-binding-mode-oneway

Conversation

@LXBStudioLLC

Copy link
Copy Markdown
Owner

Summary

Fixes the same Run.Text binding crash class as PR #20 (SizeDisplay), but across all remaining XAML files.

Root Cause

Run.Text defaults to TwoWay binding mode in WPF. When bound to a read-only property (getter only, init-only, or a computed property like List.Count), WPF throws XamlParseException at template instantiation time, crashing the entire view when data is present.

Fixes (7 bindings across 4 files)

File Binding Property type
BrowserSelectionView.xaml:116 StatusText regular setter (display-only)
DashboardView.xaml:55 Email, Phone regular setter (display-only)
DashboardView.xaml:59 Experience.Count read-only (List.Count)
DashboardView.xaml:62 Skills.Count read-only (List.Count)
FindJobsView.xaml:68 Company, Location init-only
LLMSettingsView.xaml:70 StatusText regular setter (display-only)
LLMSettingsView.xaml:73 ModelCountText regular setter (display-only)

Every Run Text="{Binding...}" in the codebase now has Mode=OneWay.

Test Results

Passed! - 57 - Envoy.Core.Tests.dll
Passed! - 61 - Envoy.GhostDetection.Tests.dll
Passed! - 17 - Envoy.Discovery.Tests.dll
Total: 135 passed, 0 failed

Checklist

  • Only XAML files touched (4 files, 7 one-line edits)
  • Build succeeds (Release)
  • All 135 tests pass
  • No CAPTCHA/bot-detection code

Run.Text defaults to TwoWay in WPF. When bound to a read-only or
init-only property, WPF throws XamlParseException at template
instantiation, crashing the entire view.

PR #20 fixed SizeDisplay in LLMSettingsView. This PR fixes the
remaining 7 Run.Text bindings across 4 XAML files:

- BrowserSelectionView.xaml: StatusText
- DashboardView.xaml: Email, Phone, Experience.Count, Skills.Count
- FindJobsView.xaml: Company, Location
- LLMSettingsView.xaml: StatusText, ModelCountText

Every Run Text={Binding...} in the codebase now has Mode=OneWay.
@LXBStudioLLC LXBStudioLLC merged commit e69789d into main Jul 5, 2026
2 checks passed
@LXBStudioLLC LXBStudioLLC deleted the fix/run-binding-mode-oneway branch July 5, 2026 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant