Skip to content

feat: add knowledge status#53

Merged
lucaseduoli merged 15 commits intomainfrom
knowledge-status
Sep 24, 2025
Merged

feat: add knowledge status#53
lucaseduoli merged 15 commits intomainfrom
knowledge-status

Conversation

@deon-sanchez
Copy link
Copy Markdown
Collaborator

@deon-sanchez deon-sanchez commented Sep 19, 2025

This pull request introduces support for displaying and managing document processing status in the knowledge management UI. It adds a new StatusBadge component, tracks task-related files and their statuses, and ensures that both backend and in-progress files are shown with accurate status indicators. The changes improve user feedback and make the UI more robust when handling asynchronous document processing and deletion.

Status tracking and UI enhancements:

  • Added a new StatusBadge component (frontend/src/components/ui/status-badge.tsx) and an animated processing icon (frontend/src/components/ui/animated-processing-icon.tsx) to visually indicate document status such as "processing", "active", "failed", etc. [1] [2]
  • Updated the File interface in useGetSearchQuery.ts to include an optional status field, and made several fields optional to better support files in different states.
  • Modified the knowledge page UI (frontend/src/app/knowledge/page.tsx) to merge backend results with in-progress task files, display their status, and use the new StatusBadge for each file. Also improved table rendering and formatting for chunk counts and scores. [1] [2] [3]

Task context and file state management:

  • Enhanced the TaskContext to track files associated with tasks, including their status and metadata, and to update or remove files based on task completion or failure. This ensures the UI reflects both backend and in-progress files accurately. [1] [2] [3] [4] [5]

API and mutation improvements:

  • Improved the document deletion mutation to use onSettled with a delay for query invalidation, ensuring UI updates after deletions are more reliable. Also made minor API parameter and error handling tweaks. [1] [2] [3] [4]

@deon-sanchez deon-sanchez self-assigned this Sep 19, 2025
@lucaseduoli lucaseduoli self-assigned this Sep 23, 2025
@lucaseduoli lucaseduoli changed the title Knowledge status feat: add knowledge status Sep 23, 2025
@lucaseduoli lucaseduoli merged commit be8e13a into main Sep 24, 2025
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.

3 participants