fix: resolve openclaw partial artifacts and eliminate legacy fallback code

lib/app/app_controller_desktop_thread_actions.dart

@@ -742,6 +742,7 @@ extension AppControllerDesktopThreadActions on AppController {
               math.max(1, pollDelay.inSeconds))
           .ceil(),
     );
+    var artifactRetries = 0;
     for (var attempt = 0; attempt < maxAttempts; attempt += 1) {
       if (disposedInternal) {
         return;
@@ -774,6 +775,17 @@ extension AppControllerDesktopThreadActions on AppController {
           continue;
         }
         if (aiGatewayPendingSessionKeysInternal.contains(sessionKey)) {
+          final hasRequiredExts = current.requiredArtifactExtensions.isNotEmpty;
+          final hasEnoughArtifacts = !hasRequiredExts ||
+              current.requiredArtifactExtensions.every((ext) {
+                return result.artifacts.any(
+                  (a) => a.relativePath.toLowerCase().endsWith(ext.toLowerCase()),
+                );
+              });
+          if (!hasEnoughArtifacts && artifactRetries < 3) {
+            artifactRetries += 1;
+            continue;
+          }
           await applyGatewayChatResultInternal(
             sessionKey: sessionKey,
             target: target,

lib/runtime/external_code_agent_acp_desktop_transport.dart

@@ -175,6 +175,14 @@ class ExternalCodeAgentAcpDesktopTransport implements GoTaskServiceClient {
         );
       }
       final result = _recoveredResultFromTaskSnapshot(snapshot);
+      final resultArtifacts = _castMap(result['artifacts']);
+      final artifactItems = resultArtifacts['items'] ?? resultArtifacts;
+      final hasArtifacts = result.isNotEmpty &&
+          (artifactItems is List && artifactItems.isNotEmpty ||
+           result['artifacts'] is List && (result['artifacts'] as List).isNotEmpty);
+      if (!hasArtifacts && status == 'completed' && attempt < attempts - 1) {
+        continue;
+      }
       if (result.isNotEmpty) {
         return goTaskServiceResultFromAcpResponse(
           <String, dynamic>{
@@ -295,14 +303,14 @@ class ExternalCodeAgentAcpDesktopTransport implements GoTaskServiceClient {
     String status,
   ) {
     final error = _castMap(snapshot['error']);
-    final message = _firstNonEmptyDisplayText(
-      <String, dynamic>{...error, ...snapshot},
-      const <String>['message', 'error', 'errorMessage', 'reason', 'code'],
-    );
-    final code = _firstNonEmptyDisplayText(
-      <String, dynamic>{...error, ...snapshot},
-      const <String>['code', 'errorCode'],
-    );
+    final rawMessage =
+        error['message'] ??
+        error['error'] ??
+        snapshot['message'] ??
+        snapshot['error'];
+    final message = rawMessage?.toString().trim() ?? '';
+    final rawCode = error['code'] ?? snapshot['code'];
+    final code = rawCode?.toString().trim() ?? '';
     final result = <String, dynamic>{
       'success': false,
       'status': status,
@@ -316,68 +324,6 @@ class ExternalCodeAgentAcpDesktopTransport implements GoTaskServiceClient {
     return result;
   }
 
-  String _firstNonEmptyDisplayText(
-    Map<String, dynamic> values,
-    List<String> keys,
-  ) {
-    for (final key in keys) {
-      final value = _displayText(values[key]).trim();
-      if (value.isNotEmpty) {
-        return value;
-      }
-    }
-    return '';
-  }
-
-  String _displayText(Object? value, [Set<Object>? visited]) {
-    final seen = visited ?? <Object>{};
-    if (value == null) {
-      return '';
-    }
-    if (value is String) {
-      return value.trim();
-    }
-    if (value is Map) {
-      if (!seen.add(value)) {
-        return '';
-      }
-      final map = value.cast<String, dynamic>();
-      for (final key in const <String>[
-        'output',
-        'summary',
-        'resultSummary',
-        'message',
-        'content',
-        'text',
-        'delta',
-        'output_text',
-      ]) {
-        final extracted = _displayText(map[key], seen);
-        if (extracted.isNotEmpty) {
-          return extracted;
-        }
-      }
-      for (final key in const <String>['result', 'payload', 'data']) {
-        final extracted = _displayText(map[key], seen);
-        if (extracted.isNotEmpty) {
-          return extracted;
-        }
-      }
-      return '';
-    }
-    if (value is List) {
-      if (!seen.add(value)) {
-        return '';
-      }
-      return value
-          .map((item) => _displayText(item, seen))
-          .where((item) => item.isNotEmpty)
-          .join('\n')
-          .trim();
-    }
-    return value.toString().trim();
-  }
-
   Map<String, dynamic> _castMap(Object? value) {
     if (value is Map<String, dynamic>) {
       return value;

test/runtime/assistant_execution_target_test.dart

@@ -4174,13 +4174,13 @@ void main() {
           controller,
           'openclaw-missing-screenshot',
           'ready',
-          const Duration(milliseconds: 500),
+          const Duration(seconds: 10),
         );
         await _waitForThreadArtifactSyncStatusWithin(
           controller,
           'openclaw-missing-screenshot',
           'no-exported-artifacts',
-          const Duration(milliseconds: 500),
+          const Duration(seconds: 10),
         );
 
         final thread = controller.requireTaskThreadForSessionInternal(