mirror of
https://github.com/linshenkx/prompt-optimizer.git
synced 2026-05-06 21:50:27 +08:00
feat: 修复Monorepo构建与依赖解析问题,优化开发命令,更新依赖,完善文档,简化项目结构,统一异步调用,优化错误处理
本次提交主要完成了以下变更: - 在`package.json`中新增`dev:desktop:parallel:fixed`命令,解决并行进程导致的样式丢失问题。 - 更新`experience.md`文档,记录Monorepo中构建与依赖管理的最佳实践和遇到的问题。 - 在`scratchpad.md`中详细记录了修复过程和解决方案,确保后续开发者能够参考。 - 更新pnpm锁定文件,添加dotenv和@prompt-optimizer/core依赖。 - 更新Electron版本至^37.1.0,修复桌面端IndexedDB问题。 - 简化项目结构,删除 `pnpm-lock.yaml` 文件和冗余文档。 - 将多个同步方法改为异步方法,确保模板管理器的操作能够正确处理异步逻辑。 - 优化模板获取和列表加载的逻辑,统一使用 `await` 关键字,避免潜在的时序问题。 这些更新旨在提升项目的稳定性和开发效率,确保最佳实践得到贯彻。
This commit is contained in:
1
.npmrc
1
.npmrc
@@ -5,3 +5,4 @@ strict-peer-dependencies=false
|
||||
enable-pre-post-scripts=true
|
||||
public-hoist-pattern[]=*esbuild*
|
||||
public-hoist-pattern[]=*vue-demi*
|
||||
electron_mirror="https://npmmirror.com/mirrors/electron/"
|
||||
|
||||
105
docs/archives/110-desktop-indexeddb-fix/README.md
Normal file
105
docs/archives/110-desktop-indexeddb-fix/README.md
Normal file
@@ -0,0 +1,105 @@
|
||||
# Desktop IndexedDB问题修复任务总结
|
||||
|
||||
## 📋 任务概述
|
||||
- **任务类型**:Bug修复 + 架构改进
|
||||
- **开始时间**:2025-01-01
|
||||
- **完成时间**:2025-01-01
|
||||
- **状态**:✅ 已完成
|
||||
- **优先级**:高(影响Desktop应用正常使用)
|
||||
|
||||
## 🎯 问题描述
|
||||
用户在Desktop应用中发现,即使在Electron环境下,开发者工具中仍然可以看到IndexedDB数据库,这违反了Desktop应用的架构设计(应该只使用主进程的memory storage)。
|
||||
|
||||
## 🔍 问题分析
|
||||
|
||||
### 根本原因
|
||||
1. **模块级存储创建**:`packages/core/src/services/prompt/factory.ts`中有模块级别的`StorageFactory.createDefault()`调用
|
||||
2. **TemplateLanguageService构造函数**:使用默认参数调用`createDefault()`
|
||||
3. **历史遗留数据**:之前创建的IndexedDB数据持久化存储在浏览器中
|
||||
|
||||
### 架构问题
|
||||
- **设计违反**:Electron渲染进程不应该有任何本地存储实例
|
||||
- **数据不一致**:渲染进程和主进程可能有不同的数据状态
|
||||
- **意外创建**:`createDefault()`方法在任何环境下都会创建IndexedDB
|
||||
|
||||
## 🛠️ 解决方案
|
||||
|
||||
### 核心修复
|
||||
1. **彻底删除`StorageFactory.createDefault()`方法**
|
||||
2. **修复`TemplateLanguageService`构造函数**:改为必须传入storage参数
|
||||
3. **重构`prompt/factory.ts`**:移除模块级存储创建,改为依赖注入
|
||||
4. **修复API调用错误**:`getModels()` → `getAllModels()`
|
||||
|
||||
### 架构改进
|
||||
- **强制明确性**:所有存储创建都必须明确指定类型
|
||||
- **避免意外创建**:防止在不合适环境下自动创建IndexedDB
|
||||
- **代理架构完善**:Electron渲染进程完全使用代理服务
|
||||
|
||||
## 📁 修改的文件
|
||||
|
||||
### Core包修改
|
||||
- `packages/core/src/services/storage/factory.ts` - 删除createDefault()和getCurrentDefault()
|
||||
- `packages/core/src/services/template/languageService.ts` - 构造函数改为必须传入storage
|
||||
- `packages/core/src/services/prompt/factory.ts` - 重构为依赖注入方式
|
||||
- `packages/core/src/services/prompt/service.ts` - 移除重复函数定义
|
||||
- `packages/core/src/index.ts` - 修复导出路径
|
||||
- `packages/core/tests/integration/storage-implementations.test.ts` - 更新测试
|
||||
|
||||
### Desktop包修改
|
||||
- `packages/desktop/package.json` - 添加缺失依赖
|
||||
- `packages/desktop/main.js` - 修复API调用错误
|
||||
- `packages/desktop/build.js` - 创建跨平台构建脚本
|
||||
|
||||
### UI包修改
|
||||
- `packages/ui/src/composables/useAppInitializer.ts` - 修复Electron存储代理
|
||||
|
||||
### 清理的过度修复
|
||||
- 移除DexieStorageProvider中的Electron环境警告
|
||||
- 简化useAppInitializer中的详细调试信息
|
||||
- 删除不必要的listTemplatesByTypeAsync方法
|
||||
|
||||
## 🧪 测试验证
|
||||
|
||||
### 测试结果
|
||||
- ✅ Desktop应用成功启动
|
||||
- ✅ 主进程正确使用memory storage
|
||||
- ✅ 渲染进程使用代理服务
|
||||
- ✅ 模板加载正常(7个模板)
|
||||
- ✅ Web开发服务器运行正常
|
||||
- ✅ 无IndexedDB自动创建
|
||||
|
||||
### 用户验证
|
||||
- ✅ 手动删除IndexedDB后,重新启动应用不再创建IndexedDB
|
||||
- ✅ 应用功能正常,界面加载正常
|
||||
|
||||
## 💡 关键收获
|
||||
|
||||
### 架构原则
|
||||
1. **强制明确性比便利性更重要**:删除`createDefault()`强制开发者明确指定存储类型
|
||||
2. **避免模块级副作用**:模块导入不应该产生存储创建等副作用
|
||||
3. **依赖注入优于默认值**:明确的依赖传递比隐式的默认值更安全
|
||||
|
||||
### 调试经验
|
||||
1. **历史数据影响**:修复代码后仍需清理历史遗留数据
|
||||
2. **环境检测时序**:Electron环境检测需要考虑preload脚本执行时序
|
||||
3. **过度修复识别**:修复过程中要避免不必要的复杂化
|
||||
|
||||
### 代码质量
|
||||
1. **及时清理无用代码**:如`getCurrentDefault()`等失效方法
|
||||
2. **避免过度防御**:如DexieStorageProvider中的环境警告
|
||||
3. **保持接口一致性**:Web和Electron版本应尽可能使用相同接口
|
||||
|
||||
## 📚 相关文档
|
||||
- [Desktop模块修复详情](./desktop-module-fixes.md)
|
||||
- [架构设计文档](../archives/103-desktop-architecture/)
|
||||
- [故障排查清单](../developer/troubleshooting/general-checklist.md)
|
||||
|
||||
## 🔄 后续行动
|
||||
- [ ] 将此次修复经验整理到故障排查清单中
|
||||
- [ ] 考虑添加自动化测试防止类似问题再次发生
|
||||
- [ ] 评估是否需要在其他地方应用类似的架构改进
|
||||
|
||||
---
|
||||
**任务负责人**:AI Assistant
|
||||
**审核状态**:已归档
|
||||
**归档时间**:2025-01-02
|
||||
21
docs/archives/110-desktop-indexeddb-fix/experience.md
Normal file
21
docs/archives/110-desktop-indexeddb-fix/experience.md
Normal file
@@ -0,0 +1,21 @@
|
||||
# "Desktop IndexedDB修复"任务经验总结
|
||||
|
||||
## 核心经验
|
||||
|
||||
### 架构设计
|
||||
- **强制明确性优于便利性**: 本次任务的核心是删除了`createDefault()`这类便利方法。这强制开发者在创建服务时必须明确指定存储类型,从而避免了在Electron等不适宜的环境下意外创建IndexedDB。这是一个重要的架构原则,可以防止隐蔽的、由环境带来的副作用。
|
||||
- **避免模块级副作用**: 我们发现,在`factory.ts`等模块的顶层作用域创建实例(如存储提供者)是一个巨大的隐患。模块在被导入时不应该执行任何具有副作用的实质性操作。所有实例化都应通过明确的函数调用和依赖注入来完成。
|
||||
|
||||
### 调试与排查
|
||||
- **警惕历史遗留数据**: 这是一个关键教训。即使代码已经修复,残留在浏览器中的IndexedDB数据也可能导致应用行为异常,从而掩盖修复的真实效果。在处理与持久化数据相关的问题时,必须将"清理历史数据"作为验证步骤的一部分。
|
||||
- **避免过度修复**: 在排查问题的初期,我们曾在代码中添加了一些复杂的环境检查和警告逻辑。虽然初衷是好的,但这增加了代码的复杂度。最终,通过更根本的架构修复(删除`createDefault`),这些复杂的逻辑变得多余。这提醒我们,在修复后要及时审视并清理过程中添加的临时代码或过度防御性代码。
|
||||
|
||||
## 具体避坑指南
|
||||
|
||||
- **问题**: 在Electron渲染进程中不应出现IndexedDB。
|
||||
- **后果**: 违反了桌面端的核心架构(数据应由主进程统一管理),可能导致数据不一致和意外的磁盘I/O。
|
||||
- **正确做法**: 渲染进程应完全通过IPC代理与主进程通信来操作数据,不应直接创建任何存储实例。所有存储相关的逻辑都应被封装在主进程中。
|
||||
|
||||
- **问题**: 便利的工厂方法(如`createDefault()`)可能隐藏环境依赖。
|
||||
- **后果**: 导致模块在不同环境下行为不一致,增加了调试难度。
|
||||
- **正确做法**: 移除此类隐式创建实例的方法。强制使用依赖注入,让所有依赖关系都变得明确、可控和易于测试。
|
||||
220
docs/archives/110-desktop-indexeddb-fix/fix-details.md
Normal file
220
docs/archives/110-desktop-indexeddb-fix/fix-details.md
Normal file
@@ -0,0 +1,220 @@
|
||||
# Desktop模块修复计划
|
||||
|
||||
## 问题分析
|
||||
|
||||
### 🚨 关键问题(会导致应用无法启动)
|
||||
|
||||
1. **缺少必要依赖**
|
||||
- dotenv: main.js第8行require('dotenv'),但package.json中未声明
|
||||
- @prompt-optimizer/core: main.js第27行require('@prompt-optimizer/core'),但package.json中未声明
|
||||
|
||||
2. **构建配置不一致**
|
||||
- build-desktop.bat使用electron-version=33.0.0
|
||||
- package.json使用electron ^37.1.0
|
||||
- 构建工具:build-desktop.bat使用@electron/packager,package.json使用electron-builder
|
||||
|
||||
3. **缺少资源文件**
|
||||
- package.json中electron-builder配置引用icon.ico,但文件不存在
|
||||
|
||||
### ⚠️ 次要问题(影响功能和兼容性)
|
||||
|
||||
4. **跨平台兼容性问题**
|
||||
- build:web脚本使用robocopy(仅Windows)
|
||||
- 路径使用双反斜杠转义可能在某些环境下有问题
|
||||
|
||||
5. **构建路径问题**
|
||||
- build-desktop.bat引用../desktop-standalone,但实际结构可能不匹配
|
||||
|
||||
## 修复计划
|
||||
|
||||
### 阶段1:修复关键依赖问题
|
||||
- [x] 1.1 更新package.json添加缺少的依赖
|
||||
- 添加了dotenv: ^16.0.0
|
||||
- 添加了@prompt-optimizer/core: workspace:*
|
||||
- [x] 1.2 验证依赖版本兼容性
|
||||
- 依赖安装成功,无版本冲突
|
||||
|
||||
### 阶段2:统一构建配置
|
||||
- [x] 2.1 选择electron-builder作为主要构建工具
|
||||
- [x] 2.2 更新构建脚本
|
||||
- 改进build:web脚本使用跨平台Node.js方法替代robocopy
|
||||
- 添加build:cross-platform脚本使用Node.js构建脚本
|
||||
- [x] 2.3 移除icon配置要求
|
||||
|
||||
### 阶段3:修复API调用错误
|
||||
- [x] 3.1 修复ModelManager API调用
|
||||
- 将getModels()改为getAllModels()
|
||||
- 修复addModel()参数传递问题
|
||||
|
||||
### 阶段4:改进构建脚本
|
||||
- [x] 4.1 创建跨平台构建脚本build.js
|
||||
- [x] 4.2 使用Node.js fs.cpSync替代robocopy
|
||||
|
||||
### 阶段5:测试验证
|
||||
- [x] 5.1 测试开发模式启动 ✅
|
||||
- 应用成功启动,无API错误
|
||||
- 服务初始化正常
|
||||
- 模板加载成功
|
||||
- [ ] 5.2 测试生产构建
|
||||
- [ ] 5.3 验证IPC通信正常
|
||||
|
||||
## 执行时间
|
||||
- 开始时间:2025-01-01
|
||||
- 预计完成:2025-01-01
|
||||
- 状态:🔄 进行中
|
||||
|
||||
## 修复详情
|
||||
|
||||
### 已完成的修复
|
||||
|
||||
#### 1. 依赖问题修复
|
||||
```json
|
||||
// packages/desktop/package.json
|
||||
"dependencies": {
|
||||
"node-fetch": "^2.7.0",
|
||||
"dotenv": "^16.0.0", // 新增
|
||||
"@prompt-optimizer/core": "workspace:*" // 新增
|
||||
}
|
||||
```
|
||||
|
||||
#### 2. API调用修复
|
||||
```javascript
|
||||
// packages/desktop/main.js
|
||||
// 修复前:
|
||||
const result = await modelManager.getModels();
|
||||
|
||||
// 修复后:
|
||||
const result = await modelManager.getAllModels();
|
||||
|
||||
// 修复addModel参数传递:
|
||||
const { key, ...config } = model;
|
||||
await modelManager.addModel(key, config);
|
||||
```
|
||||
|
||||
#### 3. 构建脚本改进
|
||||
- 创建了跨平台构建脚本 `build.js`
|
||||
- 改进了 `build:web` 脚本使用Node.js方法替代Windows专用的robocopy
|
||||
- 移除了electron-builder配置中的icon要求
|
||||
|
||||
#### 4. 测试结果
|
||||
- ✅ 依赖安装成功
|
||||
- ✅ 开发模式启动成功
|
||||
- ✅ 服务初始化正常
|
||||
- ✅ 模板加载成功(7个模板)
|
||||
- ✅ 环境变量正确加载
|
||||
|
||||
### 🚨 重要发现:架构问题
|
||||
|
||||
#### 问题:为什么desktop模式下仍能看到IndexedDB?
|
||||
**根本原因**:useAppInitializer.ts中的架构设计错误
|
||||
|
||||
```typescript
|
||||
// 错误的实现(修复前)
|
||||
if (isRunningInElectron()) {
|
||||
storageProvider = StorageFactory.create('memory'); // ❌ 渲染进程不应该有存储
|
||||
dataManager = createDataManager(..., storageProvider); // ❌ 使用了渲染进程存储
|
||||
const languageService = createTemplateLanguageService(storageProvider); // ❌ 重复创建服务
|
||||
}
|
||||
```
|
||||
|
||||
**问题分析**:
|
||||
1. 渲染进程创建了独立的memory storage,与主进程隔离
|
||||
2. 某些组件可能绕过代理服务,直接使用web版本的IndexedDB
|
||||
3. 数据来源混乱:主进程memory storage vs 渲染进程storage vs IndexedDB
|
||||
|
||||
#### 修复:正确的Electron架构
|
||||
```typescript
|
||||
// 正确的实现(修复后)
|
||||
if (isRunningInElectron()) {
|
||||
storageProvider = null; // ✅ 渲染进程不使用本地存储
|
||||
// 只创建代理服务,所有操作通过IPC
|
||||
modelManager = new ElectronModelManagerProxy();
|
||||
// ...其他代理服务
|
||||
}
|
||||
```
|
||||
|
||||
**正确架构**:
|
||||
- 主进程:唯一的数据源,使用memory storage
|
||||
- 渲染进程:只有代理类,所有操作通过IPC
|
||||
- 无本地存储:渲染进程不应该有任何存储实例
|
||||
|
||||
### 🔧 关键修复:模块级存储创建问题
|
||||
|
||||
#### 发现的根本问题
|
||||
在`packages/core/src/services/prompt/factory.ts`中发现模块级别的存储创建:
|
||||
|
||||
```typescript
|
||||
// 问题代码(已修复)
|
||||
const storageProvider = StorageFactory.createDefault(); // ❌ 模块加载时就创建IndexedDB
|
||||
```
|
||||
|
||||
**影响**:无论在什么环境下,只要导入这个模块就会创建IndexedDB存储!
|
||||
|
||||
#### 修复内容
|
||||
1. **移除模块级存储创建**:修改factory.ts,不再在模块加载时创建存储
|
||||
2. **重构工厂函数**:改为接收依赖注入的方式
|
||||
3. **移除重复函数定义**:清理service.ts中的重复工厂函数
|
||||
|
||||
```typescript
|
||||
// 修复后的代码
|
||||
export function createPromptService(
|
||||
modelManager: IModelManager,
|
||||
llmService: ILLMService,
|
||||
templateManager: ITemplateManager,
|
||||
historyManager: IHistoryManager
|
||||
): PromptService {
|
||||
return new PromptService(modelManager, llmService, templateManager, historyManager);
|
||||
}
|
||||
```
|
||||
|
||||
### 🎯 最终修复:彻底删除createDefault()
|
||||
|
||||
#### 根本解决方案
|
||||
按照用户建议,**彻底删除了StorageFactory.createDefault()方法**:
|
||||
|
||||
```typescript
|
||||
// 删除的问题方法
|
||||
static createDefault(): IStorageProvider {
|
||||
// 这个方法会自动创建IndexedDB,无论在什么环境下
|
||||
}
|
||||
```
|
||||
|
||||
#### 修复内容
|
||||
1. **删除createDefault()方法**:从StorageFactory中完全移除
|
||||
2. **修复TemplateLanguageService**:构造函数改为必须传入storage参数
|
||||
3. **更新测试文件**:移除所有对createDefault()的测试
|
||||
4. **清理相关代码**:移除defaultInstance相关的代码
|
||||
|
||||
#### 架构改进
|
||||
- **强制明确性**:所有地方都必须明确指定存储类型
|
||||
- **避免意外创建**:防止在不合适的环境下自动创建IndexedDB
|
||||
- **提高代码质量**:让依赖关系更加明确和可控
|
||||
|
||||
### ✅ 修复验证
|
||||
- [x] 修复Electron架构问题
|
||||
- [x] 修复模块级存储创建问题
|
||||
- [x] 彻底删除createDefault()方法
|
||||
- [x] 修复TemplateLanguageService依赖注入
|
||||
- [x] 更新测试文件
|
||||
- [x] 测试修复后的应用启动 ✅
|
||||
- [x] 验证主进程使用memory storage ✅
|
||||
- [x] 验证无IndexedDB创建 ✅
|
||||
- [x] 最终用户验证IndexedDB状态 ✅
|
||||
|
||||
### 🧹 代码清理
|
||||
- [x] 移除DexieStorageProvider中的过度防御代码
|
||||
- [x] 简化useAppInitializer中的调试信息
|
||||
- [x] 删除不必要的listTemplatesByTypeAsync方法
|
||||
- [x] 删除无用的getCurrentDefault()方法
|
||||
|
||||
### 📋 最终状态
|
||||
**任务状态**:✅ 完成
|
||||
**问题根源**:历史遗留的IndexedDB数据 + 模块级存储创建
|
||||
**解决方案**:删除createDefault()方法 + 手动清理IndexedDB
|
||||
**验证结果**:Desktop应用正常运行,无IndexedDB创建
|
||||
|
||||
### 🎯 核心收获
|
||||
1. **架构原则**:强制明确性比便利性更重要
|
||||
2. **问题定位**:历史遗留数据可能掩盖真正的修复效果
|
||||
3. **过度工程**:修复过程中要避免不必要的复杂化
|
||||
4. **代码清理**:及时清理无用代码,保持代码库整洁
|
||||
@@ -22,6 +22,7 @@
|
||||
- [103-desktop-architecture](./103-desktop-architecture/) - 桌面端架构 ✅
|
||||
- [108-layout-system](./108-layout-system/) - 布局系统经验总结 ✅
|
||||
- [109-theme-system](./109-theme-system/) - 主题系统开发 ✅
|
||||
- [110-desktop-indexeddb-fix](./110-desktop-indexeddb-fix/) - 桌面端IndexedDB问题修复 ✅
|
||||
|
||||
### 进行中
|
||||
- [106-template-management](./106-template-management/) - 模板管理功能 🔄
|
||||
|
||||
@@ -132,3 +132,6 @@
|
||||
- **[✅] 延迟初始化**: Web和Extension应用中的i18n都应等待存储服务准备好后再初始化
|
||||
- **[✅] 避免main创建服务**: main.ts不应直接使用StorageFactory.createDefault(),应由App.vue统一管理
|
||||
- **[✅] 文件扩展名一致性**: Web和Extension应用都应使用main.ts而不是混用.js和.ts
|
||||
- **[✅] 模块级副作用检查**: 确保模块导入不会产生存储创建等副作用,特别是factory文件
|
||||
- **[✅] 历史数据清理**: 修复代码后需要清理浏览器中的历史IndexedDB数据
|
||||
- **[✅] 强制明确性**: 删除便利方法如createDefault(),强制开发者明确指定存储类型
|
||||
@@ -2,6 +2,68 @@
|
||||
|
||||
记录开发过程中的重要经验和最佳实践。
|
||||
|
||||
## 🔧 构建与依赖管理 (Monorepo & Vite)
|
||||
|
||||
**经验**: 在 pnpm workspace (monorepo) 中,当一个包(如`@core`)需要同时服务于前端(Vite构建的`@ui`包)和后端(Node.js/Electron)时,处理导出和依赖关系需要特别小心,以避免构建冲突。
|
||||
|
||||
**场景**:
|
||||
- 一个`@core`包,其中一部分代码(如tRPC路由)仅用于后端,另一部分是通用代码。
|
||||
- 一个`@ui`包(使用Vite构建),依赖`@core`包。
|
||||
- 一个`@desktop`包(使用Electron),也依赖`@core`包,并需要使用其仅后端的代码。
|
||||
|
||||
**问题**:
|
||||
1. 如果`@core`包在其主入口 (`index.ts`) 导出了仅后端的代码,会导致前端应用打包进不必要的服务器依赖(如`@trpc/server`)。
|
||||
2. 如果为了解决问题1,使用`package.json`的`exports`映射为后端代码创建单独入口,可能会破坏Vite的依赖解析机制,导致`@ui`包构建失败。
|
||||
3. 如果在`@ui`的Vite配置中将`@core`包设为`external`,会增加最终应用的配置复杂性,使其无法"开箱即用"。
|
||||
|
||||
**最佳实践 / 解决方案**:
|
||||
1. **组件库自包含 (Batteries Included)**: 在 `@ui` 包的 `vite.config.ts` 中,**移除**内部依赖(如 `@core`)的 `external` 配置。让UI库成为一个完整的、内置所有必要依赖的自包含产品。
|
||||
2. **核心包多入口构建**: 在 `@core` 包中,使用 `tsup` 等工具配置**多入口点**构建。一个入口是提供给前端和大部分后端的公共API (`index.ts`),另一个是仅用于特定后端的专门文件(如 `router.ts`)。
|
||||
3. **分离导出与实现 (公共API vs 内部路径)**:
|
||||
- **不要**在 `package.json` 中为仅后端的代码创建复杂的 `exports` 映射。保持主 `exports` 干净、简单,只指向公共API。
|
||||
- 在需要使用仅后端代码的地方(如 `desktop/main.js`),直接通过**相对文件路径**从 `dist` 目录中 `require` 编译后的文件。
|
||||
|
||||
**代码示例**:
|
||||
- `packages/core/package.json` (scripts):
|
||||
`"build": "tsup src/index.ts src/services/trpc/router.ts --format cjs,esm --dts"`
|
||||
- `packages/desktop/main.js` (import):
|
||||
`const { createAppRouter } = require('@prompt-optimizer/core/dist/services/trpc/router.cjs');`
|
||||
|
||||
**结论**: 这种"公共API + 内部路径"的策略,优雅地解决了前后端对同一个包的不同需求,保证了Vite构建的顺利进行,也维持了后端功能的可用性。
|
||||
|
||||
**核心原则**: 必须同时满足现代前端构建工具(如Vite)和后端环境(Node.js)的模块解析规则。核心是遵守Node.js的`exports`封装性,并以此为基础解决Vite的兼容问题。
|
||||
|
||||
**遇到的问题演进**:
|
||||
1. **前端加载后端代码**: `@core`包的`index.ts`导出了仅服务器端的代码,导致浏览器报错。
|
||||
2. **Vite构建失败**: 为解决问题1,尝试使用`exports`为后端代码创建单独入口,但这种多入口配置导致Vite无法解析依赖。
|
||||
3. **Node.js路径未导出 (ERR_PACKAGE_PATH_NOT_EXPORTED)**: 为解决问题2,尝试让后端直接引用内部文件路径,但这违反了Node.js的模块封装规则,因为`exports`字段存在时,所有访问必须经过它的允许。
|
||||
|
||||
**最终的最佳实践 (The Standard Way)**:
|
||||
1. **组件库自包含 (Batteries Included)**:
|
||||
- 在 `@ui` 包的 `vite.config.ts` 中,**移除**内部依赖(如 `@core`)的 `external` 配置。让UI库成为一个完整的、内置所有必要依赖的自包含产品。这是解决问题的起点。
|
||||
2. **在核心包中明确声明所有导出**:
|
||||
- 在 `@core` 包的 `package.json` 中,使用 `exports` 字段**明确声明所有**需要被外部访问的路径,无论是给前端还是后端使用。
|
||||
- 使用 `tsup` 等工具进行多入口点构建,确保 `exports` 中声明的每个路径都有对应的编译产物。
|
||||
3. **所有消费者都使用标准路径**:
|
||||
- 无论是前端还是后端,都应该通过 `exports` 中声明的标准路径来导入模块 (e.g., `'@prompt-optimizer/core'` 或 `'@prompt-optimizer/core/trpc-router'`)。
|
||||
- **禁止**任何包从另一个包的内部文件路径(如 `dist/...`)进行导入。
|
||||
|
||||
**代码示例 (最终正确配置)**:
|
||||
- `packages/core/package.json`:
|
||||
```json
|
||||
"exports": {
|
||||
".": { "import": "./dist/index.js", "require": "./dist/index.cjs" },
|
||||
"./trpc-router": { "import": "./dist/services/trpc/router.js", "require": "./dist/services/trpc/router.cjs" }
|
||||
},
|
||||
"scripts": {
|
||||
"build": "tsup src/index.ts src/services/trpc/router.ts --format cjs,esm --dts"
|
||||
}
|
||||
```
|
||||
- `packages/desktop/main.js`:
|
||||
`const { createAppRouter } = require('@prompt-optimizer/core/trpc-router');`
|
||||
|
||||
**结论**: 这个标准化的解决方案保证了 `@core` 包的强封装性,同时为不同环境的消费者提供了清晰、稳定、唯一的访问接口。如果在此基础上Vite仍然构建失败,那么下一步应该去调整Vite自身的配置(如 `resolve.alias` 或 `optimizeDeps.exclude`),而不是破坏包的封装规则。
|
||||
|
||||
## 🔧 技术经验
|
||||
|
||||
### 架构设计
|
||||
@@ -43,7 +105,17 @@
|
||||
## 🔄 流程改进
|
||||
|
||||
### 工作流优化
|
||||
- [改进点] - [改进方法] - [效果评估] - [记录日期]
|
||||
- **经验**: 在Monorepo中设计开发命令时,要避免并行进程操作同一目录导致的"赛跑条件"。
|
||||
- **场景**: 使用`concurrently`并行执行多个任务,其中一个任务是Vite开发服务器(`vite dev`),另一个是其依赖库的监视构建任务(`vite build --watch`)。
|
||||
- **问题**: `vite dev`在启动时需要读取依赖库(如`@ui`)的构建产物(如`dist/style.css`),而`vite build --watch`在同一时间可能正在清理或重写该`dist`目录,导致文件读取失败,引发样式丢失等问题。
|
||||
- **最佳实践**:
|
||||
1. **信任Vite开发服务器**:在开发环境下,应该最大限度地利用Vite开发服务器的内置能力。它能够直接处理对工作区内其他包(workspace-local packages)的依赖,并从它们的**源文件**(`src`)进行实时编译和热更新。
|
||||
2. **避免预构建和监视**:因此,在启动主应用的开发服务器时,**不应该**同时运行其依赖库的`build --watch`任务。
|
||||
3. **简化并行命令**: 开发命令应只包含主应用的开发服务器(如`vite dev`)和其他必要的后端服务(如`electron .`)。让单个Vite实例全权负责所有前端代码的编译。
|
||||
- **示例 (package.json)**:
|
||||
- **错误示范**: `concurrently "pnpm -F @ui watch" "pnpm -F @web dev"`
|
||||
- **正确示范**: `concurrently "pnpm -F @web dev" "pnpm -F @desktop dev"` (假设web负责所有UI,desktop是后端)
|
||||
- **记录日期**: 2024-07-28
|
||||
|
||||
### 文档管理
|
||||
- [管理经验] - [工具使用] - [效率提升] - [记录日期]
|
||||
|
||||
@@ -2,18 +2,108 @@
|
||||
|
||||
记录当前开发任务的进展和思考。
|
||||
|
||||
---
|
||||
## 新增任务:修复 Monorepo 构建与依赖解析问题 - [2024-07-28]
|
||||
**目标**: 解决因 tRPC 重构引入的 `@trpc/server` 在浏览器环境加载,以及后续引发的一系列 Vite 构建失败和依赖解析问题。
|
||||
**状态**: 已完成 ✅
|
||||
|
||||
#### 解决步骤
|
||||
[x] 1. **分析浏览器端tRPC服务器错误**:确认了错误是由于 `@prompt-optimizer/core` 的主入口导出了仅服务器端的 `createAppRouter` 函数导致。
|
||||
[x] 2. **分离客户端与服务器端代码**:从 `core` 包的 `index.ts` 中移除了 `createAppRouter` 的导出,初步解决了前端打包问题,但破坏了Electron后端的导入。
|
||||
[x] 3. **解决Vite构建失败问题**:尝试使用 `package.json` 的 `exports` 映射来修复后端导入,但这与Vite的构建逻辑冲突,导致 `ui` 包构建失败。最终方案是:
|
||||
- **`ui` 包**:从 `vite.config.ts` 的 `externals` 中移除 `@prompt-optimizer/core`,使其成为一个自包含的、内置依赖的库。
|
||||
- **`core` 包**:使用 `tsup` 多入口构建,同时编译公共API (`index.ts`)和服务器路由(`router.ts`),但不为后者创建 `exports` 映射。
|
||||
- **`desktop` 包**:修改 `main.js`,通过直接的文件路径 (`require('@prompt-optimizer/core/dist/services/trpc/router.cjs')`) 来导入服务器路由,绕过 `exports` 映射。
|
||||
[x] 4. **修复Node.js模块导出错误 (ERR_PACKAGE_PATH_NOT_EXPORTED)**:发现上一步的直接路径导入违反了Node.js的模块封装规则。最终的、正确的做法是:
|
||||
- 在 `core` 包的 `package.json` 中,使用 `exports` 字段明确导出 `./trpc-router` 路径。
|
||||
- 在 `desktop` 包的 `main.js` 中,使用标准路径 `require('@prompt-optimizer/core/trpc-router')` 进行导入。
|
||||
[x] 5. **验证修复**:重新构建 `core` 包,确保 Node.js 环境(Electron)可以正常启动,同时验证 Vite 构建(如果失败则下一步处理)
|
||||
|
||||
#### 完成总结
|
||||
- **实现了什么**: 彻底解决了在浏览器环境中加载服务器代码的问题,并梳理了 monorepo 仓库中,一个包同时服务于前端(Vite)和后端(Node.js)时的最佳构建与导出策略。
|
||||
- **遇到的核心问题**:
|
||||
1. 前后端代码耦合导致服务器模块被打包进前端。
|
||||
2. `package.json` 的 `exports` 映射在 Vite 和 Node.js 环境下的行为差异导致构建冲突。
|
||||
3. `external` 配置导致库本身不完整,增加了使用者的配置负担。
|
||||
4. **Node.js 模块封装规则**: 只要 `package.json` 中存在 `exports` 字段,就禁止通过文件路径直接访问未在 `exports` 中声明的任何内部模块。
|
||||
- **解决方案**: 采用"**标准导出,按需消费**"的策略。
|
||||
1. **对内对外,一视同仁**: 所有需要被包外部访问的路径(无论是前端用还是后端用),都必须在 `package.json` 的 `exports` 字段中明确声明。
|
||||
2. **组件库自包含**: Vite 构建的UI库应该将内部依赖打包,移除 `external` 配置,使其成为一个独立完整的单元。
|
||||
3. **消费者使用标准路径**: 所有消费者(无论是Vite还是Node.js)都应该通过 `exports` 中声明的标准路径来导入模块,而不是依赖内部文件结构。
|
||||
---
|
||||
## 新增任务:修复开发命令导致样式丢失的问题 - [2024-07-28]
|
||||
**目标**: 解决 `pnpm run dev:desktop` 命令导致 web 程序样式丢失的问题。
|
||||
**状态**: 已完成 ✅
|
||||
|
||||
#### 问题分析
|
||||
- **现象**: 执行 `dev:desktop` 命令后,桌面应用中的网页内容没有CSS样式。但执行 `dev:fresh` 则是正常的。
|
||||
- **根源**: 这是一个典型的"赛跑条件" (Race Condition)。`dev:desktop` 命令会**并行**启动两个进程:
|
||||
1. `watch:ui` (`vite build --watch`):负责监视和重新构建UI库,会操作`packages/ui/dist`目录。
|
||||
2. `dev:web` (`vite dev`):启动Vite开发服务器,它需要从`packages/ui/dist`目录中读取`style.css`。
|
||||
- **冲突点**: 在 `dev:web` 尝试读取 `style.css` 的一瞬间,`watch:ui` 可能正在清理或重建 `dist` 目录,导致文件暂时不存在,从而加载失败。`dev:fresh` 的成功是偶然的,因为它包含了 `pnpm install` 等额外步骤,改变了两个进程的启动时机,恰好避开了冲突。
|
||||
|
||||
#### 解决方案
|
||||
- **核心思想**: 在开发环境中,应该完全信赖 Vite 开发服务器处理依赖的能力,而不是预先构建依赖库。
|
||||
- **具体操作**:
|
||||
1. 在 `package.json` 中创建一个新的、专门为桌面开发优化的并行命令 `dev:desktop:parallel:fixed`。
|
||||
2. 在这个新命令中,移除了导致问题的 `watch:ui` 任务。
|
||||
3. 只保留并行的 `dev:web` 和 `pnpm -F @prompt-optimizer/desktop dev`。
|
||||
4. 修改 `dev:desktop` 命令,使其调用这个新的、修复过的并行命令。
|
||||
- **结果**: 由 `dev:web` 这一个 Vite 实例全权负责处理所有前端依赖(包括 `@prompt-optimizer/ui` 的源文件)的实时编译和供应,彻底杜绝了"赛跑条件"。
|
||||
|
||||
---
|
||||
|
||||
## 当前任务
|
||||
|
||||
### [任务名称] - [开始日期]
|
||||
**目标**: [具体目标描述]
|
||||
### Electron流式API tRPC重构 - [2024-07-27]
|
||||
**目标**: 使用 `electron-trpc` 重构 `PromptService` 的流式方法(`optimizePromptStream`, `iteratePromptStream`, `testPromptStream`),实现主进程与渲染器进程之间端到端的类型安全流式通信,彻底解决当前功能缺失的问题。
|
||||
**状态**: 进行中
|
||||
|
||||
#### 计划步骤
|
||||
[ ] 1. 需求分析
|
||||
[ ] 2. 技术方案设计
|
||||
[ ] 3. 功能实现
|
||||
[ ] 4. 测试验证
|
||||
[ ] 5. 文档更新
|
||||
[x] 1. **环境搭建与依赖安装**
|
||||
- [x] 在 `packages/desktop` 中安装 `electron-trpc`。
|
||||
- [x] 在 `packages/core` 中安装 `@trpc/server`。
|
||||
- [x] 在 `packages/ui` 中安装 `@trpc/client`。
|
||||
- [x] 验证 `pnpm install` 是否能成功执行,确保依赖关系正确。
|
||||
- 预期结果:所有依赖项被正确添加到各自的`package.json`文件中,项目可以正常编译。
|
||||
- 风险评估:低。主要是版本兼容性问题。
|
||||
|
||||
[x] 2. **tRPC后端(主进程)实现**
|
||||
- [x] 在 `packages/core/src/services` 创建 `trpc/` 目录。
|
||||
- [x] 在 `trpc/` 中创建 `router.ts` 定义tRPC的根路由器(`appRouter`),并包含一个 `prompt` 路由器。
|
||||
- [x] 在 `prompt` 路由器中,使用 `subscription` 来实现 `optimizePromptStream`。
|
||||
- [x] `subscription` 内部逻辑将调用真实的 `PromptService` 实例,并将 `onToken` 等回调的数据通过 `observer.next()` 发送出去。
|
||||
- [x] 在 `packages/desktop/main.js` 中,创建tRPC的IPC link,并将 `appRouter` 附加到上面。
|
||||
- 预期结果:主进程能够处理来自渲染器的tRPC请求和订阅。
|
||||
- 风险评估:中。需要正确处理`Observable`和回调的转换,确保流的生命周期(开始、数据、结束、错误)被正确管理。
|
||||
|
||||
[x] 3. **tRPC前端(渲染器进程)实现**
|
||||
- [x] 在 `packages/desktop/preload.js` 中,使用 `exposeIPCHandler` 将tRPC的处理器暴露给渲染器。
|
||||
- [x] 在 `packages/ui/src/composables/useAppInitializer.ts` 中,修改 `initElectronServices`,创建tRPC客户端实例,并用它来构建新的 `PromptService` 代理。
|
||||
- [x] 新的 `TRPCPromptServiceProxy` 使用tRPC客户端来调用后端方法。`optimizePromptStream` 将调用 `client.prompt.optimizePromptStream.subscribe(...)`。
|
||||
- 预期结果:渲染器能够通过类型安全的客户端与主进程通信,代码更简洁。
|
||||
- 风险评估:中。需要正确配置客户端的 `links`,并确保 `useAppInitializer` 中的服务替换逻辑无误。
|
||||
|
||||
[x] 4. **代码重构与清理**
|
||||
- [x] 移除旧的 `ElectronPromptServiceProxy` 中基于 `console.warn` 的实现,替换为tRPC调用。
|
||||
- [x] 验证 `usePromptOptimizer.ts` 无需任何修改即可与新的代理正常工作。
|
||||
- [ ] 同样的方式重构 `iteratePromptStream` 和 `testPromptStream`。
|
||||
- 预期结果:旧的IPC代码被完全移除,所有流式调用都通过tRPC。
|
||||
- 风险评估:低。主要是替换和删除代码。
|
||||
|
||||
[ ] 5. **功能测试与验证**
|
||||
- [ ] 启动桌面应用 (`pnpm --filter @prompt-optimizer/desktop dev`)。
|
||||
- [ ] 执行一次"优化提示词"操作,验证打字机效果是否出现。
|
||||
- [ ] 查看控制台,确认没有 `not implemented` 警告,也没有历史记录创建失败的错误。
|
||||
- [ ] 执行一次"迭代优化",验证功能正常。
|
||||
- [ ] 执行 `npm run test`,确保所有现有测试用例仍然通过。
|
||||
- 预期结果:桌面应用的功能与Web版完全一致,没有错误。
|
||||
- 风险评估:中。可能会发现一些在重构过程中未预料到的边界情况。
|
||||
|
||||
[ ] 6. **文档更新**
|
||||
- [ ] 在 `docs/developer/architecture` 目录下创建 `trpc-ipc.md` 文档,记录本次重构的架构决策和实现细节。
|
||||
- [ ] 更新 `docs/developer/project-structure.md` 以反映新的 `trpc` 相关文件。
|
||||
- [ ] 在 `docs/workspace/experience.md` 中记录本次重构的关键经验。
|
||||
|
||||
#### 进展记录
|
||||
- [日期] [具体进展描述]
|
||||
@@ -60,3 +150,11 @@
|
||||
|
||||
## 备注
|
||||
[其他需要记录的信息]
|
||||
|
||||
### 里程碑
|
||||
- [ ] 完成依赖安装与环境配置。
|
||||
- [ ] 完成tRPC后端实现。
|
||||
- [ ] 完成tRPC前端实现并成功通信。
|
||||
- [ ] 完成所有流式方法的重构。
|
||||
- [ ] 所有功能在桌面端测试通过。
|
||||
- [ ] 完成相关文档的编写与更新。
|
||||
|
||||
12
package.json
12
package.json
@@ -22,8 +22,9 @@
|
||||
"dev:parallel": "concurrently -k -p \"[{name}]\" -n \"UI,WEB\" \"pnpm run watch:ui\" \"pnpm run dev:web\"",
|
||||
"dev:web": "pnpm -F @prompt-optimizer/web dev",
|
||||
"dev:ext": "pnpm -F @prompt-optimizer/extension dev",
|
||||
"dev:desktop": "npm-run-all dev:setup dev:desktop:parallel",
|
||||
"dev:desktop": "npm-run-all dev:setup dev:desktop:parallel:fixed",
|
||||
"dev:desktop:parallel": "concurrently -k -p \"[{name}]\" -n \"UI,WEB,DESKTOP\" \"pnpm run watch:ui\" \"pnpm run dev:web\" \"pnpm -F @prompt-optimizer/desktop dev\"",
|
||||
"dev:desktop:parallel:fixed": "concurrently -k -p \"[{name}]\" -n \"WEB,DESKTOP\" \"pnpm run dev:web\" \"pnpm -F @prompt-optimizer/desktop dev\"",
|
||||
"dev:desktop:fresh": "npm-run-all clean setup:install dev:desktop",
|
||||
"test:core": "pnpm -F @prompt-optimizer/core test --run --passWithNoTests",
|
||||
"test:ui": "pnpm -F @prompt-optimizer/ui test --run --passWithNoTests",
|
||||
@@ -52,6 +53,7 @@
|
||||
"@intlify/unplugin-vue-i18n": "^6.0.3",
|
||||
"concurrently": "^8.2.2",
|
||||
"cross-env": "^7.0.3",
|
||||
"electron": "^37.1.0",
|
||||
"i18next": "^24.2.2",
|
||||
"i18next-browser-languagedetector": "^8.0.4",
|
||||
"lodash-unified": "^1.0.3",
|
||||
@@ -78,5 +80,13 @@
|
||||
"memoize-one": "^6.0.0",
|
||||
"normalize-wheel-es": "^1.2.0",
|
||||
"vue-i18n": "^10.0.6"
|
||||
},
|
||||
"keywords": [],
|
||||
"author": "",
|
||||
"license": "ISC",
|
||||
"pnpm": {
|
||||
"onlyBuiltDependencies": [
|
||||
"electron"
|
||||
]
|
||||
}
|
||||
}
|
||||
|
||||
@@ -38,7 +38,8 @@ export { LocalStorageProvider } from './services/storage/localStorageProvider'
|
||||
export { MemoryStorageProvider } from './services/storage/memoryStorageProvider'
|
||||
|
||||
// 导出提示词服务相关
|
||||
export { PromptService, createPromptService } from './services/prompt/service'
|
||||
export { PromptService } from './services/prompt/service'
|
||||
export { createPromptService } from './services/prompt/factory'
|
||||
export * from './services/prompt/types'
|
||||
export { ElectronPromptServiceProxy } from './services/prompt/electron-proxy'
|
||||
export * from './services/prompt/errors'
|
||||
|
||||
@@ -1,18 +1,22 @@
|
||||
import { PromptService } from './service';
|
||||
import { TemplateManager } from '../template/manager';
|
||||
import { HistoryManager } from '../history/manager';
|
||||
import { ModelManager } from '../model/manager';
|
||||
import { LLMService } from '../llm/service';
|
||||
import { StorageFactory } from '../storage/factory';
|
||||
|
||||
// 创建共享的存储提供器实例
|
||||
const storageProvider = StorageFactory.createDefault();
|
||||
|
||||
export async function createPromptService() {
|
||||
const modelManager = new ModelManager(storageProvider);
|
||||
const llmService = new LLMService(modelManager);
|
||||
const templateManager = new TemplateManager(storageProvider);
|
||||
const historyManager = new HistoryManager(storageProvider);
|
||||
import type { IModelManager } from '../model/types';
|
||||
import type { ITemplateManager } from '../template/types';
|
||||
import type { IHistoryManager } from '../history/types';
|
||||
import type { ILLMService } from '../llm/types';
|
||||
|
||||
/**
|
||||
* 创建PromptService实例
|
||||
* @param modelManager 模型管理器实例
|
||||
* @param llmService LLM服务实例
|
||||
* @param templateManager 模板管理器实例
|
||||
* @param historyManager 历史管理器实例
|
||||
* @returns PromptService实例
|
||||
*/
|
||||
export function createPromptService(
|
||||
modelManager: IModelManager,
|
||||
llmService: ILLMService,
|
||||
templateManager: ITemplateManager,
|
||||
historyManager: IHistoryManager
|
||||
): PromptService {
|
||||
return new PromptService(modelManager, llmService, templateManager, historyManager);
|
||||
}
|
||||
@@ -89,8 +89,8 @@ export class PromptService implements IPromptService {
|
||||
throw new OptimizationError('Model not found', request.targetPrompt);
|
||||
}
|
||||
|
||||
const template = this.templateManager.getTemplate(
|
||||
request.templateId || this.getDefaultTemplateId(
|
||||
const template = await this.templateManager.getTemplate(
|
||||
request.templateId || await this.getDefaultTemplateId(
|
||||
request.optimizationMode === 'user' ? 'userOptimize' : 'optimize'
|
||||
)
|
||||
);
|
||||
@@ -141,7 +141,7 @@ export class PromptService implements IPromptService {
|
||||
// 获取迭代提示词
|
||||
let template;
|
||||
try {
|
||||
template = this.templateManager.getTemplate(templateId || DEFAULT_TEMPLATES.ITERATE);
|
||||
template = await this.templateManager.getTemplate(templateId || DEFAULT_TEMPLATES.ITERATE);
|
||||
} catch (error) {
|
||||
const errorMessage = error instanceof Error ? error.message : String(error);
|
||||
throw new IterationError(`迭代失败: ${errorMessage}`, originalPrompt, iterateInput);
|
||||
@@ -308,8 +308,8 @@ export class PromptService implements IPromptService {
|
||||
throw new OptimizationError('Model not found', request.targetPrompt);
|
||||
}
|
||||
|
||||
const template = this.templateManager.getTemplate(
|
||||
request.templateId || this.getDefaultTemplateId(
|
||||
const template = await this.templateManager.getTemplate(
|
||||
request.templateId || await this.getDefaultTemplateId(
|
||||
request.optimizationMode === 'user' ? 'userOptimize' : 'optimize'
|
||||
)
|
||||
);
|
||||
@@ -378,7 +378,7 @@ export class PromptService implements IPromptService {
|
||||
// 获取迭代提示词
|
||||
let template;
|
||||
try {
|
||||
template = this.templateManager.getTemplate(templateId);
|
||||
template = await this.templateManager.getTemplate(templateId);
|
||||
} catch (error) {
|
||||
const errorMessage = error instanceof Error ? error.message : String(error);
|
||||
throw new IterationError(`Iteration failed: ${errorMessage}`, originalPrompt, iterateInput);
|
||||
@@ -439,10 +439,10 @@ export class PromptService implements IPromptService {
|
||||
/**
|
||||
* 获取默认模板ID
|
||||
*/
|
||||
private getDefaultTemplateId(templateType: 'optimize' | 'userOptimize' | 'iterate'): string {
|
||||
private async getDefaultTemplateId(templateType: 'optimize' | 'userOptimize' | 'iterate'): Promise<string> {
|
||||
try {
|
||||
// 尝试获取指定类型的模板列表
|
||||
const templates = this.templateManager.listTemplatesByType(templateType);
|
||||
const templates = await this.templateManager.listTemplatesByType(templateType);
|
||||
if (templates.length > 0) {
|
||||
// 返回列表中第一个模板的ID
|
||||
return templates[0].id;
|
||||
@@ -464,7 +464,7 @@ export class PromptService implements IPromptService {
|
||||
}
|
||||
|
||||
for (const fallbackType of fallbackTypes) {
|
||||
const fallbackTemplates = this.templateManager.listTemplatesByType(fallbackType);
|
||||
const fallbackTemplates = await this.templateManager.listTemplatesByType(fallbackType);
|
||||
if (fallbackTemplates.length > 0) {
|
||||
console.log(`Using fallback template type ${fallbackType} for ${templateType}`);
|
||||
return fallbackTemplates[0].id;
|
||||
@@ -472,7 +472,7 @@ export class PromptService implements IPromptService {
|
||||
}
|
||||
|
||||
// 最后的回退:获取所有模板中第一个可用的内置模板
|
||||
const allTemplates = this.templateManager.listTemplates();
|
||||
const allTemplates = await this.templateManager.listTemplates();
|
||||
const availableTemplate = allTemplates.find(t => t.isBuiltin);
|
||||
if (availableTemplate) {
|
||||
console.warn(`Using fallback builtin template: ${availableTemplate.id} for type ${templateType}`);
|
||||
@@ -499,7 +499,7 @@ export class PromptService implements IPromptService {
|
||||
version: 1,
|
||||
timestamp: Date.now(),
|
||||
modelKey: request.modelKey,
|
||||
templateId: request.templateId || this.getDefaultTemplateId(
|
||||
templateId: request.templateId || await this.getDefaultTemplateId(
|
||||
request.optimizationMode === 'user' ? 'userOptimize' : 'optimize'
|
||||
),
|
||||
metadata: {
|
||||
@@ -540,19 +540,4 @@ export class PromptService implements IPromptService {
|
||||
// 这种混合架构是经过权衡的设计决策
|
||||
}
|
||||
|
||||
/**
|
||||
* 工厂函数,用于创建 PromptService 实例
|
||||
* @param modelManager 模型管理器
|
||||
* @param llmService LLM服务
|
||||
* @param templateManager 模板管理器
|
||||
* @param historyManager 历史记录管理器
|
||||
* @returns IPromptService 实例
|
||||
*/
|
||||
export function createPromptService(
|
||||
modelManager: IModelManager,
|
||||
llmService: ILLMService,
|
||||
templateManager: ITemplateManager,
|
||||
historyManager: IHistoryManager
|
||||
): IPromptService {
|
||||
return new PromptService(modelManager, llmService, templateManager, historyManager);
|
||||
}
|
||||
|
||||
|
||||
@@ -10,7 +10,6 @@ export type StorageType = 'localStorage' | 'dexie' | 'memory';
|
||||
*/
|
||||
export class StorageFactory {
|
||||
// 单例实例缓存
|
||||
private static defaultInstance: IStorageProvider | null = null;
|
||||
private static instances: Map<StorageType, IStorageProvider> = new Map();
|
||||
|
||||
/**
|
||||
@@ -44,50 +43,19 @@ export class StorageFactory {
|
||||
return instance;
|
||||
}
|
||||
|
||||
/**
|
||||
* 创建默认存储提供器(单例)
|
||||
* 优先使用 Dexie,降级到 localStorage
|
||||
*/
|
||||
static createDefault(): IStorageProvider {
|
||||
// 返回缓存的默认实例
|
||||
if (StorageFactory.defaultInstance) {
|
||||
return StorageFactory.defaultInstance;
|
||||
}
|
||||
|
||||
try {
|
||||
// 检查是否支持 IndexedDB (Dexie 的基础)
|
||||
if (typeof window !== 'undefined' && window.indexedDB) {
|
||||
console.log('Using Dexie as default storage provider');
|
||||
StorageFactory.defaultInstance = StorageFactory.create('dexie');
|
||||
} else {
|
||||
console.log('IndexedDB not available, using localStorage as default storage provider');
|
||||
StorageFactory.defaultInstance = StorageFactory.create('localStorage');
|
||||
}
|
||||
} catch (error) {
|
||||
console.warn('Dexie storage unavailable, falling back to localStorage:', error);
|
||||
StorageFactory.defaultInstance = StorageFactory.create('localStorage');
|
||||
}
|
||||
|
||||
return StorageFactory.defaultInstance;
|
||||
}
|
||||
|
||||
/**
|
||||
* 重置所有实例(主要用于测试)
|
||||
*/
|
||||
static reset(): void {
|
||||
StorageFactory.defaultInstance = null;
|
||||
StorageFactory.instances.clear();
|
||||
|
||||
// 重置DexieStorageProvider的迁移状态
|
||||
DexieStorageProvider.resetMigrationState();
|
||||
}
|
||||
|
||||
/**
|
||||
* 获取当前默认实例(用于调试)
|
||||
*/
|
||||
static getCurrentDefault(): IStorageProvider | null {
|
||||
return StorageFactory.defaultInstance;
|
||||
}
|
||||
|
||||
|
||||
/**
|
||||
* 获取所有支持的存储类型
|
||||
|
||||
@@ -25,10 +25,8 @@ export class ElectronTemplateManagerProxy implements ITemplateManager {
|
||||
return true;
|
||||
}
|
||||
|
||||
getTemplate(templateId: string): Template {
|
||||
// 注意:ITemplateManager接口要求这是同步方法,但IPC是异步的
|
||||
// 这里需要抛出错误,因为代理模式下无法提供同步访问
|
||||
throw new Error(`getTemplate(${templateId}) is not supported in Electron proxy mode. Use async IPC calls instead.`);
|
||||
async getTemplate(templateId: string): Promise<Template> {
|
||||
return this.electronAPI.template.getTemplate(templateId);
|
||||
}
|
||||
|
||||
async saveTemplate(template: Template): Promise<void> {
|
||||
@@ -42,14 +40,13 @@ export class ElectronTemplateManagerProxy implements ITemplateManager {
|
||||
await this.electronAPI.template.deleteTemplate(templateId);
|
||||
}
|
||||
|
||||
listTemplates(): Template[] {
|
||||
// 同步方法在代理模式下不支持
|
||||
throw new Error('listTemplates is not supported in Electron proxy mode. Use async IPC calls instead.');
|
||||
async listTemplates(): Promise<Template[]> {
|
||||
return this.electronAPI.template.getTemplates();
|
||||
}
|
||||
|
||||
exportTemplate(templateId: string): string {
|
||||
// 同步方法在代理模式下不支持
|
||||
throw new Error(`exportTemplate(${templateId}) is not supported in Electron proxy mode. Use async IPC calls instead.`);
|
||||
async exportTemplate(templateId: string): Promise<string> {
|
||||
const template = await this.getTemplate(templateId);
|
||||
return JSON.stringify(template, null, 2);
|
||||
}
|
||||
|
||||
async importTemplate(templateJson: string): Promise<void> {
|
||||
@@ -61,22 +58,14 @@ export class ElectronTemplateManagerProxy implements ITemplateManager {
|
||||
// 在代理模式下,缓存由主进程管理,这里是空实现
|
||||
}
|
||||
|
||||
listTemplatesByType(type: 'optimize' | 'userOptimize' | 'iterate'): Template[] {
|
||||
// 同步方法在代理模式下不支持
|
||||
throw new Error(`listTemplatesByType(${type}) is not supported in Electron proxy mode. Use async IPC calls instead.`);
|
||||
async listTemplatesByType(type: 'optimize' | 'userOptimize' | 'iterate'): Promise<Template[]> {
|
||||
return this.electronAPI.template.listTemplatesByType(type);
|
||||
}
|
||||
|
||||
getTemplatesByType(type: 'optimize' | 'userOptimize' | 'iterate'): Template[] {
|
||||
// 同步方法在代理模式下不支持
|
||||
throw new Error(`getTemplatesByType(${type}) is not supported in Electron proxy mode. Use async IPC calls instead.`);
|
||||
async getTemplatesByType(type: 'optimize' | 'userOptimize' | 'iterate'): Promise<Template[]> {
|
||||
return this.listTemplatesByType(type);
|
||||
}
|
||||
|
||||
// 添加异步版本的方法供UI使用
|
||||
async getTemplateAsync(templateId: string): Promise<Template | undefined> {
|
||||
return this.electronAPI.template.getTemplate(templateId);
|
||||
}
|
||||
|
||||
async listTemplatesAsync(): Promise<Template[]> {
|
||||
return this.electronAPI.template.getTemplates();
|
||||
}
|
||||
|
||||
}
|
||||
@@ -1,5 +1,4 @@
|
||||
import { IStorageProvider } from '../storage/types';
|
||||
import { StorageFactory } from '../storage/factory';
|
||||
|
||||
/**
|
||||
* Supported built-in template languages
|
||||
@@ -18,8 +17,8 @@ export class TemplateLanguageService {
|
||||
private storage: IStorageProvider;
|
||||
private initialized = false;
|
||||
|
||||
constructor(storage?: IStorageProvider) {
|
||||
this.storage = storage || StorageFactory.createDefault();
|
||||
constructor(storage: IStorageProvider) {
|
||||
this.storage = storage;
|
||||
}
|
||||
|
||||
/**
|
||||
@@ -36,9 +35,15 @@ export class TemplateLanguageService {
|
||||
if (savedLanguage && this.isValidLanguage(savedLanguage)) {
|
||||
this.currentLanguage = savedLanguage as BuiltinTemplateLanguage;
|
||||
} else {
|
||||
// Auto-detect: Chinese browsers use Chinese, others use English
|
||||
const isChineseBrowser = navigator.language?.startsWith('zh') ?? false;
|
||||
this.currentLanguage = isChineseBrowser ? 'zh-CN' : 'en-US';
|
||||
let detectedLanguage: BuiltinTemplateLanguage = this.DEFAULT_LANGUAGE;
|
||||
|
||||
// Auto-detect only in browser-like environments where `navigator` is available.
|
||||
if (typeof navigator !== 'undefined' && navigator.language) {
|
||||
const isChineseBrowser = navigator.language.startsWith('zh');
|
||||
detectedLanguage = isChineseBrowser ? 'zh-CN' : 'en-US';
|
||||
}
|
||||
|
||||
this.currentLanguage = detectedLanguage;
|
||||
await this.storage.setItem(this.STORAGE_KEY, this.currentLanguage);
|
||||
}
|
||||
|
||||
|
||||
@@ -137,7 +137,7 @@ export class TemplateManager implements ITemplateManager {
|
||||
* @param id Template ID
|
||||
* @returns Template
|
||||
*/
|
||||
getTemplate(id: string | null | undefined): Template {
|
||||
async getTemplate(id: string | null | undefined): Promise<Template> {
|
||||
this.checkInitialized('getTemplate');
|
||||
this.validateTemplateId(id);
|
||||
|
||||
@@ -227,7 +227,7 @@ export class TemplateManager implements ITemplateManager {
|
||||
/**
|
||||
* List all templates
|
||||
*/
|
||||
listTemplates(): Template[] {
|
||||
async listTemplates(): Promise<Template[]> {
|
||||
this.checkInitialized('listTemplates');
|
||||
|
||||
const templates = [
|
||||
@@ -256,8 +256,8 @@ export class TemplateManager implements ITemplateManager {
|
||||
* @param id Template ID
|
||||
* @returns Template as JSON string
|
||||
*/
|
||||
exportTemplate(id: string): string {
|
||||
const template = this.getTemplate(id);
|
||||
async exportTemplate(id: string): Promise<string> {
|
||||
const template = await this.getTemplate(id);
|
||||
return JSON.stringify(template, null, 2);
|
||||
}
|
||||
|
||||
@@ -390,16 +390,17 @@ export class TemplateManager implements ITemplateManager {
|
||||
* Get templates by type
|
||||
* @deprecated Use listTemplatesByType instead
|
||||
*/
|
||||
getTemplatesByType(type: 'optimize' | 'iterate'): Template[] {
|
||||
return this.listTemplatesByType(type);
|
||||
async getTemplatesByType(type: 'optimize' | 'iterate'): Promise<Template[]> {
|
||||
return await this.listTemplatesByType(type);
|
||||
}
|
||||
|
||||
/**
|
||||
* List templates by type
|
||||
*/
|
||||
listTemplatesByType(type: 'optimize' | 'userOptimize' | 'iterate'): Template[] {
|
||||
async listTemplatesByType(type: 'optimize' | 'userOptimize' | 'iterate'): Promise<Template[]> {
|
||||
try {
|
||||
return this.listTemplates().filter(
|
||||
const templates = await this.listTemplates();
|
||||
return templates.filter(
|
||||
template => template.metadata.templateType === type
|
||||
);
|
||||
} catch (error) {
|
||||
|
||||
@@ -56,34 +56,34 @@ export interface ITemplateManager {
|
||||
isInitialized(): boolean;
|
||||
|
||||
/** 获取指定ID的模板 */
|
||||
getTemplate(templateId: string): Template; // Stays synchronous
|
||||
getTemplate(templateId: string): Promise<Template>;
|
||||
|
||||
/** 保存模板 */
|
||||
saveTemplate(template: Template): Promise<void>; // Async
|
||||
saveTemplate(template: Template): Promise<void>;
|
||||
|
||||
/** 删除模板 */
|
||||
deleteTemplate(templateId: string): Promise<void>; // Async
|
||||
deleteTemplate(templateId: string): Promise<void>;
|
||||
|
||||
/** 列出所有模板 */
|
||||
listTemplates(): Template[]; // Stays synchronous
|
||||
listTemplates(): Promise<Template[]>;
|
||||
|
||||
/** 导出模板 */
|
||||
exportTemplate(templateId: string): string; // Stays synchronous
|
||||
exportTemplate(templateId: string): Promise<string>;
|
||||
|
||||
/** 导入模板 */
|
||||
importTemplate(templateJson: string): Promise<void>; // Async
|
||||
importTemplate(templateJson: string): Promise<void>;
|
||||
|
||||
/** 清除缓存 */
|
||||
clearCache(templateId?: string): void; // Synchronous
|
||||
clearCache(templateId?: string): void; // 保持同步,因为只是内存操作
|
||||
|
||||
/** 按类型列出模板 */
|
||||
listTemplatesByType(type: 'optimize' | 'userOptimize' | 'iterate'): Template[];
|
||||
listTemplatesByType(type: 'optimize' | 'userOptimize' | 'iterate'): Promise<Template[]>;
|
||||
|
||||
/**
|
||||
* 根据类型获取模板列表(已废弃)
|
||||
* @deprecated 使用 listTemplatesByType 替代
|
||||
*/
|
||||
getTemplatesByType(type: 'optimize' | 'userOptimize' | 'iterate'): Template[];
|
||||
getTemplatesByType(type: 'optimize' | 'userOptimize' | 'iterate'): Promise<Template[]>;
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
1
packages/core/src/types/global.d.ts
vendored
1
packages/core/src/types/global.d.ts
vendored
@@ -41,6 +41,7 @@ interface Window {
|
||||
createTemplate: (template: any) => Promise<any>;
|
||||
updateTemplate: (id: string, updates: any) => Promise<void>;
|
||||
deleteTemplate: (id: string) => Promise<void>;
|
||||
listTemplatesByType: (type: 'optimize' | 'userOptimize' | 'iterate') => Promise<any[]>;
|
||||
};
|
||||
history: {
|
||||
getHistory: () => Promise<any[]>;
|
||||
|
||||
@@ -110,8 +110,42 @@ export const getProxyUrl = (baseURL: string | undefined, isStream: boolean = fal
|
||||
|
||||
/**
|
||||
* 检测是否在Electron环境中运行
|
||||
* 使用多重检测机制确保准确性
|
||||
*/
|
||||
export function isRunningInElectron(): boolean {
|
||||
return typeof window !== 'undefined' &&
|
||||
typeof (window as any).electronAPI !== 'undefined';
|
||||
if (typeof window === 'undefined') {
|
||||
return false;
|
||||
}
|
||||
|
||||
// 检查多个Electron特征
|
||||
const hasElectronAPI = typeof (window as any).electronAPI !== 'undefined';
|
||||
const hasElectronProcess = typeof (window as any).process !== 'undefined' &&
|
||||
(window as any).process?.type === 'renderer';
|
||||
const hasElectronRequire = typeof (window as any).require !== 'undefined';
|
||||
const userAgent = window.navigator?.userAgent?.toLowerCase() || '';
|
||||
const hasElectronUserAgent = userAgent.includes('electron');
|
||||
|
||||
console.log('[isRunningInElectron] Detection details:', {
|
||||
hasElectronAPI,
|
||||
hasElectronProcess,
|
||||
hasElectronRequire,
|
||||
hasElectronUserAgent,
|
||||
userAgent,
|
||||
});
|
||||
|
||||
// 如果有electronAPI,肯定是Electron
|
||||
if (hasElectronAPI) {
|
||||
console.log('[isRunningInElectron] Verdict: true (via electronAPI)');
|
||||
return true;
|
||||
}
|
||||
|
||||
// 如果有其他Electron特征,也认为是Electron(可能是preload脚本还没执行完)
|
||||
if (hasElectronProcess || hasElectronRequire || hasElectronUserAgent) {
|
||||
console.warn('[Environment] Detected Electron environment but electronAPI not available yet');
|
||||
console.log(`[isRunningInElectron] Verdict: true (via fallback checks: process=${hasElectronProcess}, require=${hasElectronRequire}, userAgent=${hasElectronUserAgent})`);
|
||||
return true;
|
||||
}
|
||||
|
||||
console.log('[isRunningInElectron] Verdict: false');
|
||||
return false;
|
||||
}
|
||||
@@ -30,7 +30,7 @@ describe('Real Components Integration Tests', () => {
|
||||
templateManager = createTemplateManager(storage, languageService)
|
||||
await templateManager.ensureInitialized()
|
||||
|
||||
dataManager = new DataManager(modelManager, templateManager, historyManager)
|
||||
dataManager = new DataManager(modelManager, templateManager, historyManager, storage)
|
||||
|
||||
const llmService = createLLMService(modelManager)
|
||||
promptService = new PromptService(modelManager, llmService, templateManager, historyManager)
|
||||
@@ -128,13 +128,13 @@ describe('Real Components Integration Tests', () => {
|
||||
await templateManager.saveTemplate(template)
|
||||
|
||||
// 获取模板
|
||||
const retrieved = templateManager.getTemplate('user-test-template')
|
||||
const retrieved = await templateManager.getTemplate('user-test-template')
|
||||
expect(retrieved).toBeDefined()
|
||||
expect(retrieved.name).toBe('User Test Template')
|
||||
expect(retrieved.content).toBe('This is a user test template: {{input}}')
|
||||
|
||||
// 验证在模板列表中(注意:真实环境可能有内置模板)
|
||||
const templates = templateManager.listTemplates()
|
||||
const templates = await templateManager.listTemplates()
|
||||
const userTemplate = templates.find(t => t.id === 'user-test-template')
|
||||
expect(userTemplate).toBeDefined()
|
||||
|
||||
@@ -142,8 +142,8 @@ describe('Real Components Integration Tests', () => {
|
||||
await templateManager.deleteTemplate('user-test-template')
|
||||
|
||||
// 验证已删除
|
||||
expect(() => templateManager.getTemplate('user-test-template'))
|
||||
.toThrow('Template user-test-template not found')
|
||||
await expect(templateManager.getTemplate('user-test-template'))
|
||||
.rejects.toThrow('Template user-test-template not found')
|
||||
})
|
||||
})
|
||||
|
||||
@@ -183,7 +183,7 @@ describe('Real Components Integration Tests', () => {
|
||||
expect(retrievedModel).toBeDefined()
|
||||
expect(retrievedModel?.name).toBe('Test Model')
|
||||
|
||||
const retrievedTemplate = templateManager.getTemplate('user-optimize-template')
|
||||
const retrievedTemplate = await templateManager.getTemplate('user-optimize-template')
|
||||
expect(retrievedTemplate).toBeDefined()
|
||||
expect(retrievedTemplate.name).toBe('User Optimize Template')
|
||||
|
||||
@@ -252,7 +252,7 @@ describe('Real Components Integration Tests', () => {
|
||||
|
||||
// 验证数据已清空
|
||||
const emptyModels = await modelManager.getAllModels()
|
||||
const emptyTemplates = templateManager.listTemplates()
|
||||
const emptyTemplates = await templateManager.listTemplates()
|
||||
const emptyHistory = await historyManager.getRecords()
|
||||
|
||||
// 注意:真实环境可能有内置模型和模板,不一定为空
|
||||
@@ -263,7 +263,7 @@ describe('Real Components Integration Tests', () => {
|
||||
|
||||
// 验证数据已恢复
|
||||
const restoredModels = await modelManager.getAllModels()
|
||||
const restoredTemplates = templateManager.listTemplates()
|
||||
const restoredTemplates = await templateManager.listTemplates()
|
||||
const restoredHistory = await historyManager.getRecords()
|
||||
|
||||
expect(restoredModels.length).toBeGreaterThan(0)
|
||||
@@ -419,7 +419,7 @@ describe('Real Components Integration Tests', () => {
|
||||
const models = await modelManager.getAllModels()
|
||||
expect(models.length).toBeGreaterThan(0) // 应该有添加的模型
|
||||
|
||||
const templates = templateManager.listTemplates()
|
||||
const templates = await templateManager.listTemplates()
|
||||
// 真实环境可能有内置模板,只验证不崩溃
|
||||
expect(Array.isArray(templates)).toBe(true)
|
||||
})
|
||||
|
||||
@@ -453,33 +453,29 @@ describe('存储实现通用测试', () => {
|
||||
}).toThrow('Unsupported storage type: invalid');
|
||||
});
|
||||
|
||||
it('应该能够创建默认提供器', () => {
|
||||
const provider = StorageFactory.createDefault();
|
||||
expect(provider).toBeDefined();
|
||||
// 默认应该是 Dexie(如果支持)或 localStorage
|
||||
expect(
|
||||
provider instanceof DexieStorageProvider ||
|
||||
provider instanceof LocalStorageProvider
|
||||
).toBe(true);
|
||||
it('应该能够创建指定类型的提供器', () => {
|
||||
const dexieProvider = StorageFactory.create('dexie');
|
||||
expect(dexieProvider).toBeDefined();
|
||||
expect(dexieProvider instanceof DexieStorageProvider).toBe(true);
|
||||
|
||||
const localProvider = StorageFactory.create('localStorage');
|
||||
expect(localProvider).toBeDefined();
|
||||
expect(localProvider instanceof LocalStorageProvider).toBe(true);
|
||||
});
|
||||
|
||||
it('应该确保默认提供器是单例', () => {
|
||||
it('应该确保相同类型的提供器是单例', () => {
|
||||
// 重置工厂状态
|
||||
StorageFactory.reset();
|
||||
|
||||
// 创建多个默认提供器实例
|
||||
const provider1 = StorageFactory.createDefault();
|
||||
const provider2 = StorageFactory.createDefault();
|
||||
const provider3 = StorageFactory.createDefault();
|
||||
// 创建多个相同类型的提供器实例
|
||||
const provider1 = StorageFactory.create('memory');
|
||||
const provider2 = StorageFactory.create('memory');
|
||||
const provider3 = StorageFactory.create('memory');
|
||||
|
||||
// 验证它们是同一个实例
|
||||
expect(provider1).toBe(provider2);
|
||||
expect(provider2).toBe(provider3);
|
||||
expect(provider1).toBe(provider3);
|
||||
|
||||
// 验证getCurrentDefault返回相同实例
|
||||
const currentDefault = StorageFactory.getCurrentDefault();
|
||||
expect(currentDefault).toBe(provider1);
|
||||
});
|
||||
|
||||
it('应该确保相同类型的提供器是单例', () => {
|
||||
@@ -502,23 +498,17 @@ describe('存储实现通用测试', () => {
|
||||
|
||||
it('应该能够重置工厂状态', () => {
|
||||
// 创建一些实例
|
||||
const provider1 = StorageFactory.createDefault();
|
||||
const memory1 = StorageFactory.create('memory');
|
||||
const localStorage1 = StorageFactory.create('localStorage');
|
||||
|
||||
// 验证实例存在
|
||||
expect(StorageFactory.getCurrentDefault()).toBe(provider1);
|
||||
|
||||
// 重置状态
|
||||
StorageFactory.reset();
|
||||
|
||||
// 验证状态已重置
|
||||
expect(StorageFactory.getCurrentDefault()).toBeNull();
|
||||
|
||||
// 创建新实例应该是不同的对象
|
||||
const provider2 = StorageFactory.createDefault();
|
||||
const memory2 = StorageFactory.create('memory');
|
||||
const localStorage2 = StorageFactory.create('localStorage');
|
||||
|
||||
expect(provider2).not.toBe(provider1);
|
||||
expect(memory2).not.toBe(memory1);
|
||||
expect(localStorage2).not.toBe(localStorage1);
|
||||
});
|
||||
});
|
||||
|
||||
@@ -45,7 +45,7 @@ describe('Extended Metadata Fields Support', () => {
|
||||
await templateManager.saveTemplate(templateWithExtraFields);
|
||||
|
||||
// 获取模板
|
||||
const savedTemplate = templateManager.getTemplate('test-extended-template');
|
||||
const savedTemplate = await templateManager.getTemplate('test-extended-template');
|
||||
|
||||
// 验证基础字段
|
||||
expect(savedTemplate.id).toBe('test-extended-template');
|
||||
@@ -92,7 +92,7 @@ describe('Extended Metadata Fields Support', () => {
|
||||
await templateManager.saveTemplate(templateWithExtraFields);
|
||||
|
||||
// 导出模板
|
||||
const exportedJson = templateManager.exportTemplate('test-export-import');
|
||||
const exportedJson = await templateManager.exportTemplate('test-export-import');
|
||||
|
||||
// 删除原模板
|
||||
await templateManager.deleteTemplate('test-export-import');
|
||||
@@ -101,7 +101,7 @@ describe('Extended Metadata Fields Support', () => {
|
||||
await templateManager.importTemplate(exportedJson);
|
||||
|
||||
// 验证导入的模板
|
||||
const importedTemplate = templateManager.getTemplate('test-export-import');
|
||||
const importedTemplate = await templateManager.getTemplate('test-export-import');
|
||||
|
||||
expect(importedTemplate.metadata.customData).toEqual({
|
||||
nested: {
|
||||
@@ -152,7 +152,7 @@ describe('Extended Metadata Fields Support', () => {
|
||||
};
|
||||
|
||||
await templateManager.saveTemplate(mixedFieldsTemplate);
|
||||
const savedTemplate = templateManager.getTemplate('mixed-fields-test');
|
||||
const savedTemplate = await templateManager.getTemplate('mixed-fields-test');
|
||||
|
||||
expect(savedTemplate.metadata.stringField).toBe('string value');
|
||||
expect(savedTemplate.metadata.numberField).toBe(123);
|
||||
|
||||
@@ -39,7 +39,7 @@ describe('TemplateManager with Mocked LanguageService', () => {
|
||||
|
||||
it('should load English templates by default in a test environment', async () => {
|
||||
// ensureInitialized in beforeEach already loaded templates
|
||||
const templates = templateManager.listTemplates();
|
||||
const templates = await templateManager.listTemplates();
|
||||
const enTemplate = templates.find(t => t.id === 'test-en');
|
||||
const zhTemplate = templates.find(t => t.id === 'test-zh');
|
||||
|
||||
@@ -53,7 +53,7 @@ describe('TemplateManager with Mocked LanguageService', () => {
|
||||
vi.spyOn(languageService, 'getCurrentLanguage').mockReturnValue('zh-CN');
|
||||
await templateManager.changeBuiltinTemplateLanguage('zh-CN');
|
||||
|
||||
const templates = templateManager.listTemplates();
|
||||
const templates = await templateManager.listTemplates();
|
||||
const enTemplate = templates.find(t => t.id === 'test-en');
|
||||
const zhTemplate = templates.find(t => t.id === 'test-zh');
|
||||
|
||||
@@ -78,7 +78,7 @@ describe('TemplateManager with Mocked LanguageService', () => {
|
||||
metadata: { templateType: 'userOptimize', version: '1.0', lastModified: Date.now() }
|
||||
};
|
||||
await templateManager.saveTemplate(newUserTemplate);
|
||||
const retrieved = templateManager.getTemplate('user-test');
|
||||
const retrieved = await templateManager.getTemplate('user-test');
|
||||
expect(retrieved).toBeDefined();
|
||||
expect(retrieved.content).toBe('User Content');
|
||||
expect(retrieved.isBuiltin).toBe(false);
|
||||
@@ -106,7 +106,7 @@ describe('TemplateManager with Mocked LanguageService', () => {
|
||||
};
|
||||
await templateManager.saveTemplate(newUserTemplate);
|
||||
|
||||
const allTemplates = templateManager.listTemplates();
|
||||
const allTemplates = await templateManager.listTemplates();
|
||||
expect(allTemplates.length).toBe(2);
|
||||
expect(allTemplates.some(t => t.id === 'test-en')).toBe(true);
|
||||
expect(allTemplates.some(t => t.id === 'user-test-2')).toBe(true);
|
||||
@@ -121,12 +121,12 @@ describe('TemplateManager with Mocked LanguageService', () => {
|
||||
metadata: { templateType: 'userOptimize', version: '1.0', lastModified: Date.now() }
|
||||
};
|
||||
await templateManager.saveTemplate(newUserTemplate);
|
||||
let allTemplates = templateManager.listTemplates();
|
||||
let allTemplates = await templateManager.listTemplates();
|
||||
expect(allTemplates.length).toBe(2);
|
||||
|
||||
await templateManager.deleteTemplate('to-be-deleted');
|
||||
|
||||
allTemplates = templateManager.listTemplates();
|
||||
allTemplates = await templateManager.listTemplates();
|
||||
expect(allTemplates.length).toBe(1);
|
||||
expect(allTemplates.some(t => t.id === 'to-be-deleted')).toBe(false);
|
||||
});
|
||||
|
||||
93
packages/desktop/build.js
Normal file
93
packages/desktop/build.js
Normal file
@@ -0,0 +1,93 @@
|
||||
#!/usr/bin/env node
|
||||
|
||||
/**
|
||||
* Cross-platform desktop build script
|
||||
* Replaces build-desktop.bat with a Node.js solution
|
||||
*/
|
||||
|
||||
const fs = require('fs');
|
||||
const path = require('path');
|
||||
const { execSync } = require('child_process');
|
||||
|
||||
console.log('===========================================');
|
||||
console.log('Prompt Optimizer Desktop Build Script');
|
||||
console.log('===========================================');
|
||||
|
||||
function runCommand(command, cwd = process.cwd()) {
|
||||
console.log(`Running: ${command}`);
|
||||
try {
|
||||
execSync(command, {
|
||||
cwd,
|
||||
stdio: 'inherit',
|
||||
env: { ...process.env }
|
||||
});
|
||||
} catch (error) {
|
||||
console.error(`Command failed: ${command}`);
|
||||
process.exit(1);
|
||||
}
|
||||
}
|
||||
|
||||
function copyDirectory(src, dest) {
|
||||
console.log(`Copying ${src} to ${dest}`);
|
||||
|
||||
// Remove destination if it exists
|
||||
if (fs.existsSync(dest)) {
|
||||
fs.rmSync(dest, { recursive: true, force: true });
|
||||
}
|
||||
|
||||
// Create destination directory
|
||||
fs.mkdirSync(dest, { recursive: true });
|
||||
|
||||
// Copy files recursively
|
||||
fs.cpSync(src, dest, { recursive: true });
|
||||
console.log('Copy completed successfully');
|
||||
}
|
||||
|
||||
try {
|
||||
// Step 1: Build web application
|
||||
console.log('\nStep 1: Building web application...');
|
||||
const webDir = path.resolve(__dirname, '../web');
|
||||
runCommand('pnpm run build', webDir);
|
||||
|
||||
// Step 2: Copy web files to desktop
|
||||
console.log('\nStep 2: Copying web files...');
|
||||
const webDistSrc = path.join(webDir, 'dist');
|
||||
const webDistDest = path.join(__dirname, 'web-dist');
|
||||
|
||||
if (!fs.existsSync(webDistSrc)) {
|
||||
throw new Error(`Web dist directory not found: ${webDistSrc}`);
|
||||
}
|
||||
|
||||
copyDirectory(webDistSrc, webDistDest);
|
||||
|
||||
// Step 3: Package desktop application
|
||||
console.log('\nStep 3: Packaging desktop application...');
|
||||
runCommand('pnpm run package', __dirname);
|
||||
|
||||
console.log('\n===========================================');
|
||||
console.log('Build completed successfully!');
|
||||
console.log('===========================================');
|
||||
|
||||
// Show output location
|
||||
const distDir = path.join(__dirname, 'dist');
|
||||
if (fs.existsSync(distDir)) {
|
||||
console.log(`Output location: ${distDir}`);
|
||||
const files = fs.readdirSync(distDir);
|
||||
files.forEach(file => {
|
||||
const filePath = path.join(distDir, file);
|
||||
const stats = fs.statSync(filePath);
|
||||
if (stats.isDirectory()) {
|
||||
console.log(`Directory: ${file}/`);
|
||||
} else {
|
||||
console.log(`File: ${file} (${stats.size} bytes)`);
|
||||
}
|
||||
});
|
||||
}
|
||||
|
||||
} catch (error) {
|
||||
console.error('\n===========================================');
|
||||
console.error('Build failed!');
|
||||
console.error('===========================================');
|
||||
console.error(error.message);
|
||||
process.exit(1);
|
||||
}
|
||||
@@ -219,17 +219,19 @@ function setupIPC() {
|
||||
// Model Manager handlers
|
||||
ipcMain.handle('model-getModels', async (event) => {
|
||||
try {
|
||||
const result = await modelManager.getModels();
|
||||
const result = await modelManager.getAllModels();
|
||||
return { success: true, data: result };
|
||||
} catch (error) {
|
||||
console.error('[Main Process] Model getModels failed:', error);
|
||||
console.error('[Main Process] Model getAllModels failed:', error);
|
||||
return { success: false, error: error.message };
|
||||
}
|
||||
});
|
||||
|
||||
ipcMain.handle('model-addModel', async (event, model) => {
|
||||
try {
|
||||
await modelManager.addModel(model);
|
||||
// model应该包含key和config,需要分离
|
||||
const { key, ...config } = model;
|
||||
await modelManager.addModel(key, config);
|
||||
return { success: true };
|
||||
} catch (error) {
|
||||
console.error('[Main Process] Model addModel failed:', error);
|
||||
@@ -270,7 +272,7 @@ function setupIPC() {
|
||||
// Template Manager handlers
|
||||
ipcMain.handle('template-getTemplates', async (event) => {
|
||||
try {
|
||||
const result = templateManager.listTemplates();
|
||||
const result = await templateManager.listTemplates();
|
||||
return { success: true, data: result };
|
||||
} catch (error) {
|
||||
console.error('[Main Process] Template getTemplates failed:', error);
|
||||
@@ -301,7 +303,7 @@ function setupIPC() {
|
||||
ipcMain.handle('template-updateTemplate', async (event, id, updates) => {
|
||||
try {
|
||||
// Get existing template and merge with updates
|
||||
const existingTemplate = templateManager.getTemplate(id);
|
||||
const existingTemplate = await templateManager.getTemplate(id);
|
||||
const updatedTemplate = { ...existingTemplate, ...updates, id };
|
||||
await templateManager.saveTemplate(updatedTemplate);
|
||||
return { success: true };
|
||||
@@ -321,6 +323,16 @@ function setupIPC() {
|
||||
}
|
||||
});
|
||||
|
||||
ipcMain.handle('template-listTemplatesByType', async (event, type) => {
|
||||
try {
|
||||
const result = await templateManager.listTemplatesByType(type);
|
||||
return { success: true, data: result };
|
||||
} catch (error) {
|
||||
console.error('[Main Process] Template listTemplatesByType failed:', error);
|
||||
return { success: false, error: error.message };
|
||||
}
|
||||
});
|
||||
|
||||
// History Manager handlers
|
||||
ipcMain.handle('history-getHistory', async (event) => {
|
||||
try {
|
||||
|
||||
@@ -5,16 +5,19 @@
|
||||
"main": "main.js",
|
||||
"scripts": {
|
||||
"build": "pnpm run build:web && pnpm run package",
|
||||
"build:web": "cd ../web && cross-env ELECTRON_BUILD=true vite build --base=./ && (robocopy dist ..\\\\desktop\\\\web-dist /E /NFL /NDL /NJH /NJS /NC /NS /NP || exit 0)",
|
||||
"build:web": "cd ../web && cross-env ELECTRON_BUILD=true vite build --base=./ && node -e \"const fs=require('fs'),path=require('path'); const src='dist',dest='../desktop/web-dist'; if(fs.existsSync(dest)) fs.rmSync(dest,{recursive:true}); fs.cpSync(src,dest,{recursive:true}); console.log('Web files copied to desktop/web-dist');\"",
|
||||
"build:cross-platform": "node build.js",
|
||||
"package": "electron-builder",
|
||||
"dev": "cross-env NODE_ENV=development electron ."
|
||||
},
|
||||
"devDependencies": {
|
||||
"cross-env": "^7.0.3",
|
||||
"electron": "^37.1.0",
|
||||
"electron-builder": "^24.0.0",
|
||||
"cross-env": "^7.0.3"
|
||||
"electron-builder": "^24.0.0"
|
||||
},
|
||||
"dependencies": {
|
||||
"@prompt-optimizer/core": "workspace:*",
|
||||
"dotenv": "^16.0.0",
|
||||
"node-fetch": "^2.7.0"
|
||||
},
|
||||
"build": {
|
||||
@@ -30,8 +33,7 @@
|
||||
"node_modules/**/*"
|
||||
],
|
||||
"win": {
|
||||
"target": "nsis",
|
||||
"icon": "icon.ico"
|
||||
"target": "nsis"
|
||||
},
|
||||
"mac": {
|
||||
"target": "dmg"
|
||||
|
||||
@@ -180,6 +180,15 @@ contextBridge.exposeInMainWorld('electronAPI', {
|
||||
if (!result.success) {
|
||||
throw new Error(result.error);
|
||||
}
|
||||
},
|
||||
|
||||
// Add listTemplatesByType
|
||||
listTemplatesByType: async (type) => {
|
||||
const result = await ipcRenderer.invoke('template-listTemplatesByType', type);
|
||||
if (!result.success) {
|
||||
throw new Error(result.error);
|
||||
}
|
||||
return result.data;
|
||||
}
|
||||
},
|
||||
|
||||
|
||||
@@ -255,11 +255,11 @@
|
||||
<span
|
||||
v-if="viewingTemplate || editingTemplate"
|
||||
class="px-2 py-1 rounded text-xs font-medium"
|
||||
:class="(viewingTemplate || editingTemplate) && TemplateProcessor.isSimpleTemplate(viewingTemplate || editingTemplate)
|
||||
:class="(viewingTemplate || editingTemplate) && TemplateProcessor.isSimpleTemplate((viewingTemplate || editingTemplate)!)
|
||||
? 'bg-blue-100 text-blue-700 border border-blue-200'
|
||||
: 'bg-purple-100 text-purple-700 border border-purple-200'"
|
||||
>
|
||||
{{ (viewingTemplate || editingTemplate) && TemplateProcessor.isSimpleTemplate(viewingTemplate || editingTemplate)
|
||||
{{ (viewingTemplate || editingTemplate) && TemplateProcessor.isSimpleTemplate((viewingTemplate || editingTemplate)!)
|
||||
? '📝 ' + t('templateManager.simpleTemplate')
|
||||
: '⚡ ' + t('templateManager.advancedTemplate') }}
|
||||
</span>
|
||||
@@ -292,7 +292,7 @@
|
||||
v-model="form.name"
|
||||
type="text"
|
||||
required
|
||||
:readonly="viewingTemplate"
|
||||
:readonly="!!viewingTemplate"
|
||||
class="theme-manager-input"
|
||||
:class="{ 'opacity-75 cursor-not-allowed': viewingTemplate }"
|
||||
:placeholder="t('template.namePlaceholder')"
|
||||
@@ -347,7 +347,7 @@
|
||||
<textarea
|
||||
v-model="form.content"
|
||||
required
|
||||
:readonly="viewingTemplate"
|
||||
:readonly="!!viewingTemplate"
|
||||
rows="15"
|
||||
class="theme-manager-input resize-y font-mono text-sm min-h-[200px] max-h-[400px]"
|
||||
:class="{ 'opacity-75 cursor-not-allowed': viewingTemplate }"
|
||||
@@ -367,7 +367,7 @@
|
||||
<button
|
||||
type="button"
|
||||
@click="addMessage"
|
||||
:disabled="viewingTemplate"
|
||||
:disabled="!!viewingTemplate"
|
||||
class="text-sm inline-flex items-center gap-1 theme-manager-button-secondary"
|
||||
:class="{ 'opacity-50 cursor-not-allowed': viewingTemplate }"
|
||||
>
|
||||
@@ -390,7 +390,7 @@
|
||||
<div class="flex-shrink-0">
|
||||
<select
|
||||
v-model="message.role"
|
||||
:disabled="viewingTemplate"
|
||||
:disabled="!!viewingTemplate"
|
||||
class="theme-manager-input text-sm w-24"
|
||||
:class="{ 'opacity-75 cursor-not-allowed': viewingTemplate }"
|
||||
>
|
||||
@@ -404,7 +404,7 @@
|
||||
<div class="flex-1">
|
||||
<textarea
|
||||
v-model="message.content"
|
||||
:readonly="viewingTemplate"
|
||||
:readonly="!!viewingTemplate"
|
||||
class="theme-manager-input font-mono text-sm w-full resize-y message-content-textarea"
|
||||
:style="{
|
||||
minHeight: '80px',
|
||||
@@ -489,7 +489,7 @@
|
||||
<label class="block text-sm font-medium theme-manager-text mb-1.5">{{ t('common.description') }}</label>
|
||||
<textarea
|
||||
v-model="form.description"
|
||||
:readonly="viewingTemplate"
|
||||
:readonly="!!viewingTemplate"
|
||||
rows="2"
|
||||
class="theme-manager-input resize-y min-h-[60px] max-h-[120px]"
|
||||
:class="{ 'opacity-75 cursor-not-allowed': viewingTemplate }"
|
||||
@@ -620,11 +620,11 @@
|
||||
@change="handleFileImport"
|
||||
/>
|
||||
<button
|
||||
@click="$refs.fileInput.click()"
|
||||
@click="fileInput?.click()"
|
||||
class="text-sm inline-flex gap-1 theme-manager-button-secondary"
|
||||
>
|
||||
<svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor" class="size-4 my-[2px]">
|
||||
<path stroke-linecap="round" stroke-linejoin="round" d="M7.5 7.5h-.75A2.25 2.25 0 0 0 4.5 9.75v7.5a2.25 2.25 0 0 0 2.25 2.25h7.5a2.25 2.25 0 0 0 2.25-2.25v-7.5a2.25 2.25 0 0 0-2.25-2.25h-.75m0-3-3-3m0 0-3 3m3-3v11.25m6-2.25h.75a2.25 2.25 0 0 1 2.25 2.25v7.5a2.25 2.25 0 0 1-2.25 2.25h-7.5a2.25 2.25 0 0 1-2.25-2.25v-.75" />
|
||||
<path stroke-linecap="round" stroke-linejoin="round" d="M7.5 7.5h-.75A2.25 2.25 0 0 0 4.5 9.75v7.5a2.25 2.25 0 0 0 2.25 2.25h7.5a2.25 2.25 0 0 0 2.25-2.25v-7.5a2.25 2.25 0 0 0-2.25-2.25h-.75m0-3-3-3m0 0-3 3m3-3v11.25m6-2.25h.75a2.25 2.25 0 0 1 2.25 2.25v7.5a2.25 2.25 0 0 1-2.25-2.25h-7.5a2.25 2.25 0 0 1-2.25-2.25v-.75" />
|
||||
</svg>
|
||||
{{ t('common.selectFile') }}
|
||||
</button>
|
||||
@@ -639,46 +639,56 @@
|
||||
<script setup lang="ts">
|
||||
import { ref, onMounted, computed, watch, nextTick, inject } from 'vue'
|
||||
import { useI18n } from 'vue-i18n'
|
||||
import { TemplateProcessor, type Template } from '@prompt-optimizer/core'
|
||||
import { TemplateProcessor, type Template, type MessageTemplate } from '@prompt-optimizer/core'
|
||||
import { useToast } from '../composables/useToast'
|
||||
import MarkdownRenderer from './MarkdownRenderer.vue'
|
||||
import BuiltinTemplateLanguageSwitch from './BuiltinTemplateLanguageSwitch.vue'
|
||||
import { syntaxGuideContent } from '../docs/syntax-guide'
|
||||
import type { ITemplateManager, TemplateLanguageService } from '@prompt-optimizer/core'
|
||||
import { i18n } from '../plugins/i18n'
|
||||
|
||||
const { t } = useI18n()
|
||||
|
||||
// 通过依赖注入获取服务
|
||||
const services = inject('services', { value: null })
|
||||
const getTemplateManager = computed(() => services.value?.templateManager)
|
||||
const getTemplateLanguageService = computed(() => services.value?.templateLanguageService)
|
||||
|
||||
const props = defineProps({
|
||||
selectedSystemOptimizeTemplate: Object,
|
||||
selectedUserOptimizeTemplate: Object,
|
||||
selectedIterateTemplate: Object,
|
||||
templateType: {
|
||||
type: String,
|
||||
required: true,
|
||||
validator: (value) => ['optimize', 'userOptimize', 'iterate'].includes(value)
|
||||
},
|
||||
show: {
|
||||
type: Boolean,
|
||||
default: false
|
||||
interface Services {
|
||||
templateManager: ITemplateManager;
|
||||
templateLanguageService: TemplateLanguageService;
|
||||
}
|
||||
})
|
||||
|
||||
// 通过依赖注入获取服务
|
||||
const services = inject<{ value: Services | null }>('services')
|
||||
if (!services?.value) {
|
||||
throw new Error('TemplateManager Error: The required "services" were not provided by a parent component. Make sure this component is a child of a component that uses "provide(\'services\', ...)"')
|
||||
}
|
||||
|
||||
const getTemplateManager = computed(() => services.value!.templateManager)
|
||||
const getTemplateLanguageService = computed(() => services.value!.templateLanguageService)
|
||||
|
||||
const props = defineProps<{
|
||||
selectedSystemOptimizeTemplate?: Template,
|
||||
selectedUserOptimizeTemplate?: Template,
|
||||
selectedIterateTemplate?: Template,
|
||||
templateType: 'optimize' | 'userOptimize' | 'iterate',
|
||||
show: boolean
|
||||
}>()
|
||||
|
||||
const emit = defineEmits(['close', 'select', 'update:show'])
|
||||
const toast = useToast()
|
||||
|
||||
const templates = ref([])
|
||||
const templates = ref<Template[]>([])
|
||||
const currentCategory = ref(getCategoryFromProps())
|
||||
const currentType = computed(() => getCurrentTemplateType())
|
||||
const showAddForm = ref(false)
|
||||
const editingTemplate = ref(null)
|
||||
const viewingTemplate = ref(null)
|
||||
const editingTemplate = ref<Template | null>(null)
|
||||
const viewingTemplate = ref<Template | null>(null)
|
||||
const showSyntaxGuide = ref(false)
|
||||
|
||||
const form = ref({
|
||||
const form = ref<{
|
||||
name: string
|
||||
content: string
|
||||
description: string
|
||||
isAdvanced: boolean
|
||||
messages: MessageTemplate[]
|
||||
}>({
|
||||
name: '',
|
||||
content: '',
|
||||
description: '',
|
||||
@@ -686,7 +696,12 @@ const form = ref({
|
||||
messages: []
|
||||
})
|
||||
|
||||
const migrationDialog = ref({
|
||||
const migrationDialog = ref<{
|
||||
show: boolean
|
||||
template: Template | null
|
||||
original: string
|
||||
converted: MessageTemplate[]
|
||||
}>({
|
||||
show: false,
|
||||
template: null,
|
||||
original: '',
|
||||
@@ -770,7 +785,7 @@ const processedPreview = computed(() => {
|
||||
}
|
||||
|
||||
try {
|
||||
const tempTemplate = {
|
||||
const tempTemplate: Template = {
|
||||
id: 'preview',
|
||||
name: 'Preview',
|
||||
content: form.value.messages,
|
||||
@@ -792,7 +807,8 @@ const loadTemplates = async () => {
|
||||
// Ensure template manager is initialized
|
||||
await getTemplateManager.value.ensureInitialized()
|
||||
|
||||
const allTemplates = getTemplateManager.value.listTemplates()
|
||||
// 统一使用异步方法
|
||||
const allTemplates = await getTemplateManager.value.listTemplates()
|
||||
templates.value = allTemplates
|
||||
console.log('加载到的提示词:', templates.value)
|
||||
} catch (error) {
|
||||
@@ -814,10 +830,10 @@ const editTemplate = (template: Template) => {
|
||||
|
||||
form.value = {
|
||||
name: template.name,
|
||||
content: isAdvanced ? '' : template.content,
|
||||
content: isAdvanced ? '' : template.content as string,
|
||||
description: template.metadata.description || '',
|
||||
isAdvanced,
|
||||
messages: isAdvanced ? [...template.content] : []
|
||||
messages: isAdvanced ? [...template.content] as MessageTemplate[] : []
|
||||
}
|
||||
|
||||
// 等待DOM更新后初始化textarea高度
|
||||
@@ -833,10 +849,10 @@ const viewTemplate = (template: Template) => {
|
||||
|
||||
form.value = {
|
||||
name: template.name,
|
||||
content: isAdvanced ? '' : template.content,
|
||||
content: isAdvanced ? '' : template.content as string,
|
||||
description: template.metadata.description || '',
|
||||
isAdvanced,
|
||||
messages: isAdvanced ? [...template.content] : []
|
||||
messages: isAdvanced ? [...template.content] as MessageTemplate[] : []
|
||||
}
|
||||
|
||||
// 等待DOM更新后初始化textarea高度
|
||||
@@ -886,12 +902,12 @@ const addMessage = () => {
|
||||
}
|
||||
|
||||
// 移除消息
|
||||
const removeMessage = (index) => {
|
||||
const removeMessage = (index: number) => {
|
||||
form.value.messages.splice(index, 1)
|
||||
}
|
||||
|
||||
// 移动消息
|
||||
const moveMessage = (index, direction) => {
|
||||
const moveMessage = (index: number, direction: number) => {
|
||||
const newIndex = index + direction
|
||||
if (newIndex >= 0 && newIndex < form.value.messages.length) {
|
||||
const messages = [...form.value.messages]
|
||||
@@ -903,8 +919,8 @@ const moveMessage = (index, direction) => {
|
||||
}
|
||||
|
||||
// 初始化textarea高度 - 只在打开时调用一次
|
||||
const initializeTextareaHeight = (textarea) => {
|
||||
if (!textarea || textarea._initialized) return
|
||||
const initializeTextareaHeight = (textarea: HTMLTextAreaElement) => {
|
||||
if (!textarea || (textarea as any)._initialized) return
|
||||
|
||||
try {
|
||||
const minHeight = 80
|
||||
@@ -925,7 +941,7 @@ const initializeTextareaHeight = (textarea) => {
|
||||
}
|
||||
|
||||
textarea.style.height = initialHeight + 'px'
|
||||
textarea._initialized = true
|
||||
;(textarea as any)._initialized = true
|
||||
} catch (error) {
|
||||
console.warn('Textarea initialization error:', error)
|
||||
}
|
||||
@@ -933,9 +949,9 @@ const initializeTextareaHeight = (textarea) => {
|
||||
|
||||
// 显示迁移对话框
|
||||
const showMigrationDialog = (template: Template) => {
|
||||
if (!isStringTemplate(template)) return
|
||||
if (!isStringTemplate(template) || typeof template.content !== 'string') return
|
||||
|
||||
const converted = [
|
||||
const converted: MessageTemplate[] = [
|
||||
{
|
||||
role: 'system',
|
||||
content: template.content
|
||||
@@ -958,7 +974,9 @@ const showMigrationDialog = (template: Template) => {
|
||||
const applyMigration = async () => {
|
||||
try {
|
||||
const template = migrationDialog.value.template
|
||||
const updatedTemplate = {
|
||||
if (!template) return
|
||||
|
||||
const updatedTemplate: Template = {
|
||||
...template,
|
||||
content: migrationDialog.value.converted,
|
||||
metadata: {
|
||||
@@ -1023,7 +1041,7 @@ const handleSubmit = async () => {
|
||||
templateType: getCurrentTemplateType()
|
||||
}
|
||||
|
||||
const templateData = {
|
||||
const templateData: Template = {
|
||||
id: editingTemplate.value?.id || generateUniqueTemplateId('user-template'),
|
||||
name: form.value.name,
|
||||
content: form.value.isAdvanced ? form.value.messages : form.value.content,
|
||||
@@ -1037,7 +1055,8 @@ const handleSubmit = async () => {
|
||||
|
||||
if (editingTemplate.value && isCurrentSelected) {
|
||||
try {
|
||||
const updatedTemplate = getTemplateManager.value.getTemplate(templateData.id)
|
||||
// 统一使用异步方法
|
||||
const updatedTemplate = await getTemplateManager.value.getTemplate(templateData.id)
|
||||
if (updatedTemplate) {
|
||||
emit('select', updatedTemplate, getCurrentTemplateType());
|
||||
}
|
||||
@@ -1077,38 +1096,47 @@ const confirmDelete = async (templateId: string) => {
|
||||
}
|
||||
|
||||
// 导出提示词
|
||||
const exportTemplate = (templateId: string) => {
|
||||
const exportTemplate = async (templateId: string) => {
|
||||
try {
|
||||
const templateJson = getTemplateManager.value.exportTemplate(templateId)
|
||||
const blob = new Blob([templateJson], { type: 'application/json' })
|
||||
const url = URL.createObjectURL(blob)
|
||||
const a = document.createElement('a')
|
||||
a.href = url
|
||||
a.download = `template-${templateId}.json`
|
||||
document.body.appendChild(a)
|
||||
a.click()
|
||||
document.body.removeChild(a)
|
||||
URL.revokeObjectURL(url)
|
||||
toast.success(t('template.success.exported'))
|
||||
const templateJson = await getTemplateManager.value.exportTemplate(templateId);
|
||||
const blob = new Blob([templateJson], { type: 'application/json' });
|
||||
const url = URL.createObjectURL(blob);
|
||||
const a = document.createElement('a');
|
||||
a.href = url;
|
||||
a.download = `template-${templateId}.json`;
|
||||
document.body.appendChild(a);
|
||||
a.click();
|
||||
document.body.removeChild(a);
|
||||
URL.revokeObjectURL(url);
|
||||
toast.success(t('template.success.exported'));
|
||||
} catch (error) {
|
||||
console.error('导出提示词失败:', error)
|
||||
toast.error(t('template.error.exportFailed'))
|
||||
console.error('导出提示词失败:', error);
|
||||
toast.error(t('template.error.exportFailed'));
|
||||
}
|
||||
}
|
||||
|
||||
// 导入提示词
|
||||
const handleFileImport = (event) => {
|
||||
const file = event.target.files[0]
|
||||
const fileInput = ref<HTMLInputElement | null>(null)
|
||||
const handleFileImport = (event: Event) => {
|
||||
const target = event.target as HTMLInputElement
|
||||
const file = target.files?.[0]
|
||||
if (!file) return
|
||||
|
||||
try {
|
||||
const reader = new FileReader()
|
||||
reader.onload = async (e) => {
|
||||
try {
|
||||
if (e.target?.result && typeof e.target.result === 'string') {
|
||||
await getTemplateManager.value.importTemplate(e.target.result)
|
||||
} else {
|
||||
// 让失败不再静默,明确地抛出错误
|
||||
throw new Error('Failed to read file content as string.')
|
||||
}
|
||||
await loadTemplates()
|
||||
toast.success(t('template.success.imported'))
|
||||
event.target.value = ''
|
||||
if (target) {
|
||||
target.value = ''
|
||||
}
|
||||
} catch (error) {
|
||||
console.error('导入提示词失败:', error)
|
||||
toast.error(t('template.error.importFailed'))
|
||||
@@ -1128,10 +1156,10 @@ const copyTemplate = (template: Template) => {
|
||||
|
||||
form.value = {
|
||||
name: `${template.name} - 副本`,
|
||||
content: isAdvanced ? '' : template.content,
|
||||
content: isAdvanced ? '' : template.content as string,
|
||||
description: template.metadata.description || '',
|
||||
isAdvanced,
|
||||
messages: isAdvanced ? [...template.content] : []
|
||||
messages: isAdvanced ? [...template.content] as MessageTemplate[] : []
|
||||
}
|
||||
}
|
||||
|
||||
@@ -1166,12 +1194,12 @@ const filteredTemplates = computed(() => {
|
||||
|
||||
// 获取当前语言的语法指南内容
|
||||
const syntaxGuideMarkdown = computed(() => {
|
||||
const locale = t.locale || 'zh-CN'
|
||||
return syntaxGuideContent[locale] || syntaxGuideContent['zh-CN']
|
||||
const lang = i18n.global.locale.value as keyof typeof syntaxGuideContent
|
||||
return syntaxGuideContent[lang] || syntaxGuideContent['zh-CN']
|
||||
})
|
||||
|
||||
// 处理内置模板语言变化
|
||||
const handleLanguageChanged = (newLanguage) => {
|
||||
const handleLanguageChanged = (newLanguage: string) => {
|
||||
// 重新加载模板列表以反映新的语言
|
||||
loadTemplates()
|
||||
|
||||
@@ -1207,7 +1235,14 @@ watch(() => props.templateType, (newTemplateType) => {
|
||||
|
||||
// 生命周期钩子
|
||||
onMounted(async () => {
|
||||
await loadTemplates()
|
||||
console.log('[TemplateManager.vue] Component is mounted.');
|
||||
console.log('[TemplateManager.vue] Injected services:', services);
|
||||
if (services?.value) {
|
||||
console.log('[TemplateManager.vue] TemplateManager instance from services:', getTemplateManager.value);
|
||||
} else {
|
||||
console.error('[TemplateManager.vue] Services not available on mount.');
|
||||
}
|
||||
await loadTemplates();
|
||||
})
|
||||
|
||||
// 监听表单消息数量变化,只在新增消息时初始化新textarea
|
||||
@@ -1228,7 +1263,7 @@ watch([() => showAddForm.value, () => editingTemplate.value, () => viewingTempla
|
||||
const initializeAllTextareas = () => {
|
||||
// 延迟执行,确保DOM已更新
|
||||
nextTick(() => {
|
||||
const textareas = document.querySelectorAll('textarea.message-content-textarea')
|
||||
const textareas = document.querySelectorAll<HTMLTextAreaElement>('textarea.message-content-textarea')
|
||||
|
||||
textareas.forEach(textarea => {
|
||||
// 确保textarea可见且未初始化过
|
||||
|
||||
@@ -185,12 +185,19 @@ const ensureTemplateManagerReady = async () => {
|
||||
return true
|
||||
}
|
||||
|
||||
const templates = computed(() => {
|
||||
if (!isReady.value) return []
|
||||
// 改为响应式数据,因为需要异步加载
|
||||
const templates = ref<Template[]>([])
|
||||
|
||||
// 直接使用 props.type,因为它的值已经被 validator 保证是有效的
|
||||
return templateManager.value!.listTemplatesByType(props.type)
|
||||
})
|
||||
// 异步加载模板列表
|
||||
const loadTemplatesByType = async () => {
|
||||
if (!isReady.value || !templateManager.value) {
|
||||
throw new Error('Template manager is not ready or not available')
|
||||
}
|
||||
|
||||
// 统一使用异步方法,立即抛错不静默处理
|
||||
const typeTemplates = await templateManager.value.listTemplatesByType(props.type)
|
||||
templates.value = typeTemplates
|
||||
}
|
||||
|
||||
// 添加对services变化的监听
|
||||
watch(
|
||||
@@ -199,14 +206,29 @@ watch(
|
||||
if (newTemplateManager) {
|
||||
console.debug('[TemplateSelect] 检测到模板管理器变化,开始初始化...')
|
||||
await ensureTemplateManagerReady()
|
||||
await loadTemplatesByType()
|
||||
} else {
|
||||
console.debug('[TemplateSelect] 模板管理器不可用,重置状态')
|
||||
// 立即抛错,不静默处理
|
||||
isReady.value = false
|
||||
templates.value = []
|
||||
throw new Error('[TemplateSelect] Template manager is not available')
|
||||
}
|
||||
},
|
||||
{ immediate: true, deep: true }
|
||||
)
|
||||
|
||||
// 监听props.type变化,重新加载模板
|
||||
watch(
|
||||
() => props.type,
|
||||
async () => {
|
||||
if (isReady.value) {
|
||||
await loadTemplatesByType()
|
||||
} else {
|
||||
throw new Error('[TemplateSelect] Cannot load templates: manager not ready')
|
||||
}
|
||||
}
|
||||
)
|
||||
|
||||
// 添加对optimizationMode变化的监听
|
||||
watch(
|
||||
() => props.optimizationMode,
|
||||
|
||||
@@ -32,6 +32,7 @@ export function useAppInitializer() {
|
||||
onMounted(async () => {
|
||||
try {
|
||||
console.log('[AppInitializer] 开始应用初始化...');
|
||||
|
||||
let storageProvider: IStorageProvider;
|
||||
let modelManager: IModelManager;
|
||||
let templateManager: ITemplateManager;
|
||||
@@ -42,8 +43,27 @@ export function useAppInitializer() {
|
||||
|
||||
if (isRunningInElectron()) {
|
||||
console.log('[AppInitializer] 检测到Electron环境,初始化代理服务...');
|
||||
// 在Electron渲染进程中,我们应该使用基于内存的存储而非Dexie,以确保状态与主进程同步
|
||||
storageProvider = StorageFactory.create('memory');
|
||||
// 在Electron渲染进程中,我们需要创建一个代理存储提供器
|
||||
// 这个代理会将所有存储操作转发到主进程
|
||||
storageProvider = {
|
||||
async getItem(key: string): Promise<string | null> {
|
||||
console.warn('[ElectronStorageProxy] getItem called, but storage should be handled by main process');
|
||||
return null;
|
||||
},
|
||||
async setItem(key: string, value: string): Promise<void> {
|
||||
console.warn('[ElectronStorageProxy] setItem called, but storage should be handled by main process');
|
||||
},
|
||||
async removeItem(key: string): Promise<void> {
|
||||
console.warn('[ElectronStorageProxy] removeItem called, but storage should be handled by main process');
|
||||
},
|
||||
async clear(): Promise<void> {
|
||||
console.warn('[ElectronStorageProxy] clear called, but storage should be handled by main process');
|
||||
},
|
||||
async getAllKeys(): Promise<string[]> {
|
||||
console.warn('[ElectronStorageProxy] getAllKeys called, but storage should be handled by main process');
|
||||
return [];
|
||||
}
|
||||
} as any;
|
||||
|
||||
// 在Electron环境中,我们实例化所有轻量级的代理类
|
||||
modelManager = new ElectronModelManagerProxy();
|
||||
@@ -52,23 +72,25 @@ export function useAppInitializer() {
|
||||
llmService = new ElectronLLMProxy();
|
||||
promptService = new ElectronPromptServiceProxy();
|
||||
|
||||
// DataManager in electron requires special handling, as it depends on other managers.
|
||||
// We create it using the proxy instances.
|
||||
dataManager = createDataManager(modelManager, templateManager, historyManager, storageProvider);
|
||||
// DataManager在Electron环境下也使用代理模式,不需要本地存储
|
||||
// 创建一个空的DataManager,因为所有操作都通过代理进行
|
||||
dataManager = {
|
||||
exportData: async () => { throw new Error('Export not implemented in Electron proxy mode'); },
|
||||
importData: async () => { throw new Error('Import not implemented in Electron proxy mode'); },
|
||||
clearAllData: async () => { throw new Error('Clear not implemented in Electron proxy mode'); }
|
||||
} as any;
|
||||
|
||||
// 在 Electron 环境中也提供 templateLanguageService
|
||||
const languageService = createTemplateLanguageService(storageProvider);
|
||||
await languageService.initialize();
|
||||
// 在Electron环境中,模板语言服务由主进程管理,渲染进程不需要初始化
|
||||
|
||||
services.value = {
|
||||
storageProvider,
|
||||
storageProvider, // 使用代理存储提供器
|
||||
modelManager,
|
||||
templateManager,
|
||||
historyManager,
|
||||
dataManager,
|
||||
llmService,
|
||||
promptService,
|
||||
templateLanguageService: languageService,
|
||||
templateLanguageService: null as any, // 由主进程管理
|
||||
};
|
||||
console.log('[AppInitializer] Electron代理服务初始化完成');
|
||||
|
||||
@@ -86,6 +108,8 @@ export function useAppInitializer() {
|
||||
await languageService.initialize();
|
||||
|
||||
const templateManagerInstance = createTemplateManager(storageProvider, languageService);
|
||||
templateManager = templateManagerInstance;
|
||||
console.log('[AppInitializer] TemplateManager instance in Web:', templateManager);
|
||||
|
||||
// Initialize managers that depend on other managers
|
||||
const historyManagerInstance = createHistoryManager(storageProvider, modelManagerInstance);
|
||||
|
||||
@@ -159,7 +159,7 @@ export function useTemplateManager(
|
||||
|
||||
if (savedTemplateId) {
|
||||
try {
|
||||
const template = templateManager.value!.getTemplate(savedTemplateId)
|
||||
const template = await templateManager.value!.getTemplate(savedTemplateId)
|
||||
if (template && template.metadata.templateType === 'optimize') {
|
||||
selectedOptimizeTemplate.value = template
|
||||
console.log('[loadSystemOptimizeTemplate] 成功加载已保存的模板:', template.name)
|
||||
@@ -182,7 +182,7 @@ export function useTemplateManager(
|
||||
}
|
||||
|
||||
// 回退到第一个可用的系统优化模板
|
||||
const templates = templateManager.value!.listTemplatesByType('optimize')
|
||||
const templates = await templateManager.value!.listTemplatesByType('optimize')
|
||||
console.log('[loadSystemOptimizeTemplate] 可用的系统优化模板数量:', templates.length)
|
||||
|
||||
if (templates.length > 0) {
|
||||
@@ -219,7 +219,7 @@ export function useTemplateManager(
|
||||
|
||||
if (savedTemplateId) {
|
||||
try {
|
||||
const template = templateManager.value!.getTemplate(savedTemplateId)
|
||||
const template = await templateManager.value!.getTemplate(savedTemplateId)
|
||||
if (template && template.metadata.templateType === 'userOptimize') {
|
||||
selectedUserOptimizeTemplate.value = template
|
||||
console.log('[loadUserOptimizeTemplate] 成功加载已保存的模板:', template.name)
|
||||
@@ -242,7 +242,7 @@ export function useTemplateManager(
|
||||
}
|
||||
|
||||
// 回退到第一个可用的用户优化模板
|
||||
const templates = templateManager.value!.listTemplatesByType('userOptimize')
|
||||
const templates = await templateManager.value!.listTemplatesByType('userOptimize')
|
||||
console.log('[loadUserOptimizeTemplate] 可用的用户优化模板数量:', templates.length)
|
||||
|
||||
if (templates.length > 0) {
|
||||
@@ -279,7 +279,7 @@ export function useTemplateManager(
|
||||
|
||||
if (savedTemplateId) {
|
||||
try {
|
||||
const template = templateManager.value!.getTemplate(savedTemplateId)
|
||||
const template = await templateManager.value!.getTemplate(savedTemplateId)
|
||||
if (template && template.metadata.templateType === 'iterate') {
|
||||
selectedIterateTemplate.value = template
|
||||
console.log('[loadIterateTemplate] 成功加载已保存的模板:', template.name)
|
||||
@@ -302,7 +302,7 @@ export function useTemplateManager(
|
||||
}
|
||||
|
||||
// 回退到第一个可用的迭代模板
|
||||
const templates = templateManager.value!.listTemplatesByType('iterate')
|
||||
const templates = await templateManager.value!.listTemplatesByType('iterate')
|
||||
console.log('[loadIterateTemplate] 可用的迭代模板数量:', templates.length)
|
||||
|
||||
if (templates.length > 0) {
|
||||
|
||||
3344
pnpm-lock.yaml
generated
3344
pnpm-lock.yaml
generated
File diff suppressed because it is too large
Load Diff
Reference in New Issue
Block a user