|
881
|
116
|
5
|
5
|
|
0
|
ci(diagnostic): 只读核对现有runner的维护边界
|
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793581907
|
1793596652
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
882
|
22
|
121
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793588946
|
1793588953
|
1793588953
|
0
|
0
|
0
|
Edit
Delete
|
|
883
|
116
|
6
|
5
|
|
0
|
fix(notification-client): bind delivery receipts a fix(notification-client): bind delivery receipts and prepare remaining stage inputs...
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793605952
|
1793605961
|
1793605961
|
0
|
0
|
0
|
Edit
Delete
|
|
884
|
116
|
7
|
5
|
|
0
|
docs(permission): preserve natural idle discard ev docs(permission): preserve natural idle discard evidence and incomplete recovery...
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793605980
|
1793605990
|
1793605990
|
0
|
0
|
0
|
Edit
Delete
|
|
885
|
116
|
8
|
5
|
|
0
|
feat(notification): bind signing credentials to co feat(notification): bind signing credentials to controlled M2 access...
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793610788
|
1793610798
|
1793610798
|
0
|
0
|
0
|
Edit
Delete
|
|
886
|
116
|
9
|
5
|
|
0
|
docs(credential): 补齐M2真实请求键与双侧授权验收前置
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793610828
|
1793610838
|
1793610838
|
0
|
0
|
0
|
Edit
Delete
|
|
887
|
146
|
2
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793614942
|
1793614948
|
1793614948
|
0
|
0
|
0
|
Edit
Delete
|
|
888
|
116
|
10
|
5
|
|
0
|
docs(credential): 补齐恢复与生产签收待签材料
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793617972
|
1793617980
|
1793617980
|
0
|
0
|
0
|
Edit
Delete
|
|
889
|
144
|
1
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793673556
|
1793673561
|
1793673561
|
0
|
0
|
0
|
Edit
Delete
|
|
890
|
116
|
11
|
5
|
|
0
|
feat(payment): add candidate provider port and det feat(payment): add candidate provider port and deterministic simulator; verify pure Node safety cases...
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793674092
|
1793674128
|
1793674128
|
0
|
0
|
0
|
Edit
Delete
|
|
891
|
116
|
12
|
5
|
|
0
|
docs: archive IdentityWeb launcher preparation and docs: archive IdentityWeb launcher preparation and independent v2 evidence...
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793674146
|
1793674154
|
1793674154
|
0
|
0
|
0
|
Edit
Delete
|
|
892
|
116
|
13
|
5
|
|
0
|
chore(reports): adopt Runtime static originals at chore(reports): adopt Runtime static originals at dc13b515 and clarify evidence snapshot...
|
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793674166
|
1793710134
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
893
|
143
|
1
|
1
|
|
0
|
0915版本
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793695768
|
1793695782
|
1793695782
|
0
|
0
|
0
|
Edit
Delete
|
|
894
|
144
|
2
|
1
|
|
0
|
0915 版本内容
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793695809
|
1793695849
|
1793695849
|
0
|
0
|
0
|
Edit
Delete
|
|
895
|
21
|
354
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793863976
|
1793863983
|
1793863983
|
0
|
0
|
0
|
Edit
Delete
|
|
896
|
21
|
355
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793864000
|
1793864006
|
1793864006
|
0
|
0
|
0
|
Edit
Delete
|
|
897
|
22
|
122
|
1
|
|
0
|
日志增加版本好
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793864188
|
1793864193
|
1793864193
|
0
|
0
|
0
|
Edit
Delete
|
|
898
|
22
|
123
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793864207
|
1793864212
|
1793864212
|
0
|
0
|
0
|
Edit
Delete
|
|
899
|
23
|
71
|
1
|
|
0
|
创建 Ahead_room_discontinue_rule_model.php
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793864236
|
1793864246
|
1793864246
|
0
|
0
|
0
|
Edit
Delete
|
|
900
|
23
|
72
|
1
|
|
0
|
创建 Ahead_room_discontinue_rule_model.php
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793864270
|
1793864280
|
1793864280
|
0
|
0
|
0
|
Edit
Delete
|
|
901
|
18
|
238
|
1
|
|
0
|
改用 Vue 开发代理,修复走马灯修改门店bug,美团直连移除红字备注
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793864309
|
1793864322
|
1793864322
|
0
|
0
|
0
|
Edit
Delete
|
|
902
|
18
|
239
|
1
|
|
0
|
改用 Vue 开发代理,修复走马灯修改门店bug,美团直连移除红字备注
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793864343
|
1793864354
|
1793864354
|
0
|
0
|
0
|
Edit
Delete
|
|
903
|
21
|
356
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793871747
|
1793871752
|
1793871752
|
0
|
0
|
0
|
Edit
Delete
|
|
904
|
21
|
357
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793871761
|
1793871766
|
1793871766
|
0
|
0
|
0
|
Edit
Delete
|
|
905
|
22
|
124
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793871786
|
1793871793
|
1793871793
|
0
|
0
|
0
|
Edit
Delete
|
|
906
|
22
|
125
|
1
|
|
0
|
1
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793871801
|
1793871806
|
1793871806
|
0
|
0
|
0
|
Edit
Delete
|
|
907
|
18
|
240
|
1
|
|
0
|
第三方库路径
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793871827
|
1793871831
|
1793871831
|
0
|
0
|
0
|
Edit
Delete
|
|
908
|
18
|
241
|
1
|
|
0
|
第三方库路径
|
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793871844
|
1793871850
|
1793871850
|
0
|
0
|
0
|
Edit
Delete
|
|
44
|
6
|
1
|
5
|
|
0
|
test(finance): add finance mainline smoke and hard test(finance): add finance mainline smoke and harden invoice detail formatting...
|
## Summary
- add a finance-role mainline smoke tha ## Summary
- add a finance-role mainline smoke that covers invoices, AR ledger, and finance dashboard
- treat reconciliation 403 as the current permission boundary for the finance fixture
- harden invoice detail amount formatting so null values no longer trip the page
## Validation
- E2E_SKIP_WEB_SERVER=true E2E_USE_SYSTEM_CHROME=true pnpm -C e2e exec playwright test tests/auth/finance-mainline-flow-smoke.spec.ts --project=chromium-no-auth
- result: 1 passed (9.4s)
## Note
- git fetch/clone against the current Gitea main is currently failing with upload-pack not our ref, so this patch was replayed through the Gitea contents API on top of the live main history....
|
0
|
0
|
1
|
1
|
1
|
|
0
|
1774950058
|
1779328857
|
1778625303
|
0
|
0
|
0
|
Edit
Delete
|
|
1
|
2
|
1
|
1
|
|
0
|
代码审查报告:main 分支 - 111122
|
## 代码审查自动报告
**分支**: main
**提交**: 29b5655925cbfa8c ## 代码审查自动报告
**分支**: main
**提交**: 29b5655925cbfa8c56c5447ee5bb8dc020b74c5b
**时间**: 2026-03-17 10:12:36
[2026-03-17 10:11:59] 正在调用 Qwen API 进行代码审查...
## 📋 审查摘要
- 变更文件数:0 (未提供实际代码文件)
- 主要变更:仅提供了构建/运行环境的日志信息(工作目录、仓库 URL),**未包含任何实际的源代码变更内容(Diff)**。
## 🐛 发现的问题
### 问题 1: 缺失实际代码变更内容
- **严重程度**: 严重
- **文件**: N/A (输入内容)
- **行号**: 全部
- **问题描述**: 提供的文本仅为 CI/CD 或脚本执行的日志头信息,不包含任何编程语言(如 Python, Java, Go 等)的代码逻辑、Diff 对比或文件修改内容。无法进行逻辑、性能或安全性审查。
- **建议修复**: 请提供实际的 `git diff` 输出内容,或直接粘贴修改过的源代码文件内容。
### 问题 2: 潜在敏感信息泄露
- **严重程度**: 中等
- **文件**: 日志信息
- **行号**: 第 2 行
- **问题描述**: 日志中包含了内部仓库 URL (`https://gitea.g-hi.com/juhai/testpc.git`)。如果这是企业内部私有仓库,将其暴露在公共网络或 AI 对话中可能存在信息泄露风险。
- **建议修复**: 在分享日志或调试信息时,建议对内部域名、IP 地址、仓库路径等敏感信息进行脱敏处理(例如替换为 `example.com` 或 `***`)。
### 问题 3: 时间戳异常
- **严重程度**: 轻微
- **文件**: 日志信息
- **行号**: 第 1 行
- **问题描述**: 日志时间显示为 `[2026-03-17 10:11:56]`,这是一个未来的时间(相对于当前实际时间)。这可能意味着系统时钟配置错误、测试数据伪造或日志生成逻辑存在问题。
- **建议修复**: 检查生成该日志的系统时钟设置,或确认是否为预期的测试数据。如果是生产环境日志,需立即排查时间同步服务(如 NTP)。
## ✅ 代码亮点
- 由于未提供实际代码,无法评估代码亮点。
## 📝 总体建议
本次提交的内容**无法进行有效的代码审查**,因为缺少核心的源代码变更部分。
1. **补充材料**:请重新提交包含实际代码修改(Diff)的内容。
2. **安全意识**:在未来的分享中,请注意脱敏内部仓库地址、API 密钥、密码等敏感信息。
3. **环境检查**:请验证生成日志的系统时间是否准确,避免因时间戳问题导致日志分析或审计困难。
期待您提供具体的代码变更内容,以便我进行更深入的技术审查。
---
*此 Issue 由代码审查服务自动创建*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773713556
|
1773713556
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
2
|
2
|
2
|
1
|
|
0
|
代码审查报告:main 分支 - 111122
|
## 代码审查自动报告
**分支**: main
**提交**: 29b5655925cbfa8c ## 代码审查自动报告
**分支**: main
**提交**: 29b5655925cbfa8c56c5447ee5bb8dc020b74c5b
**时间**: 2026-03-17 10:12:49
[2026-03-17 10:12:02] 正在调用 Qwen API 进行代码审查...
## 📋 审查摘要
- 变更文件数:0 (未提供有效的代码 Diff)
- 主要变更:提供的內容仅为运行日志或环境元数据(工作目录、仓库 URL),不包含任何实际的源代码变更(如 Python, Java, C++ 等)。
## 🐛 发现的问题
### 问题 1: 缺少可审查的源代码
- **严重程度**: 严重
- **文件**: N/A
- **行号**: N/A
- **问题描述**: 本次提交的内容不包含任何编程语言的代码逻辑、函数变更或配置文件的修改。仅包含两行日志输出信息。作为代码审查专家,无法基于此内容评估逻辑正确性、性能或架构问题。
- **建议修复**: 请提供实际的代码 Diff 内容(例如包含 `+` 和 `-` 标记的代码变更块),以便进行有效的技术审查。
### 问题 2: 敏感信息潜在泄露 (信息泄露)
- **严重程度**: 中等
- **文件**: 日志/输出内容
- **行号**: 第 2 行 (`仓库 URL: ...`)
- **问题描述**: 日志中明文打印了内部仓库 URL (`https://gitea.g-hi.com/juhai/testpc.git`)。如果此日志被提交到公共仓库、发送给外部人员或记录在公开系统中,可能导致内部项目结构暴露。此外,`/tmp/` 下的随机目录名虽通常无害,但结合仓库 URL 可能辅助攻击者推测构建环境。
- **建议修复**:
1. 确保日志脱敏,不要在日志中硬编码或打印完整的内部仓库 URL。
2. 检查是否误将日志文件提交到了版本控制系统中(日志文件应加入 `.gitignore`)。
### 问题 3: 系统时间配置异常
- **严重程度**: 轻微
- **文件**: 日志/输出内容
- **行号**: 第 1 行 (`[2026-03-17 ...]`)
- **问题描述**: 日志显示的时间为 `2026 年`,这是一个未来时间。这可能是由于:
1. 服务器系统时间配置错误。
2. 这是一个测试用的 Mock 数据。
3. 代码中硬编码了时间戳。
如果是生产环境日志,时间错误会导致日志分析、审计追踪和故障排查严重混乱。
- **建议修复**: 检查服务器 NTP 同步服务,或确认代码中是否存在硬编码的时间戳逻辑。
## ✅ 代码亮点
- 无(因未提供实际源代码)。
## 📝 总体建议
1. **补充材料**:当前提供的内容无法构成有效的代码审查。请补充具体的源代码变更 Diff。
2. **安全意识**:在分享日志、截图或配置文件时,请务必抹去内部域名、IP 地址、仓库路径等敏感信息。
3. **规范日志**:确保生产环境的日志时间戳准确,且不包含敏感的基础设施信息。
4. **版本控制**:确认该日志内容不是被意外 `git add` 的文件,日志文件应当被忽略。
**结论**: 无法进行实质性代码审查,请提供正确的代码变更内容。
---
*此 Issue 由代码审查服务自动创建*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773713569
|
1773713569
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
7
|
2
|
7
|
1
|
|
0
|
代码审查报告:main 分支 - 测试提交内容21
|
## 代码审查自动报告
**分支**: main
**提交**: 930eca9fb53f66e8 ## 代码审查自动报告
**分支**: main
**提交**: 930eca9fb53f66e8bd5c691867946eecb08d648e
**时间**: 2026-03-17 10:25:09
[2026-03-17 10:24:16] 正在调用 Qwen API 进行代码审查...
## 📋 审查摘要
- 变更文件数:1
- 主要变更:修改了 `pc/index.php` 入口文件,增加了基于 URL 的分支名称检测逻辑,定义了大量项目常量(路径、URL 等),并保留了 CodeIgniter 框架的标准引导代码。
## 🐛 发现的问题
### 问题 1: 存在明显的调试垃圾代码和无效字符
- **严重程度**: 中等
- **文件**: pc/index.php
- **行号**: 5-9 左右
- **问题描述**: 文件头部包含大量无意义的调试文本和字符,例如 `//ssssss`, `sss 所得到的`, `dwdwdw`, `收拾收拾`。这表明代码在提交前未经过清理,严重影响代码专业性和可读性。
- **建议修复**: 删除所有与业务逻辑无关的调试注释和乱码字符,保持代码整洁。
### 问题 2: 默认环境变量设置为 development 存在安全风险
- **严重程度**: 严重
- **文件**: pc/index.php
- **行号**: 65 左右 (`define('ENVIRONMENT', isset($_SERVER['CI_ENV']) ? $_SERVER['CI_ENV'] : 'development');`)
- **问题描述**: 如果服务器未设置 `CI_ENV` 环境变量,系统默认进入 `development` 模式。在此模式下,`display_errors` 被开启且错误报告级别为 `-1`。若此代码部署到生产环境且配置缺失,会导致敏感错误信息、堆栈跟踪甚至代码路径泄露给用户。
- **建议修复**: 将默认环境改为 `production`,或者在服务器层面强制配置 `CI_ENV`。例如:`define('ENVIRONMENT', isset($_SERVER['CI_ENV']) ? $_SERVER['CI_ENV'] : 'production');`
### 问题 3: 硬编码绝对路径导致可移植性差
- **严重程度**: 中等
- **文件**: pc/index.php
- **行号**: 173 左右 (`define('COMMONCLASS', "/mnt/data/www/CommonClass".DIRECTORY_SEPARATOR);`)
- **问题描述**: 代码中硬编码了特定的服务器绝对路径(如 `/mnt/data/...`)。这将导致代码无法在其他环境(如开发环境、测试环境或其他服务器)运行,耦合了基础设施细节。
- **建议修复**: 将路径配置移至配置文件(如 `config.php`)中,或使用相对路径结合 `__DIR__` 动态获取。
### 问题 4: 硬编码域名和 URL 结构
- **严重程度**: 中等
- **文件**: pc/index.php
- **行号**: 178-184 左右 (`define('BASE_URL', ...)` 等)
- **问题描述**: 域名(`g-hi.com`)、子域名前缀(`pc.`, `pay.`)以及协议(http/https 混用)被硬编码在入口文件中。`PERSONNEL_BASE_URL` 使用了 `http` 而其他使用 `https`,可能存在混合内容安全风险。
- **建议修复**: 将 URL 配置移至配置文件,根据环境动态加载。确保所有生产环境 URL 强制使用 `https`。
### 问题 5: 依赖 $_SERVER['PHP_SELF'] 进行逻辑判断
- **严重程度**: 轻微
- **文件**: pc/index.php
- **行号**: 10-14 左右
- **问题描述**: 使用 `$_SERVER['PHP_SELF']` 解析 URL 路径来确定分支名称 (`BRANCHNAME`)。`PHP_SELF` 依赖于 URL 重写规则和访问方式,不够可靠。如果 URL 结构变化或包含特殊字符,正则匹配可能失效或产生意外结果。
- **建议修复**: 建议使用框架的路由机制或更稳定的 `$_SERVER['REQUEST_URI']` 配合解析,并将此逻辑移至配置类而非入口文件。
### 问题 6: 代码风格不一致
- **严重程度**: 轻微
- **文件**: pc/index.php
- **行号**: 全文
- **���题描述**: 缩进混用(Tab 和空格),部分常量定义后有多余制表符(如 `define('PHP_PATH','../'); //`)。
- **建议修复**: 统一使用 PSR 标准的代码风格,配置编辑器自动格式化,移除行尾多余空白。
## ✅ 代码亮点
- 保留了 CodeIgniter 框架标准的目录路径检测逻辑(`realpath` 检查),增强了系统路径的可靠性。
- 对不同环境(development/testing/production)的错误报告级别进行了区分配置(尽管默认值有风险)。
## 📝 总体建议
该代码变更存在**严重的安全隐患**和**代码质量**问题。
1. **立即修复**:必须清理文件头部的垃圾字符,并将默认环境变量修改为 `production`,防止生产环境信息泄露。
2. **架构优化**:入口文件 (`index.php`) 应保持轻量,仅负责引导框架。所有的业务常量(路径、URL、域名)应提取到配置文件中,避免硬编码。
3. **流程规范**:提交代码前请执行代码审查和清理,避免调试代码进入版本库。
建议暂缓合并,待上述严重问题修复后再进行集成。
---
*此 Issue 由代码审查服务自动创建*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773714309
|
1773714309
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
8
|
2
|
8
|
1
|
|
0
|
代码审查报告:main 分支 - 测试提交内容21
|
## 代码审查自动报告
**分支**: main
**提交**: 930eca9fb53f66e8 ## 代码审查自动报告
**分支**: main
**提交**: 930eca9fb53f66e8bd5c691867946eecb08d648e
**时间**: 2026-03-17 10:25:46
[2026-03-17 10:24:16] 正在调用 Qwen API 进行代码审查...
## 📋 审查摘要
- **变更文件数**: 1
- **主要变更**: 修改了 `pc/index.php` 入口文件,增加了基于 URL 路径的分支名称识别逻辑(`BRANCHNAME`),定义了多个新的项目常量(如 `BASE_URL`, `COMMONCLASS` 等),并保留了 CodeIgniter 框架的基础引导代码。但代码中混入了大量无效字符和调试信息。
## 🐛 发现的问题
### 问题 1: 致命语法错误 (Syntax Error)
- **严重程度**: 严重
- **文件**: pc/index.php
- **行号**: 第 4, 5, 8 行左右
- **问题描述**: 代码中混入了非 PHP 语法的随机文本,这将导致 PHP 解析失败,网站无法访问(500 错误)。
- 第 4 行:`define('BASE_DOMAIN', 'g-hi');sss 所得到的`
- 第 5 行:`define('DEBUG_VERSION','test-');dwdwdw`
- 第 8 行:`$pattern='/pc-(.*?)\//is';收拾收拾`
- PHP 无法识别分号后的 `sss 所得到的`、`dwdwdw`、`收拾收拾` 等字符,会抛出 Parse Error。
- **建议修复**: 删除所有语句末尾的无关文本,确保每行 PHP 代码语法纯净。
```php
// 修复后示例
define('BASE_DOMAIN', 'g-hi');
define('DEBUG_VERSION', 'test-');
$pattern = '/pc-(.*?)\//is';
```
### 问题 2: 环境变量安全风险 (Environment Security)
- **严重程度**: 高
- **文件**: pc/index.php
- **行号**: 约 65 行
- **问题描述**: `ENVIRONMENT` 常量直接依赖于 `$_SERVER['CI_ENV']`。
```php
define('ENVIRONMENT', isset($_SERVER['CI_ENV']) ? $_SERVER['CI_ENV'] : 'development');
```
如果服务器配置不当(如 CGI/FastCGI 配置问题),攻击者可能通过伪造请求头或环境变量将环境强制设置为 `development`。这将导致 `display_errors` 被开启,敏感信息(路径、SQL 语句、堆栈跟踪)可能泄露给终端用户。
- **建议修复**:
1. 在生产环境中,建议在 Web 服务器配置(Nginx/Apache)中固定 `CI_ENV` 环境变量,而不是在 PHP 代码中信任 `$_SERVER`。
2. 或者在 PHP 中对 `$_SERVER['CI_ENV']` 的值进行白名单验证。
```php
$env = isset($_SERVER['CI_ENV']) ? $_SERVER['CI_ENV'] : 'production';
if (!in_array($env, ['development', 'testing', 'production'])) {
$env = 'production';
}
define('ENVIRONMENT', $env);
```
### 问题 3: 硬编码绝对路径 (Hardcoded Paths)
- **严重程度**: 中等
- **文件**: pc/index.php
- **行号**: 约 75-78 行
- **问题描述**: 代码中硬编码了服务器特定的绝对路径,如 `/mnt/data/www/CommonClass` 和 `/mnt/data/`。
```php
define('MNTDATA', dirname(ROOTWEB).DIRECTORY_SEPARATOR);//mnt/data/
define('COMMONCLASS', "/mnt/data/www/CommonClass".DIRECTORY_SEPARATOR);
```
这导致代码缺乏可移植性,如果部署到其他服务器或目录结构变更,代码将失效。
- **建议修复**: 使用相对路径,或将这些路径配置移到独立的配置文件(如 `config.php`)中,根据不同环境加载不同配置。
### 问题 4: 代码整洁度与调试残留 (Code Cleanliness)
- **严重程度**: 轻微
- **文件**: pc/index.php
- **行号**: 第 3 行,第 80 行等
- **问题描述**: 存在无意义的注释和调试残留。
- 第 3 行:`//ssssss`
- 第 80 行:`//17.3.9 增加几个定义`
- 第 4 行注释中混入中文文本 `sss 所得���的`(除了导致语法错误外,也是脏数据)。
- **建议修复**: 清理所有无意义的注释和调试代码,保持代码库整洁。
### 问题 5: URL 路径解析逻辑潜在风险
- **严重程度**: 中等
- **文件**: pc/index.php
- **行号**: 约 6-12 行
- **问题描述**: 使用 `$_SERVER['PHP_SELF']` 进行正则匹配来获取分支名。
```php
$now_url = str_replace("/index.php","",$_SERVER['PHP_SELF']);
preg_match($pattern, $now_url, $preg_url);
// ...
define('BRANCHNAME',"pc-".$preg_url[1]);
```
`$_SERVER['PHP_SELF']` 在某些服务器配置下可能被用户操纵(尽管较难)。如果 `BRANCHNAME` 后续被用于文件包含、数据库表名拼接等操作,可能存在注入风险。
- **建议修复**: 对 `$preg_url[1]` 进行严格的白名单验证(如只允许字母数字),确保 `BRANCHNAME` 安全。
## ✅ 代码亮点
- **路径存在性检查**: 代码中包含了对 `system`、`application` 和 `view` 目录是否存在的有效检查,并在检查失败时返回 503 状态码,这有助于避免服务器暴露内部错误信息。
- **错误报告分级**: 根据 `ENVIRONMENT` 不同设置了不同的 `error_reporting` 级别,符合最佳实践(尽管 `ENVIRONMENT` 的设定方式有待改进)。
## 📝 总体建议
当前代码**不可直接部署**,因为存在致命的语法错误,会导致网站立即瘫痪。
1. **紧急修复**: 首先必须删除所有导致语法错误的垃圾文本(`sss 所得到的` 等)。
2. **安全加固**: 重新评估 `ENVIRONMENT` 的设定方式,防止生产环境被意外切换为调试模式。
3. **配置分离**: 建议将硬编码的路径和域名配置提取到单独的配置文件中,便于多环境管理。
4. **代码清理**: 提交前请运行代码格式化工具,并删除所有调试注释。
5. **输入验证**: 对从 URL 解析出的分支名称进行验证,防止潜在的逻辑注入。
建议在修复上述问题后,再进行本地测试和代码审查。
---
*此 Issue 由代码审查服务自动创建*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773714346
|
1773714346
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
5
|
2
|
5
|
1
|
|
0
|
代码审查报告:main 分支 - 测试提交内容2
|
## 代码审查自动报告
**分支**: main
**提交**: e81717d4e0161b88 ## 代码审查自动报告
**分支**: main
**提交**: e81717d4e0161b8859cf8a712eeb882a87987e78
**时间**: 2026-03-17 10:21:46
[2026-03-17 10:21:19] 正在调用 Qwen API 进行代码审查...
## 📋 审查摘要
- 变更文件数:1
- 主要变更:修改了 `pc/index.php` 入口文件,添加了分支路径识别逻辑、多个常量定义,保留了 CodeIgniter 框架初始化代码
## 🐛 发现的问题
### 问题 1: 存在明显的调试/垃圾代码未清理
- **严重程度**: 中等
- **文件**: pc/index.php
- **行号**: 2-4
- **问题描述**: 代码开头存在明显的调试痕迹和垃圾字符,如 `//ssssss`、`sss 所得到的`、`dwdwdw` 等,这表明代码未经过清理就提交了
- **建议修复**: 删除所有调试注释和无意义字符,保持代码整洁
### 问题 2: $_SERVER['PHP_SELF'] 未做安全过滤
- **严重程度**: 严重
- **文件**: pc/index.php
- **行号**: 5
- **问题描述**: 直接使用 `$_SERVER['PHP_SELF']` 获取 URL 路径,该变量可被用户操控,存在 XSS 和路径注入风险
- **建议修复**: 使用 `filter_input()` 或对输入进行白名单验证,例如:
```php
$now_url = filter_input(INPUT_SERVER, 'PHP_SELF', FILTER_SANITIZE_URL);
```
### 问题 3: 正则表达式存在潜在安全风险
- **严重程度**: 中等
- **文件**: pc/index.php
- **行号**: 6-7
- **问题描述**: `preg_match` 使用用户可控的 URL 路径作为输入,正则模式 `/pc-(.*?)\//is` 可能被恶意构造的路径绕过或导致意外匹配
- **建议修复**: 对匹配结果进行白名单验证,限制分支名称只能包含字母、数字、下划线:
```php
if(isset($preg_url[1]) && !empty($preg_url[1]) && preg_match('/^[a-zA-Z0-9_-]+$/', $preg_url[1])){
```
### 问题 4: 敏感信息硬编码
- **严重程度**: 严重
- **文件**: pc/index.php
- **行号**: 3, 185-191
- **问题描述**: 域名、路径、OSS 地址等敏感配置硬编码在代码中,不同环境需要修改代码,且可能泄露基础设施信息
- **建议修复**: 将配置移至独立配置文件或环境变量,例如:
```php
define('BASE_DOMAIN', getenv('BASE_DOMAIN') ?: 'g-hi');
```
### 问题 5: 开发环境配置可能泄露到生产环境
- **严重程度**: 严重
- **文件**: pc/index.php
- **行号**: 108-110
- **问题描述**: `error_reporting(-1)` 和 `ini_set('display_errors', 1)` 在 development 模式下会显示所有错误,如果 ENVIRONMENT 判断被绕过,敏感信息将泄露
- **建议修复**: 确保生产环境强制关闭错误显示,添加额外的环境检测机制
### 问题 6: 常量定义重复
- **严重程度**: 轻微
- **文件**: pc/index.php
- **行号**: 185, 189
- **问题描述**: `BASE_URL` 和 `PC_BASE_URL` 定义完全相同,造成冗余
- **建议修复**: 删除重复定义,或明确区分两者用途
### 问题 7: 路径常量依赖特定服务器结构
- **严重程度**: 中等
- **文件**: pc/index.php
- **行号**: 179-182
- **问题描述**: `MNTDATA`、`COMMONCLASS` 等路径硬编码为 `/mnt/data/`,在不同服务器环境下会导致路径错误
- **建议修复**: 使用相对路径或可配置的路径前缀
### 问题 8: 缺少输入验证和错误处理
- **严重程度**: 中等
- **文件**: pc/index.php
- **行号**: 8-15
- **问题描述**: 分支名称提取后没有验证合法性,可能注入恶意值到常量中
- **建议修复**: 添加分支名称白名单验证,限制长度和字符集
## ✅ 代码亮点
- 保留了 CodeIgniter 框架的标准初始化结构
- 对不同环境(development/testing/production)做了错误报告区分
- 添加了路径存在性检查,避免静默失败
## 📝 总体建议
**代码质量评级:⚠️ 需要重大改进**
主要问题集中在:
1. **安全性**:多处用户输入未过滤,存在注入风险
2. **代码整洁度**:包含大量调试垃圾代码
3. **可维护性**:硬编码过多,不利于多环境部署
**优先修复建议**:
1. 立即清理调试代码和垃圾字符
2. 对所有用户输入进行严格验证和过滤
3. 将配置项移至独立配置文件
4. 添加分支名称的白名单验证机制
5. 确保生产环境不会泄露错误信息
建议在合并前进行安全测试,特别是针对 URL 参数注入和路径遍历的测试。
---
*此 Issue 由代码审查服务自动创建*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773714106
|
1773714106
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
6
|
2
|
6
|
1
|
|
0
|
代码审查报告:main 分支 - 测试提交内容2
|
## 代码审查自动报告
**分支**: main
**提交**: e81717d4e0161b88 ## 代码审查自动报告
**分支**: main
**提交**: e81717d4e0161b8859cf8a712eeb882a87987e78
**时间**: 2026-03-17 10:22:17
[2026-03-17 10:21:19] 正在调用 Qwen API 进行代码审查...
## 📋 审查摘要
- **变更文件数**: 1
- **主要变更**: 修改了 `pc/index.php` 入口文件,增加了动态分支名称定义、域名配置及一些硬编码路径,但引入了严重的语法错误和安全隐患。
## 🐛 发现的问题
### 问题 1: 严重的语法错误与输出污染 (Critical)
- **严重程度**: 严重
- **文件**: pc/index.php
- **行号**: 4-5
- **问题描述**: 代码中存在明显的调试残留文本 `sss 所得到的` 和 `dwdwdw` 直接写在 `define` 语句之后。
```php
define('BASE_DOMAIN', 'g-hi');sss 所得到的
define('DEBUG_VERSION','test-');dwdwdw
```
在 PHP 中,这些文本会被视为直接输出到浏览器。由于 `index.php` 是入口文件,这会导致在发送任何 HTTP 头(如 `header()` 重定向或错误码)之前就有输出,从而引发 **"Headers already sent"** 错误,导致整个应用无法正常运行。
- **建议修复**: 删除所有非代码的残留文本,确保 PHP 语句后没有多余字符,或将其放入注释中。
```php
define('BASE_DOMAIN', 'g-hi');
define('DEBUG_VERSION', 'test-');
```
### 问题 2: 环境变量可被用户伪造 (Security)
- **严重程度**: 严重
- **文件**: pc/index.php
- **行号**: 67
- **问题描述**: 环境变量 `ENVIRONMENT` 直接依赖于 `$_SERVER['CI_ENV']`。
```php
define('ENVIRONMENT', isset($_SERVER['CI_ENV']) ? $_SERVER['CI_ENV'] : 'development');
```
在某些服务器配置下,攻击者可以通过发送自定义 HTTP 头(如 `CI_ENV`)来伪造此变量。如果攻击者将其设置为 `development`,会导致 `display_errors` 开启,泄露服务器路径、SQL 语句等敏感信息。且默认值为 `development`,生产环境风险极高。
- **建议修复**: 不要直接从用户可控的 `$_SERVER` 变量中读取环境配置。应在服务器层面(如 Nginx/Apache 配置或 `.env` 文件)设置环境变量,并在代码中白名单验证。
```php
$env = $_SERVER['CI_ENV'] ?? 'production';
if (!in_array($env, ['development', 'testing', 'production'])) {
$env = 'production';
}
define('ENVIRONMENT', $env);
```
### 问题 3: 不安全的 URL 解析逻辑 (Logic/Security)
- **严重程度**: 中等
- **文件**: pc/index.php
- **行号**: 7-14
- **问题描述**: 使用 `$_SERVER['PHP_SELF']` 和正则表达式来解析分支名称。
```php
$now_url = str_replace("/index.php","",$_SERVER['PHP_SELF']);
$pattern='/pc-(.*?)\//is';
preg_match($pattern, $now_url, $preg_url);
```
`$_SERVER['PHP_SELF']` 在某些配置下可被用户操纵。此外,依赖 URL 路径结构来定义常量(`BRANCHNAME`)非常脆弱,一旦 URL 重写规则变化,逻辑就会失效。
- **建议修复**: 分支逻辑应通过配置文件或服务器环境变量管理,而不是在入口文件中解析 URL。如果必须解析,请使用更安全的 `$_SERVER['REQUEST_URI']` 并进行严格过滤。
### 问题 4: 硬编码绝对路径 (Maintainability)
- **严重程度**: 中等
- **文件**: pc/index.php
- **行号**: 135
- **问题描述**: 代码中硬编码了服务器绝对路径 `/mnt/data/www/CommonClass`。
```php
define('COMMONCLASS', "/mnt/data/www/CommonClass".DIRECTORY_SEPARATOR);
```
这导致代码无法在其他目录结构不同���服务器(如开发环境、测试环境)上运行,降低了可移植性。
- **建议修复**: 使用相对路径,或通过独立的配置文件(如 `config.php`)根据当前服务器环境定义路径。
### 问题 5: 调试代码未清理 (Code Quality)
- **严重程度**: 轻微
- **文件**: pc/index.php
- **行号**: 3
- **问题描述**: 存在无意义的调试注释 `//ssssss`。
- **建议修复**: 提交前清理所有无关注释和调试代码,保持代码整洁。
## ✅ 代码亮点
- **目录检查**: 保留了 CodeIgniter 标准的系统目录和应用目录存在性检查,并在失败时返回 503 状态码,这有助于防止因路径配置错误导致的白屏。
- **错误报告分级**: 根据环境不同设置了不同的错误报告级别,符合基本的安全实践(尽管环境变量的获取方式有问题)。
## 📝 总体建议
当前代码**不可直接发布到生产环境**。
1. **紧急修复**: 必须立即删除第 4-5 行的垃圾文本,否则网站将无法访问(报 500 错误或 Headers already sent)。
2. **安全加固**: 严禁通过 `$_SERVER` 直接控制 `ENVIRONMENT`,这是高危漏洞。请改为读取服务器固定配置。
3. **配置分离**: 将域名、路径、分支名等配置移入独立的配置文件(如 `application/config/config.php`),保持入口文件 `index.php` 的纯净。
4. **代码清理**: 提交前请运行代码格式化工具,并移除所有调试残留。
建议将此变更回退,修复上述严重问题后再重新提交。
---
*此 Issue 由代码审查服务自动创建*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773714137
|
1773714137
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
3
|
2
|
3
|
1
|
|
0
|
代码审查报告:main 分支 - 测试提交内容
|
## 代码审查自动报告
**分支**: main
**提交**: e8a92e1ba30259f2 ## 代码审查自动报告
**分支**: main
**提交**: e8a92e1ba30259f2607d811af202d67a1e14c6f6
**时间**: 2026-03-17 10:17:02
[2026-03-17 10:16:28] 正在调用 Qwen API 进行代码审查...
## 📋 审查摘要
- 变更文件数:0 (未提供实际代码 diff)
- 主要变更:仅提供了工作目录和仓库 URL 的日志元数据,未包含任何实际的代码变更内容。
## 🐛 发现的问题
### 问题 1: 缺少实际的代码变更内容
- **严重程度**: 严重
- **文件**: 输入内容
- **行号**: 全部
- **问题描述**: 提供的文本片段仅为执行日志或环境元数据(包含时间戳、工作目录、仓库 URL),不包含任何编程语言源代码、Git diff 标记(如 `+`, `-`, `diff --git` 等)或文件路径变更。无法进行逻辑、安全或质量审查。
- **建议修复**: 请提供实际的 `git diff` 输出内容,或具体文件的代码变更片段,以便进行有效的代码审查。
### 问题 2: 潜在的信息泄露风险
- **严重程度**: 轻微
- **文件**: 输入文本
- **行号**: 2
- **问题描述**: 文本中暴露了具体的内部仓库 URL (`https://gitea.g-hi.com/juhai/testpc.git`)。如果这是内部私有仓库,将其发送给外部 AI 服务可能存在信息泄露风险。
- **建议修复**: 在分享日志或配置信息时,建议对内部域名、IP 地址、仓库路径等敏感信息进行脱敏处理(例如替换为 `example.com` 或 `***`)。
### 问题 3: 系统时间异常(观察项)
- **严重程度**: 轻微
- **文件**: 输入文本
- **行号**: 1
- **问题描述**: 日志时间显示为 `2026-03-17`,这是一个未来时间。这可能意味着生成该日志的系统时钟配置错误,或者是测试用的模拟数据。
- **建议修复**: 检查生成该日志的服务器或容器系统时间设置,确保时间同步准确,以免影响日志审计和证书验证。
## ✅ 代码亮点
- 无实际代码可供评估。
## 📝 总体建议
当前提供的输入内容**无法进行代码审查**。这看起来像是 CI/CD 流水线或脚本执行时的日志头信息,而不是代码变更本身。
为了获得有效的审查结果,请补充以下信息:
1. **实际的代码 Diff**:使用 `git diff HEAD~1` 或类似命令生成的具体内容。
2. **具体的文件路径**:指明哪些文件被修改。
3. **上下文信息**:如果可能,说明这次变更的业务目的。
请注意信息安全,避免在公共渠道泄露内部仓库地址和系统路径。
---
*此 Issue 由代码审查服务自动创建*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773713822
|
1773713822
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
4
|
2
|
4
|
1
|
|
0
|
代码审查报告:main 分支 - 测试提交内容
|
## 代码审查自动报告
**分支**: main
**提交**: e8a92e1ba30259f2 ## 代码审查自动报告
**分支**: main
**提交**: e8a92e1ba30259f2607d811af202d67a1e14c6f6
**时间**: 2026-03-17 10:17:07
[2026-03-17 10:16:27] 正在调用 Qwen API 进行代码审查...
## 📋 审查摘要
- 变更文件数:0 (未提供实际源代码文件)
- 主要变更:提供的内容为执行日志或环境元数据,而非代码差异(Diff)
## 🐛 发现的问题
### 问题 1: 缺少实际源代码变更
- **严重程度**: 严重
- **文件**: 无
- **行号**: N/A
- **问题描述**: 提供的“代码变更内容”实际上是一段日志输出或环境信息,不包含任何编程语言(如 Python, Java, C++ 等)的源代码差异。无法对逻辑、算法、性能或具体的代码规范进行审查。
- **建议修复**: 请提供实际的 `git diff` 输出或源代码文件变更内容,以便进行有效的技术审查。
### 问题 2: 潜在的信息泄露风险
- **严重程度**: 中等
- **文件**: 日志/配置输出
- **行号**: 第 2 行
- **问题描述**: 日志中暴露了内部仓库 URL (`https://gitea.g-hi.com/juhai/testpc.git`) 和临时工作目录路径 (`/tmp/tmp.8GvVjC6ZU0`)。如果此日志被公开或在非受控环境中输出,可能泄露内部基础设施结构或项目命名规范。
- **建议修复**: 在生产环境或公开日志中,建议脱敏内部域名和具体路径信息。确保日志级别配置正确,避免敏感信息泄露。
### 问题 3: 时间戳异常
- **严重程度**: 轻微
- **文件**: 日志/配置输出
- **行号**: 第 1-2 行
- **问题描述**: 日志时间戳显示为 `2026-03-17`,这是一个未来时间(相对于当前实际时间)。这可能表明系统时钟配置错误、测试数据硬编码,或者是模拟数据。
- **建议修复**: 检查生成该日志的系统时钟同步设置(如 NTP),或确认是否为测试用的 Mock 数据。如果是生产代码,需确保时间源准确。
### 问题 4: 格式不符合 Diff 规范
- **严重程度**: 轻微
- **文件**: 输入内容
- **行号**: 全部
- **问题描述**: 提供的内容标记为 `diff` 代码块,但实际内容不包含标准的 Diff 格式头(如 `diff --git`, `index`, `---`, `+++`, `@@` 等)。这会导致自动化审查工具无法解析。
- **建议修复**: 确保使用 `git diff` 或类似工具生成标准的差异文件后再提交审查。
## ✅ 代码亮点
- 无实际代码可供评估。
## 📝 总体建议
本次提交的内容**无法进行有效的代码质量审查**,因为缺少实际的源代码变更。
1. **补充材料**:请重新提交包含实际代码逻辑变更的 Diff 内容。
2. **安全意识**:注意检查日志系统,避免将内部仓库地址、临时路径等基础设施信息打印到标准输出或公共日志中。
3. **环境检查**:核实生成日志的系统时间配置,确保时间戳准确,以免影响日志审计和故障排查。
请提供正确的代码变更内容后,我将很乐意为您进行详细的审查。
---
*此 Issue 由代码审查服务自动创建*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773713827
|
1773713827
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
757
|
85
|
2
|
4
|
|
0
|
[Design] 按省市统计设备月度新增上线数量报表
|
## 关联 Intent
[PR #1 — 按省市统计设备月度新增上线数量报表](https:// ## 关联 Intent
[PR #1 — 按省市统计设备月度新增上线数量报表](https://gitea.g-hi.com/laiqiaojie/jh-project/pulls/1)
---
## 审核顺序:PM 先确认原型,再由 Tech Lead Approve
---
## 第一步:PM 确认原型
**原型文件:**
(从仓库拉取后浏览器直接双击打开,可点击交互)
页面状态:
- 全国省份视图 — 月份筛选器 + 当月总数卡片 + 各省份横向柱状图
- 城市下钻视图 — 点击省份柱子进入,面包屑「全国 › 广东省」+ 城市排行
交互流程:点击省份柱子进入城市视图,点击面包屑「全国」或「返回全国」按钮返回。
**PM 审核 checklist:**
- [ ] 页面布局和组件符合预期?
- [ ] 交互路径(下钻、返回)正确?
- [ ] 数据定义(新增上线 = 首次激活,归属 = 门店地址)理解一致?
- [ ] 权限范围(超管看全国,运营商看自己区域)正确?
**PM 确认方式:在本 PR 评论「原型确认 ✅」即可,无需 Approve**
---
## 第二步:Tech Lead 审核(PM 确认后进行)
**数据链路:**
> 待确认:ahead_family_servers._shop_id 是否关联 ahead_yc_shop._id,Build 阶段查库验证。
**新增接口(新建控制器 DeviceOnlineReport.php):**
- POST /DeviceOnlineReport/getStats — 省份排行 + 当月总数
- POST /DeviceOnlineReport/getCityStats — 城市下钻
**权限:** agent_id 从 session 取,= 0 为超管不过滤
**安全:** month 正则校验、province_id intval()、agent_id 不信任前端
**Tech Lead 审核 checklist:**
- [ ] 数据链路正确?表关联关系有无误判?
- [ ] 接口设计合理?参数和响应格式符合规范?
- [ ] 权限处理方式正确?
- [ ] 安全检查要点覆盖?
- [ ] 前端方案可行(ECharts 按需引入)?
**Tech Lead Approve → Merge → 评审会议(里程碑内所有需求一起)→ Plan 阶段**...
|
1
|
0
|
1
|
1
|
0
|
|
0
|
1787824027
|
1789379887
|
1787829412
|
0
|
3
|
0
|
Edit
Delete
|
|
767
|
24
|
1
|
4
|
|
0
|
[后台管理] 按省市统计设备月度新增上线数量报表
|
## 关联 Issue
Close #4 — [后台] 按省市统计设备月度新增上线数量报表
(I ## 关联 Issue
Close #4 — [后台] 按省市统计设备月度新增上线数量报表
(Issue 位于另一仓库 laiqiaojie/jh-project,Gitea 跨仓库 `Close #N` 不会自动关联,merge 后需手动去 jh-project 更新 Issue 状态)
## 变更说明
- 新增 `DeviceOnlineStats` 控制器,暴露 `getProvinceStats`(按月+省份汇总新增上线设备数)、`getCityStats`(指定省份后按城市下钻)两个接口
- `Ahead_authenticate_log_model` 新增对应查询方法:联表 `ahead_authenticate_log` → `ahead_family_servers` → `ahead_yc_shop`,权限过滤复用现有 `Ahead_manage_user_model::check_is_agent()`(超管看全国,运营商强制按 `shop._agent_id` 过滤)
- 与 plan.md 的偏差:实现与计划一致;另外在 `/code-review` 中发现并修复了两个问题(均已回归验证):
- `month` 参数只校验格式未校验取值范围,如 `2026-00` 会被 `strtotime` 静默接受为上一年 12 月而不报错 → 正则改为 `(0[1-9]|1[0-2])` 严格校验月份范围
- 未绑定门店的设备(`room._shop_id` 为 0/未匹配)会被计入 `total` 但在 `GROUP BY shop._province` 时产生一个 province 为 null 的脏分组 → 加 `room._shop_id > 0` 过滤
## 测试说明
- 本地起 PHPStudy,登录后手动调用两个接口验证:正常月份返回 `{code:0, result:{total, list}}`;缺 `month`/非法月份(`2026-00`、`2026-13`、`0000-00`)均正确返回 400
- 用只读 SQL 直接查了云端开发库确认 JOIN 链路正确——本地测试库 `ahead_authenticate_log` 表只有 1 条 2021 年的历史数据,所以当月查询返回空列表是预期行为,不是 bug
- `tests/api/smoke.mjs` 里补充了对应的自动化用例,但该冒烟测试框架整体依赖另一个尚未推送到 `admin` 分支的提交(`694e34c`,属于另一项未提交的工作,不在本 Issue 范围内),为保持本 PR 改动范围干净,本次未包含测试框架文件,仅代码改动
- 需要 Tech Lead 重点看:`check_is_agent()` 权限过滤范围是否符合预期;`room._shop_id > 0` 过滤是否会误伤合法数据
## 产物归档
- Plan:[plan/device-online-stats-by-region-admin.md](https://gitea.g-hi.com/laiqiaojie/jh-project/src/branch/design/device-online-stats-by-region/plan/device-online-stats-by-region-admin.md)(位于 laiqiaojie/jh-project 仓库)
...
|
0
|
0
|
1
|
1
|
1
|
|
0
|
1788244598
|
1788246926
|
1788245236
|
0
|
0
|
0
|
Edit
Delete
|
|
769
|
24
|
2
|
4
|
|
0
|
[Web前端] 按省市统计设备月度新增上线数量报表
|
## 关联 Issue
Related to laiqiaojie/jh-project#5
Re ## 关联 Issue
Related to laiqiaojie/jh-project#5
Related to laiqiaojie/jh-project#8 — 按省市统计设备月度新增上线数量报表(Web 前端)
## 变更说明
- 新增 页面组件
- 省份视图:横向柱状图展示各省份当月新增上线设备数,点击下钻城市
- 城市视图:横向柱状图展示选定省份各城市分布,面包屑返回全国
- 使用 CSS 自定义 bar(不引入 ECharts),保持项目依赖不变
- 更新 :注册 device_online_stats 组件
- 更新 :新增 路由规则
- **修复** : 在父菜单 为空时拼出 双斜杠,导致路由匹配失败
- **修复** : 同一根因的双斜杠问题
## 测试说明
- 月份切换后图表重新加载
- 点击省份柱子进入城市视图,面包屑显示「全国 › XX省」
- 点击「全国」/「返回全国」返回省份视图
- 无数据月份显示空状态
- 点击「数据」顶部菜单 → 侧栏「设备上线分布」→ 组件正常渲染(已端到端验证)
## DB 变更(需手动执行)
## 产物归档
- Plan:(在 jh-project 仓库)
- 设计文档:
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1788247154
|
1789379914
|
1789377273
|
0
|
3
|
0
|
Edit
Delete
|
|
856
|
83
|
13
|
5
|
|
0
|
docs(integration): 蓝图补服务级对接矩阵,按 04 CHG-061 订正镜像与 H docs(integration): 蓝图补服务级对接矩阵,按 04 CHG-061 订正镜像与 H1/H5/H21...
|
## 内容
- 对接蓝图新增 §2.5 服务级对接矩阵:HR 与 02/03/04 共 20 个对 ## 内容
- 对接蓝图新增 §2.5 服务级对接矩阵:HR 与 02/03/04 共 20 个对接点,HR 侧写到 file:line,并标明 dev 实际跑通过的只有 3 条:事实写入、Scope 来源、OS 装载产品包。
- 按 04 `e50aad416`(CHG-061)订正:
- 应用清单镜像已合到 0.19.6,工作区镜像剩 5 处差异(H1);
- 4 条 HR 事实已在 Catalog 登记为 draft,H5 的 Catalog 半边完成;
- Scope 改按 Catalog producer 接受 `juhai.hr` 组织事实,H21 部分关闭,剩载荷约束与投影器现场摄取。
- 新增缺口:
- H25:OS 清单契约仍强制双后端,HR 清单里的 Fastify 声明是占位,不能单方删除;
- H26:门店归属事实没有生产者;
- H27:IdP 令牌 `aud` 是生成的 UUID,不是 `juhai.hr`;
- H28:数字智能层自带一份 0.19.5 副本。
- §2.5 补本机现场读数(2026-10-28 00:35 UTC 前后):
- HR API 未运行;
- 04 runtime 开了事实受理但缺 `FACT_APPLICATION_BINDINGS_JSON`,IdP 签发的机器令牌会被拒;
- Scope 来源 65281 不可达;
- OS 0.19.6 分支只在本机、未推 Gitea。
- 已确认三项:
- client-fact rc.3 与 rc.8 摘要实现逐行相同(H7 只剩发布);
- EES 运行时以 `READ_ONLY_PROJECTION` 拒绝已撤回人域命令;
- Scope 证据 `owner: juhai.hr` 由 04 按 HR 签名组装。
- H22:本机开发基座容器已于 00:40 UTC 重建为只绑回环,数据卷沿用。
- **CI 修复**(`a174e46`):本 PR 首次运行 26581 停在「Build locked OS compatibility sources」,原因是兼容锁指向 D3 后的 OS `1b8ae0d7`,该步却仍写死 `--filter=api-fastify`。
- 现改为经 `scripts/lib/os-host-backends.mjs` 读锁定 OS 的 `backend-profile.json` 决定宿主范围;
- `check:docs-truth` 新增 `ci-os-host-backends` 断言防回潮,对旧工作流负向判红。
- 同步落地状态:CLAUDE.md G23 / G14、两份 CHG 草稿、人域对照表。
- 已合入 `main` `46da429`:CLAUDE.md 冲突以 main 为准,只叠加本分支的 H21 订正。
## 验证(本地,PR 头 `25aecd7`,报告绑定 `a174e46`、clean)
- 通过:`check:runtime` 462/462(地板 452)、`check:ui` 46/46。用的是一次性 PostgreSQL 16 / Redis 7,只绑回环,跑完已删。
- 静态链如实停在 O22(OS main 尚无 0.19.6)。其后 governance-docs、docs-truth、lint、typecheck 通过。`check:governance` 只剩 O22,`reportProvenanceViolations` 0。
- CI 构建步骤:本地覆盖 5 种情形(nestjs-production、D3 前双宿主、缺 profile、非法 profile、profile 与 Fastify 冲突),以及锁定 OS 快照上的 turbo dry-run。
- 远端 CI 见本页检查结果。
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793145750
|
1793194416
|
0
|
0
|
1
|
0
|
Edit
Delete
|
|
870
|
116
|
4
|
5
|
|
0
|
docs(release): R1 运行器与分支保护变更规格(rc.8 合规发布前置)
|
## 内容
给现役 Gitea / runner 维护者的逐项变更规格,作为 `@juhai/ ## 内容
给现役 Gitea / runner 维护者的逐项变更规格,作为 `@juhai/*` 发布列车 `1.0.0-rc.8` 走合规通道(同源 main 合格 Platform CI → clean tag → `.gitea/workflows/publish-packages.yml`)的前置(R1)。
- 新增 `docs/R1运行器与分支保护变更规格-2026-10-28.md`:
1. Gitea 升到含上游 PR #37119(ActionRunAttempt)的正式版,`MAX_RERUN_ATTEMPTS ≥ 2`,升级前一致性备份与隔离恢复;
2. act_runner 升到下发真实 `GITHUB_RUN_ATTEMPT` 的版本,保持 `ubuntu-latest`、Docker、github.com 访问与 fork Secret 隔离;
3. main 保护 9 个字段逐一对应 `governance/lib/release-ci.mjs` 第 30–31 行;
4. 交付后验收顺序与 rc.8 发布步骤。
- `docs/正式发布材料待办-2026-10-10.md` 顶部加指针。
## 依据(2026-10-28 PDT 只读实测)
- main 最近 15 次 Platform run 全部 failure,均在第一步「Candidate tag」因 `GITHUB_RUN_ATTEMPT` 缺失失败关闭(例:run 26610 @ `485db734d`,`run_attempt=0`,`attempts/1` → 404);runner act_runner v0.2.13;版本 API 仅返回 `development`。
- `branch_protections` 为空列表。
## 影响提示
启用 main 保护后禁止直推(10-21 起 main 首父链 492 个提交中约 435 个为直推),所有会话须改走 PR,且每个 PR 要通过完整 Platform CI;规格要求先有一个 PR 全绿再落规则,避免锁死 main。
## 边界
只改文档;发布门不变,未恢复本地旁路(G-12),未打 tag、未写 Registry、未改服务。`pnpm check:doc-claims` 通过。
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793191041
|
1793191041
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
808
|
124
|
2
|
5
|
|
0
|
feat: add review invitation guardrail to manager c feat: add review invitation guardrail to manager cockpit (SCO11)...
|
## 内容
- **SCO11 评价邀请话术护栏**:五类演示规则(诱导好评、利益换评价、删改差评 ## 内容
- **SCO11 评价邀请话术护栏**:五类演示规则(诱导好评、利益换评价、删改差评、截图回传、施压),先规整全角、空格、符号再匹配防绕过;总部独立审核,审核时复查;邀请任务只能用已审核版本完成
- **CLN04 遗失物品**(远程运营台 → 事件与售后)
- 保洁只能从本人进行中的保洁单登记,范围限清洁可见区域
- 运营确认保管,贵重物品入加锁柜,保管链留痕
- 凭同包厢订单或人工依据认领,只存打码电话
- 移交本人、代领(证件除外)、警方或平台;到期处置不能由登记人操作
- **MKT01 活动计划**:客群、门店、责任人、周期、停止条件在批准前必填(草稿可不完整,兼容持久化服务的创建调用);周期外或非适用门店拒绝预约与投放
- **测试证据**:11 项只有模块、没有测试的需求补 45 条测试;GRP06 拼团无代码,改判为未实现;各项"已证明 / 仍缺"见 `docs/acceptance-unimplemented.md`
- **任务包状态**:新增 `ops/sync-package-status.mjs`,从验收矩阵算出状态,写入开发计划追踪和任务包矩阵(文档 22),带 `--check`
## 验证
- 原型测试 255/256:唯一失败是缺 `dist` 构建产物的打包测试
- 服务端可独立运行的测试 363/366:3 个失败是本工作区未编译框架 `dist`,与本改动无关
- `node ops/check-acceptance.mjs --check`、`node ops/sync-package-status.mjs --check`、`python3 scripts/verify-development-plan.py` 均通过
- 矩阵:127 项部分实现、2 项未实现(GAM05、GRP06)、83 项外部依赖、0 签认
- 浏览器验证:话术拦截与审核、遗失物品全流程、活动计划缺项拦截与批准
## 注意
- 话术规则、保管期 30 天、停止条件均为演示参数,待 D35 和门店政策确认
- 合并后如果再改源文件或测试:先运行 `node ops/sync-package-status.mjs`,再运行 `node ops/check-acceptance.mjs` 重新生成矩阵
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1791470731
|
1791510657
|
1791510657
|
0
|
0
|
0
|
Edit
Delete
|
|
773
|
85
|
8
|
4
|
|
0
|
按省市统计设备月度新增上线数量报表
|
## 功能概述
在 PC 管理后台「数据报表」模块,按省市统计各 KTV 设备每月新增上线数量,支 ## 功能概述
在 PC 管理后台「数据报表」模块,按省市统计各 KTV 设备每月新增上线数量,支持全国省份分布视图 + 城市下钻。
## 完整产物链路
| 阶段 | 产物 | 链接 | 状态 |
|------|------|------|------|
| 需求意图 | Intent 文档 | PR #1 | ✅ 已合并 |
| 设计原型 | Design 文档 | PR #3 | ✅ 已合并 |
| 技术方案(后台) | Plan 文档 | PR #6 | ✅ 已合并 |
| 开发实现(后台) | admin 仓库代码 | admin PR #1 | ✅ 已合并 |
| 开发实现(Web前端) | admin 仓库代码 | admin PR #2 | ✅ 已合并 |
| 测试验收 | — | 本 Issue | 🧪 待测试 |
## 验收标准
- 后台 API 返回省份/城市维度的月度新增上线设备数,分页正确
- PC 管理后台菜单「数据报表 > 设备上线分布」可正常点击进入
- 省份视图正常渲染(横向柱状图),点击下钻到城市视图
- 城市视图面包屑可返回全国
- 月份切换后数据刷新正确
- 无数据月份显示空状态,不报错...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1789379887
|
1789380277
|
0
|
0
|
2
|
0
|
Edit
Delete
|
|
851
|
83
|
8
|
5
|
|
0
|
feat(integration): Web/开发基座只绑回环、D-12 packages/* 受控 feat(integration): Web/开发基座只绑回环、D-12 packages/* 受控通道与 H20 出站事实注册表...
|
## 改了什么
本 PR 有三部分;已合入 main 的 PR #9(D-10 API 回环)与 ## 改了什么
本 PR 有三部分;已合入 main 的 PR #9(D-10 API 回环)与 PR #10(H23 根与 apps/* 依赖)均已并入本分支。
### 1. Web / 开发基座只绑回环(蓝图 §4.9)
- Web `next dev` / `next start` 与 UI 验收的 `next start` 加 `-H 127.0.0.1`;开发基座 `apps/api-nestjs/docker-compose.yml` 的 PostgreSQL / Redis 端口只发布到回环(此前发布到全部网卡,PG 口令 `app`、Redis 无口令)。已在运行的基座容器须 `docker compose up -d` 重建才生效。
- 在 #9 的 `scripts/lib/listen-host-guard.mjs` 内新增 `exposureViolations`(同一守卫),接入 `check:dual-backend`;单测 9/9。
- 文档:owner-matrix 去掉 D3 前「双后端 / Fastify」措辞;Scope 来源接入文档改正 IdentityLink 归属(enterprise-idp 拥有)。
### 2. D-12:`packages/*` 受控升级通道(关 H24,蓝图 §4.10)
- D3 保护此前逐字节冻结全部 `packages/*`(215 个受保护文件),台账只放行产品目录。现在同一台账 `docs/post-d3-product-updates.json` 增加 `packageUpdates`:编号 `PKG-NNN` 递增,须写来源与理由,仅限 D3 冻结清单内的 `packages/*` 文件,逐文件从冻结 hash 链式放行;任一条目不合规即整体不放行。
- `check:d3` 30/30:新增登记后放行、未登记 / 台账缺失判红,以及断链、前后相同、非 hash、缺理由、缺来源、编号格式错、编号不递增、空文件、包外路径、非冻结文件、字段类型错均判红。变异探针:删掉「理由与来源」或「链式校验」规则,测试各判红 1 项,已还原。
### 3. H20:出站事实注册表(蓝图 §4.10)
- contracts 新增 `ENTERPRISE_FACT_ENVELOPE_SCHEMAS`(`factType@schemaVersion` → 信封 Schema,当前只有 `EmploymentTransferred@1`)与 `EnterpriseFactTypeUnregisteredError`;`buildEnterpriseFactEnvelope` 按注册表分派。EmploymentTransferred v1 的 Schema 不变。
- relay 去掉全部类型强转;未登记类型或版本记 `FACT_TYPE_UNREGISTERED`,立即进死信、不调用 Publisher(部署补登记后可经既有重放恢复)。
- contracts 两个文件经台账 **PKG-001** 放行(登记前 `check:d3` 对这两个文件判红)。
- `check:app-manifest` M6 新增注册表与 `facts.owns` 双向一致检查(3 个负例)。
- 测试:contracts 新增 4 例;新文件 `apps/api-nestjs/test/enterprise-fact-relay-registry.test.ts` 真实 DB 3 例。原 `enterprise-fact-relay.test.ts` 受 D3 冻结未改,12 例仍全绿。
## 验证(本地,2026-10-27,干净提交 `effed90`)
- `check:runtime` 462/462、`check:ui` 46/46(一次性 PostgreSQL / Redis,跑完删除)
- typecheck、lint、`check:d3`、`check:dual-backend`、`check:app-manifest`、`check:docs-truth`、`check:governance-docs`、`check:migrations`、`check:database-rls` 等静态门禁通过
- `check:governance` 剩 2 项,与 main 相同:O22(D-8,等 OS 写入授权)、`osProductReleaseTargetRemotes` 2→1(D-9)
## 未覆盖
- 远端 CI 未查验(本机无 Gitea API 令牌)
- 新事实(`OrganizationUnitMoved` 等)仍待 Catalog 登记;H21(Scope 消费端写死来源应用)仍开放
- `packages/*` 内的 vitest 2.1.9 与 `client-fact` rc.3 现可经台账升级,本 PR 未做
- 00 框架同一监听默认值(他层仓);README 第 3 步提到的 `.env.example` 不存在(既有不一致)
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793111425
|
1793124194
|
0
|
0
|
3
|
0
|
Edit
Delete
|
|
849
|
83
|
6
|
5
|
|
0
|
feat(integration): AI 人事系统与企业智能OS各层对接——0.19.6、OS D feat(integration): AI 人事系统与企业智能OS各层对接——0.19.6、OS D3 单宿主验收链、OS 装载器注册证据与各层对接材料...
|
## 概要
把 AI 人事系统(`juhai.hr`)与企业智能OS各层(02 业务中台 / ## 概要
把 AI 人事系统(`juhai.hr`)与企业智能OS各层(02 业务中台 / 03 数字员工 OS / 04 技术中台 / 00 工程框架)的对接推进到「HR 侧能独立完成的都已完成」,其余项给出材料并列为待裁。完整记录见 `docs/domain/enterprise-control-plane-integration-blueprint-2026-10-27.md` §4–§5(取代 09-26 版)。
## 主要改动
- **产品真源收回 HR 仓,发布 0.19.6**:回灌 OS 已登记的 0.19.5;逐字节收录 OS 已执行的整表授权迁移 `20261010000000`,并以 `20261027000000` 前向收窄回 relay 两张表的精确列授权。产品迁移治理按全序列模拟最终授权,缺少或绕过收窄都会判红。
- **D3 受控升级台账**:`docs/post-d3-product-updates.json` 让 D3 冻结的产品字节和 `readiness.json` 只能按链放行(readiness 自干净复验值起链),C4 历史锁不变。
- **验收链跟进 OS D3(单 NestJS)**:宿主范围以目标 OS `backend-profile.json` 为唯一来源(`scripts/lib/os-host-backends.mjs`),拒绝加载残留的 Fastify 构建物;兼容锁对齐 OS `1b8ae0d7`(contracts 3.0.0);D-6 按锚定换算表把 12 项按后端累加的宿主证据地板换算为单后端。
- **注册证据改接 OS 装载器(D-5)**:lock v2 目标的 O20/O24/O26 只认 `pnpm audit:os-product:loader` 的只读证据(plan / apply 预演 / reconcile、4 个负向探针、目标提交回滚测试、目标写入 0);HR v1 写入器只留给 lock v1 目标。
- **O27 与发布证据**:按锁定后端数与注册路径取证;O27 转绿。
- **各层对接材料(草案,未改他层仓)**:人域对照表、事实登记 CHG 草案、应用清单镜像同步材料、权限决策答复草稿、框架同步差额评估、给 OS Owner 的交接说明。
- **仓内不一致修正**:app-manifest 仓库路径用工作区别名 `企业领域应用层/AI人事系统`;`authoritativeEntities` 去掉 IdentityLink(enterprise-idp 拥有);补 `ees.domain.thing` 依赖;`governance/framework-consumer.json` 地板改为 D3 换算后的 471/452。
- 已合并当前 `main`(含 CI 与 O18 显式 OS 根目录改动),冲突处两边规则都保留。
## 验证(本地工作区)
- 静态子门禁、`check:d3`、`check:app-manifest`、lint、typecheck 通过。
- HR 全量验收在合并后 HEAD 上通过:`check:runtime` 452/452、`check:ui` 46/46(一次性 PostgreSQL/Redis,跑完已删)。
- OS 产品层证据在 OS `1b8ae0d7` 快照上:产品迁移 17/17、ProductSqlRuntime、宿主运行 1/1 后端、浏览器 1/1 后端、升级审计 READY、装载器审计 passed。真实 OS 仓 0 处改动。
- 远端 CI 结果以本 PR 的运行为准,本地结果不外推。
## 剩余两项回退(`check:governance`)
1. **`osProductLayerViolations` 0 → 1(O22)**:OS main 尚不含 0.19.6。需显式授权(`OS_PRODUCT_REGISTRATION_APPLY=1`)用 OS `product:apply` 写入并进入 OS main,再复跑装载器 / 升级 / 发布审计。
2. **`osProductReleaseTargetRemotes` 2 → 1**:快照只有一个本机 remote(09-23 的证据跑在带 Gitea/GitHub 两个远端的真实 OS 上)。
本地不带 `DIGITAL_EMPLOYEE_OS_ROOT` 跑时,O18 也会按 main 上的新规则提示需要显式 OS 根目录;CI 已传入该变量。
## 待裁决
| 编号 | 问题 | 建议 |
| --- | --- | --- |
| D-2 | Scope→HR 来源授权是否续期 | 续期是 01 BA-01 首片(HR→Scope→OS 组织视图复验)的前提 |
| D-3 | 如何答复 04「HR 先接权限决策」 | 维持 F1 权限自治;可做不切真源的只读决策对照实验(草稿见 `docs/domain/reply-draft-04-permission-decision-2026-10-27.md`) |
| D-4 | F2(平台模式试点)是否重议 | 前置条件不成立:OS 未用叠加仓、未接 platform 端口模式;暂不重议 |
| D-7 | O27「拒绝 4 个目标敏感投影」的历史语义 | 定性为历史决定,审计区分观测命中与待拒绝面 |
| D-8 | O22 写入前是否区分「目标落后、准备发布」 | 维持现规则,不放宽;随写入授权一起解 |
| D-9 | `osProductReleaseTargetRemotes` 2→1 | 发布审计改为只读读取真实 OS 的远端,不设为不计棘轮 |
| D-10 | HR API 默认监听 `0.0.0.0` | 默认改为 `127.0.0.1`、需暴露时显式设 `HOST`,同步 00 框架;改前实测 04 容器可达 |
| D-11 | 框架 0.27.0 安全基线差额 | 单独一片升级 next / vitest / Capacitor 与平台 rc 包;按框架 `COMPATIBILITY.md:36` 登记 Zod 3 残余风险 |
另有新登记缺口:H20 relay 写死事实类型(扩事实前须先改)、H21 04 Scope 消费端写死组织事实来源(需 04 改)。
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793102425
|
1793111425
|
1793103977
|
0
|
0
|
0
|
Edit
Delete
|
|
122
|
6
|
4
|
5
|
|
0
|
fix(governance): Wave 4 解封 + capability-graph HIGH fix(governance): Wave 4 解封 + capability-graph HIGH 清零 + vitest 7 项真回归...
|
## 模块治理摘要
- 模块:`capability-graph` / `ai-review-qu ## 模块治理摘要
- 模块:`capability-graph` / `ai-review-queue` / `campaigns` / `marketing` / `quotes`
- 战区:Wave 4(AI/Process 嵌入式闭环)+ harness 红线收口
- lane:模块专项通过
- 阶段:W4-T03/T06 解封 → Wave 4 收口
- 分类:`模块专项通过`
- 结论:W4-T06 状态 `PARTIAL → PASS`;harness `HIGH 3 → 0`;18 个原 stale baseline 失败文件 `373/373` 全绿
- 战役卡:W4-T03(AI Review Queue 真实 DB 验收)
- 验收卡:`docs/governance/ai-process-evidence-bundle-2026-05-10.md`
- runner evidence bundle:commits `566b7fe1d` + `921a6ce55`
## 关键变更
### 1. W4-T03 真实 DB 验收解封
- `tests/api/ai-review-queue.test.ts:204` + `:262` 两处 `listRes.body.data.data.some(...)` → `listRes.body.data.some(...)`,对齐 Governance 3.0 扁平 `paginated()` 契约 `{ data: [], meta: { pagination } }`
- 真实 DB(`juhi-postgres-test:5433` / `juhi-redis-test:6380`)复跑 4/4 全绿(10.47s)
- 覆盖:未认证 401 + 白名单守卫 + list/stats/claim/approve/reject 真实链路 + execution guard
### 2. capability-graph HIGH 3 → 0
- 新增 `HR_TRAINING_EVENTS.COMPETENCY_ASSESSMENT_CREATED / _DERIVED` 两个 event 常量
- `kafka.ts` 新增 `hr_competency_assessment → HR_EVENTS` topic 映射
- `assessByUser` (`:95`) + `deriveBehaviorBasedAssessment` (`:556`) 写入包进 `prisma.$transaction({tx => create + publishEvent(tx)})`,outbox 原子化
- `event-publishing-audit` capability-graph 行:`none=1` → `full=1`
### 3. vitest 7 项真回归修复
背景:`reports/vitest.backend.latest.json` 是 2026-02-18 别的开发机产物,95 failures 绝大多数是 stale。复跑 18 个失败文件后只有 7 项真回归。
| 文件 | 修复 |
|---|---|
| `marketing.service.test.ts` 完成执行中活动 | 补第 4 个 findFirst onceValue(assertTransition / update 状态校验 / ensureExistsById / refresh 四次调用) |
| `quote.service.test.ts` 创建报价单 | biz-code mock 补 `createWithBizCodeRetry` / `generateBizCodes` / `generateBizId` |
| `campaign.service.enhanced.test.ts` 5+6 项 | beforeEach 加 `findFirst/updateMany/findFirstOrThrow.mockReset()` 防 sticky 串扰;mock 补 `findFirstOrThrow`;8 处状态机生命周期/事件发布/取消测试补第 4 个 findFirst onceValue;execute 测试将第 3 个 findFirst onceValue 改为 findFirstOrThrow onceValue |
剥离 stale 后真实回归 7 项,但 `mockReset()` 同时暴露 6 项原本被 sticky 状态掩盖的同类问题,一并修复(共 13 处 onceValue 补齐)。
## 验证命令
| 命令 | 结果 | 备注 |
| --- | --- | --- |
| `npx vitest run --workspace vitest.workspace.ts --project api tests/api/ai-review-queue.test.ts` | ✅ 4/4 passed (10.47s) | 真实 DB |
| `npx vitest run --workspace vitest.workspace.ts --project unit src/modules/campaigns/campaign.service.enhanced.test.ts src/modules/marketing/marketing.service.test.ts tests/unit/services/quote.service.test.ts` | ✅ 3 files / 71+25 tests passed | unit |
| `npx vitest run --project unit` (18 个 stale 失败文件全集) | ✅ 18 files / 373 tests passed | unit 回归 |
| `npx tsx scripts/audit-prisma-zod-contract.ts` | ✅ PASS (length=0 / required=0 / enum=0) | |
| `npx tsx scripts/audit-event-publishing.ts` | ✅ 99.01% 覆盖率,capability-graph 全覆盖 | |
| `pnpm harness report` | ✅ CRITICAL=0 / HIGH=0 / MEDIUM=8 / total=6028 | HIGH 从 3 降 0 |
| pre-commit Fast Gate (6 项) + Smart Gate (3 项���后端 type-check) | ✅ 全 PASS | NODE_OPTIONS=12G |
## 审计命中
- `audit:tenant`:未跑(变更未触及租户中间件)
- `audit:events`:✅ 0 finding(capability-graph 修复后从 7 降 0)
- `audit:route-contract`:未跑(变更未触及路由层)
- `audit:docs-single-source`:未跑(仅 governance 增量;CLAUDE/AGENTS/README/GEMINI 同步待后续 owner PR)
- `audit:prisma-zod-contract`:✅ 0 finding
- `audit:state-machines`:✅ 0 finding
## 前端 / 页面验收
- canonical 入口:`/ai-agents/review-queue`(未变更,本 PR 只动 API 测试 + 后端)
- smoke / auth-scope:未跑(无前端代码改动)
- alias / compat 回归:N/A
## 残余风险
- 风险 1:harness 仍有 `MEDIUM=8`,全部来自既有 baseline 数据(迁移安全检查 5215 LOW 分类边界 / 业务时间流分析 117 INFO / 类型债务追踪 41),非本 PR 引入。CI 治理硬门禁(`HIGH+CRITICAL+MEDIUM > 0`)会阻塞,需后续 owner 决定是否豁免或继续治理。
- 风险 2:full unit suite 仍剩 28 个 *其他* test 文件 / 135 failures(dimension-transform / social-auto-replies / opportunity.machine.test label / rma / content-* 等),均与本 PR 无关,是更老的 pre-existing 失败。建议作为独立后续 PR 处理。
- 风险 3:本 PR 修复的 `mockReset()` 模式应推广到其他使用 `prisma.$transaction` 的服务单测——下次治理可重点扫描。
## 文档同步
- [ ] `CLAUDE.md`
- [ ] `AGENTS.md`
- [ ] `README.md`
- [ ] `GEMINI.md`
- [x] 治理经验库:`docs/governance/ai-process-evidence-bundle-2026-05-10.md` 已升 PASS + 补 Real DB Suite + Test Contract Fix 章节
## 口径确认
- [x] 本 PR 只宣称模块级通过,不把模块结果外推为工作区全绿
- [x] 若存在工作区漂移,已明确标记且未误记到当前模块(135 项其他失败已列入残余风险)
- [x] 若存在热修 lane,已与全域补证据 / build-out lane 隔离
---
> ⚠️ GitHub 镜像 PR #51 因 Actions 账单问题阻塞,故改在 Gitea 创建本 PR 作为主验收入口。
🤖 Generated with Claude Code...
|
0
|
0
|
1
|
1
|
1
|
|
0
|
1778679717
|
1779328857
|
1778891027
|
0
|
0
|
0
|
Edit
Delete
|
|
94
|
6
|
2
|
5
|
|
0
|
Encode DB credentials in synced local env URLs
|
## 模块治理摘要
- 模块:
- 战区:
- lane:
- 阶段:
- 分类:`模 ## 模块治理摘要
- 模块:
- 战区:
- lane:
- 阶段:
- 分类:`模块专项通过 / 工作区漂移 / 阻断`
- 结论:
- 战役卡:
- 验收卡:
- runner evidence bundle:
## 验证命令
| 命令 | 结果 | 备注 |
| --- | --- | --- |
| | | |
| | | |
| | | |
## 审计命中
- `audit:tenant`:
- `audit:events`:
- `audit:route-contract`:
- `audit:docs-single-source`:
- 如未跑全量,请说明原因:
## 前端 / 页面验收
- canonical 入口:
- smoke / auth-scope:
- alias / compat 回归:
## 残余风险
- 风险 1:
- 风险 2:
## 文档同步
- [ ] `CLAUDE.md`
- [ ] `AGENTS.md`
- [ ] `README.md`
- [ ] `GEMINI.md`
- [ ] 治理经验库 / 专项记录
## 口径确认
- [ ] 本 PR 只宣称模块级通过,不把模块结果外推为工作区全绿
- [ ] 若存在工作区漂移,已明确标记且未误记到当前模块
- [ ] 若存在热修 lane,已与全域补证据 / build-out lane 隔离
...
|
0
|
0
|
0
|
1
|
5
|
|
0
|
1777515695
|
1779328857
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
96
|
6
|
3
|
5
|
|
0
|
claude/fervent-kirch-e2a254 -> origin/claude/fe claude/fervent-kirch-e2a254 -> origin/claude/fervent-kirch-e2a254...
|
## 模块治理摘要
- 模块:
- 战区:
- lane:
- 阶段:
- 分类:`模 ## 模块治理摘要
- 模块:
- 战区:
- lane:
- 阶段:
- 分类:`模块专项通过 / 工作区漂移 / 阻断`
- 结论:
- 战役卡:
- 验收卡:
- runner evidence bundle:
## 验证命令
| 命令 | 结果 | 备注 |
| --- | --- | --- |
| | | |
| | | |
| | | |
## 审计命中
- `audit:tenant`:
- `audit:events`:
- `audit:route-contract`:
- `audit:docs-single-source`:
- 如未跑全量,请说明原因:
## 前端 / 页面验收
- canonical 入口:
- smoke / auth-scope:
- alias / compat 回归:
## 残余风险
- 风险 1:
- 风险 2:
## 文档同步
- [ ] `CLAUDE.md`
- [ ] `AGENTS.md`
- [ ] `README.md`
- [ ] `GEMINI.md`
- [ ] 治理经验库 / 专项记录
## 口径确认
- [ ] 本 PR 只宣称模块级通过,不把模块结果外推为工作区全绿
- [ ] 若存在工作区漂移,已明确标记且未误记到当前模块
- [ ] 若存在热修 lane,已与全域补证据 / build-out lane 隔离
...
|
0
|
0
|
1
|
1
|
1
|
|
0
|
1777597467
|
1779328857
|
1778625303
|
0
|
0
|
0
|
Edit
Delete
|
|
131
|
6
|
6
|
5
|
|
0
|
Fix CRUD visible runner timeouts and permissions
|
## 模块治理摘要
- 模块:
- 战区:
- lane:
- 阶段:
- 分类:`模 ## 模块治理摘要
- 模块:
- 战区:
- lane:
- 阶段:
- 分类:`模块专项通过 / 工作区漂移 / 阻断`
- 结论:
- 战役卡:
- 验收卡:
- runner evidence bundle:
## 验证命令
| 命令 | 结果 | 备注 |
| --- | --- | --- |
| | | |
| | | |
| | | |
## 审计命中
- `audit:tenant`:
- `audit:events`:
- `audit:route-contract`:
- `audit:docs-single-source`:
- 如未跑全量,请说明原因:
## 前端 / 页面验收
- canonical 入口:
- smoke / auth-scope:
- alias / compat 回归:
## 残余风险
- 风险 1:
- 风险 2:
## 文档同步
- [ ] `CLAUDE.md`
- [ ] `AGENTS.md`
- [ ] `README.md`
- [ ] `GEMINI.md`
- [ ] 治理经验库 / 专项记录
## 口径确认
- [ ] 本 PR 只宣称模块级通过,不把模块结果外推为工作区全绿
- [ ] 若存在工作区漂移,已明确标记且未误记到当前模块
- [ ] 若存在热修 lane,已与全域补证据 / build-out lane 隔离
...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1778907435
|
1778907436
|
0
|
0
|
0
|
0
|
Edit
Delete
|