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

Hot reload - pearai #963

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

Conversation

Himanshu-Singh-Chauhan
Copy link

@Himanshu-Singh-Chauhan Himanshu-Singh-Chauhan commented Feb 12, 2025

Hey RooCode team,

I’m from the PearAI team, and I’m raising this draft PR just to share that we had implemented hot reload for the webview (React part) right before you guys did! Since you rolled out your own version shortly after, we didn’t end up using ours, but we still wanted to show you what we had worked on.
we wanted to share this with you as part of our commitment to contributing back.
Our approach was pretty similar to yours—the main difference being that you used Vite, while we implemented it on Webpack.

here is the pr raised on our fork - trypear#4
ignore the messy commit msgs.

@mrubens
PearAI boss 😎 - @nang-dev


Important

Implements hot reload for webview using Webpack, adding development scripts and updating CSP for localhost.

  • Behavior:
    • Implements hot reload for webview using Webpack in ClineProvider.ts.
    • Adds development mode check to load resources from localhost:3000.
  • Scripts:
    • Adds dev script in webview-ui/package.json to start Webpack dev server.
    • New dev-react.js script configures Webpack for hot reload.
  • Misc:
    • Updates Content-Security-Policy in ClineProvider.ts to allow localhost connections in development mode.

This description was created by Ellipsis for 3942dcf. It will automatically update as commits are pushed.

Copy link

changeset-bot bot commented Feb 12, 2025

⚠️ No Changeset found

Latest commit: 3942dcf

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@Himanshu-Singh-Chauhan Himanshu-Singh-Chauhan marked this pull request as draft February 12, 2025 15:15
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.

1 participant