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

refactor: rewrite and cleanups #9

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

refactor: rewrite and cleanups #9

wants to merge 4 commits into from

Conversation

pi0
Copy link
Collaborator

@pi0 pi0 commented Sep 21, 2023

Remade from the older private repo.

Branch: https://github.com/raidendotai/openv0/tree/refactor/cleanup

This PR rewrites repository codebase with couple of improvements:

  • Server migrated to ESM syntax
  • Removed intermediate training files and compressed on library/*.tgz
  • [breaking] Example training are removed - I would suggest to add them back also as zip files and auto extract when running server. This is best to reduce git size
  • Using prettier formatter
  • Fixed vite issues
  • Updated license (can remove my name haha)
  • Migrated to h3 for server
  • Added .env init flow

@raidendotai There are too many changes in this and naturally some are opinionated and you might not liked or preferred differently in which case, feel free to ask me to change, change yourself or even close this PR entirely we can rework same again!

@raidendotai
Copy link
Owner

looks super neat @pi0

on my side :

  • finishing few updates to modularize how component libraries are built (ie. shadcn) and different export modules (ie. react/svelte/vanilla)
  • a couple more features + prompts

then we push a major update on main, structured in your way

@pi0
Copy link
Collaborator Author

pi0 commented Sep 21, 2023

Sure! We can quickly rebase if not too much changes to this branch i guess.

@EricSimons
Copy link
Collaborator

One request: We were going to list this repo as a starter on stackblitz.com, so would love to have this continue to use node versus bun if possible (so that it can open in StackBlitz)!

library/verctor_db.tgz Outdated Show resolved Hide resolved
@pi0
Copy link
Collaborator Author

pi0 commented Sep 21, 2023

@EricSimons Here you go: https://stackblitz.com/~/github.com/raidendotai/openv0/pull/9 🚀

(Love stackblitz! It helped to quickly create fresh setup multiple times in order to fix all onboarding issues. Now usage is as easy as a pnpm run start command env file is also auto generated AND synced!!)

@EricSimons
Copy link
Collaborator

EricSimons commented Sep 21, 2023

Oh WOW, that CLI prompt for obtaining the API key is 🔥! Such an amazing UX addition for this. Thanks so much for taking the time to make sure this all worked well on StackBlitz btw!!

@pi0 pi0 mentioned this pull request Sep 27, 2023
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.

4 participants