Files
X-Financial/document/development/2026-07-16/dev-logs/bugs/release-guard-global-scope-and-integrity-fail-close.md
caoxiaozhu 787bc3a481 feat(platform): close AI expense value loop
Add tenant-safe value, telemetry, connector, commercial, and production-readiness foundations.
2026-07-17 14:14:08 +08:00

5.6 KiB
Raw Permalink Blame History

AI 发布门禁全局资产越权与快照损坏静默放行

修复记录

  • 19:48记录 AI 分阶段发布的租户越权与运行时完整性失效修复。

    • Git 提交检查:执行 git fetch --all --prune 后未发现 HEAD..origin/main 上游新提交;本地 main 比上游 ahead 17最新为 242d68c3 feat(approval): add task workflow and waiver decisions其余为审批安全、AI 费用学习、报销预审、迁移安全与会话认证等既有检查点,本次未合并或改写这些提交。
    • 修改:agent_asset_release_guard.py 为所有发布查询、启动、评测、晋级和回滚入口增加平台全局资产管理边界;未绑定租户的共享资产只允许平台管理员管理,租户级 manager 统一返回不可见。agent_asset_releases.py 与既有风险规则发布入口只从认证上下文传递平台管理员权限,不能由请求体或自报 actor 绕过。
    • 修改:expense_claim_risk_rule_loader.py 在候选快照与稳定快照均无法通过 SHA-256 完整性校验时生成强制阻断信号,expense_claim_platform_risk.py 将该信号转换为 critical/block 风险,而不是把损坏规则当成“未命中”静默跳过。
    • 操作:仅通过 apply_patch 修改源码和测试;保护现有财务规则 XLSX、历史未跟踪开发目录及其他智能体改动未执行提交、推送或数据库破坏操作。
    • 验证:在 local-x-financial-linux 容器内运行 Ruff 定向检查通过;test_agent_asset_release_guard.pytest_agent_asset_release_runtime.py 共 12 项测试全部通过,新增覆盖租户 manager 无权管理全局发布资产、平台 admin 可管理,以及双快照损坏后报销自动流转被阻断。
    • 影响:单租户管理员不再能影响所有企业共享的风险规则;受控发布元数据损坏时系统优先停流并提示平台恢复稳定版本,避免风险规则失效后继续自动审批。
  • 19:53继续修复发布质量指标可由管理人员手工伪造的问题。

    • Git 提交检查:再次执行 git fetch --all --pruneHEAD..origin/main 仍无上游新提交;本地仍 ahead 17提交范围与 19:48 检查一致,未自动合并、变基或覆盖共享工作区改动。
    • 修改:新增 agent_asset_release_monitor_auth.py,评测请求必须使用至少 32 字节独立密钥对时间戳、租户、资产、release ID、当前阶段和规范化请求体摘要做 HMAC-SHA256 签名;仅允许 5 分钟时钟窗口并使用常量时间比较。agent_asset_releases.py 在写入评测记录前验证签名,密钥未配置返回 503缺失、过期或错误签名返回 401。
    • 操作:保留服务层直接写入能力供同进程可信监控使用,但关闭普通 HTTP manager 仅凭自报数字写入“通过”证据的路径;签名绑定 release ID 和阶段,旧阶段请求不能在晋级后重复使用。
    • 验证:容器内 Ruff 定向检查通过;发布 guard/runtime 共 12 项测试通过HTTP 用例新增未签名评测返回 401同时签名监控数据仍可触发 shadow→Canary→active 和指标越界自动回滚。
    • 影响:人工管理权限与机器监控证据分离,发布门禁不再把未经认证的手工数字当成可信质量结果。
  • 21:51补齐“签名合法但指标仍可伪造”的第二层真实性修复并接通真实证据自动回滚。

    • Git 提交检查:执行 git fetch --all --prunegit status -sbgit log HEAD..@{u}git log @{u}..HEADorigin/main 无新提交,本地仍 ahead 17最新提交仍为 242d68c3 feat(approval): add task workflow and waiver decisions,其余 ahead 提交为既有审批、AI 费用学习、报销预审、迁移安全和会话认证检查点。本次未合并、变基、提交或推送,也未触碰受保护财务规则 XLSX。
    • 修改Release Monitor HTTP 契约改为禁止额外字段的空触发请求HMAC 继续绑定 tenant/asset/release/stagetotalprecision 等质量数字只能由服务端查询 append-only observation/label 聚合;即使签名正确,携带伪造汇总字段也返回 422。
    • 修改:真实 shadow/Canary/active manifest 执行追加脱敏 observation类型化 RiskDispositionEvent 追加可信标签并即时触发 Guard相同 release 聚合快照复用同一测试运行,低 precision 或结构化运行失败自动回滚 stable。即时监控失败由租户周期调度补偿collecting 不写虚假 passed。
    • 修改:新增 0018 两张发布遥测表、复合租户/release 外键、幂等唯一约束和 PostgreSQL append-only 触发器,并登记主 metadata、迁移所有权与启动前置检查将风险处置发布同步拆到独立服务使核心 risk_dispositions.py 保持 800 行以内。
    • 操作:全部源码与文档通过 apply_patch 修改;用独立 pgvector/pgvector:pg17 一次性数据库验证完整迁移链及数据库约束,未连接或变更生产数据库。
    • 验证:容器内发布 Guard/Runtime/Telemetry/Monitor/风险处置组合 44 项通过调度器、Monitor 与风险处置定向 24 项通过;迁移/schema owner 静态回归 112 项、启动前置检查 77 项、一次性 PostgreSQL 完整迁移循环 1 项通过PostgreSQL savings/commercial/financial connector/approval 并发探针共 14 项通过。Ruff 定向检查与 git diff --check 通过(文档回填后仍需最终全量复验)。
    • 影响:发布质量门禁不再信任“会签名的调用方”提交的汇总数字,而是由数据库真实执行与可信人工结论生成;遥测暂时失败只会阻止晋级,不会撤销已成功人工处置或关闭现有 stable 保护。