v2.5.59-2.5.60: Bug fix for scheduled scan UI + Repository cleanup#33
Merged
ttlequals0 merged 2 commits intomainfrom Feb 5, 2026
Merged
v2.5.59-2.5.60: Bug fix for scheduled scan UI + Repository cleanup#33ttlequals0 merged 2 commits intomainfrom
ttlequals0 merged 2 commits intomainfrom
Conversation
Remove ~500 lines of dead code and outdated documentation: Dead Code Removal: - Delete unused tool scripts (fix_database_schema_v2.py, fix_hevc_warnings.py, fix_nal_unit_false_positives.py) - Remove unused utility functions (handle_db_errors, update_state_progress) from utils.py - Remove deprecated API endpoints (/reset-stuck-scans, /recover-stuck-scan) - consolidated into /scan/recovery Documentation Cleanup: - Update all SQLite references to PostgreSQL (now required) - Remove all migration documentation (historical, no longer relevant) - Remove orphaned Docker files (docs/docker/*, docker-compose.simple.yml) - Fix dead documentation links Frontend Cleanup: - Remove 78 console.log/error/debug statements from JS files - Update recoverStuckScan() to use /scan/recovery endpoint Tests: - Update scan endpoint tests to use consolidated /scan/recovery endpoint - All 201 tests pass
Long-running integrity scans (20+ hours for 1M+ files) would crash at ~99.95% with "Instance has been deleted" errors. Root cause: ORM objects held in memory get expired by db.session.commit() calls, and if concurrent jobs delete rows, subsequent attribute access crashes. Solution: Load only needed columns and convert to plain Python dictionaries immediately after query. Dictionaries are immune to SQLAlchemy session expiration and concurrent database changes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR includes two releases:
v2.5.59 - Bug Fix
v2.5.60 - Repository Cleanup
fix_database_schema_v2.py,fix_hevc_warnings.py,fix_nal_unit_false_positives.pyhandle_db_errors()decorator,update_state_progress()helper/reset-stuck-scansand/recover-stuck-scan(use/scan/recoveryinstead)Files Changed
Bug Fix (v2.5.59):
static/js/app.js- AddedstartBackgroundScanDetection()functionCleanup (v2.5.60):
utils.py- Removed unused functionspixelprobe/api/scan_routes.py- Removed deprecated endpointsstatic/js/app.js,state.js,auth.js- Removed console.log statementsTest Plan
Docker
Image pushed:
ttlequals0/pixelprobe:2.5.60