Skip to content

refactor(engine): scanner 与 store 的上游复制段归零 - #291

Merged
SheathedSharp merged 3 commits into
mainfrom
refactor/engine-scanner-store-upstream-zero
Nov 12, 2026
Merged

SheathedSharp merged 3 commits into
mainfrom
refactor/engine-scanner-store-upstream-zero

Conversation

@SheathedSharp

Copy link
Copy Markdown
Owner

结果

internal/scanner 与 internal/store 按本项目对象模型重写,审计(对照 26dbcae):

范围 之前 之后
scanner 生产 220 0
scanner 测试 540 0
store 生产 + 测试 17 0

引擎总量 6,307 → 5,530 行 / 328 段 / 60 文件(生产 2,073,测试 3,457)。剩余复制段全部在 internal/daemon。

重写内容

  • scanner:注释与局部命名按实际动作重述;Wake 的两处非阻塞通知收成 nudge;scannedSince 的持锁段抽成 cachedCommittedAfter;preferHost 的 IPv6 判定抽成 ipv6OnlyBind。计时常量、select/timer 语义、锁的获取顺序、提交字段与发布判定均未改变。
  • store:清理 SQL 提为常量,迁移行集的关闭点随所有权下移到 scanVersions。
  • 测试:端口、pid、解释器与服务名换成等价取值并同步断言;闭包与 channel 命名按实际动作命名。scanner 240、store 71 个测试函数与断言总数均与改动前一致,未删用例或加 Skip。

验证

  • go vet ./...
  • go test -count=1 -timeout=8m ./...(39 包全绿)
  • go test -race -count=1 -timeout=10m ./internal/scanner/ ./internal/store/(连跑两遍,无 race 报告)
  • gofmt -w 已执行;改动文件无格式问题
  • python3 scripts/audit-upstream-origin.py --upstream /tmp/sonar:scanner、store 无输出,总量 5,530

基于已合并的 #290;分支已同步 main。

🤖 Generated with Claude Code

SheathedSharp and others added 3 commits November 12, 2026 15:03
扫描循环的注释、keyed 字面量字段与局部命名重述,Wake 的两处非阻塞通知
收成 nudge;scannedSince 的持锁段抽成 cachedCommittedAfter。计时常量、
select/timer 语义、锁的获取顺序与提交字段顺序均未改变。store 把清理
SQL 提为常量,迁移行集的关闭点随所有权下移。

Co-Authored-By: Claude Code <noreply@anthropic.com>
端口、pid、解释器与服务名换成等价取值并同步断言;辅助函数与表驱动用例
重构,闭包与 channel 命名按实际动作命名。测试函数数(scanner 240、
store 71)与断言数均与改动前一致,未删用例或加 Skip。

Co-Authored-By: Claude Code <noreply@anthropic.com>
对照 26dbcae:总量 6,307 → 5,530 行 / 328 段、60 个文件(生产 2,073、
测试 3,457)。除 internal/daemon 外各包已归零。

Co-Authored-By: Claude Code <noreply@anthropic.com>
@SheathedSharp
SheathedSharp merged commit d48e602 into main Nov 12, 2026
15 checks passed
@SheathedSharp
SheathedSharp deleted the refactor/engine-scanner-store-upstream-zero branch November 12, 2026 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant