[React]: Add initial usage of the new web worker-based cody web chat#62792
Conversation
5c3f7d9 to
31a7de1
Compare
| { | ||
| path: path.join(STATIC_ASSETS_PATH, 'chunks/chunk-*.js'), | ||
| maxSize: '600kb', // 2 monaco chunks are very big | ||
| maxSize: '6.45MB', // Cody web chunk is very big |
There was a problem hiding this comment.
Unfortunately, I couldn't find a way to name Cody Web chunk with a specific name.
We've already been violating this rule in different ways so I think this is tolerable
| @@ -0,0 +1,419 @@ | |||
| import React, { useEffect, useState } from 'react' | |||
There was a problem hiding this comment.
This file just a copy of the old page, no logic has been changed here
fkling
left a comment
There was a problem hiding this comment.
I didn't do a detail review but I looked through it. Since it's gated behind a flag it seems safe to merge. Just want to point out https://github.com/sourcegraph/sourcegraph/pull/63430 which also makes changes to CodyChatPage.tsx, so that needs to be coordinated somehow.
peterguy
left a comment
There was a problem hiding this comment.
Ran a functional test locally against S2; it behaved correctly. Didn't try to break it too much.
Part of https://github.com/sourcegraph/sourcegraph/issues/62448
Linear issue SRCH-573
This is highly experimental usage of the new (not currently merged but published in NPM
cody-web-experimental) packageHow to run it
rm -rf node_modulesin the root thenpnpm installsg start web-standalonenewCodyWeb: truein yourexperimentalFeaturesHow to run it locally with prototype PR in Cody repository
vk/integrate-cody-web-chat-2branchpnpm installto make sure you're up to date with all of the dependencies.cd web)pnpm buildpnpm link --global(Ignore the warning message about no binary)pnpm link --global cody-web-experimentalsg start web-standaloneto bundle the web app and launch an instance that uses S2 for the backend. You'll need to create a login on S2 that is not federated by GitHub.newCodyWeb: truein yourexperimentalFeaturesTest plan