Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Chat Streaming on Obsidian, Docker Image Version and First-Run, Chat Error Messages in Clients #589

Merged
merged 9 commits into from
Dec 22, 2023

Conversation

debanjum
Copy link
Member

@debanjum debanjum commented Dec 20, 2023

  • 447c1b9 Fix streaming chat response in Obsidian client
  • e04fe92 Fix first-run, chat error message in Obsidian, Desktop and Web clients
  • d3d47dc, b5ae64c Set Khoj app version to latest version in Docker images
  • 8d1e988 Tag Khoj Docker image built on release with the latest tag
    This align docker image release cadence with client, server releases

- Disable chat input field if getChatHistory had error as Khoj may not
  be setup correctly to chat
- Convert renderIncrementalMessage to an async method as
  MarkdownRenderer is an async method

- Simplify code, remove unneeded JSON check
Copy link
Member

@sabaimran sabaimran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just have a couple of questions around UX and the necessity of rendering raw HTML, but looks good! Good catch on the chat streaming issue in Obsidian.

@debanjum debanjum force-pushed the fix-client-streaming-and-indexing branch from 8af1270 to 4bdcec8 Compare December 21, 2023 11:50
@debanjum debanjum force-pushed the fix-client-streaming-and-indexing branch from b67cb44 to a0997b2 Compare December 22, 2023 11:11
This will allow troubleshooting by getting the actual khoj version
being used. Previously it was always set to a static 0.0.0 version

Command to build Khoj docker image with dynamically set current app version:
`docker-compose build server --build-arg VERSION=$(pipx run hatch version)'
@debanjum debanjum force-pushed the fix-client-streaming-and-indexing branch from a0997b2 to 95880fe Compare December 22, 2023 11:18
- Tag docker image with `tag_name' on release (i.e tag push)
- Else tag with 'pre' on push to master
- Else tag with 'dev' on push to PR branch

- Only tag the latest release with release tag
  Previously the latest commit on master was being tagged with the
  latest tag. This doesn't sync with the release cadence of the rest
  of Khoj
@debanjum debanjum force-pushed the fix-client-streaming-and-indexing branch from 95880fe to 6ae6988 Compare December 22, 2023 11:33
- Delete unused config directory
@debanjum debanjum force-pushed the fix-client-streaming-and-indexing branch from 6ae6988 to 074123b Compare December 22, 2023 11:42
@debanjum debanjum added the fix Fix something that isn't working as expected label Dec 22, 2023
@debanjum debanjum changed the title Fix Chat Streaming on Obsidian and First-Run, Chat Error Messages in Clients Fix Chat Streaming on Obsidian, Docker Image Version and First-Run, Chat Error Messages in Clients Dec 22, 2023
@debanjum debanjum merged commit 6879dac into master Dec 22, 2023
11 checks passed
@debanjum debanjum deleted the fix-client-streaming-and-indexing branch December 22, 2023 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Fix something that isn't working as expected
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants