θΏ™ζ˜―indexlocζδΎ›ηš„ζœεŠ‘οΌŒδΈθ¦θΎ“ε…₯任何密码
Skip to content

Conversation

@timothycarambat
Copy link
Member

move dev deps from prod build
remove unused deps

Pull Request Type

  • ✨ feat
  • πŸ› fix
  • ♻️ refactor
  • πŸ’„ style
  • πŸ”¨ chore
  • πŸ“ docs

What is in this change?

Reduces production build size of frontend for faster speed and chunking strategy

Additional Information

HighlightJS remains to be the largest chunk. We can do what we did with embed and only import a subset of languages we wish to support. Given this is a vite build for react we can include a larger subset then what is found in embed

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

move dev deps from prod build
remove unused deps
@timothycarambat timothycarambat merged commit 5e11173 into master Feb 7, 2024
@timothycarambat timothycarambat deleted the remove-phosphor-ssr-from-build branch February 7, 2024 21:36
@review-agent-prime
Copy link

frontend/vite.config.js

It's a good practice to add comments to regular expressions and complex functions to make the code more readable and maintainable. This will help other developers understand your code better.
Create Issue
See the diff
Checkout the fix

    find: /^~.+/,
    replacement: (val) => {
      // Remove the leading '~' from the string
      return val.replace(/^~/, "")
    }
git fetch origin && git checkout -b ReviewBot/Impro-fnagi97 origin/ReviewBot/Impro-fnagi97

cabwds pushed a commit to cabwds/anything-llm that referenced this pull request Jul 3, 2025
move dev deps from prod build
remove unused deps
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.

2 participants