Merge pull request #49 from shadowfish07/issue-39

refactor: 重构 OpenRouter API 客户端并改进 AI 翻译功能

Author: shadowfish07

.releaserc.json

@@ -20,7 +20,7 @@
           { "type": "docs", "release": false },
           { "type": "style", "release": false },
           { "type": "chore", "release": false },
-          { "type": "refactor", "release": "patch" },
+          { "type": "refactor", "release": false },
           { "type": "test", "release": false },
           { "type": "build", "release": false },
           { "type": "ci", "release": false },

lib/data/service/openrouter_api_client.dart

@@ -10,13 +10,21 @@ import 'package:result_dart/result_dart.dart';
 /// OpenRouter API 客户端
 /// 提供与 OpenRouter API 的交互功能，支持流式聊天完成
 class OpenRouterApiClient {
-  OpenRouterApiClient(this._sharedPreferencesService, {String? baseUrl})
-      : _baseUrl = baseUrl ?? 'https://openrouter.ai/api/v1';
+  OpenRouterApiClient(this._sharedPreferencesService,
+      {String? baseUrl, http.Client? httpClient})
+      : _baseUrl = baseUrl ?? 'https://openrouter.ai/api/v1',
+        _httpClient = httpClient ?? http.Client();
 
   final String _baseUrl;
   final SharedPreferencesService _sharedPreferencesService;
+  final http.Client _httpClient;
   String? _apiKey;
 
+  /// 释放资源
+  void dispose() {
+    _httpClient.close();
+  }
+
   /// 初始化API密钥
   Future<void> _initApiKey() async {
     if (_apiKey == null) {
@@ -37,7 +45,6 @@ class OpenRouterApiClient {
   Map<String, String> get _headers => {
         'Authorization': 'Bearer $_apiKey',
         'Content-Type': 'application/json',
-        // 'HTTP-Referer': 'https://readeck-app.zqydev.me',
         'X-Title': 'ReadeckApp',
       };
 
@@ -88,7 +95,7 @@ class OpenRouterApiClient {
       appLogger.d('发送流式聊天请求到 OpenRouter: $uri');
       appLogger.d('请求体: ${jsonEncode(requestBody)}');
 
-      final streamedResponse = await request.send();
+      final streamedResponse = await _httpClient.send(request);
 
       if (streamedResponse.statusCode != 200) {
         appLogger.w('OpenRouter API 请求失败。状态码: ${streamedResponse.statusCode}');
@@ -180,7 +187,7 @@ class OpenRouterApiClient {
 
       appLogger.d('发送聊天请求到 OpenRouter: $uri');
 
-      final response = await http.post(
+      final response = await _httpClient.post(
         uri,
         headers: _headers,
         body: jsonEncode(requestBody),
@@ -262,7 +269,7 @@ class OpenRouterApiClient {
       appLogger.d('发送流式文本完成请求到 OpenRouter: $uri');
       appLogger.d('请求体: ${jsonEncode(requestBody)}');
 
-      final streamedResponse = await request.send();
+      final streamedResponse = await _httpClient.send(request);
 
       if (streamedResponse.statusCode != 200) {
         appLogger.w('OpenRouter API 请求失败。状态码: ${streamedResponse.statusCode}');
@@ -357,7 +364,7 @@ class OpenRouterApiClient {
 
       appLogger.d('发送文本完成请求到 OpenRouter: $uri');
 
-      final response = await http.post(
+      final response = await _httpClient.post(
         uri,
         headers: _headers,
         body: jsonEncode(requestBody),
@@ -400,7 +407,7 @@ class OpenRouterApiClient {
 
       appLogger.d('获取 OpenRouter 模型列表: $uri');
 
-      final response = await http.get(uri, headers: _headers);
+      final response = await _httpClient.get(uri, headers: _headers);
 
       if (response.statusCode == 200) {
         final responseData = jsonDecode(response.body);

lib/routing/router.dart

@@ -198,6 +198,7 @@ GoRouter router(SettingsRepository settingsRepository) => GoRouter(
                 context.read(),
                 context.read(),
                 context.read(),
+                context.read(),
                 bookmark,
               );
               return ChangeNotifierProvider.value(

lib/ui/bookmarks/view_models/bookmark_detail_viewmodel.dart

@@ -4,11 +4,16 @@ import 'package:readeck_app/data/repository/bookmark/bookmark_repository.dart';
 import 'package:readeck_app/domain/models/bookmark/bookmark.dart';
 import 'package:readeck_app/domain/use_cases/bookmark_operation_use_cases.dart';
 import 'package:readeck_app/data/repository/label/label_repository.dart';
+import 'package:readeck_app/data/repository/settings/settings_repository.dart';
 import 'package:readeck_app/main.dart';
 
 class BookmarkDetailViewModel extends ChangeNotifier {
-  BookmarkDetailViewModel(this._bookmarkRepository,
-      this._bookmarkOperationUseCases, this._labelRepository, this._bookmark) {
+  BookmarkDetailViewModel(
+      this._bookmarkRepository,
+      this._bookmarkOperationUseCases,
+      this._labelRepository,
+      this._settingsRepository,
+      this._bookmark) {
     // 注册标签数据变化监听器
     _labelRepository.addListener(_onLabelsChanged);
     // 注册书签数据变化监听器
@@ -41,6 +46,7 @@ class BookmarkDetailViewModel extends ChangeNotifier {
   final BookmarkRepository _bookmarkRepository;
   final BookmarkOperationUseCases _bookmarkOperationUseCases;
   final LabelRepository _labelRepository;
+  final SettingsRepository _settingsRepository;
   Bookmark _bookmark;
 
   // AI翻译相关状态
@@ -230,6 +236,13 @@ class BookmarkDetailViewModel extends ChangeNotifier {
     try {
       appLogger.i('开始AI翻译内容');
 
+      // 检查OpenRouter API Key是否已配置
+      final apiKeyResult = await _settingsRepository.getOpenRouterApiKey();
+      if (apiKeyResult.isError() || apiKeyResult.getOrNull()?.isEmpty == true) {
+        appLogger.w('OpenRouter API Key未配置，无法进行AI翻译');
+        throw '请先在设置中配置OpenRouter API Key';
+      }
+
       _isTranslateMode = true;
       _isTranslating = true;
       _translatedContent = ''; // 清空之前的翻译内容

lib/ui/settings/widgets/ai_settings_screen.dart

@@ -131,12 +131,6 @@ class _AiSettingsScreenState extends State<AiSettingsScreen> {
                   prefixIcon: Icon(Icons.key),
                 ),
                 obscureText: true,
-                validator: (value) {
-                  if (value == null || value.trim().isEmpty) {
-                    return '请输入 API 密钥';
-                  }
-                  return null;
-                },
                 onFieldSubmitted: (_) => _saveApiKey(),
               ),
               const SizedBox(height: 16),

test/fixtures/test_data.dart

@@ -1,5 +1,7 @@
 import 'package:readeck_app/domain/models/daily_read_history/daily_read_history.dart';
 import 'package:readeck_app/domain/models/bookmark_article/bookmark_article.dart';
+import 'package:readeck_app/domain/models/bookmark/bookmark.dart';
+import 'package:readeck_app/domain/models/bookmark/label_info.dart';
 
 /// 测试用的书签文章数据
 class TestBookmarkArticleData {
@@ -52,6 +54,113 @@ class TestDailyReadHistoryData {
   }
 }
 
+/// 测试用的书签数据
+class TestBookmarkData {
+  static Bookmark createSample({
+    String? id,
+    String? title,
+    String? url,
+    String? siteName,
+    String? description,
+    bool? isMarked,
+    bool? isArchived,
+    int? readProgress,
+    List<String>? labels,
+    DateTime? created,
+    String? imageUrl,
+  }) {
+    return Bookmark(
+      id: id ?? 'test-bookmark-1',
+      title: title ?? 'Test Bookmark Title',
+      url: url ?? 'https://example.com/test',
+      siteName: siteName ?? 'example.com',
+      description: description ?? 'Test bookmark description',
+      isMarked: isMarked ?? false,
+      isArchived: isArchived ?? false,
+      readProgress: readProgress ?? 0,
+      labels: labels ?? ['tech', 'flutter'],
+      created: created ?? DateTime.parse('2024-01-01T00:00:00Z'),
+      imageUrl: imageUrl,
+    );
+  }
+
+  static List<Bookmark> createMultipleSamples(int count) {
+    return List.generate(
+      count,
+      (index) => createSample(
+        id: 'test-bookmark-${index + 1}',
+        title: 'Test Bookmark ${index + 1}',
+        url: 'https://example.com/test-${index + 1}',
+      ),
+    );
+  }
+
+  static Map<String, dynamic> createSampleJson({
+    String? id,
+    String? title,
+    String? url,
+    String? siteName,
+    String? description,
+    bool? isMarked,
+    bool? isArchived,
+    int? readProgress,
+    List<String>? labels,
+    String? created,
+    String? imageUrl,
+  }) {
+    return {
+      'id': id ?? 'test-bookmark-1',
+      'title': title ?? 'Test Bookmark Title',
+      'url': url ?? 'https://example.com/test',
+      'site_name': siteName ?? 'example.com',
+      'description': description ?? 'Test bookmark description',
+      'is_marked': isMarked ?? false,
+      'is_archived': isArchived ?? false,
+      'read_progress': readProgress ?? 0,
+      'labels': labels ?? ['tech', 'flutter'],
+      'created': created ?? '2024-01-01T00:00:00Z',
+      'image_url': imageUrl,
+    };
+  }
+}
+
+/// 测试用的标签数据
+class TestLabelData {
+  static LabelInfo createSample({
+    String? name,
+    int? count,
+    String? href,
+    String? hrefBookmarks,
+  }) {
+    return LabelInfo(
+      name: name ?? 'tech',
+      count: count ?? 10,
+      href: href ?? '/api/labels/tech',
+      hrefBookmarks: hrefBookmarks ?? '/api/bookmarks?labels=tech',
+    );
+  }
+
+  static List<LabelInfo> createMultipleSamples(int count) {
+    return List.generate(
+      count,
+      (index) => createSample(
+        name: 'label-${index + 1}',
+        count: (index + 1) * 5,
+      ),
+    );
+  }
+
+  static Map<String, dynamic> createSampleJson({
+    String? name,
+    int? count,
+  }) {
+    return {
+      'name': name ?? 'tech',
+      'count': count ?? 10,
+    };
+  }
+}
+
 /// 测试用的书签ID列表
 class TestBookmarkIds {
   static const List<String> sample1 = ['bookmark1', 'bookmark2', 'bookmark3'];