feat: optionaler BACH-Import als Standalone-API (aus OneDrive-Arbeitskopie gerettet) - #1
Merged
Merged
Conversation
Reproducible manual + headless-import smoke checks for the stdlib web viewer (:8787) and the PySide6 desktop GUI, verified against the actual source (launch_web/launch_gui signatures, toolbar labels, dark-theme color, window size) and a live HTTP 200 test of the web viewer. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…eitskopie gerettet Die Arbeitskopie unter OneDrive/.WISSEN/.db trug Arbeit, die in keinem Remote existierte: einen ungepushten Commit (SMOKE-CHECKS.md, hier per cherry-pick) und sieben uncommittete Dateien. Gerettet in den kanonischen Klon, ohne aus OneDrive zu pushen (Patch + cherry-pick statt Push aus dem Sync-Ordner). Inhalt: - indexer.import_bach_skills() / wiki_indexer.import_bach_wikis(): Standalone- API ohne Klassen-Overhead, laeuft ohne echtes BACH graceful durch und gibt immer ein Dict zurueck statt zu werfen. - Relative Imports mit Fallback auf absolute -- das Paket wird auch ohne Paket-Kontext direkt aus dem Verzeichnis importiert. - launcher.py: registriert das Verzeichnis unter dem Alias "KnowledgeDigest" in sys.modules. Noetig, weil der Deployordner ".db" heisst und das kein gueltiger Python-Name ist. - tests/test_bach_import.py: Fixture-Tests fuer beide Pfade (ohne bach.db -> available=False ohne Crash, mit synthetischer bach.db -> Daten landen in knowledge.db). Erledigt den offenen TODO-Punkt "Optionalen BACH-Import als Standalone-Funktion mit kleinem Fixture-Test absichern". Nicht uebernommen bzw. angepasst: - start.bat rief in der OneDrive-Fassung fest `python .db\launcher.py` auf -- im normal benannten Klon waere das tot. Jetzt eine Zeile, die beide Ablagen bedient, statt zweier divergenter Fassungen. - TODO.md existiert im Klon nicht mehr (upstream entfernt); der dortige Hunk drehte ausserdem nur einen Hostpfad von User auf lukas. - Die Loeschung von tests/__init__.py uebernommen wir nicht -- die Tests laufen mit der Datei, und ohne Not loeschen wir sie nicht. Tests: 140 passed (vorher 130). Ticket: T-20260913-506582780 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01RZVTLHtVLEhdALDmafKeJG
|
Welcome! 👋 Thanks for your first pull request in this repository. A maintainer will review it soon. Please make sure:
Thanks for contributing! |
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.
Kontext
Der Editable-Install von
knowledgedigestzeigte aufOneDrive/.WISSEN/.dbstatt auf den kanonischen Klon. Beim Umstellen (T-20260913-506582780) stellte sich heraus: die Arbeitskopie dort trug Arbeit, die in keinem Remote existierte — ein ungepushter Commit plus sieben uncommittete Dateien.Gerettet ohne Push aus OneDrive (cherry-pick + Patch in den Klon).
Inhalt
indexer.import_bach_skills()/wiki_indexer.import_bach_wikis()— Standalone-API, laeuft ohne echtes BACH graceful durch, gibt immer ein Dict zurueck statt zu werfen.launcher.pyregistriert das Verzeichnis unter dem AliasKnowledgeDigestinsys.modules— noetig, weil der Deployordner.dbheisst und das kein gueltiger Python-Name ist.tests/test_bach_import.py— Fixture-Tests fuer beide Pfade. Erledigt den offenen TODO-Punkt.SMOKE-CHECKS.md(cherry-pick des ungepushten Commits).Bewusst abweichend
start.batrief in der OneDrive-Fassung festpython .db\launcher.pyauf — im normal benannten Klon waere das tot. Jetzt eine Zeile fuer beide Ablagen statt zweier divergenter Fassungen.TODO.md-Hunk verworfen (Datei upstream entfernt, Hunk drehte nur einen Hostpfad).tests/__init__.pynicht uebernommen.Tests
python -m pytest -q→ 140 passed (vorher 130).Ticket: T-20260913-506582780
🤖 Generated with Claude Code
https://claude.ai/code/session_01RZVTLHtVLEhdALDmafKeJG