fix(editor): address post-rebase review feedback

Restore IME-safe enter handling, touch paste access, and pending JSON edits after the re_editor migration. Also align empty-message saves across edit surfaces.

Author: XiaoBuHaly

lib/desktop/message_edit_dialog.dart

@@ -94,8 +94,6 @@ class _MessageEditDesktopDialogState extends State<_MessageEditDesktopDialog> {
                       TextButton.icon(
                         onPressed: () {
                           final text = _controller.text.trim();
-                          // TODO: Provide user feedback (disable button or show snackbar) when content is empty.
-                          if (text.isEmpty) return;
                           Navigator.of(context).pop<MessageEditResult>(
                             MessageEditResult(content: text, shouldSend: true),
                           );
@@ -117,8 +115,6 @@ class _MessageEditDesktopDialogState extends State<_MessageEditDesktopDialog> {
                       TextButton.icon(
                         onPressed: () {
                           final text = _controller.text.trim();
-                          // TODO: Provide user feedback (disable button or show snackbar) when content is empty.
-                          if (text.isEmpty) return;
                           Navigator.of(context).pop<MessageEditResult>(
                             MessageEditResult(content: text, shouldSend: false),
                           );
@@ -151,7 +147,9 @@ class _MessageEditDesktopDialogState extends State<_MessageEditDesktopDialog> {
                     padding: const EdgeInsets.fromLTRB(16, 0, 16, 16),
                     child: Container(
                       decoration: BoxDecoration(
-                        color: isDark ? Colors.white10 : const Color(0xFFF7F7F9),
+                        color: isDark
+                            ? Colors.white10
+                            : const Color(0xFFF7F7F9),
                         borderRadius: BorderRadius.circular(12),
                         border: Border.all(
                           color: cs.outlineVariant.withValues(alpha: 0.18),

lib/desktop/setting/providers_pane.dart

@@ -808,7 +808,7 @@ class _DesktopProviderDetailPaneState
   final CodeLineEditingController _saJsonCtrl = CodeLineEditingController();
   final TextEditingController _apiPathCtrl = TextEditingController();
   Timer? _saJsonSaveTimer;
-  String _lastSavedSaJson = '';
+  String _lastPersistedSaJson = '';
 
   void _syncCtrl(TextEditingController c, String newText) {
     final v = c.value;
@@ -826,20 +826,26 @@ class _DesktopProviderDetailPaneState
     c.setTextSafely(newText);
   }
 
+  bool _hasPendingSaJsonChanges() {
+    return _saJsonSaveTimer != null || _saJsonCtrl.text != _lastPersistedSaJson;
+  }
+
   void _syncControllersFromConfig(ProviderConfig cfg) {
     _syncCtrl(_apiKeyCtrl, cfg.apiKey);
     _syncCtrl(_baseUrlCtrl, cfg.baseUrl);
     _syncCtrl(_apiPathCtrl, cfg.chatPath ?? '/chat/completions');
     _syncCtrl(_locationCtrl, cfg.location ?? '');
     _syncCtrl(_projectIdCtrl, cfg.projectId ?? '');
-    _syncCodeCtrl(_saJsonCtrl, cfg.serviceAccountJson ?? '');
-    _lastSavedSaJson = cfg.serviceAccountJson ?? '';
+    final persistedSaJson = cfg.serviceAccountJson ?? '';
+    if (!_saJsonCtrl.isComposing && !_hasPendingSaJsonChanges()) {
+      _syncCodeCtrl(_saJsonCtrl, persistedSaJson);
+    }
+    _lastPersistedSaJson = persistedSaJson;
   }
 
   Future<void> _saveSaJsonNow(SettingsProvider sp) async {
     final text = _saJsonCtrl.text;
-    if (text == _lastSavedSaJson) return;
-    _lastSavedSaJson = text;
+    if (text == _lastPersistedSaJson) return;
     final old = sp.getProviderConfig(
       widget.providerKey,
       defaultName: widget.displayName,
@@ -848,6 +854,7 @@ class _DesktopProviderDetailPaneState
       widget.providerKey,
       old.copyWith(serviceAccountJson: text),
     );
+    _lastPersistedSaJson = text;
   }
 
   void _scheduleSaJsonSave(SettingsProvider sp) {
@@ -1522,7 +1529,8 @@ class _DesktopProviderDetailPaneState
                         },
                         child: Container(
                           decoration: BoxDecoration(
-                            color: Theme.of(context).brightness == Brightness.dark
+                            color:
+                                Theme.of(context).brightness == Brightness.dark
                                 ? Colors.white10
                                 : const Color(0xFFF7F7F9),
                             borderRadius: BorderRadius.circular(10),

lib/features/chat/pages/message_edit_page.dart

@@ -52,7 +52,9 @@ class _MessageEditPageState extends State<MessageEditPage> {
           padding: const EdgeInsets.all(16),
           child: Container(
             decoration: BoxDecoration(
-              color: Theme.of(context).brightness == Brightness.dark ? Colors.white10 : const Color(0xFFF2F3F5),
+              color: Theme.of(context).brightness == Brightness.dark
+                  ? Colors.white10
+                  : const Color(0xFFF2F3F5),
               borderRadius: BorderRadius.circular(12),
             ),
             clipBehavior: Clip.antiAlias,

lib/features/chat/widgets/message_edit_sheet.dart

@@ -99,8 +99,6 @@ class _MessageEditSheetState extends State<_MessageEditSheet> {
                         onTap: () {
                           Haptics.light();
                           final text = _controller.text.trim();
-                          // TODO: Provide user feedback (disable button or show snackbar) when content is empty.
-                          if (text.isEmpty) return;
                           Navigator.of(context).pop<MessageEditResult>(
                             MessageEditResult(content: text, shouldSend: true),
                           );
@@ -140,8 +138,6 @@ class _MessageEditSheetState extends State<_MessageEditSheet> {
                         onTap: () {
                           Haptics.light();
                           final text = _controller.text.trim();
-                          // TODO: Provide user feedback (disable button or show snackbar) when content is empty.
-                          if (text.isEmpty) return;
                           Navigator.of(context).pop<MessageEditResult>(
                             MessageEditResult(content: text, shouldSend: false),
                           );
@@ -172,7 +168,9 @@ class _MessageEditSheetState extends State<_MessageEditSheet> {
               Expanded(
                 child: Container(
                   decoration: BoxDecoration(
-                    color: Theme.of(context).brightness == Brightness.dark ? Colors.white10 : const Color(0xFFF2F3F5),
+                    color: Theme.of(context).brightness == Brightness.dark
+                        ? Colors.white10
+                        : const Color(0xFFF2F3F5),
                     borderRadius: BorderRadius.circular(20),
                   ),
                   clipBehavior: Clip.antiAlias,