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

✨ feat: WebNavigator, a browser for LibreChat #5902

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jmaddington
Copy link
Contributor

Docs: LibreChat-AI/librechat.ai#239

Summary

A structured tool that allows LLMs to simulate a browser/curl requests. Includes abilities to return only text, or specific tags, use custom headers, and more.

webnavigator

Change Type

Please delete any irrelevant options.

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

Testing

I've been using this in my own LC instance for about 3 months now. I have not written unit tests for this.

Important Package Problem

The package-lock file is NOT in sync in this PR, for which I apologize. cheerio is added as a dependency. When running npm install and npm ci I get a package lock that has far more changes than seems correct compared to the latest versions in e3b5c59. I need help here as I am still a newbie with node.

Checklist

Please delete any irrelevant options.

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • I have made pertinent documentation changes.
  • A pull request for updating the documentation has been submitted.

@rubentalstra
Copy link
Collaborator

@jmaddington
please don't install client specific package in the main package.json but in client/package.json

@jmaddington
Copy link
Contributor Author

@rubentalstra what is the proper way to go about that? Do I edit the package file directly, or is there an npm command to use?

Again, I apologize for not knowing how to handle this the right way.

@rubentalstra
Copy link
Collaborator

rubentalstra commented Feb 15, 2025

  1. npm r cheerio
  2. cd client
  3. npm i cheerio

then you should submit you package-lock.json as well. maybe first remove and download the one from the repo so you are up-to-date if needed.

  1. npm I
  2. submit all changes

@jmaddington
Copy link
Contributor Author

No need to npm ci again?

Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

ESLint found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

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