Skip to content

feat: SQLite chat-history met Chainlit sidebar#1

Closed
Tomeriko96 wants to merge 1 commit into
mainfrom
feat/chat-history
Closed

feat: SQLite chat-history met Chainlit sidebar#1
Tomeriko96 wants to merge 1 commit into
mainfrom
feat/chat-history

Conversation

@Tomeriko96
Copy link
Copy Markdown
Contributor

Wat doet dit?

Voegt persistente gespreksgeschiedenis toe via een SQLite-backed Chainlit data layer, inclusief sidebar-navigatie.

Wijzigingen

  • data_layer.pySQLiteDataLayer (BaseDataLayer): slaat threads, steps en users op in data/sessions.db
  • persistence.py — lichtgewicht sessions-tabel als fallback voor de /history knop
  • app.py@cl.data_layer, @cl.header_auth_callback, @cl.on_chat_resume, /history command, resume_session action, thread-metadata sync na elke beurt

Vereisten

CHAINLIT_AUTH_SECRET in .env (zie .env.example). Zonder dit valt Chainlit terug op sessie-authenticatie zonder sidebar.

Status / openstaande punten

  • Testen: sidebar-navigatie werkt na herstart
  • Testen: on_chat_resume herstelt berichten correct
  • Overwegen: persistence.py samenvoegen met data_layer.py (twee SQLite DB's nu)
  • Overwegen: user-ID koppelen aan threads (nu alles onder "local")

🤖 Generated with Claude Code

- data_layer.py: SQLite BaseDataLayer (threads, steps, users tabellen)
- persistence.py: lichtgewicht sessions-tabel voor /history fallback
- app.py: @cl.data_layer, @cl.header_auth_callback, @cl.on_chat_resume,
  /history command, resume_session action, thread-metadata persistentie

Vereist CHAINLIT_AUTH_SECRET in .env (zie .env.example).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@Tomeriko96
Copy link
Copy Markdown
Contributor Author

Discarded — wordt opnieuw geïmplementeerd in een nieuwe branch met volledige TDD-aanpak en correcte Chainlit 2.x data layer integratie.

@Tomeriko96 Tomeriko96 closed this May 18, 2026
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