Skip to content

Fix CodeQL comments from PR 305 - #306

Merged
suguanYang merged 1 commit into
mainfrom
fix/suguanYang/pr305-codeql-comments
Aug 19, 2026
Merged

Fix CodeQL comments from PR 305#306
suguanYang merged 1 commit into
mainfrom
fix/suguanYang/pr305-codeql-comments

Conversation

@suguanYang

Copy link
Copy Markdown
Contributor

Summary

  • Resolve the CodeQL review comments on #305 (mainstaging) in worker debug scripts.
  • Drop unused tracker assignments, a redundant comparison, a duplicate re import, and import * namespace pollution.
  • Replace empty except cleanup paths with unlink(missing_ok=True) / warning logs, and mark intentional re-exports in __all__.

Test plan

  • Confirm CodeQL no longer flags the previous comments on debug_parse.py, debug_retrieval.py, and scripts/page_memory/*
  • uv run --all-packages --group lint ruff check apps/worker/scripts/debug_parse.py apps/worker/scripts/debug_retrieval.py apps/worker/scripts/page_memory
  • git diff --check

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
@suguanYang
suguanYang merged commit 7625fb0 into main Aug 19, 2026
6 checks passed
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