BossAgents 代码库质量评估报告

一、代码量统计

1.1 总体规模

1.2 核心文件分析

二、server.js 架构分析

2.1 架构特点

优点:

• 采用模块化加载设计，支持延迟加载SQLite依赖模块
• 统一错误处理机制 (setupGlobalErrorHandlers)
• 支持多种数据库后端
• 配置优先级合理: 环境变量 > config.yaml > 默认值

问题清单:

| 问题类型 | 位置 | 严重程度 | 描述 |

|----------|------|----------|------|

| 全局变量滥用 | 多处 | 中 | 使用 global._xxx 存储状态，共发现 130 处 |

| 空catch块 | 多处 | 高 | 发现 577 处空 catch 块，错误被静默吞没 |

| 单文件过大 | server.js | 中 | 主文件超过 5000 行，职责过多 |

| 路由分散 | server.js | 中 | 路由处理分散在主文件和routes目录 |

2.2 全局变量统计

// 发现的主要全局变量:
global._pvDb                    // 产品供应商数据库
global._procurementTimers       // 采购定时器
global._procurementConfirmMap   // 采购确认映射
global._pendingProcurements     // 待处理采购
global._schedulerConfig         // 调度器配置
global.__SCSAIClient             // SCSAI客户端
global._ruleEngine              // 规则引擎
global._chipwiseContexts        // ChipWise上下文
global._plmDocBrainTasks        // PLM文档任务
global._evolutionScheduler      // 演化调度器

风险: 全局变量在多请求并发场景下可能导致数据污染和竞态条件。

2.3 空catch块示例

// D:\bossagents\server\digital-staff\index.js:99
} catch (_) {}

// D:\bossagents\server.js:347
} catch (e) {}

// D:\bossagents\server\services\content-engine.js:153
} catch (_) {}

影响: 错误被静默吞没，无法追踪问题根因，生产环境调试困难。

三、数字员工实现分析

3.1 员工花名册

3.2 实现状态分析

真实实现 (8/9):

• 所有核心员工都有完整的worker实现
• 通过 CapabilityRuntime 调用规则引擎和LLM
• 支持人在回路确认机制 (requestConfirmation)
• 支持协作链任务派发

废弃实现 (1/9):

• ⟦DS-SCSAI-001⟧ 已标记为 [DEPRECATED]，统一走 CapabilityDispatcher

四、Worker实现盘点

4.1 Worker功能分析表

4.2 关键发现

采购助手:

// 三阶段执行流程
// Phase 1: 查找供应商 → requestConfirmation
// Phase 2: 跑全流程 → 等待用户确认
// Phase 3: 生成PO → 协作派单给成本优化师和供应商管家

成本优化师:

// SCSAI无数据时的降级策略
if (allParts.length === 0) {
  allParts = generateBomByRule(merged.productModel || 'M1000');
  dataSource = '规则生成(SCSAI无数据)';
}

五、数据持久化分析

5.1 持久化方式统计

| 方式 | 使用场景 | 文件数 | 风险 |

|------|----------|--------|------|

| JSON文件 | 配置、日志、缓存 | 100+ | 高 - 无事务、无并发控制 |

| SQLite (sql.js) | 主数据库 | 1 | 中 - 纯JS实现，性能受限 |

| MySQL | 可选后端 | 1 | 低 - 需配置环境变量 |

5.2 JSON文件当数据库的问题

发现的JSON数据文件:

server/data/digital-staff-logs.json    // 数字员工日志
server/data/digital-staff-config.json  // 数字员工配置
server/data/content-items.json         // 内容项
server/data/vendors.json               // 供应商
server/data/orders.json                // 订单
server/data/staff-tasks.json           // 员工任务
server/data/execution-logs.json        // 执行日志

问题代码示例:

// server/digital-staff/index.js:84-102
function loadData(filepath, defaultValue) {
    try {
        if (fs.existsSync(filepath)) {
            const content = fs.readFileSync(filepath, 'utf8').trim();
            if (!content) return defaultValue;
            return JSON.parse(content);
        }
    }
    catch (e) {
        console.warn('加载数据失败:', filepath, e.message);
        // JSON 损坏时备份并重建
        try {
            const backup = filepath + '.bak.' + Date.now();
            fs.renameSync(filepath, backup);
        } catch (_) {}
    }
    return defaultValue;
}

风险分析:

1. 并发写入冲突: 多个请求同时写入可能导致数据丢失
2. 无事务支持: 写入过程中崩溃可能导致数据损坏
3. 性能瓶颈: 大文件读写阻塞事件循环
4. 无索引支持: 查询需要全量加载

六、路由和服务盘点

6.1 路由模块清单 (72个)

6.2 服务模块清单 (75个)

七、架构问题清单

7.1 高优先级问题

7.2 中优先级问题

八、总结与建议

8.1 整体评估

8.2 改进建议

短期 (1-2周):

1. 修复所有空catch块，添加错误日志
2. 将高频读写的JSON文件迁移到SQLite
3. 清理废弃的Worker代码

中期 (1-2月):

1. 重构server.js，拆分路由处理
2. 统一数据库访问层，消除global._pvDb直接访问
3. 添加单元测试覆盖核心Worker

长期 (3-6月):

1. 引入Redis作为缓存层，替代内存全局变量
2. 完善监控和告警机制
3. 建立代码质量门禁

报告生成时间: 2026-07-22

分析范围: D:\bossagents 全量代码库