Repository navigation
fix: Bugfix-Sweep – FTS5-Trigger, GUI-Threading, Suche, Summarizer, Transit optional - #2
Merged
Merged
Conversation
tests/test_transit.py brach die gesamte Test-Collection ab, weil sqlite_transit_sync nicht auf PyPI verfügbar ist (CI rot). - Import in transit.py optional (TRANSIT_SYNC_AVAILABLE) mit klarer ImportError-Meldung in create_knowledge_sync / merge - iso_timestamp liefert festbreite UTC-Zeitstempel (Mikrosekunden), da Leases lexikografisch verglichen werden - get_task verändert die row_factory der Aufrufer-Connection nicht mehr - Sync-Tests werden ohne Paket übersprungen; neue Lifecycle-Tests für ChunkTaskManager und Konfliktauflösung laufen ohne Paket Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014VFa8w68q2cEYMqHQqfiH2
…ngest - Schema v5: FTS5-Delete-Trigger brachen jedes DELETE (SQL logic error) – Re-Ingest, index --force, Löschen und deduplicate waren defekt; Migration für bestehende DBs - Löschen: erst DB-Transaktion, dann Datei in _Papierkorb - GUI: falsche relative Imports (leere Dokumentliste), falsches Dokument nach Sortierung, SQLite-Connections pro Thread, EventBus dispatcht in den GUI-Thread - Summarizer: fehlgeschlagene Chunks -> status error; verwaiste processing-Einträge werden zurückgesetzt; Kostenschätzung aktualisiert - Suche: FTS5-Syntaxfehler -> quotierte Tokens -> LIKE-Fallback; LIKE- und Verzeichnisfilter escaped - Ingest: Re-Ingest setzt Summaries/Queue zurück, archived_path-Fallback, UTF-8-BOM, harte Chunk-Obergrenze - Kleinkram: is_active=0, content_hash-Präzedenz, Config-Deepcopy, zoll_station als Modulaufruf, plattformübergreifendes Datei-Öffnen Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014VFa8w68q2cEYMqHQqfiH2
|
Welcome! 👋 Thanks for your first pull request in this repository. A maintainer will review it soon. Please make sure:
Thanks for contributing! |
Aktuelles setuptools lehnt `license = { text = "MIT" }` zusammen mit
`license-files` ab ("project.license must be string"), wodurch bereits
`pip install -e .[test]` in CI scheiterte. Jetzt SPDX-Ausdruck
`license = "MIT"`, setuptools>=77 als Build-Requirement und ohne den
überholten License-Classifier.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014VFa8w68q2cEYMqHQqfiH2
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
2 participants
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.
Zusammenfassung
Code-Review mit anschließenden Fixes. Wichtigster Befund: Die FTS5-Delete-Trigger brachen jedes DELETE auf Chunk-Tabellen (
SQL logic error) – Re-Ingest geänderter Dateien,index --force, Löschen im Web-Viewer unddeduplicatewaren dadurch defekt. Außerdem brachtests/test_transit.pydie gesamte Test-Collection ab (CI rot), weilsqlite-transit-syncnicht auf PyPI liegt.Fixes
DELETE FROM <x>_fts WHERE rowid = old.id; bestehende DBs werden beim Öffnen migriert._Papierkorb(Web-Viewer,deduplicate).errorstattdone; verwaisteprocessing-Einträge werden zurückgesetzt; Kostenschätzung aktualisiert.COVID-19,E-Mail,c++) → quotierte Tokens → LIKE-Fallback; LIKE-Wildcards escaped; Verzeichnisfilter matcht keine Geschwister-Ordner mehr.archived_path-Fallback, UTF-8-BOM, harte 500-Wort-Obergrenze im Chunker.sqlite-transit-syncoptional (TRANSIT_SYNC_AVAILABLE), festbreite UTC-Zeitstempel für Lease-Vergleiche,get_taskverändert keine fremderow_factory.is_active=0,content_hash-Operator-Präzedenz,Config-Deepcopy,zoll_station.pyals Modulaufruf, plattformübergreifendes Datei-Öffnen.Tests
python -m pytest tests→ 214 passed, 5 skipped (vorher: Collection-Abbruch).tests/test_bugfixes_2026_10.py(37 davon schlagen auf dem alten Stand fehl), neue Lifecycle-Tests intests/test_transit.py.ruff check .: keine neuen Befunde gegenübermain(bestehende Stil-Befunde unverändert).Hinweis: Die 5 Sync-Tests laufen nur mit installiertem
sqlite-transit-sync(pip install "git+https://github.com/ellmos-ai/sqlite-transit-sync").🤖 Generated with Claude Code
https://claude.ai/code/session_014VFa8w68q2cEYMqHQqfiH2
Generated by Claude Code