我帮你分析了一下你的代码，发现了几个导致双向绑定失效的关键问题，现在给你修改方案和详细解释：

问题根源分析 #

你的附加属性实现和ViewModel的变更通知存在两处核心问题：

1. 事件重复注册导致逻辑混乱：每次附加属性值变化时都重复注册SelectedItemChanged事件，可能引发事件处理的异常流程；
2. 双向绑定不完整：只实现了「TreeView选中项变化→附加属性更新」的单向逻辑，缺少「附加属性更新→TreeView选中项同步」的反向逻辑，同时没有确保附加属性的变更能正确传递到ViewModel；
3. 属性变更通知不规范：ViewModel的SelectedTodoItem在调用OnPropertyChanged时未传递正确属性名，虽不是View→ViewModel失效的直接原因，但会导致ViewModel→View的同步异常。

完整解决方案 #

1. 修正附加属性实现 #

修改TreeViewSelectedItemBehavior类，确保双向绑定的两个方向都能正常工作，同时避免重复注册事件：

using System.Windows;
using System.Windows.Controls;

public static class TreeViewSelectedItemBehavior
{
    public static readonly DependencyProperty SelectedItemProperty =
        DependencyProperty.RegisterAttached(
            "SelectedItem",
            typeof(object),
            typeof(TreeViewSelectedItemBehavior),
            new FrameworkPropertyMetadata(
                null,
                FrameworkPropertyMetadataOptions.BindsTwoWayByDefault,
                OnSelectedItemChanged));

    public static object GetSelectedItem(DependencyObject obj)
    {
        return obj.GetValue(SelectedItemProperty);
    }

    public static void SetSelectedItem(DependencyObject obj, object value)
    {
        obj.SetValue(SelectedItemProperty, value);
    }

    private static void OnSelectedItemChanged(DependencyObject d, DependencyPropertyChangedEventArgs e)
    {
        if (d is TreeView treeView)
        {
            // 仅在TreeView加载完成后注册一次事件，避免重复注册
            if (!treeView.IsLoaded)
            {
                treeView.Loaded += TreeView_Loaded;
            }
            else
            {
                // 当附加属性从ViewModel更新时，同步TreeView的SelectedItem
                if (treeView.SelectedItem != e.NewValue)
                {
                    treeView.SelectedItem = e.NewValue;
                }
            }
        }
    }

    private static void TreeView_Loaded(object sender, RoutedEventArgs e)
    {
        if (sender is TreeView treeView)
        {
            treeView.Loaded -= TreeView_Loaded;
            treeView.SelectedItemChanged += TreeView_SelectedItemChanged;
            
            // 初始化时同步附加属性值到TreeView
            var initialSelected = GetSelectedItem(treeView);
            if (treeView.SelectedItem != initialSelected)
            {
                treeView.SelectedItem = initialSelected;
            }
        }
    }

    private static void TreeView_SelectedItemChanged(object sender, RoutedPropertyChangedEventArgs<object> e)
    {
        if (sender is TreeView treeView)
        {
            // 当TreeView选中项变化时，更新附加属性，触发ViewModel绑定更新
            var currentValue = GetSelectedItem(treeView);
            if (currentValue != e.NewValue)
            {
                SetSelectedItem(treeView, e.NewValue);
            }
        }
    }
}

2. 修正ViewModel的属性变更通知 #

修改BaseViewModel，添加基于CallerMemberName的重载，自动获取属性名，避免手动输入错误：

using System.ComponentModel;
using System.Runtime.CompilerServices;

public abstract class BaseViewModel : INotifyPropertyChanged
{
    public event PropertyChangedEventHandler PropertyChanged;

    // 利用CallerMemberName特性自动获取当前属性名，无需手动传递
    protected virtual void OnPropertyChanged([CallerMemberName] string propertyName = null)
    {
        PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
    }
}

更新MainViewModel的SelectedTodoItem属性，简化变更通知调用：

public class MainViewModel : BaseViewModel
{
    public List<TodoItemViewModel> tvTodoItems { get; set; }

    private TodoItemViewModel _selectedTodoItem;
    public TodoItemViewModel SelectedTodoItem
    {
        get => _selectedTodoItem;
        set
        {
            if (_selectedTodoItem != value)
            {
                _selectedTodoItem = value;
                // 自动获取当前属性名"SelectedTodoItem"
                OnPropertyChanged();
                // 可添加调试输出验证是否被调用
                // Console.WriteLine($"选中项更新为：{_selectedTodoItem?.Name}");
            }
        }
    }
}

3. 简化XAML绑定（可选） #

由于附加属性已设置BindsTwoWayByDefault，可以去掉XAML中的Mode=TwoWay：

<TreeView ItemsSource="{Binding tvTodoItems}" 
          helpers:TreeViewSelectedItemBehavior.SelectedItem="{Binding SelectedTodoItem}">
    <TreeView.ItemTemplate>
        <DataTemplate>
            <TextBlock Text="{Binding Name}" />
        </DataTemplate>
    </TreeView.ItemTemplate>
</TreeView>

工作原理说明 #

1. View→ViewModel方向：用户在TreeView中选择新项时，SelectedItemChanged事件触发，将新值设置到附加属性，通过双向绑定自动更新ViewModel的SelectedTodoItem；
2. ViewModel→View方向：当ViewModel的SelectedTodoItem手动更新时，附加属性值随之变化，触发同步逻辑设置TreeView的SelectedItem；
3. 初始化同步：TreeView加载完成后，会将附加属性的初始值同步到TreeView的选中项，确保初始状态正确；
4. 避免重复注册：仅在TreeView加载完成后注册一次SelectedItemChanged事件，避免重复注册导致的逻辑混乱。

现在你可以运行程序测试，选择TreeView中的项时，SelectedTodoItem的setter应该会被正常调用，双向绑定就能完全正常工作了！