|
790
|
116
|
1
|
5
|
|
0
|
fix(contracts): 注册中心规则补失败关闭与摘要规范化——三个入口此前抛 TypeErr fix(contracts): 注册中心规则补失败关闭与摘要规范化——三个入口此前抛 TypeError，同一摘要的两种写法此前分叉成两个复核键...
|
蓝图 docs/domain/registry-snapshot-blueprint.md 从迁入起 蓝图 docs/domain/registry-snapshot-blueprint.md 从迁入起就写着「失败关闭」与
「review key 仅用于人工复核幂等」两条不变量，实测三处不成立。本次按蓝图自己的口径补齐，
改动只在 contracts/src/domain/application-contract-registry/：不新增写入点、不新增宿主、
不动 Catalog 与 Schema，每条输出仍固定 activationAllowed=false / registryWriteAllowed=false /
humanApprovalRequired=true。
1. 失败关闭：assessRegistrySnapshot / assessRegistryTransition /
reviewCandidateSnapshotUpgrade 对畸形输入抛 TypeError（快照缺字段、now 不是 Date、
decisions 整个缺失都会抛），四个入口里只有 LocalManifestAdmissionPlanner.plan 是
失败关闭的。现四个入口一致返回 REJECT + 原因码。这不是洁癖：check-fixtures 把抛出当
配置错误处理，实测一次 TypeError 会让整套 15 例逐例判定塌成一条 configuration 失败。
2. 十六进制引用规范化：digest / commit 的正则带 /i 收大小写两种写法，比较与入键却
大小写敏感——同一摘要写成大写会得到另一个 reviewIdempotencyKey，同版本大小写翻转被判成
SAME_VERSION_DIGEST_CHANGED，大写回滚 pin 被判成 ROLLBACK_PIN_MUST_MATCH_CURRENT。
现一律规范化为小写后再比较、再入键，ACCEPT 回吐规范化后的值。
3. 版本文法收敛：一个模块三套（快照流与清单准入用 \d+\.\d+\.\d+，升级复核用严格 SemVer），
新增 registry-primitives.ts 作单一来源，含预发布序；DEC-009—012 此前在两份文件各写一份，
一并收敛。snapshot-upgrade-review.ts 因此净减约 60 行重复实现。
两处语义变化（蓝图新增一节已如实登记）：
- 收紧：1.02.0 这类前导零版本此前被放行，现在拒；assessRegistrySnapshot 另开始拒 latest
这类可变标签——此前它能过体检却必然过不了迁移规则。
- 放宽：1.0.0-rc.1 这类预发布版本此前被快照流整条挡在外面，现在按 SemVer 序参与迁移。
平台自己发的就是 1.0.0-rc.N，升级复核那一侧本来就这么判。风险面有限：本模块所有出口
都不激活、不写 Registry，多进人工复核队列不等于多放行制品。
证据（本分支隔离工作树，clean）：注册中心单测 44/44（原 34 + 加固 10）；
contracts check:local 302/302；check:fixtures 九个契约域套件 100 例 0 失败，
其中本套件 15 例（正 4 / 反 11，原 8 例）。篡改必红实测两次：sameHexRef 退回大小写敏感
→ 单测 1 红 + 夹具 P03 红；去掉快照入口守卫 → 整套塌成 0 例 1 配置失败。
仍为仓内候选 sdk_e2，不代表正式 Registry、Snapshot 发布或跨仓 Required Check 已上线。
清单登记的 C01.04 / C01.06 是「接线」型缺口且前置 Q01 运行宿主未裁，本次未动。
reports/fixtures.latest.json 未回绑：它是 17 套件的聚合报告，只跑 9 个套件去覆盖会缩小结论面，
应在整批（含并行会话的 public-file / IM / 跨域流程改动）落定后统一重跑回绑。
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1789742918
|
1790248458
|
1789742925
|
0
|
0
|
0
|
Edit
Delete
|
|
21
|
2
|
21
|
1
|
|
0
|
11111
|
测试
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1773910676
|
1773910796
|
1773910698
|
0
|
0
|
0
|
Edit
Delete
|
|
743
|
57
|
3
|
5
|
|
0
|
feat(script): 接入大语言模型——歌词/文案 → 拍摄脚本 → 角色确认 → 定格图
|
本仓此前零 LLM 接入（全仓只有 DashScope 的图像调用），「歌词成图」屏是纯前端
原型 本仓此前零 LLM 接入（全仓只有 DashScope 的图像调用），「歌词成图」屏是纯前端
原型（procedural.ts 画色块 + setTimeout 假装推理，不落库不接后端）。本轮把这条
动线升级为真链路 + 真持久化。
契约单源（packages/contracts/src/script.ts）
- 角色确认状态机 scriptRoleMachine、脚本状态机 shootingScriptMachine
- 景别/运镜用枚举而非自由文本：LLM 一旦自由发挥，下游 prompt 与统计就没有稳定口径
- scriptDraftSchema 是 LLM 输出的唯一判据；prompt 构造器单源，两后端禁止各拼一份
- evaluateRoleReadiness：本域核心不变量，双后端硬拦截 + 前端同源即时反馈
- 前端私有的 findBannedHits 上收为 contracts 的 findBannedWords（后端也要逐镜标记，
两份并行就是双真源）
LLM 不是权威
- sidecar /v1/script 只保证「返回长得像脚本的 JSON」，两个后端各自再 parse 一次才落库
- 模型引用未声明角色键 → 422 SCRIPT_ROLE_KEYS_DANGLING（幻觉护栏）
- 连续给不出合法 JSON → 502 LLM_OUTPUT_UNPARSEABLE，绝不返回占位脚本
- parse_json_object 只剥围栏、截首尾花括号，不做字段猜测或补齐
角色必须先确认才允许出定格图
- 判据 = status CONFIRMED 且有 referenceAssetId（没有参考图就没有一致性锚点）
- CONFIRMED --reset--> DRAFT 会解除参考图绑定：人确认的是那一张图，不是那个名字
- 出图时按 roleKeys 顺序喂参考图，与指令里「第 N 张参考图」编号严格对齐
写链纪律与物料链路同构：tenant 全覆盖、updateMany 带状态前置条件（0 行即 409）、
终态与 outbox 同 tx、能力不可用 → job BLOCKED + 诚实 reason，禁止静默降级。
品牌准绳只逐镜标记不拒绝整份脚本（脚本属创作内容），硬拦截点在出定格图这个受控出口，
显式放行需带参数且落审计事件。
顺带修掉三个既有缺陷
1. 合并遗留：sidecarEdit 用 sidecarGenerateResponseSchema 校验响应（要求 steps/
loadSeconds），但 run_hosted_edit 两个字段都不返回 —— 托管改字会在客户端边界
直接 Zod 抛错（本地路径返回这两个字段，所以一直没暴露）
2. check-migrations 的列检查只扫 CREATE TABLE，凡 ALTER TABLE ADD COLUMN 追加的
@map 列一律误报 drift（此前唯一的 ALTER 列 brand_kits.profile 没 @map 才没现形）
3. ktv-poster-master.png 血统在合并两侧打架，git log --all 证明该文件自初始化提交后
从未重新生成，取诚实标注 prototype-demo-master
验收
- 新增 pnpm check:script → reports/script-acceptance.latest.json（双后端各 8 passed
+ 1 skipped-live）；own-tests floor 10→14，棘轮已收紧
- pnpm check / check:runtime 均 exit 0
- 浏览器实测：未确认角色 → 按钮 disabled 且提示缺哪个角色；绕过前端直打后端 → 422
ROLE_NOT_CONFIRMED；跨租户 404；无凭证生成 → UI 显示 503 且不产生占位脚本
诚实边界（已写入 CLAUDE.md）
- liveLlmAsserted=false：本机无 DASHSCOPE_API_KEY，真调 LLM 的 live 断言被 skip，
门禁成立 ≠ LLM 出脚本已验证
- check:inference 在本机为 failed（动手前即为 failed）：本工作副本的模型权重是
git-lfs 指针存根，非本轮改动导致；基线表该行已从 GREEN 改为 OPEN
- ui-acceptance 报告早于本屏，未覆盖 script 屏，已标 STALE
- 新开缺口 G16（LLM 只做结构校验、无事实性校验）、G17（同步 HTTP 挂分钟级推理 /
render_jobs 索引非 CONCURRENTLY / 本地多图 I2I 未在本机验证）
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合 `/api/v1/{resources}` 规范
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1787029791
|
1787032807
|
1787032807
|
0
|
0
|
0
|
Edit
Delete
|
|
130
|
6
|
5
|
5
|
|
0
|
feat: 培训管理种子 + MQL 评分规则 + Gitea CI 迁移
|
对应 GitHub PR #52。
- backend/prisma/seeds/22-hr-tr 对应 GitHub PR #52。
- backend/prisma/seeds/22-hr-training-content.ts: 培训管理 12 张表种子
- backend/prisma/seeds/23-lead-scoring-rules.ts: MQL 默认评分规则集(修复 /lead-qualification 评估 400)
- .gitea/workflows/ci.yml: Gitea Actions CI(取代账户计费阻塞的 GitHub Actions)
- frontend course-marketplace.ts: array-guard 修复
- 已合并 origin/main 至 97a07d598,冲突全部解决
GitHub Actions 已仓库级禁用,CI 迁移到本 Gitea PR 验证。...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1778885087
|
1778891427
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
811
|
124
|
5
|
5
|
|
0
|
feat: add review invitation guardrail to manager c feat: add review invitation guardrail to manager cockpit (SCO11)...
|
基于 PR #2（`claude/sco11-invite`，已包含 PR #3）。PR #2 合并 基于 PR #2（`claude/sco11-invite`，已包含 PR #3）。PR #2 合并后，请把本 PR 的目标改为 `main`。
## 内容
- **14 项 P0 逐条证据**：为 CON03、SELF04/05/08/09/12、JRN01/03–06、CLN03、INS03、OPN02 补 58 条测试，不改源码。查到的缺口写入 `docs/acceptance-unimplemented.md`，例如：上一场私人欢迎和素材的清除逻辑缺失、限制名单没有批准生效动作、经营指标是固定样例、没有节假日价格日历
- **GRP06 拼团**（活动执行页）
- 规则须经 D29 批准才能开团
- 参与时校验团版本，防止并发超额
- 成团只产生"已成团待预约"，预约另查容量，且不等于入场
- 未成团时逐人生成"待退款核对"，不自动退款
- **GAM05 奖品额度**（活动执行页）
- 额度负责人、成本负责人、单价没登记就不能发奖
- 多个活动共用一个奖品池，数量和预算都不超
- 参与授权与音视频宣传授权分开
- **外部对接推进清单** `docs/integration-readiness.md`：按卡住的 P0 数量排序，列出每项依赖要业务方提供什么、接通前怎么降级、由哪个工程包承接
- 验收矩阵：未实现 2 项 → 0 项；有命名证据的需求 165 → 179
## 验证
- 原型测试 369/370：唯一失败是缺 `dist` 构建产物的打包测试
- `node ops/check-acceptance.mjs --check`、`node ops/sync-package-status.mjs --check`、`python3 scripts/verify-development-plan.py` 均通过
- 拼团与奖品两条流程已在浏览器中走通，控制台无报错
## 注意
- 所有拼团规则与奖品参数都是演示值，要等 D29 和 GAM05 的负责人确定后才能正式使用
- 签认仍为 0；30 项 P0 需等外部对接
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1791510471
|
1791510481
|
1791510481
|
0
|
0
|
0
|
Edit
Delete
|
|
809
|
124
|
3
|
5
|
|
0
|
feat: 补齐 9 项需求的原型规则与证据（HQR04 REV04 MKT05 LEA04 INS feat: 补齐 9 项需求的原型规则与证据（HQR04 REV04 MKT05 LEA04 INS05 SUP06 STO05 ALI05 DAT03）...
|
基于 PR #2（`claude/sco11-invite`）。PR #2 合并后请将目标分支改为 基于 PR #2（`claude/sco11-invite`）。PR #2 合并后请将目标分支改为 `main`。
## 内容
- **HQR04 视频墙**：筛选、排序、分页、轮巡规则抽到 `hq-wall-domain.mjs`，页面行为不变
- **REV04 评价统计**：按周趋势并注明覆盖范围、处理时长中位数和 P90、整改后复发、是否能宣称口碑提升
- **MKT05 活动复盘**：继续、调整、扩大、暂停、停止建议，标注为关联观察，不计算增量；复盘决定会联动门店活动执行
- **LEA04 线索分析**：去重、只计可靠关联的转化、跟进及时性、渠道费用核对后才算获客成本、观察期满才计流失
- **INS05 巡检分析**：计划完成、整改耗时（算到独立核实）、复发、交接；每个指标可追溯到来源记录；只看关闭数量时给出提醒
- **SUP06 辅助答复**：知识只管措辞且有版本；状态句只取自已查询的业务事实；事实无法确认时转人工；发送须客服确认
- **STO05 跨店替代**：冻结原承诺快照；顾客明确选择；补差须单独同意；目标店独立接收后责任才转移
- **ALI05 合作素材**：素材授权核验，缺授权、渠道不符或过期则拒绝验收；合作费用单列；合作复盘只计明确关联
- **DAT03 品牌指标**：曝光、态度、消费、复购四项并列；复购须整组观察期满才给比率；缺来源显示空值而不是 0
## 验证
- 原型测试 303/304：唯一失败是缺 `dist` 构建产物的打包测试
- 服务端可独立运行的测试 363/366：3 个失败是本工作区未编译框架 `dist`，与本改动无关
- `node ops/check-acceptance.mjs --check`、`node ops/sync-package-status.mjs --check`、`python3 scripts/verify-development-plan.py` 均通过
- 每项都在浏览器中走通主流程，控制台无报错
## 注意
- 改动到共享代码的地方：
- `services/ktv/application.mjs`：活动状态联动改用共享函数 `linkedActivityStatus`
- 合作验收新增素材授权检查：种子素材已补授权，原有流程测试不受影响
- 所有阈值和演示数据都标为演示参数，待 D19、D25 等决策确认；真实渠道和数据源仍依赖 DEP09、DEP15
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1791508977
|
1791508987
|
1791508987
|
0
|
0
|
0
|
Edit
Delete
|
|
846
|
83
|
3
|
5
|
|
0
|
fix(ci): fetch locked OS with a Gitea token header fix(ci): fetch locked OS with a Gitea token header and probe access first...
|
actions/checkout presented OS_REPO_READ_TOKEN as B actions/checkout presented OS_REPO_READ_TOKEN as Basic credentials and
Gitea answered 401 (git fell back to a username prompt) for two
different correctly scoped tokens. Fetch the locked SHA with plain git
and an "Authorization: token" header into RUNNER_TEMP instead, after a
Gitea API probe that prints only the HTTP status, so a missing secret,
unusable token or missing repository access fails with a clear reason.
This step also no longer downloads an action from github.com.
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793096204
|
1793096226
|
1793096226
|
0
|
0
|
0
|
Edit
Delete
|
|
850
|
83
|
7
|
5
|
|
0
|
ci: per-run docker PostgreSQL/Redis for acceptance ci: per-run docker PostgreSQL/Redis for acceptance, with reports rebound to e8591c7...
|
Two commits:
- e8591c7 ci: the runtime job no l Two commits:
- e8591c7 ci: the runtime job no longer uses `services:` (never started on this host-mode runner; 127.0.0.1:5432 reached an unrelated PostgreSQL on the runner host in run 26519). scripts/ci-acceptance-services.sh fails without docker; otherwise it starts per-run PostgreSQL/Redis containers on docker-assigned high ports with a random masked password and removes them in an if: always() step. G14 updated.
- 8cf50a6 chore(reports): runtime, UI and static reports rerun on the clean final source e8591c7.
Local evidence (isolated containers, recreated database):
- check:runtime 452/452 (floor 452), frozen oracle 12, outbox 1 scenario / 0 duplicates, outage health 2.
- check:ui 46/46.
- pnpm check with DIGITAL_EMPLOYEE_OS_ROOT at e12d0f2: exit 0, provenance violations 0.
- Script checks: start/connect/stop with no leftovers, repeated stop, and failures for missing docker, a non-prefixed database name and an invalid run id.
Not yet verified: docker on the runner host and its ability to pull postgres:16 / redis:7 (override with CI_POSTGRES_IMAGE / CI_REDIS_IMAGE repository variables).
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793102531
|
1793190011
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
818
|
124
|
12
|
5
|
|
0
|
feat(GOV01): renew development test registration w feat(GOV01): renew development test registration without rotating identities...
|
The registered development scope had no renewal pa The registered development scope had no renewal path: rerunning the script
only re-validated the file, and bindings expire 30 days after generation.
- renewDevelopmentRegistration() keeps tenant, store, registration, binding
and person UUIDs; only active bindings get expiresAt = now + days
(1-30, default 30). Revoked/suspended people are never revived. Each
renewal records time, authorization and previous expiry in `renewals`.
- ops/register-development-test-scope.mjs --renew "<authorization>".
- Renewed the fixture once: 2026-11-07T02:15Z -> 2026-11-08T15:33Z.
- Running registered gateways read their release copy and need a redeploy.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1791560333
|
1791560339
|
1791560339
|
0
|
0
|
0
|
Edit
Delete
|
|
847
|
83
|
4
|
5
|
|
0
|
ci: stop pulling actions from github.com on the Gi ci: stop pulling actions from github.com on the Gitea runner...
|
The Gitea runner repeatedly timed out cloning acti The Gitea runner repeatedly timed out cloning actions/checkout from
github.com (once hanging Set up job for over 10 minutes), so jobs failed
or blocked the single runner before any repository code ran.
- Checkout: plain git with the Gitea task token as Basic credentials,
all branches and tags fetched (F5 provenance needs history), detached
at github.sha, workspace cleaned.
- Node: scripts/ci-setup-node.sh downloads an exact version from
npmmirror and verifies it against the mirror's SHASUMS256.txt; both
governance jobs and the platform job use 22.23.2.
- platform-governance: upload-artifact replaced by printing the gate
reports into the run log. This file now diverges from the
@juhai/governance consumer template.
Verified locally: the checkout snippet against a file:// copy of this
repository (detached at the requested SHA, stale files removed); the
Node script in a linux/arm64 container with a test-only curl stand-in
(install, cache reuse, tampered archive rejected, unknown version
rejected).
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793096895
|
1793096902
|
1793096902
|
0
|
0
|
0
|
Edit
Delete
|
|
815
|
124
|
9
|
5
|
|
0
|
test: move remaining hardcoded booking fixtures on test: move remaining hardcoded booking fixtures onto future dates...
|
The CRM04/BRD04 20:30 failures were already fixed The CRM04/BRD04 20:30 failures were already fixed on main (3e41516,
4941806). Running the self-contained suites under a shifted clock found
two more booking fixtures that would start failing once their date passed:
- composition-fixtures.mjs booked 2026-10-20, so all seven PER07 tests
fail from 20:30 Asia/Shanghai that day.
- prototype-enhancement.persistence.test.mjs defaulted to 2026-10-12 and
used 10-13/14/15 for its other bookings.
Both now go through futureBookingDate(), which keeps weekdays and distinct
dates while staying ahead of the server day.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1791559271
|
1791559275
|
1791559275
|
0
|
0
|
0
|
Edit
Delete
|
|
756
|
85
|
1
|
4
|
|
0
|
[Intent] 按省市统计设备月度新增上线数量报表
|
Related to #8
## 需求摘要
在「数据报表 > 设备分析」下新增报表，按月筛 Related to #8
## 需求摘要
在「数据报表 > 设备分析」下新增报表，按月筛选展示各省份新激活上线设备数量柱状图排行，支持下钻到城市维度。
## 核心决策
- 统计口径：当月**新激活**上线设备（首次上线，取 ahead_authenticate_log 表的 create_time 字段）
- 地区归属：取设备绑定**门店地址**的省/市字段
- 交互方式：点击省份**下钻**到城市排行（非 Tab 切换）
- 权限：跟随现有 agent_id 角色体系，超管看全国，运营商看自己区域
## 不在范围内
- 不做 Excel 导出
- 不做地图热力图
- 不做区/县级别
- 不做跨月趋势
## 产物文件
[intent/device-online-stats-by-region.md](https://gitea.g-hi.com/laiqiaojie/jh-project/src/branch/intent/device-online-stats-by-region/intent/device-online-stats-by-region.md)
## PM 审核清单（参考 REVIEW.md）
- [ ] 需求背景和问题描述是否准确反映了原始诉求？
- [ ] 期望结果是否可验证？
- [ ] 不在范围内的排除项是否合理？
- [ ] 受影响系统判断是否正确（admin / ahead_authenticate_log）？
- [ ] 本次是否接受排期，进入 Design 阶段？...
|
1
|
0
|
1
|
1
|
2
|
|
0
|
1787817232
|
1789379898
|
1787823622
|
0
|
5
|
0
|
Edit
Delete
|
|
770
|
85
|
7
|
4
|
|
0
|
[Skill] build-complete 新增接口文档归档步骤
|
Related to #8
## 说明
在 `build-complete` skill 里新增 Related to #8
## 说明
在 `build-complete` skill 里新增一步：Plan 阶段确定接口设计后、代码开发完成时，如涉及新增/修改接口，需要生成并归档接口文档：
- Apifox（规范化 OpenAPI 文档，Claude 通过 API 增量导入）
- `docs/frontend-specs/<功能名>.md`（面向其他端的对接说明文档，固定五章结构）
背景：issue #4 实现完成后发现接口文档一直没有归档，靠事后补做；这次把它固化成流程里的正式一步，插在"提交 plan.md"和"开代码 PR"之间。
同时补充了代码仓库与 jh-project 分离时的注意事项（milestone/label 可能不存在、Close #N 跨仓库不生效）。...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1788247386
|
1789379900
|
1788247940
|
0
|
1
|
0
|
Edit
Delete
|
|
768
|
85
|
6
|
4
|
|
0
|
[Plan] 按省市统计设备月度新增上线数量报表 (后台管理)
|
Related to #8
## 关联 Issue
[#4 — [后台] 按省市统计设备月度新增 Related to #8
## 关联 Issue
[#4 — [后台] 按省市统计设备月度新增上线数量报表](https://gitea.g-hi.com/laiqiaojie/jh-project/issues/4)
## 说明
补交 Plan 阶段产物（此前误提交到已合并关闭的 design 分支，未能进入 main，见此 PR 重新提交）。
代码已实现并提交：[laiqiaojie/admin#1](https://gitea.g-hi.com/laiqiaojie/admin/pulls/1)
按 REVIEW.md「跳过评审的路径」，`plan/` 产物文件不需要 Tech Lead 单独审核，随此 PR 直接入库。...
|
0
|
0
|
1
|
1
|
1
|
|
0
|
1788245555
|
1789379899
|
1788245966
|
0
|
1
|
0
|
Edit
Delete
|
|
758
|
85
|
3
|
4
|
|
0
|
[Design] 按省市统计设备月度新增上线数量报表
|
Related to #8
## 关联 Intent
[PR #1 — 按省市统计设备月度新增上 Related to #8
## 关联 Intent
[PR #1 — 按省市统计设备月度新增上线数量报表](https://gitea.g-hi.com/laiqiaojie/jh-project/pulls/1)
---
## PM 确认原型
**需求说明文档：**
[design/device-online-stats-by-region.md](https://gitea.g-hi.com/laiqiaojie/jh-project/src/branch/design/device-online-stats-by-region/design/device-online-stats-by-region.md)
**在线原型（点击直接打开，可交互，无需登录）：**
https://claude.ai/code/artifact/b50eaa98-689b-4742-8024-cdf74f95d5c8
**页面状态：**
- 全国省份视图 — 月份筛选器 + 当月总数卡片 + 各省份横向柱状图
- 城市下钻视图 — 点击省份柱子进入，面包屑「全国 › 广东省」+ 城市排行
**交互流程：** 点击省份柱子进入城市视图，点击面包屑「全国」或「返回全国」按钮返回。
**审核 checklist：**
- [ ] 页面布局和组件符合预期？
- [ ] 交互路径（下钻、返回）正确？
- [ ] 数据定义（新增上线 = 首次上线，归属 = 门店注册地址）理解一致？
- [ ] 权限范围（超管看全国，运营商看自己区域）正确？
**确认方式：在本 PR 评论「原型确认 ✅」，然后 Approve + Merge**...
|
1
|
0
|
1
|
1
|
0
|
|
0
|
1787829439
|
1789379899
|
1787908921
|
0
|
7
|
0
|
Edit
Delete
|
|
844
|
83
|
1
|
5
|
|
0
|
docs(governance): sync dynamic baseline with curre docs(governance): sync dynamic baseline with current D3 reports...
|
Refresh the GOVERNANCE-BASELINE section of CLAUDE. Refresh the GOVERNANCE-BASELINE section of CLAUDE.md from
reports/governance.latest.json, reports/baseline-d3.json and the
2026-10-25 runtime/UI/RLS reports plus the 2026-09-23 OS audits:
static governance now passes, runtime 452, UI 46, RLS 116 tables,
own tests 69/471, single-host outbox/health scenarios, OS product
0.19.4 upgrade ready and release blocked only by remote CI and
target sign-off. Retire the native Fastify UI row and keep earlier
dual-host figures labelled as history.
Verified with check:governance-docs, check:docs-truth and
check:dual-backend (all exit 0); regenerated gate reports were not
committed.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合当前生命周期规范：既有 Personnel Core 保持 `/api/personnel/*`，HR v2 领域使用 `/api/v2/{domain}/*`，身份协议使用标准 OIDC 端点
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 涉及页面或浏览器行为时，`pnpm check:ui` 与 `pnpm check:ui:fastify` 均通过，未只验证一个后端
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793092331
|
1793092337
|
1793092337
|
0
|
0
|
0
|
Edit
Delete
|
|
803
|
121
|
1
|
5
|
|
0
|
wip(K3.3): PolicyDefinition — preserved snapshot o wip(K3.3): PolicyDefinition — preserved snapshot of uncommitted work (base 0.71, not for merge)...
|
Preservation snapshot (R0, 2026-09-29) of work tha Preservation snapshot (R0, 2026-09-29) of work that only existed as uncommitted changes in a /tmp worktree.
Nothing here was verified by this commit: no tests were ever written for it and it was not run against a database.
What the snapshot contains:
- sql/072_policy_definition.sql: governance.policy_definition_version (EXECUTION_AUTHORIZATION with maxTtlSeconds),
a ten-state lifecycle, immutable content, one ACTIVE version per tenant and type, FORCE RLS; also replaces the
authority_assignment action CHECK with governance.manage_policy / governance.approve_policy added. The file header
is a copy of the K3.2 header.
- src/contexts/policy-definition.js: the PolicyDefinition commands (create, validate, request review, approve,
reject, publish, activate, deprecate, retire, revoke) with self-approval denial and authority checks.
- registrations in commands.js, owner-roles.js, rejection-target.js, kernel.js, catalog and the payload / event
contracts.
Known state when preserved:
- Based on 0490b8e (0.71). Main is 0.103.x with migrations up to 095; 072 already exists there
(072_task_case_correlation_immutable), so the migration must be renumbered, and kernel.js / catalog / contracts
will need a real rebase.
- Missing for the slice: tests, contract document, smoke, and the binding of a GovernanceDecision to the active
policy version with drift rejection (K3.3b).
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1790675308
|
1790675308
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
812
|
124
|
6
|
5
|
|
0
|
feat(SUP01): public FAQ and pre-booking customer h feat(SUP01): public FAQ and pre-booking customer help on current main...
|
Port the public FAQ, pre-booking general help and Port the public FAQ, pre-booking general help and original help-record
targeting from codex/ktv-delivery-integration (5d56fcc) onto main's
consumer workspace reader (f295ce8), instead of the older readSerial guard.
- Signed-out and no-booking customers can open the FAQ; it exposes no
customer, store, price or device facts.
- Signed-in customers without a booking can save a store-level general
help request; payment/redeem/door topics still require the original
booking. Results reuse independent review and customer confirmation.
- Receipts locate the exact original help record; help drafts and result
notes are cleared only within the same identity and help scope.
- SUP01 gets named test mappings (consumer-help, prototype-enhancement,
persistence).
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1791558540
|
1791558556
|
1791558556
|
0
|
0
|
0
|
Edit
Delete
|
|
845
|
83
|
2
|
5
|
|
0
|
fix(ci): re-hash O18 OS Web sources from an explic fix(ci): re-hash O18 OS Web sources from an explicit locked OS checkout...
|
O18 re-hashed OS Web sources through the osRoot re O18 re-hashed OS Web sources through the osRoot recorded inside the
host UI report, an ephemeral /tmp clone that never exists on a CI
runner and disappears locally after cleanup. governance.yml therefore
failed on every Gitea run and the runtime/UI job never executed.
- O18 now only trusts --os-root / DIGITAL_EMPLOYEE_OS_ROOT and fails
closed with an explicit message when neither is given; new negative
probe product-host-ui-without-explicit-os-root (258 probes).
- The static CI job resolves compatibilityLock.osGitSha from
readiness.json, checks out luoanwu/digital-employee-os at that SHA
with secret OS_REPO_READ_TOKEN, moves it to RUNNER_TEMP, builds the
compat packages and asserts a clean tree; Node 22 to satisfy the OS
engine-strict requirement.
- Docs: CLAUDE.md commands/G9/G14/ci-gate, README, acceptance-runner
and business-module-intake now state the OS checkout requirement.
Local evidence (dirty worktree): clean OS clone at e12d0f2 makes
check:os-product pass; missing root and a tampered OS Web file both
fail O18. Full pnpm check still fails check:governance on runtime/UI
report fingerprints, which predates this change and needs a rerun of
check:runtime and check:ui on the final tree.
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793094537
|
1793095912
|
1793095912
|
0
|
0
|
0
|
Edit
Delete
|
|
814
|
124
|
8
|
5
|
|
0
|
fix(AUD-EXIT): keep fulfilling original obligation fix(AUD-EXIT): keep fulfilling original obligations after cooperation exit...
|
Exit used to freeze an exiting cooperation: delive Exit used to freeze an exiting cooperation: deliveries were rejected, so
open promises had no continued-handling path (ALI01, AC30).
- deliver accepts active or exiting cooperations, still capped by each
original obligation; after-exit entries are tagged phase:'after-exit'
with the acting principal.
- exit records which obligations were open at that moment; there is still
no action that adds new obligations, and approve stays draft-only.
- new completeExit moves exiting -> exited only when every original
obligation is delivered (OBLIGATIONS_OPEN otherwise); exited rejects
further deliveries. Finance reconcile is unaffected and funds remain
unverified.
- store workbench mirrors the rules and adds a 完成退出 action.
- ALI01/AC30 named test mapping for the new rule test.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1791559039
|
1791559044
|
1791559044
|
0
|
0
|
0
|
Edit
Delete
|
|
858
|
83
|
15
|
5
|
|
0
|
feat(contracts): 出站事实来源域按事实类型收紧（V-04，PKG-004）
|
CHG-061 独立核验 V-04：通用 Intent 记录 Schema 把 `sourceDom CHG-061 独立核验 V-04：通用 Intent 记录 Schema 把 `sourceDomain` 写死为 `hr`，CHG-061 登记的组织事实（Catalog 与 04 Scope 要求 `source_domain = organization`）即使登记进注册表也无法出站。
## 改动（D-12 台账 PKG-004，`packages/*` 冻结字节链式放行）
- 通用记录 Schema 的 `sourceDomain` 只校验非空字符串；来源域由注册表条目的信封 Schema 按事实类型收紧，`ENTERPRISE_FACT_ENVELOPE_SCHEMAS` 的类型约束要求每个条目 `source_domain` 为字面量。
- `resolveEnterpriseFactEnvelopeSchema` / `buildEnterpriseFactEnvelope` 增加默认为生产注册表的可选注册表参数；relay 调用不变。
- 生产注册表仍只有 `EmploymentTransferred@1`，行为不变；未登记类型仍以 `FACT_TYPE_UNREGISTERED` 失败关闭。
- contracts 追加 4 例（测试用注册表下 organization 来源域封装通过、hr 来源域被拒；未登记组织事实失败关闭；EmploymentTransferred 仍只接受 hr）。负向：记录 Schema 改回 `z.literal("hr")`，新增 2 例判红。
- 文档：对接蓝图 H1 / H21 / P1 按 CHG-061 落地订正并新增 §4.15；`CLAUDE.md` 真源地图与 G23 动态区同步。
## 验证（本地工作区，clean `11adf0f`，一次性 PostgreSQL / Redis）
- `check:d3` 0 问题；contracts 219/219；`check:app-manifest` 通过；lint、typecheck 通过。
- `check:runtime` 466 / 0（地板 452）；`check:ui` 46 / 46；报告回绑提交 `e51fd28`。
- `check:governance` 只剩 main 上既有的 O22（OS main 不含 0.19.6，H12 / D-8）。远端 CI 未查验。
## 未包含
生产注册表登记新事实、`app-manifest.json` 的 `facts.owns`、`@juhai/contracts` 升 pin（rc.7 的 Catalog 为 11 条）、组织 / 用工事实的 producer 与生效事务写意图。
🤖 Generated with [Claude Code](https://claude.com/claude-code)
...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793150819
|
1793150819
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
819
|
124
|
13
|
5
|
|
0
|
feat(GOV01): allow 90-day development registration feat(GOV01): allow 90-day development registration renewals and renew again...
|
At the user's request the renewal cap and defa At the user's request the renewal cap and default go from 30 to 90 days
(DEVELOPMENT_RENEWAL_MAX_DAYS); first-time generation still issues 30 days.
Renewed the fixture again with the same identities: bindings now expire
2027-01-07T15:38:57Z. Both renewals are recorded in `renewals`.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1791560579
|
1791560584
|
1791560584
|
0
|
0
|
0
|
Edit
Delete
|
|
134
|
6
|
8
|
5
|
|
0
|
test(governance): AI Agent/获客域过时 E2E 选择器对齐 canonic test(governance): AI Agent/获客域过时 E2E 选择器对齐 canonical...
|
AI Agent 员工中心 / 获客中心域审计验收后残留 E2E 过时选择器对齐 canonical AI Agent 员工中心 / 获客中心域审计验收后残留 E2E 过时选择器对齐 canonical（curl/页面实证），两 spec 单跑+合并复跑 14/14 全绿，未放宽断言或 skip。
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1779070186
|
1779328857
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
823
|
124
|
17
|
5
|
|
0
|
feat(AC79): move video wall layout controls into t feat(AC79): move video wall layout controls into the domain and add evidence...
|
AC79 (HQR04) was the only non-external acceptance AC79 (HQR04) was the only non-external acceptance case without named
evidence: group/flat, 4/9/16, paging, sorting and rotation switches must
keep the scope, and mobile focus/return must stay reachable.
- hq-wall-domain: wallControl() owns slots/sort/pin/mode/page/tour changes
and never touches store, region, type or point scope; wallSections()
derives group/flat sections for the current page.
- HeadquartersConsole/VideoWall use them instead of inline setters; the
behavior is unchanged (layout, risk-first, manual paging, focus and talk
stop rotation; group/flat keeps the page; sort/pin/layout reset to page 1).
- New AC79 test walks every switch over three scopes and checks each page
covers the scope exactly once; AC79 maps to it.
- Browser (store dev server, demo data): 北区 scope stayed 12 cameras/3
stores through 4->flat->risk->16; next-page stopped rotation; at 375px,
focus moved to the detail panel and return restored scroll and the
original focus button.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1791589290
|
1791589300
|
1791589300
|
0
|
0
|
0
|
Edit
Delete
|
|
852
|
83
|
9
|
5
|
|
0
|
fix(api): D-10 默认只监听 127.0.0.1，对外暴露须显式设 HOST
|
> 状态说明：本 PR 已于 2026-10-27 合入 `main`（合并提交 `7835e > 状态说明：本 PR 已于 2026-10-27 合入 `main`（合并提交 `7835e3b`），合并时 CI 检查尚未运行；远端 CI 结果以本 PR 的检查为准。
## 概要
执行蓝图 D-10（关闭 H22）：HR API 默认只监听 `127.0.0.1`，需要容器或局域网访问时必须显式设 `HOST`（如 `0.0.0.0`，双栈用 `::`）。改动前后都实测了 04 runtime 容器的可达性。记录见 `docs/domain/enterprise-control-plane-integration-blueprint-2026-10-27.md` §4.7。
## 改动
- **监听地址单源** `apps/api-nestjs/src/listen-host.ts`：未设或空 `HOST` → `127.0.0.1`，否则用显式值。`main.ts` 只经 `resolveListenHost()` 取值，启动日志写明是否只在回环上监听。
- **Web 默认目标**：`next.config.ts` 代理与 `useRealtime` 的 WS 默认值从 `localhost:3001` 改为 `127.0.0.1:3001`，避免 `localhost` 先解析到 `::1` 时连不上只听 IPv4 回环的 API；`.claude/launch.json` 同步。
- **防回潮**：`check:dual-backend` 新增 `scripts/lib/listen-host-guard.mjs`（默认值必须是回环，`main.ts` 不得绕过解析函数），node 单测 9 个负例；把默认值改回 `0.0.0.0` 的变异探针判红 2 处。行为单测 `apps/api-nestjs/test/listen-host.test.ts` 3 例。
- **文档同步**：CLAUDE.md（常用命令、运行时拓扑、G23 中 H22 移为已关闭）、README、后端规范、Scope 来源接入、蓝图 §4.7 / H22 / D-10、给 04 的答复草稿、框架同步评估 F-7。
## 实测证据
证据文件：`docs/evidence/d10-listen-host-2026-10-27.json`（本机 Docker Desktop；对 04 容器只做只读 GET，未改 04 的容器或配置）。
**改动前**，用宿主上的最小 HTTP 探针：
| 绑定 | 宿主回环 | 本机局域网 IP | 04 容器经 `host.docker.internal` |
| --- | --- | --- | --- |
| `0.0.0.0` | 可达 | 可达 | 可达 |
| `127.0.0.1` | 可达 | 不可达 | 可达（宿主看到来源 `127.0.0.1`） |
结论：改为回环默认后，04 的 Scope 来源探针（`SCOPE_HR_SOURCE_URL` 指向 `host.docker.internal:65281`）仍可达。
**改动后**，用真实构建的 HR API 配本仓一次性 PostgreSQL/Redis（跑完已删除）：
| `HOST` | lsof | 局域网 `/api/health` | 04 容器 `/api/health` |
| --- | --- | --- | --- |
| 未设 | `127.0.0.1` | 不可达 | 200，database/redis up |
| 空串 | `127.0.0.1` | 不可达 | 200，database/redis up |
| `0.0.0.0` | `*` | 200 | 200，database/redis up |
另外，Web 不设 `API_PROXY_TARGET` 时，经 Next 代理访问 `/api/health` 返回 200。
**全量验收**（本地，clean `9c53371`，一次性库，跑完已删，报告无口令命中）：
- `check:runtime` 455/455：比 452 多出的是新增的 3 例；冻结行为、多实例 Outbox、依赖故障三项均通过。
- `check:ui` 46/46。
- `check:governance` 仅剩已知两项回退：O22（OS main 不含 0.19.6）、`osProductReleaseTargetRemotes` 2→1；来源指纹违规 0。
- 远端 CI 以本 PR 的运行为准。
**未覆盖**：
- 未用 Scope 来源真实凭据调用签名查询，只证明网络可达。
- Linux 原生 Docker 经 `host-gateway` 连不到宿主回环，这类部署必须显式设 `HOST`。
- 仓外 65281 实例当前无进程；它的源码 `326ae3e` 不读 `HOST`，恢复前须换到含本改动的提交。
- `apps/web` 的 `next dev` / `next start` 仍按 Next 默认监听全部网卡，不在本 PR 范围。
## 待裁：00 框架 F-7
00 框架 `apps/api-nestjs/src/main.ts:8` 仍是 `process.env.HOST ?? "0.0.0.0"`；`backend-standard.md:7`、`COMPATIBILITY.md:45`、ADR-0010 与迁移手册都把 `0.0.0.0` 写成默认，理由是容器内可达。HR 因此有意与框架不同。本 PR 未改框架仓，请框架 Owner 二选一：
- **(a)** 框架也改为回环默认，容器部署在 Dockerfile / compose 里显式设 `HOST=0.0.0.0`。
- **(b)** 维持 `0.0.0.0`，在 `COMPATIBILITY.md` 登记「消费者可偏离为回环」。
无论选哪个，下一次框架同步或叠加仓 `replace: true`（F-5）都不得把 HR 的回环默认冲回去。详见 `docs/framework-sync-assessment-2026-10-27.md` F-7。
## 另外发现（不在本 PR 范围）
- `apps/api-nestjs/docker-compose.yml` 的开发 PostgreSQL（口令为 `app`）与 Redis 映射到 `0.0.0.0`，属同类暴露。
- `CLAUDE.md` 基线表 runtime 行仍是 10-25 的读数 452：报告指纹包含文档，验收后改文档会使报告失效，留待下次改文档时一并更新。
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793111729
|
1793114019
|
1793111736
|
0
|
1
|
0
|
Edit
Delete
|
|
759
|
85
|
4
|
4
|
|
0
|
[后台] 按省市统计设备月度新增上线数量报表
|
> 子任务，属于主 Issue #8「按省市统计设备月度新增上线数量报表」
## 关联 De > 子任务，属于主 Issue #8「按省市统计设备月度新增上线数量报表」
## 关联 Design PR
[PR #3 — Design: 按省市统计设备月度新增上线数量报表](https://gitea.g-hi.com/laiqiaojie/jh-project/pulls/3)
## 工作范围
负责后台接口实现：
- 新增查询接口，按月份统计各省份新增上线设备数量
- 支持下钻查询：指定省份，返回城市级别数据
- 按权限过滤：超管返回全国数据，运营商返回自己区域
## 产物
- （实现前提交，Tech Lead 审核后开始 Build）
## 参考
- 需求说明：[design/device-online-stats-by-region.md](https://gitea.g-hi.com/laiqiaojie/jh-project/src/branch/design/device-online-stats-by-region/design/device-online-stats-by-region.md)
- 在线原型：https://claude.ai/code/artifact/b50eaa98-689b-4742-8024-cdf74f95d5c8...
|
1
|
0
|
1
|
0
|
2
|
|
0
|
1787909526
|
1789380193
|
1789380193
|
0
|
1
|
0
|
Edit
Delete
|
|
760
|
85
|
5
|
4
|
|
0
|
[Web] 按省市统计设备月度新增上线数量报表
|
> 子任务，属于主 Issue #8「按省市统计设备月度新增上线数量报表」
## 关联 De > 子任务，属于主 Issue #8「按省市统计设备月度新增上线数量报表」
## 关联 Design PR
[PR #3 — Design: 按省市统计设备月度新增上线数量报表](https://gitea.g-hi.com/laiqiaojie/jh-project/pulls/3)
## 工作范围
负责 Web 前端页面实现：
- 数据报表 → 设备上线分布 页面
- 月份筛选器 + 当月总数卡片 + 省份横向柱状图
- 点击省份下钻到城市视图，支持面包屑返回
## 产物
- plan/device-online-stats-by-region-web.md（实现前提交，Tech Lead 审核后开始 Build）
## 参考
- 需求说明：[design/device-online-stats-by-region.md](https://gitea.g-hi.com/laiqiaojie/jh-project/src/branch/design/device-online-stats-by-region/design/device-online-stats-by-region.md)
- 在线原型：https://claude.ai/code/artifact/b50eaa98-689b-4742-8024-cdf74f95d5c8...
|
1
|
0
|
1
|
0
|
3
|
|
0
|
1787909556
|
1789380193
|
1789380193
|
0
|
1
|
0
|
Edit
Delete
|
|
871
|
83
|
18
|
5
|
|
0
|
feat(personnel): D-13 组织类型字段与宿主 Schema 受控通道（PKG-00 feat(personnel): D-13 组织类型字段与宿主 Schema 受控通道（PKG-006）...
|
> **叠在 #17 之上（#17 又叠在 #13 之上），请按 #13 → #17 → 本 > **叠在 #17 之上（#17 又叠在 #13 之上），请按 #13 → #17 → 本 PR 的顺序合入。** 本 PR 自身只有最后 2 个提交：`4fa64f1`（D-13）、`6912ab7`（报告回绑）。
## 背景
- F1 `OrganizationUnitMoved` 要求 `unit_type` 必填（CHG-061 Q5），而 HR 组织原本没有类型字段。
- 02 已于 10-27 定下词表（D73）：采用 CHG-061 的三值，02 侧分别映射为 `BUSINESS_UNIT` / `DEPARTMENT` / `TEAM`。
- 加字段要改 D3 冻结的宿主 `schema.prisma`。用户 2026-10-28 裁定 D-13：开受控通道并实现。
## 内容（对接蓝图 §4.17）
- **D3 受控通道**（`scripts/lib/d3-protection.mjs`）：`packageUpdates` 只多放行一个文件，即宿主 `apps/api-nestjs/prisma/schema.prisma`。既有迁移、`migration_lock.toml` 和已退役的 Fastify schema 仍不得登记；结构变更一律以新迁移追加。
- **字段**：组织新增可空的 `unitType`（`ORGANIZATION` / `DEPARTMENT` / `TEAM`）。迁移 `20261028090000_organization_unit_type` 只追加这一列；已有组织的值为 null，不做推断。
- **contracts**（台账 PKG-006）：
- 创建时可选，缺省为 null；
- 编辑时省略即保留原值，显式传 null 则清除；
- 命令视图与组织摘要必带该字段；
- 双时间快照读取该字段，D-13 之前的快照读为 null。
- **服务**：创建和编辑写入该字段，仍沿用原有乐观锁和 outbox；当前读取与双时间读取两条路径都输出该字段。
- **Web**：组织增改表单新增「组织类型」，组织详情显示类型。
- **未做**：
- OS 产品副本还没有这一列，登记为 H29。0.19.6 尚未进入 OS main，0.19.7 现在注册不了，等它进入后再随产品版本补上。
- F1 本身：等 contracts 发版后，与 F2–F4 一起实现。
- 已有组织的类型补录：需要业务按实际情况登记。
## 验证（本地，报告绑定 `4fa64f1`、clean）
- `d3-protection` 27/27，新增 2 例；2 个变异探针（去掉放行、放宽到整个 prisma 目录）都判红。
- contracts 232/232，新增 5 例。
- 真实 DB 的 HTTP 测试 3/3，覆盖：带类型 / 不带类型创建；编辑时省略保留、修改、清除；词表外取值返回 400 且不落库；用 `knownAt` 能回看到旧类型。另有 3 个服务层变异探针都判红。
- `check:runtime` 482/482，`check:ui` 46/46（在原有组织增改场景里加入了类型选择和读回），用一次性 PostgreSQL / Redis 跑，跑完已删。
- 静态检查：D3 / schema / migrations / RLS / write-guard 等门禁全部通过；静态链在 O22 处如实停止，`check:governance` 只剩 O22。
- 远端 CI 见本页。
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793194416
|
1793194416
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
861
|
83
|
17
|
5
|
|
0
|
feat(contracts): P1 前置片——CHG-061 三条 HR 事实的载荷与信封 Sc feat(contracts): P1 前置片——CHG-061 三条 HR 事实的载荷与信封 Schema（PKG-005）...
|
> **叠在 #13 之上，请先合 #13。** 本分支已合入更新后的 #13（#13 已含 > **叠在 #13 之上，请先合 #13。** 本分支已合入更新后的 #13（#13 已含 main 上的 #16）。#13 合入后，这里的差异只剩 P1 前置片与报告回绑。
## 为什么重建（2026-10-28）
PR #16 已在 main 上以 PKG-004 落地 V-04（来源域按事实类型；注册表条目的 `source_domain` 须为字面量）。本 PR 原先也用 PKG-004 实现了另一版 V-04，与 #16 冲突。重建时：
- contracts 与台账取 main 版本；
- 去掉本分支自己的 `sourceDomain` 放宽；
- 改按 #16 的注册表类型重做，台账改为 **PKG-005**，蓝图记为 **§4.16**。
## 内容（对接蓝图 §4.16，P1 / H5）
- **contracts**（`packages/contracts/src/enterprise-fact.ts`，台账 PKG-005）
- 新增 `organizationCanonicalIdSchema`（`org_`）。
- 新增 `EmploymentStarted`、`EmploymentTerminated`、`OrganizationUnitStatusChanged` v1 的载荷 Schema，逐字段对应 04 `contracts/schemas/facts/*.v1.json`（CHG-061）。组织状态载荷另要求 `change_kind` 与 `state_after` 一致。
- 三份信封 Schema 保持 ZodObject，`source_domain` 为字面量（hr / organization），满足 #16 的注册表类型（typecheck 已证明），升 pin 后可以直接登记。
- `buildEnterpriseFactEnvelope` 在注册表 Schema 解析之后统一校验两条：载荷里的实体 ID 等于 `entity_id`；组织事实的 `valid_from` 等于 `effective_at`。校验失败抛 ZodError，Relay 记为 `FACT_SCHEMA_INVALID` 并进死信。这条同样约束已登记的 `EmploymentTransferred`；HR 写入时两者本来就相同。
- **刻意不做**：出站注册表、`facts.owns`、生效事务写 Intent。原因是 M6 只认 pinned `@juhai/contracts` 的 Catalog，而 Registry 上最新的 rc.7 只有 11 条事实。发版已交给单独的 04 会话推进，真实发布前须用户同意；HR 侧的请求草稿是 `docs/domain/request-draft-04-contracts-release-chg061-2026-10-28.md`。
- **F1 `OrganizationUnitMoved`**：组织类型词表已由 02 D73 定为 CHG-061 三值（`organization` / `department` / `team`）。HR 加这个字段要改 D3 冻结的宿主 Prisma 和产品迁移，已登记为待裁决 D-13（用户已同意开受控通道并实现，另起 PR）。
- **文档**：蓝图 §4.16、H5 / P1 行，矩阵 #19 按 02 D72 订正（Site / Space / SiteOperator 的现用写点改为 EES 物包），新增 D-13；同步更新 CLAUDE.md G23、CHG 草案和发版请求草稿。
## 验证（本地，报告绑定 `b8f85d3`、clean）
- contracts 227/227（enterprise-fact 24 例，其中新增 8 例）。
- 5 个变异探针各判红：去掉实体 ID 一致、去掉状态一致、去掉 `valid_from` 一致、封装不调用一致性校验、组织事实来源域改为 hr。
- `check:d3` 通过；去掉 PKG-005 后判红（6 项）。
- `check:app-manifest`、`check:contract-consumers`、`check:validation`、lint、typecheck 均通过。
- `check:runtime` 474/474、`check:ui` 46/46，用一次性 PostgreSQL / Redis 跑完已删。静态检查链如实停在 O22，`check:governance` 只剩 O22。
- 远端 CI 结果见本页。
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793153681
|
1793194416
|
0
|
0
|
1
|
0
|
Edit
Delete
|
|
848
|
83
|
5
|
5
|
|
0
|
docs(governance): record first remote run past O18 docs(governance): record first remote run past O18 and the stale acceptance provenance...
|
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的 <!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合当前生命周期规范：既有 Personnel Core 保持 `/api/personnel/*`，HR v2 领域使用 `/api/v2/{domain}/*`，身份协议使用标准 OIDC 端点
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 涉及页面或浏览器行为时，`pnpm check:ui` 与 `pnpm check:ui:fastify` 均通过，未只验证一个后端
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793099401
|
1793099408
|
1793099408
|
0
|
0
|
0
|
Edit
Delete
|
|
853
|
83
|
10
|
5
|
|
0
|
chore(deps): H23 根与 apps/* 对齐框架 0.27.0 安全基线
|
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的 <!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合当前生命周期规范：既有 Personnel Core 保持 `/api/personnel/*`，HR v2 领域使用 `/api/v2/{domain}/*`，身份协议使用标准 OIDC 端点
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 涉及页面或浏览器行为时，`pnpm check:ui` 与 `pnpm check:ui:fastify` 均通过，未只验证一个后端
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793114286
|
1793115451
|
1793114295
|
0
|
0
|
0
|
Edit
Delete
|
|
854
|
83
|
11
|
5
|
|
0
|
docs(os-product): 回灌 OS 侧 0.19.6 导入结果与 relay 整表授权放 docs(os-product): 回灌 OS 侧 0.19.6 导入结果与 relay 整表授权放宽原因（H11）...
|
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的 <!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合当前生命周期规范：既有 Personnel Core 保持 `/api/personnel/*`，HR v2 领域使用 `/api/v2/{domain}/*`，身份协议使用标准 OIDC 端点
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 涉及页面或浏览器行为时，`pnpm check:ui` 与 `pnpm check:ui:fastify` 均通过，未只验证一个后端
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793143165
|
1793143171
|
1793143171
|
0
|
0
|
0
|
Edit
Delete
|
|
855
|
83
|
12
|
5
|
|
0
|
fix(os-product): D-9 发布审计沿快照本机远端解析到真实 OS 仓，只读 ls-r fix(os-product): D-9 发布审计沿快照本机远端解析到真实 OS 仓，只读 ls-remote 其发布远端...
|
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的 <!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合当前生命周期规范：既有 Personnel Core 保持 `/api/personnel/*`，HR v2 领域使用 `/api/v2/{domain}/*`，身份协议使用标准 OIDC 端点
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 涉及页面或浏览器行为时，`pnpm check:ui` 与 `pnpm check:ui:fastify` 均通过，未只验证一个后端
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793145336
|
1793145343
|
1793145343
|
0
|
0
|
0
|
Edit
Delete
|
|
857
|
83
|
14
|
5
|
|
0
|
含五件事：H20 台账收敛的记录、H11 回灌、PKG-002 vitest 升级、D-9，以及这次 含五件事：H20 台账收敛的记录、H11 回灌、PKG-002 vitest 升级、D-9，以及这次的冻结测试放行加 PKG-003。...
|
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的 <!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合当前生命周期规范：既有 Personnel Core 保持 `/api/personnel/*`，HR v2 领域使用 `/api/v2/{domain}/*`，身份协议使用标准 OIDC 端点
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 涉及页面或浏览器行为时，`pnpm check:ui` 与 `pnpm check:ui:fastify` 均通过，未只验证一个后端
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1793147366
|
1793147366
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
859
|
83
|
16
|
5
|
|
0
|
feat(contracts): 出站事实来源域按事实类型收紧（V-04，D-12 台账 PKG-0 feat(contracts): 出站事实来源域按事实类型收紧（V-04，D-12 台账 PKG-004）...
|
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的 <!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合当前生命周期规范：既有 Personnel Core 保持 `/api/personnel/*`，HR v2 领域使用 `/api/v2/{domain}/*`，身份协议使用标准 OIDC 端点
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 涉及页面或浏览器行为时，`pnpm check:ui` 与 `pnpm check:ui:fastify` 均通过，未只验证一个后端
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1793153388
|
1793191535
|
1793153395
|
0
|
0
|
0
|
Edit
Delete
|
|
691
|
50
|
1
|
5
|
|
0
|
fix(web): update Lyra product labels
|
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的 <!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合 `/api/v1/{resources}` 规范
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1783327279
|
1783431501
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
692
|
57
|
1
|
5
|
|
0
|
feat(material-factory): 图像模型真实推理对接 + 双后端渲染管线闭环
|
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的 <!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合 `/api/v1/{resources}` 规范
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1783381428
|
1783381594
|
1783381594
|
0
|
0
|
0
|
Edit
Delete
|
|
720
|
67
|
1
|
5
|
|
0
|
治理: 关闭对抗性验收发现的假绿装置、导出层静默数据损坏与文档幽灵资产
|
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的 <!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合 `/api/v1/{resources}` 规范
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
1
|
1
|
0
|
|
0
|
1785285580
|
1785285593
|
1785285593
|
0
|
0
|
0
|
Edit
Delete
|
|
731
|
57
|
2
|
5
|
|
0
|
chore: add material factory model weights
|
<!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的 <!--
PR 治理模板。提交前请逐项确认，确保符合 docs/standards 下的工程规范。
CI 会执行：pnpm check（命名 + 单源漂移 + 治理棘轮 + lint + typecheck）；
有测试 DB 的流水线另执行 DATABASE_URL=... pnpm check:runtime。
-->
## 变更内容
<!-- 简述这次改了什么、为什么改（聚焦 why） -->
-
## 涉及业务概念
<!-- 列出涉及的业务词典英文名，例如 store / order / verification。参见 docs/domain-glossary.md -->
-
## 检查清单
### 命名一致性
- [ ] 业务概念使用了词典中的标准英文名（未出现 shop/branch/seller/voucher/writeOff 等禁用词）
- [ ] 数据库 snake_case + 复数表名；API/JSON 字段 camelCase；类型 PascalCase；常量 SCREAMING_SNAKE_CASE
- [ ] `pnpm check:naming` 通过
- [ ] `pnpm check:schema` 通过，未新增第二份状态/契约真源
### API / 契约
- [ ] 不涉及 API 变更
- [ ] 新增/修改了 API，路由符合 `/api/v1/{resources}` 规范
- [ ] 已更新 `packages/contracts` 中的 Zod schema / DTO / 类型（前后端共用同一份）
- [ ] 无破坏前端兼容的字段删除/重命名（如有，已在描述中说明迁移方案）
### 数据库
- [ ] 不涉及数据库变更
- [ ] 新增/修改了表或字段，已通过 migration review（表名复数、字段 snake_case、外键 xxx_id、时间 _at、状态 status）
- [ ] 高增长表已考虑分区；涉及多租户的表带 tenant_id
- [ ] 写链显式带 tenant_id；跨租户读写按 404/隔离口径处理
### 数据库迁移（迁移即代码：版本/顺序/内容/环境/发布/回滚一致）
- [ ] 改 schema.prisma 的同时**已生成并提交 migration**（`pnpm db:migrate:dev --name <change>`），未只改 schema 不落迁移
- [ ] **未修改已发布过的历史 migration 文件**（内容一致：历史不可变，新变更走新迁移）
- [ ] `pnpm check:migrations` 通过（迁移历史在位 / 无未豁免高危 DDL / schema 表、enum、`@map` 列均已落迁移）
- [ ] 破坏性变更（删表/删列/改名/改类型/加 NOT NULL/加唯一约束）走 **expand→migrate→contract** 三阶段，不一次切
- [ ] 大表加索引用 `CREATE INDEX CONCURRENTLY`；大批量回填走 BullMQ 分批任务，不塞进 migration
- [ ] 高危但确需的 DDL 已加 `-- migration-allow:<原因>` 留痕，并在下方回滚方案说明
- [ ] 非开发环境只用 `migrate deploy`（禁止 `db push` / 线上手动 ALTER）；发布前 `pnpm db:migrate:status` 无 drift
### 实时 / 队列
- [ ] 不涉及
- [ ] 关键业务事件进入 BullMQ 或持久化事件表（未仅依赖 Redis Pub/Sub）
- [ ] 写库事件与真实写操作在同一 tx 写入 outbox，未在 commit 后旁路 publish
- [ ] 实时通道选型符合规范（订单/看板/通知用 SSE；设备在线/包间控制/IM 用 WebSocket）
### 质量与发布
- [ ] 通过 `pnpm lint` 与 `pnpm typecheck`
- [ ] 通过 `pnpm check:governance`，`reports/*.latest.json` 已刷新且无棘轮回退
- [ ] 涉及写链 / DB / 租户 / 状态机时，通过 `DATABASE_URL=... pnpm check:runtime`，`reports/runtime-acceptance.latest.json` 已刷新
- [ ] 已补充/更新必要的测试
- [ ] 不需要回滚方案
- [ ] 需要回滚方案，已在下方说明
## 回滚方案 / 其他说明
-
...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1786603028
|
1786603028
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
822
|
124
|
16
|
5
|
|
0
|
docs: 开发默认值口头确认，并刷新验收矩阵
|
- required-business-decisions.md：开发默认值表标为“2026-10- - required-business-decisions.md：开发默认值表标为“2026-10-09 已口头确认，待正式审批记录”，包括秒杀迟到支付转人工退款核对、转写中断片段不能校对两条补充规则；仍待定的部分（门店地址随 BD01、暂锁到期是否通知、可校对岗位、真实开票能力）单独写明。
- 这只是口头确认：代码里的策略仍是 approved:false，要等正式批准记录到位后再改；没有启用任何真实支付、设备、平台、开票或语音识别能力。
- 重新生成验收矩阵：#12/#13 合并后 main 上的矩阵已过期，重跑 903/903 通过，--check 恢复为 0。
🤖 Generated with [Claude Code](https://claude.com/claude-code)...
|
0
|
0
|
0
|
1
|
0
|
|
0
|
1791588968
|
1791588968
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
35
|
17
|
5
|
1
|
|
0
|
🔍 代码审查报告：pctest 分支 - ce12
|
## 自动代码审查报告
**分支**: pctest
**提交**: `e03027962aed5 ## 自动代码审查报告
**分支**: pctest
**提交**: `e03027962aed59b0c87ce41b201b6905760521d6`
**时间**: 2026-03-25 10:24:19
**审查模型**: qwen3.5-plus
---
[2026-03-25 10:23:14] 正在调用 Qwen API 进行代码审查...
[2026-03-25 10:24:19] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260325-102314.md
### alilog.php 中的引用:
1. **外部依赖库**:
* **文件路径**: `/mnt/data/www/CommonClass/aliyun-log-php-sdk-master/Log_Autoload.php`
* **说明**: 该文件位于项目根目录之外（`/mnt/data/www/CommonClass/...`），引入了阿里云日志服务 PHP SDK 的自动加载文件。这意味着项目运行环境依赖于此特定路径下的 SDK 代码。
2. **使用的 SDK 类**:
代码中实例化或引用了以下阿里云 SDK 类（均通过 `Log_Autoload.php` 加载）：
* `Aliyun_Log_Client`: 客户端主类，用于发起请求。
* `Aliyun_Log_Models_LogItem`: 日志项模型。
* `Aliyun_Log_Models_PutLogsRequest`: 写入日志请求模型。
* `Aliyun_Log_Exception`: SDK 异常类。
* `Aliyun_Log_Models_ListLogstoresRequest`: 列举 Logstore 请求。
* `Aliyun_Log_Models_ListTopicsRequest`: 列举 Topic 请求。
* `Aliyun_Log_Models_GetLogsRequest`: 查询日志请求。
* `Aliyun_Log_Models_LogStoreSqlRequest`: LogStore SQL 查询请求。
* `Aliyun_Log_Models_GetProjectLogsRequest`: 项目级日志查询请求。
* `Aliyun_Log_Models_ProjectSqlRequest`: 项目级 SQL 查询请求。
* `Aliyun_Log_Models_GetHistogramsRequest`: 直方图查询请求。
* `Aliyun_Log_Models_ListShardsRequest`: 列举 Shard 请求。
* `Aliyun_Log_Models_GetCursorRequest`: 获取游标请求。
* `Aliyun_Log_Models_BatchGetLogsRequest`: 批量获取日志请求。
* `Aliyun_Log_Models_MergeShardsRequest`: 合并 Shard 请求。
* `Aliyun_Log_Models_SplitShardRequest`: 分裂 Shard 请求。
3. **项目内部引用**:
* **无**: 该文件未引用项目 `application/` 目录下的任何控制器、模型、配置文件或辅助函数。
* **配置独立**: 脚本底部硬编码了阿里云的 `endpoint`, `accessKeyId`, `accessKey`, `project`, `logstore` 等信息，未使用 `application/config/` 下的配置文件（如 `config.php` 或 `redis.php` 等）。
* **独立性**: 该文件看起来像是一个独立的测试脚本或工具脚本，尚未集成到项目的 MVC 流程中。
4. **代码问题与风险提示**:
* **严重安全风险**: 文件中明文硬编码了阿里云 AccessKey ID (`LTAInvzKfeu0j7KN`) 和 AccessKey Secret (`RH94sjMSAPx72l6tTq2NtMnl7uuwLM`)。**这属于严重的安全泄露，建议立即在阿里云控制台禁用该密钥并轮换。**
* **语法错误/残留代码**:
* 第 12 行：`function putLogs(...) { sss1111` 中存在无意义的字符串 `sss1111`，会导致 PHP 解析错误。
* 第 336 行：`print(..., nextCursor, ...)` 中变量 `nextCursor` 缺少 `$` 符号，应为 `$nextCursor`，否则会导致未定义变量警告或错误。
* **调试代码泄露**: 文件中包含大量 `var_dump`, `print`, `logVarDump` 调试输出，生产环境中应移除或改为日志记录。
* **路径依赖**: 依赖绝对路径 `/mnt/data/www/...`，降低了代码的可移植性，建议改为相对路径或通过配置文件管理 SDK 路径。
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码���照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1774405459
|
1774405459
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
23
|
17
|
1
|
1
|
|
0
|
🔍 代码审查报告：pctest 分支 - 修改单个文件测试
|
## 自动代码审查报告
**分支**: pctest
**提交**: `b23272882d68e ## 自动代码审查报告
**分支**: pctest
**提交**: `b23272882d68e0b61ab119ff0f670c4dbbe7a652`
**时间**: 2026-03-20 15:06:01
**审查模型**: qwen3.5-plus
---
[2026-03-20 15:04:58] 正在调用 Qwen API 进行代码审查...
[2026-03-20 15:06:01] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260320-150458.md
基于您提供的项目结构和代码变更内容，以下是对 `application/controllers/AdMachine.php` 文件的代码分析及潜在问题审查：
### 1. 文件概述
* **功能**: 该控制器主要用于管理广告机（AdMachine）的屏保内容，提供了获取 (`getScreensaver`) 和设置 (`setScreensaver`) 屏保内容的接口。
* **框架**: 基于 CodeIgniter 框架（通过 `BASEPATH`, `FCPATH`, `load->model` 等特征判断）。
* **继承关系**: 继承自 `PcServer` 控制器 (`application/controllers/PcServer.php`)。
### 2. 代码审查与潜在风险
#### ⚠️ 高风险：模型名称拼写可疑
在 `getScreensaver` 方法中，加载了两个非常相似的模型：
1. `Ahead_shop_ad_machine_set_modelsss` (注意末尾的三个 's')
2. `Ahead_shop_ad_machine_set_model`
```php
$this->load->model('Ahead_shop_ad_machine_set_modelsss');
$this->Ahead_shop_ad_machine_set_modelsss->set("dw"); // 疑似 typo
$this->load->model('Ahead_shop_ad_machine_set_model');
$res = $this->Ahead_shop_ad_machine_set_model->get_datas(...);
```
* **问题**: `Ahead_shop_ad_machine_set_modelsss` 极大概率是 `Ahead_shop_ad_machine_set_model` 的**拼写错误**。
* **影响**: 如果项目中不存在 `Ahead_shop_ad_machine_set_modelsss.php` 模型文件，代码运行时会报错。如果存在，需确认其用途（是否用于设置某种特定状态 "dw"？），否则应予以修正以减少维护困惑。
#### ⚠️ 架构规范：控制器继承控制器
* **现状**: `AdMachine` 继承自 `PcServer`，且在文件头部手动 `include` 了父控制器文件。
* **建议**: 在 CodeIgniter 标准实践中，通常建议创建一个基类控制器放在 `application/core/` 目录下（例如 `MY_Controller.php`），而不是直接在 `controllers` 目录下让业务控制器相互继承。
* **依赖耦合**: 该类强依赖于父类 `PcServer` 中定义的属性（`$this->merchant_id`, `$this->param`, `$this->stream`）和方法（`success_response()`）。如果父类逻辑变更，此类极易受影响。
#### ⚠️ 模型加载方式
* **现状**: 模型在方法内部按需加载。
* **建议**: 如果这两个模型在该控制器的多个方法中都会用到，建议在 `__construct` 构造函数中统一加载，以提高代码整洁度。
### 3. 跨文件引用总结
根据提供的变更内容，该文件依赖以下外部资源：
| 类型 | 名称 | 路径推测 | 用途 |
| :--- | :--- | :--- | :--- |
| **父类控制器** | `PcServer` | `application/controllers/PcServer.php` | 提供基础请求处理、参数解析 (`$this->param`) 及响应格式 (`success_response`) |
| **模型 (疑似)** | `Ahead_shop_ad_machine_set_modelsss` | `application/models/` | 用于调用 `set("dw")` 方法 (需确认是否存在) |
| **模型** | `Ahead_shop_ad_machine_set_model` | `application/models/` | 用于获取 (`get_datas`) 和设置 (`set_data`) 屏保数据 |
### 4. 优化建议代码示例
如果确认 `modelsss` 是笔误，且希望规范代码，建议修改如下：
```php
<?php
defined('BASEPATH') OR exit('No direct script access allowed');
// 建���：确保 PcServer 作为基类被正确 autoload 或放在 core 目录，避免手动 include
// include FCPATH . 'application' . DIRECTORY_SEPARATOR . 'controllers' . DIRECTORY_SEPARATOR . 'PcServer.php';
class AdMachine extends PcServer
{
public function __construct()
{
parent::__construct();
// 建议：在构造函数中加载常用模型
$this->load->model('Ahead_shop_ad_machine_set_model');
// 如果 modelsss 确实存在且必要，也在此加载，否则删除
// $this->load->model('Ahead_shop_ad_machine_set_modelsss');
}
public function getScreensaver()
{
// 修正潜在的拼写错误，确认 set("dw") 是否需要在主模型上调用
// $this->Ahead_shop_ad_machine_set_model->set("dw");
$res = $this->Ahead_shop_ad_machine_set_model->get_datas($this->merchant_id, $this->param);
$this->stream['request']['result'] = $res;
$this->success_response();
}
public function setScreensaver()
{
$this->Ahead_shop_ad_machine_set_model->set_data($this->merchant_id, $this->param);
$this->success_response();
}
}
```
### 5. 待确认事项
1. **确认模型文件是否存在**: 请检查 `application/models/` 目录下是否有 `Ahead_shop_ad_machine_set_modelsss.php`。
2. **确认 "dw" 参数含义**: 确认 `$this->Ahead_shop_ad_machine_set_modelsss->set("dw")` 的具体业务逻辑，是否可合并到主模型中。
3. **确认父类依赖**: 确保 `PcServer.php` 中已正确初始化 `$this->merchant_id` 和 `$this->param`，否则此类方法会运行失败。
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码快照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773990362
|
1773990362
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
24
|
17
|
2
|
1
|
|
0
|
🔍 代码审查报告：pctest 分支 - 修改单个文件测试2
|
## 自动代码审查报告
**分支**: pctest
**提交**: `7fbec3ef95a38 ## 自动代码审查报告
**分支**: pctest
**提交**: `7fbec3ef95a3856eef25ceabf266030d5ca445e4`
**时间**: 2026-03-20 15:24:52
**审查模型**: qwen3.5-plus
---
[2026-03-20 15:23:33] 正在调用 Qwen API 进行代码审查...
[2026-03-20 15:24:52] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260320-152333.md
基于您提供的项目结构、`Book.php` 控制器代码以及跨文件引用分析，以下是对该文件的详细分析报告。这份报告补充了“项目中定义的类和方法”部分，并对代码质量、潜在风险及优化建议进行了审查。
## 1. 文件功能概述
**文件路径**: `application/controllers/Book.php`
**类名**: `Book`
**继承**: `PcServer`
**主要功能**:
该控制器主要用于处理**微信预订单**和**智慧商家 APP 预定**相关的业务逻辑。功能涵盖预订单的列表查询、详情查看、数据导出（Excel/PDF）以及退款操作。它区分了两种主要的业务场景：
1. **微信预订单** (`getList`, `getDetail`, `export`, `refund`)：基于 `ahead_book_order_model`。
2. **智慧商家 APP 预定** (`getBookList`, `getBookDetail`, `exportBook`)：基于 `ahead_book_model`。
## 2. 项目中定义的类和方法 (补充)
根据提供的 `Book.php` 内容，补充该文件中定义的类和方法信息：
### PHP 类:
| 类名 | 继承自 | 文件路径 | 描述 |
| :--- | :--- | :--- | :--- |
| `Book` | `PcServer` | `application/controllers/Book.php` | 预订单管理控制器，处理列表、详情、导出及退款逻辑 |
### 公共方法列表:
| 方法名 | 对应业务 | 描述 |
| :--- | :--- | :--- |
| `getList()` | 微信预订单 | 获取微信预订单列表，支持多条件筛选、分页及总金额统计 |
| `getDetail()` | 微信预订单 | 获取单个微信预订单的详细信息 |
| `export()` | 微信预订单 | 导出微信预订单报表 (Excel/PDF) |
| `refund()` | 微信预订单 | 执行预订单退款操作 |
| `getBookList()` | 智慧商家 APP | 获取商家端预订单列表 |
| `getBookDetail()` | 智慧商家 APP | 获取商家端预订单详情 |
| `exportBook()` | 智慧商家 APP | 导出商家端预订单报表 (Excel/PDF) |
## 3. 依赖项分析
### 引用的模型 (Models)
| 模型名 | 用途 | 备注 |
| :--- | :--- | :--- |
| `ahead_setting_clean_data_model` | 数据清洗/日期处理 | 仅在 `getList` 中用于处理清理日期 |
| `ahead_book_order_model` | 微信预订单核心逻辑 | 用于列表、详情、导出、退款 |
| `ahead_book_model` | 商家 APP 预订单核心逻辑 | 用于商家端列表、详情、导出 |
| `Export_model` / `export_model` | 通用导出逻辑 | **注意：存在大小写不一致** |
| `Ahead_finance_report_setting_model` | 导出配置保存 | 用于记录导出设置 |
### 引用的类库 (Libraries)
| 类库路径 | 用途 |
| :--- | :--- |
| `PHPExcel/Classes/MyPHPExcel.php` | 手动生成 Excel 文件 | 仅在 `export` 方法后半段使用 |
### 引用的助手函数 (Helpers)
| 助手名 | 用途 |
| :--- | :--- |
| `check` | 权限或数据校验 | 在 `getList` 和 `export` 中加载 |
## 4. 代码审查与潜在问题 (Code Review)
在分析 `Book.php` 代码后，发现以下几个值得注意的问题和优化建议：
### 4.1 严重逻辑冗余 (Critical)
**位置**: `export()` 方法 (第 220 行 - 第 270 行)
**问题**: 该方法中存在**两段独立的 Excel 生成逻辑**。
1. 第一段 (约 220 行)：调��� `$this->Export_model->export2Excel` 生成对象并保存文件。
2. 第二段 (约 237 行)：重新加载 `MyPHPExcel` 库，实例化新对象，手动设置单元格内容，再次保存文件。
**风险**:
* 第二段逻辑覆盖了第一段的文件名生成规则（第一段包含 `merchant_id` 和 `admin_id`，第二段没有）。
* 造成性能浪费，且第二段手动设置单元格的逻辑可能与第一段 `Export_model` 处理的数据不一致。
* **建议**: 删除第二段手动生成 Excel 的代码，统一使用 `Export_model` 封装好的方法，或确认是否为遗留代码未清理。
### 4.2 命名规范不一致 (Consistency)
**位置**: 模型加载
**问题**:
* 在 `export()` 方法中：`$this->load->model('Export_model');` (大写 E)
* 在 `exportBook()` 方法中：`$this->load->model('export_model');` (小写 e)
**风险**: 虽然 PHP 类名通常不区分大小写，但在文件系统区分大小写的环境（如 Linux）下可能导致加载失败。
**建议**: 统一模型命名规范，建议全部使用小写或遵循 PSR 标准。
### 4.3 输入参数获取方式不统一 (Input Handling)
**位置**: 方法参数获取
**问题**:
* `getList()`: 使用 `$this->param` (通常来自框架封装的输入处理)。
* `export()` & `exportBook()`: 直接使用 `$_GET`。
**风险**:
* 直接使用 `$_GET` 绕过了框架可能提供的安全过滤或统一参数处理机制。
* 导致代码风格不一致，维护困难。
**建议**: 统一使用 `$this->param` 或框架推荐的输入获取方式。
### 4.4 错误处理方式不统一 (Error Handling)
**位置**: `export()` 方法
**问题**:
* 大部分方法使用 `$this->error_response()` 返回标准 JSON 错误。
* `export()` 方法中多处使用 `exit('...')` 直接终止脚本。
**风险**: 直接 `exit` 会导致前端无法接收到标准的错误响应格式，难以进行统一的错误提示处理。
**建议**: 统一使用 `$this->error_response()`。
### 4.5 代码复用性 (DRY Principle)
**位置**: `getList()` 与 `export()`
**问题**: 这两个方法中关于参数解析、权限校验、`where` 条件构建的逻辑高度重复（约 80% 代码相似）。
**风险**: 如果业务规则变更（例如增加一个筛选条件），需要同时修改两处，容易遗漏。
**建议**: 将构建查询条件的逻辑提取为私有方法（例如 `_buildBookQueryWhere()`），供 `getList` 和 `export` 共同调用。
### 4.6 硬编码 (Hardcoding)
**位置**: `getList()`
**问题**: `$user_type = $this->user_type ? $this->user_type : '2';`
**风险**: 默认用户类型硬编码为 '2'，建议通过配置文件或常量管理，以便后续扩展。
## 5. 总结
`Book.php` 是一个功能丰富的控制器，涵盖了预订单管理的核心流程。但代码中存在明显的**重复逻辑**和**不一致的编码风格**，特别是在 `export` 方法中存在疑似冗余的 Excel 生成代码，建议优先清理和优化，以提高代码的可维护性和稳定性。
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码快照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773991492
|
1773991492
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
34
|
17
|
4
|
1
|
|
0
|
🔍 代码审查报告：pctest 分支 - ce1
|
## 自动代码审查报告
**分支**: pctest
**提交**: `6a1850c0a6b2b ## 自动代码审查报告
**分支**: pctest
**提交**: `6a1850c0a6b2b9a5c6f910af5ddecbf2d99c8a96`
**时间**: 2026-03-25 09:56:34
**审查模型**: qwen3.5-plus
---
[2026-03-25 09:55:38] 正在调用 Qwen API 进行代码审查...
[2026-03-25 09:56:34] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260325-095538.md
### alilog.php 中的引用:
1. **外部 SDK 依赖**:
* **文件路径**: `/mnt/data/www/CommonClass/aliyun-log-php-sdk-master/Log_Autoload.php`
* **说明**: 通过 `require_once` 引入了阿里云日志服务（SLS）PHP SDK 的自动加载文件。这是一个位于项目目录结构之外（`/mnt/data/www/CommonClass/`）的公共类库依赖。
2. **依赖的类 (来自阿里云 SDK)**:
该文件大量实例化或使用了以下 SDK 类，均依赖于上述引入的 `Log_Autoload.php`：
* `Aliyun_Log_Client`: 核心客户端类，用于建立与阿里云日志服务的连接。
* `Aliyun_Log_Models_LogItem`: 用于构建单条日志内容。
* `Aliyun_Log_Models_PutLogsRequest`: 写入日志的请求对象。
* `Aliyun_Log_Exception`: 捕获 SDK 抛出的特定异常。
* `Aliyun_Log_Models_ListLogstoresRequest`: 列举 Logstore 的请求对象。
* `Aliyun_Log_Models_ListTopicsRequest`: 列举 Topic 的请求对象。
* `Aliyun_Log_Models_GetLogsRequest`: 查询普通日志的请求对象。
* `Aliyun_Log_Models_LogStoreSqlRequest`: 执行 Logstore SQL 查询的请求对象。
* `Aliyun_Log_Models_GetProjectLogsRequest`: 执行项目级日志查询的请求对象。
* `Aliyun_Log_Models_ProjectSqlRequest`: 执行项目级 SQL 查询的请求对象。
* `Aliyun_Log_Models_GetHistogramsRequest`: 获取日志直方图的请求对象。
* `Aliyun_Log_Models_ListShardsRequest`: 列举 Shard 分片的请求对象。
* `Aliyun_Log_Models_GetCursorRequest`: 获取 Shard 游标的请求对象。
* `Aliyun_Log_Models_BatchGetLogsRequest`: 批量拉取日志的请求对象。
* `Aliyun_Log_Models_MergeShardsRequest`: 合并 Shard 的请求对象。
* `Aliyun_Log_Models_SplitShardRequest`: 分裂 Shard 的请求对象。
3. **项目内部文件引用**:
* **无**。该文件未引用项目结构列表中的任何文件（如 `application/config/config.php`, `application/controllers/*` 等）。
* 它没有使用项目的配置管理类来获取数据库连接、Redis 配置或阿里云密钥，而是直接在脚本底部硬编码了配置信息。
4. **配置耦合**:
* **硬编码凭证**: 文件中直接写死了 `endpoint`, `accessKeyId`, `accessKey`, `project`, `logstore` 等敏感配置信息。
* **影响**: 这导致该文件与环境强耦合，无法通过项目的配置文件（如 `application/config/` 下的文件）动态切换环境（开发/生产），且存在安全风险。
5. **执行模式**:
* 文件末尾包含直接执行代码（实例化 `Aliyun_Log_Client` 并调用 `putLogs`），表明该文件不仅是一个函数库，被访问时还会立即执行日志写入操作。
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码快照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1774403794
|
1774403794
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
25
|
17
|
3
|
1
|
|
0
|
🔍 代码审查报告：pctest 分支 - 修改单个文件测试3
|
## 自动代码审查报告
**分支**: pctest
**提交**: `12a4a4ad441e2 ## 自动代码审查报告
**分支**: pctest
**提交**: `12a4a4ad441e2edf19328cb83a4d1532f1985c69`
**时间**: 2026-03-20 15:30:26
**审查模型**: qwen3.5-plus
---
[2026-03-20 15:29:22] 正在调用 Qwen API 进行代码审查...
[2026-03-20 15:30:25] 审查结果已保存到：/root/gitea-code-review/logs/results/code-review-20260320-152922.md
基于您提供的项目结构和 `application/controllers/Book.php` 的文件内容，以下是对该控制器的详细代码分析报告。这份报告涵盖了功能概述、潜在缺陷、安全性问题以及优化建议。
### 1. 功能概述
`Book` 控制器主要处理与**预订单（Booking Orders）**相关的业务逻辑，分为两个主要板块：
1. **微信预订单管理**：包括列表查询 (`getList`)、详情查看 (`getDetail`)、导出报表 (`export`) 和退款处理 (`refund`)。
2. **智慧商家 App 预定**：包括列表查询 (`getBookList`)、详情查看 (`getBookDetail`) 和导出报表 (`exportBook`)。
该控制器继承自 `PcServer`，表明它依赖于父类提供的权限验证、参数处理 (`$this->param`) 和响应格式 (`success_response`, `error_response`) 等基础功能。
### 2. 代码缺陷与潜在 Bug
在代码审查中发现了以下几个明显的问题，建议优先修复：
* **数组键名拼写错误 (高危)**
* **位置**: `getList()` 方法，第 23 行。
* **代码**: `$page_size = isset($param['page_size's]) ? ...`
* **问题**: 键名写成了 `'page_size's`（多了一个 `s` 且在字符串内），导致无法正确获取前端传递的分页大小参数，始终默认值为 20。
* **修复**: 应改为 `$param['page_size']`。
* **变量名不一致 (中危)**
* **位置**: `getList()` 方法，第 16 行。
* **代码**: `$merchant_id = $this->merchant_id ? $this->merchant_ids : ...`
* **问题**: 判断使用的是 `$this->merchant_id`，但赋值使用的是 `$this->merchant_ids`（复数）。如果父类 `PcServer` 中只定义了 `merchant_id`，这里会导致 `$merchant_id` 为空或报错。
* **修复**: 确认父类属性名，通常应统一为 `$this->merchant_id`。
* **响应流中断 (中危)**
* **位置**: `export()` 和 `exportBook()` 方法。
* **代码**: `exit('导出字段不能为空');` 和 `exit($exportRes['msg']);`
* **问题**: 直接使用 `exit` 会输出纯文本，破坏了项目统一的 JSON 响应格式。前端接收到的将是非 JSON 数据，导致解析错误。
* **修复**: 应使用 `$this->error_response('...')` 或类似的标准错误处理方法。
* **模型加载大小写不一致**
* **位置**: `export()` 方法。
* **代码**: `$this->load->model('Export_model');` vs `$this->load->model('export_model');` (在 `exportBook` 中)
* **问题**: 虽然 Linux 文件系统区分大小写，但为了规范性和兼容性，建议统一模型命名的大小写（通常首字母大写或全小写需与文件名一致）。
### 3. 安全性分析
* **超全局变量直接使用**
* **位置**: `export()` 和 `exportBook()` 方法。
* **代码**: `$param = $_GET;`
* **风险**: `getList()` 使用了 `$this->param`（通常经过过滤），但导出方法直接使用 `$_GET`。这可能绕过父类中的参数清洗或安全验证逻辑，存在 SQL 注入或 XSS 风险（尽管模型层可能做了防护，但控制器层应保持一致）。
* **建议**: 统一使用 `$this->param` 或经过验证的输入变量。
* **文件导出路径与权限**
* **位置**: `export()` 方法末尾。
* **代码**: `$objPHPExcel->saveFile($filename);`
* **风险**: 文件保存路径未明确显示（取决于 `saveFile` 实现）。如果文件保存在 Web 根目录且未加访问限制，可能导致敏感数据泄露。
* **建议**: 确保导出文件存储在非公开目录，或通过脚本流式输出后删除，避免遗留敏感文件。
* **硬编码的 Excel 列索引**
* **位置**: `export()` 方法。
* **代码**: `$objPHPExcel->getActiveSheet()->setCellValue('H' . $rowCount, ...)`
* **风险**: 硬编码列号（如 'H', 'I'）。如果 `fields` 数组顺序发生变化，合计金额可能会错位到错误的列。
* **建议**: 动态计算列索引。
### 4. 依赖关系分析
根据代码内容，该控制器强依赖以下模型和库：
| 依赖项 | 类型 | 用途 | 备注 |
| :--- | :--- | :--- | :--- |
| `PcServer` | 控制器父类 | 基础权限、响应格式 | 手动 include 引入 |
| `ahead_book_order_model` | 模型 | 微信预订单 CRUD、退款、统计 | 核心业务逻辑 |
| `ahead_book_model` | 模型 | 智慧商家 App 预订单逻辑 | 核心业务逻辑 |
| `Export_model` / `export_model` | 模型 | 生成 Excel/PDF 文件 | 注意大小写一致性 |
| `Ahead_finance_report_setting_model` | 模型 | 保存导出配置记录 | 审计用途 |
| `ahead_setting_clean_data_model` | 模型 | 数据清洗日期策略 | 合规/数据保留策略 |
| `PHPExcel` | 第三方库 | Excel 文件操作 | **已废弃**，建议迁移至 PhpSpreadsheet |
### 5. 优化与重构建议
1. **修复致命 Bug**: 立即修复 `getList` 中的 `'page_size's` 拼写错误和 `$merchant_ids` 变量名问题。
2. **统一出口规范**: 移除所有 `exit()` 调用，改为返回标准 JSON 错误响应。
3. **代码复用**: `export()` 和 `exportBook()` 中有大量重复的 Excel 生成逻辑（表头设置、样式、保存）。建议提取到一个公共 Helper 方法或父类方法中。
4. **库升级**: `PHPExcel` 库已不再维护，存在安全漏洞且不支持新版 PHP。建议迁移至 `PhpSpreadsheet`。
5. **参数验证**: 在控制器层增加对 `$param` 的严格类型验证（如 `shop_id` 必须为整数），减轻模型层压力。
6. **权限检查**: 确保 `refund()` 方法也进行了严格的权限校验（当前代码直接调用模型，未见明显的权限检查逻辑，依赖模型内部实现可能存在风险）。
### 6. 总结
`Book.php` 是一个业务逻辑密集的控制器，涵盖了查询、导出和退款功能。虽然功能完整，但存在明显的拼写错误和不规范的代码实践（如直接 `exit`、混用 `$_GET`）。建议在进行下一步开发前，先完成上述的 Bug 修复和代码规范化工作，以确保系统的稳定性和安全性。
---
*此 Issue 由代码审查服务自动创建 | 审查基于提交时的代码快照*
...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1773991826
|
1773991826
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
629
|
18
|
196
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 前端编译
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `ffb241600a ## 自动代码审查报告
**分支**: pc-260616
**提交**: `ffb241600a5f4bc792f5a1d30759c2b77443372e`
**提交人**: caihongyuchy (1091045324@qq.com)
**时间**: 2026-06-09 13:58:05
---
## 📋 审查摘要
- **变更文件数**: 1
- **严重问题**: 2
- **高危问题**: 4
- **中危问题**: 3
- **建议优化**: 2
## 🐛 发现的问题
### <font color="red">[语法错误] 代码文件不完整/缺少闭合标签</font>
- **严重程度**: <font color="red">严重</font>
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: 文件末尾
- **问题描述**: 代码在 `getChecksGoodsAfterChangeEndTime` 方法的 `success` 回调中突然截断，缺失了闭合的大括号 `}`、`</script>` 以及 `</template>`。这将直接导致 Vue 单文件组件编译失败或运行时解析错误。
- **修复建议**: 补充完整的方法逻辑、闭合括号及文件结构标签。确保 `methods` 对象和 `export default` 正确闭合。
### <font color="red">[语法错误] 未导入的依赖导致运行时 ReferenceError</font>
- **严重程度**: <font color="red">严重</font>
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: 全文多处（如 `moment()`, `$()`, `layer.msg()`）
- **问题描述**: 代码中大量使用了 `moment`、`$` (jQuery)、`layer` 等全局变量，但在 `<script>` 顶部未进行 `import` 声明。在现代前端工程化环境（Webpack/Vite）中，若未配置全局注入，将直接抛出 `ReferenceError: moment is not defined` 等错误。
- **修复建议**:
```javascript
import Vue from 'vue';
import XLSX from 'xlsx';
import moment from 'moment';
import $ from 'jquery';
import layer from 'layui/layer'; // 或对应UI库路径
// 确保 daterangepicker, select2, bootstrapTable 等 jQuery 插件已通过全局脚本或 npm 包正确引入
```
### <font color="red">[跨文件调用] 调用了未定义的全局 Vue 原型属性</font>
- **严重程度**: <font color="red">高危</font>
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: 全文多处（如 `Vue.request_header`, `Vue.ctUrl`, `Vue.version`, `Vue.timeoutfun`）
- **问题描述**: 频繁通过 `Vue.xxx` 调用自定义属性。这些并非 Vue 官方 API，属于项目自定义的全局挂载。若未在 `main.js` 中通过 `Vue.prototype.$xxx = xxx` 正确挂载，或挂载时机晚于组件加载，将导致跨文件调用失败。
- **修复建议**: 确认入口文件已正确挂载。推荐改为实例调用方式以符合 Vue 规范：`this.$request_header`、`this.$ctUrl` 等，或在组件顶部显式引入配置对象。
### [逻辑 BUG] `$.map` 中的 `return` 无法中断循环或阻止后续执行
- **严重程度**: 高危
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: `saveGoods` 方法 & `getChecksGoods` 方法内
- **问题描述**: 在 `$.map(result, function(row) { ... if(重复) { return; } ... })` 中，`return` 仅表示当前迭代返回 `undefined`，**不会终止循环**，也不会阻止外层函数继续执行。若存在重复商品，`layer.msg` 会连续弹出多次，且重复数据仍可能被错误处理。
- **修复建议**: 改用 `for` 循环或 `Array.prototype.some` 进行前置查重，找到重复项后直接 `return` 终止整个方法：
```javascript
const hasDuplicate = result.some(row =>
_this.checks_goods_list.some(item => item.merchant_goods_id === row.merchant_goods_id)
);
if (hasDuplicate) {
layer.msg("商品已存在");
return;
}
// 后续批量添加逻辑...
```
### [逻辑 BUG] 对象相加导致类型错误与无用代码
- **严重程度**: 高危
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: `addExport` 方法内 `let sheet3 = sheet1 + sheet2;`
- **问题描述**: `sheet1` 和 `sheet2` 是 `XLSX.utils` 生成的 Worksheet 对象。使用 `+` 运算符会触发隐式类型转换，结果为字符串 `"[object Object][object Object]"`。该变量后续未使用，属于明显的逻辑错误与冗余代码。
- **修复建议**: 直接删除 `let sheet3 = sheet1 + sheet2;` 这行代码。
### [逻辑 BUG] `split()` 无参数调用不符合业务预期
- **严重程度**: 高危
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: `addExport` 方法内 `"goods_type_name": _this.add_type_txt.split(),`
- **问题描述**: `String.prototype.split()` 不传参数时，会返回包含原字符串的单元素数组 `['原字符串']`，而非按逗号或其他分隔符拆分。若后端期望的是拆分后的数组或纯字符串，将导致数据格式错误。
- **修复建议**: 根据实际分隔符补充参数，例如 `_this.add_type_txt.split(',')`；若只需传字符串，直接传 `_this.add_type_txt` 即可。
### [代码质量] 组件 `name` 与文件名严重不符
- **严重程度**: 中危
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: `export default { name: 'procurement_return', ... }`
- **问题描述**: 文件名为 `stock_checks.vue`（库存盘点），但组件名称定义为 `'procurement_return'`（采购退货）。这会导致 Vue DevTools 调试信息混乱，影响组件缓存（`keep-alive`）及递归组件调用。
- **修复建议**: 将 `name` 修改为与文件名一致的驼峰命名：`name: 'stock_checks'`。
### [代码质量] 多处方法名拼写错误
- **严重程度**: 中危
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: `getMunu`, `innitDate`, `getDeteil`
- **问题描述**: 存在明显的英文拼写错误：`getMunu` 应为 `getMenu`，`innitDate` 应为 `initDate`，`getDeteil` 应为 `getDetail`。虽不影响运行，但严重降低代码可读性与专业度。
- **修复建议**: 全局搜索并替换为正确拼写，保持命名规范。
### [安全隐患] 潜在 XSS 风险（DOM 操作与数据渲染）
- **严重程度**: 中危
- **文件**: `web/youc_business_operate_pc/src/views/stock_management/stock_checks.vue`
- **行号**: `initTable` 方法内 `formatter` 及多处 `$.each` 拼接
- **问题描述**: 当前 `formatter` 返回硬编码 HTML 是安全的，但代码中大量使用 jQuery 插件（Select2, BootstrapTable）直接渲染数据。若后续在 `formatter` 中拼接 `row` 字段（如 `row.goods_name`）而未做 HTML 转义，极易引发 XSS 攻击。
- **修复建议**: 优先使用 Vue 的模板语法渲染数据。若必须使用 jQuery 插件的 `formatter`，务必对动态数据进行转义：`return '<span class="text-blue detail">' + Vue.escape(row.number) + '</span>';`。
## ✅ 代码亮点
1. **状态管理清晰**：使用 `watch` 监听 `export_checkModel`、`checks_goods_list` 等数组变化，自动联动 `btnClickable` 和全选状态，交互逻辑闭环较好。
2. **API 请求封装规范**：所有 AJAX 请求统一使用 `Vue.request_header`、`Vue.version` 等标准头信息，并统一处理了 `result_code` 和 `timeoutfun`，便于后端鉴权与会话管理。
3. **用户体验细节**：在 `clearStore` 中保留了滚动条位置 `scrollTop`，避免了表格刷新导致的页面跳动，提升了操作流畅度。
## 📝 总体建议
1. **架构规范**：该文件为 Vue 前端组件，**不涉及 PHP CodeIgniter 的模型/控制器直接调用**。但代码中调用的 API 路由（如 `stock/searchStockTakingList`、`PublicData/api_getShopListByPurview`）符合 CI 框架的 `控制器/方法` 路由规范。请确保后端对应 Controller 已正确实现且权限校验完备。
2. **技术栈冲突**：在 Vue 项目中重度依赖 jQuery 及 jQuery 插件（Select2, BootstrapTable, Daterangepicker）属于**反模式**。Vue 的虚拟 DOM 更新机制与 jQuery 的直接 DOM 操作极易产生状态不同步、内存泄漏或事件绑定失效。强烈建议逐步替换为 Vue 生态组件（如 `element-ui`/`ant-design-vue` 的表格、日期选择器、下拉框）。
3. **工程化改造**：务必补充缺失的 `import` 声明，修复文件截断问题，并统一使用 `this.$nextTick` 包裹所有 jQuery DOM 初始化操作，确保 Vue 渲染完成后再执行插件挂载。
4. **代码整洁度**：清理注释掉的废弃代码块（如 `addExport` 中大段注释的 AJAX 请求），修正拼写错误，统一使用严格相等 `===` 替代 `==`，可大幅提升代码可维护性。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780984685
|
1780984685
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
476
|
18
|
183
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 1
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `f954ecb712 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `f954ecb712f639d10ad639cb3c6491c588f9e5f1`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-06-02 18:08:22
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：业务逻辑覆盖较全，报表查询、过滤与导出流程完整。但存在明显的 **N+1 查询性能瓶颈**、**SQL 条件拼接注入风险** 以及 **非面向对象的全局变量调用**。代码风格偏向传统 CodeIgniter 3 写法，部分逻辑依赖隐式类型转换与未声明的全局函数，可维护性与健壮性有待提升。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | L2-L3 / 全局 | `$CI = &get_instance();` 在类外部直接执行。若文件在框架未完全初始化时被 `include/require`，将触发致命错误。且模型加载不应放在全局作用域。 | 移除全局代码，将依赖模型加载移至类构造函数中，遵循 OOP 规范。 | ```php<br>public function __construct()<br>{<br> parent::__construct();<br> $this->load->model('Report_model');<br> // 其他常用模型可在此预加载<br>}<br>``` |
| 🔴 严重 | `get_community_revenues_list()` ~L158-165 | **SQL 注入风险**：`$pay_platform_where` 使用字符串拼接构造 `OR` 条件，并直接传入 `$where['where'][]`。若底层 Query Builder 未做严格转义，恶意构造的 `pay_platform_arr` 可突破过滤。 | 放弃手动拼接 SQL 字符串，改用框架查询构建器的参数绑定（Prepared Statements）或分组条件方法。 | ```php<br>// 推荐改用框架原生分组查询<br>$this->db->group_start();<br>foreach ($params['pay_platform_arr'] as $item) {<br> $parts = explode('_', $item);<br> $this->db->or_group_start()<br> ->where('a._pay_platform', $parts[0])<br> ->where('a._second_pay_platform', $parts[1] ?? null)<br> ->group_end();<br>}<br>$this->db->group_end();<br>``` |
| 🟠 警告 | `get_community_revenues_list()` ~L130-133 | **N+1 查询性能瓶颈**：在 `foreach ($data as &$v)` 循环中，针对 `order_type == '1'` 的每条记录都执行一次 `ahead_book_order_model->get_one()`。数据量大时将导致数据库连接耗尽与响应超时。 | 采用 **批量查询 + 内存映射** 策略。先收集所有需要查询的 `book_order_id`，一次性查出结果，再在循环中通过键值匹配赋值。 | ```php<br>$book_ids = array_unique(array_filter(array_map(function($v) {<br> return $v['order_type'] == '1' ? preg_replace('/\(退款单号:.*\)$/', '', $v['order_id']) : null;<br>}, $data)));<br>$book_orders = $this->ahead_book_order_model->get_data_by_ids($book_ids, '_id,_shop_name,_arrival_time,_end_time', '_id');<br>// 循环内直接 $book_orders[$book_order_id] 取值<br>``` |
| 🟠 警告 | `get_community_revenues_list()` ~L108 | **分页统计逻辑缺陷**：仅当 `$params['page'] == '1'` 时计算 `$count` 与 `$sum_data`，其他页码变量未定义。虽用 `??` 兜底，但违背分页常规设计，且 `== '1'` 为弱类型比较。 | 统计查询应与分页解耦，或明确约定前端传参逻辑。使用严格类型比较 `(int)$params['page'] === 1`。 | ```php<br>$is_first_page = (int)($params['page'] ?? 1) === 1;<br>if ($is_first_page) {<br> $count = $this->count($where);<br> $sum_data = $this->get_one($where, 'SUM(IF(a._type = 1, a._amount, -a._amount)) as total_amount');<br>}<br>``` |
| 🟠 警告 | `get_community_revenues_list()` ~L145 | **数组越界风险**：`explode('_', $params['pay_platform'])` 后直接访问 `$pay_platform[1]`。若传入值不含 `_`，将触发 `Undefined offset` 警告并导致逻辑错乱。 | 使用 `list()` 解构或 `isset()` 安全取值。 | ```php<br>$parts = explode('_', $params['pay_platform']);<br>$where['a._pay_platform'] = $parts[0];<br>if (isset($parts[1]) && $parts[1] !== '') {<br> $where['a._second_pay_platform'] = $parts[1];<br>}<br>``` |
| 🟡 建议 | L10 | **类名不符合 PSR-12**：使用蛇形命名 `Jh_community_shop_revenues_detail_model`，不符合 PHP 标准类名大驼峰规范。 | 重命名为 `JhCommunityShopRevenuesDetailModel`，并同步更新所有引用处。 | `class JhCommunityShopRevenuesDetailModel extends Report_model` |
| 🟡 建议 | L17-L38 | **常量键类型不一致**：`ORDER_TYPE` 数组键使用字符串 `'1'`、`'2'`，后续逻辑中又与字符串/整型混用比较。 | 统一使用整型键，或在类顶部添加 `declare(strict_types=1);` 强制类型安全。 | `const ORDER_TYPE = [1 => '在线预订', 2 => '卡券核销兑换', ...];` |
| 🟡 建议 | 全局多处 | **依赖未声明的全局函数**：`two_dimensional_arr_sort()`、`filter_emoji()`、`change_number_format()` 等未通过 `use` 或 `require` 显式引入。若未自动加载，将导致 `Fatal Error`。 | 将通用函数封装为静态工具类（如 `App\Utils\StringHelper`）或确保已正确加载对应 Helper。避免污染全局命名空间。 | `use App\Utils\ExcelHelper;`<br>`$v['user_name'] = ExcelHelper::filterSpecialChars($name);` |
> 📌 **框架适配说明**：代码呈现典型的 CodeIgniter 3 语法特征（如 `$CI = &get_instance()`、`$this->load->model()`、数组型 `$where` 构建器）。若 `phpci` 为基于 CI 的定制框架，请核对底层 `DB_driver.php` 是否完整支持该 `$where` 数组语法及自动转义机制；若为独立框架，请严格参照 `phpci` 官方文档调整模型加载、查询构建与分页组件的调用方式。
## 3. 总结与行动建议
### 🔑 优先修复的关键问题
1. **消除 SQL 拼接隐患**：立即重构 `$pay_platform_where` 的拼接逻辑，全面改用框架提供的参数化查询或 Query Builder 分组方法，杜绝注入风险。
2. **解决 N+1 查询**：将循环内的单条 `get_one()` 替换为批量查询，预计可降低 70% 以上的数据库交互耗时，显著提升报表加载速度。
3. **规范模型加载与作用域**：移除文件顶部的全局 `$CI` 调用，统一在 `__construct()` 中加载依赖模型，确保框架生命周期安全。
### 🛠 后续重构与优化方向
- **查询构建器标准化**：当前 `$where` 数组结构（如 `'join' => [...]`、`'where_in' => [...]`）属于高度定制化的封装。建议逐步迁移至框架原生 Query Builder 链式调用，提升代码可读性与跨版本兼容性。
- **输入校验与类型安全**：对 `$params['start_time']`、`$params['end_time']` 增加日期格式校验（如 `DateTime::createFromFormat`），避免 `strtotime()` 返回 `false` 导致查询异常。建议开启 `declare(strict_types=1);` 并规范参数类型声明。
- **配置与常量抽离**：将硬编码的表名（如 `ktv_online.ahead_merchant_room_type`）、支付平台映射、运营场景字典等抽离至配置文件或枚举类中，便于后续多租户/多环境切换。
- **单元测试覆盖**：针对 `get_community_revenues_list()` 的复杂过滤条件与分页逻辑，编写 PHPUnit 测试用例，重点覆盖边界条件（空数组、非法 JSON、跨页查询、退款单号解析等）。
> 💡 **提示**：若当前项目处于快速迭代期，建议先完成 🔴 与 🟠 级别问题的修复，再逐步推进 🟡 级别的重构。所有修改请务必在测试环境验证数据库执行计划（`EXPLAIN`）与内存占用情况。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780394903
|
1780394903
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
340
|
18
|
149
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 自助转房设置
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `f2ac768412 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `f2ac7684120cded91e9bab2cf25b5fcb44f0f52b`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-26 17:11:39
---
## 1. 审查摘要
- **代码质量评分**：4 / 10 分
- **总体评价**：该文件本质上是一个庞大的业务配置数组定义，但存在严重的架构违规（顶层执行代码）、语法截断问题，且将静态配置硬编码在 Model 中，违反了配置与逻辑分离原则。整体可维护性与框架兼容性较差，需进行结构性重构。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 第 1-3 行 | 在类定义外部直接执行 `$CI = &get_instance();` 和 `$CI->load->model('Simple_model');`。PHP 会在文件被 `include/require` 时立即执行，若框架尚未完成引导将触发 `Fatal Error`，且严重违反 OOP 封装原则。 | 彻底移除顶层执行代码。模型依赖应通过框架自动加载机制或构造函数处理。若 `Simple_model` 为父类，直接 `extends` 即可，无需手动 `load`。 | ```php\n// ❌ 删除以下代码\n$CI = &get_instance();\n$CI->load->model('Simple_model');\n\n// ✅ 正确做法：仅保留类定义\nclass Ahead_community_shop_model extends Simple_model\n{ ... }\n``` |
| 🔴 严重 | 末尾行 | 提供的代码片段不完整，数组定义在 `'config_params` 处突然截断，缺少闭合括号与类结束符，直接导致 `Parse Error`。 | 补全缺失的数组结构闭合符号 `]);`、属性闭合 `];` 及类闭合 `}`，确保语法合法。 | ```php\n ]\n ]\n ]\n ]\n ];\n}\n``` |
| 🟠 警告 | 第 5 行起 | 将超大型静态业务配置数组硬编码在 Model 属性中。每次实例化该模型都会将完整数组载入内存，造成不必要的内存开销，且违反“配置与业务逻辑分离”的最佳实践。 | 将 `$operational_scene_config` 迁移至独立配置文件（如 `application/config/community_shop.php`），或通过框架 Config 组件/缓存层动态读取。 | ```php\n// config/community_shop.php\nreturn [\n 'operational_scene_config' => [ /* 原数组内容 */ ]\n];\n\n// Model 中按需读取\npublic function getSceneConfig($sceneId)\n{\n $config = config('community_shop')['operational_scene_config'];\n return $config[$sceneId] ?? [];\n}\n``` |
| 🟡 建议 | 第 4 行 | 类名 `Ahead_community_shop_model` 采用蛇形命名。虽兼容旧版 CI3 规范，但不符合 PSR-12/PSR-4 现代 PHP 命名标准，不利于自动加载与团队协作。 | 建议重命名为 `AheadCommunityShopModel`，同步修改文件名，并更新项目自动加载映射。 | ```php\nclass AheadCommunityShopModel extends Simple_model\n{\n // ...\n}\n``` |
| 🟡 建议 | 全局 | **框架适配提示**：代码目录结构（`system/helpers/`、`get_instance()`、`load->model()`）高度疑似 CodeIgniter 3。若 `phpci` 为自研/魔改框架，请确认其是否完全兼容 CI3 的单例模式与模型加载机制。 | 查阅 `phpci` 官方文档确认依赖注入或模型加载方式。建议优先使用构造函数初始化或框架提供的 Service Container，避免全局单例滥用。 | ```php\n// 若框架支持 DI 或标准 CI3 规范\npublic function __construct()\n{\n parent::__construct();\n // 框架特定初始化逻辑（如需）\n}\n``` |
## 3. 总结与行动建议
### 🔑 优先修复项
1. **立即移除顶层执行代码**：删除文件开头的 `$CI = &get_instance();` 与 `$CI->load->model()`，避免文件被包含时引发致命错误。
2. **补全语法结构**：修复末尾截断的数组与类定义，确保代码可通过 `php -l` 语法检查。
3. **配置与模型解耦**：将 `$operational_scene_config` 迁移至 `config/` 目录或数据库配置表，模型仅负责读取与校验，不再承担数据定义职责。
### 🛠 后续重构与优化方向
- **引入配置缓存**：该配置数组体积较大且变更频率低，建议结合框架 Cache 组件（如 Redis/File）进行缓存，减少重复解析开销。
- **配置结构抽象化**：若后续配置项持续膨胀，建议将数组结构抽象为 `DTO` 或 `ConfigEntity` 类，配合验证规则（如 `Form_validation` 或自定义 Validator）确保数据完整性。
- **动态化管理**：当前配置为硬编码，不利于运营人员修改。建议后续将此类场景配置持久化至数据库，并提供后台管理界面进行可视化维护。
- **框架规范对齐**：请明确 `phpci` 与 CodeIgniter 的继承关系。若为独立框架，请严格遵循其官方模型加载、配置读取及生命周期规范，避免混用不同框架的 API 导致隐性兼容问题。
> 💡 **注**：本次审查基于提供的代码片段。若实际业务逻辑（如配置读取、校验、入库方法）存在于其他文件中，建议一并提供以便进行完整的链路审查。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779786699
|
1779786699
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
630
|
18
|
197
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 需求 盘点单增加时间选择
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `f17760cc43 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `f17760cc43e88f2aa1cd9b4f66da9ae98165719a`
**提交人**: chenjunfeng (developer.jeff.c@gmail.com)
**时间**: 2026-06-09 14:09:54
---
## 1. 审查摘要
- **代码质量评分**：6.5 / 10 分
- **总体评价**：业务逻辑链路基本完整，涵盖了盘点单创建、列表查询、详情计算、报表生成与导出等核心场景。但代码中存在**严重的并发安全隐患**（动态修改单例模型表名）、**隐式类型转换导致的逻辑漏洞**、**事务边界划分不当**以及**模型层直接中断执行**等反模式。整体架构偏向传统 CodeIgniter 3 风格，与现代 PHP 规范及高并发场景要求存在一定差距。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | `add_merchant_stocktaking` ~L38 | **时间比较逻辑错误**：`$now_business_date['end_business_day_str']` 为字符串格式，直接与数字 `86400*15` 相加会触发 PHP 隐式类型转换（通常转为 `0` 或截断），导致 `15天限制` 校验完全失效。 | 统一转换为时间戳后再进行数值比较，或使用 `DateTime` 对象。 | ```php<br>$nowTs = strtotime($now_business_date['end_business_day_str']);<br>$lastTs = strtotime($lastTimeBusiness['end_business_day_str']);<br>if ($nowTs > $lastTs + 86400 * 15) {<br> throwError("盘点库存截止时间只能是15天内");<br>}<br>``` |
| 🔴 严重 | `get_stocktaking_detail` ~L118<br>`get_stocktaking_report` ~L168 | **模型单例状态污染**：CI 模型为单例。通过 `set_table_name()` 动态追加别名（如 `table stocktaking`）会修改全局状态。高并发下极易导致请求 A 的别名被请求 B 误用，引发 SQL 语法错误或数据错乱。 | 放弃修改模型属性，改用 Query Builder 的 `from()` 或 `join()` 指定别名，或实例化临时模型对象。 | ```php<br>// 推荐做法：使用 DB 构造器别名<br>$this->db->from($orginalTableName . ' stocktaking');<br>$amountSum = $this->db->get()->row_array();<br>// 或使用临时实例<br>$tempModel = clone $this->Ahead_merchant_stocktaking_infos_model;<br>$tempModel->set_table_name($orginalTableName . ' stocktaking');<br>``` |
| 🔴 严重 | `stocktaking_report_export` ~L208, L213 | **模型层直接 `exit()`**：在 Model 中调用 `exit()` 会直接终止 PHP 进程，导致后续中间件、日志记录、事务回滚或框架生命周期钩子无法执行，且可能引发 `Headers already sent` 错误。 | 改为抛出异常或返回错误数组，由 Controller 层统一处理响应。 | ```php<br>if (empty($exportFields)) {<br> throw new \InvalidArgumentException('导出字段不能为空');<br>}<br>if (!$exportRes['success']) {<br> throw new \RuntimeException($exportRes['msg']);<br>}<br>``` |
| 🟠 警告 | `add_merchant_stocktaking` ~L68 | **事务边界不当**：`$this->db->trans_commit()` 执行后，紧接着调用 `add_data()`。若 `add_data` 失败，主事务已提交，导致 `stocktaking` 主表与 `update_data` 关联表数据不一致。 | 将 `add_data` 移入 `try` 块内，在 `trans_commit()` 前执行；或改为异步队列处理。 | ```php<br>// 移入事务块内<br>$this->db->trans_start();<br>try {<br> // ... insert & update_stock ...<br> if (!empty($stocktaking_update_time)) {<br> $this->Ahead_stocktaking_update_data_model->add_data(...);<br> }<br> $this->db->trans_complete(); // 推荐用 trans_complete()<br>} catch (...) { ... }<br>``` |
| 🟠 警告 | `search_stocktaking_list` ~L95 | **JSON 解析未做容错**：`json_decode($row['operater_names'], true)` 未校验数据合法性。若数据库存储了非法 JSON 字符串，将返回 `null` 并触发 `implode()` 类型警告。 | 增加空值/类型校验，或使用 `json_decode(..., true) ?: []`。 | ```php<br>$names = json_decode($row['operater_names'], true);<br>$row['admin_name'] = is_array($names) ? implode(',', $names) : $row['admin_name'];<br>``` |
| 🟠 警告 | `search_stocktaking_list` ~L78 | **权限过滤潜在越权**：`explode(",", trim($CI->priv_shop_ids, ","))` 当 `priv_shop_ids` 为空字符串时，`explode` 返回 `['']`，传入 `WHERE IN` 可能引发 SQL 异常或意外匹配。 | 增加空值判断，过滤空元素。 | ```php<br>$shopIds = array_filter(explode(",", trim($CI->priv_shop_ids, ",")), 'strlen');<br>if (!empty($shopIds)) {<br> $where['where_in'] = ['_shop_id', $shopIds];<br>}<br>``` |
| 🟡 建议 | 全局 | **命名不符合 PSR-12**：类名 `Ahead_merchant_goods_stocktaking_model` 及方法名 `add_merchant_stocktaking` 使用下划线，不符合现代 PHP 驼峰命名规范，影响自动加载与团队协作。 | 逐步重构为 `AheadMerchantGoodsStocktakingModel` 及 `addMerchantStocktaking`。若为历史遗留项目，可配置 IDE 映射或添加 `@deprecated` 注释过渡。 | `class AheadMerchantGoodsStocktakingModel extends Simple_model` |
| 🟡 建议 | `get_stocktaking_report` ~L145 | **非标准查询语法强耦合**：大量使用 `'_updated_at >= '`、`'where_in' => [...]` 等自定义键名。强依赖 `Simple_model` 内部解析逻辑，降低代码可移植性，且可能绕过框架默认的 SQL 转义。 | 建议统一使用框架原生 Query Builder 语法，或在 `Simple_model` 中明确文档化并增加参数类型校验。 | 无（需结合框架底层实现调整） |
## 3. 总结与行动建议
### 🔑 优先修复项（P0）
1. **修复时间比较漏洞**：立即将字符串时间转换为时间戳后再进行 `15天` 阈值校验，否则业务规则形同虚设。
2. **消除模型单例污染**：全面移除 `set_table_name()` 动态修改表名的写法，改用 `$this->db->from('table alias')` 或 `clone` 模型实例，彻底解决高并发下的数据串扰风险。
3. **规范事务与异常处理**：将 `add_data` 纳入事务范围；将 Model 中的 `exit()` 替换为 `throw new Exception()`，确保框架生命周期完整执行。
### 🛠 后续重构方向
1. **架构解耦**：当前 Model 承担了过多职责（数据查询、业务校验、报表组装、导出逻辑）。建议将 `stocktaking_report_export` 移至 `Service` 层或 `Job` 队列，Model 仅负责数据持久化与基础查询。
2. **查询性能优化**：
- 报表查询涉及多表 `LEFT JOIN` 与 `GROUP BY`，建议在 `ahead_merchant_goods_stocktaking` 表建立复合索引：`INDEX idx_merchant_status_time (_merchant_id, _status, _create_time)`。
- `get_stocktaking_report` 中的 `buying_price` 与 `shop_config` 已做批量查询优化，可考虑引入 Redis 缓存门店配置，减少重复 DB 查询。
3. **框架适配说明**：代码结构高度符合 **CodeIgniter 3** 规范。若 `phpci` 为内部定制框架，请重点核对 `Simple_model` 对 `where_in`、`>=` 等自定义语法的 SQL 预处理逻辑，确保已启用参数绑定（Prepared Statements）以防 SQL 注入。
> 💡 **提示**：若需进一步审查 `Simple_model` 底层实现或数据库表结构，可提供相关片段以便进行更精准的索引与查询优化评估。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1780985394
|
1780985394
|
0
|
0
|
0
|
0
|
Edit
Delete
|
|
349
|
18
|
154
|
1
|
|
0
|
🔍 代码审查报告：pc-260616 - 1
|
## 自动代码审查报告
**分支**: pc-260616
**提交**: `eb5b62a472 ## 自动代码审查报告
**分支**: pc-260616
**提交**: `eb5b62a4727dd9dcc9fcdc3423dfd4455db87803`
**提交人**: LITTLEMAIDI (11833999+littlemaidi@user.noreply.gitee.com)
**时间**: 2026-05-26 17:52:33
---
## 1. 审查摘要
- **代码质量评分**：4/10 分
- **总体评价**：该文件主要承载静态业务配置数据结构，但存在严重的框架生命周期违规、职责分离不当及代码截断问题。整体架构不符合现代 PHP 面向对象规范及主流 MVC 框架最佳实践，需进行结构性重构。
- **风险等级**：🔴 高
## 2. 问题详情
| 严重程度 | 文件/行号 | 问题描述 | 建议修改方案 | 代码示例 (可选) |
| :--- | :--- | :--- | :--- | :--- |
| 🔴 严重 | 顶部 1-3 行 | 在类外部直接执行 `$CI = &get_instance();` 与 `$CI->load->model()`。PHP 在 `include/require` 时会立即执行顶层代码，若该文件在框架核心初始化前被加载，将触发 `Fatal Error`；且破坏了模型类的封装性。 | 移除顶层执行代码。依赖加载应移至类的构造函数中，或通过框架自动加载机制（如 `config/autoload.php`）处理。 | `public function __construct() { parent::__construct(); $this->load->model('Simple_model'); }` |
| 🟠 警告 | 全文 (数组定义) | 将超大型业务配置数组硬编码在 Model 中。违反单一职责原则（Model 应专注数据交互），每次实例化都会占用大量内存，且不利于多环境配置、热更新与版本控制。 | 将配置抽离至独立配置文件（如 `application/config/scene_config.php`）或存入数据库/Redis。Model 仅负责读取与缓存。 | `// application/config/scene_config.php<br>return [ '1' => [...], '2' => [...] ];` |
| 🟠 警告 | 类定义行 | 框架识别特征（`system/` 目录、`get_instance()`、`$this->load->model()`）明确指向 **CodeIgniter 3**，而非 `phpci`。若为定制分支，请确认生命周期是否一致。类名未遵循 PSR-12 驼峰规范，且未定义构造函数。 | 确认框架基线。若基于 CI3，建议类名与文件名保持一致，并显式继承 `CI_Model` 或基类。 | `class Ahead_community_shop_model extends CI_Model { public function __construct() { parent::__construct(); } }` |
| 🟡 建议 | 数组内部 | 配置项高度重复，魔法数字（如 `'1'`, `'-1'`, `'2'`）缺乏语义化定义，后期维护易出错。 | 使用类常量或枚举定义状态值，提升可读性与类型安全。 | `const STATUS_ENABLE = '1'; const STATUS_DISABLE = '-1';` |
| 🟡 建议 | 文件末尾 | 代码片段在 `'config_params` 处意外截断，无法评估完整逻辑与闭合结构。 | 补充完整文件内容以便进行全量审查。 | 无 |
## 3. 总结与行动建议
### 🔑 优先修复项
1. **移除顶层执行代码**：立即删除文件头部的 `$CI = &get_instance();` 及模型加载语句，将其迁移至 `__construct()` 方法内，避免在文件解析阶段引发致命错误。
2. **配置与逻辑解耦**：将 `$operational_scene_config` 数组迁移至 `application/config/` 目录或配置中心。Model 中仅保留读取方法，例如：
```php
public function get_scene_config($scene_id) {
$config = config_item('scene_config'); // 或从缓存/DB读取
return $config[$scene_id] ?? [];
}
```
3. **补充完整代码**：当前片段截断严重，请提供完整文件以排查潜在的语法闭合错误、数组越界或未定义的键值引用。
### 🛠 后续重构方向
- **引入配置校验层**：针对动态下发的配置，建议在读取时增加类型校验与默认值回退机制（如使用 `filter_var` 或自定义 Validator），防止脏数据导致前端渲染异常。
- **缓存策略优化**：该配置结构变更频率低但读取频繁，建议首次加载后写入 Redis/Memcached，设置合理 TTL，避免每次请求重复解析大数组。
- **框架规范对齐**：若项目确为 `phpci` 定制框架，请查阅官方文档确认模型生命周期与自动加载规范；若实际基于 CodeIgniter 3，建议统一遵循 CI3 官方手册的 Model 编写标准，并逐步向 PSR-12 编码规范靠拢。
> 💡 **注**：受限于提供的代码片段不完整，本次审查主要聚焦于架构设计与框架生命周期规范。补充完整代码后，可进一步针对边界条件处理、数据过滤及具体业务逻辑进行深度审计。
---
*此 Issue 由代码审查服务自动创建*...
|
0
|
0
|
0
|
0
|
0
|
|
0
|
1779789153
|
1779789153
|
0
|
0
|
0
|
0
|
Edit
Delete
|