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 全局变量统计

``javascript

// 发现的主要全局变量:

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块示例

`javascript

// 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 关键发现

采购助手:

`javascript

// 三阶段执行流程

// Phase 1: 查找供应商 → requestConfirmation

// Phase 2: 跑全流程 → 等待用户确认

// Phase 3: 生成PO → 协作派单给成本优化师和供应商管家

`

成本优化师:

`javascript

// SCSAI无数据时的降级策略

if (allParts.length === 0) {

allParts = generateBomByRule(merged.productModel || 'M1000');

dataSource = '规则生成(SCSAI无数据)';

}

`

五、数据持久化分析

5.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 // 执行日志

`

问题代码示例:

`javascript

// 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 全量代码库