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
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