Skip to content

Fix SQLite threading error when cancelling scan - #1

Open
cto-new[bot] wants to merge 1 commit into
mainfrom
cto-task-okay-please-begin-test-driven-development-on-this-which-is-c
Open

Fix SQLite threading error when cancelling scan#1
cto-new[bot] wants to merge 1 commit into
mainfrom
cto-task-okay-please-begin-test-driven-development-on-this-which-is-c

Conversation

@cto-new

@cto-new cto-new Bot commented Feb 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Problem

When cancelling a scan at 100% or navigating back from the scanning screen, the application would freeze and throw a SQLite threading error.

Solution

  1. Added check_same_thread=False to SQLite connection in core/database.py
  2. Added try/except error handling around db.close() calls in tui/app.py

Powered by CTO.new

@cto-new cto-new Bot changed the title Set up Test-Driven Development (TDD) with comprehensive tests and fix TUI navigation issues Fix setuptools package discovery for flat layout Feb 6, 2026
@cto-new cto-new Bot changed the title Fix setuptools package discovery for flat layout Apply comprehensive test suite for Archive Duplicate Finder Feb 6, 2026
@cto-new cto-new Bot changed the title Apply comprehensive test suite for Archive Duplicate Finder Fix: Resolve click module import error and verify test suite Feb 6, 2026
@cto-new cto-new Bot changed the title Fix: Resolve click module import error and verify test suite Fix TUI directory list visibility and 'q' quit binding Feb 6, 2026
@cto-new cto-new Bot changed the title Fix TUI directory list visibility and 'q' quit binding Fix DuplicateIds error when adding directories in TUI Feb 6, 2026
@cto-new cto-new Bot changed the title Fix DuplicateIds error when adding directories in TUI Add responsive progress dialogs for source and target scanning Feb 6, 2026
@cto-new cto-new Bot changed the title Add responsive progress dialogs for source and target scanning Fix TUI deletion method checkbox clarity and add extensive integration tests Feb 6, 2026
@cto-new cto-new Bot changed the title Fix TUI deletion method checkbox clarity and add extensive integration tests Fix call_from_thread error in ScanningScreen progress updates Feb 6, 2026
@cto-new cto-new Bot changed the title Fix call_from_thread error in ScanningScreen progress updates Fix Textual worker thread error in ScanningScreen Feb 6, 2026
@cto-new cto-new Bot changed the title Fix Textual worker thread error in ScanningScreen Fix SQLite threading error when canceling scan at 100% Feb 6, 2026
@cto-new
cto-new Bot force-pushed the cto-task-okay-please-begin-test-driven-development-on-this-which-is-c branch from 0e3ec77 to 6c17aca Compare February 6, 2026 17:39
@cto-new
cto-new Bot force-pushed the cto-task-okay-please-begin-test-driven-development-on-this-which-is-c branch from 6c17aca to a16d646 Compare February 6, 2026 17:44
@cto-new cto-new Bot changed the title Fix SQLite threading error when canceling scan at 100% Fix SQLite threading error when cancelling scan Feb 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

0 participants