Skip to content

fix(ci): 修复日志检查夹具与提交图笔记 - #1772

Merged
1lck merged 1 commit into
previewfrom
codex/issue-1662-verification
Nov 6, 2026
Merged

1lck merged 1 commit into
previewfrom
codex/issue-1662-verification

Conversation

@1lck

@1lck 1lck commented Nov 6, 2026

Copy link
Copy Markdown
Owner

关联 Issue / Linked issue

PR 类型 / Type

  • fix:缺陷修复
  • feat:新功能或用户可见的改进
  • docs:文档、Agent Note、Skill 等规则文件
  • release:稳定版发布准备

改动说明 / What changes

背景 / Why

#1771 把纯规则 crate 加入日志扫描范围后,测试的临时仓库没有创建新目录,导致 Structured Logging 检查报 ENOENT。#1770 把提交图实现和 IDEA 对照数据迁往共享 Core 后,两篇 Agent Note 仍引用已删除路径,导致 Agent Notes 校验及看板部署失败。

改动 / Changes

  • 在日志检查的临时仓库中创建 core-rules 目录,并断言该目录中的生产代码确实受到日志规则检查。
  • 将两篇提交图 Note 的失效路径和 fixture 链接更新为共享 Core 实现及共享数据;同步修正双端提交历史的日期排序说明。
  • 不更改日志扫描器、基线策略、产品运行逻辑或检查豁免。

不在本 PR 范围内 / Out of scope


类型专属:fix

复现与根因

检查项

  • 日志测试在修复前失败、修复后通过,并新增对 core-rules 生产文件被扫描的断言;路径 scripts/logging/test-log-events.mjs。
  • [N/A] 手动复现:检查故障可自动复现。
  • 已核对 scanRepository、日志扫描根目录与 CI 测试入口;本次不更改生产 API 或契约。
  • 检查为跨平台 Node 脚本,适用于两端;不存在同类平台专有实现。
  • 仅修复夹具与失效笔记,无额外重构;没有发现需要另建 Issue 的本次范围内问题。
  • [N/A] 新增 bug-fix Note:不改变行为约束;已更新原有两篇 Note 的迁移路径和事实。

范围检查 / Scope checks

功能矩阵 / Feature matrix

  • [N/A] 新增或更新能力记录:本 PR 不改变用户可见能力或跨平台行为。
  • 已运行 ./scripts/platform-feature-matrix/verify-platform-feature-matrix.sh。
  • 未提交 .artifacts/ 下生成的矩阵视图。
  • 不更新矩阵的理由:仅修复 CI 测试夹具及既有 Note。

统一日志系统 / Structured logging

  • [N/A] 新增事件和日志行为:本 PR 仅修正日志扫描测试夹具。
  • 已运行 node scripts/logging/generate-log-events.mjs --check 和 node scripts/logging/verify-log-usage.mjs;未修改豁免名单。

界面文案 / i18n

  • [N/A] 无用户可见文案修改。
  • 已运行 node scripts/i18n/verify-app-language-localization.mjs 和 node scripts/i18n/generate-i18n.mjs --check。

逻辑归属 / Logic placement

N/A:没有业务逻辑或平台实现变更。

性能与内存 / Performance and memory

N/A:没有运行时、后台、交互或资源使用变更。

设置 / Settings

N/A:没有设置变更。

插件与共享契约 / Plugin and shared contracts

N/A:没有插件协议、宿主方法或共享契约变更。

发行包与资源 / Bundle and resources

  • 已运行 ./scripts/macos/verify/verify-runtime-bundle-immutability.sh。
  • [N/A] 没有运行时资源、下载或构建资源变更。

无用代码与重构 / Dead code and refactoring

  • 没有因本次改动变成无用的生产代码。
  • 本次未发现需要另建 refactor Issue 的既有无用代码。

验证 / Verification

命令或操作 平台 结果
node --test --test-timeout=60000 scripts/logging/test-log-events.mjs macOS 15/15 通过;修复前该夹具报 ENOENT
./scripts/agent-notes/verify-agent-notes.sh macOS 212 篇 active Notes 通过
node scripts/logging/generate-log-events.mjs --check;node scripts/logging/verify-log-usage.mjs macOS 通过,0 违规
node scripts/architecture/verify-no-host-language-specifics.mjs --base origin/preview macOS 通过
./.agents/skills/write-stable-tests/scripts/verify-test-stability.sh macOS 通过;日志用例逐项计时已生成 HTML 和 JUnit,修改的用例约 4 ms
./scripts/macos/verify/verify-runtime-bundle-immutability.sh macOS 通过
node scripts/i18n/verify-app-language-localization.mjs;node scripts/i18n/generate-i18n.mjs --check macOS 通过
./scripts/platform-feature-matrix/verify-platform-feature-matrix.sh macOS 250 项通过
Windows 同步规则桥相关 Bun 测试(真实 WASM) Windows lithe-dev 135/135 通过,最慢 47 ms
bun run build;cargo build --locked --bin lithe-windows Windows lithe-dev 通过;前端产物包含规则 WASM

未能执行的检查和原因: Windows WebView2 图形启动及双击 Shift 的实际交互未验证,用户决定本次不继续图形验收;本 PR 不更改产品运行逻辑。

提交前确认 / Final checks

  • 分支从最新 preview 创建,目标为 preview。
  • 按 Draft 创建;本地验证通过后标记 ready。
  • 没有无关改动或提交生成产物、凭据、个人路径、日志及截图。
  • 本次没有启动 Lithe 图形实例;Windows 构建进程已结束,确认无 Lithe 进程残留。

@ghfind-review ghfind-review Bot added the review: high ghfind author score; see https://ghfind.com label Nov 6, 2026
@1lck
1lck marked this pull request as ready for review November 6, 2026 06:41
@1lck
1lck merged commit 4a029e0 into preview Nov 6, 2026
17 of 30 checks passed
@xiaoyumuxi
xiaoyumuxi deleted the codex/issue-1662-verification branch November 9, 2026 05:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review: high ghfind author score; see https://ghfind.com

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant