Skip to content

[Sister] Vector Store Refactor #416

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.

Sign up for GitHub

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

Jump to bottom
Draft
bhunt02 wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

[Sister] Vector Store Refactor #416

bhunt02 wants to merge 10 commits into from

Conversation

@bhunt02
Copy link
Collaborator

@bhunt02 bhunt02 commented Oct 14, 2025

Description

better description later

Closes #412

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • This requires a run of yarn install
  • This change requires an addition/change to the production .env variables. These changes are below:
  • This change requires developers to add new .env variables. The file and variables needed are below:
  • This change requires a database query to update old data on production. This query is below:

How Has This Been Tested?

Please describe how you tested this PR (both manually and with tests)
Provide instructions so we can reproduce.

  • Test A
  • Test B

Checklist:

  • I have performed a code review of my own code (under the "Files Changed" tab on github) to ensure nothing is committed that shouldn't be (e.g. leftover console.logs, leftover unused logic, or anything else that was accidentally committed)
  • I have commented my code where needed
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that new and existing tests pass locally with my changes
  • Any work that this PR is dependent on has been merged into the main branch
  • Any UI changes have been checked to work on desktop, tablet, and mobile

@bhunt02 bhunt02 self-assigned this Oct 17, 2025
@bhunt02
migration file that's been hanging for a bit
e9af904
@bhunt02
Merge remote-tracking branch 'origin/main' into bridgette/chatbot-vec…
3428c71 
…tor-store-refactor-sister

# Conflicts:
#	packages/frontend/app/(dashboard)/course/[cid]/(settings)/settings/chatbot_knowledge_base/page.tsx
#	packages/frontend/app/(dashboard)/course/[cid]/components/chatbot/Chatbot.tsx
#	packages/frontend/app/(dashboard)/organization/lms_integrations/page.tsx
#	packages/frontend/app/api/index.ts
#	packages/server/src/lmsIntegration/lmsIntegration.service.ts
@bhunt02
this is going to break every test but i'm implementing websockets now…
aebad3a 
… for chatbot ops because there's way more routes that are liable to timeout due to external API calls
@bhunt02
beefy commit; bulk of work done for subscribers, verification of thei…
90aff5d 
…r functionality (and verifying the frontend works with the new impl)

added 'delete all queries' and further worked on new doc chunk list interface
next: fix aggregate uploads & test them
@bhunt02
why did these guys not go?
96709ce
@bhunt02
switching to other dev device
49d6252
@bhunt02
Merge remote-tracking branch 'origin/main' into bridgette/chatbot-vec…
d4fe581 
…tor-store-refactor-sister

# Conflicts:
#	packages/frontend/app/api/index.ts
#	yarn.lock
@bhunt02
very minor fixes
6318f51
@bhunt02
progress
f75c9b3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@bhunt02 bhunt02

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Meta-Issue] Miscellaneous Chatbot Improvements

2 participants

@bhunt02