Skip to content

fix: dedup drift issues across open and closed states - #20

Merged
NocodeMrLi merged 1 commit into
mainfrom
fix/drift-issue-dedup-all-states
Sep 14, 2026
Merged

NocodeMrLi merged 1 commit into
mainfrom
fix/drift-issue-dedup-all-states

Conversation

@NocodeMrLi

Copy link
Copy Markdown
Owner

Summary

  • 去重只查 --state open:关闭已解决的 NO_ACTIONABLE_DRIFT issue 会抹掉去重键,下次周运行必重开同名重复项([Drift-audit] wechat: NO_ACTIONABLE_DRIFT #9 关闭后 [Drift-audit] wechat: NO_ACTIONABLE_DRIFT #19 重生,循环往复)
  • 改为 --state all + 标题搜索:已关闭的同名 issue 同样抑制重开;--search 不可用时回退全量 all-state 列表
  • 回归断言:mock 捕获 gh 调用参数,锁定 --state all 行为

Verification

existing_open_issues only queried --state open, so closing a resolved
'no actionable drift' issue erased the dedup key and the next weekly run
re-created a same-titled duplicate (#19 re-created after #9 was closed,
with the cycle repeating for every future closure). Query --state all
with a title search so closed duplicates still suppress re-opening;
fall back to a plain all-state listing when --search is unavailable.
Verified: 255 tests OK (incl. new regression assertion on --state all),
validate_suite 117 files, sensitive scan 0 findings, drift watch clean.
@NocodeMrLi
NocodeMrLi merged commit f54df0f into main Sep 14, 2026
8 checks passed
@NocodeMrLi
NocodeMrLi deleted the fix/drift-issue-dedup-all-states branch September 14, 2026 13:48
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