[低] MainWindow.clear_cache 无任何调用

Open
#35 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
58/100
Issue type
Refactor
Clarity
Mostly clear
Activity status
Active
Tech stack
python
Domain
desktop

Research direction

Start at windows/birdlock/app/main_window.py:737-749 to see what clear_cache deletes, then grep for clear_cache and for the cache menu actions to confirm nothing calls it. A maintainer needs to decide delete versus wire-up; if wired, check how state.job and state.analysis are reset and whether preview or background threads touch the cache at the same time. Done means the method is removed, or reachable from the menu with a test covering it.

Written by the indexing model from the issue text.

Description

位置:windows/birdlock/app/main_window.py:737-749

现象:菜单里只有「缓存管理…」与「清理未使用的过程文件」,没有「清空缓存」入口,该方法也无测试调用。它还会删除当前缓存但不清理 state.job/analysis。

建议:要么接线并确认与预览/后台线程的并发安全,要么删除。

来源:2026-11-15 全仓只读审计,PR #7 未包含此项。 未在 Windows 实机复现,复现路径由代码审计推得。

Dominant language
Swift
Stars
2
Forks
2
Avg merge
1h 11m
Merged PRs (30d)
2

Getting set up

This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from cyandee/BirdLock

All issues in cyandee/BirdLock

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.